WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#19899 closed enhancement (fixed)

Redundant logic in filter_SSL() function

Reported by: deltafactory Owned by: duck_
Milestone: 3.4 Priority: normal
Severity: trivial Version: 3.3.1
Component: Multisite Keywords: has-patch commit
Focuses: Cc:

Description

I'm not sure if there's a reason it's written this way, but the logic in this line seems redundant.

http://core.trac.wordpress.org/browser/trunk/wp-includes/ms-functions.php#L1947

if ( 'http' === $arrURL['scheme'] && 'https' !== $arrURL['scheme'] )

In the interest of simplification, shouldn't this line be:

if ( 'http' === $arrURL['scheme'] )

Attachments (1)

ms-functions.php.patch (446 bytes) - added by deltafactory 2 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 scribu2 years ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 3.4

Yes, it is redundant.

comment:2 duck_2 years ago

  • Owner set to duck_
  • Resolution set to fixed
  • Status changed from new to closed

In [19759]:

Remove redundant logic from filter_SSL(). Props deltafactory. Fixes #19899.

Note: See TracTickets for help on using tickets.