From: erickson <erickson@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Fri, 20 Jun 2008 16:10:39 +0000 (+0000)
Subject: exiting early if picklist delete fails.  setting picklist org_unit
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=79c4c2dd0665bdc25f670f0695f00c2c50bff939;p=Evergreen.git

exiting early if picklist delete fails.  setting picklist org_unit

git-svn-id: svn://svn.open-ils.org/ILS/branches/acq-experiment@9894 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Acq/Picklist.pm b/Open-ILS/src/perlmods/OpenILS/Application/Acq/Picklist.pm
index a01253646a..3176133a45 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Acq/Picklist.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Acq/Picklist.pm
@@ -29,6 +29,7 @@ sub create_picklist {
     my($self, $conn, $auth, $picklist) = @_;
     my $e = new_editor(xact=>1, authtoken=>$auth);
     return $e->die_event unless $e->checkauth;
+    $picklist->org_unit($e->requestor->ws_ou) unless $picklist->org_unit;
     return $e->die_event unless $e->allowed('CREATE_PICKLIST', $picklist->org_unit);
     return OpenILS::Event->new('BAD_PARAMS')
         unless $e->requestor->id == $picklist->owner;
@@ -413,7 +414,8 @@ sub zsearch {
     $name ||= '';
     my $picklist = $e->search_acq_picklist({owner=>$e->requestor->id, name=>$name})->[0];
     if($name eq '' and $picklist) {
-        delete_picklist($self, $conn, $auth, $picklist->id);
+        my $evt = delete_picklist($self, $conn, $auth, $picklist->id);
+        return $evt unless $evt == 1;
         $picklist = undef;
     }
 
@@ -421,6 +423,7 @@ sub zsearch {
         $picklist = Fieldmapper::acq::picklist->new;
         $picklist->owner($e->requestor->id);
         $picklist->name($name);
+        $picklist->org_unit($e->requestor->ws_ou);
         $e->create_acq_picklist($picklist) or return $e->die_event;
     }
 
@@ -430,8 +433,8 @@ sub zsearch {
     while(my $resp = $req->recv(timeout=>60)) {
 
         my $result = $resp->content;
-        use Data::Dumper;
-        $logger->info("results = ".Dumper($resp));
+        #use Data::Dumper;
+        #$logger->info("results = ".Dumper($resp));
         my $count = $result->{count};
         my $total = (($count < $search->{limit}) ? $count : $search->{limit})+1;
         my $ctr = 0;