Is there any intention to port this module to D6 (or beyond)?

Thanks.

Comments

davistv’s picture

Hi there,

I didn't know anyone actually used this module, thanks for writing. I have a few sites in D5, haven't tried to upgrade them yet... Not really looking forward to that experience.

But I'll certainly take a look at this module and see what needs to be updated. Hope to have more info in a few days.

davistv’s picture

Assigned: Unassigned » davistv
davistv’s picture

Drupal 6 fixes committed to cvs. The download link should appear within 12 hours or so. If you can install and test, I'd greatly appreciate it.

Thanks!

scoutbaker’s picture

Thanks for the update. I'll put this in my queue for testing. I'm working on a test migration right now.

scoutbaker’s picture

Version: 5.x-1.2 » 6.x-2.x-dev

I've tested this on a couple different installations. One is an actual 5.x->6.x site upgrade. The other is a clean D6 test site I set up. Both of them give me the same message. As soon as the module is enabled, it comes back with:

warning: Missing argument 1 for user_edit_notify_menu() in <path to site>/sites/all/modules/user_edit_notify/user_edit_notify.module on line 93.

The same warning shows up whenever update.php is run of course.

Dan Silver’s picture

Does anyone know the status of the D6 port?

Dan Silver’s picture

I got this error too:
warning: Missing argument 1 for user_edit_notify_menu() in [removed]/sites/all/modules/user_edit_notify/user_edit_notify.module on line 93.

cornersoyo’s picture

It looks like that is the menu_hook (where the error occurs).

Anyways, it appears that all this does is accept an email address to send the notifications to. From what I can tell, if that is blank then it is supposed to default to site_mail, and then sendmail_from.

So before I even bother with this, I am wondering - Does it even work? I mean, are notifications being sent to the default email address when a profile change is made?