Merge lp:~widelands-dev/widelands/barbarians2-starting-objective into lp:widelands

Proposed by GunChleoc
Status: Merged
Merged at revision: 7989
Proposed branch: lp:~widelands-dev/widelands/barbarians2-starting-objective
Merge into: lp:widelands
Diff against target: 67 lines (+30/-2)
2 files modified
data/campaigns/bar02.wmf/scripting/mission_thread.lua (+14/-2)
data/campaigns/bar02.wmf/scripting/texts.lua (+16/-0)
To merge this branch: bzr merge lp:~widelands-dev/widelands/barbarians2-starting-objective
Reviewer Review Type Date Requested Status
Miroslav Remák code, testing Approve
Review via email: mp+293546@code.launchpad.net

Commit message

Barbarian Campaign 2: Added an objective to the first message.

Description of the change

Some players find it confusing that the second Barbarian scenario has no instructions in the first message. So, I have added some. This also requires the player to expand to the east a bit, which will get them closer to the tracks.

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

Continuous integration builds have changed state:

Travis build 1083. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/127258989.
Appveyor build 914. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_barbarians2_starting_objective-914.

Revision history for this message
bunnybot (widelandsofficial) wrote :

Bunnybot encountered an error while working on this merge proposal:

The read operation timed out

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 1083. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/127258989.
Appveyor build 914. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_barbarians2_starting_objective-914.

Revision history for this message
bunnybot (widelandsofficial) wrote :

Bunnybot encountered an error while working on this merge proposal:

The read operation timed out

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 1083. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/127258989.
Appveyor build 914. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_barbarians2_starting_objective-914.

Revision history for this message
Miroslav Remák (miroslavr256) wrote :

LGTM.

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

Thanks!

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/campaigns/bar02.wmf/scripting/mission_thread.lua'
--- data/campaigns/bar02.wmf/scripting/mission_thread.lua 2016-01-28 05:24:34 +0000
+++ data/campaigns/bar02.wmf/scripting/mission_thread.lua 2016-05-02 15:51:07 +0000
@@ -38,7 +38,19 @@
38 wake_me(2000)38 wake_me(2000)
39 campaign_message_box(story_msg_1)39 campaign_message_box(story_msg_1)
4040
41 wake_me(120000)41 local o = add_campaign_objective(obj_build_basic_economy)
42 while not check_for_buildings(p1, {
43 barbarians_lumberjacks_hut = 1,
44 barbarians_rangers_hut = 1,
45 barbarians_quarry = 1,
46 barbarians_wood_hardener = 1,
47 }) do sleep(3413) end
48 while not check_for_buildings(p1, {barbarians_sentry=1}, game.map:get_field(57,36):region(6)) do
49 sleep(1500)
50 end
51 o.done = true
52 sleep(3000)
53
42 campaign_message_box(briefing_msg_1)54 campaign_message_box(briefing_msg_1)
43 campaign_message_box(order_msg_1_small_food_economy)55 campaign_message_box(order_msg_1_small_food_economy)
4456
@@ -53,7 +65,7 @@
53 "barbarians_sentry",65 "barbarians_sentry",
54 }66 }
5567
56 local o = add_campaign_objective(obj_build_small_food_economy)68 o = add_campaign_objective(obj_build_small_food_economy)
57 while not check_for_buildings(p1, {69 while not check_for_buildings(p1, {
58 barbarians_fishers_hut = 1,70 barbarians_fishers_hut = 1,
59 barbarians_hunters_hut = 1,71 barbarians_hunters_hut = 1,
6072
=== modified file 'data/campaigns/bar02.wmf/scripting/texts.lua'
--- data/campaigns/bar02.wmf/scripting/texts.lua 2016-01-28 05:24:34 +0000
+++ data/campaigns/bar02.wmf/scripting/texts.lua 2016-05-02 15:51:07 +0000
@@ -16,6 +16,21 @@
16-- =======================================================================16-- =======================================================================
1717
1818
19obj_build_basic_economy = {
20 name = "build_basic_food_economy",
21 title = _"Build up a basic economy",
22 number = 5,
23 body = objective_text(_"Build up a basic economy",
24 _"Start collecting basic building materials, and fortify the camp."
25 .. paragraphdivider() ..
26 listitem_bullet(_"Build a lumberjack’s hut") ..
27 listitem_bullet(_"Build a ranger’s hut") ..
28 listitem_bullet(_"Build a quarry") ..
29 listitem_bullet(_"Build a wood hardener") ..
30 listitem_bullet(_"Build a sentry on the eastern border") ..
31 listitem_arrow(_"You should eventually also build a reed yard and a lime kiln."))
32}
33
19obj_build_small_food_economy = {34obj_build_small_food_economy = {
20 name = "build_small_food_economy",35 name = "build_small_food_economy",
21 title = _"Build up a small food economy",36 title = _"Build up a small food economy",
@@ -149,6 +164,7 @@
149 .. paragraphdivider() ..164 .. paragraphdivider() ..
150 -- TRANSLATORS: Thron165 -- TRANSLATORS: Thron
151 _([[For now, we are resting at the borders of the old forest and preparing for the coming days.]]))166 _([[For now, we are resting at the borders of the old forest and preparing for the coming days.]]))
167 .. new_objectives(obj_build_basic_economy)
152}168}
153169
154story_msg_2 = {170story_msg_2 = {

Subscribers

People subscribed via source and target branches

to status/vote changes: