 |
This indeed happens in older versions as well, only checked the old 2.0.3 release and it's doing that too. |
|
|
It appears that this behaviour can only be observed when doing the colour change through the Light Inspector. It must be something related to the way the entity properties are set by the Inspector, which triggers this odd behaviour in the UndoSystem. |
|
|
Ok, got the problem narrowed down to this: when undoing an operation where entity key values are added/removed *and* keyvalues of the same entity are changed, the whole EntityKeyValues list is imported back into the Entity class. In the Entity::importState() method, all the key values are removed and re-inserted from the snapshot, which is when they are disconnected and re-connected to the UndoSystem. After the call to UndoSystem::getStateSaver(), the KeyValue ends up with a state saver pointing to a nullptr Stack, that's why the _color keyvalue can't save its state before it's importing the old value. So when redoing the step, the previous _color information is lost as it hasn't been saved to the UndoSystem - because the keyvalue has been newly connected to the undosystem when the parent entity had been importing its own keyvalue pair list from the snapshot. # Some chronological notes I made during debugging: Undo: setLightProperties # EntityKeyValues are imported from UndoSystem Saving undoable's state of observed object 'EntityKeyValues' Exporting state of observed object 'EntityKeyValues' # entity state is saved now Importing state of observed object 'EntityKeyValues' Importing state into entity light_1, got 10 key values # Stack of _color keyvlue points to RedoStack Disconnecting key value from undo system _color [000000000EE67690] Disconnecting KeyValue with value '0.000 0.000 0.000' from undosystem # _color Keyvalue is disconnected from undo systemConnecting new key value to undo system _color [000000000EE67690] Connecting KeyValue with value '0.000 0.000 0.000' to undosystem # _color keyvalue is connected, but its stack points to nullptr now, due to new getStateSaver() call Saving undoable's state of observed object 'KeyValue' # ObservedUndoable cannot save its state due to stack == nullptr, old value not saved to stackImporting state of observed object 'KeyValue' [000000000EE67690] Importing value from undo system 1.000 0.000 0.000 (old value: 0.000 0.000 0.000) # Redo will now import the entity key value list only, the old value of the _color key is lost. |
|
|
Fix committed in bf034c7. |
- Anonymous
