Hello,
i don't know if that is an bug or if I missed something. EditableField does'nt work with filefield/imagefield for me. In an node view the fields are editable if enabled. I can upload an new file or image - in the case of imagefield i do see the image preview in the node. But if I reload the node view, the old image / file does appear. Other fieldtype like 'text' works fine.
Drupal 6.14,
6.x-2.x-dev editablefields (it doesn't work in early versions)
(Sorry for the english. Hope it sounds not to confusing.)
Thanks, for any help.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 614894_1.patch | 1.12 KB | andreiashu |
Comments
Comment #1
mckeen_greg commentedThe same applies when I have an Imagefield as ClicktoEdit in a view.
Comment #2
tnfno commentedI have the same problem, I can upload files but they do not show in the node. I realize that the data is (should be for filefields) saved in the background, but I think there should ba a save button s well, users might be confused if the data is actually saved.
Comment #3
Anonymous (not verified) commentedI'm having the same problem editing an imagefield in Views. The preview thumbnail is generated, but no changes are actually applied to the node.
Comment #4
peterparker commentedSubscribing to follow.
I am having the same troubles. In my case I am trying to create a few views to assist in the uploading of scanned documents and getting this to work would save the end user of my site a great deal of time.
FWIW, I am also using the FileField Paths module, which is meant to rename and relocate the file upon upload. This can't be helping my case.
Comment #5
milos.kroulik commentedsubscribe
Comment #6
Becker-and-Becker commentedsubscribing
Comment #7
andreiashu commentedIt is very important to get this working as soon as possible.
I've done some progress in the attached patch. It only works for single valued filefield fields. With multiple values it won't work.
Comment #8
andreiashu commentedComment #9
andreiashu commentedI committed a temporary fix in the 6--3 branch. Please note that it will remove the file from the node if you click on the remove button (normally on the node edit form it would remove it only when you click the save button after that).
It will only work with single valued fields for multiple values the behaviours is awkward.
My plan is to add a save button to certain (multiple values, filefield, imagefield) editablefields and then we'll get the normal behaviour: the files are attached/detached to the node only after clicking the save button. There is a ticket related to this already: #878094: Provide a Save button for some fields.
I would be interested in your thoughts about the 6--3 version though.
Please don't try to test this on a production site.
Cheers,
Andrei
Edit: I've updated the link to the commit (it points to drupalcode.org now).
Comment #11
yesct commented#1123640: Filefield and Image field half work with editablefields marked as duplicate of this issue
Comment #12
yesct commented#857938: Argument #2 is not an array in /sites/all/modules/filefield/filefield_widget.inc on line 255 might be of interest in getting this temporary fix to work (better).
Comment #13
rjbrown99 commentedHmm... I'd love to see what the commit was from #9 as I'd like to backport to 6.x-2.0. Was it the same as the one in #7? The reason I ask is because the link to the commitlog now goes to the old CVS commitlog which shows no information. Thanks.
Comment #14
andreiashu commented@rjbrown99: it is more of a hack than a permanent fix. Here is the commit. It takes over the filefield ahah callback.
Comment #15
rjbrown99 commentedThanks for the reference! My goal is just to get it working with 2.0 when using single-value filefield/imagefields. I'm not sure if I want to take over the callback or if that would have broader implications. Either way thanks for the follow up, very helpful. If I write some stuff that would be useful to others I'll post it back - probably to a new issue since this one is closed.
Comment #16
andreiashu commentedYes, it doesn't feel good to take over the AHAH callback of filefiled but it was just a quick and dirty solution.
If you make it work with 2.0 I'm happy for you to open this issue - it is better to have a history behind a patch.
Thanks
Comment #17
micheleannj commentedsubscribing