When message_notify_send_message is called and can't find the plugin for the specified notifier, an exception is thrown, but a misplaced parenthesis causes a fatal PHP error:

Wrong parameters for Exception([string $exception [, long $code [, Exception $previous = NULL]]]) in sites/all/modules/contrib/message_notify/message_notify.module on line 24

The misplaced parenthesis means the format_string $args parameter is being passed to the MessageNotifyException constructor instead of format_args. A patch is attached to resolve this error.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dwillems’s picture

Issue summary: View changes
amitaibu’s picture

Status: Active » Needs review

correct status

Status: Needs review » Needs work

The last submitted patch, message_notify-message_notify_send_message.patch, failed testing.

dwillems’s picture

Status: Needs work » Needs review
FileSize
742 bytes

Correctd paths in patch file.

Status: Needs review » Needs work

The last submitted patch, 4: message_notify-message_notify_send_message-2148987.patch, failed testing.

bluegeek9’s picture

Status: Needs work » Closed (outdated)