During custom development I find some incorrect code in postDelete().
As I see current logic use label of current config page type instead of id. I think this logic not clear because site can have multiple config page with the same label and delete them all during deletion of one - it's not good behaviour.

CommentFileSizeAuthor
#2 3117701-02-better-postdelete.patch816 bytesstomusic

Comments

stomusic created an issue. See original summary.

stomusic’s picture

Status: Active » Needs review
StatusFileSize
new816 bytes
shumer’s picture

Assigned: Unassigned » shumer

  • shumer committed e9901d0 on 8.x-2.x authored by stomusic
    Issue #3117701 by stomusic: Better postDelete()
    
shumer’s picture

Status: Needs review » Fixed

  • shumer committed 01da1ac on 8.x-2.x
    Issue #3117701 by stomusic, shumer: Better postDelete()
    

  • shumer committed 6955991 on 8.x-2.x
    Issue #3117701 by stomusic, shumer: Better postDelete()
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.