Nicer error message if a branch is not set
authorChris Cormack <chrisc@catalyst.net.nz>
Thu, 3 Sep 2009 01:16:34 +0000 (13:16 +1200)
committerGalen Charlton <gmcharlt@gmail.com>
Sat, 5 Sep 2009 23:56:22 +0000 (19:56 -0400)
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

labels/label-edit-batch.pl

index 1bcadb3..93cd07b 100755 (executable)
@@ -80,10 +80,16 @@ elsif ($op eq 'delete') {
 elsif ($op eq 'add') {
     $batch = C4::Labels::Batch->retrieve(batch_id => $batch_id);
     $batch = C4::Labels::Batch->new(branch_code => $branch_code) if $batch == -2;
-    foreach my $item_number (@item_numbers) {
-        $err = $batch->add_item($item_number);
+    if ($branch_code){
+        foreach my $item_number (@item_numbers) {
+            $err = $batch->add_item($item_number);
+        }
+        $errstr = "item(s) not added to batch $batch_id." if $err;
+    }
+    else {
+        $err = 1;
+        $errstr = "items(s) not added, the error was: Branch is not set, you please set your branch before adding items to a batch";
     }
-    $errstr = "item(s) not added to batch $batch_id." if $err;
 }
 elsif ($op eq 'new') {
     $batch = C4::Labels::Batch->new(branch_code => $branch_code);