Fixed [put_to_recall_list] (bug #24390)

put_recall_unit() already deals with erasing it from the map, rendering
this erase_unit() call redundant and harmful.

RELEASE_NOTES

@@ -49,7 +49,6 @@ KNOWN BUGS
 
 [list]
 [*]The MP server has trouble with Local player types in campaigns (bug [bug]21965[/bug]). We have decided to postpone dealing with this. In the meantime, you might try assigning such sides to the host, or running multiple instances of Wesnoth.
-[*][put_to_recall_list] doesn't put the unit to the recall list.
 [/list]
 [*]Wesnoth does not exist on the panel (bug [bug]24202[/bug]).
 [*]"Name of Game" bleeds onto "Player/Type" label when leaving/entering the app (bug [bug]24437[/bug]).

changelog

@@ -64,6 +64,7 @@ Version 1.13.2+dev:
    * add support for relative dirs in wesnoth.dofile/require
    * Added name= and write_name= attributes in [item]
    * Added description_alignment= key to [campaign]
+   * Fixed [put_to_recall_list] not working currently (bug #24390)
  * Miscellaneous and bug fixes:
    * Fix the new log code on Windows to actually use Unicode-aware functions
      in a couple of places so Wesnoth does not quit on startup when trying to

data/lua/wml-tags.lua

@@ -1368,7 +1368,6 @@ function wml_actions.put_to_recall_list(cfg)
 			unit.status.slowed = false
 		end
 		wesnoth.put_recall_unit(unit, unit.side)
-		wesnoth.erase_unit(unit)
 	end
 end