#62012 closed defect (bug) (fixed)
Remove obsolete conditional in wp-admin/user-new.php
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Users | Keywords: | good-first-bug has-patch commit |
Focuses: | administration | Cc: |
Description
As a result of mu:641, mu:1529, [12722], and [16294], there is a redundant condition in wp-admin/user-new.php
:
$username = $user_details->user_login; ... if ( null != $username && ... ) {
The null != $username
part was initially added for $username = $wpdb->get_var()
in mu:641, but now that $username
is set to $user_details->user_login
, it will never be null
in this specific line, so this part of the conditional can be removed.
Attachments (1)
Change History (8)
This ticket was mentioned in PR #7311 on WordPress/wordpress-develop by @akshat2802.
10 months ago
#1
- Keywords has-patch added; needs-patch removed
#3
@
10 months ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 58999:
@SergeyBiryukov commented on PR #7311:
10 months ago
#4
Thanks for the PR! Merged in r58999.
This ticket was mentioned in PR #7340 on WordPress/wordpress-develop by @iflairwebtechnologies.
10 months ago
#5
@martin.krcho commented on PR #7340:
10 months ago
#6
@iflair this has already been merged in https://github.com/WordPress/wordpress-develop/commit/a30b9b444fbdd30d711f7e597794eb958764ea56.
You can close this issue.
@mukesh27 commented on PR #7340:
10 months ago
#7
Already merged.
Trac ticket: https://core.trac.wordpress.org/ticket/62012
This PR removes Obsolete conditional in wp-admin/user-new.php