[fixed] 3.8.0 beta1,list-scerarios and load-scenario incompatible

[Baŝto]

 64bit archlinux:

Code: [Select]

megaglest --list-scenarios and

Code: [Select]

megaglest --load-scenario= are not compatible.

Code: [Select]

megaglest --list-scenarios lists the folder names of all scenarios

Code: [Select]

megaglest --load-scenario= expects the scenario name how it is defined in the .lng-file


Edit (tomreyn): Removed [bug] tag, not needed on the bug reports forum.

[Omega]

I'm not quite sure I understand. The name of the scenario is decided entirely by the folder. The language file has no say over the scenario name as far as I know. The scenario's XML file must be named <folderName>.xml and the language files must be named <folderName>_<language>.lng.

For example, running megaglest --list-scenarios on the latest stable release produces (EDIT: except the benchmark scenario isn't part of the release):

Code: [Select]

Scenarios found:
===========================================
amazones
amazones_light
anarchy
beehive_castle
beginner_1
beginner_2
beginner_3
benchmark
capture_enemy_flag
duel
egypt_mayhem
indian_attack
kill_the_magic
magic_can_do_it
my_island
norsemen_attack
one_hope
siege_of_despair
storming
tech_and_magic
tough_duel
waterworld
===========================================
Total: 22
I can run, say, Egypt Mayhem with megaglest --load-scenario=egypt_mayhem, which works fine (despite the fact that in-game, the translator replaces underscores with spaces and Capitalizes The First Word Of Everything). The same works for scenarios with an actual language file, like storming.

If this wasn't a misunderstanding of the commands, can you please give a specific example of how the features are incompatible?

[Baŝto]

First of all: I use the German localization.

Code: [Select]

megaglest --load-scenario=egypt_mayhem works, that’s right, this scenario name does not seem being translated. And just few scenario names are translated, at least in the German version.

I can’t run

Code: [Select]

megaglest --load-scenario=amazones instead I have to use

Code: [Select]

megaglest --load-scenario=amazonen

[filux]

wrong thinking

everything in console, inside a files, etc. is default in english and should be always in english

[Baŝto]

wrong thinking

everything in console, inside a files, etc. is default in english and should be always in english

I totally agree with that, but

Code: [Select]

--load-scenario= only works with the translated names … and that’s the problem

[Omega]

I agree that this should be considered a bug. The two commands should be 100% compatible. Ideally, we'd be able to use both the translated name and the folder name (which isn't necessarily English, by the way), but that would raise the issue of conflicts. Best to just use the folder name when launching scenarios from the command line.

[titi]

Yes, I also think thats a bug.  Regarding these command line params, even the english translation is wrong. There should be used the filenames instead .
Same for translated unit names in logfiles and so on. A translation should only be shown in the GUI, that's all.

[softcoder]

Fixed in svn rev#: 4550