http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6510

--- Comment #1 from Magnus Enger <[email protected]> 2011-06-16 10:37:20 UTC 
---
After some more digging, it looks like the problem stems from the regex on line
474 of catalogue/search.pl: 

472 for my $this_cgi ( split('&',$query_cgi) ) {
473     next unless $this_cgi;
474     $this_cgi =~ m/(.?)=(.*)/;
475     my $input_name = $1;
476     my $input_value = $2;
477     push @query_inputs, { input_name => $input_name, input_value =>
$input_value };
478         if ($input_name eq 'idx') {
479         $scan_index_to_use = $input_value; # unless $scan_index_to_use;
480         }
481 }
482 $template->param ( QUERY_INPUTS => \@query_inputs,
483                    scan_index_to_use => $scan_index_to_use );

.? is non-greedy and matches the minimum number of characters, which is the "x"
part of "idx", resulting in the behaviour described earlier. A patch proposing
to change that line to this: 

$this_cgi =~ m/(.*)=(.*)/;

is coming up...

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to