Bug 19200: (QA follow-up) Simplify call to GetBasketAsCSV
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 22 Dec 2017 10:20:13 +0000 (11:20 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 22 Dec 2017 16:15:35 +0000 (13:15 -0300)
If no profile_id is passed, GetBasketAsCSV will fallback to default itself.
No need to make the distinction in basket.pl.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

acqui/basket.pl

index 8df5f8b..92fae60 100755 (executable)
@@ -171,13 +171,7 @@ if ( $op eq 'delete_confirm' ) {
         -attachment => 'basket' . $basket->{'basketno'} . '.csv',
     );
     my $csv_profile_id = $query->param('csv_profile');
-    if (defined $csv_profile_id) {
-        # using a csv profile
-        print GetBasketAsCSV(scalar $query->param('basketno'), $query, $csv_profile_id);
-    } else {
-        # using default
-        print GetBasketAsCSV(scalar $query->param('basketno'), $query);
-    }
+    print GetBasketAsCSV( scalar $query->param('basketno'), $query, $csv_profile_id ); # if no csv_profile_id passed, using default rows
     exit;
 } elsif ($op eq 'email') {
     my $err = eval {