From 6dbcc524b8897cbc18c883b763c936f74353842e Mon Sep 17 00:00:00 2001
From: Thibaut Born <thibaut.born@kuleuven.be>
Date: Thu, 20 Jun 2024 13:12:35 +0200
Subject: [PATCH 1/3] make event_listing act consistent

context: event_listing didn't always show the results from the collection.
---
 plone/app/event/browser/event_listing.py | 32 ++++++++++++++++--------
 1 file changed, 22 insertions(+), 10 deletions(-)

diff --git a/plone/app/event/browser/event_listing.py b/plone/app/event/browser/event_listing.py
index e01de705..9b9fd089 100644
--- a/plone/app/event/browser/event_listing.py
+++ b/plone/app/event/browser/event_listing.py
@@ -66,20 +66,30 @@ def __init__(self, context, request):
             ).isoformat()
 
         if self.mode is None:
-            self.mode = "day" if self._date else "future"
+            if self._date:
+                self.mode = "day"
+            elif self.is_start_or_end_in_query():
+                self.mode = "all"
+            else:
+                self.mode = "future"
 
         self.uid = None  # Used to get all occurrences from a single event. Overrides all other settings  # noqa
 
-    @property
-    def show_filter(self):
-        ret = True
+    def is_start_or_end_in_query(self): 
         if self.is_collection:
             ctx = self.default_context
             query = queryparser.parseFormquery(ctx, ctx.query)
             if "start" in query or "end" in query:
-                # Don't show the date filter, if a date is given in the
-                # collection's query
-                ret = False
+                return True
+        return False
+
+    @property
+    def show_filter(self):
+        ret = True
+        if self.is_start_or_end_in_query():
+            # Don't show the date filter, if a date is given in the
+            # collection's query
+            ret = False
         return ret
 
     @property
@@ -150,12 +160,14 @@ def events(self, ret_mode=RET_MODE_ACCESSORS, expand=True, batch=True):
                 ctx, ctx.query, sort_on=sort_on, sort_order=sort_order
             )
             custom_query = self.request.get("contentFilter", {})
-            if "start" not in query or "end" not in query:
-                # ... else don't show the navigation bar
+            # only use custom end or start dates, if none are used by the collection
+            if "start" not in query and "end" not in query:
                 start, end = self._start_end
                 start, end = _prepare_range(ctx, start, end)
                 custom_query.update(start_end_query(start, end))
-            res = ctx.results(batch=False, brains=True, custom_query=custom_query)
+                res = ctx.results(batch=False, brains=True, custom_query=custom_query)
+            else:
+                res = ctx.results(batch=False, brains=True)
             if expand:
                 # get start and end values from the query to ensure limited
                 # listing for occurrences

From 8f125c817c061588b8506d7d81e3fdd3b85528a7 Mon Sep 17 00:00:00 2001
From: Thibaut Born <thibaut.born@kuleuven.be>
Date: Mon, 24 Jun 2024 17:29:43 +0200
Subject: [PATCH 2/3] restucture code event_listing

context: make code more readable by seperating usecases
---
 plone/app/event/browser/configure.zcml   |  22 +-
 plone/app/event/browser/event_listing.py | 289 ++++++++++++-----------
 2 files changed, 172 insertions(+), 139 deletions(-)

diff --git a/plone/app/event/browser/configure.zcml b/plone/app/event/browser/configure.zcml
index ea49f92b..cf31a1d9 100644
--- a/plone/app/event/browser/configure.zcml
+++ b/plone/app/event/browser/configure.zcml
@@ -23,10 +23,30 @@
       menu="plone_displayviews"
       title="Event listing"
       />
+    <browser:page
+      name="event_listing"
+      for="plone.app.contenttypes.interfaces.ICollection"
+      class=".event_listing.CollectionEventListing"
+      template="event_listing.pt"
+      permission="zope2.View"
+      layer="..interfaces.IBrowserLayer"
+      menu="plone_displayviews"
+      title="Event listing"
+      />
+    <browser:page
+      name="event_listing"
+      for="plone.app.contenttypes.interfaces.IFolder"
+      class=".event_listing.FolderEventListing"
+      template="event_listing.pt"
+      permission="zope2.View"
+      layer="..interfaces.IBrowserLayer"
+      menu="plone_displayviews"
+      title="Event listing"
+      />
   <browser:page
       name="event_listing"
       for="*"
-      class=".event_listing.EventListing"
+      class=".event_listing.BaseEventListing"
       template="event_listing.pt"
       permission="zope2.View"
       layer="..interfaces.IBrowserLayer"
diff --git a/plone/app/event/browser/event_listing.py b/plone/app/event/browser/event_listing.py
index 9b9fd089..dd2f4cb4 100644
--- a/plone/app/event/browser/event_listing.py
+++ b/plone/app/event/browser/event_listing.py
@@ -4,6 +4,7 @@
 from plone.app.contenttypes.behaviors.collection import ISyndicatableCollection
 from plone.app.event import _
 from plone.app.event.base import _prepare_range
+from plone.app.event.base import default_timezone
 from plone.app.event.base import expand_events
 from plone.app.event.base import get_events
 from plone.app.event.base import guess_date_from
@@ -24,30 +25,16 @@
 from zope.contentprovider.interfaces import IContentProvider
 
 
-class EventListing(BrowserView):
+class BaseEventListing(BrowserView):
     def __init__(self, context, request):
-        super().__init__(context, request)
+        super(BrowserView, self).__init__(context, request)
 
         self.now = now = localized_now(context)
 
-        # Try to get the default page
-        default = get_default_page(context)
-        self.default_context = context[default] if default else context
-
-        self.is_collection = False
-        if ISyndicatableCollection:
-            self.is_collection = ISyndicatableCollection.providedBy(
-                self.default_context
-            )
-
         # Request parameter
         req = self.request.form
 
-        b_size = int(req.get("b_size", 0))
-        if not b_size and self.is_collection:
-            collection_behavior = ISyndicatableCollection(self.default_context)
-            b_size = getattr(collection_behavior, "item_count", 0)
-        self.b_size = b_size or 10
+        self.b_size = int(req.get("b_size", 10))
         self.b_start = int(req.get("b_start", 0))
         self.orphan = int(req.get("orphan", 1))
         self.mode = req.get("mode", None)
@@ -66,32 +53,12 @@ def __init__(self, context, request):
             ).isoformat()
 
         if self.mode is None:
-            if self._date:
-                self.mode = "day"
-            elif self.is_start_or_end_in_query():
-                self.mode = "all"
-            else:
-                self.mode = "future"
-
-        self.uid = None  # Used to get all occurrences from a single event. Overrides all other settings  # noqa
-
-    def is_start_or_end_in_query(self): 
-        if self.is_collection:
-            ctx = self.default_context
-            query = queryparser.parseFormquery(ctx, ctx.query)
-            if "start" in query or "end" in query:
-                return True
-        return False
+            self.mode = "day" if self._date else "future"
 
     @property
     def show_filter(self):
-        ret = True
-        if self.is_start_or_end_in_query():
-            # Don't show the date filter, if a date is given in the
-            # collection's query
-            ret = False
-        return ret
-
+        return True
+    
     @property
     def date(self):
         dt = None
@@ -101,33 +68,36 @@ def date(self):
             except TypeError:
                 pass
         return dt
-
+    
     @property
     def _start_end(self):
         start, end = start_end_from_mode(self.mode, self.date, self.context)
         return start, end
-
-    def _get_events(self, ret_mode=RET_MODE_ACCESSORS, expand=True):
-        context = self.context
+    
+    @view.memoize
+    def events(self, ret_mode=RET_MODE_ACCESSORS, expand=True, batch=True):
+        res = self._get_events(ret_mode, expand=expand)
+        if batch:
+           res = Batch(res, size=self.b_size, start=self.b_start, orphan=self.orphan)
+        return res
+    
+    def get_query_keywords(self):
         kw = {}
-        if self.uid:
-            # In this case, restrict search for single event
-            kw["UID"] = self.uid
+        if self.path:
+            kw["path"] = self.path
         else:
-            if self.path:
-                kw["path"] = self.path
-            else:
-                # Search current and subsequent folders
-                kw["path"] = "/".join(context.getPhysicalPath())
-
-            if self.tags:
-                kw["Subject"] = {"query": self.tags, "operator": "and"}
+            # Search current and subsequent folders
+            kw["path"] = "/".join(self.context.getPhysicalPath())
 
-            if self.searchable_text:
-                kw["SearchableText"] = self.searchable_text
+        if self.tags:
+            kw["Subject"] = {"query": self.tags, "operator": "and"}
 
-        # kw['b_start'] = self.b_start
-        # kw['b_size']  = self.b_size
+        if self.searchable_text:
+            kw["SearchableText"] = self.searchable_text
+        return kw
+    
+    def _get_events(self, ret_mode=RET_MODE_ACCESSORS, expand=True):
+        kw = self.get_query_keywords()
 
         start, end = self._start_end
 
@@ -136,7 +106,7 @@ def _get_events(self, ret_mode=RET_MODE_ACCESSORS, expand=True):
         if self.mode in ("past", "all"):
             sort_reverse = True
         return get_events(
-            context,
+            self.context,
             start=start,
             end=end,
             sort=sort,
@@ -145,46 +115,11 @@ def _get_events(self, ret_mode=RET_MODE_ACCESSORS, expand=True):
             expand=expand,
             **kw,
         )
-
+    
     @view.memoize
     def events(self, ret_mode=RET_MODE_ACCESSORS, expand=True, batch=True):
         res = []
-        if self.is_collection:
-            ctx = self.default_context
-            # Whatever sorting is defined, we're overriding it.
-            sort_on = "start"
-            sort_order = None
-            if self.mode in ("past", "all"):
-                sort_order = "reverse"
-            query = queryparser.parseFormquery(
-                ctx, ctx.query, sort_on=sort_on, sort_order=sort_order
-            )
-            custom_query = self.request.get("contentFilter", {})
-            # only use custom end or start dates, if none are used by the collection
-            if "start" not in query and "end" not in query:
-                start, end = self._start_end
-                start, end = _prepare_range(ctx, start, end)
-                custom_query.update(start_end_query(start, end))
-                res = ctx.results(batch=False, brains=True, custom_query=custom_query)
-            else:
-                res = ctx.results(batch=False, brains=True)
-            if expand:
-                # get start and end values from the query to ensure limited
-                # listing for occurrences
-                start, end = self._expand_events_start_end(
-                    query.get("start") or custom_query.get("start"),
-                    query.get("end") or custom_query.get("end"),
-                )
-                res = expand_events(
-                    res,
-                    ret_mode,
-                    start=start,
-                    end=end,
-                    sort=sort_on,
-                    sort_reverse=True if sort_order else False,
-                )
-        else:
-            res = self._get_events(ret_mode, expand=expand)
+        res = self._get_events(ret_mode, expand=expand)
         if batch:
             b_start = self.b_start
             b_size = self.b_size
@@ -225,40 +160,6 @@ def ical_url(self):
         qstr = "?%s" % qstr if qstr else ""
         return f"{self.context.absolute_url()}/@@event_listing_ical{qstr}"
 
-    # COLLECTION daterange start/end determination
-    def _expand_events_start_end(self, start, end):
-        # make sane start and end values for expand_events from
-        # Collection start/end criterions.
-        # if end/min is given, it overrides start/min settings to make sure,
-        # ongoing events are shown in the listing!
-        # XXX: This actually fits most needs, but not all. Maybe someone
-        # wants to come up with some edgecases!
-        se = dict(start=None, end=None)
-
-        def safe_dt(val):
-            # convert value from DateTime to datetime
-            if hasattr(val, "asdatetime"):
-                return val.asdatetime()
-            return val
-
-        if start:
-            q = start.get("query")
-            r = start.get("range")
-            if r == "min":
-                se["start"] = q
-            elif r == "max":
-                se["end"] = q
-            elif r in ("minmax", "min:max"):
-                list(q).sort()
-                se["start"] = q[0]
-                se["end"] = q[1]
-        if end:
-            q = end.get("query")
-            r = end.get("range")
-            if r == "min":
-                se["start"] = q
-        return safe_dt(se["start"]), safe_dt(se["end"])
-
     def formatted_date(self, occ):
         provider = getMultiAdapter(
             (self.context, self.request, self), IContentProvider, name="formatted_date"
@@ -460,13 +361,7 @@ def prev_month_url(self):
         datestr = (now.replace(day=1) - timedelta(days=1)).date().isoformat()
         return self._date_nav_url("month", datestr)
 
-
-class EventListingIcal(EventListing):
-    def __call__(self, *args, **kwargs):
-        return self.ical
-
-
-class EventEventListing(EventListing):
+class EventEventListing(BaseEventListing):
     """This is an EventListing for an individual event, to list all
     occurrences batched and navigatable with all the features, the EventListing
     offers.
@@ -475,3 +370,121 @@ class EventEventListing(EventListing):
     def __init__(self, context, request):
         super().__init__(context, request)
         self.uid = IUUID(self.context)
+
+    def get_query_keywords(self):
+        kw = {}
+        kw["UID"] = IUUID(self.context)
+        return kw
+
+class CollectionEventListing(BaseEventListing):
+    def __init__(self, context, request):
+        super().__init__(context, request)
+        b_size = int(request.get("b_size", 0))
+        if not b_size:
+            collection_behavior = ISyndicatableCollection(context)
+            b_size = getattr(collection_behavior, "item_count", 0)
+        self.b_size = b_size or 10
+
+        if not self.show_filter:
+            self.mode = "all"
+
+    @property
+    def show_filter(self):
+        ctx = self.context
+        query = queryparser.parseFormquery(ctx, ctx.query)
+        if "start" in query or "end" in query:
+            # Don't show the date filter, if a date is given in the
+            # collection's query
+            return False
+        return True
+    
+    @view.memoize
+    def events(self, ret_mode=RET_MODE_ACCESSORS, expand=True, batch=True):
+        ctx = self.context
+        # Whatever sorting is defined, we're overriding it.
+        sort_on = "start"
+        sort_order = None
+        if self.mode in ("past", "all"):
+            sort_order = "reverse"
+        query = queryparser.parseFormquery(
+            ctx, ctx.query, sort_on=sort_on, sort_order=sort_order
+        )
+
+        # only use custom end or start dates, if none are used by the collection
+        custom_query = self.request.get("contentFilter", {})
+        if "start" not in query and "end" not in query:
+            # ... else don't show the navigation bar
+            start, end = self._start_end
+            start, end = _prepare_range(ctx, start, end)
+            custom_query.update(start_end_query(start, end))
+            res = ctx.results(batch=False, brains=True, custom_query=custom_query)
+        else:
+            res = ctx.results(batch=False, brains=True)
+        if expand:
+            # get start and end values from the query to ensure limited
+            # listing for occurrences
+            start, end = self._expand_events_start_end(
+                query.get("start") or custom_query.get("start"),
+                query.get("end") or custom_query.get("end"),
+            )
+            res = expand_events(
+                res,
+                ret_mode,
+                start=start,
+                end=end,
+                sort=sort_on,
+                sort_reverse=True if sort_order else False,
+            )
+        if batch:
+            res = Batch(res, size=self.b_size, start=self.b_start, orphan=self.orphan)
+        return res
+    
+    def _expand_events_start_end(self, start, end):
+        # make sane start and end values for expand_events from
+        # Collection start/end criterions.
+        # if end/min is given, it overrides start/min settings to make sure,
+        # ongoing events are shown in the listing!
+        # XXX: This actually fits most needs, but not all. Maybe someone
+        # wants to come up with some edgecases!
+        se = dict(start=None, end=None)
+
+        def safe_dt(val):
+            # convert value from DateTime to datetime
+            if hasattr(val, "asdatetime"):
+                return val.asdatetime()
+            return val
+        
+        if start:
+            q = start.get("query")
+            r = start.get("range")
+            if r == "min":
+                se["start"] = q
+            elif r == "max":
+                se["end"] = q
+            elif r in ("minmax", "min:max"):
+                list(q).sort()
+                se["start"] = q[0]
+                se["end"] = q[1]
+        if end:
+            q = end.get("query")
+            r = end.get("range")
+            if r == "min":
+                se["start"] = q
+        return safe_dt(se["start"]), safe_dt(se["end"])
+
+def convert_to_datetime(time, tzinfo):
+    if isinstance(time, DateTime):
+        return time.asdatetime()
+    elif isinstance(time, str):
+        naive_time = datetime.fromisoformat(time)
+        aware_time=  naive_time.replace(tzinfo=tzinfo)
+        return aware_time
+    return time
+
+class FolderEventListing(BaseEventListing):
+    pass 
+
+
+class EventListingIcal(BaseEventListing):
+    def __call__(self, *args, **kwargs):
+        return self.ical
\ No newline at end of file

From f60192afd38b7f2aae4201ad2f87e5c43855efc7 Mon Sep 17 00:00:00 2001
From: Thibaut Born <thibaut.born@kuleuven.be>
Date: Thu, 22 Aug 2024 17:19:34 +0200
Subject: [PATCH 3/3] update changenote

---
 news/405.bugfix | 1 +
 1 file changed, 1 insertion(+)
 create mode 100644 news/405.bugfix

diff --git a/news/405.bugfix b/news/405.bugfix
new file mode 100644
index 00000000..e3c8323d
--- /dev/null
+++ b/news/405.bugfix
@@ -0,0 +1 @@
+Fix query for when start/end dates are set on collections and refactor event_listing. [ThibautBorn]