← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~widelands-dev/widelands/bug-1507923 into lp:widelands

 

GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1507923 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1507923/+merge/275676

This fixes the sefaring bug - Jens found the solution:

https://bugs.launchpad.net/widelands/+bug/1507923/comments/14
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1507923 into lp:widelands.
=== modified file 'src/logic/production_program.cc'
--- src/logic/production_program.cc	2015-10-20 07:05:57 +0000
+++ src/logic/production_program.cc	2015-10-26 09:32:00 +0000
@@ -811,7 +811,7 @@
 {
 	switch (m_feature) {
 		case SEAFARING: {
-			if (game.map().allows_seafaring())
+			if (game.map().get_port_spaces().size() > 1)
 				return ps.program_step(game, 0);
 			else {
 				ps.set_production_result(_("No use for ships on this map!"));


Follow ups