When you are already registered and click Add to cart, there should be a "You are already registered for [title], you can view your registration." message in addition to the "This product is not available for registration at this time." error or menu without the Myself option.

CommentFileSizeAuthor
#8 1883482.diff1.32 KBdrumm
#3 1883482.diff1.97 KBdrumm

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
Issue tags: +portland2013

We will be running into this on DrupalCon sites.

drumm’s picture

drumm
he/him
Location NY, US
commented
Assigned: Unassigned » drumm
drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Active » Needs review
StatusFileSize
new 1883482.diff1.97 KB

This patch lets the user know that they have an existing registration. This will show up regardless of whether they can register someone else or not. If they can not register someone else, it explains why registration is not available. If they can register someone else, it reminds them that they are registered and shouldn't fill in their information again.

Status: Needs review » Needs work

The last submitted patch, 1883482.diff, failed testing.

drumm’s picture

drumm
he/him
Location NY, US
commented
Version: 7.x-3.x-dev » 7.x-2.x-dev
drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Needs work » Needs review

#3: 1883482.diff queued for re-testing.

jpontani’s picture

jpontani commented
Status: Needs review » Fixed

Looks good.

http://drupalcode.org/project/commerce_registration.git/commitdiff/11358d4

drumm’s picture

drumm
he/him
Location NY, US
commented
StatusFileSize
new 1883482.diff1.32 KB

Sorry, I omitted a couple things.

  • $user is needed.
  • ->uri() returns a data structure.

Patch against 7.x-2.x attached.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Fixed » Needs review
mrfelton’s picture

mrfelton commented

yes, patch in #8 is needed and is good. Thanks.

nhck’s picture

nhck commented
Category: task » bug
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

This should be committed asap. #8 should go in asap, the former patch causes some confusion.

jpontani’s picture

jpontani commented
Status: Reviewed & tested by the community » Fixed

Committed to dev. Thanks!

http://drupalcode.org/project/commerce_registration.git/commitdiff/496085a

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