[dev] [cvs] commit: imp/lib IMP.php

[Michael M Slusarz]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael M Slusarz <slusarz at horde.org>:
>
>> slusarz     2006-08-07 07:42:38 PDT
>>
>>  Modified files:
>>    lib                  IMP.php
>>  Log:
>>  Easier way to get default namespace information.
>>
>>  Revision  Changes    Path
>>  1.581     +10 -3     imp/lib/IMP.php
>>
>>  Chora Links:
>>  http://cvs.horde.org/diff.php/imp/lib/IMP.php?r1=1.580&r2=1.581&ty=u
>
> Do you want to check for an empty value like stated in the phpdoc?   
> Then you should better check with empty($mailbox). Note that at   
> least one callee is calling the method with '' as the mailbox   
> parameter.
> If you want to explicitely check for null, then you need the ===   
> operator instead.

Thanks for the catch - I was simply missing the third equal sign.  (''  
is definitely different than null in terms of expected return)

michael

___________________________________
Michael Slusarz [slusarz at horde.org]