Browse Source

Bug 6679 - [SIGNED-OFF] fix 2 perlcritic violations in C4/BackgroundJob.pm

"return" statement with explicit "undef" at line 234, column 5.  See page 199 of PBP.  (Severity: 5)

"return" statement with explicit "undef" at line 256, column 9.  See page 199 of PBP.  (Severity: 5)

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
3.10.x
Mason James 12 years ago
committed by Paul Poulain
parent
commit
642779f217
  1. 4
      C4/BackgroundJob.pm

4
C4/BackgroundJob.pm

@ -231,7 +231,7 @@ if the job status is not 'completed'.
sub results {
my $self = shift;
return undef unless $self->{'status'} eq 'completed';
return unless $self->{'status'} eq 'completed';
return $self->{'results'};
}
@ -253,7 +253,7 @@ sub fetch {
my $session = get_session($sessionID);
my $prefix = "job_$jobID";
unless (defined $session->param($prefix)) {
return undef;
return;
}
my $self = $session->param($prefix);
bless $self, $class;

Loading…
Cancel
Save