wesnoth.synchronize_selection

[lipk]

This works:

Code: Select all

      helper = wesnoth.require "lua/helper.lua"
	function f()
	  locations = helper.get_variable_array("some_array")
	  return locations
	end
      
      helper.set_variable_array("some_array", f())

This doesn't work (gives an empty array):

Code: Select all

      helper = wesnoth.require "lua/helper.lua"
      result = wesnoth.synchronize_choice(
	function ()
	  locations = helper.get_variable_array("some_array")
	  return locations
	end
      )
      
      helper.set_variable_array("some_array", result)

All on Wesnoth 1.10.5, either local or networked multiplayer.
Am I doing something wrong or is synchronize_selection broken?

[Elvish_Hunter]

Right now, I don't see anything wrong with the code. Are you attempting to use synchronize_choice in a prestart or start event? If yes, in past I noticed that such function does not work with these two events.

[lipk]

The code is triggered by a menu item which itself is added in a start event. On an additional note, I could work around the problem by coding the contents of "some_array" into a string and returning a single-field table in the anonymous function.

[Anonymissimus]

synch_choice doesn't work earlier than turn 1, yes.

The return value of the inner function needs to be of type wml table. Yours should be of the form { a, b, c, ... } instead. (For finding out why some table isn't also a wml table, I suggest dbms from the WLP.)
You could use a modified helper function instead

Code: Select all

function helper.get_wml_variable_array(var)
	local result = {}
	for i = 1, wesnoth.get_variable(var .. ".length") do
		table. insert(result, { var, wesnoth.get_variable(string.format("%s[%d]", var, i - 1)) })
	end
	return result
end

and an analog helper.set_wml_variable_array(var) (untested), so that the form of the passed-through table should be a wml table
{ { "some_array", a }, { "some_array", b }, ... }
where a, b and so on are the assumed wml tables representing a single element/container in the wml variable array some_array.

[lipk]

Ah-ha! I see now. Thankies.