I'm not sure if I'm the only one having this issue, but before, I had workbench set up to send emails out to specific people assigned to particular sections. In my organization, every section had their assigned editor. When they would update content on the site, an email is sent out to that particular editor. Now it appears that that feature is no longer valid. I have discovered that emails are only sent out by role only. Meaning a role has to be assigned to a particular section and everyone who is that role will now get an email. I couldn't divide and conquer like before. Is there any reason why this feature was taken out of Workbench Email?
Comments
Comment #2
frosev CreditAttribution: frosev at Appnovation commentedThanks for the report. We didn't take any functionality out, so this must be a bug. You're using workbench_access to assign users to sections I imagine? Would it be possible to share your configuration for workbench_email and workbench_access? I want to replicate it locally to try to debug it locally.
Thank you.
Comment #3
frosev CreditAttribution: frosev at Appnovation commentedComment #4
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedComment #5
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedFrosev
Yeah I can get you my configuration. What exactly do you need to see?
I am set up on Workbench Email and I have editors getting auto notifications. I assigned editors to different sections. And they were getting emails for their specific section with no problem. One day the emails just stopped. I couldn't figure out what it was.
It wasn't until I assigned the "Editor" role to all of sections did they start receiving emails again. Please see all my attached images.
Comment #6
frosev CreditAttribution: frosev at Appnovation commentedThat's everything I need. Thank you! I'm going to replicate this locally over the weekend, and report back. Thank you again!
Comment #7
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedfrosev,
How is everything going? Just wanted to check up to see if you were able to recreate the problem.
Best,
TG
Comment #8
frosev CreditAttribution: frosev at Appnovation commentedHi TG,
I'm still working on it. Unfortunately I didn't get much time over the weekend to try and reproduce the issue. I'm going to give it a go later tonight. I'll let you k now if I have any issues reproducing the error. Thank you.
Comment #9
frosev CreditAttribution: frosev at Appnovation commentedHi TG,
I had a few extra minutes to spare and I think I've found the issue. The code added from the following issue: https://www.drupal.org/node/2532198 is causing the issue. I'm going to try and get that fixed over the weekend and provide a patch. Once that's tested I can create a new release.
Comment #10
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedFrosev,
Thank you! Looking forward to it.
Comment #11
frosev CreditAttribution: frosev at Appnovation commentedTG,
I'm whipped up a quick patch. The previous issue was that a user that had multiple roles was getting multiple emails, even though the section was only assigned to a single role that the user belonged to. So I decided to short circuit the check for user's emails if their roles didn't match the sections, however that was short circuiting for when workbench_email was looking for single editors (when a given user is assigned to a section). I've moved the short circuit to AFTER workbench_email has accumulated all the editors.
I've attached a patch, which works for both this issue and the issue previously mentioned (https://www.drupal.org/node/2532198). Let me know if it works for your case.
Comment #12
frosev CreditAttribution: frosev at Appnovation commentedHi TG,
I was wondering if you had a chance to test the patch?
Comment #13
frosev CreditAttribution: frosev at Appnovation commentedComment #15
frosev CreditAttribution: frosev at Appnovation commentedComment #16
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedSorry,
Haven't gotten around to patching. Ran into another issue with mail. Will try this week.
Best,
TG
Comment #17
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedfrosev,
I finally got around to patching however the patch didn't take. Maybe I'm doing something wrong. Take a look at my recent attachment and let me know what you think.
Best,
V
Comment #18
frosev CreditAttribution: frosev at Appnovation commentedTG,
I ran the patch command you did on the 7.x-3.11 version of the module and it applied properly. Could you try the patch command with the
--ignore-whitespace
option. It could be a mismatch in the type of line endings we're each using.Comment #19
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedStill getting this error
Comment #20
frosev CreditAttribution: frosev at Appnovation commentedHmm, it's not ignoring the line endings. I've attached a patch where I've ignored the line endings. Could you see if that works?
Comment #21
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedStill getting this...I'm probably doing something wrong. I don't understand why it wouldn't work.
Comment #22
frosev CreditAttribution: frosev at Appnovation commentedCould you try applying the patch manually? The patch moves a few lines further down the function.
Comment #23
TheGentleman CreditAttribution: TheGentleman as a volunteer commentedI patched it in manually. I'm going to see how it works out over the next week. I'll get back to you.
Best,
TG
Comment #24
frosev CreditAttribution: frosev at Appnovation commentedThanks TG. Let me know how it goes, and happy holidays!
Comment #25
frosev CreditAttribution: frosev at Appnovation commentedHi TG,
Were you able to test the patch?
Comment #26
frosev CreditAttribution: frosev at Appnovation commentedHi TG,
Following up to see if the patch worked for you. I was thinking about creating a release for this next week.
Comment #28
frosev CreditAttribution: frosev at Appnovation commentedA release has been pushed to 7.12