fix: #3607949 Skip subscribers without an email address

David Valdezrequested to merge
issue/comment_notify-3607949:3607949-skip-empty-recipient into 8.x-1.x

Summary

  • Skips registered comment subscribers when no recipient email can be resolved.
  • Adds functional regression coverage for replies to subscribers without email addresses.
  • Adds the required process-isolation attribute to the modified functional test class.

Testing

  • Reproduced the PhpMail::doMail() null-recipient TypeError on Drupal 11.4.4 before the fix.
  • Confirmed the reply request completes successfully after the fix.
  • Confirmed the regression test fails without the guard and passes with it.
  • Full comment_notify suite: 13 tests and 359 assertions passed.
  • Drupal coding standards reported zero errors; PHP syntax checks passed.

Issue: https://www.drupal.org/project/comment_notify/issues/3607949

AI-Generated: Yes (Used OpenAI Codex to reproduce the issue, review the patch, add regression coverage, and validate the code and tests.)

Merge request reports