[4.2] Workflow Notification no recipient (#37747)

* [4.1] Workflow Notification no recipient * cs * Phase 1 convert BRANCH to PSR-12 * Phase 2 convert BRANCH to PSR-12
joomla · Jul 3, 2022 · 81da883 · 81da883
1 parent fd3dbc9
commit 81da883
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/administrator/language/en-GB/plg_workflow_notification.ini b/administrator/language/en-GB/plg_workflow_notification.ini
@@ -11,6 +11,7 @@ PLG_WORKFLOW_NOTIFICATION_ADDTEXT_DESC="This text will be sent: Title [title], c
 PLG_WORKFLOW_NOTIFICATION_ADDTEXT_LABEL="Additional Message Text"
 PLG_WORKFLOW_NOTIFICATION_ON_TRANSITION_MSG="Title: %1$s. Transition \"%2$s\" performed by %3$s. New state: %4$s."
 PLG_WORKFLOW_NOTIFICATION_ON_TRANSITION_SUBJECT="The status of \"%s\" has been changed"
+PLG_WORKFLOW_NOTIFICATION_NO_RECEIVER="No notifications sent as there are no users to send this message to."
 PLG_WORKFLOW_NOTIFICATION_NO_TITLE="Unknown title"
 PLG_WORKFLOW_NOTIFICATION_RECEIVERS_LABEL="Users"
 PLG_WORKFLOW_NOTIFICATION_SENDMAIL_LABEL="Send Notification"

diff --git a/plugins/workflow/notification/notification.php b/plugins/workflow/notification/notification.php
@@ -160,6 +160,8 @@ public function onWorkflowAfterTransition(WorkflowTransitionEvent $event)
 
         // If there are no receivers, stop here
         if (empty($userIds)) {
+            $this->app->enqueueMessage(Text::_('PLG_WORKFLOW_NOTIFICATION_NO_RECEIVER'), 'error');
+
             return;
         }