We wanted admins to be able to add people who had called in to sign up for an event after the event was closed. It was a simple one line mod fix - that could be enhanced with a warning message, but I think would do what most people want.

I'm not skilled in patch making and since the mod is so simple...

Change line 1354 in v 5.x-2.4 from

if (user_access('sign up for content') && $node->signup_status) {

to

if ((user_access('sign up for content') && $node->signup_status) || user_access('administer all signups')) {

-Preston

CommentFileSizeAuthor
#3 247573_signup_admin_no_add_to_closed.patch1.09 KBdww
#2 signup_admin_addto_closed.patch638 bytesptone

Comments

pelicani’s picture

pelicani commented

Patches are really easy, try it out when you get a chance.
http://drupal.org/patch/create
Then post a patch, it's how things are done around here.
And makes maintaining a module, manageable ; )

ptone’s picture

ptone commented
StatusFileSize
new signup_admin_addto_closed.patch638 bytes

Alrighty then ;-)

Patch attached.

dww’s picture

dww
we/he/they
commented
Title: allow admins to signup users for closed events » Hide form for admins to signup users on closed events
Assigned: Unassigned » dww
Category: feature » bug
StatusFileSize
new 247573_signup_admin_no_add_to_closed.patch1.09 KB

Actually, I think the bug is that the form is presented at all when signups are closed for an event. If signups are closed, they're closed. If the admin wants to add more people, they should have to increase the signup limit or manually reopen, IMHO. See attached patch.

duellj’s picture

duellj commented
Status: Needs review » Reviewed & tested by the community

Tested. Thanks dww.

dww’s picture

dww
we/he/they
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks for testing, duellj.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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