From 3a28a02a7f7ae8b6e5b11569601724c65519c306 Mon Sep 17 00:00:00 2001
From: Jonathan LELIEVRE <jonathan.lelievre@prestashop.com>
Date: Thu, 14 Nov 2024 17:06:15 +0100
Subject: [PATCH] Simply locale creation, that is not used anyway, to fix the
 performance drop from the initial refacto

---
 .../Addon/Module/ModuleManagerBuilder.php     | 79 +++++++++++--------
 1 file changed, 48 insertions(+), 31 deletions(-)

diff --git a/src/Core/Addon/Module/ModuleManagerBuilder.php b/src/Core/Addon/Module/ModuleManagerBuilder.php
index aff779aeb86d9..8c248c9d9e874 100644
--- a/src/Core/Addon/Module/ModuleManagerBuilder.php
+++ b/src/Core/Addon/Module/ModuleManagerBuilder.php
@@ -29,8 +29,7 @@
 use Context;
 use Doctrine\Common\Cache\Psr6\DoctrineProvider;
 use Language;
-use PrestaShop\PrestaShop\Adapter\Configuration;
-use PrestaShop\PrestaShop\Adapter\Currency\CurrencyDataProvider;
+use PrestaShop\Decimal\Operation\Rounding;
 use PrestaShop\PrestaShop\Adapter\HookManager;
 use PrestaShop\PrestaShop\Adapter\LegacyLogger;
 use PrestaShop\PrestaShop\Adapter\Module\AdminModuleDataProvider;
@@ -39,15 +38,12 @@
 use PrestaShop\PrestaShop\Adapter\Tools;
 use PrestaShop\PrestaShop\Core\Context\ApiClientContext;
 use PrestaShop\PrestaShop\Core\Context\LanguageContext;
-use PrestaShop\PrestaShop\Core\Localization\CLDR\DataLayer\LocaleReference;
-use PrestaShop\PrestaShop\Core\Localization\CLDR\LocaleDataSource;
-use PrestaShop\PrestaShop\Core\Localization\CLDR\LocaleRepository;
-use PrestaShop\PrestaShop\Core\Localization\CLDR\Reader;
-use PrestaShop\PrestaShop\Core\Localization\Currency\CurrencyDataSource;
-use PrestaShop\PrestaShop\Core\Localization\Currency\DataLayer\CurrencyInstalled;
-use PrestaShop\PrestaShop\Core\Localization\Currency\DataLayer\CurrencyReference;
-use PrestaShop\PrestaShop\Core\Localization\Currency\Repository;
 use PrestaShop\PrestaShop\Core\Localization\Locale;
+use PrestaShop\PrestaShop\Core\Localization\Number\Formatter as NumberFormatter;
+use PrestaShop\PrestaShop\Core\Localization\Specification\Number as NumberSpecification;
+use PrestaShop\PrestaShop\Core\Localization\Specification\NumberCollection;
+use PrestaShop\PrestaShop\Core\Localization\Specification\NumberSymbolList;
+use PrestaShop\PrestaShop\Core\Localization\Specification\Price as PriceSpecification;
 use PrestaShop\PrestaShop\Core\Module\ModuleManager;
 use PrestaShop\PrestaShop\Core\Module\ModuleRepository;
 use PrestaShop\PrestaShop\Core\Module\SourceHandler\SourceHandlerFactory;
@@ -226,7 +222,47 @@ private function getLanguageContext(): LanguageContext
         /** @var Language $language */
         $language = Context::getContext()->language;
 
-        $localeRepository = $this->getLocaleRepository();
+        // If locale is present in context we can use, if not we create a mock one
+        // the locale is not used by the ModuleRepository anyway only the language ID is relevant for its internal cache key generation
+        if (Context::getContext()->currentLocale) {
+            $locale = Context::getContext()->currentLocale;
+        } else {
+            $numberSymbolList = new NumberSymbolList(',', ' ', ';', '%', '-', '+', 'E', '^', '‰', '∞', 'NaN');
+            $priceSpecsCollection = new NumberCollection();
+            $priceSpecsCollection->add(
+                'EUR',
+                new PriceSpecification(
+                    '#,##0.## ¤',
+                    '-#,##0.## ¤',
+                    ['latn' => $numberSymbolList],
+                    2,
+                    2,
+                    true,
+                    3,
+                    3,
+                    'symbol',
+                    '€',
+                    'EUR'
+                )
+            );
+            $numberSpecification = new NumberSpecification(
+                '#,##0.###',
+                '-#,##0.###',
+                [$numberSymbolList],
+                3,
+                2,
+                true,
+                2,
+                3
+            );
+            $locale = new Locale(
+                $language->locale,
+                $numberSpecification,
+                $priceSpecsCollection,
+                new NumberFormatter(Rounding::ROUND_HALF_UP, 'latn')
+            );
+        }
+
         self::$languageContext = new LanguageContext(
             $language->id,
             $language->name,
@@ -236,31 +272,12 @@ private function getLanguageContext(): LanguageContext
             $language->is_rtl,
             $language->date_format_lite,
             $language->date_format_full,
-            $localeRepository->getLocale($language->locale)
+            $locale
         );
 
         return self::$languageContext;
     }
 
-    private function getLocaleRepository(): Locale\Repository
-    {
-        $localeDataReference = new LocaleReference(new Reader());
-        $localeDataSource = new LocaleDataSource($localeDataReference);
-        $cldrLocaleRepository = new LocaleRepository($localeDataSource);
-
-        $configuration = new Configuration();
-        $currencyReference = new CurrencyReference($cldrLocaleRepository);
-        $currencyDataProvider = new CurrencyDataProvider($configuration, (int) $configuration->get('PS_SHOP_DEFAULT'));
-        $currencyInstalled = new CurrencyInstalled($currencyDataProvider);
-        $currencyDataSource = new CurrencyDataSource($currencyReference, $currencyInstalled);
-        $currencyRepository = new Repository($currencyDataSource);
-
-        return new Locale\Repository(
-            $cldrLocaleRepository,
-            $currencyRepository,
-        );
-    }
-
     protected function getConfigDir()
     {
         return _PS_ROOT_DIR_ . '/app/config';