From 296ee1376450adf58a040a9da816a100fa5d2ec9 Mon Sep 17 00:00:00 2001 From: Étienne Loks Date: Thu, 12 Sep 2024 12:39:40 +0200 Subject: ⚡️ improve performance - external ID: prevent double save - user UPDATE query instead of save - remove dead code "BulkUpdatedItem" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- archaeological_operations/models.py | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) (limited to 'archaeological_operations') diff --git a/archaeological_operations/models.py b/archaeological_operations/models.py index e847429a0..4220e6cc2 100644 --- a/archaeological_operations/models.py +++ b/archaeological_operations/models.py @@ -2434,12 +2434,19 @@ def operation_post_save(sender, **kwargs): for parcel in operation.parcels.all(): parcel.copy_to_file() - # external id update + # external id, cached_labels update for parcel in operation.parcels.all(): - parcel.update_external_id(save=True) + cached_label_changed(Parcel, instance=parcel) + ContextRecord = apps.get_model("archaeological_context_records", "ContextRecord") + BaseFind = apps.get_model("archaeological_finds", "BaseFind") + Find = apps.get_model("archaeological_finds", "Find") for cr in operation.context_record.all(): - cr.update_external_id(save=True) + cached_label_changed(ContextRecord, instance=cr) + for bf in cr.base_finds.all(): + cached_label_changed(BaseFind, instance=bf) + for f in bf.find.all(): + cached_label_changed(Find, instance=f) post_save.connect(operation_post_save, sender=Operation) @@ -3127,6 +3134,7 @@ def strip_zero(value): class Parcel(LightHistorizedItem): + SLUG = "parcel" EXTERNAL_ID_KEY = "parcel_external_id" BASE_SEARCH_VECTORS = [ SearchVectorConfig("section"), -- cgit v1.2.3