Opened 5 years ago

Last modified 4 years ago

#5725 new enhancement

Auto-convert Contributor's new post to draft

Reported by: toydi Owned by: josephscott
Priority: normal Milestone: Future Release
Component: XML-RPC Version: 2.3.2
Severity: minor Keywords: has-patch needs-testing
Cc: westi

Description

Currently, wp-app.php implementation honors client-supplied app:draft element. If a contributor submits a new Atom entry without app:draft element set to 'yes', it rejects the request.

[Current Response]
401 Unauthorized
"Sorry, you do not have the right to edit/publish new posts."

Imagine when a contributor forgets to set app:draft to 'yes' and submits a new post. By reading the error message, the contributor may mistakenly think that he is not allowed to submit posts at all. A proposed workaround has been submitted as ticket #5723.

There is an alternate proposal:
rather than rejecting the request, server may ignore client-supplied app:draft element and always treats a contributor's new post as draft. Based on RFC5023 Section 13.1.1, server may ignore client-supplied app:draft element.

Attachments (1)

wp-app.diff (1.4 KB) - added by toydi 5 years ago.
a patch with the proposed enhancement

Download all attachments as: .zip

Change History (8)

toydi5 years ago

a patch with the proposed enhancement

  • Keywords has-patch added
  • Milestone changed from 2.9 to 2.8
  • Keywords needs-testing added; wp-app removed

comment:3   ryan4 years ago

  • Owner changed from anonymous to josephscott
  • Cc westi added
  • Milestone changed from 2.8 to 2.9

Punting per discussion during the WP meet-up...

Still needs testing.

Patch needs a tidy up too.

comment:7   ryan4 years ago

  • Milestone changed from 2.9 to Future Release
Note: See TracTickets for help on using tickets.