The "autoclose" option for views megarow prevents any changes from actually being saved. I don't have a solution, but I think I've tracked down the issue. In function node_form_submit_build_node (node.pages.inc line 509) drupal calls form_execute_handlers prior to returning the node object.
The submit handler for views_megarow calls drupal_exit - I think this call is short-circuiting the normal node_submit process.
Repro steps:
New drupal install - minimal profile
enable views_ui, views_megarow
create content type
create node of that type
Create new view of content showing fields with format Megarow Table
In Megarow settings, check "automatically close the megarow".
add a NID field
Add megarows link field with the links "edit|node/[nid]/edit"
Save the view.
Go to the view and click on edit.
Make some changes to the node.
Click save.
The megarow will close, but nothing else happens.
Comment | File | Size | Author |
---|---|---|---|
#2 | views_megarow-autoclose-node-save-2402179-0.patch | 944 bytes | sepehr.sadatifar |
Comments
Comment #1
delzhand CreditAttribution: delzhand commentedEven stranger, with this same setup, checkboxes can be checked and saved, but not unchecked - regardless of autoclose status.
Comment #2
sepehr.sadatifar CreditAttribution: sepehr.sadatifar commented@delzhand this worked for me, pls test.
Comment #3
delzhand CreditAttribution: delzhand commentedWorks like a charm for the main issue. I suspected the answer was going to be something like this based on the comments for node_form_submit_build_node.
Thanks!
I suspect the second issue is actually unrelated. Maybe something about php's handling of 0/false/null. I'm looking into that this morning.
Comment #4
delzhand CreditAttribution: delzhand commentedConfirmed, second issue is unrelated to Views Megarow.
https://www.drupal.org/node/2405973
Comment #6
ArtusamakGood catch, thanks for the report & the patch, it's committed: http://cgit.drupalcode.org/views_megarow/commit/?id=9bfa06ad4a075a14dce3...