#967 closed defect (bug) (fixed)
publish_post is not triggered via xml-rpc or email posting
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | low | |
Severity: | minor | Version: | 1.5 |
Component: | XML-RPC | Keywords: | bg|2nd-opinion bg|has-patch |
Focuses: | Cc: |
Description
publish_post is not triggered when someone uses a program that posts via xml-rpc (w.bloggar for instance) or when someone posts by email
Attachments (1)
Change History (13)
#3
@
20 years ago
Forgot to mention, didn't touch email posting for now, as it has the action 'publish_phone' hooked in. Maybe plugins that add an action for publish_post should silently have a publish_phone action added as well? Would produce the most expected behaviour IMHO. Either that or copy the same code block again? If that was the case it would be reproduced 6 times (and thus a possible candidate for being its own reusable function maybe?)
#4
@
20 years ago
Actually, what is needed is specific XMLRPC hooks. We can not copy hooks made with a particular purpose in mind and expect they'll work the same elsewhere.
For example with this patch, do_trackbacks() wouldn't trackback anything when making a new post, because the field to_ping wouldn't be populated yet.
#6
@
20 years ago
- Keywords bg|2nd-opinion bg|needs-patch added
- Owner changed from anonymous to skippy
- Priority changed from normal to low
- Status changed from new to assigned
#9
@
20 years ago
But according to Ryan, the new workflow uses the same process for all posts, regardless of how they come in. We could have a publish_xmlrpc hook (like publish_phone), for actions specific to that kind of posting, but those are still considered posts and should trigger publish_post.
Uploaded a patch which adds the required hooks. Actually, just lifted the same code from post.php, but haven't had a chance to test it out yet. Also adds hooks for trackbacks, enclosures, and pingbacks if specified. Quick and dirty, and in need of review before committing, but might do the trick. J.