shortsighted shipwright (ship in pond)

Bug #963799 reported by Astuur
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Unassigned

Bug Description

The shipwrigt chooses an unsuitable place to build the ship.
Maybe there could be a check, especially when two alternatives are available, to verify that the
water body to build the ship in, is not too small?
See attachment

Tags: seafaring

Related branches

Revision history for this message
Astuur (wolfsteinmetz) wrote :
Revision history for this message
Astuur (wolfsteinmetz) wrote :

Want to add that the situation shown in the picture above
crashes the game as soon as the second ship is finished,
if you do not stop the shipwright.
Space problem, probably.

Revision history for this message
Nasenbaer (nasenbaer) wrote :

Best way would be to check, whether the water area has at least two port buildspaces connected to it, to ensure, that ships would make sense.

Changed in widelands:
status: New → Confirmed
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Nice catch. :) Could you please report the problem in comment #2 as a separate issue? If there is a space problem that will probably be triggered no matter the size of the sea, if you build, say, a couple of hundred ships. (And someone will do this at some point.)

Revision history for this message
SirVer (sirver) wrote :

S2 had an index for each "pont" in its map format iirc. A water without a port had index 0. They did not use it for the ships though because the ship builder could be as stupid as the one in widelands currently is.

Nasenbaer (nasenbaer)
tags: added: seafaring
removed: gamedata gameplay
Revision history for this message
Frank Pieper (frank-pieper-1) wrote :

I think the straightest Way is to implement an Idea of connected Water-Area and let the Player select which Water gets next Ships. If he don't select to send to Ponds it never happened (but he can if he want if he think a Pond to be large enough for his Needs).

Revision history for this message
SirVer (sirver) wrote :

Setting to incomplete for bug sweeping.

Changed in widelands:
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for widelands because there has been no activity for 60 days.]

Changed in widelands:
status: Incomplete → Expired
Revision history for this message
SirVer (sirver) wrote :

I think #3 is the way to go

Changed in widelands:
importance: Undecided → Low
status: Expired → Triaged
Revision history for this message
GunChleoc (gunchleoc) wrote :

#3 would break the Atlantean scenario though - we would need to add some port spaces to the map.

GunChleoc (gunchleoc)
Changed in widelands:
status: Triaged → Fix Committed
milestone: none → build20-rc1
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.