Cartelle aggiuntive all'interno della persona (#305)

* update with structure creation

* fix upgrade

Author: luca-bellenghi

src/design/plone/contenttypes/events/common.py

@@ -163,6 +163,18 @@
                 "title": "Altri documenti",
                 "allowed_types": ("File", "Image", "Link"),
             },
+            {
+                "id": "dichiarazione-insussistenza-cause-di-inconferibilita-e-incompatibilita",  # noqa
+                "title": "Dichiarazione insussistenza cause di inconferibilità e"
+                " incompatibilità",
+                "allowed_types": ("File",),
+            },
+            {
+                "id": "emolumenti-complessivi-percepiti-a-carico-della-finanza-pubblica",  # noqa
+                "title": "Emolumenti complessivi percepiti a carico della finanza"
+                " pubblica",
+                "allowed_types": ("File",),
+            },
         ],
         "allowed_types": [],
     },
@@ -207,16 +219,7 @@ def onModify(context, event):
                     child.reindexObject(idxs=["parent"])
 
 
-def createSubfolders(context, event):
-    """
-    Create subfolders structure based on a portal_type mapping
-    """
-    if not IDesignPloneContenttypesLayer.providedBy(context.REQUEST):
-        return
-
-    subfolders_mapping = SUBFOLDERS_MAPPING.get(context.portal_type, [])
-    if not subfolders_mapping:
-        return
+def createStructure(context, subfolders_mapping):
 
     for mapping in subfolders_mapping.get("content", {}):
         if mapping["id"] not in context.keys():
@@ -251,3 +254,16 @@ def createSubfolders(context, event):
         constraints_context = ISelectableConstrainTypes(context)
         constraints_context.setConstrainTypesMode(1)
         constraints_context.setLocallyAllowedTypes(allowed_types)
+
+
+def createSubfolders(context, event):
+    """
+    Create subfolders structure based on a portal_type mapping
+    """
+    if not IDesignPloneContenttypesLayer.providedBy(context.REQUEST):
+        return
+
+    subfolders_mapping = SUBFOLDERS_MAPPING.get(context.portal_type, [])
+    if not subfolders_mapping:
+        return
+    createStructure(context, subfolders_mapping)

src/design/plone/contenttypes/indexers/events.py

@@ -36,6 +36,4 @@ def effectiveend(obj):
     for occurrence in list(occurrences):
         if occurrence.end > end:
             end = occurrence.end
-    if obj.title:
-        print(f"{obj.title} - {end}")
     return end

src/design/plone/contenttypes/profiles/default/metadata.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <metadata>
-  <version>7313</version>
+  <version>7314</version>
   <dependencies>
     <dependency>profile-redturtle.bandi:default</dependency>
     <dependency>profile-collective.venue:default</dependency>

src/design/plone/contenttypes/tests/test_substructure_creation.py

@@ -222,6 +222,8 @@ def test_persona_substructure_created(self):
         - variazione-situazione-patrimoniale" "altre-cariche
         - incarichi
         - altri-documenti
+        - dichiarazione-insussistenza-cause-di-inconferibilita-e-incompatibilita
+        - emolumenti-complessivi-percepiti-a-carico-della-finanza-pubblica
         """
         item = api.content.create(
             container=self.portal,
@@ -241,6 +243,8 @@ def test_persona_substructure_created(self):
                 "altre-cariche",
                 "incarichi",
                 "altri-documenti",
+                "dichiarazione-insussistenza-cause-di-inconferibilita-e-incompatibilita",
+                "emolumenti-complessivi-percepiti-a-carico-della-finanza-pubblica",
             ],
         )
 

src/design/plone/contenttypes/upgrades/configure.zcml

@@ -948,4 +948,11 @@
       destination="7313"
       handler=".to_730x.to_7313"
       />
+  <genericsetup:upgradeStep
+      title="Update Persona subfolder"
+      profile="design.plone.contenttypes:default"
+      source="7313"
+      destination="7314"
+      handler=".to_730x.to_7314"
+      />
 </configure>

src/design/plone/contenttypes/upgrades/to_730x.py

@@ -4,7 +4,7 @@
 from design.plone.contenttypes.utils import create_default_blocks
 from plone import api
 from Products.CMFPlone.interfaces import ISelectableConstrainTypes
-
+from design.plone.contenttypes.events.common import createStructure
 import logging
 import transaction
 
@@ -225,3 +225,43 @@ class extra:
         if i % 15 == 0:
             logger.info("Progress: {}/{}".format(i, tot))
         brain.getObject().reindexObject(idxs=["effectiveend"])
+
+
+def to_7314(context):
+    logger.info("Add new folder to Persona CT")
+    mapping1 = {
+        "content": [
+            {
+                "id": "dichiarazione-insussistenza-cause-di-inconferibilita-e-incompatibilita",  # noqa
+                "title": "Dichiarazione insussistenza cause di inconferibilità e"
+                " incompatibilità",
+                "allowed_types": ("File",),
+            },
+        ]
+    }
+    mapping2 = {
+        "content": [
+            {
+                "id": "emolumenti-complessivi-percepiti-a-carico-della-finanza-pubblica",  # noqa
+                "title": "Emolumenti complessivi percepiti a carico della finanza"
+                " pubblica",
+                "allowed_types": ("File",),
+            },
+        ]
+    }
+    pc = api.portal.get_tool(name="portal_catalog")
+    brains = pc(portal_type="Persona")
+    for brain in brains:
+        persona = brain.getObject()
+        if (
+            "dichiarazione-insussistenza-cause-di-inconferibilita-e-incompatibilita"
+            not in persona.keys()
+        ):
+            createStructure(persona, mapping1)
+            logger.info("Add dichiarazione insussistenza for {}".format(persona.title))
+        if (
+            "emolumenti-complessivi-percepiti-a-carico-della-finanza-pubblica"
+            not in persona.keys()
+        ):
+            createStructure(persona, mapping2)
+            logger.info("Add emolumenti complessivi for {}".format(persona.title))