refactor: migrated open questions page to controller (#3257)

Author: thorsten

nginx.conf

@@ -129,7 +129,7 @@ server {
             rewrite admin/api/(.*)                                  /admin/api/index.php last;
 
             # Administration pages
-            rewrite admin/(attachments|backup|comments|configuration|elasticsearch|export|glossary|group|import|instance|instances|news|password|session-keep-alive|statistics|stopwords|system|tags|update|user) /admin/front.php last;
+            rewrite admin/(attachments|backup|comments|configuration|elasticsearch|export|glossary|group|import|instance|instances|news|password|questions|session-keep-alive|statistics|stopwords|system|tags|update|user) /admin/front.php last;
 
             # REST API v3.0 and v3.1
             rewrite ^api/v3\.[01]/(.*)                              /api/index.php last;

package.json

@@ -20,7 +20,7 @@
   "type": "module",
   "scripts": {
     "build": "vite build",
-    "build:watch": "vite build --watch -d",
+    "build:watch": "vite build --watch",
     "build:prod": "vite build",
     "lint": "prettier --check .",
     "lint:fix": "prettier --write .",

phpmyfaq/.htaccess

@@ -143,7 +143,7 @@ Header set Access-Control-Allow-Headers "Content-Type, Authorization"
     # Administration API
     RewriteRule ^admin/api/(.*) admin/api/index.php [L,QSA]
     # Administration pages
-    RewriteRule ^admin/(attachments|backup|comments|configuration|elasticsearch|export|glossary|group|import|instance|instances|password|news|session-keep-alive|statistics|stopwords|system|tags|update|user) admin/front.php [L,QSA]
+    RewriteRule ^admin/(attachments|backup|comments|configuration|elasticsearch|export|glossary|group|import|instance|instances|news|password|questions|session-keep-alive|statistics|stopwords|system|tags|update|user) admin/front.php [L,QSA]
     # Private APIs
     RewriteRule ^api/(autocomplete|bookmark/delete|bookmark/create|user/data/update|user/password/update|user/request-removal|user/remove-twofactor|contact|voting|register|captcha|share|comment/create|faq/create|question/create|webauthn/prepare|webauthn/register|webauthn/prepare-login|webauthn/login) api/index.php [L,QSA]
     # Setup APIs

phpmyfaq/admin/assets/src/api/index.js

@@ -3,6 +3,7 @@ export * from './faqs';
 export * from './glossary';
 export * from './group';
 export * from './news';
+export * from './question';
 export * from './statistics';
 export * from './tags';
 export * from './upgrade';

phpmyfaq/admin/assets/src/api/question.js

@@ -0,0 +1,34 @@
+/**
+ * Fetch data for open questions management
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public License,
+ * v. 2.0. If a copy of the MPL was not distributed with this file, You can
+ * obtain one at https://mozilla.org/MPL/2.0/.
+ *
+ * @package   phpMyFAQ
+ * @author    Thorsten Rinne <thorsten@phpmyfaq.de>
+ * @copyright 2024 phpMyFAQ Team
+ * @license   http://www.mozilla.org/MPL/2.0/ Mozilla Public License Version 2.0
+ * @link      https://www.phpmyfaq.de
+ * @since     2024-12-02
+ */
+
+export const toggleQuestionVisibility = async (questionId, visibility, csrfToken) => {
+  try {
+    const response = await fetch(`./api/question/visibility/toggle`, {
+      method: 'PUT',
+      headers: {
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify({
+        questionId: questionId,
+        visibility: visibility,
+        csrfToken: csrfToken,
+      }),
+    });
+
+    return await response.json();
+  } catch (error) {
+    console.error(error);
+  }
+};

phpmyfaq/admin/assets/src/content/question.js

@@ -12,7 +12,10 @@
  * @link      https://www.phpmyfaq.de
  * @since     2023-03-04
  */
+
 import { addElement, serialize } from '../../../../assets/src/utils';
+import { toggleQuestionVisibility } from '../api/index.js';
+import { pushErrorNotification } from '../utils/index.js';
 
 export const handleOpenQuestions = () => {
   const deleteButton = document.getElementById('pmf-delete-questions');
@@ -62,3 +65,29 @@ export const handleOpenQuestions = () => {
     });
   }
 };
+
+export const handleToggleVisibility = () => {
+  const toggleVisibility = document.querySelectorAll('.pmf-toggle-visibility');
+
+  if (toggleVisibility) {
+    toggleVisibility.forEach((element) => {
+      element.addEventListener('click', async (event) => {
+        event.preventDefault();
+
+        const questionId = element.getAttribute('data-pmf-question-id');
+        const visibility = element.getAttribute('data-pmf-visibility');
+        const csrfToken = element.getAttribute('data-pmf-csrf');
+
+        const response = await toggleQuestionVisibility(questionId, visibility, csrfToken);
+
+        if (response.success) {
+          element.innerText = response.success;
+        } else {
+          pushErrorNotification(response.error);
+        }
+
+        console.log(questionId, visibility, csrfToken);
+      });
+    });
+  }
+};

phpmyfaq/admin/assets/src/index.js

@@ -59,6 +59,7 @@ import {
   handleSaveFaqData,
   handleUpdateQuestion,
   handleRefreshAttachments,
+  handleToggleVisibility,
 } from './content';
 import { handleUserList, handleUsers } from './user';
 import { handleGroups } from './group';
@@ -111,6 +112,7 @@ document.addEventListener('DOMContentLoaded', async () => {
 
   // Content → Open questions
   handleOpenQuestions();
+  handleToggleVisibility();
 
   // Content → Attachments
   handleDeleteAttachments();

phpmyfaq/admin/header.php

@@ -79,7 +79,7 @@
     'stickyfaqs',
     'stickyRecordsHeader'
 );
-$secLevelEntries['content'] .= $adminHelper->addMenuEntry('delquestion', 'question', 'ad_menu_open');
+$secLevelEntries['content'] .= $adminHelper->addMenuEntry('delquestion', 'question', 'ad_menu_open', 'questions');
 $secLevelEntries['content'] .= $adminHelper->addMenuEntry('delcomment', 'comments', 'ad_menu_comments', 'comments');
 $secLevelEntries['content'] .= $adminHelper->addMenuEntry(
     'addattachment+editattachment+delattachment',

phpmyfaq/admin/index.php

@@ -267,9 +267,6 @@
             case 'copyentry':
                 require 'faqs.editor.php';
                 break;
-            case 'question':
-                require 'open-questions.php';
-                break;
             case 'stickyfaqs':
                 require 'stickyfaqs.php';
                 break;