ad510/plausible-deniability

moving unit in past sometimes fails to make new path

ad510 opened this issue · 2 comments

It fails when moving a non-live unit immediately when it's made (because its parent is ambiguous), or moving a non-live unit when resources are negative. If the path to be moved already isn't live, a better approach is removing later segments and moves then moving the original path, like pre-stacking versions.

Moving a non-live unit when resources are negative seems to be fixed in the current master. I think the fix was introduced in e068429.

eabd455 kind of fixes moving a non-live unit immediately when it's made by not letting you move the unit (without throwing an error). I'll close the issue for now, but a better long term fix would be the one proposed in the original post and I can reopen the issue if I decide to implement that.