Closed (fixed)
Project:
Signup Integration for Ubercart
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2009 at 04:30 UTC
Updated:
22 Sep 2009 at 18:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
ezra-g commentedThanks for this patch. This is a handy feature request.
This patch needs some work. When I attempt to checkout for an event with one user who already has an account and one who does not, the profile fields are not presented for the user who does not have an account. As a result, the user is redirected back to the attendee profiles form when they attempt to checkout.
You are changing the structure of the attendee info form, so this is probably where this problem comes from.
Also, it makes sense to check whether the current user is already signed up for a particular event before prepopulating it, otherwise we could prepopulate with data that would fail validation, doesn't seem right.
If you wanted to make this feature a configurable option, that seems great to me but of course is not required. Thanks!
Comment #2
ezra-g commentedThe attached patch implements this without a configuration option.
Comment #3
ezra-g commentedThis is committed.
Comment #4
recrit commentedthanks! my use case did not implement profiles so I did not notice the profile issues you had mentioned. I don't feel it needs to be a configurable option since most users expect fields to pre-populate with their account info.