[#4827] Don't rely at all on resource object in context

ckan · Jul 1, 2019 · 13c4e26 · 13c4e26
1 parent 5ca0f20
commit 13c4e26
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/ckan/logic/auth/get.py b/ckan/logic/auth/get.py
@@ -161,12 +161,11 @@ def resource_show(context, data_dict):
 def resource_view_show(context, data_dict):
 
     model = context['model']
-    resource = context.get('resource')
-    if not resource:
-        resource_view = model.ResourceView.get(data_dict['id'])
-        if not resource_view:
-            raise logic.NotFound(_('Resource view not found, cannot check auth.'))
-        resource = model.Resource.get(resource_view.resource_id)
+
+    resource_view = model.ResourceView.get(data_dict['id'])
+    if not resource_view:
+        raise logic.NotFound(_('Resource view not found, cannot check auth.'))
+    resource = model.Resource.get(resource_view.resource_id)
 
     return authz.is_authorized('resource_show', context, {'id': resource.id})