Move logic from sms_entity_delete() and _sms_entity_postsave() to the phone number provider.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dpi created an issue. See original summary.

dpi’s picture

PR: https://github.com/dpi/smsframework/pull/41

Added two new methods to phone number provider service.

  • Moved contents of sms_entity_delete() to PhoneNumberProvider::deletePhoneVerificationByEntity().
  • Moved contents of _sms_entity_postsave() to PhoneNumberProvider::updatePhoneVerificationByEntity().

No functional changes, no new tests added.

  • dpi committed 7ec9953 on 8.x-1.x
    Issue #2731399 by dpi: Refactored phone number logic out of hooks
    
    Fixed...
dpi’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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