nodebb-plugin-facebook-post 1.0.12 → 1.0.14

package/library.js

@@ -135,6 +135,24 @@ function sanitizeExcerpt(text, maxLen) {
   return s.slice(0, Math.max(0, maxLen - 1)).trimEnd() + '…';
 }
 
+function normalizeAllowedGroupsValue(value) {
+  if (!value) return '';
+  if (Array.isArray(value)) {
+    return value.map(String).map(v => v.trim()).filter(Boolean).join(',');
+  }
+  const s = String(value).trim();
+  if (!s) return '';
+  if (s.startsWith('[')) {
+    try {
+      const parsed = JSON.parse(s);
+      if (Array.isArray(parsed)) {
+        return parsed.map(String).map(v => v.trim()).filter(Boolean).join(',');
+      }
+    } catch (_) {}
+  }
+  return s;
+}
+
 function parseAllowedGroupsList() {
   const s = trimStr(settings.allowedGroups);
   if (!s) return [];
@@ -333,6 +351,7 @@ Plugin.init = async function (params) {
   router.get('/api/admin/plugins/facebook-post', render);
 
   router.get('/api/facebook-post/can-post', middleware.ensureLoggedIn, async (req, res) => {
+    res.set('Cache-Control', 'no-store');
     try {
       await loadSettings();
       if (!settings.enabled) return res.json({ allowed: false, reason: 'disabled' });

package/package.json

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

package/static/lib/composer.js

@@ -4,7 +4,10 @@
 (function () {
   async function canPost() {
     try {
-      const res = await fetch('/api/facebook-post/can-post', { credentials: 'same-origin' });
+      const res = await fetch('/api/facebook-post/can-post?_=' + Date.now(), {
+        credentials: 'same-origin',
+        cache: 'no-store',
+      });
       if (!res.ok) return { allowed: false };
       return await res.json();
     } catch {
@@ -13,7 +16,6 @@
   }
 
   function injectUI($composer) {
-    if (!$composer || !$composer.length) return;
     if ($composer.find('[data-fbpost-wrap]').length) return;
 
     const html = `
@@ -35,12 +37,7 @@
       </div>
     `;
 
-    // More robust placement across themes
-    const $targets = $composer.find(
-      '.composer-body, .composer-content, .write, .composer .write, .composer .composer-body, .composer .composer-content'
-    );
-    const $target = $targets.first();
-
+    const $target = $composer.find('.write, .composer-body, .composer-content').first();
     if ($target.length) $target.prepend(html);
     else $composer.prepend(html);
 
@@ -51,7 +48,6 @@
       $placeWrap.toggle($enabled.is(':checked'));
     });
 
-    // Attach once per composer instance
     $(window).off('filter:composer.submit.fbpost')
       .on('filter:composer.submit.fbpost', function (ev2, submitData) {
         const enabled = $enabled.is(':checked');
@@ -66,23 +62,18 @@
       });
   }
 
-  async function onComposerLoaded(ev, data) {
+  $(window).on('action:composer.loaded', async function (ev, data) {
     try {
-      const $composer = $(data && data.composer ? data.composer : $('.composer')).first();
-      if (!$composer.length) return;
+      const $composer = data && data.composer ? $(data.composer) : $('.composer');
+      if (!$composer || !$composer.length) return;
 
       const perm = await canPost();
       if (!perm.allowed) return;
 
-      // Defer injection slightly (composer DOM may still be mounting)
-      setTimeout(() => injectUI($composer), 0);
+      injectUI($composer);
     } catch {
       // ignore
     }
-  }
-
-  $(window).on('action:composer.loaded', onComposerLoaded);
-
-  // Some themes trigger different events; listen to opened as well.
-  $(window).on('action:composer.opened', onComposerLoaded);
+  });
 })();
+