Is there one? How? How do maintain the field data and restore the invoice template?

Wouldn't it have been better to put this in a 6.x.3 branch to not oblige people to remain with an alpha version (the annoying notices that upgrades are available etc.)?

Comments

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Postponed (maintainer needs more info)

???

I don't think there is need for an upgrade path. The API is redesigned, so upgrading only requires extra work if you had modules installed that depend or integrate with Extra Fields Pane. Tokens remained the same, database schema remained the same. (However, some bugs have been fixed since 6.x-2.0-beta1, I plan to release a second beta in a few weeks, after I have done some extra testing.)

I may not have understand your question well. What kind of issues do you get?

ñull’s picture

ñull commented
Status: Postponed (maintainer needs more info) » Active

Well after installing beta1 my invoice template uc_order-customer.tpl.php did not show the variable any more that worked before. As far as I can tell tokens don't work in these templates the ones that are stored in the /sites/all/themes or /sites/default/themes folder that is.

Then I read the remark on the project home page of that change. Reverting back to alpha did bring back the variable $order->extra_fields[ucxf_cif_billing] that I used.

megachriz’s picture

megachriz
Primary language Dutch
commented

Could be because of a bug. See #1376430: Displaying custom-field information in uc_order-customer.tpl.php. Can you try it again with the latest dev?

ñull’s picture

ñull commented

Yes with latest dev de variable $extra_extra_billing_cif became available, in my case, which still the same is a change. So it would be good that somewhere in the documentation you mention this change from $order->extra_fields[ucxf_..._billing] to $extra_extra_billing_... Or to be more precisely that is explained that in previous versions both variables were supported and that now only the second method remains.

megachriz’s picture

megachriz
Primary language Dutch
commented
Assigned: Unassigned » megachriz
Status: Active » Needs work

Hm, I saw this was reported before: #1397154: Regression $order->extra_fields gone in 6.x-2.x .

Thanks for the tip to update the documentation. I will do so, but not right now.

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Needs work » Fixed

I've added a UPGRADE.txt to the module with instructions for developers what changes there were between 6.x-2.0-alpha1 and 6.x-2.0-beta1.

See commit.

Thanks for notifying me about this shortcoming in the documentation.

ñull’s picture

ñull commented

Great work! Thanks!

Status: Fixed » Closed (fixed)

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

maknet’s picture

maknet commented

I am so incredibly confused on what i'm _supposed_ to do. I would appreciate any help here.

Basically I put in the new beta1 and as per the OP, my information in my invoice template has also disappeared.

In the past, i could print using:
$extra_extra_information_current

In the upgrade.txt file, i tried to run update.php and it says "no update".

Old-method:
$extra_extra_information_current

I've tried to print using:
$orders['extra_extra_information_current']
$orders->extra_extra_information_current
$orders->information_current
$information_current
$extra_information_current
$extra_extra_information_current
and nothing shows up. Any help here is appreciated.

Thanks.

megachriz’s picture

megachriz
Primary language Dutch
commented

There is a bug in the current beta1. I plan to release a beta2 soon, after I've tested the current dev enough.

megachriz’s picture

megachriz
Primary language Dutch
commented

@maknet
See #1376430: Displaying custom-field information in uc_order-customer.tpl.php for the original issue about the extra information bug. $extra_extra_information_current should work with the latest dev version.