Call to undefined function uc_order_load() in uc_taxes.install during upgrade 6.x-2 ->7.x-3 [#1480150]

I'm experimenting the D6 to D7 upgrade and everything looks fine with core/other modules, but Ubercart gives the following during update.php

An AJAX HTTP error occurred. HTTP Result Code: 200 Debugging information follows. Path: .../update.php?op=selection&token=...&id=5&op=do StatusText: OK ResponseText: Fatal error: Call to undefined function uc_order_load() in ...\sites\all\modules\ubercart\uc_taxes\uc_taxes.install on line 239

I found #587572: uc_price is undefined in uc_order_get_total(), but it's related to uc_store/uc_price whilst this seems due to uc_taxes..

Comment	File	Size	Author
#9	uc_taxes-bug-uc_order_load-1480150.patch	766 bytes	kenorb
#9
#8	uc_taxes-bug-uc_order_load-1480150.patch	759 bytes	kenorb
#8
#7	uc_taxes-bug-uc_order_load-1480150.patch	772 bytes	kenorb
#7

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mandreato CreditAttribution: mandreato commented 13 March 2012 at 09:23

It occours during update #7007

Comment #2

TR CreditAttribution: TR commented 13 March 2012 at 20:56

Did you have uc_order disabled during the upgrade?

Comment #3

mandreato CreditAttribution: mandreato commented 13 March 2012 at 21:19

Yes: I've disabled all the modules before upgrade.php

Comment #4

TR CreditAttribution: TR commented 15 March 2012 at 20:36

This is a basically a core Drupal issue, see #773828: Upgrade of disabled modules fails.

We *might* be able to avoid this particular issue by including uc_order.module in uc_taxes.install, however that also might cause other issues - I don't know what other dependencies would bring in. As a workaround, you could simply re-enable the modules and run any uncompleted updates.

Comment #5

mandreato CreditAttribution: mandreato commented 2 July 2012 at 11:59

Status:

Active

» Fixed

I've bypassed the issue by following these steps:
1) upgrade Drupal core
2) upgrade all contrib modules but Ubercart
3) substitue the Ubercart folder with the 7.x-3.x-dev one
4) activate Order, Product and Store of Ubercart - core group
5) update.php
6) activate other UC sub-modules (Cart, etc.)

Comment #6

16 July 2012 at 12:01

Status:

Fixed

» Closed (fixed)

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

Comment #7

kenorb CreditAttribution: kenorb commented 5 November 2013 at 12:49

Issue summary:	View changes
Status:	Closed (fixed)	» Needs review

File	Size
uc_taxes-bug-uc_order_load-1480150.patch	772 bytes

This is still happening, so it's not fixed.

Comment #8

kenorb CreditAttribution: kenorb commented 6 November 2013 at 12:53

File	Size
uc_taxes-bug-uc_order_load-1480150.patch	759 bytes

Adding better patch.
Similar to uc_store_update_7007() where uc_store is actually enabling potencially disabled modules.

Comment #9

kenorb CreditAttribution: kenorb commented 6 November 2013 at 12:57

File	Size
uc_taxes-bug-uc_order_load-1480150.patch	766 bytes

1 file was hidden/shown/deleted

File	Size
uc_taxes-bug-uc_order_load-1480150.patch	759 bytes

Comment #10

longwave

he/him

English

CreditAttribution: longwave commented 6 November 2013 at 13:54

What relevance is uc_cart here, if the problem is uc_order_load() is not available?

Ideally I think we should not actually be calling uc_order_load() or uc_order_save() from the .install file, but only doing the exact database operations that are needed.