From 548149d82328551ca479910aee986fa5258c4c76 Mon Sep 17 00:00:00 2001
From: Andrea Cecchi <andrea.cecchi85@gmail.com>
Date: Fri, 20 Sep 2024 15:16:07 +0200
Subject: [PATCH] Fix settings (#3)

* add serializer/deserializer for controlpanel

* remove unused field and rename field in settings

* code cleanup
---
 src/iosanita/policy/interfaces.py             | 21 ++-------
 src/iosanita/policy/restapi/configure.zcml    |  1 +
 .../restapi/services/search_filters/get.py    | 46 ++-----------------
 src/iosanita/policy/testing.py                |  2 +-
 4 files changed, 9 insertions(+), 61 deletions(-)

diff --git a/src/iosanita/policy/interfaces.py b/src/iosanita/policy/interfaces.py
index a083340..00e0b06 100644
--- a/src/iosanita/policy/interfaces.py
+++ b/src/iosanita/policy/interfaces.py
@@ -43,29 +43,16 @@ class IIoSanitaSettings(Interface):
         value_type=TextLine(),
     )
 
-    search_sections = SourceText(
-        title=_("search_sections_label", default="Sezioni ricerca"),
+    quick_search = SourceText(
+        title=_("quick_search_label", default="Scorcatoie di ricerca"),
         description=_(
-            "search_sections_help",
-            default="Inserire una lista di sezioni per la ricerca.",
+            "quick_search_help",
+            default="Inserire una lista di scorciatoie per la ricerca.",
         ),
         default="",
         required=False,
     )
 
-    show_modified_default = Bool(
-        title=_("show_modified_default_label", default="Mostra la data di modifica"),
-        description=_(
-            "show_modified_default_help",
-            default="Questo è il valore di default per decidere se mostrare "
-            "o meno la data di modifica nei contenuti che hanno la behavior "
-            "abilitata. E' poi possibile sovrascrivere il default nei singoli "
-            'contenuti (nel tab "Impostazioni").',
-        ),
-        default=True,
-        required=False,
-    )
-
     show_dynamic_folders_in_footer = Bool(
         title=_("show_dynamic_folders_in_footer_label", default="Footer dinamico"),
         description=_(
diff --git a/src/iosanita/policy/restapi/configure.zcml b/src/iosanita/policy/restapi/configure.zcml
index 2d6e335..99c772b 100644
--- a/src/iosanita/policy/restapi/configure.zcml
+++ b/src/iosanita/policy/restapi/configure.zcml
@@ -7,6 +7,7 @@
   <include package=".services" />
 
 
+
   <!-- enable controlpanel on plone.restapi -->
   <adapter
       factory=".controlpanel.IoSanitaSettingsControlpanel"
diff --git a/src/iosanita/policy/restapi/services/search_filters/get.py b/src/iosanita/policy/restapi/services/search_filters/get.py
index 2e0ffe5..204f163 100644
--- a/src/iosanita/policy/restapi/services/search_filters/get.py
+++ b/src/iosanita/policy/restapi/services/search_filters/get.py
@@ -1,10 +1,7 @@
 # -*- coding: utf-8 -*-
-from AccessControl.unauthorized import Unauthorized
 from iosanita.policy.interfaces import IIoSanitaSettings
 from plone import api
-from plone.restapi.interfaces import ISerializeToJsonSummary
 from plone.restapi.services import Service
-from zope.component import getMultiAdapter
 
 import json
 
@@ -12,51 +9,14 @@
 class SearchFiltersGET(Service):
     def reply(self):
         return {
-            "sections": self.get_section_data(field_id="search_sections"),
+            "quick_search": self.get_data_from_registry(field_id="quick_search"),
         }
 
-    def get_section_data(self, field_id):
+    def get_data_from_registry(self, field_id):
         try:
             values = api.portal.get_registry_record(
                 field_id, interface=IIoSanitaSettings, default="[]"
             )
         except KeyError:
             return []
-        utils = api.portal.get_tool(name="plone_utils")
-
-        sections = []
-        for setting in json.loads(values or "[]"):
-            items = []
-            for section_settings in setting.get("items") or []:
-                for uid in section_settings.get("linkUrl") or []:
-                    try:
-                        section = api.content.get(UID=uid)
-                    except Unauthorized:
-                        # private folder
-                        continue
-                    if not section:
-                        continue
-                    item_infos = getMultiAdapter(
-                        (section, self.request),
-                        ISerializeToJsonSummary,
-                    )()
-                    children = section.listFolderContents(
-                        contentFilter={"portal_type": utils.getUserFriendlyTypes()}
-                    )
-                    item_infos["items"] = [
-                        getMultiAdapter(
-                            (x, self.request),
-                            ISerializeToJsonSummary,
-                        )()
-                        for x in children
-                    ]
-                    item_infos["title"] = section_settings.get("title", "")
-                    items.append(item_infos)
-            if items:
-                sections.append(
-                    {
-                        "rootPath": setting.get("rootPath", ""),
-                        "items": items,
-                    }
-                )
-        return sections
+        return json.loads(values or "[]")
diff --git a/src/iosanita/policy/testing.py b/src/iosanita/policy/testing.py
index 2e51f10..0b9c1c4 100644
--- a/src/iosanita/policy/testing.py
+++ b/src/iosanita/policy/testing.py
@@ -9,6 +9,7 @@
 import collective.feedback
 import collective.volto.dropdownmenu
 import collective.volto.enhancedlinks
+import collective.volto.formsupport
 import collective.volto.slimheader
 import collective.volto.socialsettings
 import collective.volto.subfooter
@@ -16,7 +17,6 @@
 import iosanita.policy
 import plone.app.caching
 import souper.plone
-import collective.volto.formsupport
 
 
 class TestLayer(ContentTypesTestLayer):