udata 11.1.1.dev1__py3-none-any.whl → 11.1.1.dev3__py3-none-any.whl

udata/__init__.py

@@ -4,5 +4,4 @@
 udata
 """
 
-__version__ = "11.1.1.dev"
 __description__ = "Open data portal"

udata/core/topic/models.py

@@ -1,5 +1,6 @@
 from blinker import Signal
 from flask import url_for
+from mongoengine.errors import DoesNotExist
 from mongoengine.signals import post_delete, post_save
 
 from udata.api_fields import field
@@ -47,8 +48,12 @@ class TopicElement(Auditable, db.Document):
     @classmethod
     def post_delete(cls, sender, document, **kwargs):
         """Trigger reindex when element is deleted"""
-        if document.topic and document.element and hasattr(document.element, "id"):
-            reindex.delay(*as_task_param(document.element))
+        try:
+            if document.topic and document.element and hasattr(document.element, "id"):
+                reindex.delay(*as_task_param(document.element))
+        except DoesNotExist:
+            # Topic might have been deleted, causing dereferencing to fail
+            pass
         cls.on_delete.send(document)
 
 
@@ -86,6 +91,11 @@ class Topic(db.Datetimed, Auditable, Linkable, db.Document, Owned):
     on_create = Signal()
     on_update = Signal()
 
+    @classmethod
+    def post_delete(cls, sender, document, **kwargs):
+        """Delete associated TopicElements when a Topic is deleted"""
+        TopicElement.objects(topic=document).delete()
+
     def __str__(self):
         return self.name
 
@@ -126,4 +136,5 @@ class Topic(db.Datetimed, Auditable, Linkable, db.Document, Owned):
 
 post_save.connect(Topic.post_save, sender=Topic)
 post_save.connect(TopicElement.post_save, sender=TopicElement)
+post_delete.connect(Topic.post_delete, sender=Topic)
 post_delete.connect(TopicElement.post_delete, sender=TopicElement)

udata/migrations/2025-10-01-delete-orphaned-topic-elements.py

@@ -0,0 +1,30 @@
+"""
+Delete TopicElements that are not associated with an existing Topic
+"""
+
+import logging
+
+import mongoengine
+
+from udata.core.topic.models import TopicElement
+
+log = logging.getLogger(__name__)
+
+
+def migrate(db):
+    log.info("Processing orphaned TopicElements...")
+
+    topic_elements = TopicElement.objects.no_cache().all()
+    deleted_count = 0
+
+    for element in topic_elements:
+        try:
+            # Try to access the topic - will raise DoesNotExist if topic was deleted
+            if element.topic:
+                _ = element.topic.id
+        except mongoengine.errors.DoesNotExist:
+            # Topic doesn't exist, delete the orphaned element
+            element.delete()
+            deleted_count += 1
+
+    log.info(f"Deleted {deleted_count} orphaned TopicElements")

udata/tests/test_topics.py

@@ -15,7 +15,7 @@ from udata.core.topic.factories import (
     TopicFactory,
     TopicWithElementsFactory,
 )
-from udata.core.topic.models import Topic
+from udata.core.topic.models import Topic, TopicElement
 from udata.search import reindex
 from udata.tests.helpers import assert_emit
 
@@ -142,3 +142,15 @@ class TopicModelTest:
         with pytest.raises(ValidationError):
             topic = TopicFactory()
             TopicElementFactory(topic=topic, element=DiscussionFactory())
+
+    def test_topic_deletion_deletes_associated_elements(self):
+        """Test that deleting a topic also deletes its associated TopicElements"""
+        topic = TopicWithElementsFactory()
+
+        element_ids = [elem.id for elem in topic.elements]
+        assert len(element_ids) > 0
+
+        topic.delete()
+
+        for element_id in element_ids:
+            assert TopicElement.objects(id=element_id).first() is None

{udata-11.1.1.dev1.dist-info → udata-11.1.1.dev3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: udata
-Version: 11.1.1.dev1
+Version: 11.1.1.dev3
 Summary: Open data portal
 Author-email: Opendata Team <opendatateam@data.gouv.fr>
 Maintainer-email: Opendata Team <opendatateam@data.gouv.fr>
@@ -126,7 +126,6 @@ Requires-Dist: werkzeug==2.2.3
 Requires-Dist: wtforms[email]==3.2.1
 Requires-Dist: wtforms-json==0.3.5
 Provides-Extra: dev
-Requires-Dist: bumpx==0.3.10; extra == "dev"
 Requires-Dist: flask-shell-ipython==0.5.3; extra == "dev"
 Requires-Dist: invoke==2.2.0; extra == "dev"
 Requires-Dist: pip-tools==7.4.1; extra == "dev"

{udata-11.1.1.dev1.dist-info → udata-11.1.1.dev3.dist-info}/RECORD

@@ -1,4 +1,4 @@
-udata/__init__.py,sha256=z-x_LDWBBmN_X2NfUodR8vC1VPSQji8MAbSrTr3qUFM,102
+udata/__init__.py,sha256=U0HEYqKCLOY43O1UCVeuAb3b3SSX1pPhsJGpHJmK67k,75
 udata/api_fields.py,sha256=XI0XoM1fxO4DEzxGptOAB5SL_fJr-u58-bfQVAvzgBg,36549
 udata/app.py,sha256=gRQSi4Scqu6rj0czVsmBRkNM4wNRkBhY7y7NWbqY8UM,7249
 udata/assets.py,sha256=H5Hrc2vnKM0IFLyWfLXmJ2Kj35w1i8W1D8Cgy8_cUj4,657
@@ -242,7 +242,7 @@ udata/core/topic/api_fields.py,sha256=zMjoTSoe7-Q8F-i7Ry5EAUv-ds4Lzao3jfETZq-Vg0
 udata/core/topic/apiv2.py,sha256=WVyZDNew8m0qVFn9RLKW0SeLQHuUwWIcGAfomzua7sw,6449
 udata/core/topic/factories.py,sha256=Rhx12aTrYFZScu53IhYNP0zJKfiVZPba34ybZ7gGSwo,3010
 udata/core/topic/forms.py,sha256=OkWFo0FkSnjZ2LURoq2i2cpingFz1TrOEI3TJ7qQ8pk,3465
-udata/core/topic/models.py,sha256=9EFnSmlyC2mf8tJYJ2eESbjPicz--nqd6lX9tf3qIjQ,4221
+udata/core/topic/models.py,sha256=kQ3EMqss5G4f6jKC528NNAoQdPxZ3I73Gv63Mt31Yvs,4656
 udata/core/topic/parsers.py,sha256=oteskLLFje-zufAYzSl1zsy-MLac2hpXhGI0S3igA1g,4560
 udata/core/topic/permissions.py,sha256=RtFPPlxuU_Bv7ip6LDO4AoPrKFnIOEs9cCMXaSSmEdk,118
 udata/core/topic/tasks.py,sha256=_OrPfuZ9rKENp4p4IkoyV-Yqp9bxI3nNxhpfKrbgl-w,357
@@ -395,6 +395,7 @@ udata/migrations/2025-07-18-compute-last-update.py,sha256=K3wSSl9oC8TRbHlvgKeekg
 udata/migrations/2025-07-18-redo-count-discussion.py,sha256=BOFTjf0zq_ahnVlYQC-d8fuoLixxyk2vP6JYJXhmvbs,941
 udata/migrations/2025-07-22-fix-duplicate-contact-points.py,sha256=ZEw2PX1E3jJiDNWLj6Y52zBAIHS-kZSLjBP6gUmA_ho,1789
 udata/migrations/2025-07-30-purge-old-harvest-dynamic-fields.py,sha256=ijeu6WvX6etOJrv0cEMCRJzj1pSetBguF-aUB1-2y_0,1017
+udata/migrations/2025-10-01-delete-orphaned-topic-elements.py,sha256=Mhx5ANOihZL4botxtjvfsll-xKBtkVQBkPcSq0BJ-Ec,788
 udata/migrations/__init__.py,sha256=RBCBDaTlLjuMs_Qzwji6Z6T4r7FCGXhESKoxQbT5qAA,11221
 udata/models/__init__.py,sha256=77OriDFm4dJzQDIb-7ADKSkf9GAxkXbMHeWXoYYnTsk,1459
 udata/mongo/__init__.py,sha256=y4Rv-kq3o_kcEulcNpePLzocXPBNpx3Jd82G-VZPaMc,1421
@@ -647,7 +648,7 @@ udata/tests/test_rdf.py,sha256=XlUjRFcvTeZx1DTwwHL4pHQTcFDcAuQzaGOY_jJMDL4,4863
 udata/tests/test_routing.py,sha256=jtCZuT229WhGvJU843DtzIrdUXN5f7dym-hwz5dOFLo,10844
 udata/tests/test_storages.py,sha256=OG_PnKtPlZmmhKnbbLrvL-EMEg35wAMsTC76jVEAOLU,9602
 udata/tests/test_tags.py,sha256=y8x17-m_opa8YKjOWU60Xz6FR20FpBz61e7JJeyTwzM,3748
-udata/tests/test_topics.py,sha256=Z31k-9ETwfKouK-L93J7bXkCbraluhdi02E_b9is_8M,5211
+udata/tests/test_topics.py,sha256=yhP-15jQ2ZVI0etfmHZfTcYEsuAeX7j-Wvg-1UiIT0E,5646
 udata/tests/test_transfer.py,sha256=_0pBwYs3T7OSZ7bO3KmQ81SjwCJyT4EVf8YYHXOkwdk,7779
 udata/tests/test_uris.py,sha256=MxafZ0SyzSNRomVpZnH1ChzWaHOi1MQiXe1gmKnBc6o,8517
 udata/tests/test_utils.py,sha256=3BGnlvw-GOE6tLHQteo-uUeYuzq4rsjePOuytFGkpOg,7967
@@ -764,9 +765,9 @@ udata/translations/pt/LC_MESSAGES/udata.mo,sha256=U0abG-nBwCIoYxRZNsc4KOLeIRSqTV
 udata/translations/pt/LC_MESSAGES/udata.po,sha256=eCG35rMzYLHXyLbsnLSexS1g0N_K-WpNHqrt_8y6I4E,48590
 udata/translations/sr/LC_MESSAGES/udata.mo,sha256=IBcCAdmcvkeK7ZeRBNRI-wJ0jzWNM0eXM5VXAc1frWI,28692
 udata/translations/sr/LC_MESSAGES/udata.po,sha256=yFxHEEB4behNwQ7JnyoYheiCKLNnMS_NV4guzgyzWcE,55332
-udata-11.1.1.dev1.dist-info/licenses/LICENSE,sha256=V8j_M8nAz8PvAOZQocyRDX7keai8UJ9skgmnwqETmdY,34520
-udata-11.1.1.dev1.dist-info/METADATA,sha256=W2UTVkIVY8Df-s4y-Ir0obAdrXnKhv4frDnVjcu0BnU,6768
-udata-11.1.1.dev1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-udata-11.1.1.dev1.dist-info/entry_points.txt,sha256=v2u12qO11i2lyLNIp136WmLJ-NHT-Kew3Duu8J-AXPM,614
-udata-11.1.1.dev1.dist-info/top_level.txt,sha256=EF6CE6YSHd_og-8LCEA4q25ALUpWVe8D0okOLdMAE3A,6
-udata-11.1.1.dev1.dist-info/RECORD,,
+udata-11.1.1.dev3.dist-info/licenses/LICENSE,sha256=V8j_M8nAz8PvAOZQocyRDX7keai8UJ9skgmnwqETmdY,34520
+udata-11.1.1.dev3.dist-info/METADATA,sha256=r1EWJq_LyZno2tzXai0fyT23th5ogI4FyVV2c5TfvRk,6723
+udata-11.1.1.dev3.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+udata-11.1.1.dev3.dist-info/entry_points.txt,sha256=v2u12qO11i2lyLNIp136WmLJ-NHT-Kew3Duu8J-AXPM,614
+udata-11.1.1.dev3.dist-info/top_level.txt,sha256=EF6CE6YSHd_og-8LCEA4q25ALUpWVe8D0okOLdMAE3A,6
+udata-11.1.1.dev3.dist-info/RECORD,,