feat: make course optimizer scan only published version

openedx · Dec 4, 2024 · d9a3754 · d9a3754
1 parent 927b8c0
commit d9a3754
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/cms/djangoapps/contentstore/rest_api/v0/views/course_optimizer.py b/cms/djangoapps/contentstore/rest_api/v0/views/course_optimizer.py
@@ -64,7 +64,7 @@ def post(self, request: Request, course_id: str):
         if not has_studio_read_access(request.user, course_key):
             self.permission_denied(request)
 
-        check_broken_links.delay(request.user.id, course_key_string, request.LANGUAGE_CODE)
+        check_broken_links.delay(request.user.id, course_id, request.LANGUAGE_CODE)
         return JsonResponse({'LinkCheckStatus': UserTaskStatus.PENDING})
 
 
@@ -180,8 +180,10 @@ def get(self, request: Request, course_id: str):
                     except ValueError:
                         # Wasn't JSON, just use the value as a string
                         pass
+
         # print('DTO')
         # print(broken_links_dto)
+
         # mock dto for testing
         # broken_links_dto = {
         #     'sections': [

diff --git a/cms/djangoapps/contentstore/tasks.py b/cms/djangoapps/contentstore/tasks.py
@@ -1166,7 +1166,8 @@ def scan_course(course_key):
           [<block_id>, <broken_link>]
         """
         broken_links = []
-        verticals = modulestore().get_items(course_key, qualifiers={'category': 'vertical'})
+        verticals = modulestore().get_items(course_key, qualifiers={'category': 'vertical'}, revision=ModuleStoreEnum.RevisionOption.published_only)
+        print('verticals: ', verticals)
         blocks = []
 
         for vertical in verticals:
@@ -1187,7 +1188,7 @@ def scan_course(course_key):
                     broken_links.append([str(usage_key), url])
 
         return broken_links
-    
+
     user = validate_user()
 
     self.status.set_state('Scanning')