XML- RPC, blogger_editPost, Publish Status not working

[bobby_drk]

In the blogger_editPost function in xmlrpc.php is not working properly. Currently you cannot change publish status. It looks like we are missing a line of code.

$post_status = ($publish) ? 'publish' : 'draft';

When I inserted this line into the function before the

$postdata = compact('ID', 'post_content', 'post_title', 'post_category', 'post_status', 'post_excerpt');

I was able to gain control of the publish status.

[redsweater]

I agree there appears to be a problem with the blogger_editPost method.

Something like what bobby_drk suggested sounds like a good fix, but you should be careful to add the proposed line a bit earlier, before the first line that tests against post_status

if ( ('publish' == $post_status) && !current_user_can('publish_posts') )

I suspect this has not generally caused problems because very, very few clients probably use the blogger_editPost method. bobby_drk, if you can help it I would recommend not using it, and instead using the metaweblog variant. That will get things working for you instantly without having to wait for the patch to go through.

[josephscott]

This brings up the question of keeping the blogger.* methods up to date with the changes/updates/improvements that have been done with metaWeblog.* methods. I'm inclined to say no and direct people to using the metaWeblog.* methods instead.

[mdawaffe]

If the methods don't work, we should either fix them or remove them.

[markoheijnen]

I agree with mdawaffe. The method should be fixed or blogger API should be removed.
The patch does needs a look and unit tests before it get committed

[wonderboymusic]

In 34573:

XML-RPC: In wp_xmlrpc_server::blogger_editPost(), make use of the $publish arg (the 6th arg passed to the method) to specify publish or draft. Restores the arg, which I removed in [31092], because it was unused cruft.

Props mdawaffe.
Fixes #10764.