The attached patch should be applied with patch -p1.

It provides a new submodule for sharedemail that leverages sharedemail specifically for a site making use of the Subuser module.

From the README:

  1. Subusers use the same email address as their parent account.
  2. Only Subuser accounts may use an email address otherwise in use. (Accept subuser parent accounts sharing with their own subusers.)
  3. Subusers may not change their email address.

This patch is dependent on the hook introduced in #895924: Allow PHP definitions of users that cannot share email and the new Subuser API functions introduced in #14 of #459694: Limit number of subusers

CommentFileSizeAuthor
#5 sharedemail.896520-5.patch4.5 KBbtmash
#1 sharedemail.896520-2.patch4.5 KBGrayside
sharedemail.subuser_integration.patch4.54 KBGrayside

Comments

Grayside’s picture

Grayside commented
StatusFileSize
new sharedemail.896520-2.patch4.5 KB

Updated for changes in #895924: Allow PHP definitions of users that cannot share email. Also, corrected flawed behavior surrounding bad argument order in array_diff().

christefano’s picture

christefano commented

This is great! We're at an LA Drupal meetup and this patch applied successfully! I am really Chrisefano.

Grayside’s picture

Grayside commented

That's not very believable. You failed to spell your name correctly.

If the patch applied, and worked correctly, maybe someone could mark this RTBC ;P

christefano’s picture

christefano commented

Pat says, "You realize there's beer at these meetups, right?" :)

btmash’s picture

btmash commented
StatusFileSize
new sharedemail.896520-5.patch4.5 KB

The code looks good to me. Attaching a patch where I removed some of the whitespacing (superficial changes). So right now, its +1 Reviewed by community. It needs testing at this point.

christefano’s picture

christefano commented
Version: 6.x-1.x-dev » 6.x-1.4
Assigned: Unassigned » christefano
Status: Needs review » Fixed
Issue tags: +Subuser

Thanks, all! This has been committed and will appear in the 6.x-1.4 release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

dpantele’s picture

dpantele commented
Status: Closed (fixed) » Needs work

You haven't commited the hook_sharedemail_policy_info() functionality which is mentioned in http://drupal.org/node/895924

mrpauldriver’s picture

mrpauldriver commented

Has subuser integration found it's way in to the Drupal 7 dev version?

baikho’s picture

baikho commented
Issue summary: View changes
Status: Needs work » Closed (outdated)