Skip to content
Snippets Groups Projects

Resolve "Rethink permission handling for subject/courses"

Merged Hangzhi Yu requested to merge 18-rethink-permission-handling-for-subject-courses into master
All threads resolved!
1 file
+ 6
1
Compare changes
  • Side-by-side
  • Inline
@@ -56,7 +56,12 @@ class SubjectType(PermissionsTypeMixin, DjangoFilterMixin, DjangoObjectType):
@classmethod
def get_queryset(cls, queryset, info):
return get_objects_for_user(info.context.user, "cursus.view_subject", Subject)
# FIXME: Permissions... this is just a workaround,
# because cursus.view_subject would have to be assigned manually
if not has_person(info.context.user):
raise PermissionDenied()
return queryset
# return get_objects_for_user(info.context.user, "cursus.view_subject", Subject)
@staticmethod
def resolve_courses(root, info, **kwargs):
Loading