nodebb-plugin-discord-onekite 1.1.20 → 1.1.22

package/README.md

@@ -1,8 +1,7 @@
-# nodebb-plugin-discord-onekite v1.1.4.11
+# nodebb-plugin-discord-onekite v1.1.4.13
 
-Fixes ACP saving:
-- Inputs now use `data-field` attributes (required by NodeBB `settings` module), so values persist.
-
-Toast:
-- Title: "Succès"
-- Message: "Paramètres enregistrés"
+Fixes ACP not applying JS / not persisting:
+- Uses `acpScripts` + modules key `admin/plugins/discord-onekite.js` to ensure admin script loads.
+- Removes bottom-right save button partial; uses only the top "Enregistrer les paramètres" button.
+- Persists categories via hidden CSV field `data-field="cids"`.
+- Toast uses title "Succès" and message "Paramètres enregistrés".

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-discord-onekite",
-  "version": "1.1.20",
+  "version": "1.1.22",
   "description": "Discord webhook notifier for Onekite (NodeBB v4.x only)",
   "main": "library.js",
   "license": "MIT",

package/plugin.json

@@ -22,7 +22,10 @@
     }
   ],
   "templates": "templates",
+  "acpScripts": [
+    "static/lib/admin.js"
+  ],
   "modules": {
-    "../admin/plugins/discord-onekite.js": "static/lib/admin.js"
+    "admin/plugins/discord-onekite.js": "static/lib/admin.js"
   }
 }

package/static/lib/admin.js

@@ -4,18 +4,44 @@
 define('admin/plugins/discord-onekite', ['settings'], function (settings) {
   const ACP = {};
 
+  function parseCsv(v) {
+    if (!v) return [];
+    return String(v).split(',').map(s => s.trim()).filter(Boolean);
+  }
+  function toCsv(arr) {
+    return (arr || []).map(String).filter(Boolean).join(',');
+  }
+  function setSelectedCids(cids) {
+    const set = new Set((cids || []).map(String));
+    $('#cidsMulti option').each(function () {
+      $(this).prop('selected', set.has(String($(this).val())));
+    });
+  }
+  function getSelectedCids() {
+    return $('#cidsMulti option:selected').map(function () { return String($(this).val()); }).get();
+  }
+
+  function toastSuccess() {
+    if (window.app && typeof app.alert === 'function') {
+      app.alert({ type: 'success', title: 'Succès', message: 'Paramètres enregistrés' });
+    } else if (window.app && typeof app.alertSuccess === 'function') {
+      app.alertSuccess('Succès: Paramètres enregistrés');
+    }
+  }
+
   ACP.init = function () {
     const $form = $('.discord-onekite-settings');
     if (!$form.length) return;
 
-    settings.sync('discord-onekite', $form);
+    settings.sync('discord-onekite', $form, function () {
+      setSelectedCids(parseCsv($('#cids').val()));
+    });
 
-    $('#save').off('click.discordOnekite').on('click.discordOnekite', function (e) {
+    $('#onekite-save').off('click.discordOnekite').on('click.discordOnekite', function (e) {
       e.preventDefault();
+      $('#cids').val(toCsv(getSelectedCids()));
       settings.persist('discord-onekite', $form, function () {
-        if (window.app && typeof app.alertSuccess === 'function') {
-          if (typeof app.alert === 'function') { app.alert({ type: 'success', title: 'Succès', message: 'Paramètres enregistrés' }); } else { app.alertSuccess('Paramètres enregistrés'); }
-        }
+        toastSuccess();
       });
     });
   };

package/templates/admin/plugins/discord-onekite.tpl

@@ -1,6 +1,11 @@
 <div class="acp-page-container">
-  <h4>Discord Onekite</h4>
-  <p class="text-muted">Notifications Discord via webhook.</p>
+  <div class="d-flex justify-content-between align-items-start mb-3">
+    <div>
+      <h4 class="mb-1">Discord Onekite</h4>
+      <p class="text-muted mb-0">Notifications Discord via webhook.</p>
+    </div>
+    <button id="onekite-save" type="button" class="btn btn-primary">Enregistrer les paramètres</button>
+  </div>
 
   <form role="form" class="discord-onekite-settings">
     <div class="mb-3">
@@ -16,17 +21,19 @@
     </div>
 
     <div class="mb-3">
-      <label class="form-label" for="cids">Catégories à notifier</label>
-      <select class="form-select" id="cids" data-field="cids" multiple size="12">
+      <label class="form-label" for="cidsMulti">Catégories à notifier</label>
+
+      <select class="form-select" id="cidsMulti" multiple size="12">
         <!-- BEGIN categories -->
         <option value="{categories.cid}" <!-- IF categories.selected -->selected<!-- ENDIF categories.selected -->>{categories.name}</option>
         <!-- END categories -->
       </select>
+
+      <input type="hidden" id="cids" data-field="cids" value="{settings.cids}" />
+
       <p class="form-text text-muted">
         Si aucune catégorie n’est sélectionnée : <strong>toutes les catégories</strong> seront notifiées.
       </p>
     </div>
-
-    <!-- IMPORT admin/partials/save_button.tpl -->
   </form>
 </div>