gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 32030: Remove warnings from tests 

Use of uninitialized value $provider in string eq

Signed-off-by: Jonathan Field <jonathan.field@ptfs-europe.com>

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Jonathan Druart 2022-10-26 15:04:40 +02:00 committed by Tomas Cohen Arazi
parent 46a7434fbf
commit ca3f2fb24b
No known key found for this signature in database
GPG Key ID: 0A272EA1B2F3C15F
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Koha/REST/V1/ERM/EHoldings/Packages.pm
View File

@ -71,7 +71,7 @@ sub add {
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Packages::Local::add($c);
@ -88,7 +88,7 @@ sub update {
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Packages::Local::update($c);
@ -103,7 +103,7 @@ sub delete {
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Packages::Local::delete($c);

6
Koha/REST/V1/ERM/EHoldings/Titles.pm
View File

@ -71,7 +71,7 @@ sub add{
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Titles::Local::add($c);
@ -89,7 +89,7 @@ sub update {
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Titles::Local::update($c);
@ -104,7 +104,7 @@ sub delete {
my $c = shift->openapi->valid_input or return;
my $provider = $c->validation->param('provider');
if ( $provider eq 'ebsco' ) {
if ( $provider && $provider eq 'ebsco' ) {
die "invalid action";
} else {
return Koha::REST::V1::ERM::EHoldings::Titles::Local::delete($c);