I have been working a module to improve commerce packing algorithm. The module was ready and I was about to release a dev version of the module. So I decide to update this module and I got a bunch of errors. Some of the error are from the module that I have been working on. However, I believe if I figure out how to solve the error below it would fix the rest of the modules my module.

Here is the error that I have been getting:

Notice: Trying to get property of non-object in commerce_physical_product_line_item_dimensions() (line 154 of /var/www/thursby2/sites/all/modules/commerce_physical/commerce_physical.module).

Any thoughts about this particular error ? Anyone have seen this before?

Here is the version I was using on this module:

Comments

darol100’s picture

Issue summary: View changes
StatusFileSize
new13.48 KB
agileadam’s picture

I recently updated Commerce Physical Product and Physical Fields and ended up getting this error during checkout (while attempting to get UPS shipping rates).

I've attached a one-liner patch that corrects the issue.

darol100’s picture

Status: Active » Reviewed & tested by the community

I have review the patch of comment #2 and it works like charm. It will nice if we can push this patch ASAP, because I'm working on contribute module that depends on this module and I will not be able to make a release if it have these errors. I have change to RTCB.

rszrama’s picture

Category: Support request » Bug report
Status: Reviewed & tested by the community » Fixed

Thanks for the fix.

  • rszrama committed 2bbca91 on 7.x-1.x
    Issue #2480355 by agileadam: properly bound a for loop by a line item's...

Status: Fixed » Closed (fixed)

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