Removed test try/catch and comment

sakaiproject · Jan 22, 2025 · 21a6c22 · 21a6c22
1 parent 06de7cf
commit 21a6c22
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/webapi/src/main/java/org/sakaiproject/webapi/controllers/ConversationsController.java b/webapi/src/main/java/org/sakaiproject/webapi/controllers/ConversationsController.java
@@ -280,15 +280,10 @@ private EntityModel entityModelForTopicBean(TopicTransferBean topicBean) {
 	@PostMapping(value = "/sites/{siteId}/topics/{topicId}/posts", produces = MediaType.APPLICATION_JSON_VALUE)
     public EntityModel<PostTransferBean> createPost(@PathVariable String siteId, @PathVariable String topicId, @RequestBody PostTransferBean postBean) throws ConversationsPermissionsException {
 
-        try {
-            checkSakaiSession();
-            postBean.siteId = siteId;
-            postBean.topic = topicId;
-            return entityModelForPostBean(conversationsService.savePost(postBean, true));
-        } catch (Exception e) {
-            e.printStackTrace();
-        }
-        return null;
+        checkSakaiSession();
+        postBean.siteId = siteId;
+        postBean.topic = topicId;
+        return entityModelForPostBean(conversationsService.savePost(postBean, true));
     }
 
 	@GetMapping(value = "/sites/{siteId}/topics/{topicId}/posts", produces = MediaType.APPLICATION_JSON_VALUE)

diff --git a/webcomponents/tool/src/main/frontend/packages/sakai-conversations/src/SakaiTopic.js b/webcomponents/tool/src/main/frontend/packages/sakai-conversations/src/SakaiTopic.js
@@ -116,7 +116,7 @@ export class SakaiTopic extends reactionsAndUpvotingMixin(SakaiElement) {
         this.topic.posts = posts;
 
         // We've clicked on a topic and it has no posts. Ergo, it has been "viewed".
-        //if (!this.topic?.posts?.length) this.topic.viewed = true;
+        if (!this.topic?.posts?.length) this.topic.viewed = true;
 
         update();
         this.dispatchEvent(new CustomEvent("topic-updated", { detail: { topic: this.topic, dontUpdateCurrent: true }, bubbles: true }));