Posted by yello73 on April 27, 2009 at 10:23am
Jump to:
| Project: | Flexifield |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I downloaded your dev snapshots and made the following test:
- Created a Content type (1) with two images field
- Created a Content type (2) with a multiple flexifield using (1)
Uploading images now works correctly but if I create a node with:
-------
a-- image1
b-- image2
-------
a-- image3
b-- image4
--------
If I remove image2 and save the node, It also removes image4.
Any idea?
Thanks.
Comments
#1
I can replicate this, and I found what's causing the problem. It has to do with how the filefield module implements its filefield_widget_process function in determining which "remove" button got clicked: like many CCK modules, it's written with the assumption that each field exists within a form only once, and that's not the case when using flexifield, but flexifield has not yet become sufficiently popular within the community for field module authors to take it into account.
I need to take some time to think about whether there's a good way to fix this within flexifield, or if I need to submit a patch to filefield.
#2
I submitted a request to quicksketch to change filefield: #447572: Name construction of remove button causes incompatibility with flexifield.
If you're feeling anxious to get this working, you can make the change on your local copy of filefield_widget.inc, but you might want to wait to see what quicksketch thinks about this.
#3
Quicksketch committed the patch, so this is now fixed as long as you use releases of filefield after 3.0 (e.g., 3.1 once that gets released, or else the dev snapshot).
#4
Automatically closed -- issue fixed for 2 weeks with no activity.