Posted by Petr Illek on April 25, 2009 at 11:32pm
| Project: | Ubercart |
| Version: | 6.x-2.0-rc1 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I got this errors after cron run first time. After the second run, its seems to be OK...
# warning: Illegal offset type in C:\dev\www\sites\all\modules\ubercart\uc_product_kit\uc_product_kit.module on line 233.
# warning: Illegal offset type in C:\dev\www\sites\all\modules\ubercart\uc_product_kit\uc_product_kit.module on line 236.
# warning: Illegal offset type in C:\dev\www\sites\all\modules\ubercart\uc_product_kit\uc_product_kit.module on line 245.
Reposted from u.o
Comments
#1
Did you just create a product kit before cron ran? Was there anything strange going on with it at all? Would cron be editing your product kits for some reason?
#2
Actually, that looks like a big enough problem to be fixed. Usually, node_update functions get the form values from the node edit form, but they could also get the actual node object when node_save() is called by some other code. Unfortunately, these aren't always the same things.
This patch ought to fix it, which in turn should keep those errors from showing up during cron, whatever was causing it. It might also help with people trying to import product kits.
#3
Applied cleanly... no errors on cron run or normal product kit update. Not really up to installing a whole new site just to test it, though. So, works for me? Sufficient to be committed? Back at ya...
#4
Yeah, worked for me too. Committed, so I hope it actually fixes the cron error.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.