nodebb-plugin-simple-contact 1.0.0 → 1.1.0

package/library.js

@@ -12,24 +12,16 @@ ContactPlugin.init = async function (params) {
     const router = params.router;
     const middleware = params.middleware;
 
-    console.log('--- Contact Plugin Initialized ---'); // בדיקה שהתוסף נטען
-
-    // Route for the user facing page
     router.get('/contact', middleware.buildHeader, renderContactPage);
     router.get('/api/contact', renderContactPage);
-
-    // API Route to submit form
     router.post('/api/contact/send', middleware.applyCSRF, handleContactSubmission);
 
-    // Route for Admin Panel
     router.get('/admin/plugins/contact', middleware.admin.buildHeader, renderAdminPage);
     router.get('/api/admin/plugins/contact', renderAdminPage);
-    
-    // API to mark as handled
     router.post('/api/admin/plugins/contact/handle', middleware.admin.buildHeader, markAsHandled);
+    router.post('/api/admin/plugins/contact/delete', middleware.admin.buildHeader, deleteRequest);
 };
 
-// Render User Page (עם תיקון פירורי הלחם)
 async function renderContactPage(req, res) {
     res.render('contact', {
         title: 'צור קשר',
@@ -40,11 +32,8 @@ async function renderContactPage(req, res) {
     });
 }
 
-// Handle Form Submission (עם הלוגיקה המשופרת והבטוחה)
 async function handleContactSubmission(req, res) {
     const data = req.body;
-
-    // Validation
     if (!data.fullName || !data.email || !data.content) {
         return res.status(400).json({ error: 'נא למלא את כל שדות החובה.' });
     }
@@ -63,41 +52,28 @@ async function handleContactSubmission(req, res) {
     };
 
     try {
-        // Save to DB
         await db.setObject(key, contactData);
         await db.sortedSetAdd('contact-requests:sorted', contactId, contactId);
 
-        // Send Notification to Admins
-        try {
-            const adminUids = await groups.getMembers('administrators', 0, -1);
-            
-            if (adminUids && adminUids.length > 0) {
-                const notification = await notifications.create({
-                    type: 'new-contact',
-                    bodyShort: `פנייה חדשה מאת ${contactData.fullName}`,
-                    bodyLong: contactData.content,
-                    nid: 'contact:' + contactId,
-                    path: '/admin/plugins/contact',
-                    from: 0 
-                });
-                
-                await notifications.push(notification, adminUids);
-            } else {
-                console.warn('Contact Plugin: No administrators found to notify.');
-            }
-        } catch (notifyErr) {
-            console.error('Contact Plugin Notification Error:', notifyErr);
-            // לא מכשילים את הבקשה אם רק ההתראה נכשלה
+        const adminUids = await groups.getMembers('administrators', 0, -1);
+        if (adminUids && adminUids.length > 0) {
+            const notification = await notifications.create({
+                type: 'new-contact',
+                bodyShort: `פנייה חדשה מאת ${contactData.fullName}`,
+                bodyLong: contactData.content,
+                nid: 'contact:' + contactId,
+                path: '/admin/plugins/contact',
+                from: 0 
+            });
+            await notifications.push(notification, adminUids);
         }
 
         res.json({ success: true, message: 'הפנייה נשלחה בהצלחה.' });
     } catch (err) {
-        console.error('Contact Plugin Error:', err);
-        res.status(500).json({ error: 'שגיאה פנימית בשליחת הטופס.' });
+        res.status(500).json({ error: 'שגיאה פנימית.' });
     }
 }
 
-// Render Admin Page
 async function renderAdminPage(req, res) {
     const ids = await db.getSortedSetRevRange('contact-requests:sorted', 0, -1);
     let items = [];
@@ -107,19 +83,26 @@ async function renderAdminPage(req, res) {
         items = await db.getObjects(keys);
     }
 
+    const waitingRequests = [];
+    const handledRequests = [];
+
     items.forEach(item => {
-        // המרת תאריך למחרוזת קריאה
         item.date = new Date(parseInt(item.timestamp)).toLocaleString();
+        if (item.handled) {
+            handledRequests.push(item);
+        } else {
+            waitingRequests.push(item);
+        }
     });
 
-    res.render('admin/plugins/contact', { requests: items });
+    res.render('admin/plugins/contact', { 
+        waitingRequests: waitingRequests, 
+        handledRequests: handledRequests 
+    });
 }
 
-// Mark as Handled logic
 async function markAsHandled(req, res) {
     const id = req.body.id;
-    if (!id) return res.status(400).json({ error: 'Missing ID' });
-
     try {
         await db.setObjectField('contact-request:' + id, 'handled', true);
         res.json({ success: true });
@@ -128,23 +111,24 @@ async function markAsHandled(req, res) {
     }
 }
 
-// Add link to main navigation
+async function deleteRequest(req, res) {
+    const id = req.body.id;
+    try {
+        await db.delete('contact-request:' + id);
+        await db.sortedSetRemove('contact-requests:sorted', id);
+        res.json({ success: true });
+    } catch (err) {
+        res.status(500).json({ error: err.message });
+    }
+}
+
 ContactPlugin.addNavigation = async function (header) {
-    header.navigation.push({
-        route: '/contact',
-        iconClass: 'fa-envelope',
-        text: 'צור קשר'
-    });
+    header.navigation.push({ route: '/contact', iconClass: 'fa-envelope', text: 'צור קשר' });
     return header;
 };
 
-// Add link to admin menu
 ContactPlugin.addAdminNavigation = async function (header) {
-    header.plugins.push({
-        route: '/plugins/contact',
-        icon: 'fa-envelope',
-        name: 'פניות צור קשר'
-    });
+    header.plugins.push({ route: '/plugins/contact', icon: 'fa-envelope', name: 'פניות צור קשר' });
     return header;
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-simple-contact",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "תוסף טופס צור קשר עם התראות למנהלים וניהול פניות",
   "main": "library.js",
   "nbbpm": {

package/plugin.json

@@ -1,20 +1,11 @@
 {
   "id": "nodebb-plugin-simple-contact",
   "name": "Simple Contact Form",
-  "description": "A contact form with admin notifications.",
-  "url": "http://localhost",
   "library": "./library.js",
   "hooks": [
     { "hook": "static:app.load", "method": "init" },
     { "hook": "filter:navigation.available", "method": "addNavigation" },
     { "hook": "filter:admin.header.build", "method": "addAdminNavigation" }
   ],
-  "scripts": [
-    "public/js/client.js"
-  ],
-  "acpScripts": [
-    "public/js/admin.js"
-  ],
-  "modules": {},
   "templates": "public/templates"
 }

package/public/templates/admin/plugins/contact.tpl

@@ -4,7 +4,8 @@
         <div class="panel panel-warning">
             <div class="panel-heading">ממתין - פניות לטיפול</div>
             <div class="panel-body">
-                <table class="table table-striped">
+                {{{ if waitingRequests.length }}}
+                <table class="table table-striped table-hover">
                     <thead>
                         <tr>
                             <th>תאריך</th>
@@ -17,8 +18,7 @@
                         </tr>
                     </thead>
                     <tbody id="contact-list-waiting">
-                        {{{ each requests }}}
-                        {{{ if !./handled }}}
+                        {{{ each waitingRequests }}}
                         <tr data-id="{./id}">
                             <td>{./date}</td>
                             <td>{./fullName}</td>
@@ -27,13 +27,17 @@
                             <td>{./content}</td>
                             <td><span class="label label-warning">ממתין</span></td>
                             <td>
-                                <button class="btn btn-sm btn-primary mark-handled">סמן כטופל</button>
+                                <button class="btn btn-xs btn-primary mark-handled" data-id="{./id}">סמן כטופל</button>
                             </td>
                         </tr>
                         {{{ end }}}
-                        {{{ end }}}
                     </tbody>
                 </table>
+                {{{ else }}}
+                <div class="alert alert-info text-center">
+                    אין פניות ממתינות כרגע.
+                </div>
+                {{{ end }}}
             </div>
         </div>
 
@@ -42,7 +46,8 @@
         <div class="panel panel-success">
             <div class="panel-heading">מטופל - ארכיון פניות</div>
             <div class="panel-body">
-                <table class="table table-striped">
+                {{{ if handledRequests.length }}}
+                <table class="table table-striped table-hover">
                     <thead>
                         <tr>
                             <th>תאריך</th>
@@ -55,8 +60,7 @@
                         </tr>
                     </thead>
                     <tbody id="contact-list-handled">
-                        {{{ each requests }}}
-                        {{{ if ./handled }}}
+                        {{{ each handledRequests }}}
                         <tr data-id="{./id}" class="success">
                             <td>{./date}</td>
                             <td>{./fullName}</td>
@@ -65,14 +69,150 @@
                             <td>{./content}</td>
                             <td><span class="label label-success">טופל</span></td>
                             <td>
-                                </td>
+                                <button class="btn btn-xs btn-danger delete-request" data-id="{./id}">מחק</button>
+                            </td>
                         </tr>
                         {{{ end }}}
-                        {{{ end }}}
                     </tbody>
                 </table>
+                {{{ else }}}
+                <div class="alert alert-info text-center">
+                    ארכיון הפניות ריק.
+                </div>
+                {{{ end }}}
             </div>
         </div>
 
     </div>
-</div>
+</div>
+
+<script>
+    (function() {
+        if (typeof define === 'function' && define.amd) {
+            define('admin/plugins/contact', [], function() { return {}; });
+        }
+
+        function showMessage(type, msg) {
+            if (window.app && typeof window.app.require === 'function') {
+                window.app.require('alerts').then(function(alerts) {
+                    if (type === 'success') {
+                        alerts.success(msg);
+                    } else {
+                        alerts.error(msg);
+                    }
+                }).catch(function(err) {
+                    console.error('Failed to load alerts via app.require:', err);
+                    fallbackAlert(type, msg);
+                });
+                return;
+            }
+
+            const req = window.require || window.requirejs;
+            if (req) {
+                req(['alerts'], function(alerts) {
+                    if (alerts) {
+                        if (type === 'success' && alerts.success) alerts.success(msg);
+                        else if (alerts.error) alerts.error(msg);
+                        else fallbackAlert(type, msg);
+                    } else {
+                        fallbackAlert(type, msg);
+                    }
+                });
+                return;
+            }
+
+            if (window.app && window.app.alert) {
+                window.app.alert({
+                    type: type === 'success' ? 'success' : 'danger',
+                    message: msg,
+                    timeout: 3000
+                });
+                return;
+            }
+
+            fallbackAlert(type, msg);
+        }
+
+        function fallbackAlert(type, msg) {
+            alert((type === 'success' ? 'הצלחה: ' : 'שגיאה: ') + msg.replace(/<[^>]*>/g, ''));
+        }
+
+        function initContactPage() {
+            $('.mark-handled').off('click');
+            $('.delete-request').off('click');
+
+            $('.mark-handled').on('click', function() {
+                var btn = $(this);
+                var id = btn.data('id');
+
+                if (window.app && window.app.require) {
+                    window.app.require('bootbox').then(function(bootbox) {
+                        bootbox.confirm('האם אתה בטוח שברצונך לסמן פנייה זו כטופלה?', function(confirm) {
+                            if (confirm) performHandle(id);
+                        });
+                    });
+                } else if (window.bootbox) {
+                     window.bootbox.confirm('האם אתה בטוח שברצונך לסמן פנייה זו כטופלה?', function(confirm) {
+                        if (confirm) performHandle(id);
+                    });
+                } else {
+                    if (confirm('האם אתה בטוח שברצונך לסמן פנייה זו כטופלה?')) performHandle(id);
+                }
+            });
+
+            $('.delete-request').on('click', function() {
+                var btn = $(this);
+                var id = btn.data('id');
+
+                if (window.app && window.app.require) {
+                    window.app.require('bootbox').then(function(bootbox) {
+                        bootbox.confirm('האם אתה בטוח שברצונך למחוק פנייה זו לצמיתות?', function(confirm) {
+                            if (confirm) performDelete(id);
+                        });
+                    });
+                } else if (window.bootbox) {
+                    window.bootbox.confirm('האם אתה בטוח שברצונך למחוק פנייה זו לצמיתות?', function(confirm) {
+                        if (confirm) performDelete(id);
+                    });
+                } else {
+                     if (confirm('האם אתה בטוח שברצונך למחוק פנייה זו לצמיתות?')) performDelete(id);
+                }
+            });
+        }
+
+        function performHandle(id) {
+            $.post(config.relative_path + '/api/admin/plugins/contact/handle', {
+                id: id,
+                _csrf: config.csrf_token
+            }, function(data) {
+                if (data.success) {
+                    showMessage('success', 'הפנייה סומנה כטופלה בהצלחה');
+                    ajaxify.refresh();
+                }
+            }).fail(function() {
+                showMessage('error', 'שגיאה בעדכון הפנייה');
+            });
+        }
+
+        function performDelete(id) {
+            $.post(config.relative_path + '/api/admin/plugins/contact/delete', {
+                id: id,
+                _csrf: config.csrf_token
+            }, function(data) {
+                if (data.success) {
+                    showMessage('success', 'הפנייה נמחקה בהצלחה');
+                    ajaxify.refresh();
+                }
+            }).fail(function() {
+                showMessage('error', 'שגיאה במחיקת הפנייה');
+            });
+        }
+
+        $(document).ready(initContactPage);
+        $(window).on('action:ajaxify.end', function(event, data) {
+            if (data.tpl_url === 'admin/plugins/contact') {
+                initContactPage();
+            }
+        });
+    })();
+</script>

package/public/templates/contact.tpl

@@ -1,7 +1,9 @@
 <div class="row">
     <div class="col-lg-8 col-lg-offset-2">
         <div class="panel panel-default">
-            <div class="panel-heading"><h3 class="panel-title">צור קשר עם ההנהלה</h3></div>
+            <div class="panel-heading">
+                <h3 class="panel-title">צור קשר עם ההנהלה</h3>
+            </div>
             <div class="panel-body">
                 <form id="contact-form" role="form">
                     <div class="form-group">
@@ -20,10 +22,90 @@
                         <label for="content">תוכן הפנייה *</label>
                         <textarea class="form-control" id="content" name="content" rows="6" required></textarea>
                     </div>
+                    
                     <button type="submit" class="btn btn-primary" id="submit-btn">שלח פנייה</button>
                 </form>
+
                 <div id="contact-alert" class="alert" style="display:none; margin-top: 20px;"></div>
             </div>
         </div>
     </div>
-</div>
+</div>
+
+<script>
+    (function() {
+        function initContactForm($) {
+            var form = $('#contact-form');
+            var btn = $('#submit-btn');
+            var alertBox = $('#contact-alert');
+
+            form.off('submit').on('submit', function(e) {
+                e.preventDefault();
+
+                btn.prop('disabled', true).text('שולח...');
+                alertBox.hide().removeClass('alert-success alert-danger');
+
+                var formData = {
+                    fullName: form.find('#fullName').val(),
+                    username: form.find('#username').val(),
+                    email: form.find('#email').val(),
+                    content: form.find('#content').val(),
+                    _csrf: config.csrf_token
+                };
+
+                $.ajax({
+                    url: config.relative_path + '/api/contact/send',
+                    type: 'POST',
+                    data: formData,
+                    success: function(response) {
+                        alertBox.addClass('alert-success')
+                                .text(response.message || 'הפנייה נשלחה בהצלחה!')
+                                .fadeIn();
+                        form[0].reset();
+                        btn.text('שלח פנייה');
+                    },
+                    error: function(xhr) {
+                        var msg = (xhr.responseJSON && xhr.responseJSON.error) ? xhr.responseJSON.error : 'אירעה שגיאה בשליחה.';
+                        alertBox.addClass('alert-danger')
+                                .text(msg)
+                                .fadeIn();
+                    },
+                    complete: function() {
+                        btn.prop('disabled', false).text('שלח פנייה');
+                    }
+                });
+            });
+        }
+
+        
+        function safeBoot() {
+            if (typeof jQuery === 'undefined') {
+                setTimeout(safeBoot, 50);
+                return;
+            }
+
+            var $ = jQuery;
+            
+            initContactForm($);
+
+            $(window).off('action:ajaxify.end.contact').on('action:ajaxify.end.contact', function(ev, data) {
+                if (data.url === 'contact' || (ajaxify.data.template && ajaxify.data.template.name === 'contact')) {
+                    initContactForm($);
+                }
+            });
+        }
+
+        if (typeof define === 'function' && define.amd) {
+            define('forum/contact', ['jquery'], function($) {
+                return {
+                    init: function() {
+                        initContactForm($);
+                    }
+                };
+            });
+        } 
+        
+        safeBoot();
+
+    })();
+</script>

package/public/js/admin.js

@@ -1,36 +0,0 @@
-'use strict';
-
-/* globals $, app, socket */
-
-// הקוד הזה ירוץ ברגע שפאנל הניהול נטען
-$(document).ready(function () {
-    // האזנה לאירוע ש-NodeBB מסיים לטעון דף חדש (Ajaxify)
-    $(window).on('action:ajaxify.end', function (event, data) {
-        // בדיקה: האם הגענו לדף של התוסף שלנו?
-        if (data.tpl_url === 'admin/plugins/contact') {
-            initializeAdminPage();
-        }
-    });
-
-    // אם אנחנו כבר בדף הזה ברגע הטעינה הראשונה
-    if (ajaxify.data.template && ajaxify.data.template['admin/plugins/contact']) {
-        initializeAdminPage();
-    }
-});
-
-function initializeAdminPage() {
-    console.log('✅ Contact Admin Page logic triggered via Hook');
-
-    // לוגיקה לסימון כטופל
-    $('.mark-handled-btn').on('click', function () {
-        var id = $(this).data('id');
-        var btn = $(this);
-
-        $.post('/api/admin/plugins/contact/handle', { id: id }, function (data) {
-            if (data.success) {
-                btn.closest('tr').addClass('table-success');
-                btn.replaceWith('<span class="badge bg-success">טופל</span>');
-            }
-        });
-    });
-}

package/public/js/client.js

@@ -1,54 +0,0 @@
-'use strict';
-
-// החלק הזה מטפל בלוגיקה של הטופס
-$(document).ready(function() {
-    $(window).on('action:ajaxify.end', function(event, data) {
-        if (data.url === 'contact' || ajaxify.data.template.name === 'contact') {
-            $('#contact-form').off('submit').on('submit', function(e) {
-                e.preventDefault();
-                var btn = $('#submit-btn');
-                var alertBox = $('#contact-alert');
-                var form = $(this);
-
-                btn.prop('disabled', true);
-                
-                var formData = {
-                    fullName: form.find('#fullName').val(),
-                    username: form.find('#username').val(),
-                    email: form.find('#email').val(),
-                    content: form.find('#content').val()
-                };
-
-                $.ajax({
-                    url: config.relative_path + '/api/contact/send',
-                    type: 'POST',
-                    data: formData,
-                    headers: { 'x-csrf-token': config.csrf_token },
-                    success: function(response) {
-                        alertBox.removeClass('alert-danger').addClass('alert-success')
-                                .text(response.message).show();
-                        form[0].reset();
-                    },
-                    error: function(xhr) {
-                        // הצגת הודעת שגיאה מפורטת אם השרת החזיר כזו
-                        var msg = (xhr.responseJSON && xhr.responseJSON.error) ? xhr.responseJSON.error : 'שגיאה בשליחה';
-                        alertBox.removeClass('alert-success').addClass('alert-danger')
-                                .text(msg).show();
-                    },
-                    complete: function() {
-                        btn.prop('disabled', false);
-                    }
-                });
-            });
-        }
-    });
-});
-
-// החלק הזה "מרגיע" את NodeBB שלא יחפש מודול חסר
-define('forum/contact', [], function() {
-    return {
-        init: function() {
-            console.log('Contact page loaded.');
-        }
-    };
-});