[wesnoth-commits] [wesnoth/wesnoth] eacf26: Fix [store_unit]'s stored coordinates for recalls

[Steve Cotton]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: eacf26a2a8e24313e5c42befd42b71409486b020
      https://github.com/wesnoth/wesnoth/commit/eacf26a2a8e24313e5c42befd42b71409486b020
  Author: Steve Cotton <steve at octalot.co.uk>
  Date:   2022-01-12 (Wed, 12 Jan 2022)

  Changed paths:
    M data/lua/wml-tags.lua
    A data/test/scenarios/put_to_recall_and_modify.cfg
    M data/test/scenarios/test_modify_unit.cfg
    M wml_test_schedule

  Log Message:
  -----------
  Fix [store_unit]'s stored coordinates for recalls

Units on the recall list might have x,y coordinates that are on the map, which
therefore need to be replaced with "recall,recall" within [store_unit]. The
existing code created a temporary variable, changed the coordinates, and then
returned the unchanged original instead of the temporary.

Add a new test that `[put_to_recall_list]` followed by `[modify_unit]`
doesn't move the unit back to the map.

(cherry picked from commit 096d8aba1474e9c6583d240e7eedbcd66957f327)