Constructionsite help button points to wrong building

Bug #1789896 reported by Benedikt Straub
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
High
Unassigned

Bug Description

- Build constructionsites for a blacksmithy and a small armour smithy
- Open the window for the blacksmithy constructionsite
- Open the window for the armour smithy constructionsite (now you have 2 windows open)
- Click the help button in the blacksmithy´s window. It opens the help window for the armour smithy.

Tags: ui

Related branches

GunChleoc (gunchleoc)
Changed in widelands:
status: New → Confirmed
importance: Undecided → High
Nasenbaer (nasenbaer)
Changed in widelands:
assignee: nobody → Nasenbaer (nasenbaer)
status: Confirmed → Fix Committed
assignee: Nasenbaer (nasenbaer) → nobody
Revision history for this message
Nasenbaer (nasenbaer) wrote :

Fix committed in bzr rev 8804.
The problem was easy to fix, but not obvious to see:

BuildingWindow reinitialized the BuildingHelpWindow with registry "Constructionsite" each time a ConstructionsiteWindow was opened. This is fixed by using the registry of the building-to-be instead.

Revision history for this message
GunChleoc (gunchleoc) wrote :

Thanks for the fix. Great to have you back :)

Could you please create a branch and a merge request in the future? We have some automated checks running on Travis & AppVeyor to compile with various compilers and run the test suite etc. This has already prevented a lot of bugs from reaching trunk ;)

Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build20-rc1

Changed in widelands:
status: Fix Committed → Fix Released
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.