Closed (fixed)
Project:
Shared Email
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
4 Feb 2011 at 09:31 UTC
Updated:
18 Sep 2014 at 22:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
kiliweb commentedIt'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
-
http://www.kiliweb.fr
Comment #2
Rj-dupe-1 commentedThanks.
Comment #3
wesayso commentedHere'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.
Comment #4
wesayso commentedComment #5
christefano commentedThanks, wesayso! I'll review this in the next few days when I have the time.
Comment #6
christefano commentedComment #7
geerlingguy commentedSubscribe. 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.
Comment #8
christefano commentedA 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
Comment #9
geerlingguy commentedNice. I should comment on Drupal 7 port issues more often. This exact thing has happened twice in three days!
Comment #10
geerlingguy commentedIt 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 :)
Comment #11
christefano commentedSure, 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?
Comment #12
michelleI just installed the dev and it seems to be working fine.
Thanks!
Michelle
Comment #13
christefano commentedThis is great feedback. Thanks, everyone. I'm marking this "needs review" in case anyone can come back and mark it RTBC.
Comment #14
cedeweyI'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.
Comment #15
michelleNo problems on my site, either.
Michelle
Comment #16
doublejosh commentedLooking forward to this release.
Comment #17
christefano commentedThanks 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.
Comment #18
kess commentedHello guys,
do you perhaps have a final (stable) release of that module ?
Thx, bye Kess
Comment #19
konradesko commentedIt would be nice to have stable version of module for D7 :)
Comment #20
Bernsch commented7.x dev Version also works for me.
It's time to release a stable version of the module for D7!?
Comment #21
christefano commentedOkay, sounds like a new release is in order. I'll roll a new release during our next Drupal sprint day on Saturday.
Comment #22
Bernsch commented@christefano: Can we roll a stable 7.x-1.0 release of Shared Email pleace!?
Comment #23
kumkum29 commentedHello,
have you planned to created a stable version of this module for D7?
Thanks for this.
Comment #24
christefano commentedA version for Drupal 7 has been released:
https://www.drupal.org/node/2333141
Thanks, everybody.