Even though an event manager can specify multiple slots per registrant for their event, there is no easy place for the registrant do this. Currently, the registrant has to know their registration id, and can goto mysite.com/registration/(regid)/edit and change the spaces.

I propose that we add a textbox above the attending box that has a description showing how many spaces you can use.

CommentFileSizeAuthor
#8 2129347-regslots-8.patch10.26 KBZarabadoo
#7 2129347-regslots-7.patch7.23 KBjaperry
#6 2129347-regslots-2.patch2.02 KBjaperry
#2 Screenshot_11_11_13_10_11_AM.png115.93 KBjaperry
#1 2129347-regslots-1.patch2.8 KBjaperry
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Assigned: Unassigned » Zarabadoo
FileSize
2129347-regslots-1.patch2.8 KB

Here is the first draft of the patch. It needs theming love. Assigning to Al.

I presume that making the text field smaller, and putting text in there saying 'seats' or 'attendance count' would work well here.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
FileSize
Screenshot_11_11_13_10_11_AM.png115.93 KB

With this patch, you'll see it looking like this. We'll need to removed the debugging statements, and do some other cleanup, but theming is probably the most missing part of this patch right now.

screenshot

ezra-g’s picture

ezra-g CreditAttribution: ezra-g commented

In the interest of keeping the registration functionality simple, why not hide the quantity field entirely and default it to 1?

ezra-g’s picture

ezra-g CreditAttribution: ezra-g commented

After discussing with japerry, I think it's reasonable to keep this feature. I'd propose "# of attendees" as the description text and that we use the Form API to set a default length for the field to 2 characters.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented

We should also coordindate this patch with this patch in #2090075: Registration status shows "Open" while the registration has not been opened, if user has administer permissions by adding it to our makefile until there is a fix committed for registration. Admins shouldn't seeing a different event status than normal users.

This is the patch to add.
https://drupal.org/files/issues/2090075-removeadminbypass-1.patch

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Status: Active » Needs work
FileSize
2129347-regslots-2.patch2.02 KB

Second patch puts in a 'party size' indicator and moves the size down. Just needs a little theme love before getting committed.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
FileSize
2129347-regslots-7.patch7.23 KB

Here is a patch that cleans up the counts a bit more.

Zarabadoo’s picture

Zarabadoo CreditAttribution: Zarabadoo commented
FileSize
2129347-regslots-8.patch10.26 KB

This patch will allow it to format better. It really does need some labels, text changes, and possibly to have elements rearranged. I played around with adding and removing variations of those, and the css changes in this patch along with japerry's work should be quite flexible.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Priority: Normal » Major

  • Commit 80cc001 on 7.x-3.x by japerry: 
    Issue #2129347 by japerry, Zarabadoo: Allow registrants of an event...
japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Status: Needs work » Fixed

Committed Al's changes, this adds the basic functionality for events. Improvements, if any, can be done later in a new issue.

Status: Fixed » Closed (fixed)

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