Opened 4 years ago

Closed 3 years ago

#11211 closed defect (bug) (fixed)

make_clickable issue with brackets and trailing ,

Reported by: westi Owned by:
Priority: normal Milestone: 2.9
Component: Formatting Version: 2.9
Severity: normal Keywords: has-patch make_clickable
Cc: westi

Description

The following is incorrectly linked:

Example: WordPress, test (some text), I love example.com (http://example.com), it is brilliant

Example: WordPress, test (some text), I love example.com (<a href="http://example.com)" rel="nofollow">http://example.com)</a>, it is brilliant

Attachments (1)

make_clickable.11211.diff​ (1.1 KB) - added by filosofo 3 years ago.

Download all attachments as: .zip

Change History (4)

comment:1   westi — 4 years ago

Test case added to wordpress-tests

​http://svn.automattic.com/wordpress-tests/wp-testcase/test_includes_formatting.php

test_real_world_examples()

filosofo — 3 years ago

comment:2   filosofo — 3 years ago

  • Keywords has-patch make_clickable added; needs-patch removed

Patch passes the TestMakeClickable for me, including "test_real_world_examples"

comment:3   westi — 3 years ago

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

(In [12250]) Ensure that make_clickable links correctly when the link is placed in brackets and has trailing punctuation. Fixes #11211 props filosofo.

Note: See TracTickets for help on using tickets.