[dev] Re: [patch] Define kronolith isStored

[Chuck Hagenbuch]

Let's keep dev discussions on the list when possible...

Quoting Roel Gloudemans <roel at gloudemans.info>:

> The best place to implement the exists call is on the lowest level 
> possible (to gain the best performance). So it should be implemented 
> in sql.php, mcal.php etc.

Sure.

> Except in sql.php the getByUID call is not implemented. For the other 
> drivers; is my assumption correct that eventID == UID? (it would make 
> sense to me for at least the ical driver)

For ical, sure. Though unfortunately the ical.php driver isn't actually 
used yet; it was my attempt to clean up the remote calendar code and 
put it into its own actual driver.

I think Kolab works the way you describe, but I might be wrong. Mcal is 
the same as SQL, but is basically a deprecated driver at this point - 
the limitations it has are too much to make it possible to support 
syncing and the other more advanced features we're getting in to now.

In a sense you're doing a very specific search with a true or false 
result. Wouldn't be much more work to return the ids of events that 
match a search for UID="foo". How about that angle?

-chuck

-- 
"But she goes not abroad in search of monsters to destroy." - John 
Quincy Adams