WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#51619 closed defect (bug) (fixed)

Undefined $current_role variable in wp-admin/user-new.php

Reported by: ravipatel Owned by: SergeyBiryukov
Priority: normal Milestone: 5.6
Component: Users Version: 5.6
Severity: normal Keywords: has-patch
Cc: Focuses: coding-standards

Description

Undefined variable $current_role. Applied clean code

Attachments (1)

51619-user-new.php.patch​ (689 bytes ) - added by ravipatel 6 years ago.
Removed undefine variable and manage clean code

Download all attachments as: .zip

Change History (3)

@ravipatel
6 years ago

Removed undefine variable and manage clean code

#1 @SergeyBiryukov
6 years ago

  • Milestone Awaiting Review → 5.6
  • Owner set to SergeyBiryukov
  • Status new → reviewing
  • Summary Coding standard : Undefined varible → Undefined $current_role variable in wp-admin/user-new.php

Hi there, thanks for the patch!

This appears to be a remnant of a variable that was used up to WordPress 2.6.x, when the Add New User form was still in the wp-admin/users.php file, as seen in lines 268, 278, 442 here​.

Since the form was moved into a new file in [9614], the variable appears to be unused.

The instances remaining in wp-admin/users.php were removed in [28296], so this one can be removed too.

#2 @SergeyBiryukov
6 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 49298:

Coding Standards: Remove unused $current_role variable from Add New User form.

Follow-up to [9614], [28296].

Props ravipatel.
Fixes #51619.

Note: See TracTickets for help on using tickets.