From 28a1de6319da8b481b9b5ec08f070bce65e17bb3 Mon Sep 17 00:00:00 2001
From: Dylan William Hardison <dylan@hardison.net>
Date: Fri, 4 Aug 2017 13:09:47 -0400
Subject: Bug 1387459 - Tidy the Bugzilla/Config/*.pm modules

---
 Bugzilla/Config/GroupSecurity.pm | 143 ++++++++++++++++++++-------------------
 1 file changed, 72 insertions(+), 71 deletions(-)

(limited to 'Bugzilla/Config/GroupSecurity.pm')

diff --git a/Bugzilla/Config/GroupSecurity.pm b/Bugzilla/Config/GroupSecurity.pm
index 016bc82f7..c9d6421d3 100644
--- a/Bugzilla/Config/GroupSecurity.pm
+++ b/Bugzilla/Config/GroupSecurity.pm
@@ -17,80 +17,81 @@ use Bugzilla::Group;
 our $sortkey = 900;
 
 sub get_param_list {
-  my $class = shift;
-
-  my @param_list = (
-  {
-   name => 'makeproductgroups',
-   type => 'b',
-   default => 0
-  },
-
-  {
-   name => 'chartgroup',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => 'editbugs',
-   checker => \&check_group
-  },
-
-  {
-   name => 'insidergroup',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => '',
-   checker => \&check_group
-  },
-
-  {
-   name => 'timetrackinggroup',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => 'editbugs',
-   checker => \&check_group
-  },
-
-  {
-   name => 'querysharegroup',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => 'editbugs',
-   checker => \&check_group
-  },
-
-  {
-   name => 'comment_taggers_group',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => 'editbugs',
-   checker => \&check_comment_taggers_group
-  },
-
-  {
-   name => 'debug_group',
-   type => 's',
-   choices => \&_get_all_group_names,
-   default => 'admin',
-   checker => \&check_group
-  },
-
-  {
-   name => 'usevisibilitygroups',
-   type => 'b',
-   default => 0
-  },
-
-  {
-   name => 'strict_isolation',
-   type => 'b',
-   default => 0
-  } );
-  return @param_list;
+    my $class = shift;
+
+    my @param_list = (
+        {
+            name    => 'makeproductgroups',
+            type    => 'b',
+            default => 0
+        },
+
+        {
+            name    => 'chartgroup',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => 'editbugs',
+            checker => \&check_group
+        },
+
+        {
+            name    => 'insidergroup',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => '',
+            checker => \&check_group
+        },
+
+        {
+            name    => 'timetrackinggroup',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => 'editbugs',
+            checker => \&check_group
+        },
+
+        {
+            name    => 'querysharegroup',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => 'editbugs',
+            checker => \&check_group
+        },
+
+        {
+            name    => 'comment_taggers_group',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => 'editbugs',
+            checker => \&check_comment_taggers_group
+        },
+
+        {
+            name    => 'debug_group',
+            type    => 's',
+            choices => \&_get_all_group_names,
+            default => 'admin',
+            checker => \&check_group
+        },
+
+        {
+            name    => 'usevisibilitygroups',
+            type    => 'b',
+            default => 0
+        },
+
+        {
+            name    => 'strict_isolation',
+            type    => 'b',
+            default => 0
+        }
+    );
+    return @param_list;
 }
 
 sub _get_all_group_names {
-    my @group_names = map {$_->name} Bugzilla::Group->get_all;
-    unshift(@group_names, '');
+    my @group_names = map { $_->name } Bugzilla::Group->get_all;
+    unshift( @group_names, '' );
     return \@group_names;
 }
 
-- 
cgit v1.2.3-24-g4f1b