Remove backlinks when deleting an objects containing a LnkSet #5398
Conversation
|
@jsflax This should fix the issue with Set. |
| @@ -810,7 +810,7 @@ size_t Cluster::erase(ObjKey key, CascadeState& state) | |||
| dict.remove_backlinks(state); | |||
| } | |||
| } | |||
| else if (col_type == col_type_LinkList) { | |||
| else if (col_type == col_type_LinkList || col_type == col_type_Link) { | |||
There was a problem hiding this comment.
Clearly this works, but what's the difference between col_type_Link and col_type_TypedLink
There was a problem hiding this comment.
We don't actually support col_type_TypedLink yet, so that is dead code right now. TypedLink is currently only used internally as part of Mixed. The idea is that a column of type col_type_Link or col_type_LinkList can only link to a single static destination type, while a TypedLink has the flexibility to point to any type dynamically because it stores the destination object key and the destination object type. However, the issue in this PR is that LnkLst has type col_type_LinkList while LnkSet has type col_type_Link with the set attribute.
What, How & Why?
Fixes #5387
☑️ ToDos