#43782: Lua: remove_extra() not working
Open Date: 2022-02-04 22:08
Last Update: 2022-02-04 22:45
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/43782
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=43782
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-02-04 22:45 Updated by: cazfi
* Component Update from (None) to General
Comment:
so maybe remove_extra() got broken.
I think it has been broken on S3_0 (for resources) either always or for a long time (might have gotten broken when their reappearance in case of terrain changing back to one supporting them was fixed) - and in earlier branches resources were not even considered extras.
Fix *seems* simple, but kind of risky in that it might break something else, and more critical - this might be something that we don't want to push in before 3.0.0 even with RC2 route, but want to expose to testing for entire 3.0.1 cycle. While it's annoying that it breaks that ruleset mentioned in the forums, it's not a regression either (it was not possible to remove resources in 2.6)
I'll submit the fix soon anyway, so we do have it available whatever we decide.
---------------------------------------------------------------------
Ticket Status:
Reporter: cazfi
Owner: (None)
Type: Bugs
Status: Open
Priority: 5 - Medium
MileStone: (None)
Component: General
Severity: 5 - Medium
Resolution: None
---------------------------------------------------------------------
Ticket details:
ddeanbrown in #43584:
Tested with RC1, assert errors are gone, but remove_extra() is not working.
Also tried connecting to "turn_begin" signal and running on turn 1, instead of connecting to "map_generated" signal, and saw same problem, so maybe remove_extra() got broken.
--
Ticket information of Freeciv project
Freeciv Project is hosted on OSDN
Project URL: https://osdn.net/projects/freeciv/
OSDN: https://osdn.net
URL for this Ticket:
https://osdn.net/projects/freeciv/ticket/43782
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=43782