Hello

I use cck and imagefield HEAD with drupal 5.0-beta1, but I can't get it working :
- I have an imagefield (optional, unique) on a "link" content type
- when I edit the node, if I put and image and click on "update", I get it successfully uploaded to the server but displayed twice
- when I submit the node, the image doesn't appear, the

is empty
- if I try to edit the node again, the imagefield is empty

Any workaroud please ?

Comments

dopry’s picture

Do you have any additional information?
Is the file on the server in the files folder?
Is there a database entry for the file?
Do normal uploads work for you?

imagefield hasn't been branched for DRUPAL-5 yet, and has just been ported to the DRUPAL-5 API. I still need to update the module to work with cck. Feel free to have a hand at it. I won't be putting a lot of effort into it until CCK is stable on DRUPAl-5.

quicksketch’s picture

StatusFileSize
new1.48 KB

Here's a patch which gets imagefield working with Drupal 5 / CCK HEAD. It solves the duplicate image problem and adds dependency checking to the .info file. The "Delete" checkbox is not working still with this patch, which also is not working in the 4.7 branch. A separate patch will be required for that problem.

quicksketch’s picture

Status: Active » Needs review
dopry’s picture

not quite the right fix. there is a bug in content.module, content_form_alter calls content_form twice, which means the file data gets prepped twice. The session stuff also seems to almost be working now. I will revisit this when one of the other cck committers has had time to commit the form_alter patch.

quicksketch’s picture

Status: Needs review » Active

Right you are. I found that this patch doesn't work for multiple field types. I've got a much better patch coming for issue #73174 which should incorporate the necessary changes there (it's all related).

quicksketch’s picture

Title: drupal 5 + cck HEAD + imagefield HEAD, doesn't work » Update HEAD .info file for Drupal 5
Status: Active » Needs review
StatusFileSize
new784 bytes

Full compatibility for Drupal 5 is included in the patch for issue #73174. However, the info file still needs to be updated to include the dependency checking. Here's a trimmed patch with just the info file changes.

dopry’s picture

Status: Needs review » Fixed

hrm... I patch my local repo, but forgot to commit this hrm...

Anonymous’s picture

Status: Fixed » Closed (fixed)