fix: do not pass the "None" string to forum when course_id is not specified

Author: Agrendalath

openedx/core/djangoapps/django_comment_common/comment_client/models.py

@@ -177,12 +177,13 @@ def save(self, params=None):
 
     def delete(self, course_id=None):
         course_key = get_course_key(self.attributes.get("course_id") or course_id)
+        course_id = str(course_key) if course_key else None
         if is_forum_v2_enabled(course_key):
             response = None
             if self.type == "comment":
-                response = forum_api.delete_comment(comment_id=self.attributes["id"], course_id=str(course_key))
+                response = forum_api.delete_comment(comment_id=self.attributes["id"], course_id=course_id)
             elif self.type == "thread":
-                response = forum_api.delete_thread(thread_id=self.attributes["id"], course_id=str(course_key))
+                response = forum_api.delete_thread(thread_id=self.attributes["id"], course_id=course_id)
             if response is None:
                 raise CommentClientRequestError("Forum v2 API call is missing")
         else:
@@ -224,15 +225,16 @@ def handle_update(self, params=None):
         if params:
             request_params.update(params)
         course_id = self.attributes.get("course_id") or request_params.get("course_id")
+        course_id = str(course_id) if course_id else None
         course_key = get_course_key(course_id)
         if is_forum_v2_enabled(course_key):
             response = None
             if self.type == "comment":
-                response = self.handle_update_comment(request_params, str(course_key))
+                response = self.handle_update_comment(request_params, course_id)
             elif self.type == "thread":
-                response = self.handle_update_thread(request_params, str(course_key))
+                response = self.handle_update_thread(request_params, course_id)
             elif self.type == "user":
-                response = self.handle_update_user(request_params, str(course_key))
+                response = self.handle_update_user(request_params, course_id)
             if response is None:
                 raise CommentClientRequestError("Forum v2 API call is missing")
         else:
@@ -319,13 +321,14 @@ def perform_http_post_request(self):
 
     def handle_create(self, params=None):
         course_id = self.attributes.get("course_id") or params.get("course_id")
+        course_id = str(course_id) if course_id else None
         course_key = get_course_key(course_id)
         if is_forum_v2_enabled(course_key):
             response = None
             if self.type == "comment":
-                response = self.handle_create_comment(str(course_key))
+                response = self.handle_create_comment(course_id)
             elif self.type == "thread":
-                response = self.handle_create_thread(str(course_key))
+                response = self.handle_create_thread(course_id)
             if response is None:
                 raise CommentClientRequestError("Forum v2 API call is missing")
         else: