[wp-trac] [WordPress Trac] #38645: Improve roles reinit when switching between sites
WordPress Trac
noreply at wordpress.org
Fri Nov 4 09:26:05 UTC 2016
#38645: Improve roles reinit when switching between sites
-------------------------------------------+------------------------------
Reporter: johnjamesjacoby | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Role/Capability | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch needs-testing early | Focuses: multisite
-------------------------------------------+------------------------------
Comment (by flixos90):
Replying to [comment:7 johnjamesjacoby]:
> Unless we go back and rename `WP_User::for_ blog()`, it's hard to
endorse anything else.
I agree with @knutsp that we should use `for_site()`. We shouldn't be
locked in by the old naming conventions, so everything new should be
properly named. Eventually we will be able to rename many of the old
things over (long) time, but introducing new functionality in that way
seems wrong to me.
Another issue that just came to mind is whether that function should exist
in that form at all: Let's imagine `WP_Roles` is extended by a future
`WP_Network_Roles`, what would a `for_site()` / `for_blog()` method do? It
might be a better idea to find a more generic name.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38645#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list