Just wondering if there was any planning for a D7 release?

CommentFileSizeAuthor
#3 drupal7-1050822.patch7.49 KBwesayso

Comments

kiliweb’s picture

It's impossible to make a Drupal 7 version of this module, since there is no hook_user_validate.
The validation is only handle by the user module of D7.

I suggest you to edit user.module, and comment the line 1213

form_set_error('mail', t('The e-mail address %email is already taken.', array('%email' => $form_state['values']['mail'])));

-
http://www.kiliweb.fr

Rj-dupe-1’s picture

Thanks.

wesayso’s picture

Title: D7 Plans? » Here's a patch for Drupal 7
Status: Active » Needs review
StatusFileSize
new7.49 KB

Here's a patch to convert this module from Drupal 6 to Drupal 7.

Drupal 7 dropped hook_user() and doesn't have a hook_user_validation(), so I used hook_form_FORM_ID_alter() to change the validation handler for the user registration and profile forms. The new validation handler munges the email address if necessary, calls the user module's default validation handler, then unmunges the email address before returning.

wesayso’s picture

Title: Here's a patch for Drupal 7 » Drupal 7
christefano’s picture

Assigned: Unassigned » christefano

Thanks, wesayso! I'll review this in the next few days when I have the time.

christefano’s picture

Category: feature » task
geerlingguy’s picture

Subscribe. I'm using the same method in a custom module on my site, but I love the simplicity of this module, and would love to see a D7 version.

christefano’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Closed (fixed)

A 7.x-1.x version based on #3 has just been released. It will appear in the next few minutes on the project page and at http://drupal.org/node/1191196

geerlingguy’s picture

Nice. I should comment on Drupal 7 port issues more often. This exact thing has happened twice in three days!

geerlingguy’s picture

Status: Closed (fixed) » Active

It looks like the 7.x version isn't showing on the project main page - can you get that to show up? Until that happens, this issue should probably remain open, so those looking to use the D7 version have a place to get it :)

christefano’s picture

Sure, let's leave this open. I'm not using Shared Email with Drupal 7 yet.

Is anyone who's using Drupal 7 able to report anything about the 7.x-1.x-dev version?

michelle’s picture

I just installed the dev and it seems to be working fine.

Thanks!

Michelle

christefano’s picture

Status: Active » Needs review

This is great feedback. Thanks, everyone. I'm marking this "needs review" in case anyone can come back and mark it RTBC.

cedewey’s picture

Status: Needs review » Reviewed & tested by the community

I've used this module and it seems to be working great. I've changed this to reviewed and tested since it has been weeks without any complaints.

michelle’s picture

No problems on my site, either.

Michelle

doublejosh’s picture

Looking forward to this release.

christefano’s picture

Thanks for the feedback, everyone. I'll roll a new release when I have time (probably this week on Drupal Coworking Friday). Just wanted everyone to know I'm still paying attention to this issue.

kess’s picture

Hello guys,
do you perhaps have a final (stable) release of that module ?

Thx, bye Kess

konradesko’s picture

It would be nice to have stable version of module for D7 :)

Bernsch’s picture

7.x dev Version also works for me.
It's time to release a stable version of the module for D7!?

christefano’s picture

Okay, sounds like a new release is in order. I'll roll a new release during our next Drupal sprint day on Saturday.

Bernsch’s picture

@christefano: Can we roll a stable 7.x-1.0 release of Shared Email pleace!?

kumkum29’s picture

Issue summary: View changes

Hello,

have you planned to created a stable version of this module for D7?

Thanks for this.

christefano’s picture

Title: Drupal 7 » Release a Drupal 7.x-1.0 version
Status: Reviewed & tested by the community » Fixed

A version for Drupal 7 has been released:

   https://www.drupal.org/node/2333141

Thanks, everybody.

Status: Fixed » Closed (fixed)

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