Lack of interest Move Bookmark postDelete method to behavior

This suggestion has been closed automatically because it did not receive enough votes over an extended period of time. If you wish to see this, please search for an open suggestion and, if you don't find any, post a new one.
Kruzya

Kruzya

Well-known member
Currently, for correct work bookmark on custom entities, developer should call method like _postDeleteBookmarks() in _postDelete() in own entity. For reactions, no one additional method don't should be called, because for reactions in structure added additional behavior XF:Reactable.

I suggest implement behavior XF:Bookmarkable with same logic: remove bookmarks when entity is deleted. Because this is "breaking change", method _postDeleteBookmarks() should be exists, but with empty body.
 
Upvote 1
This suggestion has been closed. Votes are no longer accepted.

Similar threads

V
Fixed Additional queries behaviour on resource updates due to reactions and bookmarks
Replies
1
Views
1K
XF Bug Bot
XF Bug Bot
RobinHood
Move Bookmarks Menu - Add Bookmarks Icon to user nav menu and remove tab
Replies
8
Views
1K
hakyangler
hakyangler
AndyB
  • Suggestion Suggestion
Implemented Please move Bookmarks to sub-navigation menu
2
Replies
33
Views
3K
ibnesayeed
ibnesayeed
Back
Top Bottom