summaryrefslogtreecommitdiffstats
path: root/extensions/AntiSpam/lib
diff options
context:
space:
mode:
authorDylan William Hardison <dylan@hardison.net>2017-09-07 17:04:07 +0200
committerGitHub <noreply@github.com>2017-09-07 17:04:07 +0200
commit946e9116c2a2a4ccbc44f657827087bf44ba295a (patch)
treec110b20d0f189946915d737180f78e835c27e9b1 /extensions/AntiSpam/lib
parent8feb55f32ccd688d2b51105f4d56985abf0baa93 (diff)
downloadbugzilla-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/AntiSpam/lib')
-rw-r--r--extensions/AntiSpam/lib/Config.pm8
1 files changed, 1 insertions, 7 deletions
diff --git a/extensions/AntiSpam/lib/Config.pm b/extensions/AntiSpam/lib/Config.pm
index b6d1234d9..e35a7f001 100644
--- a/extensions/AntiSpam/lib/Config.pm
+++ b/extensions/AntiSpam/lib/Config.pm
@@ -23,7 +23,7 @@ sub get_param_list {
{
name => 'antispam_spammer_exclude_group',
type => 's',
- choices => \&_get_all_group_names,
+ choices => \&get_all_group_names,
default => 'canconfirm',
checker => \&check_group
},
@@ -79,10 +79,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;