[dev] Re: [cvs] commit: midas/lib Campaigns.php

[Jan Schneider]

Zitat von Marko Djukic <mdjukic at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
> > Zitat von Marko Djukic <tech at oblo.com>:
> >
> > > mdjukic     2003/09/17 14:35:09 PDT
> > >
> > >   Modified files:
> > >     lib                  Campaigns.php
> > >   Log:
> > >   autocreate dir if first entry into db, don't do notification here
> >
> > Shouldn't this be the default case in this method?
>
> is that a question as in "should we change it to default = true"? now

Yes.

> it's set
> to false. and it has been so since the very early versions. i guess my
> own gut
> feeling would be for an app to be explicitly told to go autocreate rather
> than
> it happening always.

I don't really care. I just recognized that we step by step switch all calls
to autocreate the folders, so it looks like this is now the "default" case.
That is probably because the sql driver now bails out if you want to create
a file in a not-existing folder. This didn't happen before.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft