fix: do not raise runtime errors when checking if the MySQL backend is enabled (#182)

The edx-platform incorrectly passes `str(None)` in many places when course ID
is not present. While this change silently ignores incorrect values provided
by the platform, it is better than having runtime errors in production.

Author: Agrendalath

CHANGELOG.rst

@@ -16,6 +16,16 @@ Unreleased
 
 *
 
+0.3.3 – 2025-08-12
+******************
+
+Fixed
+=====
+
+* Do not raise runtime errors if an incorrect course ID is provided when
+  checking if the MySQL backend is enabled.
+
+
 0.3.0 – 2025-04-23
 ******************
 

forum/__init__.py

@@ -2,4 +2,4 @@
 Openedx forum app.
 """
 
-__version__ = "0.3.2"
+__version__ = "0.3.3"

forum/backend.py

@@ -13,6 +13,7 @@ def is_mysql_backend_enabled(course_id: str | None) -> bool:
     try:
         # pylint: disable=import-outside-toplevel
         from forum.toggles import ENABLE_MYSQL_BACKEND
+        from opaque_keys import InvalidKeyError
         from opaque_keys.edx.keys import CourseKey
     except ImportError:
         return True
@@ -21,7 +22,10 @@ def is_mysql_backend_enabled(course_id: str | None) -> bool:
     if isinstance(course_id, CourseKey):
         course_key = course_id  # type: ignore[unreachable]
     elif isinstance(course_id, str):
-        course_key = CourseKey.from_string(course_id)
+        try:
+            course_key = CourseKey.from_string(course_id)
+        except InvalidKeyError:
+            pass
 
     return ENABLE_MYSQL_BACKEND.is_enabled(course_key)