LP#838525 Store date of birth as SQL DATE type

Upgrade script translates DoB ahead 3 hours before translating to DATE
(from TIMESTAMPTZ) to ensure dates stored in alternate daylight savings
TZ (e.g. -05 vs. -04) are translated to the proper date.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

JBAS-673 Create PO from multiple pages of lineitems.

Maintain a list of selected lieitems across multiple pages of lineitems
in a selection list.  When creating a PO from selected lineitems,
include all selected lineitems in the new PO.

See also KMAIN-1865.

Code based on 5f309573.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1007 (KMAIN-1642) ACQ items navigation improvements

This code is based on the original code at e595e444.

1. When drawing the lineitem copies page, select() the value in the Item
    Count Count text input so staff can more quickly modify the value.

2. After adding copies, focus the first owning lib selector.

3. When an owning lib selector is focused, pressing Enter causes focus
  to jump to the next lib selector in the list.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1829 Bib Call Number on Holds

Problem: Call Number wouldn't show up in list view of
patron holds screen when status was "Waiting for Copy."

Solution: Add check to Call Number column for specific
copy, and if nonexistant, pull from MARC info for selected
item.

Signed-off-by: Kyle Huckins <khuckins@catalystdevworks.com>
---

Modified by berick to avoid bib call number / MARC processing when a
call number is available, since this is already a heavy API call.
Replaced some tabs.  Added some comments.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1830 Backspace clears patron register barcode

Backspace Now Completely Clears Barcode Field When Doing A
Patron Registration.

Signed-off-by: Nick Duncan <nDuncan@catalystitservices.com>
Modified by Bill E. for style, to avoid unnecessary dojo.attach, and
to add some comments.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Log patron purge count

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Kent/future expire dates

1. Set the expire age to 21 instead of 18 for Kent SD.
2. Ensure that no expire dates are set to before "now".

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Log inserted patron count

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 set expire time to 6am to reduce DST issues

Set patron expire time to 6am so that expire dates spanning DST
boundaries don't make it appear as though the date is off by one day for
clients that don't honor timezone data.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Kent classroom cards data

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 student importer improved error handling

Includes various small fixes, but mainly the ability to report when a
commit call fails, which is primarily important when processing in
batch.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Ecard/classroom DO NOT MERGE alert

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1037 Student E-cards quick report SQL

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1129 Remove custom language_filter additions

Custom language filter is unnecessary, since we already have coded value
map for item_lang.  Removing the code allows language and other
CCVM-filtered queries to complete.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-1129 Cross-port patron reg set-expire-date

Copying to KCLS template.

Originally:

commit 6807a06a11d309e94dbc5e286e45a5fdd3eaecab
Author: Jeff Godin <jgodin@tadl.org>
Date:   Wed Oct 30 15:42:15 2013 -0400

    Add button to update expire date field

    When updating/renewing a patron (expired or not), it would be
    convenient to have a means of populating the expire date field with
    a value of perm_interval + today. Currently, staff are required to
    manually enter a date, or could toggle the profile from one profile
    then back to the desired profile, but this is not ideal.

    This commit adds a new button next to the expire date, which when
    activated will re-calculate the date based on the current profile's
    perm_interval and today's date.

    This is similar to how the expire date is populated when creating a
    new user, or when changing the profile group. A button is used here
    so that the updating of the expire date is an intentional process,
    not one that happens upon any edit.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-947 SIP monthly login counts reporter

Generate monthly sip login counts and email.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-947 SIP login counts script

Script for counting SIP connections by SIP username by year, month, or
day.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-952 Added content by record ID

Use the new (2.7) style added content by record ID lookup.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

TPAC/ecom IE10 SSL alert messages

Signed-off-by: Bill Erickson <berickxx@gmail.com>

RT86770 PO print lineitem counts & order date

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-935 ACQ unified search progress dialog repair

Launch progress dialog before building the lineitem table when unified
search displays lineitem / bib results, otherwise, in some cases, it's
launched after the table renders and thus after the .hid() call is made
and is never closed.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-821 SCKO logout avoid focusing buttons

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-821 SCKO logout warning language mods

Also ensure focus returns to scan box after Continue Session is
selected.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-821 SCKO post-barcode timer; cleanup

Reinstate the timeout that occurs between entering a barcode and failing
to enter a password.

Remove some trailing references to the defunct keepMeLoggedIn function.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1827 guardian required set onload

Ensure that the parent/guardian (ident_value2) field gets the correct
value for 'required' on patron load to ensure it's not required for
non-juvenile patrons.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-941 CC form address warning message

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-935 Support blanket order printing

Adds po_items, which track blanket orders (and other misc charges)
to PO print template.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1643 invoice ident focus repairs

Invoice vendor ID box loads asynchronously.  Give the page a second to
load before attempting to focus and sanity check that the DOM node exists
before attempting to focus it.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-916 Holds max loops linger on checked out

Avoid canceling a hold that has reached its max target attempts when any
copies mapped to the hold are currently checked out.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-914 Prevent creation of extra lieitem copies

Prevent the API call which creates/modifies copies for a linteitem in
the lineitem copy UI from running if one is already in flight.  This is
done both by disabling the Save Changes button and by checking an
internal flag (in case something besides the button is causing the
action to run).

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-869 Silence Perl warning messages

Avoid a series of heavily-logged but harmless warning messages.

 * Add some missing CDBI fields
 * Sanity checks on string comparisons and hash deletes in hold
   targeter and circulate code.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm

JBAS-825 sitemap generator CRONTAB entry

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-821 aviod unnecessary timer resets

KCLS custom code resets the patron timer even when the patron is not
logged in.  This causes the logout warning to appear even when no patron
is logged in.  Avoid this by preventing the timer from starting when no
patron is logged in.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-821 LP1476370 SCKO logout warning xport

 * Show a logout warning dialog 20 seconds before discontinuing
   a self-check patron login session.

 * Reset the inactivity timer with each checkout.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-822 Adopt stock login timer logic

In prep for LP#1474051, which modifies the timeout behavior, start using
(as much as possible) the stock login timer code.  This will ease
migration when the logout warning code is merged in.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1850 Date of Birth Is Mandatory

User Is Now Unable To Bypass The Required Date Of Birth Field
During Patron Registration

Signed-off-by: Nick Duncan <nDuncan@catalystitservices.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
berick: repaired merge conflict; minor formatting repairs

Conflicts:
Open-ILS/web/js/ui/default/actor/user/register.js

KMAIN-1849 Birthdate Correction Error

Patron Registration Now Validates The Year From The DOB Field
To Make Sure Inaccurate Information Is Rejected

Signed-off-by: Nick Duncan <nDuncan@catalystitservices.com>
modified:   Open-ILS/web/js/ui/default/actor/user/register.js

KMAIN-1827 Juvenile Registration Require Guardian

During registration a juvenile is now required to fill out Guardian field

Signed-off-by: Nick Duncan <nDuncan@catalystitservices.com>
modified:   KCLS/openils/var/templates_kcls/actor/user/register_table.tt2
modified:   Open-ILS/web/js/ui/default/actor/user/register.js

berick: minor formating cleanup

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1825 Rename Patron Registration Billing-Address

Patron Registration Now Shows Address Options As Mailing && Residence

Signed-off-by: Nick Duncan <nDuncan@catalystitservices.com>

KMAIN 1836 Ability to remove copies

Removed reverse method from dojo query, use query to grab last
(item count variable) items instead

Signed-off-by: Michael Glass <mglass@catalystitservices.com>
Cross-ported by Bill.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-1643 Text Cursor Automatically Placed on Vendor Invoice ID

Set the focus to the Vendor Invoice ID field on invoice create
and view screens.

Signed-off-by: Dan Reuther <dreuther@catalystitservices.com>
modified:   Open-ILS/web/js/ui/default/acq/invoice/view.js

KMAIN 266 Note batch dropdown

Add batch note/alert field to purchase order line item batch apply.
Add loop which adds notes/alerts to lineitems.
Add notice to user that an alert code is required.

modified:   KCLS/openils/var/templates_kcls/acq/common/li_table.tt2
modified:   Open-ILS/web/js/ui/kcls/acq/common/li_table.js

Signed-off-by: Michael Glass (mglass@catalystitservices.com)
berick: did some minor formatting cleanup.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

KMAIN-AQ27 - Don't show 'amp' with '&' character.

Replace &amp with & in selection list display and in line item display.

modified:   Open-ILS/web/js/ui/kcls/acq/common/li_table.js

Signed-off-by: Dan Reuther <dreuther@catalystitservices.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-773 Merge custom PO view changes to stock

Use stock PO view code with custom changes instead of using custom files
which are getting out of sync.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
KCLS/openils/var/templates_kcls/acq/po/view.tt2
Open-ILS/web/js/ui/default/acq/po/view_po.js
Open-ILS/web/js/ui/kcls/acq/po/view_po.js

JBAS-773 Merge custom ACQ changes into stock li-table

Avoid using a custom version of the lineitem table code, since it is
getting out of sync with stock.  Instead, return to using the stock
template and JS with custom changes merged in.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
Open-ILS/web/js/ui/kcls/acq/common/li_table.js

Conflicts:
Open-ILS/web/js/ui/default/acq/common/li_table.js

JBAS-910 query parser merge conflict repair

Signed-off-by: Bill Erickson <berickxx@gmail.com>

Classroom cards generator; BSD cards

Generator creates 50 cards per school listed in a CSV file containing
school name and address info.  BSD info file and BSD classroom cards
included.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 skip ecards/classrooms expire extension

Avoid extending the expire date for Student Ecards and Classroom
Databases accounts.  Minor reformatting for readability.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 day_phone capture improvements

Capture the day_phone value and don't treat invalid phones as an error
condition preventing import of a student.  If an invalid phone is
encountered, treat it as unset.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 remoteauth/ezproxy checks expire date

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 class cards use default dob 1900-01-01

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 classroom cards generation / BSD CSV

Accounts are loaded via CVS the same as student accounts.  CSV for the
BSD acccounst included.  They are slightly different from student
accounts:

 * expire date now() + 10 years.
 * different alert message
 * different profile, ident_type, ident_value

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 student importer max expire date

Avoid expire dates far in the future on invalid DoB's by assuming no
students are younger than 2 years old for the purposes of calculating
the expire date.

Note, this does not affect the stored DoB.  We store whatever is sent.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 hold policies match group exactly

Due to conflicting weights on various match points, the hold matrix
matchpoints for student groups has to match the usr_grp exactly and
cannot be set to the parent group of Schools.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 ezproxy/remoteauth DB permission check (Students)

Instead of calling authenticate procedures, ensure the user is valid,
knows their password, then check whether the user has the
ACCESS_EBOOKS_AND_DATABASES permission.

This was added to support Schools cards, where users can access
databases and e-books, but cannot log into the catalog.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 Student groups and policies SQL

Schools
 -- Student Ecards
 -- Classroom Cards

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-886 Student CSV importer script

See --help for options.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-909 CollectionHQ floating case/when repair

Avoid logging every copy as floating by using CASE WHEN NULL correctly
in the CHQ query.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-885 eg-updator util and sip controls

Added support for the following commands:

start-util
stop-util
start-sip
stop-sip

"util" includes z39, connexion, and hold targeter.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

Sqitch DB host defaults to 127.0.0.1

This works better with default Postgres configs, specifically for
setting localhost no-password logins on dev servers.  Host can always be
overridden via --db-host.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 remove empty sqitch target 01.2.4.1-2.5.7-deps

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch revert sanity checks

1. Revert only the most recent change during deployment failures by
   default (sqitch deploy --mode change).  This can be overridden at the
   command line.

2. Clear the contents of revert/kcls-2.4-base-prod.sql so that under no
   circumstances can the base schema be reverted.

3. Document revert and verify scripts to indicate that the contents are
   intentially empty for pre-sqitch deployments.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 Send sample data to exports directory

Drop extracted sample data directly into the
KCLS/sql/sample-data/exports directory.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch.conf more DB defaults

Default to --db-user evergreen --db-host localhost.  Either can be
overridden at run time via --db-user / --db-host.

Replace tabs with spaces consistent w/ EG codebase.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 Rename data insert test for clarity

Rename deployment test function evergreen.mangle_data_on_deploy()
to evergreen.insert_in_deploy() to better reflect its behavior.

The test only needs to be used before INSERT's.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-654 Port blanket PO DB changes to sqitch

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-651 Port checkout-ok DB changes to sqitch

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch test for data mangling on deploy

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 remove 'data' sqitch instance; not needed

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch sample data continued

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 initial sqitch data/sample-data stuff

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 clean up sql dir with legacy dir

Move old SQL files into a new legacy directory (for now).
Legacy directory may later be deleted.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sample data generator via sqitch

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch deploy avoids creating existing users

When deploying to a PG instance that already has KCLS users, avoid
trying to create them, which would result in a sqitch deployment
failure.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 sqitch config auto-verify; set engine=pg

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-837 we are now a 2.5.7 alpha, still need to compare schema against prod

JBAS-837 bring system up to 2.5.x

JBAS-837 remove bogus ~ files

JBAS-837 bring up to kcls-2.4

JBAS-837 Initial sqitch configuration

JBAS-769 data sanity check; add 245p

Don't assume the nodes below a MARC tag are element nodes.
Apparently, in some cases, they were text nodes or possibly
some other node type.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 CHS title/author fix reversed

Signed-off-by: Bill Erickson <berickxx@gmail.com>

RT#85191 Legacy billing payment print error repair

Some legacy data has NULL values (e.g. last_billing_note) where current
data has empty strings.  This leads to exceptions and annoying, but
ultimately harmless error dialogs in the XUL client.  Add some sanity
checking to the code to avoid that.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 Holds pull list uses alt title/author as well

The CHS and pull list UI's use the same template code to display holds.
Modifying the template for one, means modifying for both.  This udpates
the hold pull list template to use the alternate title/author extraction
code (for 245ab and 100ac display) for consistency with the display
template.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 CHS BibTemplate no longer required

Title/Author info comes from parsing the XML directly now.  No need for
BibTemplate.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 CHS title/author subfield trimming

Title now shows 245a,b and author is 100a,c

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 Clear holds shelf increase query timeout

Increase the timeout for the query which finds which holds need clearing
from 1 minute to 10 minutes.  This mostly affects non-production
servers, which are often slower.

Note the query is still subject any configured PG statement_timeout
settings.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-654 Blanket PO local SQL upgrade

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 Increase CHS print dialog timeout

With larger print list, the print dialog needs more time before closing
to ensure all print data is successfully delivered to the printer.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-769 Clear holds shelf cache poll

When called with a new API name, the clear-shelf process returns the
cache key first thing to the caller (via respond_complete).  Instead
of waiting for the clear-shelf API call to complete, which often times
out, the client polls the server with the cache key at regular intervals
to see if the print data has been cached.  Once found, printing is
initiated.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-651 Grand opening copy status SQL

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1464709 Seamless non-standard copy status checkout

Add a new boolean configuration option to copy statuses which allow such
copies to be checked out without requiring an alert or override.

New column is config.copy_status.checkout_ok.  Any status with this
boolean enabled will be treated the same as Available, Reshelving, and
Holds Shelf status copies in that the status will be considered
checkout-able.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-231 blanket PO local template cross-ports

Port blanket PO changes to local templates and scripts.

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1440114 Blanket order release notes

Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1440114 Blanket PO finalize from PO view

Adds new "Finalize Blanket Order" button to purchase page for PO's that
are activated and have at least on blanket charge.  Finalizing the PO
disencumbers all blanket charges and marks the PO as complete.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
Open-ILS/web/js/dojo/openils/acq/nls/acq.js

Conflicts:
Open-ILS/web/js/dojo/openils/acq/nls/acq.js

Conflicts:
Open-ILS/web/js/dojo/openils/acq/nls/acq.js

LP#1440114 Direct charge blanket orders

Support "blanket" (long-lived, multi-invoice) orders via a new "blanket"
boolean on invoice item types.  Blanket charges can be invoiced multiple
times by creating a new fund_debit for each invoice item linked to a
blanket po_item.

This change also adds the amounts paid over time for blanket charges to
the Amount Paid summary information for purchase orders containing the
charges.

Adds a new Invoice Item Type of "Blanket Order".

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Invoice.pm

LP#1380803 Update PO summary amounts

Refresh the PO summary amounts (spent, encumbered, estimated) each time
an amount-changing event occurs.  These include changing the lineitem
price, adding/removing a direct charge, and adding/removing copies via
the copy grid.

Note that adding/removing copies via the batch-updater alread causes a
page refresh, which updates the summary amounts.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1380803 Include direct charges in PO esimated price

Move the estimated PO price calculation into the middle layer, along
with the total encumbered and spent calculation.  Add a new
PO.amount_estimated field for carrying the data.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Conflicts:
Open-ILS/web/js/ui/default/acq/po/view_po.js

JBAS-827 Add collections XML file cleanup to CRONTAB

Signed-off-by: Bill Erickson <berickxx@gmail.com>

JBAS-827 Disable circ purge by default in CRONTAB

And do some light rearrangement of cron entries for consistency.

Signed-off-by: Bill Erickson <berickxx@gmail.com>