emrt.necd.content 2.5.50__zip → 2.5.51__zip

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/docs/HISTORY.txt

@@ -1,6 +1,10 @@
 Changelog
 =========
 
+2.5.51 (2024-05-23)
+
+- Bug #270288 - Fix observation not reindexing after new comments
+
 2.5.50 (2024-04-30)
 
 - Task #260973 - Add formatting

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/emrt/necd/content/configure.zcml

@@ -193,6 +193,12 @@
     handler="emrt.necd.content.notifications.observation_finalisation_request.notification_lr"
     />
 
+  <!-- New comment, reindex number of replies -->
+  <subscriber
+    for="plone.app.discussion.interfaces.IComment zope.lifecycleevent.IObjectAddedEvent"
+    handler="emrt.necd.content.subscribers.new_discussion_comment"
+    />
+
   <!-- Crons view -->
   <browser:page
       name="cron_update_vocabularies"

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/emrt/necd/content/profiles/default/metadata.xml

@@ -1,5 +1,5 @@
 <metadata>
-  <version>2.5.50</version>
+  <version>2.5.51</version>
     <dependencies>
         <dependency>profile-plone.app.dexterity:default</dependency>
         <dependency>profile-plone.app.ldap:ldap</dependency>

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/emrt/necd/content/subscribers.py

@@ -3,6 +3,9 @@ from DateTime import DateTime
 from plone import api
 from Products.CMFCore.utils import getToolByName
 
+from emrt.necd.content.observation import IObservation
+from emrt.necd.content.utils import find_parent_with_interface
+
 
 def question_transition(question, event):
     if event.action in ['approve-question']:
@@ -158,3 +161,9 @@ def observation_transition(observation, event):
 
 
     observation.reindexObject()
+
+
+def new_discussion_comment(comment, event):
+    with api.env.adopt_roles(roles=['Manager']):
+        observation = find_parent_with_interface(IObservation, comment)
+        observation.reindexObject()

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/emrt/necd/content/upgrades/configure.zcml

@@ -823,4 +823,17 @@
 
   </genericsetup:upgradeSteps>
 
+  <genericsetup:upgradeSteps
+    source="2.5.50"
+    destination="2.5.51"
+    profile="emrt.necd.content:default">
+
+    <genericsetup:upgradeStep
+      title="Reindex observation discussion counts"
+      description="[refs #270288] observation not reindexed when new discussion comments are added."
+      handler=".evolve2551.run"
+      />
+
+  </genericsetup:upgradeSteps>
+
 </configure>

{emrt.necd.content-2.5.50 → emrt.necd.content-2.5.51}/emrt/necd/content/upgrades/evolve2550.py

@@ -53,4 +53,4 @@ def run(_):
         # log progress
         if a_tenth and idx % a_tenth == 0:
             transaction.savepoint(optimistic=True)
-            LOGGER.info("Done %s/ < %s.", idx, brains_len)
+            LOGGER.info("Done %s/%s.", idx, brains_len)

emrt.necd.content-2.5.51/emrt/necd/content/upgrades/evolve2551.py

@@ -0,0 +1,31 @@
+from logging import getLogger
+
+import transaction
+from DateTime import DateTime
+
+import plone.api as api
+
+
+LOGGER = getLogger(__name__)
+
+
+def run(_):
+    catalog = api.portal.get_tool("portal_catalog")
+    brains = catalog(portal_type="Observation", modified={"range": "min", "query": DateTime("2024/02/19")})
+
+    brains_len = len(brains)
+    a_tenth = brains_len / 10
+
+    LOGGER.info("Found %s brains...", brains_len)
+
+    observations = (b.getObject() for b in brains)
+
+    for idx, observation in enumerate(observations, start=1):
+        LOGGER.info(observation.absolute_url(1))
+
+        observation.reindexObject()
+
+        # log progress
+        if a_tenth and idx % a_tenth == 0:
+            transaction.savepoint(optimistic=True)
+            LOGGER.info("Done %s/%s.", idx, brains_len)