nodebb-plugin-facebook-post 1.0.10 → 1.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-facebook-post",
-  "version": "1.0.10",
+  "version": "1.0.11",
   "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

@@ -5,14 +5,15 @@
   function toCsv(arr) {
     return (arr || []).map(String).map(s => s.trim()).filter(Boolean).join(',');
   }
+  function fromCsv(s) {
+    return (s || '').split(',').map(v => v.trim()).filter(Boolean);
+  }
 
   function init() {
     const $form = $('.facebook-post-settings');
     if (!$form.length) return;
 
     require(['settings', 'alerts'], function (Settings, alerts) {
-      Settings.load('facebook-post', $form);
-
       const $select = $('#allowedGroupsSelect');
       const $hidden = $('#allowedGroups');
 
@@ -20,15 +21,21 @@
         $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();
+      // Prevent full page refresh if user hits Enter
+      $form.off('submit.fbpost').on('submit.fbpost', function (e) { e.preventDefault(); return false; });
+
+      // IMPORTANT: wait for Settings.load to finish before reading hidden input
+      Settings.load('facebook-post', $form, function () {
+        const existing = fromCsv($hidden.val());
+        if (existing.length) {
+          $select.val(existing);
+        } else {
+          $select.val([]);
+        }
+        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();