Project:Install Profile API
Version:6.x-2.0
Component:Other
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

Comments

#1

Status:active» needs review

Visually confirmed by backtracing the imagecache_action_save() function in imagecache.module.

I'm not using install_imagecache_add_preset() in any of my code. Could somebody please test this patch?

AttachmentSize
imagecache.inc_.data-fix.patch 0 bytes

#2

Weird .. no patch upload, here it is again.

AttachmentSize
imagecache.inc_.data-fix.patch 723 bytes

#3

Status:needs review» fixed

Committing this bug fix, finally!

#4

Status:fixed» closed (fixed)

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