widelands-dev team mailing list archive

[GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>]

Review: Needs Fixing

I have proofread and fixed some stuff and then playtested the tutorials. We still have the following issues to resolve:

Economy: function _try_add_objective doesn't exist. What was it supposed to do?

Basic control:

1. We should bzr move campaigns/tutorial01.wmf to Basic Control.

2. Objective "connect the quarry to the headquarters" is never achieved. Since we don't have access to roads yet, you could check for the quarry being constructed.

3. I accidentally built a lumberjack instead of the second quarry. Tutorial tore it down, like it's supposed to. Then the call to census_and_statistics(cs.fields[1]) failed

[/home/bratzbert/sources/widelands/new-tutorials/src/scripting/lua_errors.h:29] BaseImmovable no longer exists!
stack traceback:
	[string "map:scripting/mission_thread.lua"]:238: in function 'build_a_quarry'
	[string "map:scripting/mission_thread.lua"]:143: in function 'learn_to_move'
	[string "map:scripting/mission_thread.lua"]:106: in function 'build_lumberjack'
	[string "map:scripting/mission_thread.lua"]:25: in function <[string "map:scripting/mission_thread.lua"]:5>

I have yet to look at the campaign.
-- 
https://code.launchpad.net/~widelands-dev/widelands/new-tutorials/+merge/238682
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/new-tutorials.