Collectors message contains bad names

Bug #1451977 reported by Albert Einstein
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
High
Unassigned

Bug Description

To reproduce:

1. Begin the game in Collectors mode in any language (including English!)
2. Wait for the first message
3. Read

You will see that the names should be (for example) "Wood lance", not "wood_lance". In other languages it would be "Drewniana lanca" instead of "wood_lance".

So the names are hardcoded, not taken from wares names.

Related branches

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Thanks for reporting.

I see the same thing here with r7453.

>So the names are hardcoded, not taken from wares names.

Sorta. Wares have two names, an internal one and a display name. The internal name is always the same so it can be uniquely identified and compared regardless of language chosen. The display name is the translatable one which is (or should at least) be shown in the currently selected language.

For any text shown to players, including these messages the display names should be used. I'm fairly sure this used to be the case too.

Changed in widelands:
importance: Undecided → High
milestone: none → build19-rc1
status: New → Triaged
tags: added: internationalization lowhangingfruit
Revision history for this message
GunChleoc (gunchleoc) wrote :

We only added the functionality to get at the display name for wares etc. when we automated the building help. So, this bug has always been there, but it's easy to fix. Thanks for reporting :)

Changed in widelands:
assignee: nobody → GunChleoc (gunchleoc)
status: Triaged → In Progress
GunChleoc (gunchleoc)
Changed in widelands:
status: In Progress → Fix Committed
GunChleoc (gunchleoc)
Changed in widelands:
assignee: GunChleoc (gunchleoc) → nobody
GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.