WordPress.org

Make WordPress Core

Opened 7 weeks ago

Last modified 6 weeks ago

#49872 reviewing defect (bug)

PHP Notice: Undefined index: path in /wp-includes/functions.php on line 905

Reported by: jbouganim Owned by: SergeyBiryukov
Milestone: 5.5 Priority: normal
Severity: normal Version: 4.4
Component: Pings/Trackbacks Keywords: has-patch needs-unit-tests
Focuses: Cc:

Description

Some URLs here do not have a 'path' and thus throw this notice.
Instead of only checking on /wp-includes/functions.php L904:

<?php
if ( false !== $url_parts )

should also check

<?php
if ( false !== $url_parts && !empty($url_parts['path']) )

Attachments (1)

49872.patch (616 bytes) - added by mukesh27 6 weeks ago.
Initial patch.

Download all attachments as: .zip

Change History (6)

@mukesh27
6 weeks ago

Initial patch.

#1 @mukesh27
6 weeks ago

  • Component changed from General to Pings/Trackbacks
  • Keywords has-patch added
  • Severity changed from minor to normal
  • Version changed from 5.4 to 4.4

Hi, @jbouganim,

Welcome to WordPress Trac! Thanks for the ticket.

Added initial patch as suggested.

#2 @Otto42
6 weeks ago

Can you provide an example of a URL that would not have a path component?

#3 @jbouganim
6 weeks ago

@Otto42 Sure.

<?php
$url_parts = parse_url("https://redirect.viglink.com?key=2fb2035e82f6fb61e704bd9&u=https%3A%2F%2Fwww.michaels.com%2Fsearch%3Fq%3Dbandana");

Will not have a path index.

#4 @Otto42
6 weeks ago

  • Keywords needs-unit-tests added

Great! So probably need some unit tests added to go with this as well.

#5 @SergeyBiryukov
6 weeks ago

  • Milestone changed from Awaiting Review to 5.5
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.