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.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to