[dev] [imp] DIMP not showing most headers

[Michael M Slusarz]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael M Slusarz <slusarz at horde.org>:
>
>> Quoting Liam Hoekenga <liamr at umich.edu>:
>>
>>> DIMP isn't showing the From, Cc, To headers (or rather, they're
>>> blank).  I just see the Subject and Date.  I see this in my logs..
>>>
>>> [Fri Jun 20 16:31:12 2008] [warn] mod_fcgid: stderr: PHP Notice:
>>> Undefined property:  stdClass::$display in
>>> /path/to/horde/dimp/lib/Views/ShowMessage.php on line 78
>>>
>>> Are they related?  This is DIMP from CVS + Horde 3.2, IMP 4.2
>>
>> DIMP 1.x (and MIMP 1.1x) are always going to require the most
>> up-to-date versions of IMP - otherwise, there would be no way we
>> could fix bugs in DIMP/MIMP.  Therefore, you need to update IMP to
>> the latest version in the FW_3 branch.  I added something to DIMP's
>> INSTALL to say this, but I guess I never committed it.
>
> Uh, when did we decide this? This breaks with any release policy we
> ever had. As long as DIMP and IMP are separate applications we have
> to keep BC in the stable branch.

We might as well copy all of imp/lib into DIMP or MIMP right now then.  Or do no bugfixing in DIMP and MIMP and be happy with the versions now released and get on with further development in HEAD.  Because it is *impossible* to fix bugs in IMP without affecting MIMP or DIMP.  We previously decided that dimp and mimp are not separate applications.  They are add-ons to imp.  They appear in separate directories/CVS modules simply because of historical reasons.  But we can't treat them like separate applications because they *aren't* separate applications.  I was never working under the impression that once IMP 4.2 was released that the API was frozen between these applications.

The new search code in IMP is thus unable to be used with DIMP.  No spellcheck fixes (which require changes in the UI part of things) can be done.  Various MIME escaping/encoding fixes - which there have been a bunch lately - are not BC between 4.2 and 4.2.1.  If this is unacceptable, then we should release as DIMP 1.1 instead.

michael

--
___________________________________
Michael Slusarz [slusarz at horde.org]