I can't startup my system

CommentFileSizeAuthor
#6 1570038-5.patch395 bytestheladebug

Comments

tr’s picture

Category: bug » support

That class resides in uc_order/uc_order.info.inc, so make sure the include file is present and clear your caches. If you've upgraded your version of Ubercart, be sure to run update.php. Rebuild your class registry if necessary with http://drupal.org/project/registry_rebuild

socialnicheguru’s picture

Status: Active » Fixed

thanks. I have been going crazy all day with this

Status: Fixed » Closed (fixed)

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

theladebug’s picture

Status: Closed (fixed) » Active
tr’s picture

Status: Active » Postponed (maintainer needs more info)

? If you expect help, I suggest you provide some information. At a minimum, say what you've tried, which better include the advice given in #1 ...

theladebug’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new395 bytes

This is still happening, along with the salesforce module in my case. It's because the entity metadata file isnt listed in the uc_order.info file.

Attached is a patch for it.

theladebug’s picture

Sorry TR, I could have probably re-opened and patched at the same time :)

Status: Needs review » Needs work

The last submitted patch, 1570038-5.patch, failed testing.

longwave’s picture

Status: Needs work » Fixed

This isn't an Ubercart issue. As I described #1989168: Fatal error: Class 'UcOrderProductMetadataController' not found in .../sites/all/modules/entity/entity.info.inc, the Salesforce module is calling entity_entity_property_info() when this function should be called as a hook, which would automatically include the right file. This same issue would affect other modules that implement hook_entity_property_info() and put them in a .info.inc file. Ubercart uses the class in question, and does not suffer from this error, because it uses the hook system correctly.

longwave’s picture

Category: support » task

Having said that I guess this wouldn't hurt, as class files are autoloaded as needed. Committed to 7.x-3.x.

theladebug’s picture

Thanks for clarifying, hopefully adding it to the .info file will save you some headaches in the future when other modules make the same mistake and throw a similar error message.

Before I committed this, I double checked some other modules which were working properly in my situation and the only difference was that the other modules had this file listed in their .info files, so it seems like this helps accomodate other modules which fail to use the entity hooks properly.

Status: Fixed » Closed (fixed)

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