nodebb-plugin-facebook-post 1.0.21 → 1.0.23

package/library.js

@@ -339,44 +339,89 @@ Plugin.addAdminNavigation = async function (header) {
 };
 
 Plugin.onPostCreate = async function (hookData) {
+  const winston = require.main.require('winston');
   try {
-    if (!hookData?.post || !hookData?.data) return hookData;
+    if (!hookData?.post || !hookData?.data) {
+      winston.info('[facebook-post] onPostCreate: hookData manquant (post ou data absent)');
+      return hookData;
+    }
     hookData.post.fbPostEnabled = bool(hookData.data.fbPostEnabled);
     const fbPlaceId = trimStr(hookData.data.fbPlaceId);
     if (fbPlaceId) hookData.post.fbPlaceId = fbPlaceId;
-  } catch {
-    // swallow
+    winston.info(`[facebook-post] onPostCreate: pid=${hookData.post.pid} fbPostEnabled=${hookData.post.fbPostEnabled} fbPlaceId=${hookData.post.fbPlaceId || '(none)'}`);
+  } catch (e) {
+    winston.error(`[facebook-post] onPostCreate erreur: ${e?.message || e}`);
   }
   return hookData;
 };
 
 Plugin.onPostSave = async function (hookData) {
   const winston = require.main.require('winston');
+  winston.info('[facebook-post] onPostSave: hook déclenché');
   try {
     await loadSettings();
-    if (!settings.enabled) return;
+    winston.info(`[facebook-post] onPostSave: settings.enabled=${settings.enabled} fbPageId=${settings.fbPageId ? 'défini' : 'MANQUANT'} fbPageAccessToken=${settings.fbPageAccessToken ? 'défini' : 'MANQUANT'}`);
 
-    const ctx = await getPostContext(hookData && hookData.post ? hookData.post : hookData);
-    if (!ctx) return;
+    if (!settings.enabled) {
+      winston.info('[facebook-post] onPostSave: abandon — plugin désactivé dans les paramètres');
+      return;
+    }
 
-    const allowed = await userIsAllowed(ctx.post.uid);
-    if (!allowed) return;
+    const rawPost = hookData && hookData.post ? hookData.post : hookData;
+    winston.info(`[facebook-post] onPostSave: données brutes reçues — pid=${rawPost?.pid} fbPostEnabled=${rawPost?.fbPostEnabled}`);
+
+    const ctx = await getPostContext(rawPost);
+    if (!ctx) {
+      winston.warn('[facebook-post] onPostSave: abandon — impossible de récupérer le contexte du post (pid invalide ?)');
+      return;
+    }
+    winston.info(`[facebook-post] onPostSave: contexte récupéré — pid=${ctx.post.pid} tid=${ctx.topic.tid} cid=${ctx.topic.cid} uid=${ctx.user.uid} isMainPost=${ctx.post.isMainPost} index=${ctx.post.index} reputation=${ctx.user.reputation}`);
 
-    if (!shouldProcessPost(ctx)) return;
+    const allowed = await userIsAllowed(ctx.post.uid);
+    if (!allowed) {
+      const groups = parseAllowedGroupsList();
+      winston.info(`[facebook-post] onPostSave: abandon — uid=${ctx.post.uid} n'est pas dans les groupes autorisés: [${groups.join(', ')}]`);
+      return;
+    }
+    winston.info(`[facebook-post] onPostSave: uid=${ctx.post.uid} autorisé par les groupes`);
+
+    const process = shouldProcessPost(ctx);
+    if (!process) {
+      const isFirstPost = (ctx.post.isMainPost === true) || (ctx.post.index === 0);
+      const fbEnabled = bool(ctx.post.fbPostEnabled);
+      const repOk = (ctx.user.reputation || 0) >= settings.minimumReputation;
+      const whitelist = parseCsvInts(settings.categoriesWhitelist);
+      const catOk = whitelist.length === 0 || whitelist.includes(parseInt(ctx.topic.cid, 10));
+      winston.info(`[facebook-post] onPostSave: abandon — shouldProcessPost=false. Détail: isFirstPost=${isFirstPost} fbPostEnabled=${fbEnabled} reputationOk=${repOk}(${ctx.user.reputation}>=${settings.minimumReputation}) categoryOk=${catOk}(cid=${ctx.topic.cid} whitelist=[${whitelist}])`);
+      return;
+    }
+    winston.info('[facebook-post] onPostSave: shouldProcessPost=true, publication en cours…');
 
     const Posts = require.main.require('./src/posts');
 
     const already = await Posts.getPostField(ctx.post.pid, 'fbPostedId');
-    if (already) return;
+    if (already) {
+      winston.info(`[facebook-post] onPostSave: abandon — déjà publié (fbPostedId=${already})`);
+      return;
+    }
+
+    const imageUrls = extractImageUrlsFromContent(ctx.post.content || '').filter(isForumHosted);
+    winston.info(`[facebook-post] onPostSave: images hébergées sur le forum trouvées: ${imageUrls.length} [${imageUrls.join(', ')}]`);
 
     const fbId = await postToFacebook(ctx);
     if (fbId) {
       await Posts.setPostField(ctx.post.pid, 'fbPostedId', fbId);
       await Posts.setPostField(ctx.post.pid, 'fbPostedAt', Date.now());
+      winston.info(`[facebook-post] onPostSave: publication réussie — fbId=${fbId} pid=${ctx.post.pid}`);
+    } else {
+      winston.warn('[facebook-post] onPostSave: postToFacebook a retourné un fbId vide (pas d\'erreur levée)');
     }
   } catch (e) {
     const detail = e?.response ? JSON.stringify(e.response.data) : (e?.message || e);
-    winston.error(`[facebook-post] Failed: ${detail}`);
+    winston.error(`[facebook-post] onPostSave: ERREUR: ${detail}`);
+    if (e?.response?.config?.url) {
+      winston.error(`[facebook-post] onPostSave: URL appelée: ${e.response.config.url} — status: ${e.response.status}`);
+    }
   }
 };
 

package/package.json

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