Make WordPress Core

Opened 17 years ago

Closed 10 years ago

#5725 closed enhancement (wontfix)

Auto-convert Contributor's new post to draft

Reported by: toydi's profile toydi Owned by: josephscott's profile josephscott
Milestone: Priority: normal
Severity: minor Version: 2.3.2
Component: XML-RPC Keywords: has-patch needs-testing
Focuses: Cc:

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 17 years ago.
a patch with the proposed enhancement

Download all attachments as: .zip

Change History (9)

@toydi
17 years ago

a patch with the proposed enhancement

#1 @FFEMTcJ
16 years ago

  • Keywords has-patch added
  • Milestone changed from 2.9 to 2.8

#2 @Denis-de-Bernardy
16 years ago

  • Keywords needs-testing added; wp-app removed

#3 @ryan
16 years ago

  • Owner changed from anonymous to josephscott

#4 @westi
15 years ago

  • Cc westi added

#5 @Denis-de-Bernardy
15 years ago

  • Milestone changed from 2.8 to 2.9

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

#6 @westi
15 years ago

Still needs testing.

Patch needs a tidy up too.

#7 @ryan
15 years ago

  • Milestone changed from 2.9 to Future Release

#8 @markoheijnen
10 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Closing this ticket since AtomPub was removed in [21818]

Note: See TracTickets for help on using tickets.