Skip to content
This repository has been archived by the owner on Oct 31, 2018. It is now read-only.

Sort list of calendars by calendarorder #884

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

intrixius
Copy link

basic solution to fix request owncloud/calendar#742
(No GUI implementation, so currently only possible to change the calendar order in the database table 'clndr_calendars')

basic solution to fix request owncloud/calendar#742
(No GUI implementation, so currently only possible to change the calendar order in the database table 'clndr_calendars')
@intrixius
Copy link
Author

at first this seemed to work, but now I'm adding more calendars (with a corresponding calendarorder), but somehow they are not sorted anymore correctly...
Is there any other place where it gets resorted?

Edit: it seems that the sorting only happens on owned calendars, not shared ones...

Instead of sorting the calendars in the SQL query (which only affected owned calendars and shared ones), now usort is used.
@intrixius
Copy link
Author

I have changed the way of sorting, now also shared calendars are sorted

@DeepDiver1975 DeepDiver1975 added this to the backlog milestone Oct 13, 2015
@DeepDiver1975
Copy link
Contributor

From my pov sorting the calendars is not a server side job but something to be done on the client side.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants