nodebb-plugin-gemach-directory 1.0.1 → 1.0.3

package/library.js

@@ -7,13 +7,18 @@
  *   חדש (SocketPlugins.gemachDirectory.submit) - הוא נשמר כ"ממתין" ולא
  *   מוצג לאף אחד עד שמנהל מאשר אותו.
  * - כשמוצע גמ"ח חדש, נשלחת התראת NodeBB אמיתית (פעמון) לכל חברי קבוצת
- *   "administrators" שלא כיבו אצל עצמם את ההתראות (ראו getNotifyPreference/
- *   setNotifyPreference למטה - כל מנהל שולט בעצמו, לא משפיע על אחרים).
+ *   "administrators" - כדי שמנהל ידע שיש משהו ממתין לאישור.
  * - אישור/דחייה (SocketPlugins.gemachDirectory.approve/reject) פתוחים רק
  *   למנהלים (נבדק בשרת, לא רק בממשק) - אישור מעביר את הגמ"ח לרשימה
  *   הציבורית באופן מיידי, דחייה מוחקת אותו לגמרי (בלי לצבור "זבל").
  * - הרשימה הציבורית (SocketPlugins.gemachDirectory.listApproved) פתוחה
- *   לכולם, כולל גולשים לא-מחוברים.
+ *   לכולם, כולל גולשים לא-מחוברים. כל גמ"ח מוחזר עם שם המשתמש שהעלה אותו
+ *   (submittedByUsername/submittedByUserslug) - כדי שהלקוח יוכל להציג
+ *   קרדיט ("מאת @שם") ולדעת אם המשתמש המחובר הוא הבעלים.
+ * - עריכה/מחיקה (SocketPlugins.gemachDirectory.edit/remove) פתוחים למי
+ *   שהעלה את הגמ"ח (נבדק לפי socket.uid מול השדה submittedBy שנשמר בזמן
+ *   ההעלאה - אי אפשר לזייף) *או* למנהל - מנהל יכול לערוך/למחוק כל גמ"ח,
+ *   משתמש רגיל רק את שלו.
  *
  * חלק הלקוח (הצגת הרשימה, הטופס להוספה, פאנל האישור למנהל) *לא* נמצא כאן -
  * הוא קובץ נפרד (gemach-directory-client.js) שמודבק ב-Custom JS של הפורום,
@@ -36,6 +41,10 @@ const GEMACH_KEY = id => `gemachDirectory:item:${id}`;
 // מפה של uid -> '0'/'1' - האם המנהל הזה רוצה לקבל התראות על גמ"חים חדשים.
 // חסר מפתח = כברירת מחדל כן (כדי שמנהלים קיימים לא "יפספסו" בלי לשים לב).
 const NOTIFY_PREFS_OBJECT = 'gemachDirectory:notifyPrefs';
+// הנתיב שאליו התראת "גמ"ח חדש ממתין" מפנה - הפוסט האמיתי של נושא "רשימת
+// גמחים" בפורום הזה. אם הנושא אי-פעם יימחק וייווצר מחדש, צריך לעדכן כאן
+// את מספר הפוסט (רואים אותו ב-URL כשפותחים את הנושא: /post/<המספר>).
+const NOTIFY_TARGET_PATH = '/post/151093';
 
 const MAX_LENGTHS = {
 	name: 120,
@@ -142,6 +151,60 @@ function registerSocketHandlers() {
 		await db.setObjectField(NOTIFY_PREFS_OBJECT, socket.uid, enabled ? '1' : '0');
 		return { ok: true };
 	};
+
+	// עריכת גמ"ח קיים - הבעלים (submittedBy) או מנהל בלבד. עובד גם על גמ"ח
+	// ממתין וגם על גמ"ח מאושר, בלי לשנות את הסטטוס שלו.
+	SocketPlugins.gemachDirectory.edit = async function (socket, data) {
+		requireLogin(socket);
+		const id = data && data.id;
+		if (!id) throw new Error('[[error:invalid-data]]');
+
+		const gemach = await db.getObject(GEMACH_KEY(id));
+		if (!gemach) throw new Error('[[error:no-such-gemach]]');
+		await requireOwnerOrAdmin(socket, gemach);
+
+		const name = sanitizeText(data && data.name, MAX_LENGTHS.name);
+		const city = sanitizeText(data && data.city, MAX_LENGTHS.city);
+		const category = sanitizeText(data && data.category, MAX_LENGTHS.category);
+		const contact = sanitizeText(data && data.contact, MAX_LENGTHS.contact);
+		const description = sanitizeText(data && data.description, MAX_LENGTHS.description);
+
+		if (!name || !city || !category || !contact) {
+			throw new Error('[[error:invalid-data]]');
+		}
+
+		await db.setObject(GEMACH_KEY(id), { name, city, category, contact, description });
+		return { ok: true };
+	};
+
+	// מחיקת גמ"ח - הבעלים או מנהל בלבד. מוחק לגמרי מכל הרשימות (ממתין/מאושר).
+	SocketPlugins.gemachDirectory.remove = async function (socket, data) {
+		requireLogin(socket);
+		const id = data && data.id;
+		if (!id) throw new Error('[[error:invalid-data]]');
+
+		const gemach = await db.getObject(GEMACH_KEY(id));
+		if (!gemach) return { ok: true };
+		await requireOwnerOrAdmin(socket, gemach);
+
+		await db.sortedSetRemove(PENDING_SET, id);
+		await db.sortedSetRemove(APPROVED_SET, id);
+		await db.delete(GEMACH_KEY(id));
+
+		return { ok: true };
+	};
+
+	// נקרא מהלקוח כשמנהל נכנס לנושא "רשימת גמחים" עצמו - מסמן כ"נקראו" רק
+	// את התראות ה"גמ"ח חדש ממתין" (לא נוגע בשום התראה אחרת של המנהל), כי
+	// הוא כבר רואה את כל הבקשות הממתינות ממש עכשיו בעמוד.
+	SocketPlugins.gemachDirectory.markPendingNotificationsRead = async function (socket) {
+		await requireAdmin(socket);
+		const ids = await db.getSortedSetRange(PENDING_SET, 0, -1);
+		if (!ids.length) return { ok: true };
+		const nids = ids.map(id => `gemach-directory:${id}`);
+		await notifications.markReadMultiple(nids, socket.uid);
+		return { ok: true };
+	};
 }
 
 async function getGemachsFromSet(setKey, newestFirst) {
@@ -149,8 +212,27 @@ async function getGemachsFromSet(setKey, newestFirst) {
 		await db.getSortedSetRevRange(setKey, 0, -1) :
 		await db.getSortedSetRange(setKey, 0, -1);
 	if (!ids.length) return [];
-	const gemachs = await db.getObjects(ids.map(GEMACH_KEY));
-	return gemachs.filter(Boolean);
+	const gemachs = (await db.getObjects(ids.map(GEMACH_KEY))).filter(Boolean);
+	return attachSubmitterInfo(gemachs);
+}
+
+// מוסיף לכל גמ"ח את שם המשתמש (לקרדיט) ואת ה-userslug (לקישור לפרופיל)
+// של מי שהעלה אותו - כדי שהלקוח לא יצטרך שאילתת משתמש נפרדת לכל כרטיס.
+async function attachSubmitterInfo(gemachs) {
+	const uids = gemachs.map(g => g.submittedBy).filter(Boolean);
+	if (!uids.length) return gemachs;
+
+	const users = await user.getUsersFields(uids, ['uid', 'username', 'userslug']);
+	const byUid = {};
+	users.forEach((u) => { byUid[u.uid] = u; });
+
+	return gemachs.map((g) => {
+		const submitter = byUid[g.submittedBy];
+		return Object.assign({}, g, {
+			submittedByUsername: submitter ? submitter.username : null,
+			submittedByUserslug: submitter ? submitter.userslug : null,
+		});
+	});
 }
 
 async function notifyAdmins(gemach) {
@@ -167,7 +249,7 @@ async function notifyAdmins(gemach) {
 		// nid כולל את מזהה הגמ"ח - כך שכל הצעה חדשה היא התראה "חדשה" נפרדת
 		nid: `gemach-directory:${gemach.id}`,
 		bodyShort: `גמ"ח חדש ממתין לאישור: ${gemach.name} (${gemach.city})`,
-		path: '/',
+		path: NOTIFY_TARGET_PATH,
 		from: gemach.submittedBy,
 	});
 	await notifications.push(notification, targetUids);
@@ -187,6 +269,16 @@ async function requireAdmin(socket) {
 	}
 }
 
+// מרשה גישה רק למי שהעלה את הגמ"ח הזה (submittedBy) או למנהל.
+async function requireOwnerOrAdmin(socket, gemach) {
+	requireLogin(socket);
+	if (String(gemach.submittedBy) === String(socket.uid)) return;
+	const isAdmin = await user.isAdministrator(socket.uid);
+	if (!isAdmin) {
+		throw new Error('[[error:no-privileges]]');
+	}
+}
+
 // חיתוך אורך + הסרת תווי בקרה - הגנת שרת בסיסית. ההגנה האמיתית מפני
 // XSS היא ב-escaping בצד הלקוח בזמן הצגה (ראו gemach-directory-client.js),
 // כי הנתונים האלה מוזרקים שם ישירות ל-innerHTML.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-gemach-directory",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Server-side storage + admin approval queue for a community gemach directory, filterable by city and category.",
   "main": "library.js",
   "keywords": [