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

[bugs at bugs.horde.org]

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.