[kovarex] [1.1.19] Undoing deconstruction of building with modules creates extra module request

[Menelaj]

Steps to reproduce:

1. Create a building with some modules in it.
2. Mark the building for deconstruction by deconstruction planner (even with no roboports nearby).
3. Before the building is deconstructed, undo deconstruction by Ctrl+Z.

Expected result:
building with the initial number of modules and with no deconstruction marks or module insertion requests (or with the same request as before the deconstruction, if there was such a request).

Actual result:
an unnecessary module insertion request is created (the number of requested modules is equal to the number of modules in the building).

Moreover, there is even stranger behavior, depending, apparently, on the coordinates of the building: repeating steps 2 and 3 may or may not create more insertion requests - up to tens and hundreds in total. See screenshot: all assemblers have two real modules; for all assemblers, steps 2 and 3 are repeated several times. As a result, for some assemblers, there are requests for only two additional modules; for the others, requests for tens of modules are created.

Interestingly, canceling the deconstruction mark with "Shift + deconstruction planner" behaves differently: extra requests are not created, but all previous requests (even appropriate ones) are erased.

May be connected to:
viewtopic.php?f=48&t=93000 - it seems that "item request proxy not being updated when inserting modules manually" is not the case
viewtopic.php?f=47&t=92930 - very similar steps to reproduce
viewtopic.php?f=11&t=83571 - maybe the bug was introduced somewhere around

[kovarex]

Thanks for the report, it is now fixed for the next release.