hello Everyone,

I don't know if i miss something or, but in the template workflow interface we can save notification for roles. I try it and it doesn't work. When you look into code there nothing to do that just a comment on line 146 of maestro_notification.class.php
$query->condition('a.notify_type', MaestroAssignmentTypes::USER, '='); //@TODO: add support for ROLE and GROUP types

CommentFileSizeAuthor
#1 role-notifications-1195216-1.patch2.24 KBbhauff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bhauff’s picture

Status: Needs work » Needs review
FileSize
2.24 KB

I have worked on implementing support for sending notifications to all users in the roles selected in the workflow task editor. This patch adds another query to get rows with a notify_type of MaestroAssignmentTypes::ROLE. It then does one more query per returned row to grab the users of the role. Finally, it appends the users with appropriate maestro variables to the original result. I am sure there are more optimized solutions, however, this patch has enabled our Drupal site to utilize this advertised feature of Maestro.

The patch was applied against the master branch, but was also tested against the 7.x-1.1 release.

wimds’s picture

I get no results when trying to notify (email) my OG members when a task is assigned to their OG.
Is this patch also working for OG?

_randy’s picture

Ok, I see -- the issue for OG notifications was posted recently as well. Will look in to it along with the role issue above.