[dev] Re: [cvs] commit: luxor/lib/Driver sql.php luxor/lib Tagger.php

Mike Cochrane mike at graftonhall.co.nz
Sat Feb 22 01:01:54 PST 2003


> ----- Message from jan at horde.org ---------
> 
> Zitat von Mike Cochrane <mike at graftonhall.co.nz>:
> 
> > mikec       2003/02/20 19:59:16 PST
> >
> >   Modified files:
> >     lib/Driver           sql.php
> >     lib                  Tagger.php
> >   Log:
> >   Store the last modified times of files in the database. Not used by one
> > day should be used to only update modified files.
> 
> Do you have further plans on fileId() and createFileId() that I don't see
> currently? Wasn't fileId()'s purpose to always return a file's id if one
> exists yet or not? Now we always have to check if a file has an id already
> while it used to create one automagically before your change.
> 
> Jan.

Yeah, it was but calling getFiletime() every time we wanted a fileid seemed like
a lot of overkill, not we only called it when we need it.

getFiletime() will have a larger overhead when a cvs backend it being used and
it's not the HEAD tag we're dealing with.

- Mike :-)



More information about the dev mailing list