After selecting a user or group of users and choosing email from the drop down, then update:

I get:

warning: Missing argument 2 for advuser_multiple_email_confirm_page() in /home/admin/public_html/dev/modules/advuser/advuser.module on line 350.

This is in 5.1

CommentFileSizeAuthor
#2 advuser-5.x-1.x-dev.patch.txt5.67 KBAnonymous (not verified)

Comments

Anonymous’s picture

I'll take a look later today.

Anonymous’s picture

Assigned: Unassigned »
Status: Active » Needs review
StatusFileSize
new5.67 KB

A long today.

The attached patch should fix this issue. I've also reordered the administration page to put the individual fields into common groups for better appeal and I renamed the 'Roles' selection title to 'Notification Roles'.

Please test and let me know what you think. I will work on getting this into CVS and update the download package toward the middle of next week.

Earnie

Anonymous’s picture

Hi Earnie,
How do you apply the patch?
I suppose you have to replace certain part of the advuser.module file. I couldn´t figure out how.
Is it by replacing the whole sections: User operations, AdvUser operations?
Sorry to ask this, but I have never patched before. ;(

Wouldn´t it be simpler to post the new advuser.module file instead?
Thanks.

Anonymous’s picture

For some it would be easier. Anyway as soon as I get a round tuit I plan on adding the patch to CVS and giving a new download file. I just don't have the time this month. I had hoped to have it already committed by now but life got in the way.

Look for a program named patch. Patch will take the difference file and apply it to the module. If you use *nix then it is probably supplied. If you use *dows then google for it.

Anonymous’s picture

Status: Needs review » Reviewed & tested by the community

I'll be commiting this patch within the next 24 hours. If anyone has issues with it then speak up now.

Anonymous’s picture

Status: Reviewed & tested by the community » Fixed

Assuming I did the Drupal CVS thing correctly, this should be resolved.

Anonymous’s picture

I forgot to say wait on the rebuild of the snapshot, which I believe should happen tonight, before downloading.

Anonymous’s picture

Status: Fixed » Closed (fixed)