[horde] PHP's Mime Magic Support

[Michael M Slusarz]

Quoting Rob Brandt <bronto at csd-bes.net>:

| Quoting Michael M Slusarz <slusarz at bigworm.colorado.edu>:
| 
| > Quoting Rob Brandt <bronto at csd-bes.net>:
| >
| > | I have been struggling with this for more than a week too.  I learned
| > | yesterday (although this is unconfirmed) that:
| > |
| > | * Despite mime-magic being listed as a requirement in the INSTALL
| doc, it
| > | is not; and
| >
| > This is NOT true.  It is listed under _reccommended_ modules.
|
|        "The following PHP options are either recommended or are required
| by a specific Horde
|         application (as noted in [brackets])."
|           c...
|           d...
|           e...
|           f...
|           g. Mimetype functions {enable-mime-magic}
| 
| The problem is that item g lists no modules that make use of the
| functions,

This is not the location where we really should be saying what modules use 
this function.  It is probably only used by IMP at this point, but in the 
future it may be used by other modules (that's why it's in Horde, not 
IMP).  Notice, it says *recommended* in the header - mimetype functions are 
NOT required for any module at this time so there is no need to mention any 
modules that use it.

| and there is no note that the minimum required version of PHP
| does not support them.

I added a note to INSTALL with this information.

| Any newbie is going to read that and think "I'd
| better put those in" so that any unknown module that needs them will have
| them.

These people should read the INSTALL file more closely.

| And, I would argue, that for a different category of prerequisites
| (required vs. recommended) the item lettering should start over at a. to
| make the differences especially clear.

A valid point.

| Test.php should also make a note
| when the test for mime-type fails, that it is not supported by <4.3.

Also a valid point.

I'll look into fixing these two later.

michael

______________________________________________
Michael Slusarz [slusarz at bigworm.colorado.edu]
The University of Colorado at Boulder