Needs review
Project:
Flexinode
Version:
4.7.x-1.x-dev
Component:
Field type: image
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2006 at 22:04 UTC
Updated:
25 Jan 2007 at 22:46 UTC
Jump to comment: Most recent file
Comments
Comment #1
netupgrades commentedFound a solution that seems to work. With little testing done, I do note that it does give you a little checkbox to delete images. Thanks for that patch.
Comment #2
iandit commentedhmmm... may I ask where you got the patch?
Comment #3
Bèr Kessels commentedMaking this a feature request. This is somthing that should be dealt with in image field inc.
Comment #4
mo6This patch adds a checkbox under each image field which can be selected to delete the image after submitting the node. The image field is grouped with the checkbox.
Comment #5
Bèr Kessels commentedI`ll have to give it a more thourough review and test, but it looks good.
I am no fan of the 'delete' checkbox, myself, i think it is very un-userfriendly. However, its done like this all trough Drupal, so as a quick fix, I think this is by far the best option. Maybe we can open a differrent issue later, to improve this interface issue separately, but not now.
That said, I would love reviews from others too, so folks, if you need this, please test and post your findings here! Thanks.
A small note, nothing serious to George: if you patch, please try to use the flags -u and -p, its the Drupal standard and it makes patches a lot more readable. This is not critique, its just to help you learn The Drupal Way of things ;).
Comment #6
mo6Here's a new (unified) patch which changes the weight of the fields inside the (image) fieldset. Also the name of the checkbox has been changed (a little).
Comment #7
mo6I noticed that the patch doesn't actually remove the image files. After some digging into the internals of flexinode and field_image (and some caffeine) I found out why. Here's a new patch that works (hopefully!)