Real message translation fails with ICU MessageFormat

[mx03]

Symfony version(s) affected

5.4.9

Description

When using real messages the message dont get parsed with ICU MessageFormat.

How to reproduce

Example from https://symfony.com/doc/current/translation/message_format.html is used:

<?php
require_once __DIR__ . '/vendor/autoload.php';

$translator = new Symfony\Component\Translation\Translator("de");

// prints "Ryan has invited you for his party!"
echo $translator->trans('{organizer_gender, select,
    female {{organizer_name} has invited you for her party!}
    male   {{organizer_name} has invited you for his party!}
    other  {{organizer_name} have invited you for their party!}
}', [
    'organizer_name' => 'Ryan',
    'organizer_gender' => 'male',
]);

Expected:

Ryan has invited you for his party!

Result:

{male, select,
    female {{Ryan} has invited you for her party!}
    male   {{Ryan} has invited you for his party!}
    other  {{Ryan} have invited you for their party!}
}

With a resource:

require_once __DIR__ . '/vendor/autoload.php';

$translator = new Symfony\Component\Translation\Translator("de");

$translator->addLoader('mo', new Symfony\Component\Translation\Loader\MoFileLoader());
$translator->addResource('mo', __DIR__."/messages+intl-icu.de.mo", "de", "messages+intl-icu");

// prints "Ryan has invited you for his party!"
echo $translator->trans('{organizer_gender, select,
    female {{organizer_name} has invited you for her party!}
    male   {{organizer_name} has invited you for his party!}
    other  {{organizer_name} have invited you for their party!}
}', [
    'organizer_name' => 'Ryan',
    'organizer_gender' => 'male',
]);

Source(mo file):

{organizer_gender, select,
    female {{organizer_name} has invited you for her party!}
    male   {{organizer_name} has invited you for his party!}
    other  {{organizer_name} have invited you for their party!}
}

Translation(mo file):

{organizer_gender, select,
    female {{organizer_name} a ..!}
    male   {{organizer_name} b ..!}
    other  {{organizer_name} c ..!}
}

Result:

Ryan b ..!

Possible Solution

No response

Additional Context

No response

[HypeMC]

Looks like a bug in the docs symfony/symfony-docs#15402, this works just fine:

<?php

require_once __DIR__ . '/vendor/autoload.php';

use Symfony\Component\Translation\MessageCatalogueInterface;

$translator = new Symfony\Component\Translation\Translator("de");

// prints "Ryan has invited you for his party!"
echo $translator->trans('{organizer_gender, select,
    female {{organizer_name} has invited you for her party!}
    male   {{organizer_name} has invited you for his party!}
    other  {{organizer_name} have invited you for their party!}
}', [
    'organizer_name' => 'Ryan',
    'organizer_gender' => 'male',
], 'messages'.MessageCatalogueInterface::INTL_DOMAIN_SUFFIX);

[mx03]

Yes thats working.
A default setting would be great.

current Translator::trans:

//...
if (null === $domain) {
    $domain = 'messages';
}
//...

with default setting Translator::trans:

//...
if (null === $domain) {
    $domain = $this->defaultDomain;
}
//...

And/Or if the loaded resources are use the ICU MessageFormat with the domain suffix, the fallback is automatically set from messages to messages.MessageCatalogueInterface::INTL_DOMAIN_SUFFIX.

[carsonbot]

Hey, thanks for your report!
There has not been a lot of activity here for a while. Is this bug still relevant? Have you managed to find a workaround?

[carsonbot]

Friendly ping? Should this still be open? I will close if I don't hear anything.

[carsonbot]

Hey,

I didn't hear anything so I'm going to close it. Feel free to comment if this is still relevant, I can always reopen!

[PhoneixS]

Yes, it's still relevant.

Currently, if you have the real string as id, you can't use ICU correctly. You need to do one of:

• Add "+intl-icu" to the domain. Then when you extract the strings with bin/console translation:extract pt --force it generates files as "domain+intl-icu+intl-icu.pt" that you need to rename manually to "domain+intl-icu.pt".
• Or make translations for the original language so it generates the "+intl-icu" files, but having a translation file for the original language is what you want to avoid if you use the real string as an id.

[PhoneixS]

@mx03 have you managed to solve the problems I have posted? Or at least circumvent them?