If a user is deleted, currently its subscriptions are not deleted.
And so we've got a fatal error when generating a log, because we call $subscription->getOwner()->id()

Let's fix that

CommentFileSizeAuthor
#6 3095633-6.patch12.08 KBflocondetoile
#4 3095633-4.patch6.51 KBflocondetoile
#3 3095633-3_failing_test.patch3.41 KBflocondetoile

Comments

flocondetoile created an issue. See original summary.

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Issue summary: View changes
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Status: Active » Needs review
StatusFileSize
new 3095633-3_failing_test.patch3.41 KB

a failing test

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
StatusFileSize
new 3095633-4.patch6.51 KB

The fix. Need an hook_update_N().

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Title: Subscriptions should be deleted if owner is deleted » Subscriptions and Logs should be deleted if its owner is deleted
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
StatusFileSize
new 3095633-6.patch12.08 KB

With the hook_update_N. Also fix the unrelated and randomized fail (because of email sent order).

  • flocondetoile committed e906a05 on 8.x-1.x 
    Issue #3095633 by flocondetoile: Subscriptions and Logs should be...
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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