Needs review
Project:
Ubercart Addresses
Version:
6.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
3 Jan 2013 at 19:58 UTC
Updated:
4 Jan 2013 at 09:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
megachrizUbercart Addresses 6.x-1.x alters the checkout form by a form alter hook. It adds the address select field to the delivery and billing panes when it thinks that they are enabled. The reason of the conflict is that Ubercart Addresses checks the settings directly for if the billing pane is enabled. Ubercart Product Checkout Panes temporary overrides this setting on runtime and Ubercart Addresses fails to know that.
I've came up with a simple solution: check if the billing pane exists in the form instead of checking the setting. The issue counts for the delivery pane as well, though there it was only an issue the other way around: the address select field was not added when the delivery pane was hidden by default, but temporary turned on by Ubercart Product Checkout Panes.
The attached patch should fix the issue. Please test if it doesn't introduce any side effects. Play with the Ubercart Product Checkout Panes settings to find out.
Comment #3
megachrizPatch is created against the latest dev.