[wp-trac] [WordPress Trac] #29920: Autosave breaks if post revision fields contains a value that isn't a string
WordPress Trac
noreply at wordpress.org
Sat Oct 11 00:17:26 UTC 2014
#29920: Autosave breaks if post revision fields contains a value that isn't a
string
--------------------------+------------------------------
Reporter: mattheu | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Autosave | Version:
Severity: normal | Resolution:
Keywords: | Focuses:
--------------------------+------------------------------
Comment (by mattheu):
I think you're right that this is only intended to be used internally.
Probably lucky it worked as much as it did.
The use case here is that I have a small plugin for meta & taxonomy
revisions. This data may be a string or array. I did look at your patch on
#20564 and my plugin is somewhat based on this. I'll comment on that
ticket separately with feedback.
I guess the real problem I ran into was to get my meta revisions show in
the revisions UI. Thinking about it a bit more - I think that a better
solution would be to make the value returned by `wp_get_revision_ui_diff`
filterable.
A separate issue is the triggering of autosaves/revisions for meta
changes. This actually happens in a few places, and again a better
solution might be an action/filter that could be used to handle this.
I would love to see something like this in core - and I think that pushing
for better hooks that allow for a solid plugin solution would be a good
way forward.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29920#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list