Tutorial fails if player is in road building mode when military building finishes
Bug #1286576 reported by
Hans Joachim Desserud
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| widelands |
Fix Released
|
Low
|
Unassigned | ||
| widelands (Debian) |
Fix Released
|
Unknown
|
|||
Bug Description
Forwarded from the Debian bug tracker. Original description:
"in the tutorial game, the player must build military buildings, and when
they are finished, the tutorial proceeds with introducing mines.
if the player stays in road building mode after building the military
buildings, or if he is just building a road to an additional military
building, and the tutorial takes control to build a road into the
mountains, it is (unexpectedly, to the tutorial script) not presented
with the build menu, but with the road building menu (consisting of only
a red 'x' to abort road building), and fails with a in-game message "Lua
Coroutine Fa...":
[/tmp/buildd/
"map:init"]: 132: attempt to index local 'panel' (a nil value)"
For more details, see the Debian bug report.
I was able to reproduce this issue in r6843. I also noticed that if I tried to trigger it on earlier events it seemed to close road building mode before continuing.
Related branches
lp:~widelands-dev/widelands/new-tutorials
- SirVer: Approve
- GunChleoc: Approve
- wl-zocker: Needs Resubmitting
-
Diff: 155489 lines (+86758/-41659)220 files modifiedcampaigns/atl01.wmf/scripting/texts.lua (+3/-3)
campaigns/bar01.wmf/elemental (+9/-8)
campaigns/bar01.wmf/extra_data (+3/-1)
campaigns/bar01.wmf/player_names (+7/-3)
campaigns/bar01.wmf/player_position (+4/-2)
campaigns/bar01.wmf/scripting/helper_functions.lua (+22/-0)
campaigns/bar01.wmf/scripting/init.lua (+13/-10)
campaigns/bar01.wmf/scripting/mission_thread.lua (+120/-220)
campaigns/bar01.wmf/scripting/secret_village.lua (+158/-0)
campaigns/bar01.wmf/scripting/starting_conditions.lua (+17/-22)
campaigns/bar01.wmf/scripting/texts.lua (+351/-263)
campaigns/bar02.wmf/elemental (+1/-1)
campaigns/bar02.wmf/scripting/init.lua (+2/-2)
campaigns/bar02.wmf/scripting/texts.lua (+40/-119)
campaigns/campaigns.conf (+6/-12)
campaigns/emp01.wmf/scripting/mission_thread.lua (+2/-2)
campaigns/emp01.wmf/scripting/starting_conditions.lua (+1/-1)
campaigns/emp01.wmf/scripting/texts.lua (+9/-10)
campaigns/emp02.wmf/scripting/mission_thread.lua (+7/-7)
campaigns/emp02.wmf/scripting/starting_conditions.lua (+5/-5)
campaigns/emp02.wmf/scripting/texts.lua (+4/-4)
campaigns/t01.wmf/elemental (+0/-10)
campaigns/t01.wmf/extra_data (+0/-2)
campaigns/t01.wmf/player_names (+0/-6)
campaigns/t01.wmf/player_position (+0/-3)
campaigns/t01.wmf/scripting/init.lua (+0/-71)
campaigns/t01.wmf/scripting/initial_messages.lua (+0/-41)
campaigns/t01.wmf/scripting/khantrukhs_talking.lua (+0/-63)
campaigns/t01.wmf/scripting/story_messages.lua (+0/-20)
campaigns/t01.wmf/scripting/texts.lua (+0/-173)
campaigns/tutorial01_basic_control.wmf/elemental (+9/-9)
campaigns/tutorial01_basic_control.wmf/extra_data (+3/-1)
campaigns/tutorial01_basic_control.wmf/player_names (+7/-9)
campaigns/tutorial01_basic_control.wmf/player_position (+4/-4)
campaigns/tutorial01_basic_control.wmf/scripting/helper_functions.lua (+32/-0)
campaigns/tutorial01_basic_control.wmf/scripting/helper_functions_demonstration.lua (+159/-0)
campaigns/tutorial01_basic_control.wmf/scripting/init.lua (+35/-0)
campaigns/tutorial01_basic_control.wmf/scripting/mission_thread.lua (+189/-598)
campaigns/tutorial01_basic_control.wmf/scripting/starting_conditions.lua (+10/-0)
campaigns/tutorial01_basic_control.wmf/scripting/texts.lua (+280/-221)
campaigns/tutorial02_warfare.wmf/elemental (+10/-0)
campaigns/tutorial02_warfare.wmf/extra_data (+4/-0)
campaigns/tutorial02_warfare.wmf/objective (+4/-0)
campaigns/tutorial02_warfare.wmf/player/1/messages (+4/-0)
campaigns/tutorial02_warfare.wmf/player/2/messages (+4/-0)
campaigns/tutorial02_warfare.wmf/player_names (+16/-0)
campaigns/tutorial02_warfare.wmf/player_position (+6/-0)
campaigns/tutorial02_warfare.wmf/port_spaces (+7/-0)
campaigns/tutorial02_warfare.wmf/scripting/init.lua (+20/-0)
campaigns/tutorial02_warfare.wmf/scripting/mission_thread.lua (+105/-0)
campaigns/tutorial02_warfare.wmf/scripting/starting_conditions.lua (+69/-0)
campaigns/tutorial02_warfare.wmf/scripting/texts.lua (+213/-0)
campaigns/tutorial02_warfare.wmf/version (+11/-0)
campaigns/tutorial03_seafaring.wmf/elemental (+10/-0)
campaigns/tutorial03_seafaring.wmf/extra_data (+4/-0)
campaigns/tutorial03_seafaring.wmf/objective (+4/-0)
campaigns/tutorial03_seafaring.wmf/player/1/messages (+4/-0)
campaigns/tutorial03_seafaring.wmf/player_names (+10/-0)
campaigns/tutorial03_seafaring.wmf/player_position (+5/-0)
campaigns/tutorial03_seafaring.wmf/port_spaces (+11/-0)
campaigns/tutorial03_seafaring.wmf/scripting/helper_functions.lua (+28/-0)
campaigns/tutorial03_seafaring.wmf/scripting/init.lua (+28/-0)
campaigns/tutorial03_seafaring.wmf/scripting/mission_thread.lua (+84/-0)
campaigns/tutorial03_seafaring.wmf/scripting/starting_conditions.lua (+164/-0)
campaigns/tutorial03_seafaring.wmf/scripting/texts.lua (+194/-0)
campaigns/tutorial03_seafaring.wmf/version (+11/-0)
campaigns/tutorial04_economy.wmf/elemental (+10/-0)
campaigns/tutorial04_economy.wmf/extra_data (+4/-0)
campaigns/tutorial04_economy.wmf/objective (+4/-0)
campaigns/tutorial04_economy.wmf/player/1/messages (+4/-0)
campaigns/tutorial04_economy.wmf/player/2/messages (+4/-0)
campaigns/tutorial04_economy.wmf/player_names (+16/-0)
campaigns/tutorial04_economy.wmf/player_position (+6/-0)
campaigns/tutorial04_economy.wmf/port_spaces (+7/-0)
campaigns/tutorial04_economy.wmf/scripting/helper_functions.lua (+29/-0)
campaigns/tutorial04_economy.wmf/scripting/init.lua (+31/-0)
campaigns/tutorial04_economy.wmf/scripting/mission_thread.lua (+140/-0)
campaigns/tutorial04_economy.wmf/scripting/starting_conditions.lua (+171/-0)
campaigns/tutorial04_economy.wmf/scripting/texts.lua (+410/-0)
campaigns/tutorial04_economy.wmf/version (+11/-0)
campaigns/tutorials.conf (+28/-0)
po/map_plateau.wmf/map_plateau.wmf.pot (+1/-1)
po/maps/maps.pot (+61/-109)
po/mp_scenario_island_hopping.wmf/mp_scenario_island_hopping.wmf.pot (+1/-1)
po/mp_scenario_smugglers.wmf/mp_scenario_smugglers.wmf.pot (+1/-1)
po/scenario_atl01.wmf/scenario_atl01.wmf.pot (+5/-5)
po/scenario_bar01.wmf/cs.po (+1669/-510)
po/scenario_bar01.wmf/da.po (+532/-270)
po/scenario_bar01.wmf/de.po (+1843/-570)
po/scenario_bar01.wmf/en_GB.po (+622/-300)
po/scenario_bar01.wmf/eo.po (+1067/-0)
po/scenario_bar01.wmf/es.po (+652/-310)
po/scenario_bar01.wmf/fi.po (+583/-373)
po/scenario_bar01.wmf/fr.po (+1861/-571)
po/scenario_bar01.wmf/gd.po (+1820/-444)
po/scenario_bar01.wmf/gl.po (+651/-310)
po/scenario_bar01.wmf/hu.po (+1641/-499)
po/scenario_bar01.wmf/it.po (+1720/-523)
po/scenario_bar01.wmf/ja.po (+1643/-276)
po/scenario_bar01.wmf/la.po (+1025/-0)
po/scenario_bar01.wmf/lt.po (+995/-0)
po/scenario_bar01.wmf/nb.po (+986/-327)
po/scenario_bar01.wmf/nl.po (+1789/-530)
po/scenario_bar01.wmf/nn.po (+1058/-0)
po/scenario_bar01.wmf/pl.po (+1770/-535)
po/scenario_bar01.wmf/pt.po (+1671/-519)
po/scenario_bar01.wmf/pt_BR.po (+1675/-507)
po/scenario_bar01.wmf/ru.po (+861/-313)
po/scenario_bar01.wmf/scenario_bar01.wmf.pot (+517/-290)
po/scenario_bar01.wmf/si.po (+511/-266)
po/scenario_bar01.wmf/sk.po (+1619/-497)
po/scenario_bar01.wmf/sv.po (+1625/-493)
po/scenario_bar01.wmf/vi.po (+1011/-0)
po/scenario_bar02.wmf/scenario_bar02.wmf.pot (+127/-127)
po/scenario_dummy.wmf/scenario_dummy.wmf.pot (+2/-2)
po/scenario_emp01.wmf/scenario_emp01.wmf.pot (+112/-117)
po/scenario_emp02.wmf/scenario_emp02.wmf.pot (+6/-6)
po/scenario_t01.wmf/cs.po (+0/-525)
po/scenario_t01.wmf/da.po (+0/-425)
po/scenario_t01.wmf/de.po (+0/-548)
po/scenario_t01.wmf/en_GB.po (+0/-426)
po/scenario_t01.wmf/eo.po (+0/-426)
po/scenario_t01.wmf/es.po (+0/-428)
po/scenario_t01.wmf/fi.po (+0/-448)
po/scenario_t01.wmf/fr.po (+0/-543)
po/scenario_t01.wmf/gd.po (+0/-550)
po/scenario_t01.wmf/gl.po (+0/-429)
po/scenario_t01.wmf/he.po (+0/-425)
po/scenario_t01.wmf/hu.po (+0/-525)
po/scenario_t01.wmf/it.po (+0/-547)
po/scenario_t01.wmf/ja.po (+0/-426)
po/scenario_t01.wmf/nb.po (+0/-468)
po/scenario_t01.wmf/nl.po (+0/-536)
po/scenario_t01.wmf/pl.po (+0/-524)
po/scenario_t01.wmf/pt.po (+0/-530)
po/scenario_t01.wmf/pt_BR.po (+0/-537)
po/scenario_t01.wmf/ru.po (+0/-534)
po/scenario_t01.wmf/scenario_t01.wmf.pot (+0/-389)
po/scenario_t01.wmf/si.po (+0/-426)
po/scenario_t01.wmf/sk.po (+0/-516)
po/scenario_t01.wmf/sv.po (+0/-529)
po/scenario_t01.wmf/vi.po (+0/-435)
po/scenario_t02.wmf/he.po (+0/-718)
po/scenario_tutorial01_basic_control.wmf/cs.po (+835/-655)
po/scenario_tutorial01_basic_control.wmf/de.po (+943/-701)
po/scenario_tutorial01_basic_control.wmf/el.po (+473/-593)
po/scenario_tutorial01_basic_control.wmf/en_GB.po (+486/-593)
po/scenario_tutorial01_basic_control.wmf/eo.po (+520/-587)
po/scenario_tutorial01_basic_control.wmf/es.po (+528/-593)
po/scenario_tutorial01_basic_control.wmf/fi.po (+468/-594)
po/scenario_tutorial01_basic_control.wmf/fr.po (+951/-709)
po/scenario_tutorial01_basic_control.wmf/gd.po (+952/-710)
po/scenario_tutorial01_basic_control.wmf/gl.po (+528/-593)
po/scenario_tutorial01_basic_control.wmf/hu.po (+886/-677)
po/scenario_tutorial01_basic_control.wmf/it.po (+871/-677)
po/scenario_tutorial01_basic_control.wmf/ja.po (+1095/-729)
po/scenario_tutorial01_basic_control.wmf/la.po (+518/-593)
po/scenario_tutorial01_basic_control.wmf/lt.po (+479/-595)
po/scenario_tutorial01_basic_control.wmf/nb.po (+594/-597)
po/scenario_tutorial01_basic_control.wmf/nl.po (+913/-689)
po/scenario_tutorial01_basic_control.wmf/nn.po (+528/-593)
po/scenario_tutorial01_basic_control.wmf/pl.po (+913/-677)
po/scenario_tutorial01_basic_control.wmf/pt.po (+849/-673)
po/scenario_tutorial01_basic_control.wmf/pt_BR.po (+854/-674)
po/scenario_tutorial01_basic_control.wmf/ru.po (+541/-597)
po/scenario_tutorial01_basic_control.wmf/scenario_tutorial01_basic_control.wmf.pot (+442/-554)
po/scenario_tutorial01_basic_control.wmf/sk.po (+828/-652)
po/scenario_tutorial01_basic_control.wmf/sv.po (+849/-664)
po/scenario_tutorial01_basic_control.wmf/tr.po (+488/-593)
po/scenario_tutorial01_basic_control.wmf/uk.po (+476/-595)
po/scenario_tutorial02_warfare.wmf/cs.po (+1561/-0)
po/scenario_tutorial02_warfare.wmf/de.po (+1695/-0)
po/scenario_tutorial02_warfare.wmf/el.po (+544/-0)
po/scenario_tutorial02_warfare.wmf/en_GB.po (+604/-0)
po/scenario_tutorial02_warfare.wmf/eo.po (+663/-0)
po/scenario_tutorial02_warfare.wmf/es.po (+653/-0)
po/scenario_tutorial02_warfare.wmf/fi.po (+553/-0)
po/scenario_tutorial02_warfare.wmf/fr.po (+1712/-0)
po/scenario_tutorial02_warfare.wmf/gd.po (+1712/-0)
po/scenario_tutorial02_warfare.wmf/gl.po (+655/-0)
po/scenario_tutorial02_warfare.wmf/hu.po (+1512/-0)
po/scenario_tutorial02_warfare.wmf/it.po (+1613/-0)
po/scenario_tutorial02_warfare.wmf/ja.po (+1589/-0)
po/scenario_tutorial02_warfare.wmf/la.po (+619/-0)
po/scenario_tutorial02_warfare.wmf/nb.po (+963/-0)
po/scenario_tutorial02_warfare.wmf/nl.po (+1648/-0)
po/scenario_tutorial02_warfare.wmf/nn.po (+648/-0)
po/scenario_tutorial02_warfare.wmf/pl.po (+1623/-0)
po/scenario_tutorial02_warfare.wmf/pt.po (+1567/-0)
po/scenario_tutorial02_warfare.wmf/pt_BR.po (+1578/-0)
po/scenario_tutorial02_warfare.wmf/ru.po (+865/-0)
po/scenario_tutorial02_warfare.wmf/scenario_tutorial02_warfare.wmf.pot (+473/-0)
po/scenario_tutorial02_warfare.wmf/sk.po (+1517/-0)
po/scenario_tutorial02_warfare.wmf/sv.po (+1522/-0)
po/scenario_tutorial02_warfare.wmf/tr.po (+577/-0)
po/scenario_tutorial02_warfare.wmf/uk.po (+548/-0)
po/scenario_tutorial03_seafaring.wmf/scenario_tutorial03_seafaring.wmf.pot (+404/-0)
po/scenario_tutorial04_economy.wmf/scenario_tutorial04_economy.wmf.pot (+712/-0)
po/texts/texts.pot (+44/-39)
po/tribe_atlanteans/tribe_atlanteans.pot (+1/-1)
po/tribe_barbarians/tribe_barbarians.pot (+1/-1)
po/tribe_empire/tribe_empire.pot (+1/-1)
po/tribes/tribes.pot (+1/-1)
po/widelands/widelands.pot (+65/-23)
po/widelands_console/widelands_console.pot (+1/-1)
po/win_conditions/win_conditions.pot (+1/-1)
po/world/world.pot (+1/-1)
scripting/coroutine.lua (+11/-11)
scripting/format_scenario.lua (+7/-6)
scripting/formatting.lua (+4/-4)
scripting/messages.lua (+178/-0)
scripting/set.lua (+18/-18)
scripting/table.lua (+2/-2)
scripting/ui.lua (+14/-1)
src/logic/campaign_visibility.cc (+3/-3)
src/ui_fsmenu/campaign_select.cc (+76/-35)
src/ui_fsmenu/campaign_select.h (+4/-2)
src/wlapplication.cc (+29/-10)
src/wlapplication.h (+1/-0)
txts/developers (+7/-0)
| Changed in widelands (Debian): | |
| status: | Unknown → Confirmed |
| Changed in widelands: | |
| assignee: | nobody → wl-zocker (wl-zocker) |
Revision history for this message
| SirVer (sirver) wrote | #1 |
| Changed in widelands: | |
| status: | Triaged → Incomplete |
| Changed in widelands: | |
| status: | Incomplete → In Progress |
| Changed in widelands: | |
| status: | In Progress → Fix Committed |
| Changed in widelands: | |
| assignee: | wl-zocker (wl-zocker) → nobody |
| Changed in widelands: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| GunChleoc (gunchleoc) wrote | #2 |
| Changed in widelands (Debian): | |
| status: | Confirmed → Fix Released |
To post a comment you must log in.
Setting to incomplete for bug sweeping.