User tests: Successful: 0 Unsuccessful: 1 alikon
Automatically created pull request based on core-translation repository changes
Status | New | ⇒ | Pending |
Category | ⇒ | Installation Language & Strings |
i dunno why only it-IT want to use %d
instead of the %s
in INSTL_DATABASE_ENCRYPTION_MSG_FILE_FIELD_BAD
for me this change it is wrong
The text "INSTL_DATABASE_ENCRYPTION_MSG_FILE_FIELD_BAD" is used here:
https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/src/Helper/DatabaseHelper.php#L309
https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/src/Helper/DatabaseHelper.php#L332
https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/src/Helper/DatabaseHelper.php#L342
In all cases the sprintf parameter is a translated language string for a field label and so a string, so the %d
is an error for sure.
I've upvoted the already proposed correct Italian translation with %s
on Crowdin .. let's hope it helps.
so we can close this pr ?
Isn’t it updated automatically when the changed translation has been made on Crowdin?
dunno but this pr like it is , it is wrong
Isn’t it updated automatically when the changed translation has been made on Crowdin?
Yes once its approved & synced back into the repo. This PR should be kept open until the next stable release is in preperation.
ok so let me mark it as unsuccessful in the hope that someone "cum grano salis"
will understand
ps
my last experience in crowdin was not useful at all
I have tested this item
There is no point in testing this unsuccessful but better contact the TT in question so this can be fixed in crowdin. This PR will be open for any other installation language change which will come as we decided that we always have to update the version and the dates in the xml.
i'm out
i've already tryied to contact that "ghost team" without any feedback
now i'm tired
hope yuo'll have more success
than me
sorry to stick with rules
whatever this is simply wrong so i'll close it
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-01-20 20:26:51 |
Closed_By | ⇒ | alikon | |
Labels |
Added:
Language Change
?
|
As long as that issue is not fixed in crowdin this PR will come up again and again.
waiting for a good pr
sure not this 1
anyway feel free to commit this "shit" .... who cares ?
I dont intent to merge this change for obvious reasons ;)
I see %s and %d ‘a changing in the Italian pack again. Can you validate please @alikon ?