Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I noticed, that any link that contains "(" or ")" is not clickable. I believe it has to do with regular expressions, but I'm unable to solve this by myself as I'm hopeless in dealling with them.
Comment | File | Size | Author |
---|---|---|---|
#15 | drupal7.filter-url-parenthesis-15.patch | 672 bytes | bobodrone |
#13 | drupal8.filter-url-parenthesis.13.patch | 1.9 KB | sun |
#10 | drupal_filter-728380-10.patch | 1.25 KB | Albert Volkman |
#6 | drupal.filter-url-parenthesis.6.patch | 1.28 KB | sun |
#2 | url-parenthesis.patch | 1.15 KB | mr.baileys |
Comments
Comment #1
sunComment #2
mr.baileysI tinkered with this for half an hour but was unable to tweak the regex so it accepted parenthesis inside the URL. FWIW, I did write the test case which should pass.
Comment #3
sunComment #5
sunPostponing on #161217: URL filter breaks generated href tags
Comment #6
sunAccording to RFC2396 chapter 2.3. Unreserved Characters, "(" and ")" can be used unescaped, as long as the context allows the characters to appear.
Our context allows it:
Comment #8
codekarate CreditAttribution: codekarate commentedThis appears to be a related issue. I noticed that URLS with exclamation points "!" also fail. RFC2396 chapter 2.3. Unreserved Characters states that "!" should also be allowed.
Comment #9
sunLet's incorporate that. Needs to be fixed in HEAD first.
Comment #10
Albert Volkman CreditAttribution: Albert Volkman commentedRe-roll of @sun's patch for 8.x. Still broken, though.
Comment #11
sunComment #13
sunExclamation marks have been fixed elsewhere already.
Let's see whether attached patch passes the tests.
Comment #15
bobodrone CreditAttribution: bobodrone commentedI run into this bug today and have backported the patch for Drupal 7 (without test).
/ BoboDrone
Comment #16
mr.baileysMarked #1868664: "(" or ")" character in text field truncates hyperlink as duplicate.
@bobodrone: thanks for your work on backporting the patch. However, this will need to be fixed in D8 before being backported toD7, so setting version back to D8.
Comment #17
mr.baileys#13: drupal8.filter-url-parenthesis.13.patch queued for re-testing.
Comment #18
Hanno CreditAttribution: Hanno commentedfixed in patch #1657886: Filter "Convert URLs into links" doesn't support multilingual web addresses
Comment #19
Andre-Bis this already in D7? when yes, with which version did this patch move in?