Category:
bug reportPriority:
criticalStatus:
activeWhen using checkout on a cart that contains only file products (uc_cart_is_shippable() returns FALSE), the delivery pane is invisible.
However, the 'Next' button on the 'Customer information' pane, has the following HTML:
<input type="submit" class="form-submit" onclick="return uc_cart_next_button_click(this, 'delivery', 'customer');" value="Next" id="edit-panes-customer-next" name="op"/>Since the delivery pane is invisible, the form gets submitted which results in error messages.
There are other issues, like the payment pane, which is uncollapsed by default in this case, and more.
I marked the bug as critical since this happens with the default settings, i.e. will happen to anyone installing this module.
Workaround:
uncheck the following in the checkout settings page:
[ ] Use collapsing checkout panes with next buttons during checkout.
[ ] Collapse a pane when its next button is clicked.



Re: Errors on checkout when delivary pane invisible
Hmm.. I did have to debug quite a few issues related to collapsing panes recently... what version of UC are you running? (With my luck I bet it's the latest.
)
Re: Re: Errors on checkout when delivary pane invisible
Yes, the latest (7e). sorry..
Re: Re: Re: Errors on checkout when delivary pane invisible
Looking through the Bazaar logs, revision 452 had the fixes - which was fortunately after Alpha 7e released. (Fortunately for me, since I may have fixed the problem, but not for folks running 7e.) If you can't upgrade to the latest Bazaar version, you can try diff-ing uc_cart.module and uc_cart_checkout_pane.inc to see what changed to fix it.
Of course, going to the latest version means you'll need the workflow-ng module, and we haven't fully tested updates from previous versions to this. You might just try the workaround until the Alpha 8.
Thanks! I see the fix you
Thanks!
I see the fix you put in rev 452 - nice and clean to mark each pane as shippable
Maybe I'll backport it into my own 7e..
Do you think it also fixes the tax calculation issue (http://www.ubercart.org/issue/1406/when_cart_not_shippable_taxes_are_cal...)? My guess is no, but I'm checking just in case..
Re: Thanks! I see the fix you
Nah, I don't know exactly where that bug is coming from, but I'll point it out to Lyle and see if he knows.