Now that a fix for #1424038: Order administration: "copy address" feature is committed to Ubercart Addresses, Extra Fields Pane should be a little bit adjusted in order to still work with that change. Two changes are needed:

  • The "Orders" test should be adjusted, because the panes are renamed respectively from "delivery" and "billing" to "ship_to" and "bill_to". The test expect the old pane names.
  • A small change in uc_extra_fields_pane_uc_order() is needed to let the copy address feature in the order administration work for extra fields.

Unfortunately, a patch (which I already have ready) can't be committed yet until the 7.x-1.x branch of Ubercart Addresses gets a new release. Because if I commit a change now, the branch tests on drupal.org will fail.

A patch will follow anyway.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MegaChriz’s picture

Patch attached.

MegaChriz’s picture

The patch still can't be committed, because the testbot keeps checking out an older release of Ubercart Addresses (7.x-1.0-alpha2), while Ubercart Addresses 7.x-1.0-beta1 is needed to prevent the branch test from failing.

After the patch in #1 is committed, a first release candidate can be released for Extra Fields Pane 7.x-1.x.

MegaChriz’s picture

Issue summary: View changes
Status: Needs review » Fixed

Committed #1. Hopefully, the testbot will grab the right version of uc_addresses this time so the branch tests will pass. Fingers crossed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.