Hi Devs, Sergiu,
> > - I wasn't able to create new wiki documents. Is there something special > > I should do? Is it not supported yet? Is Konqueror alone guilty? > > - Also, the error I get doesn't seem right (File already exists), since > > the file I want to create definitely doesn't exist. > Adding pages / spaces is supported and unlike with some other features (like deleting documents) creating documents work pretty ok with most of the clients I tested. You should file a JIRA issue for this, I will test it asap. But I need to install KDE first ;) > > > - Since it uses Basic authentication, we should advertise that > > authentication is not secure, thus it should only be used inside trusted > > intranets, over VLANs or over SSL. > I think this information should go into http://dev.xwiki.org/xwiki/bin/view/Drafts/XWikiWebDAV which is totally out-dated now, i'll start working on it. > > > Code related: > > - Looking a bit over the code, I noticed that we're using only a bit > > from the jackrabit-jcr-server library, and I was wondering if it would > > be possible to refactor all the code that needs it to use something > > else, so that we have one dependency in minus. > > - Also, I noticed that you're using slf4j as the logging framework, > > which doesn't fit with the rest of the XWiki code, since we're using > > commons-logging. Still, this is also the framework used by jackrabbit, > > so we can't get rid of it completely. Asiri, could you update the code > > to use commons-logging? > > - Although there are integration tests, some unit tests would be great. > > I know it's hard for the moment to do proper unit testing, since most of > > the code works closely with the old core, which doesn't contribute to a > > good testing scenario. > > > Will look into these. Also, I'm digging up a patch which will integrate webdav service into xwiki ui as in http://91.121.237.216/xwiki/bin/view/Main/. > > > Congratulations for the good work, Asiri! > Thanks. Btw, is it possible for me to get commit rights for the webdav module ? It's bit painful to go with patches; not for me but for the person i continuously bug... :) > And I forgot one more thing: > - The code is currently in com.xpn.xwiki.plugin.webdav, which I don't > quite agree with. The plugin package is for plugins, and it is somewhat > deprecated (since we're planning to switch completely to components and > remove the notion of 'plugins' per se). Thus, I'd like to rename it to > com.xpn.xwiki.webdav for the moment (except the actual webdav plugin, > which is rightly placed in the plugin package). > +1 - Asiri > > -- > Sergiu Dumitriu > http://purl.org/net/sergiu/ > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
