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

Proposed by GunChleoc
Status: Merged
Merged at revision: 7909
Proposed branch: lp:~widelands-dev/widelands/bug-1540827
Merge into: lp:widelands
Diff against target: 25 lines (+3/-3)
1 file modified
data/maps/MP_Scenarios/Smugglers.wmf/scripting/smuggling.lua (+3/-3)
To merge this branch: bzr merge lp:~widelands-dev/widelands/bug-1540827
Reviewer Review Type Date Requested Status
TiborB Approve
Review via email: mp+290164@code.launchpad.net

Commit message

Fixed bug with establishing smuggling routes in the Smugglers scenario.

Description of the change

I only tested that the smuggling routes get established now.

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

LGTM, I have now tested it but should work OK

review: Approve
Revision history for this message
GunChleoc (gunchleoc) wrote :

@bunnybot merge

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 896. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/118652618.
Appveyor build 728. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1540827-728.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/maps/MP_Scenarios/Smugglers.wmf/scripting/smuggling.lua'
2--- data/maps/MP_Scenarios/Smugglers.wmf/scripting/smuggling.lua 2016-01-28 05:24:34 +0000
3+++ data/maps/MP_Scenarios/Smugglers.wmf/scripting/smuggling.lua 2016-03-26 09:22:28 +0000
4@@ -10,7 +10,7 @@
5 -- Find a warehouse for in the given area
6 function find_warehouse(region)
7 for idx,f in ipairs(region) do
8- if f.immovable and f.immovable.descr.type_name == "barbarians_warehouse" then
9+ if f.immovable and f.immovable.descr.type_name == "warehouse" then
10 return f.immovable
11 end
12 end
13@@ -47,10 +47,10 @@
14 end
15
16 if not send_whf.immovable or
17- send_whf.immovable.descr.type_name ~= "barbarians_warehouse" or
18+ send_whf.immovable.descr.type_name ~= "warehouse" or
19 send_whf.immovable.owner ~= send_plr or
20 not recv_whf.immovable or
21- recv_whf.immovable.descr.type_name ~= "barbarians_warehouse" or
22+ recv_whf.immovable.descr.type_name ~= "warehouse" or
23 recv_whf.immovable.owner ~= recv_plr
24 then
25 send_to_all(smuggling_route_broken:bformat(

Subscribers

People subscribed via source and target branches

to status/vote changes: