When I add a user to the private group, he receives a blank email. This also happens when I remove a user from that private group.
The corresponding lines: 367 to 421 in profiles/drupal_commons/modules/features/commons_core/commons_core.module

Comments

sergiu.popa’s picture

sergiu.popa commented

Bug found: Line 387 should be 'commons_core', instead of 'commons',

Here the entire function:

drupal_mail('commons_core', "og $op", $user->mail, language_default(), array('group' => $group));
mstef’s picture

mstef commented
Status: Active » Closed (fixed)

Excellent find! I completely missed this and assume it was a problem created when we merged in the commons.module into the commons_core feature.

Thanks!

Committed and will be included in the 1.4 release going out in a few hours.

mstef’s picture

mstef commented

https://github.com/acquia/commons/commit/f98f7ffc41d37c03ff37376b2fecbe3...

t-readyroc’s picture

t-readyroc commented
Version: » 6.x-2.10
Category: bug » support
Priority: Major » Normal
Status: Closed (fixed) » Active

Sorry to reopen this issue, but after ~30mins of searching around, I still can't find where to edit this notification message (or whether it's even possible). Can either of you point me in the right direction?

Andrea C’s picture

Andrea C commented

Have you check in admin/og/og ?

t-readyroc’s picture

t-readyroc commented

Yep, I have. Not to be found there. You can find templates on invites, request denied, request membership, & a couple more there, but not this particular notification message.

mstef’s picture

mstef commented

Last I checked, which was back in the early 6.x-1.x days, it was a hard-coded message implemented in hook_mail() within commons_core (so, no way to edit from the UI). That might have changed since then though...

lsolesen’s picture

lsolesen commented
Issue summary: View changes
Status: Active » Closed (won't fix)

Closing out old issues. Please reopen if still relevant.

lsolesen’s picture

lsolesen commented
Status: Closed (won't fix) » Active

Closed by mistake. This might still be a valid issue. Please close it with the solution, if you already solved it.