Tickets :: Comment on [#3032] WebDAV / iCal support for Kronolith

* Your Email Address
* Spam protection	Enter the letters below: .. .. .. ,.__ \|\|\/\|\|__\| \./ [__) \__\|\| \|\| \| \| [__)
Comment	>> If you mean that I should have named webdav.php ics.php, I didn't do > >> that because I didn't want to overwrite existing code. If this is > >> committed into CVS, I also don't see the use of the ics.php anymore, > >> because the WebDAV module can do exactly the same and more... > > > > Exactly, that's what I was talking about. There is no need to keep > the current ics.php if webdav.php provides the same features, but we > also don't want another script. > > > >> 3) You should already check there if RPC/webdav.php exists at all for > >> BC reasons, because it has only been added in Horde 3.1. > >> > >> I don't really understand what you mean, I've written this patch > >> using the latest CVS from Kronolith, but with the existing stable > >> Horde 3.0.6/3.0.7. This RPC / webdav is already present there, > >> however, I haven't seen any component yet that uses it. > > > > My fault, I thought I only added the webdav driver in 3.1. But it > still doesn't make sense to make this work with Horde 3.0, because > the 3.1 driver already implements PROPFIND for example. So you should > rely on Horde 3.1, reuse what's there, detect it, and simply send an > HTTP error to the client if you recognized an older version.
Attachment
Watch this ticket