[wp-trac] [WordPress Trac] #20383: Strip trailing punctuation with canonical URLs

[WordPress Trac]

#20383: Strip trailing punctuation with canonical URLs
---------------------------------------------+-----------------------------
 Reporter:  nacin                            |       Owner:  SergeyBiryukov
     Type:  defect (bug)                     |      Status:  accepted
 Priority:  normal                           |   Milestone:  4.8
Component:  Canonical                        |     Version:
 Severity:  normal                           |  Resolution:
 Keywords:  has-patch has-unit-tests commit  |     Focuses:
---------------------------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * keywords:  has-patch dev-feedback has-unit-tests => has-patch has-unit-
     tests commit


Comment:

 I think the regex in [attachment:20383.2.patch] might accidentally catch
 international characters in some environments, so I'd like to keep an
 explicit list of punctuation symbols to remove.

 [attachment:20383.3.patch] adds more symbols to the list and more tests.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/20383#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform