From 0fcf95f16c942d9841f2779ceb3da54b477be305 Mon Sep 17 00:00:00 2001
From: Mauro Amico <mauro.amico@gmail.com>
Date: Tue, 3 Sep 2024 11:50:51 +0200
Subject: [PATCH] split news behaviors (comunicatistampa) (#273)

* split news behaviors
* changelog
* fix test
* add subobject in comunicatostampa
* Update CHANGES.rst
---
 CHANGES.rst                                   |  4 +++
 .../contenttypes/behaviors/configure.zcml     |  8 ++++++
 .../behaviors/news_additional_fields.py       | 25 +++++++++++--------
 .../plone/contenttypes/events/common.py       | 20 +++++++++++++++
 .../plone/contenttypes/tests/test_ct_news.py  |  2 +-
 5 files changed, 47 insertions(+), 12 deletions(-)

diff --git a/CHANGES.rst b/CHANGES.rst
index 189272b8..2f6a8e9a 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -4,6 +4,10 @@ Changelog
 6.2.15 (unreleased)
 -------------------
 
+- Add design.plone.contenttypes.behavior.news_base behavior (news metadata, without blocks)
+  [mamico]
+- Add folder creation (multimedia + allegati) for "ComunicatiStampa" CT (if exists)
+  [mamico]
 - Add "Emolumenti a carico della finanza pubblica" to Persona
   Add "Dichiarazioni di insussistenza e incompatibilità" to Persona
   [lucabel]  
diff --git a/src/design/plone/contenttypes/behaviors/configure.zcml b/src/design/plone/contenttypes/behaviors/configure.zcml
index 10d55afa..9c7ed276 100644
--- a/src/design/plone/contenttypes/behaviors/configure.zcml
+++ b/src/design/plone/contenttypes/behaviors/configure.zcml
@@ -38,6 +38,14 @@
       marker=".news_additional_fields.INewsAdditionalFields"
       />
 
+  <plone:behavior
+      name="design.plone.contenttypes.behavior.news_base"
+      title="Metadati news base (senza descrizione_estesa)"
+      description="Adds fields."
+      provides=".news_additional_fields.INewsAdditionalFieldsBase"
+      marker=".news_additional_fields.INewsAdditionalFieldsBase"
+      />
+
   <plone:behavior
       name="design.plone.contenttypes.behavior.argomenti"
       title="Argomenti"
diff --git a/src/design/plone/contenttypes/behaviors/news_additional_fields.py b/src/design/plone/contenttypes/behaviors/news_additional_fields.py
index 149879b1..5111322c 100644
--- a/src/design/plone/contenttypes/behaviors/news_additional_fields.py
+++ b/src/design/plone/contenttypes/behaviors/news_additional_fields.py
@@ -16,16 +16,7 @@
 
 
 @provider(IFormFieldProvider)
-class INewsAdditionalFields(model.Schema):
-    descrizione_estesa = BlocksField(
-        title=_("descrizione_estesa", default="Descrizione estesa"),
-        required=True,
-        description=_(
-            "descrizione_estesa_help",
-            default="Descrizione dettagliata e completa.",
-        ),
-    )
-
+class INewsAdditionalFieldsBase(model.Schema):
     numero_progressivo_cs = schema.TextLine(
         title=_(
             "numero_progressivo_cs_label",
@@ -121,10 +112,22 @@ class INewsAdditionalFields(model.Schema):
         fields=["notizie_correlate"],
     )
     # custom fieldsets and order
-    form.order_before(descrizione_estesa="ILeadImageBehavior.image")
     form.order_before(numero_progressivo_cs="ILeadImageBehavior.image")
     form.order_before(a_cura_di="ILeadImageBehavior.image")
 
+
+@provider(IFormFieldProvider)
+class INewsAdditionalFields(INewsAdditionalFieldsBase):
+    descrizione_estesa = BlocksField(
+        title=_("descrizione_estesa", default="Descrizione estesa"),
+        required=True,
+        description=_(
+            "descrizione_estesa_help",
+            default="Descrizione dettagliata e completa.",
+        ),
+    )
+
+    form.order_before(descrizione_estesa="ILeadImageBehavior.image")
     textindexer.searchable("descrizione_estesa")
 
 
diff --git a/src/design/plone/contenttypes/events/common.py b/src/design/plone/contenttypes/events/common.py
index 70d69fbc..0d1268c3 100644
--- a/src/design/plone/contenttypes/events/common.py
+++ b/src/design/plone/contenttypes/events/common.py
@@ -100,6 +100,26 @@
             },
         ],
     },
+    "ComunicatoStampa": {
+        "content": [
+            {
+                "id": "multimedia",
+                "title": "Multimedia",
+                "allowed_types": (
+                    "Image",
+                    "Link",
+                ),
+            },
+            {
+                "id": "documenti-allegati",
+                "title": "Documenti allegati",
+                "allowed_types": (
+                    "File",
+                    "Image",
+                ),
+            },
+        ]
+    },
     "Persona": {
         "content": [
             {
diff --git a/src/design/plone/contenttypes/tests/test_ct_news.py b/src/design/plone/contenttypes/tests/test_ct_news.py
index 4239d9c2..1256782a 100644
--- a/src/design/plone/contenttypes/tests/test_ct_news.py
+++ b/src/design/plone/contenttypes/tests/test_ct_news.py
@@ -125,11 +125,11 @@ def test_news_item_fields_default_fieldset(self):
                 "image_caption",
                 "preview_image",
                 "preview_caption",
-                "descrizione_estesa",
                 "numero_progressivo_cs",
                 "a_cura_di",
                 "a_cura_di_persone",
                 "luoghi_correlati",
+                "descrizione_estesa",
                 "tassonomia_argomenti",
                 "tipologia_notizia",
             ],