Menu

#105 grapher.cgi 'view' lc(undef) is always true.

fix-for-1.0.x
open
nobody
grapher (29)
5
2007-03-12
2007-03-01
Anonymous
No

xj@wp.pl

In cricket-1.0.5, line 254:

my($view) = lc $gQ ....
if(defined($view)) ....

so it's always true. My sugest is:

if (defined($gQ->param('view'))){
my($view)=lc $gQ->param('view');
$targRef->{'auto-view'}=$view;
}

Paweł Piątek

Discussion

  • Terje Bless

    Terje Bless - 2007-03-12

    Logged In: YES
    user_id=8470
    Originator: NO

    Updating Summary field to make bug more easily found. cf. dup in Bug #1678793.

     
  • Terje Bless

    Terje Bless - 2007-03-12
    • summary: view --> grapher.cgi 'view' lc(undef) is always true.
     
  • Nobody/Anonymous

    Logged In: NO

    more clean this way:

    my($view) = $gQ->param('view');
    if (defined($view)) {
    $targRef->{'auto-view'} = lc $view;
    }

     

Log in to post a comment.