[dev] [cvs] commit: imp compose.php imp/lib Compose.php

Michael M Slusarz slusarz at mail.curecanti.org
Sun Nov 27 18:03:52 PST 2005


Quoting Marko Djukic <mdjukic at horde.org>:

> mdjukic     2005-11-27 16:33:06 PST
>
>   Modified files:
>     .                    compose.php
>     lib                  Compose.php
>   Log:
>   Don't need the setting up of the imp_contents and headers in the 
> library. Use the library function resumeDraft()
>
>   Revision  Changes    Path
>   2.877     +8 -52     imp/compose.php
>   1.155     +2 -10     imp/lib/Compose.php
>
>   Chora Links:
>   http://cvs.horde.org/diff.php/imp/compose.php?r1=2.876&r2=2.877&ty=u
>   http://cvs.horde.org/diff.php/imp/lib/Compose.php?r1=1.154&r2=1.155&ty=u

Please don't use globals for objects like $imp_headers or 
$imp_contents.  They are not static objects like prefs or browser - in 
fact there is a very good chance that there may be more than one 
imp_headers object defined per pageload. This is the reason we have the 
singleton() methods.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]


More information about the dev mailing list