[dev] Re: dev Digest, Vol 892, Issue 5

[Michael Rubinsky]

Ooops.  Seems pretty obvious now that you say that ;)  Any objections to 
creating a separate config parameter for browse and export?

Thanks for clearing that up!
Mike

--On Wednesday, October 27, 2004 4:52 PM -0700 dev-request at lists.horde.org 
wrote:

> From: Jan Schneider <jan at horde.org>
> Subject: Re: [dev] Turba browse.php patch
> To: dev at lists.horde.org
> Message-ID: <20041027214334.vgylxsj604c44c4s at jan.dip.ammma.net>
> Content-Type: text/plain;        charset=ISO-8859-15; 
format="flowed"
>
> 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.
>