There is an amazing module called FAPE (drupal.org/project/fape) that allows links to be generated to allow editing of individual fields (i.e. admin/field/edit/node/1/body). It currently integrates with contextual module (turn on fape + contextual and use the IPE on a panelizer node), but could be extended to a larger use case.
The task here is...
-- update panopoly magic to include the edit form as part of the configuration as per http://drupal.org/files/fape-added-to-ipe.png
-- empower the "update field" button to save the field + update the preview
-- create a patch for FAPE that disables the hard requirement of contextual
Comment | File | Size | Author |
---|---|---|---|
#7 | 1772824-reverso.patch | 1.43 KB | beeradb |
#3 | fape-panopoly-magic.patch | 8.76 KB | beeradb |
#2 | fape-panopoly-core.patch | 601 bytes | beeradb |
#2 | 1772824-fape.patch | 6.98 KB | beeradb |
#1 | fape-panoply-widgets.patch | 545 bytes | beeradb |
Comments
Comment #1
beeradb CreditAttribution: beeradb commentedWe'll call this Alpha 1.
FAPE is not yet added to the build so you'll need to manually install it if you want to test this.
There's still a lot of kinks to work out in this implementation, so here's some of the stuff that's wrong.
This is generally working, but there's definitely quite a bit of polish to add.
Comment #2
beeradb CreditAttribution: beeradb commentedHere's alpha 2.
Improvements:
Issues:
This patch requires #1788170: Allow the context object to be altered by form submitters and validators and #1785056: Remove contextual dependency
Comment #3
beeradb CreditAttribution: beeradb commentedAck. The second patch was wrong. This should be used in replacement of the 1772824-fape.patch.
Comment #4
populist CreditAttribution: populist commentedAlright! This is looking pretty sharp and I went ahead and committed all work to date in #1 and #2. I even enabled the fape.module by default (and removed the conditional logic) since it was so cool. In terms of the issues:
-- Image widget still auto submits to next step after changing image
I believe I fixed this with http://drupalcode.org/project/panopoly_admin.git/commitdiff/f182ffc. There was trouble with the auto submit code and our auto upload code conflicting.
-- There is some weirdness when moving backwards between steps.
What wierdness are you seeing? I think it is wierd you can move backwards anyway and that there are empty steps, but thats a ctools issue :)
-- Need to resolve handling of body fields. For some reason custom form state attributes (the modified entity) gets cleared out between steps for body fields.
How can i replicate this and does your patch to panels context fix this or http://drupal.org/files/fape-panoply-widgets.patch? I wasn't able to replicate it.
Comment #5
beeradb CreditAttribution: beeradb commentedTry moving backwards with an image widget. It doesn't properly re-populate the form fields when moving from step 2 back to step 1 (assuming you changed the image in step 1)
No. That patch is just for improved handling for fieldsets. Currently the body field is special cased in the submitter to save after step 1. It's not optimal, but it works. See the if statement in panopoly_magic_fape_submit().
Comment #6
populist CreditAttribution: populist commentedComment #7
beeradb CreditAttribution: beeradb commentedHere's a patch which fixes moving backwards when editing image fields
Comment #8
populist CreditAttribution: populist commentedLooks good. I tested, committed, and pushed this patch. Marking this issue as fixed since core FAPE support is on the scene, but will require ongoing testing.