[wp-trac] [WordPress Trac] #20299: Preview changes on a published post makes all post meta "live"

[WordPress Trac]

#20299: Preview changes on a published post makes all post meta "live"
-------------------------------------------------+-------------------------
 Reporter:  jakemgold                            |       Owner:
     Type:  defect (bug)                         |  adamsilverstein
 Priority:  normal                               |      Status:  assigned
Component:  Revisions                            |   Milestone:  Future
 Severity:  major                                |  Release
 Keywords:  has-patch needs-testing dev-         |     Version:  3.3.1
  feedback                                       |  Resolution:
                                                 |     Focuses:
                                                 |  administration
-------------------------------------------------+-------------------------

Comment (by azaozz):

 Replying to [comment:48 adamsilverstein]:

 Looking at 20299.4.patch, there are some inconsistencies:
 - Autosave has nothing to do with previews, maybe better to patch it in a
 separate ticket. Also, the patch just "pokes a hole" in it letting plugins
 pass through arbitrary data. Maybe that needs some structuring.
 - `$( document.getElementById( meta ) ).val()` should probably be `$( '#'
 + meta  ).val()`, or `document.getElementById( meta ).value` if you don't
 want jQuery.
 - Namespaces are not supported in PHP 5.2, fatal error :)

 Having a patch that makes core dependent on a plugin feels off. I
 understand it is just for testing, but thinking we need a better way.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/20299#comment:50>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform