feat: Add language support for courses

cms/api.py

@@ -82,7 +82,11 @@ def filter_and_sort_catalog_pages(
         page.product.current_price,
         page.title,
     )
-    default_sorting_key = lambda page: (page_run_dates[page], page.title)  # noqa: E731
+    default_sorting_key = lambda page: (  # noqa: E731
+        page.language.priority,
+        page_run_dates[page],
+        page.title,
+    )
 
     # Best Match and Start Date sorting has same logic
     sorting_key_map = defaultdict(

cms/api_test.py

@@ -85,7 +85,6 @@ def test_filter_and_sort_catalog_pages_with_default_sorting(sort_by):
     initial_program_pages = [
         run.course.program.page for run in [second_program_run, first_program_run]
     ]
-
     all_pages, program_pages, course_pages = filter_and_sort_catalog_pages(
         initial_program_pages,
         initial_course_pages,
@@ -114,13 +113,13 @@ def test_filter_and_sort_catalog_pages_with_default_sorting(sort_by):
     assert past_run.course not in (
         None if page.is_external_course_page else page.course for page in course_pages
     )
-
-    # Pages should be sorted by next run date
+    # Pages should be sorted by language and then next run date (When language priority is the same)
     assert [page.program for page in program_pages] == [
         first_program_run.course.program,
         second_program_run.course.program,
         later_external_program_page.program,
     ]
+
     expected_course_run_sort = [
         non_program_run,
         first_program_run,
@@ -135,6 +134,41 @@ def test_filter_and_sort_catalog_pages_with_default_sorting(sort_by):
         run.course for run in expected_course_run_sort
     ]
 
+    # Pages should be sorted by language then next run date (When language priority is the different)
+    first_program_run.course.program.page.language.priority = 2
+    first_program_run.course.program.page.save()
+    second_program_run.course.program.page.language.priority = 1
+    second_program_run.course.program.page.save()
+    later_external_course_page.language.priority = 3
+    later_external_program_page.save()
+
+    all_pages, program_pages, course_pages = filter_and_sort_catalog_pages(
+        initial_program_pages,
+        initial_course_pages,
+        external_course_pages,
+        external_program_pages,
+        sort_by,
+    )
+
+    assert [page.program for page in program_pages] == [
+        second_program_run.course.program,
+        first_program_run.course.program,
+        later_external_program_page.program,
+    ]
+
+    expected_course_run_sort = [
+        later_external_course_page,
+        non_program_run,
+        first_program_run,
+        second_program_run,
+        future_enrollment_end_run,
+        earlier_external_course_page,
+    ]
+    # The sort should also include external course pages as expected
+    assert [page.course for page in course_pages] == [
+        run.course for run in expected_course_run_sort
+    ]
+
 
 @pytest.mark.parametrize(
     (

cms/factories.py

@@ -56,7 +56,12 @@
     WebinarPage,
     WhoShouldEnrollPage,
 )
-from courses.factories import CourseFactory, PlatformFactory, ProgramFactory
+from courses.factories import (
+    CourseFactory,
+    CourseLanguageFactory,
+    PlatformFactory,
+    ProgramFactory,
+)
 
 factory.Faker.add_provider(internet)
 
@@ -78,6 +83,7 @@ class ProgramPageFactory(wagtail_factories.PageFactory):
     subhead = factory.fuzzy.FuzzyText(prefix="Subhead ")
     thumbnail_image = factory.SubFactory(wagtail_factories.ImageFactory)
     background_image = factory.SubFactory(wagtail_factories.ImageFactory)
+    language = factory.SubFactory(CourseLanguageFactory)
     parent = factory.SubFactory(wagtail_factories.PageFactory)
     certificate_page = factory.RelatedFactory(
         "cms.factories.CertificatePageFactory", "parent"
@@ -108,6 +114,7 @@ class CoursePageFactory(wagtail_factories.PageFactory):
     subhead = factory.fuzzy.FuzzyText(prefix="Subhead ")
     thumbnail_image = factory.SubFactory(wagtail_factories.ImageFactory)
     background_image = factory.SubFactory(wagtail_factories.ImageFactory)
+    language = factory.SubFactory(CourseLanguageFactory)
     parent = factory.SubFactory(wagtail_factories.PageFactory)
     certificate_page = factory.RelatedFactory(
         "cms.factories.CertificatePageFactory", "parent"
@@ -141,6 +148,7 @@ class ExternalCoursePageFactory(wagtail_factories.PageFactory):
     subhead = factory.fuzzy.FuzzyText(prefix="Subhead ")
     thumbnail_image = factory.SubFactory(wagtail_factories.ImageFactory)
     background_image = factory.SubFactory(wagtail_factories.ImageFactory)
+    language = factory.SubFactory(CourseLanguageFactory)
     parent = factory.SubFactory(wagtail_factories.PageFactory)
 
     class Meta:
@@ -170,6 +178,7 @@ class ExternalProgramPageFactory(wagtail_factories.PageFactory):
     subhead = factory.fuzzy.FuzzyText(prefix="Subhead ")
     thumbnail_image = factory.SubFactory(wagtail_factories.ImageFactory)
     background_image = factory.SubFactory(wagtail_factories.ImageFactory)
+    language = factory.SubFactory(CourseLanguageFactory)
     parent = factory.SubFactory(wagtail_factories.PageFactory)
 
     class Meta:

cms/migrations/0078_add_courseware_page_language.py

@@ -0,0 +1,81 @@
+# Generated by Django 4.2.17 on 2025-01-03 08:36
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+def populate_course_language(apps, schema_editor):
+    """Prepopulate the course language for the course pages"""
+
+    CoursePage = apps.get_model("cms.CoursePage")
+    ExternalCoursePage = apps.get_model("cms.ExternalCoursePage")
+    ProgramPage = apps.get_model("cms.ProgramPage")
+    ExternalProgramPage = apps.get_model("cms.ExternalProgramPage")
+
+    CourseLanguage = apps.get_model("courses.CourseLanguage")
+    # English is the default language for all the courses
+    course_language_english, _ = CourseLanguage.objects.get_or_create(
+        name="English", priority=1
+    )
+
+    CoursePage.objects.update(language=course_language_english)
+    ExternalCoursePage.objects.update(language=course_language_english)
+    ProgramPage.objects.update(language=course_language_english)
+    ExternalProgramPage.objects.update(language=course_language_english)
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("courses", "0042_add_course_language"),
+        ("cms", "0077_alter_certificatepage_ceus"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="coursepage",
+            name="language",
+            field=models.ForeignKey(
+                blank=True,
+                help_text="The course/program language for this page",
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.AddField(
+            model_name="externalcoursepage",
+            name="language",
+            field=models.ForeignKey(
+                blank=True,
+                help_text="The course/program language for this page",
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.AddField(
+            model_name="externalprogrampage",
+            name="language",
+            field=models.ForeignKey(
+                blank=True,
+                help_text="The course/program language for this page",
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.AddField(
+            model_name="programpage",
+            name="language",
+            field=models.ForeignKey(
+                blank=True,
+                help_text="The course/program language for this page",
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.RunPython(
+            populate_course_language, reverse_code=migrations.RunPython.noop
+        ),
+    ]

cms/migrations/0079_make_language_non_nullable.py

@@ -0,0 +1,53 @@
+# Generated by Django 4.2.17 on 2025-01-03 08:39
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("cms", "0078_add_courseware_page_language"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="coursepage",
+            name="language",
+            field=models.ForeignKey(
+                help_text="The course/program language for this page",
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="externalcoursepage",
+            name="language",
+            field=models.ForeignKey(
+                help_text="The course/program language for this page",
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="externalprogrampage",
+            name="language",
+            field=models.ForeignKey(
+                default="",
+                help_text="The course/program language for this page",
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+            preserve_default=False,
+        ),
+        migrations.AlterField(
+            model_name="programpage",
+            name="language",
+            field=models.ForeignKey(
+                default="",
+                help_text="The course/program language for this page",
+                on_delete=django.db.models.deletion.PROTECT,
+                to="courses.courselanguage",
+            ),
+            preserve_default=False,
+        ),
+    ]

cms/models.py

@@ -922,6 +922,14 @@ class ProductPage(MetadataPageMixin, WagtailCachedPageMixin, Page):
     class Meta:
         abstract = True
 
+    language = models.ForeignKey(
+        "courses.CourseLanguage",
+        null=False,
+        blank=False,
+        on_delete=models.PROTECT,
+        help_text="The course/program language for this page",
+    )
+
     description = RichTextField(
         blank=True, help_text="The description shown on the product page"
     )
@@ -1030,6 +1038,7 @@ class Meta:
         use_json_field=True,
     )
     content_panels = Page.content_panels + [  # noqa: RUF005
+        FieldPanel("language"),
         FieldPanel("external_marketing_url"),
         FieldPanel("marketing_hubspot_form_id"),
         FieldPanel("subhead"),

cms/models_test.py

@@ -83,6 +83,7 @@
 from cms.wagtail_hooks import create_product_and_versions_for_courseware_pages
 from courses.factories import (
     CourseFactory,
+    CourseLanguageFactory,
     CourseRunCertificateFactory,
     CourseRunFactory,
     ProgramCertificateFactory,
@@ -2173,7 +2174,7 @@ def _create_external_course_page(superuser_client, course_id, slug):
     Asserts:
         Response status code is 302 (successful redirection).
     """
-
+    language = CourseLanguageFactory.create()
     post_data = {
         "course": course_id,
         "title": "Icon Grid #6064",
@@ -2182,6 +2183,7 @@ def _create_external_course_page(superuser_client, course_id, slug):
         "content-count": 0,
         "slug": slug,
         "action-publish": "action-publish",
+        "language": language.id,
     }
     response = superuser_client.post(
         reverse(

courses/admin.py

@@ -13,6 +13,7 @@
 
 from .models import (
     Course,
+    CourseLanguage,
     CourseRun,
     CourseRunCertificate,
     CourseRunEnrollment,
@@ -422,3 +423,12 @@ class PlatformAdmin(TimestampedModelAdmin):
     model = Platform
     list_display = ["id", "name", "created_on", "updated_on"]
     search_fields = ["name"]
+
+
+@admin.register(CourseLanguage)
+class CourseLanguageAdmin(admin.ModelAdmin):
+    """Admin for CourseLanguage"""
+
+    model = CourseLanguage
+    list_display = ["id", "name", "priority"]
+    search_fields = ["name"]

courses/factories.py

@@ -14,6 +14,7 @@
 
 from .models import (
     Course,
+    CourseLanguage,
     CourseRun,
     CourseRunCertificate,
     CourseRunEnrollment,
@@ -38,6 +39,15 @@ class Meta:
         model = Company
 
 
+class CourseLanguageFactory(DjangoModelFactory):
+    """Factory for Course Language"""
+
+    name = factory.Sequence(lambda n: f"Language_{n}")
+
+    class Meta:
+        model = CourseLanguage
+
+
 class PlatformFactory(DjangoModelFactory):
     """Factory for Platform"""