VoxeLibre/VoxeLibre
VoxeLibre
/
VoxeLibre
56
133
Fork 173
Code Issues 935 Pull Requests 78 Projects 4 Releases Wiki Activity

22 translation files have syntax errors #3540

New Issue
Open
opened 2023-03-14 01:34:15 +01:00 by Wuzzy · 6 comments
Wuzzy commented 2023-03-14 01:34:15 +01:00
Member
Copy Link

MineClone2 version: 7b53b6e45f

What happened?

I've written a quick and dirty Python script to automatically test locale files (*.tr and template.txt) for syntax errors such as missing placeholders or bad escaping.

I'm running this script on various games now to see what happens and in MCL2 the script has found a lot of errors.

This is the test result:

Error (./HELP/mcl_tt/locale/mcl_tt.fr.tr:48): Missing placeholder(s) in translation
Error (./HELP/mcl_tt/locale/mcl_tt.pl.tr:6): Missing placeholder(s) in translation
Error (./HELP/doc/doc/locale/doc.fr.tr:27): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.fr.tr:6): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.fr.tr:99): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:56): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:57): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:81): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:126): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:56): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:57): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:81): Missing placeholder(s) in translation
Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:126): Missing placeholder(s) in translation
Error (./HELP/mcl_doc/locale/mcl_doc.fr.tr:53): Missing placeholder(s) in translation
Error (./HELP/mcl_doc/locale/mcl_doc.de.tr:54): Missing placeholder(s) in translation
Error (./HELP/mcl_doc/locale/mcl_doc.pl.tr:53): Missing placeholder(s) in translation
Error (./HUD/awards/locale/awards.de.tr:4): Missing placeholder(s) in translation
Error (./HUD/awards/locale/awards.ru.tr:9): Placeholder symbol '@1' is not in the correct ascending order in source string
Error (./HUD/mcl_death_messages/locale/mcl_death_messages.fr.tr:9): Missing placeholder(s) in translation
Error (./ITEMS/REDSTONE/mesecons_lightstone/locale/mesecons_lightstone.es.tr:3): Missing placeholder(s) in translation
Error (./ITEMS/REDSTONE/mesecons_commandblock/locale/mesecons_commandblock.es.tr:4): Missing placeholder(s) in translation
Error (./ITEMS/mcl_banners/locale/mcl_banners.fr.tr:54): Missing placeholder(s) in translation
Warning (./ITEMS/mcl_beehives/locale/mcl_beehives.ja.tr): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/screwdriver.zh_TW.tr:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/screwdriver.zh_TW.tr): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/screwdriver.ru.tr:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/screwdriver.ru.tr): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/screwdriver.ja.tr:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/screwdriver.ja.tr): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/screwdriver.fr.tr:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/screwdriver.fr.tr): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/screwdriver.pl.tr:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/screwdriver.pl.tr): No textdomain in file!
Error (./ITEMS/mcl_farming/locale/mcl_farming.fr.tr:19): Missing placeholder(s) in translation
Error (./ITEMS/mcl_buckets/locale/mcl_buckets.fr.tr:23): Missing placeholder(s) in translation
Error (./ITEMS/mcl_core/locale/mcl_core.pt_BR.tr:221): Invalid line type (neither empty, comment or translation)
Error (./ITEMS/mcl_books/locale/mcl_books.pl.tr:5): Missing placeholder(s) in translation
Error (./ITEMS/mcl_books/locale/mcl_books.pl.tr:6): Missing placeholder(s) in translation
Warning (./ITEMS/mcl_honey/locale/mcl_honey.ja.tr): No textdomain in file!
Error (./ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape(s) in translation
Error (./ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation
Error (./ENVIRONMENT/lightning/locale/lightning.es.tr:2): Bad escape(s) in translation
Error (./ENVIRONMENT/lightning/locale/lightning.es.tr:2): Missing placeholder(s) in translation
Error (./HUD/mcl_achievements/locale/template.txt:69): Invalid line type (neither empty, comment or translation)
Warning (./ITEMS/mcl_beehives/locale/template.txt): No textdomain in file!
Warning (./ITEMS/screwdriver/locale/template.txt:1): Missing space between '#' and 'textdomain'. Intentional?
Warning (./ITEMS/screwdriver/locale/template.txt): No textdomain in file!
Warning (./ITEMS/mcl_honey/locale/template.txt): No textdomain in file!
------------
TEST RESULT:
------------
939 file(s) checked. Test FAILED!
22 file(s) with error(s).
10 file(s) with warnings.

What should happen:

The script should return 0 errors.

Possible fixes

Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference.

If you happen to speak the language, fix the translation yourself.

You might also check if running the locale update script fixes some of these already.

Steps to reproduce

  1. Go to https://codeberg.org/Wuzzy/minetest_check_translations
  2. Read the source code and README so you understand what this script does
  3. Copy the Python script to the mods directory of MCL2
  4. Run it
MineClone2 version: 7b53b6e45f62f43b8e79b999c9bd8d73ba2e9f64 ### What happened? I've written a quick and dirty Python script to automatically test locale files (`*.tr` and `template.txt`) for syntax errors such as missing placeholders or bad escaping. I'm running this script on various games now to see what happens and in MCL2 the script has found a lot of errors. This is the test result: ``` Error (./HELP/mcl_tt/locale/mcl_tt.fr.tr:48): Missing placeholder(s) in translation Error (./HELP/mcl_tt/locale/mcl_tt.pl.tr:6): Missing placeholder(s) in translation Error (./HELP/doc/doc/locale/doc.fr.tr:27): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.fr.tr:6): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.fr.tr:99): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:56): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:57): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:81): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt_BR.tr:126): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:56): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:57): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:81): Missing placeholder(s) in translation Error (./HELP/doc/doc_items/locale/doc_items.pt.tr:126): Missing placeholder(s) in translation Error (./HELP/mcl_doc/locale/mcl_doc.fr.tr:53): Missing placeholder(s) in translation Error (./HELP/mcl_doc/locale/mcl_doc.de.tr:54): Missing placeholder(s) in translation Error (./HELP/mcl_doc/locale/mcl_doc.pl.tr:53): Missing placeholder(s) in translation Error (./HUD/awards/locale/awards.de.tr:4): Missing placeholder(s) in translation Error (./HUD/awards/locale/awards.ru.tr:9): Placeholder symbol '@1' is not in the correct ascending order in source string Error (./HUD/mcl_death_messages/locale/mcl_death_messages.fr.tr:9): Missing placeholder(s) in translation Error (./ITEMS/REDSTONE/mesecons_lightstone/locale/mesecons_lightstone.es.tr:3): Missing placeholder(s) in translation Error (./ITEMS/REDSTONE/mesecons_commandblock/locale/mesecons_commandblock.es.tr:4): Missing placeholder(s) in translation Error (./ITEMS/mcl_banners/locale/mcl_banners.fr.tr:54): Missing placeholder(s) in translation Warning (./ITEMS/mcl_beehives/locale/mcl_beehives.ja.tr): No textdomain in file! Warning (./ITEMS/screwdriver/locale/screwdriver.zh_TW.tr:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/screwdriver.zh_TW.tr): No textdomain in file! Warning (./ITEMS/screwdriver/locale/screwdriver.ru.tr:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/screwdriver.ru.tr): No textdomain in file! Warning (./ITEMS/screwdriver/locale/screwdriver.ja.tr:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/screwdriver.ja.tr): No textdomain in file! Warning (./ITEMS/screwdriver/locale/screwdriver.fr.tr:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/screwdriver.fr.tr): No textdomain in file! Warning (./ITEMS/screwdriver/locale/screwdriver.pl.tr:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/screwdriver.pl.tr): No textdomain in file! Error (./ITEMS/mcl_farming/locale/mcl_farming.fr.tr:19): Missing placeholder(s) in translation Error (./ITEMS/mcl_buckets/locale/mcl_buckets.fr.tr:23): Missing placeholder(s) in translation Error (./ITEMS/mcl_core/locale/mcl_core.pt_BR.tr:221): Invalid line type (neither empty, comment or translation) Error (./ITEMS/mcl_books/locale/mcl_books.pl.tr:5): Missing placeholder(s) in translation Error (./ITEMS/mcl_books/locale/mcl_books.pl.tr:6): Missing placeholder(s) in translation Warning (./ITEMS/mcl_honey/locale/mcl_honey.ja.tr): No textdomain in file! Error (./ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape(s) in translation Error (./ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation Error (./ENVIRONMENT/lightning/locale/lightning.es.tr:2): Bad escape(s) in translation Error (./ENVIRONMENT/lightning/locale/lightning.es.tr:2): Missing placeholder(s) in translation Error (./HUD/mcl_achievements/locale/template.txt:69): Invalid line type (neither empty, comment or translation) Warning (./ITEMS/mcl_beehives/locale/template.txt): No textdomain in file! Warning (./ITEMS/screwdriver/locale/template.txt:1): Missing space between '#' and 'textdomain'. Intentional? Warning (./ITEMS/screwdriver/locale/template.txt): No textdomain in file! Warning (./ITEMS/mcl_honey/locale/template.txt): No textdomain in file! ------------ TEST RESULT: ------------ 939 file(s) checked. Test FAILED! 22 file(s) with error(s). 10 file(s) with warnings. ``` ### What should happen: The script should return 0 errors. ### Possible fixes Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference. If you happen to speak the language, fix the translation yourself. You might also check if running the locale update script fixes some of these already. ### Steps to reproduce 1. Go to https://codeberg.org/Wuzzy/minetest_check_translations 2. Read the source code and README so you understand what this script does 3. Copy the Python script to the `mods` directory of MCL2 4. Run it
👍 2
Wuzzy added the
bug
unconfirmed
 labels 2023-03-14 01:34:15 +01:00
Wuzzy changed title from 60 translation files have syntax errors to 55 translation files have syntax errors 2023-03-14 01:41:11 +01:00
Wuzzy changed title from 55 translation files have syntax errors to 22 translation files have syntax errors 2023-03-14 11:46:24 +01:00
Wuzzy commented 2023-03-14 11:49:06 +01:00
Author
Member
Copy Link

OK, it turned out my script had a lot of false-positives so I've rewrote and re-run it, and they seem legit now. But there are still 22 errors left, and they seem legit. Plus some warnings.

OK, it turned out my script had a lot of false-positives so I've rewrote and re-run it, and they seem legit now. But there are still 22 errors left, and they seem legit. Plus some warnings.
👍 2
ancientmarinerdev commented 2023-03-14 15:13:15 +01:00
Owner
Copy Link

OK, it turned out my script had a lot of false-positives so I've rewrote and re-run it, and they seem legit now. But there are still 22 errors left, and they seem legit. Plus some warnings.

Phew. That seemed a big bit of work! This looks more manageable. Thanks for running this and sharing the results.

Oh, and thanks for MineClone2 and all the energy you invested in it! :D

> OK, it turned out my script had a lot of false-positives so I've rewrote and re-run it, and they seem legit now. But there are still 22 errors left, and they seem legit. Plus some warnings. Phew. That seemed a big bit of work! This looks more manageable. Thanks for running this and sharing the results. Oh, and thanks for MineClone2 and all the energy you invested in it! :D
ancientmarinerdev added
#P3: elevated
translation
 and removed
unconfirmed
 labels 2023-03-14 15:14:46 +01:00
Wuzzy commented 2023-03-16 15:23:33 +01:00
Author
Member
Copy Link

After the French update, the script now returns:

Error (./mods/HELP/mcl_tt/locale/mcl_tt.pl.tr:6): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:56): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:57): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:81): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:126): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:56): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:57): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:81): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:126): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/mcl_doc/locale/mcl_doc.de.tr:54): Missing placeholder(s) in translation: '@1' 
Error (./mods/HELP/mcl_doc/locale/mcl_doc.pl.tr:53): Missing placeholder(s) in translation: '@1' 
Error (./mods/HUD/awards/locale/awards.de.tr:4): Missing placeholder(s) in translation: '@1' 
Error (./mods/HUD/awards/locale/awards.ru.tr:9): Placeholder '@1' in source string was repeated
Error (./mods/HUD/awards/locale/awards.ja.tr:46): Placeholder '@2' in translation doesn't exist in source string
Error (./mods/ITEMS/REDSTONE/mesecons_lightstone/locale/mesecons_lightstone.es.tr:3): Missing placeholder(s) in translation: '@1' 
Error (./mods/ITEMS/REDSTONE/mesecons_commandblock/locale/mesecons_commandblock.es.tr:4): Missing placeholder(s) in translation: '@1' '@2' 
Warning (./mods/ITEMS/mcl_beehives/locale/mcl_beehives.ja.tr): No textdomain in file
Error (./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr:90): Missing placeholder(s) in translation: '@2' 
Error (./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr:91): Missing placeholder(s) in translation: '@2' 
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.zh_TW.tr:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.zh_TW.tr): No textdomain in file
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ru.tr:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ru.tr): No textdomain in file
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ja.tr:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ja.tr): No textdomain in file
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.fr.tr:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.fr.tr): No textdomain in file
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.pl.tr:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/screwdriver.pl.tr): No textdomain in file
Error (./mods/ITEMS/mcl_core/locale/mcl_core.pt_BR.tr:221): Invalid line type (neither empty, comment, textdomain or translation)
Error (./mods/ITEMS/mcl_books/locale/mcl_books.pl.tr:5): Missing placeholder(s) in translation: '@1' 
Error (./mods/ITEMS/mcl_books/locale/mcl_books.pl.tr:6): Missing placeholder(s) in translation: '@1' 
Warning (./mods/ITEMS/mcl_honey/locale/mcl_honey.ja.tr): No textdomain in file
Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape code in translation: '@ '
Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation: '@1' 
Error (./mods/ENVIRONMENT/lightning/locale/lightning.es.tr:2): Bad escape code in translation: '@ '
Error (./mods/ENVIRONMENT/lightning/locale/lightning.es.tr:2): Missing placeholder(s) in translation: '@1' 
Error (./mods/HUD/mcl_achievements/locale/template.txt:69): Invalid line type (neither empty, comment, textdomain or translation)
Error (./mods/ITEMS/mcl_armor/locale/template.txt:34): Non-empty translation in template file
Error (./mods/ITEMS/mcl_deepslate/locale/template.txt:53): Non-empty translation in template file
Warning (./mods/ITEMS/mcl_beehives/locale/template.txt): No textdomain in file
Error (./mods/ITEMS/mcl_enchanting/locale/template.txt:133): Non-empty translation in template file
Warning (./mods/ITEMS/screwdriver/locale/template.txt:1): Missing space between '#' and 'textdomain'
Warning (./mods/ITEMS/screwdriver/locale/template.txt): No textdomain in file
Warning (./mods/ITEMS/mcl_honey/locale/template.txt): No textdomain in file
------------
TEST RESULT:
------------
939 file(s) checked. Test FAILED!
19 file(s) with error(s).
10 file(s) with warnings.
53 file(s) with empty translations.

(Note I've added a new error type to the script which is why the number of errors went down by only 2.)

If you have any questions, just ask.

After the French update, the script now returns: ``` Error (./mods/HELP/mcl_tt/locale/mcl_tt.pl.tr:6): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:56): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:57): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:81): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt_BR.tr:126): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:56): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:57): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:81): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/doc/doc_items/locale/doc_items.pt.tr:126): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/mcl_doc/locale/mcl_doc.de.tr:54): Missing placeholder(s) in translation: '@1' Error (./mods/HELP/mcl_doc/locale/mcl_doc.pl.tr:53): Missing placeholder(s) in translation: '@1' Error (./mods/HUD/awards/locale/awards.de.tr:4): Missing placeholder(s) in translation: '@1' Error (./mods/HUD/awards/locale/awards.ru.tr:9): Placeholder '@1' in source string was repeated Error (./mods/HUD/awards/locale/awards.ja.tr:46): Placeholder '@2' in translation doesn't exist in source string Error (./mods/ITEMS/REDSTONE/mesecons_lightstone/locale/mesecons_lightstone.es.tr:3): Missing placeholder(s) in translation: '@1' Error (./mods/ITEMS/REDSTONE/mesecons_commandblock/locale/mesecons_commandblock.es.tr:4): Missing placeholder(s) in translation: '@1' '@2' Warning (./mods/ITEMS/mcl_beehives/locale/mcl_beehives.ja.tr): No textdomain in file Error (./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr:90): Missing placeholder(s) in translation: '@2' Error (./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr:91): Missing placeholder(s) in translation: '@2' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.zh_TW.tr:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.zh_TW.tr): No textdomain in file Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ru.tr:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ru.tr): No textdomain in file Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ja.tr:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.ja.tr): No textdomain in file Warning (./mods/ITEMS/screwdriver/locale/screwdriver.fr.tr:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.fr.tr): No textdomain in file Warning (./mods/ITEMS/screwdriver/locale/screwdriver.pl.tr:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/screwdriver.pl.tr): No textdomain in file Error (./mods/ITEMS/mcl_core/locale/mcl_core.pt_BR.tr:221): Invalid line type (neither empty, comment, textdomain or translation) Error (./mods/ITEMS/mcl_books/locale/mcl_books.pl.tr:5): Missing placeholder(s) in translation: '@1' Error (./mods/ITEMS/mcl_books/locale/mcl_books.pl.tr:6): Missing placeholder(s) in translation: '@1' Warning (./mods/ITEMS/mcl_honey/locale/mcl_honey.ja.tr): No textdomain in file Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Bad escape code in translation: '@ ' Error (./mods/ITEMS/mcl_fire/locale/mcl_fire.es.tr:15): Missing placeholder(s) in translation: '@1' Error (./mods/ENVIRONMENT/lightning/locale/lightning.es.tr:2): Bad escape code in translation: '@ ' Error (./mods/ENVIRONMENT/lightning/locale/lightning.es.tr:2): Missing placeholder(s) in translation: '@1' Error (./mods/HUD/mcl_achievements/locale/template.txt:69): Invalid line type (neither empty, comment, textdomain or translation) Error (./mods/ITEMS/mcl_armor/locale/template.txt:34): Non-empty translation in template file Error (./mods/ITEMS/mcl_deepslate/locale/template.txt:53): Non-empty translation in template file Warning (./mods/ITEMS/mcl_beehives/locale/template.txt): No textdomain in file Error (./mods/ITEMS/mcl_enchanting/locale/template.txt:133): Non-empty translation in template file Warning (./mods/ITEMS/screwdriver/locale/template.txt:1): Missing space between '#' and 'textdomain' Warning (./mods/ITEMS/screwdriver/locale/template.txt): No textdomain in file Warning (./mods/ITEMS/mcl_honey/locale/template.txt): No textdomain in file ------------ TEST RESULT: ------------ 939 file(s) checked. Test FAILED! 19 file(s) with error(s). 10 file(s) with warnings. 53 file(s) with empty translations. ``` (Note I've added a new error type to the script which is why the number of errors went down by only 2.) If you have any questions, just ask.
ancientmarinerdev added
looking for contributor
#P4 priority: medium
 and removed
#P3: elevated
 labels 2023-04-17 20:52:26 +02:00
ancientmarinerdev added this to the 3 - Near future milestone 2023-04-17 20:52:39 +02:00
Laudrin commented 2024-01-14 02:33:20 +01:00
Copy Link

If the errors still persist, I would like to help fixing them for the German translation if you tell me what I have to do when there is the error message Missing placeholder(s) in translation: '@1' (or @2) in your list.

Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference.
If you happen to speak the language, fix the translation yourself.

Can you give me an example how to fix the syntax errors or missing placeholders?

If the errors still persist, I would like to help fixing them for the German translation if you tell me what I have to do when there is the error message `Missing placeholder(s) in translation: '@1' ` (or @2) in your list. > Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference. > If you happen to speak the language, fix the translation yourself. Can you give me an example how to fix the syntax errors or missing placeholders?
the-real-herowl commented 2024-01-14 03:31:37 +01:00
Owner
Copy Link

If the errors still persist, I would like to help fixing them for the German translation if you tell me what I have to do when there is the error message Missing placeholder(s) in translation: '@1' (or @2) in your list.

Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference.
If you happen to speak the language, fix the translation yourself.

Can you give me an example how to fix the syntax errors or missing placeholders?

Basically, translation files look like

Original string in English=Übersetzte Zeichenfolge auf Deutsch

and the original string may have stuff like @1, @2 and so on in them, which are obviously placeholders for something that can be filled when the string is used. The translated string should contain the placeholders appropriately placed in the same amount (and there should be no space between @ and the number).

However, I don't think you would find any in the German translation in particular.

> If the errors still persist, I would like to help fixing them for the German translation if you tell me what I have to do when there is the error message `Missing placeholder(s) in translation: '@1' ` (or @2) in your list. > > > Easiest fix: Clear the offending strings. Maybe move a copy of the broken string to the bottom as a comment for future reference. > > If you happen to speak the language, fix the translation yourself. > > Can you give me an example how to fix the syntax errors or missing placeholders? Basically, translation files look like ``` Original string in English=Übersetzte Zeichenfolge auf Deutsch ``` and the original string may have stuff like `@1`, `@2` and so on in them, which are obviously placeholders for something that can be filled when the string is used. The translated string should contain the placeholders appropriately placed in the same amount (and there should be no space between `@` and the number). However, I don't think you would find any in the German translation in particular.
Laudrin commented 2024-01-14 04:52:07 +01:00
Copy Link

Ah, thank you, I think I now understand! :-)

Hope I can post the corrected versions here:

./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr, lines 90 and 91

The number you have entered (@1) is too big, it must be at most @2.=Die eingegebene Zahl (@1) ist zu groß, maximal ist @2 erlaubt.
The number you have entered (@1) is too small, it must be at least @2.=Die eingegebene Zahl (@1) ist zu klein, minimal ist @2 erlaubt.

Not sure about this one:
./mods/HELP/mcl_doc/locale/mcl_doc.de.tr, line 54

, =,

But I found and fixed in line 53:

This blocks drops the following when mined by shears: @1=Dieser Block wird folgendes abwerfen, wenn er mit einer Schere abgebaut wird: @1

I think the next one is a false positive, seems okay to me:

./mods/HUD/awards/locale/awards.de.tr, line 4

@1’s awards:=Auszeichnungen von @1:

The achievements file is incomplete. If the English template locale/translations file is complete, I may be able to add the missing translations later today.

Ah, thank you, I think I now understand! :-) Hope I can post the corrected versions here: ./mods/ITEMS/mcl_enchanting/locale/mcl_enchanting.de.tr, lines 90 and 91 ``` The number you have entered (@1) is too big, it must be at most @2.=Die eingegebene Zahl (@1) ist zu groß, maximal ist @2 erlaubt. The number you have entered (@1) is too small, it must be at least @2.=Die eingegebene Zahl (@1) ist zu klein, minimal ist @2 erlaubt. ``` Not sure about this one: ./mods/HELP/mcl_doc/locale/mcl_doc.de.tr, line 54 ``` , =, ``` But I found and fixed in line 53: ``` This blocks drops the following when mined by shears: @1=Dieser Block wird folgendes abwerfen, wenn er mit einer Schere abgebaut wird: @1 ``` I think the next one is a false positive, seems okay to me: ./mods/HUD/awards/locale/awards.de.tr, line 4 ``` @1’s awards:=Auszeichnungen von @1: ``` The achievements file is incomplete. If the English template locale/translations file is complete, I may be able to add the missing translations later today.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
mcl_chests_redo
improve_planks
fix-has_room
opencollective-link
upstream/unbreaking_on_crossbows
mcl_mobs_cleanup
mob_ignore_fix
slabs_block_light
hardness_blast_res_fixes
buildable_flowers
burning_in_thunder
redstone-update
add_shader_support
hammers_spears
EE_armor
mapgen_update
projectile-refactor
world_aligned_textures
zombie_texture_improvements
mcl_weathered_nodes
orb_of_exchange
fix_pumpkin_names
patch_skins
hollow_logs_ref_recovery
copper-adaptation-refs-recovery
add_asset_attributions_1
fix_pumpkin_typo_pumkin
cherry_grove
lush_caves
release/0.86.1
mobile_bow_workaround
release/0.85.1
optimize_jukebox_sounds
optimize_theme_ogg
Optimize_backgrounds_jpg
archaeology
spanish-translations
fish_bucket_nametag
villager-golem-fixes
fix_beetroot_fortune_drops
fix_sliding
illager_banner_description
release_0.83.1
skelly_horse_thunder
mobs_armor
mobsarmorfix
smoother_biome_borders
jukebox_sound_distance
cobble_abm_height
callable-tool-rightclick-functions
effects_rewrite
respiration
make-new-music-louder
damage
texture_conversion_wool
dehardcoded_signs
mcl_oxidization
Texture-Rename2
Bamboo-Piston-Dig
hardcore_mode_test
update_german_translation_attempt2
test_mob_tweaks_branch
csm-enabled
move_physics_tweak
water_footstep
biome_skycolor
bambootoo
lower_ow_mapgen
release/0.81.1
revert_chest_models
mcl-craftguide-drop-sfinv
mcl-structures-fixes
turtlestriderbeefox
arrow_node_collision
pvp_knockback
icebergs
light_dump
testing
mobs-redo
wood_api
mobs_head_rotation
bonemeal
dyable-leather-armor
new_mapgen_api
newmobs-master
formspec_overhaul
mobs
mapgen
mineclone5
chat-command-builder-2
formspec-v4
doc-refactoring
production
objects
0.87.2
0.87.1
0.87.0
0.86.2
0.86.1
0.86.0
0.85.0
0.84.0
0.83.1
0.83.0
0.82.0
0.81.1
0.81.0
0.80.1
0.80.0
0.79.0
0.78.0
0.77.0
0.76.0
0.75.0
0.74.0
0.73.1
0.73.0
0.72.1
0.72.0
0.71.0
0.70.0
0.69.1
0.69.0
0.68.0
0.67.2
0.67.1
0.67.0
0.66.2
0.66.1
0.66.0
0.65.2
0.65.1
0.65.0
0.64.0
0.63.0
0.62.0
0.61.0
0.60.0
0.59.2
0.59.1
0.59.0
0.58.1
0.58.0
0.57.1
0.57.0
0.56.2
0.56.1
0.56.0
0.55.1
0.55.0
0.54.1
0.54.0
0.53.4
0.53.3
0.53.2
0.53.1
0.53.0
0.52.1
0.52.0
0.51.1
0.51.0
0.50.2
0.50.1
0.50.0
0.49.0
0.48.1
0.48.0
0.47.1
0.47.0
0.46.0
0.45.1
0.45.0
0.44.0
0.43.0
0.42.0
0.21.0
Labels
Clear labels   
#P1 CRITICAL

for crashes, catastrophic bugs, etc.

  
#P2: HIGH

highest priority level

  
#P3: elevated

2nd-highest priority level

  
#P4 priority: medium

for things that aren't elevated but have been prioritised

  
#P6: low

low priority level

  
#Review

Tickets that may improve or fix an issue but will need a review after a short period of time

  
annoying

non-bugs that annoys people

  
API

   
bug

   
code quality

   
combat

   
commands

   
compatibility

Label to cover migration issues from old worlds or mcl5

  
configurability

   
contribution inside

   
controls

   
core feature

   
creative mode

   
delayed for engine release

   
documentation

   
duplicate

   
enhancement

enhancement request/suggestion

  
environment

Issues that affect weather or light

  
gameplay

   
graphics

   
ground content conflict

mapgen gc-specific quirks

  
GUI/HUD

   
help wanted

   
incomplete feature

   
invalid / won't fix

Invalid, or will not fix

  
items

   
looking for contributor

   
mapgen

   
meta

Relating to VoxeLibre on Mesehub and elsewhere

  
mineclone2+

fit for MineClone2-Plus

  
Minecraft >= 1.13

   
Minecraft >= 1.17

   
missing feature

   
mobile

involving Minetest on mobile devices (android)

  
mobs

   
mod support

Issues related to mod support

  
model needed

A model is needed to be created or updated

  
multiplayer

Issues affecting multiplayer

  
Needs adoption

PR needs author action or to be adopted so changes can be implemented to get the PR closer to merge

  
needs discussion

   
needs engine change

   
needs more information

   
needs research

   
nodes

   
non-Minecraft feature

   
non-mob entities

   
performance

   
player

issues related to the player

  
possible close

   
redstone

   
release notes

This ticket will need information adding to release notes

  
schematics

   
Skyblock

   
sounds

   
Testing / Retest

This needs testing or retesting

  
tools

Related to the tools scripts, which aren't run in game

  
translation

   
unconfirmed
  
mcl5

Possibly relevant for MineClone5

  
mcla

Possibly relevant for Mineclonia

  
Media missing

media files for this feature are missing - contributions welcome

No Label
#P1 CRITICAL
#P2: HIGH
#P3: elevated
#P4 priority: medium
#P6: low
#Review
annoying
API
bug
code quality
combat
commands
compatibility
configurability
contribution inside
controls
core feature
creative mode
delayed for engine release
documentation
duplicate
enhancement
environment
gameplay
graphics
ground content conflict
GUI/HUD
help wanted
incomplete feature
invalid / won't fix
items
looking for contributor
mapgen
meta
mineclone2+
Minecraft >= 1.13
Minecraft >= 1.17
missing feature
mobile
mobs
mod support
model needed
multiplayer
Needs adoption
needs discussion
needs engine change
needs more information
needs research
nodes
non-Minecraft feature
non-mob entities
performance
player
possible close
redstone
release notes
schematics
Skyblock
sounds
Testing / Retest
tools
translation
unconfirmed
mcl5
mcla
Media missing
Milestone
Clear milestone
No items
No Milestone
3 - Near future
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: VoxeLibre/VoxeLibre#3540
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?