misuse of the audit decorator add return

OnroerendErfgoed · Jul 29, 2015 · c68c29f · c68c29f
1 parent 046856c
commit c68c29f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/atramhasis/audit.py b/atramhasis/audit.py
@@ -64,6 +64,7 @@ def advice(parent_object, *args, **kw):
             )
         elif 'scheme_id' not in request.matchdict.keys():
             log.error('Misuse of the audit decorator. The url must at least contain a {scheme_id} parameter')
+            return fn(parent_object, *args, **kw)
         else:
             visit_log = ConceptschemeVisitLog(conceptscheme_id=request.matchdict['scheme_id'])
         response = fn(parent_object, *args, **kw)

diff --git a/atramhasis/views/views.py b/atramhasis/views/views.py
@@ -306,6 +306,7 @@ def __init__(self, request):
         self.localizer = request.localizer
         self._ = TranslationStringFactory('atramhasis')
 
+    @audit
     @view_config(route_name='labeltypes')
     def labeltype_list_view(self):
         return self.get_list(LabelType)