black, flake, zpretty

RedTurtle · Aug 16, 2023 · 60ca0d7 · 60ca0d7
1 parent f657122
commit 60ca0d7
Show file tree

Hide file tree

Showing 18 changed files with 135 additions and 84 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -8,21 +8,21 @@ default_stages: [commit, push]
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.3.0
+    rev: v4.4.0
     hooks:
       - id: trailing-whitespace
       - id: end-of-file-fixer
       - id: check-yaml
   - repo: https://github.com/psf/black
-    rev: 23.1.0
+    rev: 23.7.0
     hooks:
       - id: black
 #        args: ["--line-length=88", "--check", "--diff", "--force-exclude=migrations", "src/"]
         args: ["--line-length=88", "--force-exclude=migrations", "src/"]
         types: [python]
         entry: black
   - repo: https://github.com/PyCQA/flake8.git
-    rev: "3.9.2"
+    rev: "6.1.0"
     hooks:
       - id: flake8
         name: flake8
@@ -31,7 +31,7 @@ repos:
         types: [python]
         args: ["--max-complexity=30", "--max-line-length=88", "--ignore=DJ01,DJ08,W503,ANN101", "--exclude=docs/*", "src/", "setup.py"]
   - repo: https://github.com/pycqa/isort
-    rev: 5.10.1
+    rev: 5.12.0
     hooks:
       - id: isort
         name: isort (python)
@@ -45,11 +45,11 @@ repos:
         language: pygrep
         types: [python]
   - repo: https://github.com/collective/zpretty
-    rev: 2.4.1
+    rev: 3.1.0
     hooks:
       - id: zpretty
         name: zpretty
   - repo: https://github.com/regebro/pyroma
-    rev: "3.2"
+    rev: '4.2'
     hooks:
       - id: pyroma
diff --git a/src/design/plone/contenttypes/browser/manage_content/templates/check_servizi.pt b/src/design/plone/contenttypes/browser/manage_content/templates/check_servizi.pt
@@ -13,7 +13,12 @@
 
     </metal:block>
     <metal:block fill-slot="style_slot">
-      <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.4.0/css/all.min.css" integrity="sha512-iecdLmaskl7CVkqkXNQ/ZH/XLlvWZOJyj7Yy7tcenmpD1ypASozpmT/E0iPtmFIB46ZmdtAc9eNBvH0H/ZpiBw==" crossorigin="anonymous" referrerpolicy="no-referrer" />
+      <link crossorigin="anonymous"
+            href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.4.0/css/all.min.css"
+            integrity="sha512-iecdLmaskl7CVkqkXNQ/ZH/XLlvWZOJyj7Yy7tcenmpD1ypASozpmT/E0iPtmFIB46ZmdtAc9eNBvH0H/ZpiBw=="
+            referrerpolicy="no-referrer"
+            rel="stylesheet"
+      />
       <style>
        li.categoria{margin-bottom:2em}
        #content-header,
@@ -93,42 +98,61 @@
                Ogni redattore pu&ograve; vedere in questa lista tutti i servizi e,
                cliccando su ognuno, andare alla vista di dettaglio del servizio,
             editarlo e compilare tutti i campi obbligatori.</p>
-            <p class="lead"
-             tal:condition="servizi">Per quanto riguarda la colonna <strong>canale di accesso</strong>,
+          <p class="lead"
+             tal:condition="servizi"
+          >Per quanto riguarda la colonna
+            <strong>canale di accesso</strong>,
             nei servizi, abbiamo canale fisico e canale digitale. Il canale digitale,
-            è calcolato su due campi: uno testuale descrittivo e uno contenente il link per
-            l'accesso. È obbligatorio compilare o il canale fisico oppure i due campi relativi al canale
+            &egrave; calcolato su due campi: uno testuale descrittivo e uno contenente il link per
+            l'accesso. &Egrave; obbligatorio compilare o il canale fisico oppure i due campi relativi al canale
             digitale per seguire le direttive indicate da AGID.
-            Nella tabella sotto si troverà scritto "D" nel caso nel servizio siano compilati
+            Nella tabella sotto si trover&agrave; scritto "D" nel caso nel servizio siano compilati
             i due campi del canale digitale, "F" nel caso sia compilato il canale fisico e
             "D e F" nel caso siano compilati tutti i campi relativi al canale di accesso.</p>
-            <p class="lead"
-             tal:condition="servizi">Si fa notare, infine, che la seguente lista di servizi
-             presenta tutti quei servizi in stato pubblicato e <strong>non scaduti</strong> secondo
-             le logiche di pubblicazione del CMS Plone, ovvero con una data di scadenza ancora da raggiugnere.</p>
+          <p class="lead"
+             tal:condition="servizi"
+          >Si fa notare, infine, che la seguente lista di servizi
+             presenta tutti quei servizi in stato pubblicato e
+            <strong>non scaduti</strong>
+             secondo
+            le logiche di pubblicazione del CMS Plone, ovvero con una data di scadenza ancora da raggiugnere.</p>
 
-        <div class="forms">
-          <div tal:condition="view/cds">
-            <form action="${context/absolute_url}/@@check_servizi">
-              <input type="Submit" value="Cerca senza condizioni di servizio" />
-            </form>
-          </div>
-          <div tal:condition="not:view/cds">
-            <form action="${context/absolute_url}/@@check_servizi">
-              <input type="hidden" name="condizioni_di_servizio" value="True" />
-              <input type="Submit" value="Cerca con condizioni di servizio" />
-            </form>
-          </div>
-          <div class="download_button">
+          <div class="forms">
+            <div tal:condition="view/cds">
+              <form action="${context/absolute_url}/@@check_servizi">
+                <input type="Submit"
+                       value="Cerca senza condizioni di servizio"
+                />
+              </form>
+            </div>
+            <div tal:condition="not:view/cds">
+              <form action="${context/absolute_url}/@@check_servizi">
+                <input name="condizioni_di_servizio"
+                       type="hidden"
+                       value="True"
+                />
+                <input type="Submit"
+                       value="Cerca con condizioni di servizio"
+                />
+              </form>
+            </div>
+            <div class="download_button">
               <form action="${context/absolute_url}/@@download_check_servizi">
                 <!-- <input type="Submit" value="Download" /> -->
-                <input tal:condition="view/cds" type="hidden" name="condizioni_di_servizio" value="True" />
-                <button type="submit" class="download-button">
-                  <i class="fas fa-download"></i> Download servizi
+                <input name="condizioni_di_servizio"
+                       type="hidden"
+                       value="True"
+                       tal:condition="view/cds"
+                />
+                <button class="download-button"
+                        type="submit"
+                >
+                  <i class="fas fa-download"></i>
+                   Download servizi
                 </button>
               </form>
+            </div>
           </div>
-        </div>
 
 
 
@@ -166,7 +190,7 @@
                       <th><span>Tempi e scadenze</span></th>
                       <th><span>Unit&agrave; org. responsabile</span></th>
                       <th><span>Contatti</span></th>
-                      </tr>
+                    </tr>
                   </thead>
                   <tbody>
                     <tr tal:repeat="servizio python:servizi[categoria]['children']">

diff --git a/src/design/plone/contenttypes/profiles/default/types/News_Item.xml b/src/design/plone/contenttypes/profiles/default/types/News_Item.xml
@@ -5,15 +5,21 @@
         i18n:domain="plone"
 >
 
-  <property name="title" i18n:translate="">Notizie e comunicati stampa</property>
+  <property name="title"
+            i18n:translate=""
+  >Notizie e comunicati stampa</property>
 
-  <property name="allowed_content_types" purge="False">
+  <property name="allowed_content_types"
+            purge="False"
+  >
     <element value="Document" />
     <element value="Image" />
     <element value="File" />
   </property>
 
-  <property name="behaviors" purge="False">
+  <property name="behaviors"
+            purge="False"
+  >
     <element value="volto.preview_image" />
     <element value="design.plone.contenttypes.behavior.news" />
     <element value="design.plone.contenttypes.behavior.argomenti_news" />
@@ -22,7 +28,9 @@
     <element value="plone.translatable" />
     <element value="kitconcept.seo" />
     <element value="collective.taxonomy.generated.tipologia_notizia" />
-    <element remove="True" value="design.plone.contenttypes.behavior.argomenti" />
+    <element remove="True"
+             value="design.plone.contenttypes.behavior.argomenti"
+    />
   </property>
 
 </object>
diff --git a/src/design/plone/contenttypes/profiles/default/types/Servizio.xml b/src/design/plone/contenttypes/profiles/default/types/Servizio.xml
@@ -6,8 +6,12 @@
 >
 
   <!-- Basic properties -->
-  <property name="title" i18n:translate="">Servizio</property>
-  <property name="description" i18n:translate=""/>
+  <property name="title"
+            i18n:translate=""
+  >Servizio</property>
+  <property name="description"
+            i18n:translate=""
+  />
 
   <property name="allow_discussion">False</property>
   <property name="factory">Servizio</property>
@@ -17,7 +21,9 @@
   <!-- Hierarchy control -->
   <property name="global_allow">True</property>
   <property name="filter_content_types">True</property>
-  <property name="allowed_content_types" purge="False">
+  <property name="allowed_content_types"
+            purge="False"
+  >
     <element value="CartellaModulistica" />
     <element value="Document" />
     <element value="Image" />

diff --git a/src/design/plone/contenttypes/profiles/fix_syndication/registry.xml b/src/design/plone/contenttypes/profiles/fix_syndication/registry.xml
@@ -2,10 +2,10 @@
 <registry xmlns:i18n="http://xml.zope.org/namespaces/i18n"
           i18n:domain="plone"
 >
-    <!--
+  <!--
         Done exactly like plone 6 do: this interface is moved, so ISiteSyndicationSettings right now is
         pointing to plone.base.interfaces.syndication.ISiteSyndicationSettings. New ISiteSyndicationSettings
         under CMFPlone is a sort of proxy to the new one.
     -->
-    <records interface="Products.CMFPlone.interfaces.syndication.ISiteSyndicationSettings" />
+  <records interface="Products.CMFPlone.interfaces.syndication.ISiteSyndicationSettings" />
 </registry>
diff --git a/src/design/plone/contenttypes/restapi/deserializers/documento.py b/src/design/plone/contenttypes/restapi/deserializers/documento.py
@@ -131,10 +131,10 @@ def __call__(
                         errors.append(
                             new_error("Il campo {} è obbligatorio".format(field))
                         )
-                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e siamo
-                    # su un sito che ha avuto upgrade alla versione pnrr può essere che
-                    # dei campi obbligatori un tempo non lo fossero e quindi arriviamo
-                    # fino a qui
+                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e
+                    # siamo su un sito che ha avuto upgrade alla versione pnrr può
+                    # essere che dei campi obbligatori un tempo non lo fossero e quindi
+                    # arriviamo fino a qui
                     if field not in data and not text_in_block(
                         getattr(self.context, field)
                     ):

diff --git a/src/design/plone/contenttypes/restapi/deserializers/news.py b/src/design/plone/contenttypes/restapi/deserializers/news.py
@@ -131,10 +131,10 @@ def __call__(
                         errors.append(
                             new_error("Il campo {} è obbligatorio".format(field))
                         )
-                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e siamo
-                    # su un sito che ha avuto upgrade alla versione pnrr può essere che
-                    # dei campi obbligatori un tempo non lo fossero e quindi arriviamo
-                    # fino a qui
+                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e
+                    # siamo su un sito che ha avuto upgrade alla versione pnrr può
+                    # essere che dei campi obbligatori un tempo non lo fossero e quindi
+                    # arriviamo fino a qui
                     if field not in data and not text_in_block(
                         getattr(self.context, field)
                     ):

diff --git a/src/design/plone/contenttypes/restapi/deserializers/servizio.py b/src/design/plone/contenttypes/restapi/deserializers/servizio.py
@@ -135,10 +135,10 @@ def __call__(
                         errors.append(
                             new_error("Il campo {} è obbligatorio".format(field))
                         )
-                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e siamo
-                    # su un sito che ha avuto upgrade alla versione pnrr può essere che
-                    # dei campi obbligatori un tempo non lo fossero e quindi arriviamo
-                    # fino a qui
+                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e
+                    # siamo su un sito che ha avuto upgrade alla versione pnrr può
+                    # essere che dei campi obbligatori un tempo non lo fossero e quindi
+                    # arriviamo fino a qui
                     if field not in data and not text_in_block(
                         getattr(self.context, field)
                     ):

diff --git a/src/design/plone/contenttypes/restapi/deserializers/unitaorganizzativa.py b/src/design/plone/contenttypes/restapi/deserializers/unitaorganizzativa.py
@@ -138,10 +138,10 @@ def __call__(
                             new_error("Il campo {} è obbligatorio".format(field))
                         )
 
-                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e siamo
-                    # su un sito che ha avuto upgrade alla versione pnrr può essere che
-                    # dei campi obbligatori un tempo non lo fossero e quindi arriviamo
-                    # fino a qui
+                    # Se siamo nella patch siamo in modifica. Se siamo in modifica e
+                    # siamo su un sito che ha avuto upgrade alla versione pnrr può
+                    # essere che dei campi obbligatori un tempo non lo fossero e quindi
+                    # arriviamo fino a qui
                     if field not in data and not text_in_block(
                         getattr(self.context, field)
                     ):

diff --git a/src/design/plone/contenttypes/restapi/serializers/dxfields.py b/src/design/plone/contenttypes/restapi/serializers/dxfields.py
@@ -25,6 +25,7 @@
 import json
 import re
 
+
 RESOLVEUID_RE = re.compile(".*?/resolve[Uu]id/([^/]*)/?(.*)$")
 KEYS_WITH_URL = ["linkUrl", "navigationRoot", "showMoreLink"]
 

diff --git a/src/design/plone/contenttypes/restapi/serializers/summary.py b/src/design/plone/contenttypes/restapi/serializers/summary.py
@@ -1,5 +1,4 @@
 # -*- coding: utf-8 -*-
-import logging
 from collective.taxonomy import PATH_SEPARATOR
 from collective.taxonomy.interfaces import ITaxonomy
 from design.plone.contenttypes.interfaces import IDesignPloneContenttypesLayer
@@ -30,6 +29,7 @@
 from zope.interface import Interface
 from zope.schema import getFieldsInOrder
 
+import logging
 import re
 
 
@@ -260,11 +260,21 @@ def __call__(self, force_all_metadata=False):
         else:
             res["compensi"] = json_compatible([])
 
-        if safe_hasattr(self.context, "compensi-file") and getattr(self.context, "compensi-file").getFolderContents():
+        if (
+            safe_hasattr(self.context, "compensi-file")
+            and getattr(self.context, "compensi-file").getFolderContents()
+        ):
             res["compensi_file"] = getattr(self.context, "compensi-file").absolute_url()
 
-        if safe_hasattr(self.context, "importi-di-viaggio-e-o-servizi") and getattr(self.context, "importi-di-viaggio-e-o-servizi").getFolderContents():
-            res["importi_di_viaggio_e_o_servizi"] = getattr(self.context, "importi-di-viaggio-e-o-servizi").absolute_url()
+        if (
+            safe_hasattr(self.context, "importi-di-viaggio-e-o-servizi")
+            and getattr(
+                self.context, "importi-di-viaggio-e-o-servizi"
+            ).getFolderContents()
+        ):
+            res["importi_di_viaggio_e_o_servizi"] = getattr(
+                self.context, "importi-di-viaggio-e-o-servizi"
+            ).absolute_url()
 
         if "atto_di_nomina" not in res:
             res["atto_di_nomina"] = None

diff --git a/src/design/plone/contenttypes/restapi/types/adapters.py b/src/design/plone/contenttypes/restapi/types/adapters.py
@@ -3,13 +3,13 @@
 from design.plone.contenttypes import _
 from design.plone.contenttypes.interfaces import IDesignPloneContenttypesLayer
 from plone.restapi.types.adapters import ObjectJsonSchemaProvider
-from redturtle.volto.types.adapters import (
-    TextLineJsonSchemaProvider as BaseTextLineJsonSchemaProvider,
-)
 from plone.restapi.types.interfaces import IJsonSchemaProvider
 from plone.restapi.types.utils import get_fieldsets
 from plone.restapi.types.utils import get_jsonschema_properties
 from plone.restapi.types.utils import iter_fields
+from redturtle.volto.types.adapters import (
+    TextLineJsonSchemaProvider as BaseTextLineJsonSchemaProvider,
+)
 from zope.component import adapter
 from zope.component import getUtility
 from zope.i18n import translate
@@ -19,6 +19,7 @@
 from zope.schema.interfaces import ITextLine
 from zope.schema.interfaces import IVocabularyFactory
 
+
 DATAGRID_FIELDS = ["value_punto_contatto", "timeline_tempi_scadenze"]
 
 

diff --git a/src/design/plone/contenttypes/tests/test_ct_servizio.py b/src/design/plone/contenttypes/tests/test_ct_servizio.py
@@ -1,11 +1,11 @@
 # -*- coding: utf-8 -*-
 """Setup tests for this package."""
-from design.plone.contenttypes.testing import (
+from design.plone.contenttypes.testing import (  # noqa
     DESIGN_PLONE_CONTENTTYPES_API_FUNCTIONAL_TESTING,
-)  # noqa
-from design.plone.contenttypes.testing import (
+)
+from design.plone.contenttypes.testing import (  # noqa
     DESIGN_PLONE_CONTENTTYPES_INTEGRATION_TESTING,
-)  # noqa
+)
 from plone import api
 from plone.app.testing import setRoles
 from plone.app.testing import SITE_OWNER_NAME