[#6896] Fix MIME::rfc822Explode
Summary Fix MIME::rfc822Explode
Queue Horde Framework Packages
Queue Version HEAD
Type Bug
State Resolved
Priority 2. Medium
Owners Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester Thomas Jarosch <thomas (dot) jarosch (at) intra2net (dot) com>
Created 06/11/08 (645 days ago)
Due
Updated 06/11/08 (645 days ago)
Assigned 06/11/08 (645 days ago)
Resolved 06/11/08 (645 days ago)
Attachments framework-MIME-fix-rfc822Explode.patch Download
Milestone
Patch No

History
06/11/08 Chuck Hagenbuch Comment #9 Reply to this comment
I've removed it from docs/CHANGES, thanks. You run it by going to 
framework/MIME/tests and then executing:



pear rt



(or pear run-tests)
06/11/08 Thomas Jarosch Comment #8 Reply to this comment
for a simple example test.
Thanks, Chuck. How do I execute it? :-)



btw: I think it's not worth mentioning in the "CHANGES" file as the bug

was introduced only recently when Michael cleaned up the function,

the new code was luckily not part of the Horde 3.2 release.


06/11/08 Chuck Hagenbuch Comment #7
State ⇒ Resolved		 Reply to this comment
Fixed, thanks. See 
http://cvs.horde.org/co.php?r=1.1&f=framework%2FMIME%2Ftests%2Fbug_6896.phpt 
for a simple example test.
06/11/08 CVS Commit Comment #6 Reply to this comment
Changes have been made in CVS for this ticket:

http://cvs.horde.org/diff.php/framework/MIME/MIME.php?r1=1.190&r2=1.191&ty=u
http://cvs.horde.org/diff.php/horde/docs/CHANGES?r1=1.1103&r2=1.1104&ty=u
06/11/08 CVS Commit Comment #5 Reply to this comment
Changes have been made in CVS for this ticket:

http://cvs.horde.org/co.php/framework/MIME/tests/bug_6896.phpt?r=1.1
06/11/08 Chuck Hagenbuch Assigned to Chuck Hagenbuch
Taken from Thomas Jarosch
 
06/11/08 Thomas Jarosch Comment #4 Reply to this comment
Thomas, can you please provide a test showing what this fixes?
I don't know how to write a unit test in PHP but basically this 
stopped working:



print_r(MIME::rfc822Explode("addr1@email.com, addr2@email.com"));



The resulting array should contain two entries instead of one.


06/11/08 Chuck Hagenbuch Priority ⇒ 2. Medium
 
06/11/08 Chuck Hagenbuch Taken from Michael Slusarz
Taken from Horde DevelopersHorde Developers
Assigned to Thomas Jarosch
 
06/11/08 Chuck Hagenbuch Comment #3
Assigned to Horde DevelopersHorde Developers		 Reply to this comment
Thomas, can you please provide a test showing what this fixes?
06/11/08 Thomas Jarosch Comment #2
State ⇒ Assigned
Assigned to Michael Slusarz		 Reply to this comment
Michael, you did the last change on that file, assigning to you.


06/11/08 Thomas Jarosch Comment #1
State ⇒ Unconfirmed
New Attachment: framework-MIME-fix-rfc822Explode.patch Download
Patch ⇒
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Fix MIME::rfc822Explode
Type ⇒ Bug
Priority ⇒ 1. Low		 Reply to this comment
Hello together,



while debugging why multiple email addresses in Turba stopped working

if found out MIME::rfc822Explode() is currently broken.

Attached patch fixes the issue.



Thomas