Posted by Jaza on October 29, 2009 at 2:51am
| Project: | Popups: Add and Reference |
| Version: | 6.x-1.0 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Jaza |
| Status: | needs review |
Issue Summary
This is related to:
#339196: Image uploading through popups
I've made a patch to make FileField and ImageField uploads work in a popup. I posted the Popups API part of my patch to that thread. I'm posting here the 'add and reference' node form part of the patch.
This is a simple patch that adds a hidden form field to node forms, when they're output in a popup. This form field's value is now checked by popups, instead of using any custom HTTP headers (which don't work when file uploads are involved).
| Attachment | Size |
|---|---|
| popups_reference_image_uploading.patch | 484 bytes |
Comments
#1
Form field properties needed '#' at the start... no idea why it worked before, but it doesn't work for me now.
Updated patch attached.
#2
This looks really promising. Subscribing, will try to find time to test. (In combination with your other patch, naturally.)
#3
subscribing
#4
Updated patch, for this thread and for:
#339196: Image uploading through popups
See the other thread for details.
#5
Jaza,
I still have errors when uploading the image.
A page is loaded with a textarea containing some JSON code.
A screenshoot is attached.
Regards
#6
subscribing
#7
subscribing
#8
subscribing
#9
I have applied the patch, but I'm still getting a "Bad Response form submission" error when I try to upload an image within the popup. Any ideas why this is?
#10
subscribing