Closed (fixed)
Project:
ImageField
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Jul 2006 at 19:37 UTC
Updated:
16 Nov 2006 at 00:47 UTC
Jump to comment: Most recent file
Comments
Comment #1
willdashwood commentedShouldn't it be:
$values[$delta] = array_merge((array)$node_field[$delta], (array)_imagefield_file_load($file['fid']));I was having the same problems until I made that change. Now all appears to be working fine.
Comment #2
RobRoy commentedJust need the (array) on the second arg.
This patch also includes false/true capitalization.
Comment #3
drewish commentedHumm, I changed it around a bit so that the _imagefield_file_load() returns an array in either case.
Comment #4
FredCK commentedMaybe the following is a clearer replacement... just place:
Instead of:
I'm setting it as "critical", as I'm having this issue on every single installation.
Comment #5
dopry commentedApplied a modified version that added an array cast to node_field[delta] in the array_merge. committed to 4.7.
Comment #6
beauregard commentedI had the same problem and downloaded the newest version of imagefield.module. But the problem remained.
Then I tried to apply imagefield_array_fix.patch, the problem still remained.
Then I tried only to change 1 code of line as willdashwood suggested. And now it seems working.
Comment #7
m3avrck commentedConfirmed getting the same array_merge() errors.
Comment #8
(not verified) commented