This is an automated email from the ASF dual-hosted git repository.

brondsem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/allura.git


The following commit(s) were added to refs/heads/master by this push:
     new f8e56d41e fix bugs on the discussion moderation form
f8e56d41e is described below

commit f8e56d41e5e0408cabb82d2322bcaa5ce0dd6501
Author: Dave Brondsema <[email protected]>
AuthorDate: Fri Aug 9 11:21:43 2024 -0400

    fix bugs on the discussion moderation form
---
 Allura/allura/controllers/discuss.py | 2 +-
 Allura/allura/lib/widgets/discuss.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Allura/allura/controllers/discuss.py 
b/Allura/allura/controllers/discuss.py
index d0006c6e4..0d732f43c 100644
--- a/Allura/allura/controllers/discuss.py
+++ b/Allura/allura/controllers/discuss.py
@@ -559,12 +559,12 @@ class ModerationController(BaseController, 
metaclass=h.ProxiedAttrMeta):
 
             if posted:
                 if delete:
+                    count += 1
                     # full, for real, delete since this is post was never 
visible and denied by admin.
                     posted.delete()
                     # If we just deleted the last post in the
                     # thread, delete the thread.
                     if posted.thread and posted.thread.num_replies == 0:
-                        count += 1
                         posted.thread.delete()
                 elif spam and posted.status != 'spam':
                     count += 1
diff --git a/Allura/allura/lib/widgets/discuss.py 
b/Allura/allura/lib/widgets/discuss.py
index 302a8af40..1b8f440e6 100644
--- a/Allura/allura/lib/widgets/discuss.py
+++ b/Allura/allura/lib/widgets/discuss.py
@@ -86,7 +86,7 @@ class ModeratePosts(ew.SimpleForm):
       (function($){
           var tbl = $('form table');
           var checkboxes = $('input[type=checkbox]', tbl);
-          $('a[href=#]', tbl).click(function () {
+          $('a[href="#"]', tbl).click(function () {
               checkboxes.each(function () {
                   if(this.checked) { this.checked = false; }
                   else { this.checked = true; }

Reply via email to