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

Birthday-calendar: properly handle deletion of addressbook #11305

Closed
ulifan opened this issue Sep 20, 2018 · 5 comments
Closed

Birthday-calendar: properly handle deletion of addressbook #11305

ulifan opened this issue Sep 20, 2018 · 5 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: caldav Related to CalDAV internals feature: contacts menu feature: dav needs info

Comments

@ulifan
Copy link

ulifan commented Sep 20, 2018

Steps to reproduce

  1. Be a normal user: Open a new nextcloud-acount
  2. Enter (or import) some contacts with birthdays entries
  3. Verify that birthdays are being displayed in the birthday-calendar
  4. Create another contacts-database (e.g. name it "Contacts_2")
  5. Enter (or import) more contacts but for now into "Contacts_2".

Remark: The following step is done because in my example i assume, that the entries into "Contacts_2" are more complete and more accurate!

  1. Delete the first contacts-database (only "Contacts_2" are left)
  2. Check for birthdays in the birthday-calendar: All birthdays (even from "Contacts" and "Contacts_2")
    are still there! No chance to remove the birthdays from the deleted "Contacts". If you
    had double entries (one from "Contacts" the second from "Contacts_2" you will see two entries.

Remark: Birthdays can be removed then only if you delete the birthday-calendar, but in this case
you run into the new problem: Not being able to recreate any birthday-calendar (as a user)

Expected behaviour

With deleting "Contacts", the birthdays should be deleted as well, but the birthday-calendar should remain there!

Actual behaviour

With deleting "Contacts", all birthday-data is still in the birthday-calendar, and cannot be deleted.

Nextcloud version: 13.0.0

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #3003 (Disable birthday contacts calendar generation for user), #4451 ("Delete Calendar" Issue), #3759 (Contact delete tooltip), #410 (Birthday calendar not generated for shared contacts), and #9334 (Calendar of contact's birthdays are not editable after re-sync).

@skjnldsv skjnldsv added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Jun 12, 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 Sep 14, 2019
@georgehrke georgehrke changed the title Deleting contacts not necessarily delets birthday-entries in birthday calendar Birthday-calendar: properly handle deletion of addressbook Sep 14, 2019
@georgehrke georgehrke added this to the Nextcloud 18 milestone Sep 14, 2019
@rullzer rullzer removed this from the Nextcloud 18 milestone Dec 9, 2019
@rxkris
Copy link

rxkris commented Jan 30, 2020

Still existing in 18.0.0, using the following steps:

  • Create an addressbook
  • Add contacts with birthday
  • Delete addressbook
  • Birthday events still available

Reason: birthday-calender sync only iterates existing adressbooks (./apps/dav/lib/CalDAV/BirthdayService.php => function: syncUser)

CC: @nickvergessen

@ChristophWurst ChristophWurst added the feature: caldav Related to CalDAV internals label Dec 17, 2021
@szaimen
Copy link
Contributor

szaimen commented Jan 9, 2023

Hi, please update to 24.0.8 or better 25.0.2 and report back if it fixes the issue. Thank you!

@szaimen szaimen added needs info 0. Needs triage Pending check for reproducibility or if it fits our roadmap and removed 1. to develop Accepted and waiting to be taken care of labels Jan 9, 2023
@szaimen szaimen closed this as completed Mar 6, 2023
@dranuhl
Copy link

dranuhl commented Jun 12, 2023

This issue still exists with NextCloud 26.0.2, Contacts 5.3.0 and Calendar 4.3.5.

@szaimen
Copy link
Contributor

szaimen commented Jun 12, 2023

Hi, please open a new issue with up-to-date information. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: caldav Related to CalDAV internals feature: contacts menu feature: dav needs info
Projects
None yet
Development

No branches or pull requests

9 participants