widelands

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

  1. bug-1507923
  2. Merge into trunk
Proposed by GunChleoc
Status: Merged
Merged at revision: 7574
Proposed branch: lp:~widelands-dev/widelands/bug-1507923
Merge into: lp:widelands
Diff against target: 12 lines (+1/-1)
1 file modified
src/logic/production_program.cc (+1/-1)
To merge this branch: bzr merge lp:~widelands-dev/widelands/bug-1507923
Reviewer Review Type Date Requested Status
TiborB Approve
Review via email: mp+275676@code.launchpad.net

Description of the change

This fixes the sefaring bug - Jens found the solution:

https://bugs.launchpad.net/widelands/+bug/1507923/comments/14

To post a comment you must log in.
Revision history for this message
TiborB (tiborb95) wrote :

looks harmless :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/logic/production_program.cc'
2--- src/logic/production_program.cc 2015-10-20 07:05:57 +0000
3+++ src/logic/production_program.cc 2015-10-26 09:32:00 +0000
4@@ -811,7 +811,7 @@
5 {
6 switch (m_feature) {
7 case SEAFARING: {
8- if (game.map().allows_seafaring())
9+ if (game.map().get_port_spaces().size() > 1)
10 return ps.program_step(game, 0);
11 else {
12 ps.set_production_result(_("No use for ships on this map!"));

Subscribers

People subscribed via source and target branches

to status/vote changes: