[wp-trac] [WordPress Trac] #47678: Modernize/simplify current_user_can()
WordPress Trac
noreply at wordpress.org
Mon Sep 16 03:53:07 UTC 2019
#47678: Modernize/simplify current_user_can()
-------------------------------------+-------------------------------------
Reporter: jrf | Owner: pento
Type: enhancement | Status: assigned
Priority: normal | Milestone: 5.3
Component: Role/Capability | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch has-unit- | Focuses: performance, coding-
tests commit needs-dev-note | standards
-------------------------------------+-------------------------------------
Comment (by jrf):
@SergeyBiryukov Thanks for committing those patches <3
> I think it's still preferable to show a proper error message on PHP <
5.6, at least until a switch to PHP 7.x is made.
I agree, though to be honest, this would indicate that the PHP version
check is done too late in the process flow, rather than that this patch is
invalid.
All the same, let's leave that for later.
> 1. Could you refresh the patch without changing the function signature
for now?
Done. See the newly uploaded patch. Passing build for the patch: https
://travis-ci.com/WordPress/wordpress-develop/builds/127739735
> 2. Could you create a new ticket for reconsidering the revert of
[45624]?
Would you mind explaining why this should be moved to a new ticket ?
I'd personally rather we keep it in this ticket as comments 23 up to 29
contain a detailed analysis of the impact, which would otherwise need to
be re-iterated completely in a new ticket.
It has already been established that the changes made via this ticket need
a dev-note, as the function signature of a number of methods has changed,
`Walker:walk()` is just one of those methods and should IMO not have
special status just because the VIP platform overloads the method.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47678#comment:59>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list