nodebb-plugin-chat-search 0.0.1 → 0.0.3

package/library.js

@@ -23,7 +23,6 @@ async function searchGlobal(socket, data) {
     }
 
     let targetUid = socket.uid;
-    // בדיקת הרשאות (אדמין צופה במישהו אחר)
     if (data.targetUid && parseInt(data.targetUid, 10) !== parseInt(socket.uid, 10)) {
         const isAdmin = await user.isAdministrator(socket.uid);
         if (!isAdmin) {
@@ -38,12 +37,10 @@ async function searchGlobal(socket, data) {
     let allResults = [];
 
     for (const roomId of roomIds) {
-        // בדיקת חברות
         const inRoom = await messaging.isUserInRoom(targetUid, roomId);
         if (!inRoom) continue;
 
         try {
-            // 1. שליפת הודעות
             const messages = await messaging.getMessages({
                 callerUid: socket.uid,
                 uid: targetUid,
@@ -60,32 +57,41 @@ async function searchGlobal(socket, data) {
             );
 
             if (matches.length > 0) {
-                // --- תיקון שליפת המשתתפים ---
-                // במקום להסתמך על roomData, נשלוף UIDs ואז Users
+                // שליפת משתמשים
                 const uids = await messaging.getUidsInRoom(roomId, 0, -1);
-                const usersData = await user.getUsersFields(uids, ['uid', 'username', 'picture']);
+                const usersData = await user.getUsersFields(uids, ['uid', 'username']);
                 
-                // סינון המשתמש שבו אנו צופים
-                const participants = usersData
-                    .filter(u => parseInt(u.uid, 10) !== parseInt(targetUid, 10))
-                    .map(u => u.username)
-                    .join(', ');
+                // סינון המשתמש הנוכחי (משאירים רק את הפרטנרים)
+                const otherUsers = usersData.filter(u => parseInt(u.uid, 10) !== parseInt(targetUid, 10));
+
+                // --- לוגיקת הקיצור (2 שמות + ועוד X) ---
+                let displayName = '';
+                
+                if (otherUsers.length === 0) {
+                    displayName = 'צ\'אט עצמי'; // או משתמש מחוק
+                } else if (otherUsers.length <= 2) {
+                    // אם יש 1 או 2, מציגים את כולם
+                    displayName = otherUsers.map(u => u.username).join(', ');
+                } else {
+                    // אם יש יותר מ-2, לוקחים את ה-2 הראשונים ומוסיפים את היתרה
+                    const firstTwo = otherUsers.slice(0, 2).map(u => u.username).join(', ');
+                    const remaining = otherUsers.length - 2;
+                    displayName = `${firstTwo} ועוד ${remaining} משתמשים`;
+                }
+                // ----------------------------------------
 
-                // --- תיקון שליפת שם החדר ---
                 const roomData = await messaging.getRoomData(roomId);
-                let roomName = (roomData && roomData.roomName) || participants || `חדר ${roomId}`;
+                // שם החדר: אם יש שם מוגדר לקבוצה - קח אותו, אחרת קח את השמות שיצרנו
+                let roomName = (roomData && roomData.roomName) || displayName;
 
-                // --- תיקון שם השולח (במקרה שחסר בהודעה) ---
-                // אם חסר אובייקט user בהודעה, נשלים אותו מתוך usersData ששלפנו הרגע
                 matches.forEach(m => {
                     if (!m.user || !m.user.username) {
                         const sender = usersData.find(u => parseInt(u.uid, 10) === parseInt(m.fromuid, 10));
                         m.user = sender || { username: 'Unknown' };
                     }
-
                     m.roomName = roomName;
-                    m.chatWith = participants;
                     m.targetUid = targetUid;
+                    // מחקנו את chatWith כי כבר לא צריך אותו
                 });
                 
                 allResults = allResults.concat(matches);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-chat-search",
-  "version": "0.0.1",
+  "version": "0.0.3",
   "description": "A plugin to search text within NodeBB chats",
   "main": "library.js",
   "nbbpm": {

package/static/lib/main.js

@@ -1,59 +1,110 @@
 'use strict';
 
+// מצב גלובלי לזיכרון בין מעברי דפים
+window.chatSearchState = window.chatSearchState || {
+    query: '',
+    resultsHtml: '',
+    isOpen: false,
+    lastScroll: 0
+};
+
 $(document).ready(function () {
-    console.log('[Chat Search] Loaded (AJAX Fix).');
+    console.log('[Chat Search] Loaded (Instant No-Flicker).');
+
+    let observer = null;
+
+    // האזנה למעבר דפים
+    $(window).on('action:ajaxify.start', function () {
+        // לפני שהדף מתחלף, ננסה לשמור את האלמנט בזיכרון אם נרצה להחזירו (אופציונלי)
+        // כרגע אנו מסתמכים על בנייה מחדש מהירה מאוד
+    });
 
-    // בכל מעבר דף, אנחנו מוחקים את הישן ומתחילים חיפוש חדש של הקונטיינר
     $(window).on('action:ajaxify.end', function (ev, data) {
-        $('#global-chat-search-container').remove(); // איפוס
-        
-        // בדיקה: האם אנחנו בעמוד צ'אטים?
-        // גם לפי URL וגם לפי Template
+        // אם יש observer ישן, ננתק אותו כדי לא להעמיס
+        if (observer) observer.disconnect();
+
         const isChatUrl = data.url.match(/^(user\/[^\/]+\/)?chats/);
         const isChatTemplate = data.template && data.template.name === 'chats';
 
         if (isChatUrl || isChatTemplate) {
-            waitForElementAndAdd();
+            initFastInjection();
+        } else {
+            // יציאה מאזור הצ'אטים - איפוס הזיכרון
+            window.chatSearchState = { query: '', resultsHtml: '', isOpen: false, lastScroll: 0 };
         }
     });
 
     $(window).on('action:chat.loaded', function (ev, data) {
+        highlightActiveChat();
         handleScrollToMessage();
     });
 
-    // הרצה ראשונית (למקרה של ריענון מלא)
-    if (ajaxify.data.template && ajaxify.data.template.name === 'chats') {
-        waitForElementAndAdd();
+    // בדיקה ראשונית
+    if (ajaxify.data && ajaxify.data.template && ajaxify.data.template.name === 'chats') {
+        initFastInjection();
     }
 
-    function waitForElementAndAdd() {
+    function initFastInjection() {
+        // 1. ניסיון הזרקה מיידי
+        tryInject();
+
+        // 2. הפעלת MutationObserver לזיהוי שינויים ב-DOM בזמן אמת (מונע קפיצות)
+        const targetNode = document.body;
+        const config = { childList: true, subtree: true };
+
+        observer = new MutationObserver(function(mutationsList) {
+            for(let mutation of mutationsList) {
+                if (mutation.type === 'childList') {
+                    // אם נוספו אלמנטים לדף, נבדוק אם הסרגל הגיע
+                    const container = findContainer();
+                    if (container.length > 0 && container.find('#global-chat-search-container').length === 0) {
+                        addGlobalSearchBar(container);
+                    }
+                }
+            }
+        });
+
+        observer.observe(targetNode, config);
+
+        // 3. גיבוי: Interval מהיר מאוד (50ms) למקרה שה-Observer פספס
         let attempts = 0;
-        const interval = setInterval(function() {
+        const interval = setInterval(() => {
             attempts++;
-            
-            // חיפוש הקונטיינר החדש שנוצר
-            let container = $('[component="chat/nav-wrapper"]'); 
-            if (container.length === 0) container = $('.chats-page').find('.col-md-4').first(); 
-
-            // ברגע שמצאנו - מזריקים ועוצרים
-            if (container.length > 0) {
-                addGlobalSearchBar(container);
-                clearInterval(interval);
-            } else if (attempts >= 20) { // מנסים במשך 4 שניות
+            if (tryInject()) {
+                // לא עוצרים את האינטרוול מיד, כי לפעמים NodeBB מרענן פעמיים
+                if (attempts > 20) clearInterval(interval);
+            } else if (attempts > 40) { // 2 שניות
                 clearInterval(interval);
             }
-        }, 200); // בדיקה מהירה כל 200ms
+        }, 50);
+    }
+
+    function findContainer() {
+        // סדר עדיפויות למציאת הקונטיינר
+        let container = $('[component="chat/nav-wrapper"]'); 
+        if (container.length === 0) container = $('.chats-page').find('.col-md-4').first();
+        return container;
+    }
+
+    function tryInject() {
+        const container = findContainer();
+        if (container.length > 0) {
+            addGlobalSearchBar(container);
+            return true;
+        }
+        return false;
     }
 
     function addGlobalSearchBar(container) {
         if ($('#global-chat-search-container').length > 0) return;
 
+        // ה-HTML של התיבה
         const searchHtml = `
             <div id="global-chat-search-container" style="padding: 10px; background: #fff; border-bottom: 1px solid #ddd; margin-bottom: 10px;">
                 <div class="input-group">
-                    <input type="text" id="global-chat-search" class="form-control" placeholder="חפש הודעה..." style="font-size: 14px;">
+                    <input type="text" id="global-chat-search" class="form-control" placeholder="חפש הודעה..." style="font-size: 14px; height: 34px;">
                     <span class="input-group-btn">
-                        <button class="btn btn-primary" id="btn-chat-search" type="button"><i class="fa fa-search"></i></button>
+                        <button class="btn btn-primary" id="btn-chat-search" type="button" style="height: 34px;"><i class="fa fa-search"></i></button>
                     </span>
                 </div>
                 <div id="global-search-results" style="margin-top: 5px; max-height: 400px; overflow-y: auto; background: white; border: 1px solid #eee; display:none;"></div>
@@ -62,10 +113,39 @@ $(document).ready(function () {
 
         container.prepend(searchHtml);
         
+        // --- שחזור מיידי של המצב (מונע הבהוב של התוכן) ---
+        const input = $('#global-chat-search');
+        const results = $('#global-search-results');
+        
+        if (window.chatSearchState.query) {
+            input.val(window.chatSearchState.query);
+        }
+        
+        if (window.chatSearchState.isOpen && window.chatSearchState.resultsHtml) {
+            results.html(window.chatSearchState.resultsHtml).show();
+            // שחזור מיקום הגלילה
+            if (window.chatSearchState.lastScroll > 0) {
+                results.scrollTop(window.chatSearchState.lastScroll);
+            }
+            highlightActiveChat();
+        }
+        // ------------------------------------------------
+
+        // הגדרת אירועים (Events)
         $('#btn-chat-search').off('click').on('click', executeSearch);
-        $('#global-chat-search').off('keypress').on('keypress', function (e) {
+        
+        // שמירת גלילה
+        results.on('scroll', function() {
+            window.chatSearchState.lastScroll = $(this).scrollTop();
+        });
+
+        input.off('keypress').on('keypress', function (e) {
             if (e.which === 13) executeSearch();
         });
+        
+        input.on('input', function() {
+            window.chatSearchState.query = $(this).val();
+        });
     }
 
     function executeSearch() {
@@ -74,12 +154,15 @@ $(document).ready(function () {
 
         if (!query) {
             resultsContainer.hide();
+            window.chatSearchState.isOpen = false;
+            window.chatSearchState.resultsHtml = '';
             return;
         }
 
         let targetUid = ajaxify.data.uid || app.user.uid;
 
         resultsContainer.show().html('<div class="text-center" style="padding:10px;"><i class="fa fa-spinner fa-spin"></i> מחפש...</div>');
+        window.chatSearchState.isOpen = true;
 
         socket.emit('plugins.chatSearch.searchGlobal', { 
             query: query,
@@ -87,12 +170,14 @@ $(document).ready(function () {
         }, function (err, messages) {
             if (err) {
                 console.error(err);
-                resultsContainer.html('<div class="alert alert-danger" style="margin:5px;">שגיאה בחיפוש</div>');
+                resultsContainer.html('<div class="alert alert-danger" style="margin:5px;">שגיאה</div>');
                 return;
             }
 
             if (!messages || messages.length === 0) {
-                resultsContainer.html('<div class="text-center" style="padding:10px; color:#777;">לא נמצאו תוצאות.</div>');
+                const noRes = '<div class="text-center" style="padding:10px; color:#777;">לא נמצאו תוצאות.</div>';
+                resultsContainer.html(noRes);
+                window.chatSearchState.resultsHtml = noRes;
                 return;
             }
 
@@ -100,37 +185,50 @@ $(document).ready(function () {
             
             messages.forEach(msg => {
                 const date = new Date(msg.timestamp).toLocaleDateString();
-                
-                let baseUrl = window.location.pathname; 
+                let baseUrl = window.location.pathname.replace(/\/chats\/.*$/, '/chats');
                 if (baseUrl.endsWith('/')) baseUrl = baseUrl.slice(0, -1);
-                if (baseUrl.match(/\/[0-9]+$/)) baseUrl = baseUrl.replace(/\/[0-9]+$/, '');
 
                 const chatLink = baseUrl + '/' + msg.roomId + '?mid=' + msg.mid;
                 const senderName = (msg.user && msg.user.username) ? msg.user.username : 'Unknown';
-                
-                let participantsHtml = '';
-                if (msg.chatWith) {
-                    participantsHtml = `<div style="font-size:11px; color:#005999; margin-bottom:2px;"><i class="fa fa-users"></i> עם: <strong>${msg.chatWith}</strong></div>`;
-                }
 
                 html += `
-                    <li class="list-group-item search-result" style="cursor:pointer; border-bottom: 1px solid #f0f0f0; padding: 8px;" onclick="ajaxify.go('${chatLink}')">
-                        <div style="font-size:13px; color:#333; margin-bottom:2px;">
+                    <li class="list-group-item search-result" data-roomid="${msg.roomId}" style="cursor:pointer; border-bottom: 1px solid #f0f0f0; padding: 10px 8px;" onclick="ajaxify.go('${chatLink}')">
+                        <div style="font-size:13px; color:#333; margin-bottom:5px;">
+                            <span class="pull-left text-muted" style="font-size:10px; margin-right: 8px;">${date}</span>
                             <strong>${msg.roomName}</strong> 
-                            <span class="pull-left text-muted" style="font-size:10px;">${date}</span>
                         </div>
-                        ${participantsHtml}
-                        <div style="font-size:12px; color:#444; background: #f9f9f9; padding: 4px; border-radius: 3px; border-right: 2px solid #007bff;">
+                        <div style="font-size:12px; color:#444; background: #f9f9f9; padding: 6px; border-radius: 4px; border-right: 3px solid #007bff;">
                             <strong>${senderName}:</strong> ${msg.content}
                         </div>
                     </li>
                 `;
             });
             html += '</ul>';
+            
             resultsContainer.html(html);
+            
+            // עדכון הזיכרון
+            window.chatSearchState.resultsHtml = html;
+            window.chatSearchState.lastScroll = 0;
+            
+            highlightActiveChat();
         });
     }
 
+    function highlightActiveChat() {
+        // מנסים לקחת RoomID מה-URL או מהמידע של NodeBB
+        let currentRoomId = ajaxify.data.roomId;
+        if (!currentRoomId) {
+             const match = window.location.pathname.match(/chats\/(\d+)/);
+             if (match) currentRoomId = match[1];
+        }
+
+        if (!currentRoomId) return;
+
+        $('.search-result').css('background-color', '');
+        $('.search-result[data-roomid="' + currentRoomId + '"]').css('background-color', '#eef6ff');
+    }
+
     function handleScrollToMessage() {
         const params = new URLSearchParams(window.location.search);
         const mid = params.get('mid');
@@ -140,8 +238,8 @@ $(document).ready(function () {
         let attempts = 0;
         const scrollInt = setInterval(() => {
             attempts++;
-            if (scrollToId(mid) || attempts > 10) clearInterval(scrollInt);
-        }, 500);
+            if (scrollToId(mid) || attempts > 15) clearInterval(scrollInt);
+        }, 300);
     }
 
     function scrollToId(mid) {