[Solved] Attempt to index global 'game' (a nil value)

[BrokenScience]

I recently moved some code from control to a new file to reduce clutter. Unfortunately it no longer works because of an error I did not get before. The file is required in control.lua and is called through on_init and on_configuration_changed, both of which do have access to the game table (I checked). Could the movement from control.lua mean I no longer have access to the game table?

I included the new file (mind the debug prints I commented out) and the lines left in control are:

Code: Select all

require("code.ordering")

if not item_order then item_order = {} end
if not frame_name then frame_name = 0 end

script.on_init(build_orderer())

script.on_configuration_changed(build_orderer())

I do hope it's not a parenthesis problem, but it probably is. That would be embarrassing.

[Jap2.0]

Don't know anything about lua, but I've done a bit of programming, so I know what mismatched parenthesis are. The only things I found were:

Code: Select all

function build_orderer()--debuggery)

Code: Select all

function order(items, orderer)--, debuggery)

Code: Select all

if can_be_ordered(orderer, orderer[items[i]]) then--, debuggery) then

which both have extra parenthesis, but it looks like -- denotes comments, so they wouldn't be it (not saying that it doesn't irritate me...)

[Mooncat]

Change

Code: Select all

script.on_init(build_orderer())

script.on_configuration_changed(build_orderer())

to

Code: Select all

script.on_init(build_orderer)

script.on_configuration_changed(build_orderer)

( Remove the pair of () )

Without (), you are registering the delegate of build_orderer as the event handler, which is the intended and usual way to do so.
But with (), you are executing build_orderer() and registering its result as the event handler. It leads to at least 2 problems:
1. build_orderer() is executed once control.lua is loaded, even before on_init is invoked. "game" is not ready and hence it is nil.
2. build_orderer() does not return anything. Nothing will be registered.

(I know this, because I have made the same mistake before )

[BrokenScience]

Yep, it was the parentheses I included as part of the build_order() function.

That is a small detail that creates an incredibly complex problem.