Database editor's views expect to have the "id" key available for removed items. However, spinedb_api deleted the key when rolling back changes. With this PR, we don't delete the id anymore but use a new 'is_id_valid' flag to signal that the id has gone out of scope.
Fixes spine-tools/Spine-Toolbox#2291
Checklist before merging
[x] Unit tests have been added/updated accordingly
Database editor's views expect to have the "id" key available for removed items. However,
spinedb_apideleted the key when rolling back changes. With this PR, we don't delete the id anymore but use a new 'is_id_valid' flag to signal that the id has gone out of scope.Fixes spine-tools/Spine-Toolbox#2291
Checklist before merging