WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#9088 closed defect (bug) (fixed)

string replacement changed to incorrect regex replacement

Reported by: pne Owned by: anonymous
Priority: normal Milestone: 2.7.2
Component: General Version:
Severity: normal Keywords: has-patch commit
Cc: matt@…

Description

I saw that from 2.7 (r10049) to 2.7.1 (r10482), wp-admin/link-manager.php changed by turning 

str_replace('www.', '', $short_url)

into 

preg_replace('/^www./i', '', $short_url)

I think that the regex should be 

/^www\./i

with a backslash before the dot, since dot is special in regexes ('match any character'), which is probably not what is intended here.

Attachments (1)

9088.diff​ (623 bytes) - added by sivel 4 years ago.

Download all attachments as: .zip

Change History (4)

sivel — 4 years ago

comment:1 sivel — 4 years ago

  • Cc matt@… added
  • Keywords has-patch commit added; regex removed

comment:2 automattor — 4 years ago

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

(In [10717]) Escape dot in regex. Props pne, sivel. fixes #9088 for trunk

comment:3 ryan — 4 years ago

(In [10718]) Escape dot in regex. Props pne, sivel. fixes #9088 for 2.7

Note: See TracTickets for help on using tickets.