 |
UbercartOne cart to rule them all... |
|
| Donate to Ubercart | Affiliates | ||
Losing cust checkout data on error.
Submitted by Blessed_Steve on Tue, 01/15/2008 - 10:25
When a customer makes an error on the checkout form such as, missing city in delivery panel, other data, notably the shipping quote and CC info are lost and the form repaints. Is there a central place I can look at in terms of preserving whatever info was supplied prior to an error? Also, since the shipping quote is triggered on change of zone field, on the repainted form there is just teh passive recalculate button and no shipping method. How are people dealing with this?
As a follow up, in uc_quote.js, the function setQuoteCallbacks defines three condition under which a quote is generated. 1. The delivery_zone field changes Any suggestions on how to inplement a fourth condition: An error has occurred and we are repainting the checkout page with data? thanks. I'd say an onError event handler: http://www.devguru.com/Technologies/ecmascript/QuickRef/evhan_onerror.ht... http://www.w3schools.com/js/js_onerror.asp But you might want to dig deeper to get the value of the error and include some helpful text (or reset whatever field happens to be the cause of the error, if any.) My .02. Maybe there are 5 conditions needed! The one I am going after is not a javascript error but rather when a customer causes an error by not filling in a required field, for instance. I fill out form, but leave required phone number blank. when the checkout page repaint with the error on top saying "phone number is required", I want the page to automatically trigger a quote recalc or.. ideally, display the old quote with the custmers choice from the previous screen still selected. onError refers to JavaScript errors, not Drupal form errors. I don't think that would be very useful because it's probably the quote fetching that causes the error, and just repeating it won't help. I tried putting <?phpbut it didn't work. I think the form errors are reset by the time the page is displayed again. This is a longstanding problem... sooo... I'm gonna add it to the beta fixin' post. It might be wise to turn this into an issue. Ah, makes sense, I was figuring if the Ajax calls were throwing an error, but I see what you're doing now. My bad Well, for quotes, Lyle should have nailed these with commits 742 and 744. For the payment details, that's a little more tricky... Those form fields are being generated by Drupal but not on the initial page load. They come from a separate request which knows naught of the data used to populate the rebuilt form w/ errors highlighted. Gonna look into it this afternoon... Ok, I believe I resolved issues with the payment details on the Livetest. I'm struggling with Drupal's Forms API in a contest over who gets to validate what data when, and it wants to validate required fields first. That's unfortunate, because it will cause the payment pane not to expand when both the payment pane and a required field from somewhere else on the form fail validation. It's cool, though, because at least we're preserving the data... just a little bit annoying. And of course, this can be avoided by a site not using collapsible fields if they really don't want the hassle for their customers. So, test it out and I'll be committing this for beta 5. Thanks, I'm looking forward to giving it a go! Hello. I've just begun working on an existing install of Ubercart and I am experiencing this problem. I was wondering if this issue still exists in the latest stable release and also, how can one find the version number of an existing installation. That information isn't showing up in the Drupal modules list. Thanks very much. If the version number isn't showing up, it's probably a download from our Bazaar site. This issue has been fixed for a while, though. You might check to make sure they aren't using any incompatible contribs and then update the code to the latest version. We've posted some steps for that in this FAQ. That's extremely helpful. Thank you very much. I've updated to the latest version and I've narrowed down our issue to a contributed module. The module is uc_nodetype, referenced here: http://www.ubercart.org/forum/development/2121/uc_nodetype_module_purcha... When credit card info on the checkout page fails validation the data contained in uc_nodetype's section of the form is lost. I've been looking through diffs of the uc_quote code to see what was done there to allow it to maintain it's state after a failed validation. I'm unfamiliar with the module so I'm a bit unsure of which code is core to storing that state. I was hoping someone could point me towards a good starting point. Is it as simple as storing the form values in the sesssion during validation and retrieving them in form_alter()? Or, are there other gotchas i should be aware of? Thanks for taking the time.
|
|
Joined: 08/07/2007