Tickets :: [#5710] regex problem in imp/lib/MIME/Viewer/html.php

6.0.0-RC7

6/27/26

History
Attachments
Comment
Watch
Download

Summary	regex problem in imp/lib/MIME/Viewer/html.php
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	ericr (at) horde (dot) org
Created	9/10/07 (6865 days ago)
Due
Updated	10/2/07 (6843 days ago)
Assigned	10/2/07 (6843 days ago)
Resolved	10/2/07 (6843 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

185	Jan Schneider	Comment #6 State ⇒ Resolved	Reply to this comment
The bug was actually that one of the regexp replacements appeared two times in the list.

103	Jan Schneider	State ⇒ Assigned

241	Eric Rostetter	Comment #5	Reply to this comment
Sorry, the bug is still a bug in the regex, but this time it displays normal href urls wrong, and not email (mailto) urls... Just wanted to clarify that...

111	Eric Rostetter	Comment #4 State ⇒ Unconfirmed	Reply to this comment
This bug seems to be back again. The following _seems_ to fix it for me. Index: html.php =================================================================== RCS file: /repository/imp/lib/MIME/Viewer/html.php,v retrieving revision 1.75.2.25 diff -r1.75.2.25 html.php 65c65 < '<a target="_blank" ', --- 'a target="_blank" ',

327	Jan Schneider	Comment #3 State ⇒ Resolved Assigned to Jan Schneider	Reply to this comment
Fixed in CVS. We could really use a unit test for this.

217	Jan Schneider	Comment #2 Version ⇒ HEAD	Reply to this comment
This is actually broken in HEAD too.

105	ericr (at) horde (dot) org	Comment #1 Priority ⇒ 2. Medium State ⇒ Unconfirmed Queue ⇒ IMP Summary ⇒ regex problem in imp/lib/MIME/Viewer/html.php Type ⇒ Bug	Reply to this comment
With version 1.75.2.23 (which consolidates some regex patterns) email addresses can show up in the output as "<email@domain.com" instead of "email@domain.com". Reverting to 1.75.2.22 restores the correct display.