Bug 622080 - Change group which can moderate quips

r/a=mkanat

diff --git a/Bugzilla/Install.pm b/Bugzilla/Install.pm
index 9d4426bbe8c0a529ad59f7e44b9ccabaa4b15798..6fbaa0b751fdd385e70d2e3cb1da2f1edc433fa4 100644 (file)
--- a/Bugzilla/Install.pm
+++ b/Bugzilla/Install.pm
@@ -149,6 +149,10 @@ use constant SYSTEM_GROUPS => (
         description  => 'Can not be impersonated by other users',
         inherited_by => ['bz_sudoers'],
     },
+    {
+        name         => 'bz_quip_moderators',
+        description  => 'Can moderate quips',
+    },
 );
 
 use constant DEFAULT_CLASSIFICATION => {

diff --git a/docs/en/xml/using.xml b/docs/en/xml/using.xml
index 0dbff34bcbffb4c47adf0d8ecb9a60f5f85cf98a..6d575c0c1095566df5f8e7eb35d6459c870b47e6 100644 (file)
--- a/docs/en/xml/using.xml
+++ b/docs/en/xml/using.xml
@@ -1375,7 +1375,18 @@
              Indicates user can configure whine reports for self.
             </para>
           </listitem>
-        </varlistentry>                
+        </varlistentry>
+
+        <varlistentry>
+          <term>
+             bz_quip_moderators
+          </term>
+          <listitem>
+            <para>
+             Indicates user can moderate quips.
+            </para>
+          </listitem>
+        </varlistentry>
  
         <varlistentry>
           <term>

diff --git a/quips.cgi b/quips.cgi
index 97993d488a48a3bff9d69e8eace8bba241102105..74c0047a1ebc7464f12d2f0ea508319a714819ae 100755 (executable)
--- a/quips.cgi
+++ b/quips.cgi
@@ -79,7 +79,7 @@ if ($action eq "add") {
     check_hash_token($token, ['create-quips']);
     # Add the quip 
     my $approved = (Bugzilla->params->{'quip_list_entry_control'} eq "open")
-                   || Bugzilla->user->in_group('admin') || 0;
+                   || $user->in_group('bz_quip_moderators') || 0;
     my $comment = $cgi->param("quip");
     $comment || ThrowUserError("need_quip");
     trick_taint($comment); # Used in a placeholder below
@@ -91,8 +91,8 @@ if ($action eq "add") {
 }
 
 if ($action eq 'approve') {
-    $user->in_group('admin')
-      || ThrowUserError("auth_failure", {group  => "admin",
+    $user->in_group('bz_quip_moderators')
+      || ThrowUserError("auth_failure", {group  => "bz_quip_moderators",
                                          action => "approve",
                                          object => "quips"});
 
@@ -131,8 +131,8 @@ if ($action eq 'approve') {
 }
 
 if ($action eq "delete") {
-    Bugzilla->user->in_group("admin")
-      || ThrowUserError("auth_failure", {group  => "admin",
+    $user->in_group('bz_quip_moderators')
+      || ThrowUserError("auth_failure", {group  => "bz_quip_moderators",
                                          action => "delete",
                                          object => "quips"});
     my $quipid = $cgi->param("quipid");

diff --git a/template/en/default/admin/params/query.html.tmpl b/template/en/default/admin/params/query.html.tmpl
index 34ea043811f59b180183c6491da38d33bcc7ea07..c5bac664190db8358a27c2145fbf8d2f6f076673 100644 (file)
--- a/template/en/default/admin/params/query.html.tmpl
+++ b/template/en/default/admin/params/query.html.tmpl
@@ -32,7 +32,7 @@
                                 </li>
                                 <li>
                                   moderated - quips can be entered, but need to be approved
-                                  by an admin before they will be shown.
+                                  by a moderator before they will be shown.
                                 </li>
                                 <li>
                                   closed - no new additions to the quips list are allowed.

diff --git a/template/en/default/list/quips.html.tmpl b/template/en/default/list/quips.html.tmpl
index 62395d27f1b8e65c009cde82d5978a82f80a047c..512add9957a11b730d1c9b9a0bc3283d4250bc90 100644 (file)
--- a/template/en/default/list/quips.html.tmpl
+++ b/template/en/default/list/quips.html.tmpl
@@ -37,7 +37,7 @@
   <p>
     <font color="red">
       Your quip '<tt>[% added_quip FILTER html %]</tt>' has been added.
-      [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('admin') %]
+      [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('bz_quip_moderators') %]
         It will be used as soon as it gets approved.
       [% END %]
     </font>
@@ -66,7 +66,7 @@
   <p>
     You can extend the quip list. Type in something clever or funny or boring
     (but not obscene or offensive, please) and bonk on the button.
-    [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('admin') %]
+    [% IF Param("quip_list_entry_control") == "moderated" AND !user.in_group('bz_quip_moderators') %]
       Note that your quip has to be approved before it is used.
     [% END %]
   </p>
@@ -86,7 +86,7 @@
 [% END %]
 
 [% IF show_quips %]
-  [% IF !user.in_group('admin') %]
+  [% IF !user.in_group('bz_quip_moderators') %]
     <h2>
       Existing quips:
     </h2>
@@ -163,7 +163,7 @@
   <p>
     Those who like their wisdom in large doses can
     <a href="quips.cgi?action=show">view
-    [% IF user.in_group('admin') %]
+    [% IF user.in_group('bz_quip_moderators') %]
       and edit
     [% END %]
     the whole quip list</a>.