WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#38190 closed defect (bug) (fixed)

Minor fixes for plugin-install.php

Reported by: ChrisWiegman Owned by: helen
Priority: normal Milestone: 4.7
Component: General Version: 4.7
Severity: trivial Keywords: has-patch
Cc: Focuses: administration

Description

Fixes two minor issues in wp-admin/plugin-install.php

1.) Conditionals on the tab check
2.) Duplicate if statements (no break in between)

Attachments (1)

plugin-install.diff​ (693 bytes ) - added by ChrisWiegman 10 years ago.
Diff to fix minor plugin install differences.

Download all attachments as: .zip

Change History (7)

@ChrisWiegman
10 years ago

Diff to fix minor plugin install differences.

#1 @ChrisWiegman
10 years ago

  • Severity normal → trivial

#2 @ChrisWiegman
10 years ago

  • Keywords has-patch added

This ticket was mentioned in ​Slack in #core by chriswiegman. ​View the logs.
10 years ago

This ticket was mentioned in ​Slack in #core by helen. ​View the logs.
10 years ago

#5 @helen
10 years ago

  • Milestone Awaiting Review → 4.7

Introduced in [38172]. Worth fixing so nobody later is left wondering if there's some secret WordPressy reason there's a duplicate conditional like that.

#6 @helen
10 years ago

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

In 39336:

Plugin install: De-duplicate a conditional, introduced in [38172].

props ChrisWiegman.
fixes #38190.

Note: See TracTickets for help on using tickets.