Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juampynr created an issue. See original summary.

Sagar Ramgade’s picture

Status: Active » Needs review
FileSize
1.25 KB

Yes, patch attached implements the hook_ENTITY_TYPE_delete().

juampynr’s picture

Status: Needs review » Needs work

Looks good. Mind adding test coverage for this? The tests already create a user so it would be a matter of deleting it and then checking that the set of consumers are gone.

Sagar Ramgade’s picture

Status: Needs work » Needs review
FileSize
2.23 KB

Patch with tests, Looks good?

  • juampynr committed 9c4b730 on
    Issue #2615772 by juampy, Sagar Ramgade: Delete consumers when a user is...
juampynr’s picture

Issue summary: View changes
Status: Needs review » Fixed

Not really, I made my own implementation because:

* There were a few legacy statements related with tables from the Drupal 7 version that don't exist in the 8 one.
* The tests's assertion was not passing.

Thanks anyway!

Status: Fixed » Closed (fixed)

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