WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#20159 closed defect (bug) (fixed)

Typo in sites/users network admin bulk actions

Reported by: duck_ Owned by: duck_
Priority: normal Milestone: 3.4
Component: Network Admin Version: 3.0
Severity: normal Keywords:
Cc: Focuses: multisite

Description 

if ( $_GET['action'] != -1 || $_POST['action2'] != -1 )
    $doaction = $_POST['action'] != -1 ? $_POST['action'] : $_POST['action2'];

Checking $_GET for action and $_POST for action2. The line can be dropped since it will always be true and it doesn't matter if $doaction is set to -1.

Looks like it's from [13609] and [13918].

Attachments (1)

20159.diff​ (1.3 KB ) - added by duck_ 15 years ago.

Download all attachments as: .zip

Change History (2)

@duck_
15 years ago

#1 @duck_
15 years ago

  • Owner set to duck_
  • Resolution → fixed
  • Status new → closed

In [20092]:

Drop unnecessary if-statements which also have some strange logic. Fixes #20159.

Note: See TracTickets for help on using tickets.