Make WordPress Core

Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#38961 closed defect (bug) (invalid)

REST API: User with `promote_users` cap can't add existing users to a site

Reported by: ocean90 Owned by: jeremyfelt
Priority: normal Milestone:
Component: REST API Version: 4.7
Severity: normal Keywords: has-unit-tests needs-patch
Cc: Focuses: multisite

Description

Adding an existing user to a site should be done by performing a PUT request to the user resource on the other site, see ticket:38526:7. This works if the current user can edit users.

But core also allows users with the promote_users capability to add existing users:

Attachments (1)

38961.patch (1.6 KB ) - added by ocean90 10 years ago.

Download all attachments as: .zip

Change History (9)

@ocean90
10 years ago

#1 @rmccue
10 years ago

  • Keywords needs-patch added

I'd be tempted to leave this until 4.8 and take a holistic look at multisite behaviour across all resources to make sure it works well. (We fixed #38526 in 4.7 since it was a bug, not to add full multisite functionality.)

@jeremyfelt Is this important enough to pull into 4.7?

This ticket was mentioned in Slack in #core by helen. View the logs.


10 years ago

#3 @jeremyfelt
10 years ago

  • Milestone Awaiting Review4.7
  • Owner set to jeremyfelt
  • Status newassigned

Taking a look at this now for 4.7.

#4 @jeremyfelt
10 years ago

  • Milestone 4.74.8

I started poking at this, but it seems like something that should be thought through some more first. Let's leave it at edit_users for now and then follow up in early 4.8.

This ticket was mentioned in Slack in #core-restapi by jeremyfelt. View the logs.


10 years ago

#6 @larsonreever
9 years ago

take a holistic look at multisite behaviour across all resources to make sure it works well.

Last edited 9 years ago by larsonreever (previous) (diff)

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


9 years ago

#8 @jeremyfelt
9 years ago

  • Milestone 4.8
  • Resolutioninvalid
  • Status assignedclosed

The original functionality for adding existing users was disabled in 4.7.3 via [40106] as it was not part of a clear strategy. Let's close this ticket in favor of #39544, which aims to improve the users endpoint as a whole.

Note: See TracTickets for help on using tickets.