From f559a15d0537821bc507277b2ef4ef097872f154 Mon Sep 17 00:00:00 2001 From: David Lawrence Date: Mon, 7 Mar 2016 05:16:25 +0000 Subject: Bug 1252862 - Remove calls to delete_token() in several places where it is unnecessary --- extensions/BMO/Extension.pm | 1 - extensions/EditTable/Extension.pm | 1 - extensions/Push/lib/Admin.pm | 3 +-- 3 files changed, 1 insertion(+), 4 deletions(-) diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm index 35ce9b8d6..a72f3d1be 100644 --- a/extensions/BMO/Extension.pm +++ b/extensions/BMO/Extension.pm @@ -258,7 +258,6 @@ sub bounty_attachment { unless $input->{reporter_email}; check_hash_token($input->{token}, ['bounty', $bug->id]); - delete_token($input->{token}); my @fields = qw( reporter_email amount_paid reported_date fixed_date awarded_date publish ); my %form = map { $_ => $input->{$_} } @fields; diff --git a/extensions/EditTable/Extension.pm b/extensions/EditTable/Extension.pm index 675b3fe90..d353513e1 100644 --- a/extensions/EditTable/Extension.pm +++ b/extensions/EditTable/Extension.pm @@ -88,7 +88,6 @@ sub page_before_template { my $edits = []; if ($data) { check_hash_token($input->{token}, [$table_name]); - delete_token($input->{token}); $data = from_json($data)->{data}; $edits = dclone($data); diff --git a/extensions/Push/lib/Admin.pm b/extensions/Push/lib/Admin.pm index e11415ea6..5b0bb8e70 100644 --- a/extensions/Push/lib/Admin.pm +++ b/extensions/Push/lib/Admin.pm @@ -13,7 +13,7 @@ use warnings; use Bugzilla; use Bugzilla::Error; use Bugzilla::Extension::Push::Util; -use Bugzilla::Token qw(check_hash_token delete_token); +use Bugzilla::Token qw(check_hash_token); use Bugzilla::Util qw(trim detaint_natural trick_taint); use base qw(Exporter); @@ -31,7 +31,6 @@ sub admin_config { if ($input->{save}) { my $token = $input->{token}; check_hash_token($token, ['push_config']); - delete_token($token); my $dbh = Bugzilla->dbh; $dbh->bz_start_transaction(); _update_config_from_form('global', $push->config); -- cgit v1.2.3-24-g4f1b