Closed (fixed)
Project:
ImageField
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Aug 2006 at 15:09 UTC
Updated:
15 Sep 2006 at 23:15 UTC
Using CVS versions of CCK and Imagefield.
When creating a module, Imagefield doesn't work.
I select an image from the Hard Drive, and the path is displayed in the text field.
I click on update, but nothing changes.
Once the node has been saved, imagefield will work [with a small problem].
The images get added to the node, but initially don't 'preview' properly.
All that is displayed is Small bar like ||
Once the node is saved the images appear correctly, and when editing the node again, the preview works correctly.
Great Module, essential for CCK really. Will post some ideas as 'Feature Requests'.
Regards
Alan
Comments
Comment #1
StevenSokulski commentedI've been having this same problem. Initially my imagefields were set to required, but I could not make them work. After reading this I decided to try it with 'required' turned off. I was able to create the node and then add the images to it but it is now a two step process.
Comment #2
dodorama commentedI'm having the same problem with imagefield.module 1.9 and CCK 4.7.0 (11 Aug 2006) with drupal 4.7.3. I can add an image only if I edit the node after creation.
Comment #3
kurrik commentedI was getting this issue as well. Normally I'd live with it, but I'm deploying a site for a customer and telling them that they can't upload an image until the node is created was not an option, so I poked around the module code for a bit.
In the imagefield_widget function, it seems that when $op == 'form', the $node_field variable is not initialized correctly. From line 327 in imagefield.module, I changed the cvs code:
to
(just copying and pasting from directly above it)
The patch has solved this issue for me.
This is my first Drupal contribution, so please let me know if I'm going about this the wrong way. I've changed the status of this issue to patch(code needs review).
Otherwise, this has been a fantastic module.
Comment #4
dodorama commentedI'm not a php coder so I can't tell you the patch is well written; but I tested it and it works.
Thanks
Comment #5
dodorama commentedAfter further testing it seems that it worked just once. But as soon as I have time I will test it again with a fresh installation since right now I've got a lot of module on for testing and things are messing up a bit...
Comment #6
rickvug commentedI am experiencing this as well, just thought that add my voice to the chorus. This is the only bug in the module that I've found so far and I imagine that most other people are in the same boat. Thank you for stepping up to the plate to add this imagefield module - it was badly needed and is the reason I can now use CCK. Your work is greatly appreciated.
Cheers,
Rick
Comment #7
introfini commentedGreat work!
Thanks,
introfini
Comment #8
dopry commentedcommited to head and 4.7. it seems content.module isn't behaving as i'd expect it to and node_field isn't making its way from prepare form values. I also got a patch pending against cck that will resolve this issue as well. thanks.
Comment #9
(not verified) commented