diff options
author | Frédéric Buclin <LpSolit@gmail.com> | 2013-09-28 00:37:03 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2013-09-28 00:37:03 +0200 |
commit | 48bc6d3981dfedf1183a76f2ccc17d8889edaa56 (patch) | |
tree | 0c938bcb0ac1b62ea4db658c727fa5d0b6d51145 /Bugzilla | |
parent | 5d0e6171ebce1410aedb378aa64e65abbcf916eb (diff) | |
download | bugzilla-48bc6d3981dfedf1183a76f2ccc17d8889edaa56.tar.gz bugzilla-48bc6d3981dfedf1183a76f2ccc17d8889edaa56.tar.xz |
Bug 851267: Bugzilla times out when a user has several thousands of votes
r=dkl a=justdave
Diffstat (limited to 'Bugzilla')
-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 991bfc05d..05863bf02 100644 --- a/Bugzilla/CGI.pm +++ b/Bugzilla/CGI.pm @@ -473,9 +473,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; } } @@ -529,7 +529,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 { |