[dev] Turba browse.php patch

Jan Schneider jan at horde.org
Wed Oct 27 12:43:34 PDT 2004


Zitat von Michael Rubinsky <mike at theupstairsroom.com>:

> Unless I'm missing something, I don't see why we should  check
> $curSource['export']) in browse.php.  Doing it this way prevents an
> addressbook from being browsed if 'export' = false.  According to
> sources.php.dist the export variable is used to allow or disallow an
> addressbook from being exported to a .csv or other type of text file.
> Setting it to false currently stops it from being browsed as well.  This
> small patch fixes this...

The only thing you miss is the historical reason for this. The rationale
behind that is that we want to catch all *huge* address books with that
configuration, like Bigfoot or campus/company LDAP sources. You don't want
to export nor browse these directories.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/


More information about the dev mailing list