Uncaught exception 'Symfony\Component\Intl\Exception\ResourceBundleNotFoundException' with message

[in-session]

Preconditions and environment

• 2.4.6-p6
• New Relic ESI block Tracelog

Steps to reproduce

Expected result

Actual result

Uncaught exception 'Symfony\Component\Intl\Exception\ResourceBundleNotFoundException' with message 'The resource bundle "/htdocs/vendor/symfony/intl/Resources/data/currencies/de_DE.php" does not exist.' in htdocs/vendor/symfony/intl/Data/Bundle/Reader/PhpBundleReader.php:38

Additional information

If I look at the path, it should actually be htdocs/vendor/symfony/intl/Resources/data/currencies/de.php default language "de_DE" is used but symfony/intl expects "de".

magento2/lib/internal/Magento/Framework/Currency/Data/Currency.php

Lines 270 to 285 in 71370a9

	public function getSymbol($currency = null, $locale = null): ?string
	{
	if ($currency === null && $locale === null) {
	return $this->options['symbol'];
	}
	$params = $this->checkParams($currency, $locale);
	if (!empty($params['currency'])) {
	$locale = $this->displayLocale[$params['currency']] ?? $params['locale'];
	$symbol = Currencies::getSymbol($params['currency'], $locale);
	} else {
	$symbol = Currencies::getSymbol($params['name'], $params['locale']);
	}
	return !empty($symbol) ? $symbol : null;
	}

Release note

No response

Triage and priority

• Severity: S0 - Affects critical data or functionality and leaves users without workaround.
• Severity: S1 - Affects critical data or functionality and forces users to employ a workaround.
• Severity: S2 - Affects non-critical data or functionality and forces users to employ a workaround.
• Severity: S3 - Affects non-critical data or functionality and does not force users to employ a workaround.
• Severity: S4 - Affects aesthetics, professional look and feel, “quality” or “usability”.

[m2-assistant]

Hi @in-session. Thank you for your report.
To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:

• @magento give me 2.4-develop instance - upcoming 2.4.x release
• For more details, review the Magento Contributor Assistant documentation.
• Add a comment to assign the issue: @magento I am working on this
• To learn more about issue processing workflow, refer to the Code Contributions.

---

Join Magento Community Engineering Slack and ask your questions in #github channel.
⚠️ According to the Magento Contribution requirements, all issues must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.
🕙 You can find the schedule on the Magento Community Calendar page.
📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket.

[in-session]

The problem relates to all Symfony\Component\Intl components. if i see it correctly magento tries to use the language iso codes ISO-639-1 with ISO-3166-1. There should be a check which then falls back $isValidLanguage = Languages::exists($languageCode); to ISO-639-1 if not available.

[in-session]

I think that this issue is due to the problem: #37780

[m2-assistant]

Hi @engcom-Hotel. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: 👇

• 1. Verify that issue has all the required information. (Preconditions, Steps to reproduce, Expected result, Actual result).
• 2. Verify that issue has a meaningful description and provides enough information to reproduce the issue.
• 3. Add Area: XXXXX label to the ticket, indicating the functional areas it may be related to.
• 4. Verify that the issue is reproducible on 2.4-develop branch
  Details

  - Add the comment @magento give me 2.4-develop instance to deploy test instance on Magento infrastructure.
  - If the issue is reproducible on 2.4-develop branch, please, add the label Reproduced on 2.4.x.
  - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here!
• 5. Add label Issue: Confirmed once verification is complete.
• 6. Make sure that automatic system confirms that report has been added to the backlog.

[engcom-Hotel]

Hello @in-session,

Thanks for the report and collaboration!

We have tried to reproduce the issue in the latest development branch of Magento i.e. 2.4-develop, but it seems the issue is not reproducible for us. We have followed the below steps in order to reproduce the issue:

Scenario 1

1. Setup fresh Magento instance
2. Goto Stores -> Settings -> Configuration -> General -> General
3. Change Locale to German(Germany)

We are not getting the error mentioned in the main description.

Scenario 2

1. Setup fresh Magento instance
2. Goto Stores -> Settings -> Configuration -> General -> General
3. Change Locale to German(Germany)
4. Create products and try to debug the files vendor/symfony/intl/Data/Bundle/Reader/PhpBundleReader.php and lib/internal/Magento/Framework/Currency/Data/Currency.php while getting the product from the frontend

We are not getting the error mentioned in the main description.

Please let us know if we missed anything for this issue.

Thanks

[in-session]

@engcom-Hotel When I make an error_log here, I always get the value de_DE, but the Currencies::getName and Currencies::getSymbol are 'de' expected. The problem has only arisen through the integration with NewRelic and when, for example, product sliders are stored in their own ESI block. We received 50 GB of Tracelogs within 7 days. You won't be able to replicate this so easily with a dev instance. Can you please categorise this as S1 and open Jira Issus. To reproduce you should set the locale from de_DE and the currencies to EUR.

Perhaps @ihor-sviziev @mrtuvn has an idea for this issus?

Here, however, Symfony\Component\Intl expects 'DE' so that it can load the data from htdocs/vendor/symfony/intl/Resources/data/currencies

https://symfony.com/doc/5.x/components/intl.html#currencies

$currencies = Currencies::getNames('de');
// => ['AFN' => 'Afghanischer Afghani', 'EGP' => 'Ägyptisches Pfund', ...]

$currency = Currencies::getName('EUR', 'de');
// => 'Indische Rupie'


$symbol = Currencies::getSymbol('EUR', 'de');
// => '€'

/**
     * @param array|string|null $options
     * @param string|null $locale
     * @throws CurrencyException
     * @SuppressWarnings(PHPMD.NPathComplexity)
     * @SuppressWarnings(PHPMD.CyclomaticComplexity)
     */
    public function __construct($options = null, $locale = null)
    {
        $callOptions = $options;

        if (is_array($options) && isset($options['display'])) {
            $this->options['display'] = $options['display'];
        }
        $this->setLocale($locale);
        
        //error_log("setLocale: " . print_r($locale, true));
        
        if (!isset($this->options['currency']) || !is_array($options)) {
            $this->options['currency'] = $this->getShortName($options, $this->options['locale']);
        }
        if (!isset($this->options['name']) || !is_array($options)) {
            $this->options['name'] = $this->getName($options, $this->options['locale']);
        }
        if (!isset($this->options['symbol']) || !is_array($options)) {
            $this->options['symbol'] = $this->getSymbol($options, $this->options['locale']);
        }
        if ($this->options['currency'] === null && $this->options['name'] === null) {
            throw new CurrencyException(__(
                'Currency "%1" not found',
                $options
            ));
        }
        if ((is_array($callOptions) && !isset($callOptions['display']))
            || (!is_array($callOptions) && $this->options['display'] == self::NO_SYMBOL)) {
            if (!empty($this->options['symbol'])) {
                $this->options['display'] = self::USE_SYMBOL;
            } elseif (!empty($this->options['currency'])) {
                $this->options['display'] = self::USE_SHORTNAME;
            }
        }
    }

This returns de_DE which is incorrect.

    /**
     * Returns the actual or details of other currency symbols.
     *
     * @param string|null $currency
     * @param string|null $locale
     * @return string|null
     * @throws CurrencyException
     */
    public function getSymbol($currency = null, $locale = null): ?string
    {
        if ($currency === null && $locale === null) {
            return $this->options['symbol'];
        }
        $params = $this->checkParams($currency, $locale);

        if (!empty($params['currency'])) {
            $locale = $this->displayLocale[$params['currency']] ?? $params['locale'];
            $symbol = Currencies::getSymbol($params['currency'], $locale);
            error_log("setLocale: " . print_r($locale, true));            
        } else {
            $symbol = Currencies::getSymbol($params['name'], $params['locale']);
        }

        return !empty($symbol) ? $symbol : null;
    }

As soon as we insert it hard-coded, the problem no longer exists:

    /**
     * Returns the actual or details of other currency symbols.
     *
     * @param string|null $currency
     * @param string|null $locale
     * @return string|null
     * @throws CurrencyException
     */
    public function getSymbol($currency = null, $locale = null): ?string
    {
        if ($currency === null && $locale === null) {
            return $this->options['symbol'];
        }
        $params = $this->checkParams($currency, $locale);

        if (!empty($params['currency'])) {
            $locale = $this->displayLocale[$params['currency']] ?? $params['locale'];
            $symbol = Currencies::getSymbol($params['currency'], 'de');
        } else {
            $symbol = Currencies::getSymbol($params['name'], $params['locale']);
        }

        return !empty($symbol) ? $symbol : null;
    }

[engcom-Hotel]

Hello @in-session,

Thanks for the reply!

We have more dig into this issue and tried to debug the following method call:

magento2/lib/internal/Magento/Framework/Currency/Data/Currency.php

Lines 313 to 331 in 6a18520

	public function getName($currency = null, $locale = null): ?string
	{
	if ($currency === null && $locale === null) {
	return $this->options['name'];
	}
	$params = $this->checkParams($currency, $locale);
	if (!empty($params['currency'])) {
	$name = Currencies::getName($params['currency'], $params['locale']);
	} else {
	$name = Currencies::getName($params['name'], $params['locale']);
	}
	if (empty($name)) {
	return null;
	}
	return $name;
	}

Actually here we are trying to get the Name from the method call Currencies::getName with param de and de_DE. But what we have observed here is, we are getting the same and correct Euro name in return. Please refer to the below screenshots for reference:

If we pass de

If we pass de_DE

But we have also gone through the Symfony documentation here: https://symfony.com/doc/6.4/components/intl.html#currencies and it requires de as a second parameter.

But still we are unable to reproduce the issue as mentioned in the main description.

Thanks

[in-session]

@engcom-Hotel

thanks for your check i have looked at everything again and i think this only affects the getSymbol.

public function getSymbol($currency = null, $locale = null): ?string
    {
        if ($currency === null && $locale === null) {
            return $this->options['symbol'];
        }
        $params = $this->checkParams($currency, $locale);

        if (!empty($params['currency'])) {
            $locale = $this->displayLocale[$params['currency']] ?? $params['locale'];
            $symbol = Currencies::getSymbol($params['currency'], $locale);
            // $symbol = Currencies::getSymbol($params['currency'], 'de');
        } else {
            $symbol = Currencies::getSymbol($params['name'], $params['locale']);
        }

        return !empty($symbol) ? $symbol : null;
    }

But I have found a way to reproduce this:

Firstly, you should change the following value for EUR in Symfony\Component\Intl\Resources\data\currencies\de.php::377

        'EUR' => [
            '€ New',
            'Euro',
        ],

As a result, you will not yet see any changes in the frontend. If you change the value $symbol = Currencies::getSymbol($params['currency'], $locale); to $symbol = Currencies::getSymbol($params['currency'], 'de');
Only then will the prices be displayed with '€ New'

This is because there is no de_DE.php in the intl and it uses en.php as a fallback.

This means that all locations always use the default en.php, like:

• de_DE
• it_IT
• it_CH
• eu_ES
• and many more...

Therefore it should be checked here whether the locale exists, if not the language code should be manipulated from de_DE to de and als fallback en, as a rough example

public function getSymbol($currency = null, $locale = null): ?string
  {
      if ($currency === null && $locale === null) {
          return $this->options['symbol'];
      }
      $params = $this->checkParams($currency, $locale);

      if (!empty($params['currency'])) {
          $locale = $this->displayLocale[$params['currency']] ?? $params['locale'];
          
          // Check whether the locale code is valid
          $isValidLocale = Locales::exists($locale);
          
          if (!$isValidLocale) {                
              $croppedLocale = substr($params['locale'], (strpos($params['locale'], '_') + 1));
              $symbol = Currencies::getSymbol($params['currency'], $croppedLocale);
          } else {
              $symbol = Currencies::getSymbol($params['currency'], $locale);                
          }           
          
      } else {
          $symbol = Currencies::getSymbol($params['name'], $params['locale']);
      }

      return !empty($symbol) ? $symbol : null;
  }

This is also the reason why the value in the Issus is not taken here, the MXN cannot be displayed correctly because de.php is not loaded #37780

[engcom-Hotel]

Hello @in-session,

Thanks for the reply!

We have debugged the issue and are able to reproduce it, but there is a catch, let me explain here.

Please refer to the below screenshot from method vendor/symfony/intl/Data/Bundle/Reader/BundleEntryReader:readEntry, when the $locale is de_DE:

But after that on line no. 136, the below method has been called:

$currentLocale = Locale::getFallback($currentLocale);

Which then called getFallback method, the documentation of the method is as follows:

Returns the fallback locale for a given locale.
For example, the fallback of "fr_FR" is "fr". The fallback of "fr" is the default fallback locale configured with setDefaultFallback(). The default fallback locale has no fallback.
Returns:
null|string The ICU locale code of the fallback locale, or null if no fallback exists

Now this method will return de, which is available on the vendor/symfony/intl/Resources/data/currencies/ path, hence it is returning the correct value for us:

It seems to us an expected behaviour, let me know in case we have missed anything.

Thanks