Looks like we might need to a different approach for parsing the XML to be cross-browser compatible.

For now, ownCloud uses a different but less accurate implementation (it assumes that the prefix is always "d:") but works for our use case: https://github.com/owncloud/core/blob/jsocclient/core/js/files/ie8davclient.js#L116

Ideal would be to fine-tune that approach to be more accurate.

Currently we only get the body as string.

Would be nice to have it automatically parsed (or have a helper method) to make it possible to get the actual error message.

I wasn't able to use the webinterface to my calendars in my local nextcloud instance, because of an underscore in the url.

Changing the line to:
var parts = url.match(/^(?:([A-Za-z]+):)?(\/{0,3})([0-9.\-A-Za-z\_]+)(?::(\d+))?(?:\/([^?#]*))?(?:\?([^#]*))?(?:#(.*))?$/); did fix the issue.

I added a "\_" to the hostname-part of the regex.

See https://github.com/evert/davclient.js/blob/master/lib/client.js#L205

Right now the developer has to check if the request was successful in the then "block".
It would be easier for developers, if instead of resolving, davclient.js would automatically reject the promise if the response code is not a 2xx one.

The regex in the parseURL function can't parse IPv6 addresses.

corresponding line: https://github.com/evert/davclient.js/blob/fc7519d2c81f689dba55d0a4504803c5774741ff/lib/client.js#L364

Steps to reproduce

1. create an owncloud instance accepting a domain with _ (for example test_123.thismycooldomain.org
2. Install calendar app
3. try to create new calendar --> Uncaught (in promise) TypeError: Cannot read property '0' of null in Chrome debugger console

var parts = url.match(/^(?:([A-Za-z]+):)?(/{0,3})([0-9.-A-Za-z]+)(?::(\d+))?(?:/([^?#]))?(?:?([^#]))?(?:#(.*))?$/);

should be

var parts = url.match(/^(?:([A-Za-z]+):)?(/{0,3})([0-9.-A-Za-z_]+)(?::(\d+))?(?:/([^?#]))?(?:?([^#]))?(?:#(.*))?$/);

@evert after #21 is merged, would be good to tag a release as v0.1 or so.

We are now using bower to pull dependencies in owncloud core and it's better to point to an actual tag than a commit id: https://github.com/owncloud/core/blob/master/bower.json#L31

Hi @PVince81 ,

I thought I would contact you through a github issue. You are the number 1 contributer to this project, doubling my own contributions.

How would you feel about officially taking over this project? I don't think I will take it anywhere, any time soon. Even if you don't spend a lot of time on this, I feel this project is in better hands with you instead of me. How would you feel about this? If you are interested, let me know how you want to deal with this repo.

I can transfer ownership to your account, I can add it to the fruux org, I can keep it under my account or place it anywhere else you'd like.

Lemme know!

Evert

Could use karma + jasmine

jQuery 3 Upgrade

See the full upgrade 3.0 guide

Deferred

.then()

• /lib/client.js

Event

.bind()

• /lib/client.js
• /tests/client.test.js

@evert bower complains that the tag 0.1.0 doesn't match package.json which you forgot to update.

If you have no objection I'm going to make a 0.1.1 tag with the matching package.json update.
(unless you want to reposition the tag, but usually that's not good practice)