From e537a7f7f4a3b2e985813198b1bcc41bcc9cbce5 Mon Sep 17 00:00:00 2001
From: Marc Durepos <marc@bemade.org>
Date: Fri, 8 Nov 2024 15:47:39 -0500
Subject: [PATCH] caldav_sync: getting ready for launch of 0.6.0

---
 caldav_sync/README.rst                    | 39 ++++++++++++++++++++---
 caldav_sync/static/description/index.html | 16 ++++++++++
 caldav_sync/tests/test_calendar.py        | 38 ++++------------------
 3 files changed, 57 insertions(+), 36 deletions(-)

diff --git a/caldav_sync/README.rst b/caldav_sync/README.rst
index 301aabd..21979f5 100644
--- a/caldav_sync/README.rst
+++ b/caldav_sync/README.rst
@@ -3,7 +3,7 @@ CalDAV Synchronization
 
 Bemade Inc.
 
-Copyright (C) 2023-June Bemade Inc. (<https://www.bemade.org>).
+Copyright (C) 2023-June Bemade Inc. (https://www.bemade.org).
 Author: Marc Durepos (Contact : marc@bemade.org)
 
 This program is under the terms of the GNU Lesser General Public License (LGPL-3)
@@ -44,13 +44,44 @@ Usage
 Technical Details
 -----------------
 
-- The module extends the `calendar.event` model to add CalDAV synchronization
+* The module extends the `calendar.event` model to add CalDAV synchronization
   functionality.
-- It uses the `icalendar` library to format events and the `caldav` library to
+* It uses the `icalendar` library to format events and the `caldav` library to
   interact with CalDAV servers.
-- Polling for changes on the CalDAV server can be triggered manually by
+* Polling for changes on the CalDAV server can be triggered manually by
   triggering the scheduled action in Odoo.
 
+Change Log
+----------
+
+17.0.0.6.0
+^^^^^^^^^^
+
+* Fixed an issue where synchronizing events created duplicate events on every sync.
+* Completely revamped and synchronization of recurring events in both directions.
+
+  * Making a recurring event in Odoo correctly creates the recurring event on the server.
+  * Modifying the base event of a recurrence with "all events" or "future events" in
+    Odoo reflects correctly on the server.
+  * Modifying a non-base event correctly updates on the server in all 3 modes (this
+    event only, all events, future events).
+  * Modifying a base recurring event on the CalDAV server correctly updates the events
+    on Odoo after a synchronization.
+  * Deleting a whole recurring sequence from Odoo correctly deletes the sequence from
+    the CalDAV server.
+  * Deleting a single event or a whole recurring sequence on the CalDAV server
+    correctly synchronizes to Odoo after a synchronization.
+
+* CalDAV (iCalendar) UIDs are now correctly shared among events of a same recurrence in
+  Odoo. This corrects a number of issues around updating and deleting events from both
+  the Odoo and CalDAV server side.
+
+Issues & Requests
+-----------------
+
+Please submit issues on Bemade's Gitlab at https://git.bemade.org/bemade/bemade-addons
+or via our website at https://www.bemade.org.
+
 License
 -------
 
diff --git a/caldav_sync/static/description/index.html b/caldav_sync/static/description/index.html
index a09ded5..193bd1d 100644
--- a/caldav_sync/static/description/index.html
+++ b/caldav_sync/static/description/index.html
@@ -32,6 +32,11 @@
         <p><strong>License:</strong> GNU Lesser General Public License (LGPL-3)</p>
         <h2>Overview</h2>
         <p>The CalDAV Synchronization module for Odoo allows users to synchronize their calendar events with CalDAV servers. This enables seamless integration of Odoo calendar with external applications like Apple Calendar or Thunderbird.</p>
+        <h2>Warning</h2>
+        <p>This module is in early development stages and should not be considered stable.
+        Please test it on an unimportant calendar before using it with your important data.
+        We are working hard to make improvements based on user feedback, so please reach
+        out if you run into any issues or need a specific feature.</p>
         <h2>Features</h2>
         <ul>
             <li>Synchronize Odoo calendar events with CalDAV servers.</li>
@@ -64,6 +69,17 @@
         </ol>
         <h2>Technical Details</h2>
         <p>The module extends the <code>calendar.event</code> model to add CalDAV synchronization functionality. It uses the <code>icalendar</code> library to format events and the <code>caldav</code> library to interact with CalDAV servers. Polling for changes on the CalDAV server can be triggered manually by triggering the scheduled action in Odoo.</p>
+        <h2>Support</h2>
+        <p>If you have any issues with the module or feature requests, please reach out
+            to us by creating an issue on our Gitlab at
+            <a href="https://git.bemade.org/bemade/bemade-addons">
+                https://git.bemade.org/bemade/bemade-addons
+            </a>
+            or by submitting a request on our website at
+            <a href="https://www.bemade.org">
+                https://www.bemade.org</a>
+            </a>
+        </p>
     </div>
     <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js"></script>
     <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.16.0/umd/popper.min.js"></script>
diff --git a/caldav_sync/tests/test_calendar.py b/caldav_sync/tests/test_calendar.py
index 1cc57c0..fde0324 100644
--- a/caldav_sync/tests/test_calendar.py
+++ b/caldav_sync/tests/test_calendar.py
@@ -45,9 +45,10 @@ def _patch_caldav_with_events_from_ics(ics_paths, user, last_modified=None):
         mock_calendar.side_effect = calendar_side_effect
 
         def event_by_uid_side_effect(self, uid):
-            for event in self.events:
+            for event in self.events():
                 if str(event.icalendar_component.get("uid")) == uid:
                     return event
+            return DEFAULT
 
         ical_events = []
         if ics_paths:
@@ -59,8 +60,10 @@ def _patch_caldav_with_events_from_ics(ics_paths, user, last_modified=None):
                 ical_events.append(icalendar.Calendar.from_ical(ical_content))
         if last_modified:
             for event in ical_events:
-                event["last-modified"] = last_modified
-                event["dtstamp"] = last_modified
+                for subcomponent in event.subcomponents:
+                    if subcomponent.name == "VEVENT":
+                        subcomponent["last-modified"] = icalendar.vDate(last_modified)
+                        subcomponent["dtstamp"] = icalendar.vDate(last_modified)
 
         base_events = [event for event in ical_events if not event.get("recurrence-id")]
         for base_event in base_events:
@@ -271,35 +274,6 @@ class TestCalendarEvent(TransactionCase, CaldavTestCommon):
             )
         )
 
-    def test_multiple_user_attendees_event_to_server_create(self):
-        with self._patch_all_3_users_davclients() as (_, mock_calendar):
-            self._create_multi_user_test_event()
-            self.assertEqual(mock_calendar.add_event.call_count, 3)
-
-    def test_event_to_server_delete(self):
-        with self._patch_all_3_users_davclients() as (_, mock_calendar):
-            self._create_multi_user_test_event().unlink()
-            self.assertEqual(
-                mock_calendar.event_by_uid.return_value.delete.call_count, 3
-            )
-
-    def test_event_to_server_update(self):
-        with self._patch_all_3_users_davclients() as (_, mock_calendar):
-            self._create_multi_user_test_event().write(
-                {"start": datetime.now() + timedelta(days=14)}
-            )
-            self.assertEqual(mock_calendar.save_event.call_count, 3)
-
-    def test_recurrent_event_to_server(self):
-        with self._patch_all_3_users_davclients() as (_, mock_calendar):
-            self._create_multi_user_test_event().write(
-                {
-                    "recurrency": True,
-                }
-            )
-            args = mock_calendar.save_event.call_args
-            self.assertEqual(mock_calendar.save_event.call_count, 3)
-
     @contextmanager
     def _patch_all_3_users_davclients(self):
         with patch("caldav.DAVClient") as MockDAVClient: