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

6.0.0-beta1

12/20/25

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	09/10/2007 (6676 days ago)
Due
Updated	10/02/2007 (6654 days ago)
Assigned	10/02/2007 (6654 days ago)
Resolved	10/02/2007 (6654 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

10/02/2007 05:42:18 PM	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.

10/02/2007 03:10:10 PM	Jan Schneider	State ⇒ Assigned

10/02/2007 01:27:24 PM	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...

10/02/2007 01:26:11 PM	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" ',

09/11/2007 07:58:32 AM	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.

09/11/2007 07:56:21 AM	Jan Schneider	Comment #2 Version ⇒ HEAD	Reply to this comment
This is actually broken in HEAD too.

09/10/2007 05:34:10 PM	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.