[open-ils-commits] r18937 - in branches/rel_2_0/Open-ILS: src/perlmods/OpenILS/Application src/perlmods/OpenILS/Application/Circ web/opac/skin/default/js (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2010-12-08 16:57:12 -0500 (Wed, 08 Dec 2010)
New Revision: 18937

Modified:
   branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Circ/Holds.pm
   branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Serial.pm
   branches/rel_2_0/Open-ILS/web/opac/skin/default/js/holds.js
Log:
Backport r18800 from trunk (I thought I already had! r18932 depends on it)

Place holds on issuances from the OPAC. Also a couple very minor API
documentation fixes.



Modified: branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Circ/Holds.pm
===================================================================
--- branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Circ/Holds.pm	2010-12-08 20:00:08 UTC (rev 18936)
+++ branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Circ/Holds.pm	2010-12-08 21:57:12 UTC (rev 18937)
@@ -94,10 +94,11 @@
                 'exception is returned',
         params => [
             { desc => 'Authentication token',               type => 'string' },
-            { desc => 'Hold object for hold to be created', type => 'object' }
+            { desc => 'Hold object for hold to be created',
+                type => 'object', class => 'ahr' }
         ],
         return => {
-            desc => 'Undef on success, -1 on missing arg, event (or ref to array of events) on error(s)',
+            desc => 'New ahr ID on success, -1 on missing arg, event (or ref to array of events) on error(s)',
         },
     }
 );
@@ -110,11 +111,14 @@
         desc  => "If the recipient is not allowed to receive the requested hold, " .
                  "call this method to attempt the override",
         params => [
-           { desc => 'Authentication token',               type => 'string' },
-           { desc => 'Hold object for hold to be created', type => 'object' }
+            { desc => 'Authentication token',               type => 'string' },
+            {
+                desc => 'Hold object for hold to be created',
+                type => 'object', class => 'ahr'
+            }
         ],
         return => {
-            desc => 'Undef on success, -1 on missing arg, event (or ref to array of events) on error(s)',
+            desc => 'New hold (ahr) ID on success, -1 on missing arg, event (or ref to array of events) on error(s)',
         },
     }
 );

Modified: branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Serial.pm
===================================================================
--- branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Serial.pm	2010-12-08 20:00:08 UTC (rev 18936)
+++ branches/rel_2_0/Open-ILS/src/perlmods/OpenILS/Application/Serial.pm	2010-12-08 21:57:12 UTC (rev 18937)
@@ -450,7 +450,7 @@
     global := If true, return only one representative version of a conceptual issuance regardless of the number of subscriptions, otherwise return all issuance objects meeting the requested criteria, including conceptual duplicates. Valid values are 0 (false) and 1 (true, default).
     order  := date_published sort direction, either "asc" (chronological, default) or "desc" (reverse chronological)
     limit  := Number of issuances to return.  Useful for paging results, or finding the oldest or newest
-    offest := Number of issuance to skip before returning results.  Useful for paging.
+    offset := Number of issuance to skip before returning results.  Useful for paging.
     orgid  := OU id used to scope retrieval, based on distribution.holding_lib
     depth  := OU depth used to range the scope of orgid
     type   := Holding type filter. Valid values are "basic", "supplement" and "index". Can be a scalar (one) or arrayref (one or more).

Modified: branches/rel_2_0/Open-ILS/web/opac/skin/default/js/holds.js
===================================================================
--- branches/rel_2_0/Open-ILS/web/opac/skin/default/js/holds.js	2010-12-08 20:00:08 UTC (rev 18936)
+++ branches/rel_2_0/Open-ILS/web/opac/skin/default/js/holds.js	2010-12-08 21:57:12 UTC (rev 18937)
@@ -589,7 +589,7 @@
 
 function holdsSetFormatSelector() {
 	var type = holdArgs.type;
-	if( type == 'C' || type == 'V' || holdArgs.editHold ) return;
+	if( type == 'C' || type == 'V' || type == "I" || holdArgs.editHold ) return;
 
 	var data				= holdsGetFormats();
 	var avail_formats	= data.avail_formats;
@@ -729,6 +729,7 @@
 		titleid : holdArgs.record,
 		mrid : holdArgs.metarecord,
 		volume_id : holdArgs.volume,
+		issuanceid : holdArgs.issuance,
 		copy_id : holdArgs.copy,
 		hold_type : holdArgs.type,
 		patronid : holdArgs.recipient.id(),