Add converter for query param browse

Author: Tschuppi81

src/onegov/agency/collections/agencies.py

@@ -36,10 +36,8 @@ class ExtendedAgencyCollection(AdjacencyListCollection[ExtendedAgency]):
     def __init__(
         self,
         session: Session,
-        # FIXME: These really should be float/int, we just need to add
-        #        convertes to the path configuration...
         root_pdf_modified: str | None = None,
-        browse: str | None = None
+        browse: int | None = None
     ) -> None:
         super().__init__(session)
         self.root_pdf_modified = root_pdf_modified

src/onegov/agency/layout.py

@@ -128,7 +128,7 @@ def browsed_agency(self) -> ExtendedAgency | None:
             isinstance(self.model, ExtendedAgencyCollection)
             and self.model.browse
         ):
-            return self.model.by_id(self.model.browse)  # type:ignore[arg-type]
+            return self.model.by_id(self.model.browse)
         return None
 
     @cached_property

src/onegov/agency/path.py

@@ -44,11 +44,12 @@ def get_people(
 @AgencyApp.path(
     model=ExtendedAgencyCollection,
     path='/organizations',
+    converters={'browse': int}
 )
 def get_agencies(
     app: AgencyApp,
     root_pdf_modified: str | None = None,
-    browse: str | None = None
+    browse: int | None = None
 ) -> ExtendedAgencyCollection:
     return ExtendedAgencyCollection(app.session(), root_pdf_modified, browse)
 

src/onegov/agency/views/agencies.py

@@ -259,7 +259,7 @@ def push_history_state(response: BaseResponse) -> None:
         response.headers.add(
             'X-IC-PushURL',
             request.class_link(
-                ExtendedAgencyCollection, {'browse': str(self.id)})
+                ExtendedAgencyCollection, {'browse': self.id})
         )
 
     return render_macro(