[#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/2008 (71 days ago)
Due
Updated 06/11/2008 (71 days ago)
Assigned 06/11/2008 (71 days ago)
Resolved 06/11/2008 (71 days ago)
Attachments framework-MIME-fix-rfc822Explode.patch Download
Milestone
Patch No

History
06/11/2008 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/2008 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/2008 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/2008 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/2008 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/2008 Chuck Hagenbuch Taken from Thomas Jarosch
Assigned to Chuck Hagenbuch
 
06/11/2008 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/2008 Chuck Hagenbuch Priority ⇒ 2. Medium
 
06/11/2008 Chuck Hagenbuch Taken from Michael Slusarz
Taken from Horde DevelopersHorde Developers
Assigned to Thomas Jarosch
 
06/11/2008 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/2008 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/2008 Thomas Jarosch Comment #1
New Attachment: framework-MIME-fix-rfc822Explode.patch Download
Patch ⇒
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Fix MIME::rfc822Explode
Type ⇒ Bug
Priority ⇒ 1. Low
State ⇒ Unconfirmed		 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