Based on the fact that it is looking for 'action' as index to the passed array, I think the function install_imagecache_add_preset, line 30 needs to be changed from:
$action['data'] = $data;
to
$action['data'] = $data['data'];
Otherwise, the data does not get properly inserted into the table.
Also, as currently structured, the comment up top is incorrect: "Each action is itself an array with indexes of action, weight" ... it does not look for 'weight' index
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | imagecache.inc_.data-fix.patch | 723 bytes | James Andres |
| #1 | imagecache.inc_.data-fix.patch | 0 bytes | James Andres |
Comments
Comment #1
James Andres commentedVisually confirmed by backtracing the
imagecache_action_save()function inimagecache.module.I'm not using
install_imagecache_add_preset()in any of my code. Could somebody please test this patch?Comment #2
James Andres commentedWeird .. no patch upload, here it is again.
Comment #3
James Andres commentedCommitting this bug fix, finally!