[cvs] [Wiki] changed: NewMIMELib

[Chuck Hagenbuch]

chuck  Thu, 03 May 2007 11:34:10 -0700

Modified page: http://wiki.horde.org/NewMIMELib
New Revision:  1.8
Change log:  Troll doesn't exist anymore, add some of my opinions

@@ -24,11 +24,19 @@
 **MIME_Structure** - No major changes

 **MIME_Headers** - Remove deprecated code.  Other than that, no major
changes.

-**MIME_Contents** - I believe that the benefits of code sharing is fairly
slight when faced with the disadvantages of BC issues.  Therefore, I
propose removing MIME_Contents completely and instead maintaining the code
entirely within IMP (and Troll).  Since the output should really be
controlled at the app level, it doesn't make too much sense to try to have
a shared library handle it.  Remove caching code from MIME_Contents. 
Possibly have IMP's MIME_Part or MIME_Viewer extending class do the
retrieval of the text of a part from the server instead of MIME_Contents.
+**MIME_Contents** - I believe that the benefits of code sharing is fairly
slight when faced with the disadvantages of BC issues.  Therefore, I
propose removing MIME_Contents completely and instead maintaining the code
entirely within IMP.  Since the output should really be controlled at the
app level, it doesn't make too much sense to try to have a shared library
handle it.  Remove caching code from MIME_Contents.  Possibly have IMP's
MIME_Part or MIME_Viewer extending class do the retrieval of the text of a
part from the server instead of MIME_Contents.
+
+//I agree - especially since Troll is now gone and MIMP and DIMP use IMP,
this belongs entirely in IMP. --ChuckHagenbuch//

 **MIME_Viewer** - MIME_Viewer should be extended to do more than just
render the MIME_Part in html format.  It should do a better job of
indicating the parts underneath it, whether a part is an 'attachment' or
not, prepare the summary information for the part (instead of
MIME_Contents).  Once again, some of this additional functionality may be
best implemented by extending MIME_Viewer in IMP.
+
+//I think that we should retain the {{Viewer}} concept as something
separate from the MIME libraries, but that things like message/rfc822 and
encrypted part handling probably should be email-specific.
--ChuckHagenbuch//

 ++ Relevant Tickets

 [http://bugs.horde.org/ticket/?id=3580 Bug #3580]
+
+++ Naming Notes
+
+All code that remains in the framework should be changed to have the prefix
{{Horde_Mime}} (instead of just {{MIME}} - the case change is also
intentional). The code should also be updated to use PHP 5 features.