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

* Your Email Address
* Spam protection	Enter the letters below: ..__ .__ .___..__ \|[__)[ __ \| [ __ \__\|[__)[_./ \| [_./
Comment	> 2) You should reuse the already existing ics.php. > > > > I don't see how this is possible... Currently this file does nothing > more that download the calendar in the iCal format, the only possible > solution would be to implement the webdav object in this file, but > that's not really a good solution. > > > > Also in ics.php there is exactly the same code as in the api.php. I > choose to only include the code from api.php to minimize redundancy > of the code. > > > > 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... > > > > 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.
Attachment
Watch this ticket