Update and fix place_simple_hold()
authorDan Wells <dbw2@calvin.edu>
Tue, 12 Mar 2013 14:52:55 +0000 (10:52 -0400)
committerDan Wells <dbw2@calvin.edu>
Tue, 12 Mar 2013 14:52:55 +0000 (10:52 -0400)
commit7e2d64328b1ad7ea212f1cbbc86fc23d60907c2c
tree073e7218c519348533a83fa6934a697188f7cd72
parent877ae8956ac7a64f9ed951ac602c1dfbcf92bed6
Update and fix place_simple_hold()

This commit makes the following changes to the place_simple_hold()
subroutine:
 - clarify that the patron parameter is an ID
 - use a more typical default location for the oils_header.pl file
 - use a Fieldmapper object rather than raw JSON to construct the
hold request
 - add freeze/thaw to hold process to prevent custom status issues
 - add comments where customization is likely needed

We should probably revisit the freeze/thaw process in the future,
but this at least gets us to a working state.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
iNCIPit.cgi