diff options
author | Dave Lawrence <dlawrence@mozilla.com> | 2013-09-30 22:01:18 +0200 |
---|---|---|
committer | Dave Lawrence <dlawrence@mozilla.com> | 2013-09-30 22:01:18 +0200 |
commit | 1941de37c0f7e49069fee49cfebcaa842a58c420 (patch) | |
tree | f7cd5c314a7e3f0933b10153cbe71f5e8eedbc1a /Bugzilla/CGI.pm | |
parent | df32bbf5aabe22fc393182b021d12fb22940508f (diff) | |
download | bugzilla-1941de37c0f7e49069fee49cfebcaa842a58c420.tar.gz bugzilla-1941de37c0f7e49069fee49cfebcaa842a58c420.tar.xz |
Bug 922246 - backport upstream bug 851267 to bmo/4.2 for performance improvement with large number of votes
Diffstat (limited to 'Bugzilla/CGI.pm')
-rw-r--r-- | Bugzilla/CGI.pm | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm index 58a681491..c003e9295 100644 --- a/Bugzilla/CGI.pm +++ b/Bugzilla/CGI.pm @@ -505,9 +505,9 @@ sub redirect_search_url { # GET requests that lacked a list_id are always redirected. POST requests # are only redirected if they're under the CGI_URI_LIMIT though. - my $uri_length = length($self->self_url()); - if ($self->request_method() ne 'POST' or $uri_length < CGI_URI_LIMIT) { - print $self->redirect(-url => $self->self_url()); + my $self_url = $self->self_url(); + if ($self->request_method() ne 'POST' or length($self_url) < CGI_URI_LIMIT) { + print $self->redirect(-url => $self_url); exit; } } @@ -561,7 +561,7 @@ sub url_is_attachment_base { $regex =~ s/\\\%bugid\\\%/\\d+/; } $regex = "^$regex"; - return ($self->self_url =~ $regex) ? 1 : 0; + return ($self->url =~ $regex) ? 1 : 0; } sub set_dated_content_disp { |