diff options
author | Dylan William Hardison <dylan@hardison.net> | 2017-09-07 17:04:07 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-07 17:04:07 +0200 |
commit | 946e9116c2a2a4ccbc44f657827087bf44ba295a (patch) | |
tree | c110b20d0f189946915d737180f78e835c27e9b1 /extensions/RestrictComments | |
parent | 8feb55f32ccd688d2b51105f4d56985abf0baa93 (diff) | |
download | bugzilla-946e9116c2a2a4ccbc44f657827087bf44ba295a.tar.gz bugzilla-946e9116c2a2a4ccbc44f657827087bf44ba295a.tar.xz |
Bug 1397747 - Move _get_all_group_names to Bugzilla::Config::Common::_get_all_group_names, and remove copy-pasted code
Diffstat (limited to 'extensions/RestrictComments')
-rw-r--r-- | extensions/RestrictComments/lib/Config.pm | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/extensions/RestrictComments/lib/Config.pm b/extensions/RestrictComments/lib/Config.pm index 55cf11187..e5dbc518c 100644 --- a/extensions/RestrictComments/lib/Config.pm +++ b/extensions/RestrictComments/lib/Config.pm @@ -23,14 +23,14 @@ sub get_param_list { { name => 'restrict_comments_group', type => 's', - choices => \&_get_all_group_names, + choices => \&get_all_group_names, default => '', checker => \&check_group }, { name => 'restrict_comments_enable_group', type => 's', - choices => \&_get_all_group_names, + choices => \&get_all_group_names, default => '', checker => \&check_group }, @@ -39,10 +39,4 @@ sub get_param_list { return @param_list; } -sub _get_all_group_names { - my @group_names = map {$_->name} Bugzilla::Group->get_all; - unshift(@group_names, ''); - return \@group_names; -} - 1; |