add changes to the views

breathecode/admissions/views.py

@@ -85,11 +85,11 @@ class AcademyTeacherView(APIView, GenerateLookupsMixin):
 
     @capable_of('read_member')
     def get(self, request, academy_id):
-
         handler = self.extensions(request)
+
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = ProfileAcademy.objects.filter(academy__id=academy_id,
                                               role__slug__in=['teacher', 'assistant'
@@ -296,7 +296,7 @@ def get(self, request, format=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         try:
             if isinstance(request.user, AnonymousUser):
@@ -363,7 +363,7 @@ def get(self, request):
 
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return Response(cache, status=status.HTTP_200_OK)
+        #     return cache
 
         items = CohortUser.objects.all()
 
@@ -567,7 +567,7 @@ def get(self, request, format=None, cohort_id=None, user_id=None, academy_id=Non
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         if user_id is not None:
             item = CohortUser.objects.filter(cohort__academy__id=academy_id,
@@ -1089,7 +1089,7 @@ def get(self, request, cohort_id=None, academy_id=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         if cohort_id is not None:
             if cohort_id.isnumeric():
@@ -1144,7 +1144,7 @@ def get(self, request, cohort_id=None, academy_id=None):
 
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return Response(cache, status=status.HTTP_200_OK)
+        #     return cache
 
         if cohort_id is not None:
             item = None
@@ -1589,7 +1589,7 @@ def get(self, request, syllabus_id=None, syllabus_slug=None, version=None, acade
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         if academy_id is None:
             raise ValidationException('Missing academy id', slug='missing-academy-id')
@@ -1752,7 +1752,7 @@ def get(self, request):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = CohortUser.objects.all()
 

breathecode/assignments/views.py

@@ -328,9 +328,10 @@ class CohortTaskView(APIView, GenerateLookupsMixin):
     @capable_of('read_assignment')
     def get(self, request, cohort_id, academy_id):
         handler = self.extensions(request)
+
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return Response(cache, status=status.HTTP_200_OK)
+        #     return cache
 
         items = Task.objects.all()
         lookup = {}
@@ -523,9 +524,10 @@ class TaskMeView(APIView):
 
     def get(self, request, task_id=None, user_id=None):
         handler = self.extensions(request)
+
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return Response(cache, status=status.HTTP_200_OK)
+        #     return cache
 
         if not user_id:
             user_id = request.user.id

breathecode/events/views.py

@@ -180,7 +180,7 @@ def get(self, request, event_id=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = get_my_event_types(request.user)
         lang = get_user_language(request)
@@ -232,7 +232,7 @@ def get(self, request):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse('{}', content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         lang = get_user_language(request)
 
@@ -416,7 +416,7 @@ def get(self, request, academy_id=None, event_id=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         if event_id is not None:
             single_event = Event.objects.filter(id=event_id, academy__id=academy_id).first()

breathecode/feedback/views.py

@@ -78,7 +78,7 @@ def get(self, request, format=None, academy_id=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = Answer.objects.filter(academy__id=academy_id)
         lookup = {}

breathecode/marketing/views.py

@@ -1157,7 +1157,7 @@ def get(self, request, course_slug=None):
 
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         lang = get_user_language(request)
 

breathecode/mentorship/views.py

@@ -717,10 +717,10 @@ class MentorView(APIView, HeaderLimitOffsetPagination):
     @capable_of('read_mentorship_mentor')
     def get(self, request, mentor_id=None, academy_id=None):
         handler = self.extensions(request)
-        cache = handler.cache.get()
 
+        cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         if mentor_id is not None:
             mentor = MentorProfile.objects.filter(id=mentor_id, services__academy__id=academy_id).first()

breathecode/payments/views.py

@@ -1053,7 +1053,7 @@ def get(self, request):
 
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return Response(cache, status=status.HTTP_200_OK)
+        #     return cache
 
         lang = get_user_language(request)
         utc_now = timezone.now()

breathecode/registry/views.py

@@ -148,7 +148,7 @@ def get(self, request, academy_id=None):
         handler = self.extensions(request)
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = AssetTechnology.objects.all()
         lookup = {}
@@ -483,10 +483,10 @@ class AssetView(APIView, GenerateLookupsMixin):
 
     def get(self, request, asset_slug=None):
         handler = self.extensions(request)
-        cache = handler.cache.get()
 
+        cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         lang = get_user_language(request)
 
@@ -760,16 +760,16 @@ class AcademyAssetView(APIView, GenerateLookupsMixin):
 
     @capable_of('read_asset')
     def get(self, request, asset_slug=None, academy_id=None):
+        handler = self.extensions(request)
+
+        cache = handler.cache.get()
+        if cache is not None:
+            return cache
 
         member = ProfileAcademy.objects.filter(user=request.user, academy__id=academy_id).first()
         if member is None:
             raise ValidationException("You don't belong to this academy", status.HTTP_400_BAD_REQUEST)
 
-        handler = self.extensions(request)
-        cache = handler.cache.get()
-        if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
-
         if asset_slug is not None:
             asset = Asset.get_by_slug(asset_slug, request)
             if asset is None or (asset.academy is not None and asset.academy.id != int(academy_id)):
@@ -1022,7 +1022,7 @@ def get(self, request, academy_id=None):
         handler = self.extensions(request)
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = AssetComment.objects.filter(asset__academy__id=academy_id)
         lookup = {}
@@ -1127,7 +1127,7 @@ def get(self, request, category_slug=None, academy_id=None):
         handler = self.extensions(request)
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = AssetCategory.objects.filter(academy__id=academy_id)
         lookup = {}
@@ -1211,7 +1211,7 @@ def get(self, request, keyword_slug=None, academy_id=None):
         handler = self.extensions(request)
         cache = handler.cache.get()
         if cache is not None:
-            return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+            return cache
 
         items = AssetKeyword.objects.filter(academy__id=academy_id)
         lookup = {}
@@ -1306,7 +1306,7 @@ def get(self, request, cluster_slug=None, academy_id=None):
         # cache has been disabled because I cant get it to refresh then keywords are resigned to assets
         # cache = handler.cache.get()
         # if cache is not None:
-        #     return HttpResponse(cache, content_type='application/json', status=status.HTTP_200_OK)
+        #     return cache
 
         items = KeywordCluster.objects.filter(academy__id=academy_id)
         lookup = {}