Attachment 10760 Details for Bug 8413 – Bug 8413 Space in barcodes breaks GET request in benchmark_staff.pl

[patch] Bug 8413 Space in barcodes breaks GET request in benchmark_staff.pl

Bug-8413-Space-in-barcodes-breaks-GET-request-in-b.patch (text/plain), 1.33 KB, created by Mirko Tietgen on 2012-07-10 22:05:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mirko Tietgen

Created: 2012-07-10 22:05:14 UTC

Size: 1.33 KB

patch

obsolete

>From eb4faa1444a026c36d8b0951320a31ff8eb25aad Mon Sep 17 00:00:00 2001
>From: Mirko Tietgen <5p4m@gmx.de>
>Date: Tue, 10 Jul 2012 23:55:18 +0200
>Subject: [PATCH] Bug 8413 Space in barcodes breaks GET request in benchmark_staff.pl
>Content-Type: text/plain; charset="utf-8"
>
>GET requests in benchmark_staff.pl test 6 do not work if a space character is part of the barcode. That seems highly unlikely to happen in barcodes, but is possible if no real barcodes are used but a substitute, like a copy of the call number. Space character needs to be changed to %20 for the request to work.
>---
> misc/load_testing/benchmark_staff.pl |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
>diff --git a/misc/load_testing/benchmark_staff.pl b/misc/load_testing/benchmark_staff.pl
>index ba659c9..19fb6bd 100644
>--- a/misc/load_testing/benchmark_staff.pl
>+++ b/misc/load_testing/benchmark_staff.pl
>@@ -325,6 +325,7 @@ if ($steps=~ /6/) {
>             my $rand_itemnumber = int(rand($itemnumber_max)+1);
>             $sth->execute($rand_itemnumber);
>             ($rand_barcode) = $sth->fetchrow();
>+            ($rand_barcode) =~ s/ /%20/g;
>         }
>         push @issues,"$baseurl/circ/circulation.pl?borrowernumber=$rand_borrowernumber&barcode=$rand_barcode&issueconfirmed=1";
>         push @returns,"$baseurl/circ/returns.pl?barcode=$rand_barcode";
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 8413: 10759 | 10760 | 10765 | 11397