Closed (fixed)
Project:
Edit
Version:
7.x-1.0-alpha6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
19 Mar 2013 at 14:32 UTC
Updated:
29 May 2014 at 18:18 UTC
Jump to comment: Most recent
Comments
Comment #1
wim leersConfirmed, and expanded.
Comment #2
wim leersSolving this is now blocked on #1947802: Clean up Edit's PHP to make it more maintainable: Drupal 8-style in-place editor definitions.
Comment #3
wim leersInstead of this being blocked on #1947802: Clean up Edit's PHP to make it more maintainable: Drupal 8-style in-place editor definitions, I decided that it's better to solve the different parts of that problem in subissues. Hence this is one such subissue.
I'm happy to report that by cleaning up fape.inc, restructuring it, diving deep into Node & Entity APIs and carefully comparing code flow and object dumps, I was able to solve this problem :)
For future reference: most of the issue was cleaning up (fape.inc is now highly consistent instead of highly inconsistent), the key part of the solution to this particular problem happened in
_edit_node_object_prepare(). I also managed to get rid of the entity reloading after saving in pages.inc, and fixed a bug in the re-rendering of the node author.http://drupalcode.org/project/edit.git/commit/7dd2f86
Comment #5
longtom commented[non-issue posted in incorrect location]