patch to delete image in flexinode

netupgrades - August 29, 2006 - 22:04
Project:Flexinode
Version:4.7.x-1.x-dev
Component:Field type: image
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (code needs review)
Description

What are people doing to delete images from flexinodes?

The only thing I can imagine now is to go in w/ an ftp program and delete images. Is there another way?

Thanks for any sort of tip ---

#1

netupgrades - August 30, 2006 - 11:42
Title:how to delete images from flexinodes» patch to delete image in flexinode

Found 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.

#2

iandit - October 5, 2006 - 00:16

hmmm... may I ask where you got the patch?

#3

Bèr Kessels - January 2, 2007 - 10:26
Component:User interface» Field type: image
Category:support request» feature request

Making this a feature request. This is somthing that should be dealt with in image field inc.

#4

george@dynapres.nl - January 25, 2007 - 16:17
Status:active» patch (code needs review)

This 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.

AttachmentSize
flexinode-field-image-delete.patch1.04 KB

#5

Bèr Kessels - January 25, 2007 - 17:26

I`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 ;).

#6

george@dynapres.nl - January 25, 2007 - 21:45

Here'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).

AttachmentSize
flexinode-field-image-delete-2.patch2.32 KB

#7

george@dynapres.nl - January 25, 2007 - 22:46

I 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!)

AttachmentSize
flexinode-field-image-delete-3.patch2.4 KB
 
 

Drupal is a registered trademark of Dries Buytaert.