nodebb-plugin-facebook-post 1.0.6 → 1.0.8

package/library.js

@@ -138,7 +138,7 @@ function sanitizeExcerpt(text, maxLen) {
 function parseAllowedGroupsList() {
   const s = trimStr(settings.allowedGroups);
   if (!s) return [];
-  return s.split(',').map(x => x.trim()).filter(Boolean);
+  return s.split(',').map(v => v.trim()).filter(Boolean);
 }
 
 async function userIsAllowed(uid) {
@@ -323,7 +323,9 @@ Plugin.init = async function (params) {
   }
 
   async function render(req, res) {
-    const allGroups = await getAllGroups();
+    await loadSettings();
+    const selected = new Set(parseAllowedGroupsList());
+    const allGroups = (await getAllGroups()).map(g => Object.assign({}, g, { selected: selected.has(g.name) }));
     res.render('admin/facebook-post', { allGroups });
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-facebook-post",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "Auto-post new NodeBB topics to a fixed Facebook Page (text + NodeBB uploads + place id).",
   "main": "library.js",
   "dependencies": {

package/static/lib/admin.js

@@ -2,6 +2,10 @@
 'use strict';
 
 (function () {
+  function toCsv(arr) {
+    return (arr || []).map(String).map(s => s.trim()).filter(Boolean).join(',');
+  }
+
   function init() {
     const $form = $('.facebook-post-settings');
     if (!$form.length) return;
@@ -9,14 +13,28 @@
     require(['settings', 'alerts'], function (Settings, alerts) {
       Settings.load('facebook-post', $form);
 
+      const $select = $('#allowedGroupsSelect');
+      const $hidden = $('#allowedGroups');
+
+      function syncHidden() {
+        $hidden.val(toCsv($select.val() || []));
+      }
+
+      // initial sync (after Settings.load fills hidden input)
+      const existing = ($hidden.val() || '').split(',').map(s => s.trim()).filter(Boolean);
+      if (existing.length) {
+        $select.val(existing);
+      }
+      syncHidden();
+
+      $select.off('change.fbpost').on('change.fbpost', syncHidden);
+      $form.off('submit.fbpost').on('submit.fbpost', function (e) { e.preventDefault(); return false; });
+
       $('#save').off('click.fbpost').on('click.fbpost', function (e) {
         e.preventDefault();
+        syncHidden();
         Settings.save('facebook-post', $form, function () {
-          if (alerts && typeof alerts.success === 'function') {
-            alerts.success('[[admin/settings:saved]]');
-          } else if (window.app && typeof window.app.alertSuccess === 'function') {
-            window.app.alertSuccess('[[admin/settings:saved]]');
-          }
+          alerts.success('[[admin/settings:saved]]');
         });
       });
     });

package/static/templates/admin/facebook-post.tpl

@@ -1,21 +1,5 @@
 <div class="acp-page-container">
   <h2>Facebook/Instagram Post (Worker)</h2>
-  <p class="text-muted">
-    Enfile les publications dans une <strong>queue Redis</strong>. Un worker séparé consomme la queue et poste sur Facebook,
-    et sur Instagram (si activé et si une image est présente).
-  </p>
-
-  <div class="alert alert-info">
-    Variables d’environnement :
-    <ul class="mb-0">
-      <li><code>NODEBB_FB_PAGE_ID</code></li>
-      <li><code>NODEBB_FB_PAGE_ACCESS_TOKEN</code></li>
-      <li><code>NODEBB_FB_GRAPH_VERSION</code> (optionnel)</li>
-      <li><code>NODEBB_FB_QUEUE_REDIS_URL</code></li>
-      <li><code>NODEBB_FB_WORKER_SECRET</code></li>
-      <li><code>NODEBB_IG_USER_ID</code> (si Instagram activé)</li>
-    </ul>
-  </div>
 
   <form class="facebook-post-settings" role="form">
     <div class="form-check mb-3">
@@ -31,15 +15,16 @@
     <hr/>
 
     <h4>Accès</h4>
-    <p class="text-muted">Groupes NodeBB autorisés à voir la case “Publier sur Facebook” dans le composer (et donc déclencher Facebook + Insta).</p>
+    <p class="text-muted">Sélectionne les groupes autorisés à voir la case “Publier sur Facebook” dans le composer.</p>
 
     <div class="mb-3">
-      <label class="form-label" for="allowedGroups">Groupes autorisés</label>
-      <select id="allowedGroups" name="allowedGroups" class="form-select" multiple size="12">
+      <label class="form-label" for="allowedGroupsSelect">Groupes autorisés</label>
+      <select id="allowedGroupsSelect" class="form-select" multiple size="12">
         <!-- BEGIN allGroups -->
-        <option value="{allGroups.name}">{allGroups.name}</option>
+        <option value="{allGroups.name}" <!-- IF allGroups.selected -->selected<!-- ENDIF -->>{allGroups.name}</option>
         <!-- END allGroups -->
       </select>
+      <input type="hidden" id="allowedGroups" name="allowedGroups" value="">
       <p class="form-text">Sans fallback : si aucun groupe n’est sélectionné, personne ne verra la case.</p>
     </div>
 
@@ -90,6 +75,6 @@
       <label class="form-check-label" for="enablePlaceTagging">Activer le tag de lieu (Place ID / location_id)</label>
     </div>
 
-    <button id="save" class="btn btn-primary">Enregistrer</button>
+    <button type="button" id="save" class="btn btn-primary">Enregistrer</button>
   </form>
 </div>