Conflict with Ubercart Product Checkout Panes module? [#1879278]

Hello,

I am using this module as well as Ubercart Product Checkout Panes. I use UPCP to hide certain panes in the checkout process whenever someone has a specific product in their cart. One of those panes that I need to hide is the Billing Information pane. As described at this link (http://drupal.org/node/1878358), when I hide that pane the Saved Addresses dropdown menu still appears at the very bottom of the checkout page. Can you please let me know if there is a workaround that would allow me to hide that field with the rest of the hidden pane?

Thanks!

Comment	File	Size	Author
#1	uc_addresses-check-billing-pane-1879278-1.patch	6.79 KB	MegaChriz
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

MegaChriz CreditAttribution: MegaChriz commented 3 January 2013 at 20:41

Status:

Active

» Needs review

File	Size
uc_addresses-check-billing-pane-1879278-1.patch	6.79 KB

Ubercart 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.