Skip to content

Commit

Permalink
fix filterset_kwargs and model lookup
Browse files Browse the repository at this point in the history
  • Loading branch information
bryanlandia committed Jul 8, 2023
1 parent 1cc3156 commit 8adcd58
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions openedx/core/djangoapps/appsembler/openedx_api/filters.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,19 +86,14 @@ class AllowedCourseOrgFilterSet(filters.FilterSet):
def filter_allowed_org(self, queryset, name, value):
import pdb; pdb.set_trace()

try:
user_allowed_org = self.request.user.organizations.first()
except Organization.DoesNotExist:
raise # TODO: do something else

try:
lookup = self.MODEL_COURSE_ORG_LOOKUPS[self.queryset.model]
except KeyError:
raise # TODO: do something else

if model in self.OPAQUE_KEY_FIELD_LOOKUP_MODELS:
if self.queryset.model in self.OPAQUE_KEY_FIELD_LOOKUP_MODELS:
return queryset.filter(**{lookup: "{}{}+".format(COURSE_PREFIX, user_allowed_org)})
elif model in self.STRING_ORG_NAME_LOOKUP_MODELS:
elif self.queryset.model in self.STRING_ORG_NAME_LOOKUP_MODELS:
return queryset.filter(**{lookup: user_allowed_org.short_name})
else:
return queryset.filter(**{lookup: user_allowed_org})
Expand Down Expand Up @@ -140,8 +135,9 @@ def get_filterset_kwargs(self, request, queryset, view):
except Organization.DoesNotExist:
raise # TODO: do something else
q_params = copy.deepcopy(request.query_params)
q_params.update({"allowed_org": user_allowed_org})
return {
"data": q_params.update({"allowed_org": user_allowed_org}),
"data": q_params,
"queryset": queryset,
"request": request,
}

0 comments on commit 8adcd58

Please sign in to comment.