Workers exiting warehouse do not follow flag

Bug #1162918 reported by _aD
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
cghislai

Bug Description

When a warehouse is destroyed or dismantled, workers do not exit at the flag and join the road network; they act "lost" until they find a flag. They should exit to the flag and join the road network.

Related branches

_aD (ad-simplypeachy)
Changed in widelands:
status: New → Confirmed
Revision history for this message
SirVer (sirver) wrote :

I think when a warehouse is destroyed by an enemy, this behavior is wanted. But I agree that it should not happen when the player burns it down herself.

One way would be to check for a baseflag and launch the workers when there is one - if there is no baseflag, make the workers all lost. This still looks strange, when the warehouse is destroyed by ripping his base flag, but that cant't be helped.

Changed in widelands:
importance: Undecided → Low
Revision history for this message
_aD (ad-simplypeachy) wrote :

Agreed. Panic from a warehouse being destroyed would be suitable :-)

Revision history for this message
Nicolai Hähnle (nha) wrote :

Let me put it this way: Who the hell cares about an orderly exit from a building that has just caught on fire? :P Also, it's a cute analogy to Settlers 2. Note that it is also consistent with what happens when you destroy other buildings.

We are bikeshedding here, but I vote to leave the behaviour as is.

Revision history for this message
cghislai (charlyghislain) wrote :

I vote for panic only if building is destroyed, but not dismantled.
Also, see 1144465 and 923702

Revision history for this message
SirVer (sirver) wrote :

sgtm.

Revision history for this message
Hans Joachim Desserud (hjd) wrote :
Changed in widelands:
assignee: nobody → cghislai (charlyghislain)
Revision history for this message
cghislai (charlyghislain) wrote :

There is one issue that needs discussion...
Currently, all workers in stock are only freed if the warehouse is destroyed, leading the the explosion of carriers. I guess we all agree we should keep that. However, upon dismantling, currently only workers that are not yet incorporated in stock (such as soldiers) leave the building. So either we release all stock (maybe not carriers) when dismantling, either we don't release anything and rely on the player to empty it beforehand, either we only release soldiers. But the current behaviour is not straightforward imho.

Revision history for this message
cghislai (charlyghislain) wrote :

While i like the option to release all builders except carriers, it is a bit awkward to see 30 builders leaving the house and not a single one stopping to actually dismantle it...

Changed in widelands:
status: Confirmed → In Progress
milestone: none → build18-rc1
Revision history for this message
SirVer (sirver) wrote :

Why? that is the way the system of Widelands works: everyone has his/her job and they never take the job of others. When there is no builder around, no one will dismantle the house. If none of them got the job, they will happily walk away - being glad that they where not chosen for the stupid task.

Changed in widelands:
status: In Progress → Fix Committed
Revision history for this message
SirVer (sirver) wrote :

Released in build-18 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.