Handle invalid agencies in NCIP::ILS::Evergreen.

We modify NCIP::ILS::Evergreen->find_location_failover to return an
undefined value in the event of failure to lookup an agency.

We then modify the requestitem and cancelrequestitem handlers to
return a Problem message if the lookup fails for their selection
org. units.  These appear to be the only two places possilby
affected by a failure to find a location.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

"Fix" templates/includes/problem.inc.

During testing I noticed that the Problem messages had an extra
newline in the ProblemType field.  This apparently causes problems
for Auto-Graphics' parser, so this commit removes it.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Add error handling to NCIP::ILS::Evergreen->cancelrequestitem.

We did not check the return value of open-ils.circ.hold.cancel, because
why would it fail?  Turns out there are a couple of reasons it could
fail, including not setting up permissions for the staff user correctly.

The cancel_hold helper function now returns the return value of the
open-ils.circ.hold.cancel method.  This method returns 1 on succes or
a reference ot an event hash on failure.

The cancelrequestitem method now returns a Problem response message
of Temporary Processing Failure if cancel_hold returns a ref.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Some defensive coding in NCIP::ILS.pm.

XML::LibXML::Simple will return {} for an empty field.  We need to
check this on barcode input, at least, and undef the variable.  This
is done in the base class find_user_barcode and find_item_barcode
methods.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in NCIP::ILS::Evergreen->lookupuser().

I had IdnetifierValue instead of IdentifierValue.  Is that something
I fixed in place during testing and forgot to fix in the repo?

Signed-off-by: Jason Stephenson <jason@sigio.com>

A little more defensive programming in NCIP::ILS::Evergreen.

If for some reason actor.usr.card does not have the id of a valid
barcode and we get nothing from it, we use the barcode that we
were given in the LookupUser request.

Signed-off-by: Jason Stephenson <jason@sigio.com>

A little defensive programming parsing oils_ncip.xml.

If one leaves the <patrons> block alone, one gets the following Perl
error:

Can't use string as a HASH ref while "strict refs" in use at
/home/opensrf/NCIPServer/lib/NCIP/ILS/Evergreen.pm line 2626.

This commit attempts to avoid that.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Move oils_ncip.xml.example to examples/.

I realized that NCIPServer would possibly read oils_ncip.xml.example
if it were left in t/config_sample given that we use the Config::Merge
module.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add oils_ncip.xml.example in t/config_sample.

This is the example configuration file needed for Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Reimplement requestitem handler in NCIP::ILS::Evergreen.

Testing with Auto-Graphics suggests that our original logic was too
complicated, so we simplify it to only handle what Auto-Graphics
sends us.  We may want to implement a more generic handler in the
future when NCIPServer is used with more vendors.  However, it seems
fairly typical for responders to also be vendor specific.

While simplifying the logic, this commit gets rid of some unneeded
funtions.

We also fix a typo in the RequestItemResponse.inc template.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix an oops in NCIP::ILS::Evergreen.

Forgot the service name on a simplereq call around line 800.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Changes to NCIP::ILS::Evergreen as a result of testing.

These modifications are based on testing with Auto-Graphics and
reflect how they say they will be sending data in production. They
will return the ItemId that we are now returning in the response
to RequestItem so we can use that to find the appropriate hold to
cancel.

The above necessitated some internal changes to the code that
amounted to a simplification of placing holds (in some cases) as
well as a simplification in canceling holds. There is still some
cruft hanging around that should probably go away, but I think
we're safe for now.

We've also update some templates to fix issues while testing the
above mentioned changes, specifically RequestItemResponse.inc and
CancelRequestItemResponse.inc.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Modify NCIP::ILS::Evergreen->requestitem.

We now return AgencyId in the ItemId response field when returning a
database id and not barcode. If we can't figure out the AgencyId to
return, we set the ItemId/ItemIdentifierType to SYSNUMBER.

There's room for improvement here, but no time right now.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Receive or abort transit in NCIP::ILS::Evergreen->checkoutitem.

If the copy is in transit to or from the working organizational
unit of the NCIP staff user, then we have to either receive or
abort the transit before the check out can proceed.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Change CheckOutItemResponse.inc for whitespace.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix another template typo.

That's what happens when you make big changes in testing.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in UserId.in template include.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add ItemId to RequestItem output in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fixup RequestItemResponse template include file.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Return only 1 barcode for UserId in NCIP::ILS::Evergreen->lookupuser.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Alter the template includes so that fields are in xsd order.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Return the RequestId in the RequestItem message.

Signed-off-by: Jason Stephenson <jason@sigio.com>

"Fix" ItemOptionalFields.inc.

Because Auto-Graphics has problems with "carriage returns."

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix another "typo/thinko" in NCIP::ILS::Evergreen.

Missed the 't' on an aout retrieve and ended pulling out an ou,
instead.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a mistake in NCIP::ILS:Evergreen->find_target_via_bibliographic_id

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a typo on line 2790 of NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Modify NCIP::ILS::Evergreen based on communication and testing.

Auto-Graphics is sending our target branch code in the ToAgencyId
field of the InitiationHeader.  They are not sending some of the
optional fields that we are looking for.  Most of the changes in
this commit revolve around using the ToAgencyId as a fallback
when the other fields are not available.  The focus of these
changes are the AcceptItem and RequestItem messages since the
local location is most important there.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix an inversion of author and title in NCIP::ILS::Evergreen->acceptitem.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Another template change that was missed in the previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Templates changes for Auto-graphics.

They requested that we remove "carriage returns" from the output. I'm not
100% certain if he meant all carriage returns or if he just wanted the
weird spacing that some elements had where the open tag is on one line
followed by the value and the close tag on the next line. I don't really
know exactly why that was happening, but it seemed to have something to
do with trying to make the value come out on a line by its own between
the open and close tags. This change puts those elements on one line.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Try setting the configuration directory from environment again.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Get NCIP configuration directory from the environment.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Set application/xml content type in NCIP::Dancing.

I tried setting the default in config.yml, and it did not work.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add output of phone numbers in NCIP::ILS::Evergreen->lookupuser.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add NCIP::PhysicalAddress object.

I started to add this to the code last night to make addresses even
more abstract and extensible. Instead, I took a short cut. In the
light of the new day, I decided to go back and add this class as the
right thing to do.

We modify NCIP::ILS::Evergreen, NCIP::User::AddressInformation, and
the include for UserOptionalFields to make use of the new object.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Get rid of the namespace prefix in templates and includes.

Signed-off-by: Jason Stephenson <jason@sigio.com>

More fixes for includes/LookupUserResponse.inc.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fixup a lot of stuff based on a first round of real testing.

Add ncip: namespace qualifier in templates and includes.

Reorder some fields because apparently the field order matters,
at least to xmllint when using the schema.

Make some code changes in NCIP::ILS::Evergreen to make sure that
date fields have the proper format. (This still needs testing, so
we'll see.)

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix response.tt and includes/data.inc issue.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a case issue in NCIP::Const::AUTHENTICATIONINPUT_MESSAGES.

We had LookUpUser and it should have been LookupUser.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a problem with acceptitem in NCIP::ILS::Evergreen.

There was a logical error when checking RequestedActionType for
the "Hold For Pickup" action.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Do for renewitem what was done for checkoutitem in previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix checkoutitem in NCIP::ILS::Evergreen.

Even though, the function documentation for checkout(.full) says
a copy object works, it apparently doesn't, so I switched to using
the copy_id parameter with $copy->id().

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix another typo in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix something found by perl -c from the previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add find_copy_details_by_item to NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

More Doh! type things in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Get the pcrud method name correct.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix NCIP::ILS::Evergreen->place_hold.

We better take into account what open-ils.circ.holds.create returns.

We check for an array of errors and use the first one.

We also look up the hold for the id returned. I forgot that it only
returns the id and not the whole hold object created.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix some warnings in NCIP::ILS::Evergreen.

Try to fix thes:

Use of uninitialized value in string eq at
/home/opensrf/NCIPServer/lib/NCIP/ILS/Evergreen.pm line 2553.
Use of uninitialized value in string eq at
/home/opensrf/NCIPServer/lib/NCIP/ILS/Evergreen.pm line 2589.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo on line 2748 of NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix some more stuff in NCIP::ILS::Evergreen.

Running with perl -c caught another syntax error, and I saw that
I need hashrefs in a few places where I missed the {}.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix some things caught by perl -c in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add handling of ItemElementType in NIP::ILS::Evergreen.

This includes some template changes to output the ItemOptionalFields
element data.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo on line 2380 of NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Last fix just gave us a different warning.

Try to fix it for real this time.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Try to fix a warning when checking penalties.

Use of uninitialized value in pattern match (m//) at
/home/opensrf/NCIPServer/lib/NCIP/ILS/Evergreen.pm line 1595.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in function name in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix copy and paste problem with previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add UserOptionalFields to more templates.

Also add checks for it in more messages for NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add NCIP::ILS::Evergreen->handle_user_elements for flexibility.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add handling of NeedBeforeDate to NCIP::ILS::Evergreen->requestitem.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix problems with _problem_from_event in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix mistake in NCIP::ILS::Evergreen on line 924.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Alter NCIP::ILS::Evergreen->_hold_search() to do just one thing.

Remove the code that had it return a problem if no holds were
found and make it the caller's responsibility to check if a hold
was returned or nothing.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typos in NCIP::ILS::Evergreen caught by perl -c.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add cancelrequestitem handler to NCIP::ILS::Evergreen.

Also add the CancelRequestItemResponse.inc to the templates.

Signed-off-by: Jason Stephenson <jason@sigio.com>

More NCIP::ILS::Evergreen->requestitem() improvements.

We check if item is a NCIP::Problem and return it if it is.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a comment typo in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Modify NCIP::ILS::Evergreen->requestitem to try a little harder.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Simplify NCIP::ILS::Evergreen->retrieve_org_unit_by_shortname().

Use open-ils.actor.org_unit.retrieve_by_shortname instead of a
pcrud search.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a little something caught by perl -c.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Alter find_bibliograpic_id to find_biblioraphic_ids in NCIP::ILS.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add duplicate request check to NCIP::ILS::Evergreen->requestitem

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix a typo in previous commit.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add RequestItem support to NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in previous commit: $enty for $entry.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add $code option to NCIP::ILS->find_bibliographic_id.

I wonder now, if I should not have overriden the method in the
Evergreen driver, but I thought this would be generically useful.

We will probably want a method that can return all of the IDs at
some point, but for now, any subclasses that need that, can
implement it, themselves.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Check for repeated BibliographId in NCIP::ILS->find_bibliographic_id.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in find_bibliographic_id in NCIP::ILS.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Improve find_bibliographic_id in NCIP::ILS.

Add a check for BibligraphicDescription under ItemOptionalFields.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add find_bibliographic_id to NCIP::ILS.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Improve comments in NCIP::ILS.

Add an array example to find_user_barcode and find_item_barcode.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Update the CheckOutItemResponse.inc template file.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add checkoutitem to NCIP::ILS::Evergreen.

Also add a couple of utility methods to check if a copy is allowed
to circulate and fill holds, as well as one to retrieve a copy
location object by id.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Simplify delete_copy in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Bump _problem_from_event's IQ by a couple of points.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add some "smarts" to _problem_from_event in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix text_code for textcode typo in NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix set_timezone to set_time_zone in NCIP::ILS::Evergreen->renewitem.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add AuthenticationInput checks for more messages in NCIP::ILS.

We add the AUTHENTICATIONINPUT_MESSAGES constant to hold a list of
messages that may have AuthenticationInput set, and then we check
if our message is in the list when trying to find a user barcode
and the UserId field was not present.

Signed-off-by: Jason Stephenson <jason@sigio.com>

UserId does not repeat if present in a message.

Alter NCIP::ILS->find_user_barcode() to not make an array ref out
of UserId inputs. It is not supposed to repeat.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add error checking to NCIP::ILS::Evergreen->checkinitem().

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add RenewItem support for NCIP::ILS::Evergreen.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix typo in NCIP::ILS::Evergreen->checkinitem().

Signed-off-by: Jason Stephenson <jason@sigio.com>

Fix some things in NCIP::ILS::Evergreen picked up by perl -c.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Move circulation, copy and user checks to check_circ_details method.

Before implementing renewitem in NCIP::ILS::Evergreen, it became
apparent that we'd want to make the same checks for the copy being
checked out, owned by or checked out at our working ou, and being
checked out to the provided user, if any, before proceeding with
renewal. It makes sense therefore to move the checks from being
inside checkinitem to being its own function that returns a
NCIP::Problem or undef.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add check for more copies in NCIP::ILS::Evergreen->delete_copy.

We now check if there are other copies on the call number before
we delete it.

We also fix a thinko when we were checking for additional call
numbers on the bib. We had map where we needed grep.

Signed-off-by: Jason Stephenson <jason@sigio.com>