# [PATCH (bugfix)] gitweb: Fix passing parameters to git_project_search_form

2 messages from 2012-03-02 to 2012-03-02. Participants: Jakub Narebski, Junio C Hamano.
Thread: https://gitlist.dev/t/29817

## Jakub Narebski, 2012-03-02 22:50

Subject: [PATCH (bugfix)] gitweb: Fix passing parameters to git_project_search_form
Message-ID: <1330728601-27124-1-git-send-email-jnareb@gmail.com>
URL: https://gitlist.dev/e/1330728601-27124-1-git-send-email-jnareb%40gmail.com

```
The git_project_search_form() subroutine, introduced in a1e1b2d
(gitweb: improve usability of projects search form, 2012-01-31) didn't
get its arguments from caller correctly.  Gitweb worked correctly
thanks to sticky-ness of form fields in CGI.pm... but it make UTF-8
fix for project search not working.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
---
Brown paper bug (a1e1b2d is in master).  I am extremly sorry about that.

 gitweb/gitweb.perl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 7398be1..e2e6a73 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -5263,7 +5263,7 @@ sub git_patchset_body {
 # . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .
 
 sub git_project_search_form {
-	my ($searchtext, $search_use_regexp);
+	my ($searchtext, $search_use_regexp) = @_;
 
 	my $limit = '';
 	if ($project_filter) {
-- 
1.7.9

```

## Junio C Hamano, 2012-03-02 23:55

Subject: Re: [PATCH (bugfix)] gitweb: Fix passing parameters to git_project_search_form
Message-ID: <7v1upablsl.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7v1upablsl.fsf%40alter.siamese.dyndns.org
In-Reply-To: <1330728601-27124-1-git-send-email-jnareb@gmail.com>

```
Jakub Narebski <jnareb@gmail.com> writes:

> The git_project_search_form() subroutine, introduced in a1e1b2d
> (gitweb: improve usability of projects search form, 2012-01-31) didn't
> get its arguments from caller correctly.  Gitweb worked correctly
> thanks to sticky-ness of form fields in CGI.pm... but it make UTF-8
> fix for project search not working.
>
> Signed-off-by: Jakub Narebski <jnareb@gmail.com>
> ---
> Brown paper bug (a1e1b2d is in master).  I am extremly sorry about that.

Heh, mistakes happen.  Thanks for taking care of this.

>
>  gitweb/gitweb.perl |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 7398be1..e2e6a73 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -5263,7 +5263,7 @@ sub git_patchset_body {
>  # . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . .
>  
>  sub git_project_search_form {
> -	my ($searchtext, $search_use_regexp);
> +	my ($searchtext, $search_use_regexp) = @_;
>  
>  	my $limit = '';
>  	if ($project_filter) {

```