From 10ce36ea14f0392cf5f32e1cb19c3f2eeccc8d87 Mon Sep 17 00:00:00 2001
From: Michael Terry <mterry@edx.org>
Date: Thu, 17 Feb 2022 12:18:36 -0500
Subject: [PATCH] feat: hide Old Mongo courses from catalog

This is step one of removing access altogether. But first, let's
not advertise them.

DEPR-58
DEPR-123
---
 lms/djangoapps/course_api/serializers.py            | 2 +-
 lms/djangoapps/course_api/tests/test_serializers.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lms/djangoapps/course_api/serializers.py b/lms/djangoapps/course_api/serializers.py
index adcf578e74..d9188a51c4 100644
--- a/lms/djangoapps/course_api/serializers.py
+++ b/lms/djangoapps/course_api/serializers.py
@@ -123,7 +123,7 @@ class CourseSerializer(serializers.Serializer):  # pylint: disable=abstract-meth
         Represents whether course is hidden in LMS
         """
         catalog_visibility = course_overview.catalog_visibility
-        return catalog_visibility in ['about', 'none']
+        return catalog_visibility in ['about', 'none'] or course_overview.id.deprecated  # Old Mongo should be hidden
 
     def get_blocks_url(self, course_overview):
         """
diff --git a/lms/djangoapps/course_api/tests/test_serializers.py b/lms/djangoapps/course_api/tests/test_serializers.py
index b80cc63799..7bba11d540 100644
--- a/lms/djangoapps/course_api/tests/test_serializers.py
+++ b/lms/djangoapps/course_api/tests/test_serializers.py
@@ -78,7 +78,7 @@ class TestCourseSerializer(CourseApiFactoryMixin, ModuleStoreTestCase):
             'effort': '6 hours',
             'pacing': 'instructor',
             'mobile_available': True,
-            'hidden': False,
+            'hidden': True,  # because it's an old mongo course
             'invitation_only': False,
 
             # 'course_id' is a deprecated field, please use 'id' instead.