JBAS-1787 More SQL insert wrappers

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

JBAS-537 Mark missing pieces in-transit items

Cancel active transits before checking an item back out to the most
recent patron during mark-missing-pieces.  Otherwise, the checkout will
fail.

Commit also removes an unnecessary transaction from the first half of
the API and its associated FIXME messages.

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

JBAS-1850 Reset lineitem selection every page

Clear the list of selected lineitems with every lineitem list page
(selection list, PO, etc.).  This prevents an error where lineitems from
a different selection list would be selected for inclusion in a new PO,
leading the undefined JS errors referencing nonexisting lineitems.

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

JBAS-1850 ACQ vlagent falls back to browser localStorage

Not related specifically to JBAS-1850, just uncovered during testing.

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

JBAS-1799 Skip 2.10 SQL reingests

Skip the partial reingests that came as part of the 2.9->2.10 SQL
upgrade.  Together they are no faster than a full re-ingest, which we
can run with more fine-grained control.

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

Sample user data address cleanup

Scrub some interesting auto-generated sample data addresses.

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

Teacher cards short username handles characters

Fixes bug in teacher account generator so that it properly handles
student_id values less than 4 characters for both numbers and letters.

Previously it would fail padding non-numeric IDs.

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

Renton Risdon Middle classroom cards

Also remove some errant spaces from the Renton classroom info file.

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

JBAS-1786 Remote auth 2.10-style password test

Teach our custom version of remoteauth.cgi to use the new 2.10-style
in-database password check.

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

JBAS-1788 KCLS 2.10 Release Notes

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

JBAS-1768 X-port XSS login form repairs

Cross-porting 472bd5ae1486fc0349581b02e666d0e8b8d5c143 to our custom
login form.

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

JBAS-1786 Protect Stripe org settings

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

JBAS-1786 KCLS 2.9 -> 2.10 SQL Upgrade

Of note, using iter_count 8 for password encryption.

All inserts are wrapped.

Composed of 2 upgrades, a main, and a data reingest.

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

JBAS-1790 SIP uses adopt-syslog, applies syslog ident

Be sure syslog settings applied to the SIP server persist through
OpenSRF configuring its on syslog client.  Apply a specific 'sip' syslog
identity.

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

JBAS-1792 Remote auth logging handles barcodes

Fix an issue in remoteauth.cgi logging where it failed to log barcodes
when they were passed via the generic "user" parameter.

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

JBAS-1741 Targeter SQL wrap inserts

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

JBAS-1741 Targeter fetches holds in batches

Avoid clobbering the jabber servers with hundreds of thousands of hold
ID messages by grouping the hold IDs into messages of 10k IDs each.

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

JBAS-1741 Hold targeter v2 SQL updates

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

JBAS-1741 Hold targeter v2 gets KCLS has-checked-out-copy

Cross-port code to implement KCLS policy of leaving holds alive so long
as a checked out copy exists regardless of whether the targeter has
reached max org unit target attempt loops.

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

LP#1596595: Stamping upgrade scripts for hold targeter refactoring

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1596595 AOUS lookup batch by org id

Org unit setting value lookup for batches of org units, instead of the
traditional batches by setting name.

Perl live test included.

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

LP#1596595 Hold targeter release notes

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

LP#1596595 Hold targeter refactoring and optimization.

* New open-ils.hold-targeter service

* Ports hold targeter code to a Perl utility function, communicating w/
  the DB via cstore instead of storage.

* Adds a new global flag 'circ.holds.retarget_interval' for configuring
  the hold retarget interval in the database.

* Adds a new DB function to regenerating hold copy maps to make map
  deletion and creation more efficient.

* Adds an option for targeting holds in newest to oldest order.

* Caches all org unit settings per targeter run.

* Adds support for "skip_viable" option.  This tells the hold targeter
  to avoid modifying any holds that target viable copies.  AKA "fix
  broken" mode.

  For example, you might run in skip_viable mode with a retarget
  interval of 24hr once a day to repair non-viable holds, then also run
  the targeter in regular mode once a day with a retarget interval of 48
  hours to give staff 2 days to process viable holds.

* Hold target loops logic changes:

 ** Org units with fewer target attempts are prioritized during loop
    processing.  So, instead of segregating org units into 2 categetories,
    those attempted in the current loop and those not attempted, sort those
    not attempted by the number number of times they have been attempted.
    Within each grouping, prioritize by target weight/proximity as before.

 ** All org units that have been attempted less than the max configured
    amount are on the table for targeting, not just those that have been
    targeted less than the current loop max.  If no orgs with
    less-than-current-max attempts are found, try orgs that match the
    current max (but are still less than the configured max).

 ** When activated, target looping treats the pickup lib like any
    other org unit.  If a targeted copy at the pickup lib remains
    un-captured, at re-target time, a copy at a different branch is
    chosen (if one is available) even if other copies at the pickup
    lib are targetable.

* Parallel targeting support baked into service.

  Teach the targeter to process a subset of holds based on the number of
  parallel targeters at play and the parallel targeting slot each targeter
  instance occupies.

  As with the existing hold targeter, group holds by their metarecord to
  avoid multiple targeter processes targeting the same sets of potential
  copies.

* Logging / code refactoring and clean up.

* New hold_targeter_v2.pl script for batch hold targeting.  Existing
  targeter remains for backwards-compat.

hold_targeter_v2.pl options:

--verbose
    Print process counts

--parallel <parallel-process-count>
    Number of parallel hold processors to run.  This overrides any
    value found in opensrf.xml

--target-all
    Target all active holds, regardless of when they were last targeted.

--skip-viable
    Avoid modifying holds that currently target viable copies.
    In other words, only (re)target holds in a non-viable state.

--retarget-interval
    Override the 'circ.holds.retarget_interval' global_flag value.

--parallel-init-sleep
    Time to wait between starting each parallel instance.  Useful for
    avoiding dog-piling the DB.

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

JBAS-1581 Backstage cron updates and docs

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

JBAS-1874 Merge patron checkout history

https://bugs.launchpad.net/evergreen/+bug/1712657

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

JBAS-1554 Circ history delete untracked

Script to find and delete circulation history rows for patrons who
disabled circ history after the initial migration but before the new
history delete code was deployed.

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

JBAS-1554 Circ history checker log cleanup

Log each affected user once (instead of once per circ) for ease of
manual cleanup.

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

JBAS-1554 Circ history checker log correct offset

Log the correct 0-based offset when reporting patrons that have
un-migrated circulations.  This makes manually repairing them easier.

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

JBAS-1554 Circ history migration check script

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

JBAS-1867 Disable lost.auto event creation

Piling lost.auto event creation on top of checkout.due events for auto
mark lost often results dying A/T drone processes.  Cause is still
unknown.  Since we're not currently using lost.auto events, disable
lost.auto creation pending a resolution.

This primarily affects the 28-day auto mark lost reactor.

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

JBAS-1817 Hide BC screen reader links via local CSS

BC screen reader links are not hidden as expected using only the code
delivered by their API.  Add some custom local CSS to hide them.

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

JBAS-1817 BC header/footer API updates

Update self-reg (patron card app) to use the new BC header/footer/etc
API.

TODO: resolve issue where screen reader navigation is visible.

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

Backstage quarterly bib export count email

Send an email after the quarterly bib export file has been generated and
uploaded with the count of bib records in the file.

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

Backstage monthly authority CRONTAB updates

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

JBAS-1554 Aged circ SQL update drop repair cont.

Avoid trying to drop functions that may not yet exist.

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

EDI pusher always uses passive FTP

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

JBAS-1148 Set cat date on new vandelay records

Set bre.cataloging_date = now for all bib records created as new from
the MARC Batch Import UI.

API supports setting this optionally via new set_cat_date option.

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

JBAS-1554 User circ history transitional API

Retain dummy versions of the old circ history API calls to avoid any
unexpected explosions during the 2.9.003 upgrade and BC circ history
code migration.

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

JBAS-1554 Custom purge circs SQL params repair

Update action.purge_circulations_custom definition to include the start
and end time function parameters to ensure the existing function is
replaced instead of creating a new function.

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

JBAS-1554 Aged circ SQL update drop repair

DROP some functions before a view they depend on is replaced.

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

JBAS-1792 Log remoteauth (ezproxy) login failures

Log all of the reasons a remoteauth login might have failed, including
bad passwords.  Useful for debugging student e-card authentication
issues.

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

JBAS-1554 Create aged circ index concurrently

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

JBAS-1554 Avoid duplicate circ_ids vars

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

JBAS-1554 Aged circ index timeout

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

JBAS-1554 Aged circ parent circ index

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

JBAS-1554 Apply 1 year circ purge config

Also avoid overwriting the config within the nightly purge script.
That's confusing.

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

JBAS-1554 C/O history tpac display repair (kcls template)

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

JBAS-1554 C/O history phase 2 SQL

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

JBAS-1419 CO history deployment SQL tweaks

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

JBAS-1419 CO history statement timeout

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

JBAS-1419 CO history usr index (LP#1615736)

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

JBAS-1419 CO history handles serials copies

Cross-port 6479b578.

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

JBAS-1419 Aged circ API/SQL cross-port

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

Stamping upgrade scripts for aged circs display branch

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

LP#1497335 Aged/All circulation API access

Various SQL, IDL, and API changes for accessing aged circulations,
primarily via all_circulation objects, for imporoved staff client
integration.

*. Support open-ils.pcrud access to the action.all_circulation DB view /
   'combcirc' class.

*. Add missing parent_circ, checkin_scan_time, checkin_workstation to
   combcirc class.

*. Add 'usr' field to combcirc.  The action.all_circulation VIEW will
   return NULL as the 'usr' column value when returning data for an
   aged_circulation.

*. Add virtual 'active_circ' and 'aged_circ' fields to combcirc for
   fleshing the related action.circulation or action.aged_circulation
   object.

* Adds SQL functions for action.all_circ_chain and
  action.summarize_all_circ_chain, which pull data from
  aged_circulation.

*. API calls
open-ils.circ.[prev_]renewal_chain.retrieve_by_circ[.summary] now return
data for active and aged circulations by using action.all_circ_chain and
action.summarize_all_circ_chain.  When using these APIs, a null value
in the 'usr' column is the indication that a given circulation or circ
chain summary represents an aged circulation.

* API open-ils.circ.copy_details.retrieve will now optionally return
  aged circ data within the copy circ history.

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

JBAS-1326 C/O history TPAC cross-port

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

JBAS-1326 C/O History SQL Backport

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

LP#1312699: Minor modifications to patron checkout history deletions

Minor modifications to make patron checkout history deletions work with
the new table.  open-ils.actor.history.circ.clear
extended to take an optional list of circ_ids, to limit what's deleted
from the history.

Signed-off-by: Jake Litrell <jake@masslnc.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1312699 - Add feature to allow user to edit their checkout history.

In the checkout history page, a new column and action selector is provided to allow
the paton to indicate items they would just as soon not want to see again in the
history list for whatever reason.  NOTE: This is not a PURGE function; it simply
suppresses display of items in the history list (and exported history files).
Internally, the circulations are kept by Evergreen for several reasons which are
not affected by this functionality.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Jake Litrell <jake@masslnc.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1564079 Stamping upgrade script

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1564079 Checkout history skips nonexistent items

Modify the user checkout history migration process
(2.9.3-2.10.0-upgrade-db.sql and 0960.schema.decouple_co_history.sql) to
avoid inserting history rows for circulations whose copies do not
exist in the database.

For reference, this is a rare condition that cannot be recreated in a
modern EG system short of modifying constraints.

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

LP#1564079 Checkout history handles serials

Remove the target_copy foreign key constraint on action.usr_circ_history
so that it does not prevent creation of rows from circulation's that
link to serial.unit objects.  Similar to action.circulation, the foreign
key constraint is enforced via a evergreen.fake_fkey_tgr trigger
instead.

This modifies the 2.9.3-2.10.0-upgrade-db.sql and 0960 upgrade scripts,
since without this change, the scripts may fail.

A separate upgrade script is added to drop the constraint for those who
have already successfully run the the 2.9.3-2.10.0-upgrade-db.sql script
(i.e. those who don't use serials).  For simplicity, the
evergreen.fake_fkey_tgr trigger is only applied in this separate upgrade
script (in addition to the base schema, of course).

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Conflicts:
Open-ILS/src/sql/Pg/version-upgrade/2.9.3-2.10.0-upgrade-db.sql

LP#1527342 Stamping upgrade script decouple checkout history

Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1527342 Patron checkout history table release notes

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

LP#1527342 Patron co history remove on purge

Delete checkout history rows when purging a user.

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

LP#1527342 Patron checkout history SQL/IDL

Adds a new table action.usr_circ_history for tracking opt-in checkout
history.  History is maintained via trigger on action.circulation.

Includes updates to html/email/csv checkout history templates to
gracefully handle NULL checkin_time values, since history starts as soon
as an item is checked out.

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

JBAS-1581 Backstage docs continued

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

JBAS-1722 Automate backstage bib collisions processing

Adds a separate command for processing bib collisions files.

./backstage-agent.sh -p <admin-pass> -c

Leaving as a separate command for now so that I can run it manually the
first time or two.  Will later integrate this into the quarterly results
processing task.

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

JBAS-1581 Backstage cron updates and docs

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

JBAS-826 Revert custom hold counts part II

Remove all traces of the custom holds-per-bib count code.

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

JBAS-1766 More insert-on-deploy wrappers

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

JBAS-1766 KCLS 2.9 sample SQL data update

Updated sample bib record set and new data export for 2.9 data.

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

JBAS-1766 2.9 SQL upgrade wrap inserts

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

JBAS-1742 2.9 authority ingest after 2.9.002 SQL

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

CAT-103 Add missing API call to XUL constants files

FM_ACP_BATCH_RETRIEVE_VIA_ACN_LIST.authoritative is referenced by XUL
but not defined.  Add it to the constants file.

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

JBAS-673 Post 2.9 multi-page repairs

Only attempt to render copy details for lineitems that are part of the
currently displayed page of lineitems.  Alt fix for LP#1208613.

To ease debugging and eventual support by the browser client, use the
borwser's localStorage object when not in XUL mode.

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

Revert "LP#1208613: Reset lineitem cache when resetting table"

Avoid clearing the lineitem cache on each page of lineitems.  We need
the cache to for multi-page selection-to-PO support.  If the original
bugs addressed by this commit resurface, we can resolve it by
maintaining 2 caches, the stock liCache and a separate parallel cache
used just for multi-page actions.

https://bugs.launchpad.net/evergreen/+bug/1208613

https://kcevergreen.atlassian.net/browse/JBAS-673

This reverts commit 341babf860a20539529e29db252545031b01ed5f.

Authority linking scripts CRONTAB synchronization

Update the sample CRONTAB to match how linking new/modified authority
records scripts are run in production.

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

Student importer now uses Tukwila (not Foster)

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

Disable SIP stderr logging by default

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

JBAS-826 Record hold counts for bib-level only

Modify the holds-per-record counting code to only look at title-level
holds, for now anyway.  This is makes the query a lot faster, which
is a concern given that we still see several million API calls to
open-ils.circ.bre.holds.count per day.

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

JBAS-1554 C/O history logging mods

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

JBAS-1554 C/O history deployment script mods

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

JBAS-1554 Patron C/O history migration phase 1

New circ history table and functions, plus script to migrate circ
history in batches.

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

JBAS-1736 Remove unused, stranded transaction in LI fetcher

Remove unnecessary and un-closed xact=>1 in custom lineitem fetching code.
This avoids leaving cstore backends stranded.

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

JBAS-1736 Prevent barcode changes on items in controlled statuses

Disable barcode edit text input when a copy is in a non-editable
status in the "Update Items" and "Create Or Re-Barcode Items" interfaces.

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

CAT-103 Clean up pauseComp()

Remove instances of pauseComp() in copy_browser.js
and cat/util.js in favor of .authoritative calls.

Make open-ils.search.asset.copy.batch.retrieve
authoritative.

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

CAT-47 Checkout receipt shows due date time for hourly checkouts

Due date time (hour and minute) shows on checkout receipts
when the circulation duration for the checked out item
is less than a day.

Signed-off-by: Victoria Lewis <vLewis@catalystdevworks.com>
modified:   Open-ILS/xul/staff_client/chrome/content/util/print.js

JBAS-826 Revert custom hold counts part I

Teach the API to call the stock holds-per-record counting code,
bypassing the custom KCLS code.

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

JBAS-1405 Revert custom hold queue stats code

Return to using the custom hold queue position calculation code.
This merely moves the logic from the DB back into the Perl code.  The
query logic is unchanged.

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

LP#1619703 Transfer ACQ lineitem XUL UI

Support for marking a bib record as a lineitem transfer destination for
the XUL client.  Adds a new menu entry under the 'Actions For This
Record' menu in the main bib record UI.

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

LP#1619703 Transfer ACQ lineitem browser UI

Supports marking a bib record as a lineitem transfer destination in the
web staff bib record detail interface.  Found in the 'Mark For:' selector
menu.

Supports inititiating the transfer from the embedded ACQ purchase order
interface.  ACQ action is found in the Actions selector for each
lineitem.

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

LP#1619703 Transfer ACQ lineitem to bib API

New open-ils.acq API call for transfering a lineitem and its assets
to a new bib record.

Any asset.call_number's created from the lineitem are migrated to the
new target bib record.

Any monograph parts linked to copies created from the lineitem are
duplicated (if need) on the target bib record and part copy maps are
modified to use the new parts.

API: open-ils.acq.lineitem.transfer_to_bib(auth, li_id, target_bib_id)

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

JBAS-1767 PG 9.6 replace connect-by (perm funcs)

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

CAT-80 Set focus to distribution formula

Refactor focusCopyLibSelector to accept a boolean
parameter and have the option to move foucs onto
the distribution formula field instead of the first
copy's owning library selector.

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

JBAS-1767 PG 9.6 remove connect-by

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

eg-updater -a start-util starts edi_webrick process

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

JBAS-1665 Self-reg kiosk mode additions (submit another, etc.)

* Hide submit-another button in kiosk mode
* Adds a black border around the verification requirements "wall of
  text"
* Clicking on a card image now selects the image -- no longer necessary
  to click exactly on the radio input.

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

JBAS-1732 Lake Washington school/classroom cards

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

JBAS-1761 Skykomish school cards

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