[AIDAPP-309]: After changing an article from public to internal, the article is still available via the link on the portal

app-modules/portal/src/DataTransferObjects/KnowledgeBaseArticleData.php

@@ -46,6 +46,6 @@ public function __construct(
         public string $name,
         public ?string $lastUpdated,
         public ?string $content,
-        public ?array $tags,
+        public ?array $tags
     ) {}
 }

app-modules/portal/src/Http/Controllers/KnowledgeManagementPortal/KnowledgeManagementPortalArticleController.php

@@ -49,28 +49,32 @@ public function show(KnowledgeBaseCategory $category, KnowledgeBaseItem $article
     {
         $article->increment('portal_view_count');
 
-        return response()->json([
-            'category' => KnowledgeBaseCategoryData::from([
-                'id' => $category->getKey(),
-                'name' => $category->name,
-                'description' => $category->description,
-            ]),
-            'article' => KnowledgeBaseArticleData::from([
-                'id' => $article->getKey(),
-                'categoryId' => $article->category_id,
-                'name' => $article->title,
-                'lastUpdated' => $article->updated_at->format('M d Y, h:m a'),
-                'content' => tiptap_converter()->record($article, attribute: 'article_details')->asHTML($article->article_details),
-                'tags' => $article->tags()
-                    ->orderBy('name')
-                    ->select([
-                        'id',
-                        'name',
-                    ])
-                    ->get()
-                    ->toArray(),
-            ]),
-            'portal_view_count' => $article->portal_view_count,
-        ]);
+        if ($article->public === true) {
+            return response()->json([
+                'category' => KnowledgeBaseCategoryData::from([
+                    'id' => $category->getKey(),
+                    'name' => $category->name,
+                    'description' => $category->description,
+                ]),
+                'article' => KnowledgeBaseArticleData::from([
+                    'id' => $article->getKey(),
+                    'categoryId' => $article->category_id,
+                    'name' => $article->title,
+                    'lastUpdated' => $article->updated_at->format('M d Y, h:m a'),
+                    'content' => tiptap_converter()->record($article, attribute: 'article_details')->asHTML($article->article_details),
+                    'tags' => $article->tags()
+                        ->orderBy('name')
+                        ->select([
+                            'id',
+                            'name',
+                        ])
+                        ->get()
+                        ->toArray(),
+                ]),
+                'portal_view_count' => $article->portal_view_count,
+            ]);
+        }
+
+        return response()->json([]);
     }
 }

portals/knowledge-management/src/Pages/ViewArticle.vue

@@ -81,10 +81,13 @@
 
         get(props.apiUrl + '/categories/' + route.params.categoryId + '/articles/' + route.params.articleId).then(
             (response) => {
-                category.value = response.data.category;
-                article.value = response.data.article;
+                if (response.data) {
+                    category.value = response.data.category;
+                    article.value = response.data.article;
+                    portalViewCount.value = response.data.portal_view_count;
+                }
+
                 loading.value = false;
-                portalViewCount.value = response.data.portal_view_count;
             },
         );
     }
@@ -104,11 +107,12 @@
                                 { name: category.name, route: 'view-category', params: { categoryId: category.id } },
                             ]"
                             currentCrumb="Articles"
+                            v-if="article"
                         ></Breadcrumbs>
 
                         <div class="grid space-y-2">
                             <div class="flex flex-col gap-3">
-                                <div class="prose max-w-none">
+                                <div class="prose max-w-none" v-if="article">
                                     <h1 class="font-semibold">{{ article.name }}</h1>
                                     <div class="flex mb-4">
                                         <div class="text-gray-500 flex items-center space-x-1 mr-2">
@@ -124,6 +128,19 @@
                                     <hr class="my-4" />
                                     <div v-html="DOMPurify.sanitize(article.content)"></div>
                                 </div>
+                                <div class="flex items-center justify-center min-h-screen bg-gray-100" v-else>
+                                    <div class="text-center">
+                                        <p class="text-lg font-semibold text-gray-800">
+                                            The link you are attempting to access is invalid.
+                                        </p>
+                                        <router-link
+                                            :to="{ name: 'home' }"
+                                            class="mt-4 inline-block px-4 py-2 text-white bg-gradient-to-br from-primary-500 to-primary-800 m-3 p-2 rounded"
+                                        >
+                                            Return Home
+                                        </router-link>
+                                    </div>
+                                </div>
                             </div>
                         </div>
                     </main>