sms_user_delete should be renamed because it conflicts with hook_entity_type_delete.

The sms_user_delete function is defined by sms_user, however the hook responds to the main sms.module.

If a user entity is deleted, this hook is invoked by EntityStorageBase.php:374.

This bug does not affect Drupal 7 versions of SMS Framework, as the function was previously known as sms_user_delete_user_info().

Comments

dpi created an issue. See original summary.

almaudoh’s picture

This was fixed in #1915022: No phone number validation on user registration form for D7 but it appears some parts of that fix didn't get ported to D8. I discover now that the final patch in that issue is smaller than what was committed. The actual change is at http://cgit.drupalcode.org/smsframework/commit/?id=5906460 and we can fix this issue by porting that change to D8.

dpi’s picture

Status: Active » Fixed

Committed in 81d7e61

Status: Fixed » Closed (fixed)

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