Needs review
Project:
jEditable inline content editing
Version:
7.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Jan 2012 at 23:51 UTC
Updated:
22 Apr 2018 at 04:26 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
netboss commentedHi,
I'm experiencing the same reported issue here.
Logged-in as admin (user/1), I'm able to successfully edit and save the text field, but as as soon as I left the page and come back, or reload it, the changes disappear.
Also, trying to edit a field logged-in as another role (authenticated user, which have already been granted the "use jeditable" permission), the field does not update and I receive an "Access denied" message instead.
Server: Apache/2.2.14 (Ubuntu)
PHP: 5.3.2-1ubuntu4.11
Drupal: 7.11-dev
And by the way, great module!
~!v@n
Comment #2
kevin.mcnamee@mailbox.org commentedThe problem is two-fold. First, the code is only performing a node_save(). It does not do any form validation as would occur when editing the field in a node edit form. This means that node_save will fail if the value input cannot be stored in the database field, e.g.the code will not stop you trying to put text in an integer field.
Secondly, the value that is displayed is simply the text that you entered. The code does not do a node_load after the node_save to be sure the value was actually stored in the database. Hence the refresh issue.
/Kevin
Comment #3
wlofgren commentedPlacing the node in an entity wrapper seems to work for me.
Comment #4
wlofgren commented