Repository navigation
Make TLV compatible with master-branch => composer.json #71
Description
The current master will use a composer.json for the language files: https://github.com/phpbb/phpbb/blob/master/phpBB/language/en/composer.json
We should add some functions to check this file:
- validate the composer json
- Check for the right version in
"version": "4.0.0-a1-dev", - Need to check for the information from former iso.txt https://github.com/phpbb/phpbb/blob/3.3.x/phpBB/language/en/iso.txt
"extra": {
"language-iso": "en",
"english-name": "British English",
"local-name": "British English",
"phpbb-version": "4.0.0-a1-dev"
}
- Maybe improve core code, to add reCAPTCHA and plural rules to the composer.json, too.
Activity
battye commentedon Feb 10, 2022
Hi @Crizz0 - I'm taking a look at this now. Should this just be the lowest level of error message (notice) at this stage because not many language packs have composer.json files yet? Or do you want to change the rules so it becomes mandatory now and translators become familiar with adding the json file in there?
Crizz0 commentedon Feb 10, 2022
This should introduce a new optional version 4.0 to validate against.
With default 3.3 composer.json should be not asked for.
But for 4.0 this should be mandatory then, yes. But the iso.txt and what else is replaced by the composer.json should not be there, and flagged as additional file.
Validate 4.0 composer.json, check for the right version. phpbb#71
battye commentedon Oct 20, 2022
@Crizz0 Could you please explain this line a bit more?
Is that something different to what was merged in #59?
Crizz0 commentedon Oct 20, 2022
Hi,
phpBB-master removed the current config-file for phpBB 3.3.x and lower: the phpBB/language/en/iso.txt is gone.
Instead there is a composer.json used now: https://github.com/phpbb/phpbb/blob/master/phpBB/language/en/composer.json
The validator needs to be updated for this composer.json.
Although the composer.json is the new config-file for translations, there are other files which contain "setup information" like: https://github.com/phpbb/phpbb/blob/master/phpBB/language/en/common.php#L45-L53 or https://github.com/phpbb/phpbb/blob/master/phpBB/language/en/captcha_recaptcha.php#L42
So maybe it is worth to update the phpBB-core in that relation, that those information are all within the new composer.json. Which will lead to a further update to the TVL.
Maybe @marc1706 has some thoughts on this, too. :)
battye commentedon Oct 20, 2022
Is the change in battye@fbd2740 what you had in mind for the composer.json? (e.g. check that the 'extra' information is present, that the four attributes are there, that the version number matches)
I'll test it again, but I think if the iso.txt file is removed from the master package then it would automatically not check for it in the translation package (because it checks that the file structure is the same, so if it's not in the source then it will pretend it doesn't exist). There are a few mentions of
sourceIsoin the code but I'm pretty sure that's just shorthand for the language name likedeorfroren.Crizz0 commentedon Sep 27, 2025
See #84