#175 exception handling + update crab

OnroerendErfgoed · May 4, 2023 · e4024a7 · e4024a7
1 parent 5b24155
commit e4024a7
Show file tree

Hide file tree

Showing 7 changed files with 344 additions and 379 deletions.
diff --git a/crabpy_pyramid/__init__.py b/crabpy_pyramid/__init__.py
@@ -334,6 +334,7 @@ def includeme(config):
         config.add_request_method(get_adressenregister, 'adressenregister_gateway')
         config.include('crabpy_pyramid.routes.adressenregister')
         config.scan('crabpy_pyramid.views.adressenregister')
+        config.scan("crabpy_pyramid.views.exceptions")
 
 
 def main(global_config, **settings):

diff --git a/crabpy_pyramid/renderers/adressenregister.py b/crabpy_pyramid/renderers/adressenregister.py
@@ -1,4 +1,3 @@
-import crabpy
 import pycountry
 from crabpy.gateway import adressenregister
 from pyramid.renderers import JSON
@@ -24,7 +23,7 @@ def list_provincie_adapter(obj, request):
         "niscode": obj.niscode,
         "naam": obj.naam,
         "gewest": {
-            "id": obj.gewest,
+            "id": obj.gewest_niscode,
         },
     }
 
@@ -48,7 +47,7 @@ def list_gemeente_adapter(obj, request):
     """
     return {
         "niscode": obj.niscode,
-        "naam": obj.naam,
+        "naam": obj.naam(),
         "provincie": {"niscode": obj.provincie_niscode}
     }
 
@@ -138,7 +137,7 @@ def item_provincie_adapter(obj, request):
         "niscode": obj.niscode,
         "naam": obj.naam,
         "gewest": {
-            "id": obj.gewest,
+            "niscode": obj.gewest_niscode,
         },
     }
 

diff --git a/crabpy_pyramid/routes/adressenregister.py b/crabpy_pyramid/routes/adressenregister.py
@@ -7,24 +7,24 @@ def includeme(config):
     )
     crabpy_pyramid.add_route(
         config,
-        "adressenregister_get_gewest_by_id",
-        "/adressenregister/gewesten/{gewest_id}",
+        "adressenregister_get_gewest_by_niscode",
+        "/adressenregister/gewesten/{gewest_niscode}",
     )
 
     crabpy_pyramid.add_route(
         config,
         "adressenregister_list_provincies",
-        "/adressenregister/gewesten/{gewest_id}/provincies",
+        "/adressenregister/gewesten/{gewest_niscode}/provincies",
     )
     crabpy_pyramid.add_route(
         config,
         "adressenregister_get_provincie",
-        "/adressenregister/provincies/{provincie_id}",
+        "/adressenregister/provincies/{provincie_niscode}",
     )
     crabpy_pyramid.add_route(
         config,
         "adressenregister_list_deelgemeenten",
-        "/adressenregister/gewesten/{gewest_id}/deelgemeenten",
+        "/adressenregister/gewesten/{gewest_niscode}/deelgemeenten",
     )
     crabpy_pyramid.add_route(
         config,
@@ -34,18 +34,18 @@ def includeme(config):
     crabpy_pyramid.add_route(
         config,
         "adressenregister_get_deelgemeente_by_id",
-        "/adressenregister/deelgemeenten/{deelgemeente_id}",
+        "/adressenregister/deelgemeenten/{deelgemeente_niscode}",
     )
 
     crabpy_pyramid.add_route(
         config,
         "adressenregister_list_gemeenten_by_provincie",
-        "/adressenregister/provincies/{provincie_id}/gemeenten",
+        "/adressenregister/provincies/{provincie_niscode}/gemeenten",
     )
     crabpy_pyramid.add_route(
         config,
         "list_gemeenten_adressenregister",
-        "/adressenregister/gewesten/{gewest_id}/gemeenten",
+        "/adressenregister/gewesten/{gewest_niscode}/gemeenten",
     )
 
     crabpy_pyramid.add_route(