Add tests for invalid patron and invalid item

Test for a blocked patron attempting to check out a book

Fix ILS::checkout to handle more error conditions properly (but I
still can't cope with Bob attempting to check out a book that
Alice already has out.

Test renewing an item that's identified by title rather than by
item id.

Add code to renew items that are specified by title, rather than
by item id.  This is very odd, but the spec seems to allow it.

Define an empty array of fields to be checked in the responses
for those messages that don't require any field checking.  This
eliminates "TODO" failures from the test harness.

Test that when a second hold is placed on an item, the queue
position is properly reported.

Add second patron to stub ILS, to test multiple holds on an
item.

Generalize configuration of Net::Server::PreFork to allow for
arbitrary configuration parameters.

Clean up after the test is completed by checking the item back
in.

Minor cleanup: be consistent in use of "LOG_" in syslog levels.

Initial tests for Item Information message

Extend reporting of circulation status

Initial test for Renew All

text fields in the response messages may be empty

Add support for the configuration parameters

    <min_servers value="N" />
    <max_servers value="N" />

which parallel the paramters for Net::Server::PreFork.

write_msg() is used by both the server and the test suite.  In
the test suite, we don't want to call syslog, so only call syslog
if we're writing the message to stdout (which only happens in the
server).

Fix check renewal permissions check

Initial set of tests for Renew

Fix minor bugs in error condition for Renew Response

Initial test suite for Checkin

Confirm the actual title of the item being held

Only attempt to add the Patron ID to the Checkin Response message
if the ILS actually provides a valid $patron structure

Add a bunch more tests

minor fixups for hold management

Hold messages can have a patron password in them

Initial test case

Add more tests

I'm beginning to wonder how many times it'll take me to get the
password check right.  But this time I actually ran it through
the test harness before checking it in.

add missing value to debug message

Get the test right for checking the patron password in
handle_patron_enable()

Initial set of tests

Fix bugs in Patron Enable: A valid patron with an invalid
password should generate a true patron status message, but
shouldn't enable the patron.

Fix language handling in Block Patron: return the patron's
language, rather than sending 'unknown'.

Initial set of tests for Checkout Response

Eliminate warning message for uninitialized variable use

Fix spelling mistake

Fix handling of the "renewal ok" status field in the ILS

Finish first pass at ILS::Patron documentation

Initial, incomplete, documentation for ILS::Patron

Missed an unused function in the last code cleanup

Code cleanups

Initial pass at documentation for ILS::Item

Delete unused access method

Add code to build HTML documentation from the podfiles.

Finish first pass at documentation for ILS module

Minor code cleanups

Adjust tests to agree with the code.  Since the server reports
that a patron is valid, but the password was wrong, we need to
make sure that that's what the tests expect.

Sequence numbers are a single digit, so when it hits 10, wrap
around to 0.

Adjust the test data so that it works with the test cases.

If the patron is valid, but the patron password is provided, and
wrong, still report that the patron's good.  This seems to be
what clients are expecting, and since we can "trust" them, we
will give them full disclosure.

'0' is a valid sequence number (it comes after '9'), so we have
to see if the sequence number is set, not just if it's true.

minor cleanups

Add a bunch of tests, and stock field checks

Make sure to set BL: Patron Valid when the patron is, in fact,
valid (uncovered by the tests)

Fix formatting, export FID_PATRON_BIRTHDATE FID_PATRON_CLASS

Add support for FID_PATRON_BIRTHDATE (PB) and FID_PATRON_CLASS
(PC) to Patron Information Response, for Envisionware terminals.

Add stub methods sip_birthdate and ptype, to support Envisionware
terminals (and probably others).

Change port for raw socket connection from 5300 (chosen at random
by your's truly) to 6001, as per Envisionware email.

Very basic tests.  This will be a big test case.  I suspect that
the test harness will have to be enhanced in a variety of ways,
including some way of indicating fields that SHOULD NOT appear,
as well as just fields that are required.

Generalize handling of tagged fields, so we can reuse field
specifications between test scenarios.

Fix Patron Information message handling to parallel the situation
in the handling of Patron Status messages: if the password's not
provided by the terminal, then we don't worry about checking the
password, but if the terminal gives us the password, then it has
to match before we say anything about the patron.

Replace call to nonexistent method 'hold_items_count' with
correct code.

Return an item title, even if the checkout failed (assuming that
the item ID was valid).

Clean up handling of Patron Status Response messages when there's
no password provided by the terminal.  I have a strong suspicion
that there exist terminals that expect to get patron status or
patron information even if they don't provide a password.

Create code to test tagged field values, create tests for Patron
Status message

Initial test environment

Clean up handling of SIP boolean flags, and fix interface to
write_msg().  Since it never used the $server parameter, get rid
of it, and add an optional $file parameter, so that write_msg()
can be use by the test harness.

Remove hold_items_count method, and move the code to drop a hold
into a Patron method, rather than embedding it in the ils
transaction level.

Implement code to handle Renew All messages.

Minor cleanups.

Move add_count() into Sip.pm

Consistently use methods to access fields in Transaction status
objects.

Minor cleanups.

Minor cleanups.

move available() method out of Transaction and into Item.

add messages for Renew, Renew All, and Patron Enable

Move add_count() to Sip.pm from Sip::MsgType.pm

Implement stubs necessary for Renew All support.

Consistently use methods to access fields rather than getting
at the fields directly.

Switch to AUTOLOAD-based field access

Stub code to handle renewals

Add stub code necessary to support item renewal

Add support for Renew message

Change IP on service so that it will work regardless of the IP of
the host.

Stub code to handle status returned by a "hold" transaction.

Clean up some test messages so they work against the existing
test stub.  Add messages for placing and cancelling a hold.

Add support for Hold message

Add support for placing, altering, and cancelling holds

Use the new $patron->block() method.  Implement support for the
Patron Enable message

Move block() from the ILS into the ILS::Patron.  Create the
opposite enable() method to support the Patron Enable message

Get handling of institution IDs right.  Make "block_patron" a
method of the ILS::Patron object, rather than an ILS method

Add support for Status Update message

Add stub to handle Status Update message

use the new ILS::check_inst_id to verify the institution ID that
we receive from the terminal.

Also implement support for Item Information message.

Added stub routines that provide support for the Item Information
Response message.

Sip::timestamp now takes an optional parameter (seconds since the
epoch) and generates a timestamp for that time.  Default
continues to be timestamp of 'now'.

new ILS takes an institution configuration as input, so the name
of the ILS institution is the $institution->{id}

Added code to verify the institution id, and to manage due dates
within the stub item db.

Delete handling of "FID_FEE_ITEMS", since it doesn't exist (the
Patron Information Response message documentation is wrong).
Correct handling of Address, Email, and Home Phone fields in
Patron Information Response in response to corrections from 3M

Delete non-existent "FID_FEE_ITEMS", since we heard back from 3M
that the documentation about the Patron Information Response
message was incorrect.

Don't use "ILS::pay_fee()", use "$ils->pay_fee()"

Change label for tag generation

More documentation

Initial documentation for the ILS.pm portable interface between
the SIP code and the ILS.

adding loadable module support