Do not return subscriptions on findAllCalendars()

[georgehrke]

Signed-off-by: Georg Ehrke developer@georgehrke.com

[georgehrke]

/compile amend /

[raimund-schluessler]

Hm, maybe I did something wrong, but on this branch nextcloud/tasks#942 and after npm install and npm run build I still see subscriptions appearing in the app.

Also, how does the Calendar app load subscriptions? Wouldn't this change also affect it?

[georgehrke]

Also, how does the Calendar app load subscriptions? Wouldn't this change also affect it?

We set a special HTTP Header, so that the server returns Subscription as calendar collections. (so we can fetch cached events from the server)

[georgehrke]

Hm, maybe I did something wrong, but on this branch nextcloud/tasks#942 and after npm install and npm run build I still see subscriptions appearing in the app.

Let me check that right now

[georgehrke]

For some mysterious reason, the server is reporting subscriptions as calendars, although it should report them as subscription: https://github.com/sabre-io/dav/blob/master/lib/CalDAV/Subscriptions/Plugin.php#L34

[georgehrke]

Ok, now this is embarrassing 🙈
Commit incoming ...

[georgehrke]

This should obviously only be done in the calendar, not in cdav:
09207e0

[georgehrke]

@raimund-schluessler Please check again

[raimund-schluessler]

@raimund-schluessler Please check again

Subscriptions don't show up anymore 👍