[Tickets #5601] Re: is_null() check in imp/lib/MIME/Contents.php

bugs at bugs.horde.org bugs at bugs.horde.org
Tue Aug 7 19:30:24 UTC 2007


DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=5601
-----------------------------------------------------------------------
 Ticket             | 5601
 Updated By         | Michael Slusarz <slusarz at horde.org>
 Summary            | is_null() check in imp/lib/MIME/Contents.php
 Queue              | IMP
 Version            | 4.1.4
 Type               | Bug
-State              | Feedback
+State              | Resolved
 Priority           | 1. Low
 Owners             | Michael Slusarz
-----------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2007-08-07 12:30) wrote:

> Seems like it would be better to trap the error earlier, at the point 
> that $this->_message gets set to null (or never gets set to 
> not-null).

In HEAD/4.2 we do a much better job of trying to catch precisely this in
IMP_Contents' constructor.

> But I'm not strongly opposed to putting the check in. 
> Michael, any opinion?

I'll go ahead and put it in for 4.1.5, but not HEAD.  In HEAD, if this
ever does occur, we need to catch it earlier anyway.



More information about the bugs mailing list