8 years agoAvoid referencing nonexistent AgencyId vars
Jeff Godin [Tue, 30 Jul 2013 17:00:08 +0000 (13:00 -0400)]
Avoid referencing nonexistent AgencyId vars

In fail() and do_user_lookup_error_stanza(), we were attempting to
call staff_log() with values for FromAgencyId and ToAgencyId using
variables which did not exist.

Also, in fail() we were attempting to use those same variables in
the creation of an XML NCIP response.

The above issues would result in "Use of uninitialized value"
warnings, as well preventing use of "use strict".

This commit removes the references to the nonexistent variables,
resulting in the same log and xml output.

A future commit may provide a means for these functions to log the
FromAgencyId and ToAgencyId.

Signed-off-by: Jeff Godin <>
8 years agoRemove duplicate request detection
Jeff Godin [Tue, 30 Jul 2013 15:04:53 +0000 (11:04 -0400)]
Remove duplicate request detection

Per MeLCat staff 2013-07-26, duplicate request detection is no
longer needed, and can cause its own problems. It should be removed.

This commit removes the duplicate NCIP message detection.

Signed-off-by: Jeff Godin <>
8 years agoMerge pull request #3 from Dyrcona/last_post
Hekman Library Technology Team [Mon, 25 Mar 2013 14:26:25 +0000 (07:26 -0700)]
Merge pull request #3 from Dyrcona/last_post

A little defensive coding around last_post.txt.

8 years agoA little defensive coding around last_post.txt.
Jason Stephenson [Thu, 21 Mar 2013 13:24:30 +0000 (09:24 -0400)]
A little defensive coding around last_post.txt.

If the file doesn't exist, rather than die, try to create it.  Only
die if we fail to create it.

Signed-off-by: Jason Stephenson <>
8 years agoMore customization markers, and general cleanup
Dan Wells [Tue, 12 Mar 2013 15:17:12 +0000 (11:17 -0400)]
More customization markers, and general cleanup

First, add more CUSTOMIZATION markers wherever a value likely needs
per-system configuration.

Second, clean up and clarify the code in a few places.  This includes
standardizing some variable names.  In particular, $visid and
$barcode were both used interchangeably, and this commit standardizes
on $visid for anything which isn't an actual local item barcode (i.e.
it is a fake or external system "barcode" value).

Signed-off-by: Dan Wells <>
8 years agoRemove force-to-available code from check_in_item
Dan Wells [Tue, 12 Mar 2013 15:12:14 +0000 (11:12 -0400)]
Remove force-to-available code from check_in_item

check_in_item() currently tries to force the checked-in copy to the
"Available" status.  This seems unnecessary, and possibly damaging.

Instead, let the EG check in process set the status as it sees fit
(most often to "Reshelving").

Signed-off-by: Dan Wells <>
8 years agoClarify item_request(), add missing $U
Dan Wells [Tue, 12 Mar 2013 15:07:48 +0000 (11:07 -0400)]
Clarify item_request(), add missing $U

First, we clarify the item_request() sub by:
 - adding customization notes
 - making sure we retain our custom status after hold placement
 - identifying Item vs User Agency code branches

Second, we add the common $U AppUtils variable which was missing from
the last commit.

Signed-off-by: Dan Wells <>
8 years agoUpdate and fix place_simple_hold()
Dan Wells [Tue, 12 Mar 2013 14:52:55 +0000 (10:52 -0400)]
Update and fix place_simple_hold()

This commit makes the following changes to the place_simple_hold()
 - clarify that the patron parameter is an ID
 - use a more typical default location for the 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 <>
8 years agoComment out incomplete, unused subs
Dan Wells [Tue, 12 Mar 2013 14:30:11 +0000 (10:30 -0400)]
Comment out incomplete, unused subs

Signed-off-by: Dan Wells <>
8 years agoWhitespace revision, take II
Dan Wells [Mon, 11 Mar 2013 22:11:42 +0000 (18:11 -0400)]
Whitespace revision, take II

perltidy can result in code formatting which is a little extreme
when left at the 80-characters-wide default, so rejoin the split
$doc->findvalue() calls which were negatively affecting

Also, Evergreen code standards is for using four spaces, not tabs,
so correct for that small mixup as well.

Signed-off-by: Dan Wells <>
8 years agocleaning up service detection block by way of including code from atz fork
Jon Scott [Mon, 11 Mar 2013 20:22:37 +0000 (13:22 -0700)]
cleaning up service detection block by way of including code from atz fork

8 years agoattempt to clean up whitespace and get standardized with eg code formatting asap :
Jon Scott [Mon, 11 Mar 2013 19:57:04 +0000 (12:57 -0700)]
attempt to clean up whitespace and get standardized with eg code formatting asap :
perltidy cuddled else and 's/ {4}/\t/g' version

8 years agoadded comments for removing local customizations / reading ini file going forward
Jon Scott [Mon, 11 Mar 2013 19:10:47 +0000 (12:10 -0700)]
added comments for removing local customizations / reading ini file going forward

9 years agoAdded NCIPmsgs : ItemCheckedIn, RenewItem, ItemShipped, ItemRequested, ItemRequestCan...
Jon Scott [Wed, 12 Sep 2012 00:25:02 +0000 (17:25 -0700)]
Added NCIPmsgs : ItemCheckedIn, RenewItem, ItemShipped, ItemRequested, ItemRequestCancelled, ItemRenewed, ItemReceived, ItemCheckedOut, CheckOutItem, CheckInItem and AcceptItem

9 years agolatest version
Jon Scott [Tue, 11 Sep 2012 21:06:29 +0000 (14:06 -0700)]
latest version

9 years agoUpdate README
Jon Scott [Fri, 11 May 2012 21:10:17 +0000 (14:10 -0700)]

9 years agowanted the ncip ItemRenewed to be able to override std.limits on renewals so ...
Jon Scott [Fri, 11 May 2012 21:05:05 +0000 (14:05 -0700)]
wanted the ncip ItemRenewed to be able to override std.limits on renewals so ...

9 years agoUpdate README
Jon Scott [Fri, 11 May 2012 14:14:26 +0000 (07:14 -0700)]

9 years agoStanding Penalties Added - Want to return blocking info to NCIP request
Jon Scott [Fri, 11 May 2012 13:55:07 +0000 (06:55 -0700)]
Standing Penalties Added - Want to return blocking info to NCIP request

9 years agoNCIPmsgs/LookupUser.ncip demo added
Jon Scott [Thu, 26 Apr 2012 19:29:04 +0000 (12:29 -0700)]
NCIPmsgs/LookupUser.ncip demo added

9 years agoUpdate iNCIPit.cgi
Jon Scott [Thu, 26 Apr 2012 17:31:28 +0000 (10:31 -0700)]
Update iNCIPit.cgi

9 years agoUpdate README
Jon Scott [Thu, 26 Apr 2012 17:22:20 +0000 (10:22 -0700)]

9 years agoUpdate README
Jon Scott [Thu, 26 Apr 2012 14:37:29 +0000 (07:37 -0700)]

9 years agoUpdate README
Jon Scott [Thu, 26 Apr 2012 12:50:52 +0000 (05:50 -0700)]

9 years agoUpdate README
Wolf Halton [Thu, 26 Apr 2012 12:44:50 +0000 (05:44 -0700)]

9 years agofirst commit
Jon Scott [Wed, 25 Apr 2012 14:22:45 +0000 (07:22 -0700)]
first commit