Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Contacts Birthdays' showing up as task list #5504

Closed
andreasjacobsen93 opened this issue Jun 21, 2017 · 16 comments · Fixed by #16503
Closed

'Contacts Birthdays' showing up as task list #5504

andreasjacobsen93 opened this issue Jun 21, 2017 · 16 comments · Fixed by #16503
Assignees
Milestone

Comments

@andreasjacobsen93
Copy link
Member

Steps to reproduce

When using contacts and calendar, the 'Contacts Birthday' calendar is created.

Expected behaviour

Should not be shown as task list.

Actual behaviour

Is shown as task list, see nextcloud/tasks#88

Server configuration

Operating system: Ubuntu 17.04

Web server: Apache

Database: MySQL

PHP version: PHP 7

Nextcloud version: 12.0.0.29

Updated from an older Nextcloud/ownCloud or fresh install: Fresh install

Where did you install Nextcloud from: Nextcloud.com

@ethanopp
Copy link

ethanopp commented Jul 7, 2017

+1

@MorrisJobke
Copy link
Member

This should be handled in the tasks app, because this is just a normal calendar. And we can't do anything "clever" on the server side to list it in half of the requests and in the other half (tasks) not.

@raimund-schluessler
Copy link
Member

This should be handled in the tasks app, because this is just a normal calendar. And we can't do anything "clever" on the server side to list it in half of the requests and in the other half (tasks) not.

That is not really true. The tasks app only shows calendars which have the VTODO option set. See here: https://github.com/nextcloud/tasks/blob/master/js/app/services/calendarservice.js#L109-L122

So the birthday calendar should only be VEVENT and not VTODO. This is what 6eb1bc5 was about. If the birthday calendar also holds VTODO this is a server bug, because Tasks app cannot do anything against that.

@raimund-schluessler
Copy link
Member

@andreasjacobsen93 Could you please check whether or not your birthday calendar is VEVENT or also VEVENT,VTODO ?

@MorrisJobke
Copy link
Member

Thanks for the explanation. Didn't knew that. Sorry for the noise.

@MorrisJobke
Copy link
Member

@georgehrke @rullzer Mind to have a quick look into it?

@MorrisJobke
Copy link
Member

Looks like we already set this properly:

'components' => 'VEVENT',

Could it be that the calendar was updated to also hold the VTODO?

@MorrisJobke MorrisJobke added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Aug 30, 2017
@raimund-schluessler
Copy link
Member

Would be good if @andreasjacobsen93 could confirm if the calendar holds VTODO (which I really think).

@leongrdic
Copy link

I just checked my Nextcloud 12 installation and the contact_birthdays calendars had components set to VEVENT,VTODO.
Running a simple MySQL query solved this problem:

update oc_calendars set components="VEVENT" where displayname="Contact birthdays";

So yeah, this is a server issue.

@skjnldsv
Copy link
Member

skjnldsv commented Jun 4, 2019

@georgehrke shall we take position here?
Either close it or set it as to develop? :)

@georgehrke
Copy link
Member

We already fixed that as Morris pointed out, but I’m not sure whether we added a repair step for it. Will check tomorrow when I’m back home

@skjnldsv skjnldsv removed stale Ticket or PR with no recent activity labels Jun 12, 2019
@0xbb
Copy link

0xbb commented Jul 22, 2019

I just checked my Nextcloud 12 installation and the contact_birthdays calendars had components set to VEVENT,VTODO.
Running a simple MySQL query solved this problem:

update oc_calendars set components="VEVENT" where displayname="Contact birthdays";

So yeah, this is a server issue.

Thanks for this fix!
Nextcloud 16 still seem to add VTODO to birthday calendars.

@georgehrke georgehrke self-assigned this Jul 22, 2019
@georgehrke georgehrke added this to the Nextcloud 17 milestone Jul 22, 2019
@georgehrke georgehrke added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jul 22, 2019
@georgehrke

This comment has been minimized.

@georgehrke

This comment has been minimized.

@georgehrke georgehrke added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Jul 22, 2019
@georgehrke
Copy link
Member

Fix is in #16503

@0xbb
Copy link

0xbb commented Jul 22, 2019

Thank you!

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

Successfully merging a pull request may close this issue.

9 participants