imio.smartweb.core 1.3__py3-none-any.whl → 1.3.1__py3-none-any.whl

imio/smartweb/core/browser/configure.zcml

@@ -96,6 +96,22 @@
       layer="imio.smartweb.core.interfaces.IImioSmartwebCoreLayer"
       />
 
+  <adapter
+      factory="imio.smartweb.core.browser.sitemap.SitemapNavtreeStrategy"
+      provides="imio.smartweb.core.browser.sitemap.ISmartwebNavtreeStrategy"
+      for="*
+           imio.smartweb.core.interfaces.IImioSmartwebCoreLayer"
+      />
+
+  <browser:page
+      name="sitemap_builder_view"
+      for="*"
+      class=".sitemap.CatalogSiteMap"
+      allowed_attributes="siteMap"
+      permission="zope.Public"
+      layer="imio.smartweb.core.interfaces.IImioSmartwebCoreLayer"
+      />
+
   <browser:page
       name="events_view"
       for="*"

imio/smartweb/core/browser/search/search.py

@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
 
+from imio.smartweb.core.contents.rest.search.endpoint import get_default_view_url
 from plone import api
 from Products.CMFPlone.browser.search import Search
 from Products.CMFPlone.utils import normalizeString
@@ -68,6 +69,20 @@ class Search(Search):
 
         return json.dumps(response)
 
+    @property
+    def are_views_available(self):
+        news = False if get_default_view_url("news") == "" else True
+        events = False if get_default_view_url("events") == "" else True
+        directory = False if get_default_view_url("directory") == "" else True
+        return json.dumps({"news": news, "events": events, "directory": directory})
+
+    @property
+    def context_user_has_roles(self):
+        # is current user has more role(s) than just authenticated
+        if len(api.user.get_roles()) > 1:
+            return True
+        return False
+
     @property
     def current_language(self):
         return api.portal.get_current_language()[:2]

imio/smartweb/core/browser/sitemap.py

@@ -1,49 +1,108 @@
+from Acquisition import aq_inner
 from BTrees.OOBTree import OOBTree
 from imio.smartweb.core.config import DIRECTORY_URL
 from imio.smartweb.core.config import EVENTS_URL
 from imio.smartweb.core.config import NEWS_URL
-from imio.smartweb.core.contents.rest.utils import get_auth_sources_response
-from imio.smartweb.core.contents.rest.utils import get_entity_id
-from plone import api
-from plone.base.interfaces import IPloneSiteRoot
+from imio.smartweb.core.contents.rest.directory.endpoint import DirectoryEndpointGet
+from imio.smartweb.core.contents.rest.events.endpoint import EventsEndpointGet
+from imio.smartweb.core.contents.rest.news.endpoint import NewsEndpointGet
+from imio.smartweb.core.interfaces import IImioSmartwebCoreLayer
+from plone.app.layout.navigation.navtree import buildFolderTree
 from plone.app.layout.sitemap.sitemap import SiteMapView
-from plone.registry.interfaces import IRegistry
+from plone.base.interfaces import IPloneSiteRoot
+from Products.CMFPlone.browser.navigation import CatalogSiteMap as BaseCatalogSiteMap
+from Products.CMFPlone.browser.navtree import (
+    SitemapNavtreeStrategy as BaseSitemapNavtreeStrategy,
+)
+from Products.CMFPlone.browser.navtree import (
+    SitemapQueryBuilder as BaseSitemapQueryBuilder,
+)
 from Products.CMFCore.utils import getToolByName
 from Products.CMFPlone.utils import normalizeString
-from zope.component import getUtility
+from zope.component import getMultiAdapter
+from zope.interface import implementer
+from zope.interface import Interface
 
 import logging
+import Missing
 
 logger = logging.getLogger("imio.smartweb.core")
 
 
+FRIENDLY_TYPES = [
+    "Collection",
+    "Image",
+    "Link",
+    "imio.smartweb.Folder",
+    "imio.smartweb.Page",
+    "imio.smartweb.PortalPage",
+    "imio.smartweb.Procedure",
+    "imio.smartweb.CirkwiView",
+    "imio.smartweb.DirectoryView",
+    "imio.smartweb.EventsView",
+    "imio.smartweb.NewsView",
+]
+
+
+def get_endpoint_data(obj, request):
+    """Retourne les données issues du bon endpoint pour un objet donné"""
+    endpoint_mapping = {
+        "imio.smartweb.DirectoryView": DirectoryEndpointGet,
+        "imio.smartweb.EventsView": EventsEndpointGet,
+        "imio.smartweb.NewsView": NewsEndpointGet,
+    }
+
+    endpoint_class = endpoint_mapping.get(obj.portal_type)
+    if not endpoint_class:
+        return {}
+
+    endpoint = endpoint_class()
+    batch_size = 1000 if obj.portal_type == "imio.smartweb.DirectoryView" else 365
+    return (
+        endpoint.reply_for_given_object(
+            obj, request, fullobjects=0, batch_size=batch_size
+        )
+        or {}
+    )
+
+
+def format_sitemap_items(items, base_url):
+    """Formatte les items pour le sitemap"""
+    formatted_items = []
+    for item in items:
+        item_id = normalizeString(item.get("title"))
+        item_uid = item.get("id")
+        lastmod = item.get("modified") or "1970-01-01T00:00:00Z"
+        formatted_items.append(
+            {
+                "loc": f"{base_url}/{item_id}?u={item_uid}",
+                "lastmod": lastmod,
+                "Title": item_id,
+                "Description": item.get("description", ""),
+                "getURL": f"{base_url}/{item_id}?u={item_uid}",
+                "getRemoteUrl": Missing.Value,
+                "currentItem": False,
+                "currentParent": False,
+                "normalized_review_state": "published",
+                "normalized_portal_type": "imio-smartweb-directory-item",
+            }
+        )
+    return formatted_items
+
+
 class CustomSiteMapView(SiteMapView):
+    """Custom sitemap view. (get items for sitemap.xml.gz)"""
 
     def objects(self):
         """Returns the data to create the sitemap."""
-
-        friendlytypes = [
-            "Link",
-            "imio.smartweb.Folder",
-            "LIF",
-            "LRF",
-            "imio.smartweb.Page",
-            "imio.smartweb.Procedure",
-            "File",
-            "Collection",
-            "imio.smartweb.PortalPage",
-            "Image",
-            "imio.smartweb.CirkwiView",
-        ]
+        friendlytypes = FRIENDLY_TYPES
 
         catalog = getToolByName(self.context, "portal_catalog")
-        query = {}
-        utils = getToolByName(self.context, "plone_utils")
-        query["portal_type"] = utils.getUserFriendlyTypes(friendlytypes)
-        registry = getUtility(IRegistry)
-        typesUseViewActionInListings = frozenset(
-            registry.get("plone.types_use_view_action_in_listings", [])
-        )
+        query = {
+            "portal_type": getToolByName(
+                self.context, "plone_utils"
+            ).getUserFriendlyTypes(friendlytypes)
+        }
 
         is_plone_site_root = IPloneSiteRoot.providedBy(self.context)
         if not is_plone_site_root:
@@ -56,93 +115,82 @@ class CustomSiteMapView(SiteMapView):
             value = (item.modified.micros(), item.modified.ISO8601())
             default_page_modified[key] = value
 
-        # The plone site root is not catalogued.
         if is_plone_site_root:
             loc = self.context.absolute_url()
             date = self.context.modified()
-            # Comparison must be on GMT value
-            modified = (date.micros(), date.ISO8601())
-            default_modified = default_page_modified.get(loc, None)
-            if default_modified is not None:
-                modified = max(modified, default_modified)
-            lastmod = modified[1]
-            yield {
-                "loc": loc,
-                "lastmod": lastmod,
-                # 'changefreq': 'always',
-                #  hourly/daily/weekly/monthly/yearly/never
-                # 'prioriy': 0.5, # 0.0 to 1.0
-            }
+            modified = max(
+                (date.micros(), date.ISO8601()), default_page_modified.get(loc, (0, ""))
+            )
+            yield {"loc": loc, "lastmod": modified[1]}
 
         query["is_default_page"] = False
         for item in catalog.searchResults(query):
             loc = item.getURL()
             date = item.modified
-            # Comparison must be on GMT value
-            modified = (date.micros(), date.ISO8601())
-            default_modified = default_page_modified.get(loc, None)
-            if default_modified is not None:
-                modified = max(modified, default_modified)
-            lastmod = modified[1]
-            if item.portal_type in typesUseViewActionInListings:
-                loc += "/view"
-            yield {
-                "loc": loc,
-                "lastmod": lastmod,
-            }
-
-        auth_sources = {
-            "directory": {
-                "entity_reg_var": "smartweb.directory_entity_uid",
-                "main_rest_view": "smartweb.default_directory_view",
-                "vocabulary": "imio.smartweb.vocabulary.RemoteDirectoryEntities",
-                "url": DIRECTORY_URL,
-            },
-            "events": {
-                "entity_reg_var": "smartweb.events_entity_uid",
-                "main_rest_view": "smartweb.default_events_view",
-                "vocabulary": "imio.smartweb.vocabulary.RemoteEventsEntities",
-                "url": EVENTS_URL,
-            },
-            "news": {
-                "entity_reg_var": "smartweb.news_entity_uid",
-                "main_rest_view": "smartweb.default_news_view",
-                "vocabulary": "imio.smartweb.vocabulary.RemoteNewsEntities",
-                "url": NEWS_URL,
-            },
-        }
-        for auth_source_key, auth_source_value in auth_sources.items():
-            entity_uid = api.portal.get_registry_record(
-                auth_source_value.get("entity_reg_var")
-            )
-            if entity_uid is None:
-                return self.request.response.setStatus(404, "No entity found")
-            entity_id = get_entity_id(auth_source_value.get("vocabulary"), entity_uid)
-            auth_source_uid = api.portal.get_registry_record(
-                f"smartweb.default_{auth_source_key}_view", default=None
+            modified = max(
+                (date.micros(), date.ISO8601()), default_page_modified.get(loc, (0, ""))
             )
-            if auth_source_uid is None:
-                return self.request.response.setStatus(
-                    404, "No default authentic source found"
-                )
-            obj = api.content.get(UID=auth_source_uid)
-            if obj is None:
-                logger.warning(
-                    f"Seems that a main authentic view (for {auth_source_key}) is not found"
-                )
-                continue
-            auth_source_view_url = obj.absolute_url()
-            results = get_auth_sources_response(
-                auth_source_key, normalizeString(entity_id), (60 * 60 * 24)
-            ).json()
-            if results is None or results.get("type") == "ValueError":
+            yield {"loc": loc, "lastmod": modified[1]}
+
+        for brain in catalog(
+            portal_type=[
+                "imio.smartweb.EventsView",
+                "imio.smartweb.NewsView",
+                "imio.smartweb.DirectoryView",
+            ]
+        ):
+            obj = brain.getObject()
+            data = get_endpoint_data(obj, obj.REQUEST)
+            yield from format_sitemap_items(data.get("items", {}), obj.absolute_url())
+
+
+@implementer(IImioSmartwebCoreLayer)
+class CatalogSiteMap(BaseCatalogSiteMap):
+    def siteMap(self):
+        context = aq_inner(self.context)
+
+        queryBuilder = SitemapQueryBuilder(context)
+        query = queryBuilder()
+        strategy = getMultiAdapter((context, self), ISmartwebNavtreeStrategy)
+        base_folder_tree = buildFolderTree(
+            context, obj=context, query=query, strategy=strategy
+        )
+
+        for child in base_folder_tree.get("children"):
+            obj = child.get("item").getObject()
+            data = get_endpoint_data(obj, obj.REQUEST)
+            if not data:
                 continue
-            for item in results.get("items"):
-                item_id = normalizeString(item.get("title"))
-                item_uid = item.get("id")
-                loc = f"{auth_source_view_url}/{item_id}?u={item_uid}"
-                lastmod = item.get("modified")
-                yield {
-                    "loc": loc,
-                    "lastmod": lastmod,
-                }
+            child["children"] = format_sitemap_items(
+                data.get("items", []), obj.absolute_url()
+            )
+
+        return base_folder_tree
+
+
+class SitemapQueryBuilder(BaseSitemapQueryBuilder):
+    """Construit la requête pour le sitemap avec des types de contenu personnalisés."""
+
+    def __init__(self, context):
+        self.context = context
+
+    def __call__(self):
+        query = {}
+        custom_query = getattr(self.context, "getCustomNavQuery", None)
+        if custom_query and callable(custom_query):
+            query = custom_query()
+        query["portal_type"] = FRIENDLY_TYPES
+        query["review_state"] = "published"
+        return query
+
+
+class ISmartwebNavtreeStrategy(Interface):
+    """Marker interface for the smartweb navtree strategy."""
+
+
+@implementer(ISmartwebNavtreeStrategy)
+class SitemapNavtreeStrategy(BaseSitemapNavtreeStrategy):
+
+    def nodeFilter(self, node):
+        # Return all nodes in sitemap even if they are exclude from nav
+        return True

imio/smartweb/core/contents/rest/base.py

@@ -15,9 +15,12 @@ class BaseEndpoint(object):
     language = "fr"
     remote_endpoint = ""
 
-    def __init__(self, context, request):
+    def __init__(self, context, request, fullobjects=1, batch_size=0):
+
         self.context = context
         self.request = request
+        self.fullobjects = fullobjects
+        self.batch_size = batch_size
 
     def __call__(self):
         results = get_json(self.query_url, timeout=20)
@@ -46,7 +49,7 @@ class BaseEndpoint(object):
         item[f"{field}_full_scale"] = (
             f"{item['@id']}/@@images/{field}/?cache_key={modified_hash}"
         )
-        del item[field]
+        item.pop(field, None)
 
     def construct_query_string(self, params):
         params = "&".join(params)

imio/smartweb/core/contents/rest/campaign/endpoint.py

@@ -107,7 +107,7 @@ class CampaignEndpoint(BaseTsEndpoint):
             extra_params = [f"{k}={v}" for k, v in self.request.form.items()]
             extra_params = "&".join(extra_params)
             campaign_id = self.context.linked_campaign
-            url = f"{wcs_api}/cards/imio-ideabox-projet/list?campagne={campaign_id}&full=on&filter-statut=Vote|Enregistr%C3%A9e&filter-statut-operator=in&{extra_params}"
+            url = f"{wcs_api}/cards/imio-ideabox-projet/list?filter-campagne={campaign_id}&full=on&filter-statut=Vote|Enregistr%C3%A9e&filter-statut-operator=in&{extra_params}"
         return url
 
 
@@ -124,7 +124,7 @@ class ZonesEndpoint(BaseTsEndpoint):
     def query_url(self):
         wcs_api = get_ts_api_url("wcs")
         campaign_id = self.context.linked_campaign
-        return f"{wcs_api}/cards/imio-ideabox-zone/list?campagne={campaign_id}"
+        return f"{wcs_api}/cards/imio-ideabox-zone/list?filter-campagne={campaign_id}"
 
 
 @implementer(IExpandableElement)
@@ -140,7 +140,7 @@ class TsTopicsEndpoint(BaseTsEndpoint):
     def query_url(self):
         wcs_api = get_ts_api_url("wcs")
         campaign_id = self.context.linked_campaign
-        return f"{wcs_api}/cards/imio-ideabox-theme/list?campagne={campaign_id}"
+        return f"{wcs_api}/cards/imio-ideabox-theme/list?filter-campagne={campaign_id}"
 
 
 @implementer(IExpandableElement)

imio/smartweb/core/contents/rest/directory/endpoint.py

@@ -13,7 +13,7 @@ from zope.interface import Interface
 
 class BaseDirectoryEndpoint(BaseEndpoint):
     def __call__(self):
-        results = super(BaseDirectoryEndpoint, self).__call__()
+        results = super(BaseDirectoryEndpoint, self).__call__() or {}
         if not results.get("items"):
             return results
         orientation = self.context.orientation
@@ -45,10 +45,13 @@ class BaseDirectoryEndpoint(BaseEndpoint):
             "metadata_fields=taxonomy_contact_category",
             "metadata_fields=topics",
             "metadata_fields=has_leadimage",
-            "fullobjects=1",
+            "fullobjects={}".format(self.fullobjects),
             "sort_on=sortable_title",
-            "b_size={}".format(self.context.nb_results),
         ]
+        if self.batch_size == 0:
+            params.append("b_size={}".format(self.context.nb_results))
+        else:
+            params.append("b_size={}".format(self.batch_size))
         if self.context.selected_categories is not None:
             for category in self.context.selected_categories:
                 params.append(f"taxonomy_contact_category.query={category}")
@@ -74,6 +77,11 @@ class DirectoryEndpointGet(BaseService):
     def reply(self):
         return DirectoryEndpoint(self.context, self.request)()
 
+    def reply_for_given_object(self, obj, request, fullobjects=1, batch_size=0):
+        return DirectoryEndpoint(
+            obj, request, fullobjects=fullobjects, batch_size=batch_size
+        )()
+
 
 class DirectoryFiltersEndpointGet(BaseService):
     def reply(self):

imio/smartweb/core/contents/rest/events/endpoint.py

@@ -12,7 +12,7 @@ from zope.interface import Interface
 
 class BaseEventsEndpoint(BaseEndpoint):
     def __call__(self):
-        results = super(BaseEventsEndpoint, self).__call__()
+        results = super(BaseEventsEndpoint, self).__call__() or {}
         if not results or not results.get("items"):
             return results
         orientation = self.context.orientation
@@ -43,9 +43,12 @@ class BaseEventsEndpoint(BaseEndpoint):
             "metadata_fields=has_leadimage",
             "metadata_fields=UID",
             "sort_on=event_dates",
-            "fullobjects=1",
-            "b_size={}".format(self.context.nb_results),
+            "fullobjects={}".format(self.fullobjects),
         ]
+        if self.batch_size == 0:
+            params.append("b_size={}".format(self.context.nb_results))
+        else:
+            params.append("b_size={}".format(self.batch_size))
         if self.context.selected_event_types is not None:
             for event_type in self.context.selected_event_types:
                 params.append(f"event_type={event_type}")
@@ -70,6 +73,11 @@ class EventsEndpointGet(BaseService):
     def reply(self):
         return EventsEndpoint(self.context, self.request)()
 
+    def reply_for_given_object(self, obj, request, fullobjects=1, batch_size=0):
+        return EventsEndpoint(
+            obj, request, fullobjects=fullobjects, batch_size=batch_size
+        )()
+
 
 class EventsFiltersEndpointGet(BaseService):
     def reply(self):

imio/smartweb/core/contents/rest/news/endpoint.py

@@ -12,7 +12,7 @@ from zope.interface import Interface
 
 class BaseNewsEndpoint(BaseEndpoint):
     def __call__(self):
-        results = super(BaseNewsEndpoint, self).__call__()
+        results = super(BaseNewsEndpoint, self).__call__() or {}
         if not results.get("items"):
             return results
         orientation = self.context.orientation
@@ -46,9 +46,12 @@ class BaseNewsEndpoint(BaseEndpoint):
             "metadata_fields=UID",
             "sort_on=effective",
             "sort_order=descending",
-            "b_size={}".format(self.context.nb_results),
-            "fullobjects=1",
+            "fullobjects={}".format(self.fullobjects),
         ]
+        if self.batch_size == 0:
+            params.append("b_size={}".format(self.context.nb_results))
+        else:
+            params.append("b_size={}".format(self.batch_size))
         params = self.construct_query_string(params)
         url = f"{NEWS_URL}/{self.remote_endpoint}?{params}"
         return url
@@ -70,6 +73,11 @@ class NewsEndpointGet(BaseService):
     def reply(self):
         return NewsEndpoint(self.context, self.request)()
 
+    def reply_for_given_object(self, obj, request, fullobjects=1, batch_size=0):
+        return NewsEndpoint(
+            obj, request, fullobjects=fullobjects, batch_size=batch_size
+        )()
+
 
 class NewsFiltersEndpointGet(BaseService):
     def reply(self):

imio/smartweb/core/contents/rest/search/endpoint.py

@@ -57,7 +57,10 @@ def get_default_view_url(view_type):
     if not record:
         return ""
     with api.env.adopt_user(username="admin"):
-        return api.content.get(UID=record).absolute_url()
+        obj = api.content.get(UID=record)
+        if not obj:
+            return ""
+        return obj.absolute_url()
 
 
 def get_views_mapping(navigation_root):

imio/smartweb/core/contents/rest/utils.py

@@ -69,10 +69,10 @@ def get_auth_sources_response(
                 "v": wf_status,
             },
         ],
-        "metadata_fields": ["title", "modified"],
+        "metadata_fields": ["title", "modified", "id"],
         "sort_on": "effective",
         "sort_order": "descending",
-        "fullobjects": True,
+        "fullobjects": False,
         "b_start": 0,
         "b_size": 4000,
     }
@@ -81,7 +81,7 @@ def get_auth_sources_response(
     auth = get_wca_token(client_id, client_secret)
     headers = {
         "Accept": "application/json",
-        "Content-Type": "text/plain",
+        "Content-Type": "application/json",
         "Authorization": auth,
     }
     url = f"{url}/@querystring-search"

imio/smartweb/core/interfaces.py

@@ -5,6 +5,7 @@ from collective.messagesviewlet.interfaces import ICollectiveMessagesviewletLaye
 from collective.solr.browser.interfaces import IThemeSpecific
 from imio.smartweb.common.interfaces import IImioSmartwebCommonLayer
 from plone.app.contenttypes.interfaces import IPloneAppContenttypesLayer
+from Products.CMFPlone.browser.interfaces import ISiteMap
 from zope.interface import Interface
 
 
@@ -22,6 +23,7 @@ class IImioSmartwebCoreLayer(
     ILayerSpecific,
     IThemeSpecific,
     ICollectiveMessagesviewletLayer,
+    ISiteMap,
 ):
     """Marker interface that defines a browser layer."""
 

imio/smartweb/core/tests/test_ideabox.py

@@ -85,6 +85,38 @@ class TestIdeabox(ImioSmartwebTestCase):
         self.assertEqual(campaign_view.title, "Sprint iMio Fall 2024")
         self.assertEqual(campaign_view.id, "sprint-imio-fall-2024")
 
+    # CampaignNameChooser : Give th Title to our object thanks to e-guichet object Title.
+    @patch("imio.smartweb.core.subscribers.get_basic_auth_json")
+    @patch("imio.smartweb.core.subscribers.get_value_from_registry")
+    @patch("imio.smartweb.core.contents.rest.campaign.content.get_basic_auth_json")
+    @patch("imio.smartweb.core.contents.rest.campaign.content.get_value_from_registry")
+    def test_campaign_name_chooser(
+        self,
+        m_get_value_from_registry_namechooser,
+        m_get_basic_auth_json_namechooser,
+        m_get_value_from_registry,
+        m_get_basic_auth_json,
+    ):
+        from imio.smartweb.core.contents.rest.campaign.content import CampaignView
+        from zope.component import queryAdapter
+        from zope.container.interfaces import INameChooser
+
+        m_get_value_from_registry.return_value = (
+            "https://staging3-formulaires.guichet-citoyen.be/api"
+        )
+        m_get_basic_auth_json.return_value = self.json_campaign_raw_mock
+
+        m_get_value_from_registry_namechooser.return_value = (
+            "https://staging3-formulaires.guichet-citoyen.be/api"
+        )
+        m_get_basic_auth_json_namechooser.return_value = self.json_campaign_raw_mock
+        campaign_view = CampaignView(id="test-campaign", title=None)
+        campaign_view.linked_campaign = "2"
+        name_chooser = queryAdapter(self.folder, INameChooser)
+        generated_id = name_chooser.chooseName(None, campaign_view)
+        self.assertEqual(campaign_view.title, "Sprint iMio Fall 2024")
+        self.assertEqual(generated_id, "sprint-imio-fall-2024")
+
     @requests_mock.Mocker()
     @patch("imio.smartweb.core.subscribers.get_basic_auth_json")
     @patch("imio.smartweb.core.subscribers.get_value_from_registry")
@@ -110,7 +142,7 @@ class TestIdeabox(ImioSmartwebTestCase):
         url = endpoint.query_url
         self.assertEqual(
             url,
-            "https://demo-formulaires.guichet-citoyen.be/api/cards/imio-ideabox-projet/list?campagne=2&full=on&filter-statut=Vote|Enregistr%C3%A9e&filter-statut-operator=in&",
+            "https://demo-formulaires.guichet-citoyen.be/api/cards/imio-ideabox-projet/list?filter-campagne=2&full=on&filter-statut=Vote|Enregistr%C3%A9e&filter-statut-operator=in&",
         )
 
         m.get(url, text=json.dumps({}))

imio/smartweb/core/tests/test_rest.py

@@ -18,7 +18,6 @@ from plone.app.testing import TEST_USER_ID
 from plone.app.testing import TEST_USER_PASSWORD
 from plone.namedfile.file import NamedBlobImage
 from plone.restapi.testing import RelativeSession
-from unittest import mock
 from unittest.mock import patch
 from urllib.parse import urlparse
 from urllib.parse import parse_qs
@@ -29,7 +28,6 @@ from ZPublisher.pubevents import PubStart
 from imio.smartweb.core.viewlets.ogptags import OgpTagsViewlet
 
 import json
-import os
 import requests_mock
 import transaction
 
@@ -444,8 +442,8 @@ class SectionsFunctionalTest(ImioSmartwebTestCase):
             "metadata_fields=UID&"
             "sort_on=effective&"
             "sort_order=descending&"
-            "b_size=20&"
             "fullobjects=1&"
+            "b_size=20&"
             "translated_in_en=1".format(self.rest_news.selected_news_folder),
         )
         m.get(url, text=json.dumps({}))