onEntityDelete and onEntityRevisionDelete do not work properly. This patch seems to fix some bugs for me. When deleting a field onFieldDelete does not seems to (always?) be called - onEntityDelete is, but throws a fatal error as it expects the entity ID to always be an integer. Also added a dependency in the install file that fixed an error when installing this with drush.

CommentFileSizeAuthor
#5 editor_note-fatal-errors-1.patch3.94 KBpatrickharris
#3 editor_note-fatal-errors-1.patch3.66 KBpatrickharris
#2 editor_note-fatal-errors-1.patch3.27 KBpatrickharris
editor_note-fatal-errors-1.patch3.04 KBpatrickharris

Comments

patrickharris created an issue. See original summary.

patrickharris’s picture

patrickharris commented
Issue summary: View changes
StatusFileSize
new editor_note-fatal-errors-1.patch3.27 KB
patrickharris’s picture

patrickharris commented
StatusFileSize
new editor_note-fatal-errors-1.patch3.66 KB
patrickharris’s picture

patrickharris commented
Issue summary: View changes
patrickharris’s picture

patrickharris commented
StatusFileSize
new editor_note-fatal-errors-1.patch3.94 KB
Rostislav Sergeenkov’s picture

Rostislav Sergeenkov commented
Assigned: Unassigned » veranika_biblis

  • veranika_biblis committed 4dcbc2a on 8.x-1.x 
    Issue #3096726 by patrickharris, Rostislav Sergeenkov: onEntityDelete...
veranika_biblis’s picture

veranika_biblis commented
Status: Active » Needs review
Rostislav Sergeenkov’s picture

Rostislav Sergeenkov commented
Rostislav Sergeenkov’s picture

Rostislav Sergeenkov commented
Status: Needs review » Fixed
Rostislav Sergeenkov’s picture

Rostislav Sergeenkov commented
Status: Fixed » Closed (fixed)