Integrate Monograph Parts UI

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

Teach egEmbedFrame how to not throw warnings, and also how to be used more than once per app

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

Add "delete" and "undelete" actions to the MARC editor

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

Enable next/prev/first/last/results buttons on all new search results

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

Small-ize the "set default" button, and move it "into" the tab

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

Use tabs instead of Actions For this Record, and do away with that menu altogether (YAY!)

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

always calculate context menu for egMarcEditEditable on the fly

This way, the correct context menus get generated for a new
field/subfield entry or when a field's tag is changed.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

set value ctype for RDA 33x$a

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

teach MARC editor how to build context menus for subfield values

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: allow coded value maps to be associated with subfields

This change allows a set of values from a record attribute's
coded value map to be associated with a MARC subfield and
returned by the tag-table service.  This is for supporting
the use of controlled vocabularies for subfield values in
the MARC editor.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1442815: add RDA Content Type, Media Type, and Carrier Type attributes to seed data

Note that a record attribute reingest is required for the
attributes to be available for search purposes.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1442815: teach record attributes about SKOS

Add columns to the database for storing URIs of SKOS
vocabularies and concepts.  In particular:

- add a vocabulary column to config.record_attr_definition
  to store the URI of the published vocabulary associated
  with the record attribute
- add a concept_uri column to config.coded_value_map

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1442796: make XPath-based record attribute definitions work

Record attribue definitions that use XPath to extract their values
could break ingest; this patch fixes that.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Register a pile of standard hotkeys

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

Teach access keys how to deal with ng-click actions and empty hrefs

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

Integrate Manage Authorities UI -- will require modification to de-XUL-ify in a later sprint

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

Integrate URL Verification UI -- will require modification to de-XUL-ify in a later sprint

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

Batch Edit iframe template

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

Integrate batch edit (including bucket batch edit entry point)

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

Add support for standalone grid menu items even when there is a menu label

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

Embed vandelay

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

Make space saving configurable on embedded iframes

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

Flat Text MARC editor support

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

Allow global delimiter override

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

Propagate undo changes into FFs

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

Styling cleanup

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

Honor "obscure DOB" YAOUS

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

FF context menus, generated from DB data

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

Resolve after saving

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

Fixed field editor!

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

New directive for FF editing

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

Fetch FF pos map and value map, per record type

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

Make Date of Birth label red if DOB is emtpy

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

Add Profile to the column list on patron search results

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

Remove unnecessary dropdown-toggle attr

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

Close other context menus before showing this one

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

indicator value lists now use "description" to, well, label descriptions

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: (erratum) create Pg enum type to represent MARC record type

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: (erratum) use "description" rather than "value"

This makes the indicator list have the same shape as the
subfield code list.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

use contextItemGenerator instead of contextItemContainer

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Typo (missing space)

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

Tooltips for holdable/visible on status and location in item summary

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

AngularJS egTagTable service + context menus for tags, subfield codes, and indicator values

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Do not change the tab on iframe changes when using the summary bar buttons

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

Use ng-show instead of ng-if to avoid an extra scope injection

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

Toward dirty data flag and better prev/next/start/end buttons

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

Add buttons to call appropriate opac functions from the record summary bar

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

Allow UIs wrapping the opac to call functions in the opac iframe

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

Allow the iframe code to pass a handle to the iframe out to the parent scope

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

Respect items-out tally settings

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

Give the Holds Shelf a template and print button, and expose both that and the Pull List in the workstation print interface

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

Add support for a context menu item generator function on the scope (ContextItemGenerator), and pull ContextItemContainer from the parent scope

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

Move the MARC namespace to MARC21 so someone can make a more generic superclass one day

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

Use menu grouping on holds-for-this-record grid

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

Teach autogrid about grouping Actions menu options

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

Add "show holds", "show patrons", "show recent circs", and "show triggered events" to various grids

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

Allow Record Summary to collapse (and remember the choice)

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

Add authority control sets support to the MARC namespace.  Some TTS support will be required for Angular use.

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

LP#1427287: methods to retrieve entire tag table

This patch adds methods to open-ils.cat for
retrieving either the entire tag table or just the list
of MARC fields, either from the stock MARC format
seed definition or as potentially customized for the user's
workstation OU:

open-ils.cat.tag_table.all.retrieve.stock
open-ils.cat.tag_table.all.retrieve.local
open-ils.cat.tag_table.field_list.retrieve.stock
open-ils.cat.tag_table.field_list.retrieve.local

The user must have UPDATE_MARC permission.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Allow hotkeys to work in form elements -- we may need to use the underlying Mousetrap library directly, however

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

LP#1427287: update fm_IDL.xml add ADMIN_TAG_TABLE permission

A question: should cataloging administrators get
system-level ADMIN_TAG_TABLE by default?

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: tag table seed data for MARC 21

This was converted from marcedit-tooltips.xml
using the marc_tooltips_to_sql script introduced in the
previous commit.  As a consequence, the tag table
data is somewhat out of date; this will be addressed
later.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: script to migrate MARC tooltips data file

marc_tooltips_to_sql is a script that converts an
Evergreen MARC tooltips file to a set of SQL statements
for seeding config.marc_field, config.marc_subfield,
config.record_attr_definition, and config.coded_value_map
with tag table data.

Example usage:

marc_tooltips_to_sql --input Open-ILS/xul/staff_client/server/locale/en-US/marcedit-tooltips.xml \
                     --output /tmp/bibs.sql --type biblio

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1427287: database schema for storing MARC tag+subfield metadata

This patch defines several tables, views, and functions for
storing information about MARC tags and subfields recognized
by Evergreen:

Tables:

* config.marc_format:   list of MARC formats; at present, however,
                        only MARC21 is supported
* config.marc_field:    list of fields recognized by the Evergreen
                        database.
* config.marc_subfield: list of subfields recognized by Evergreen

Views:

* config.marc_field_for_ou
* config.marc_subfield_for_ou

These two views assist in the process of determining the set of
MARC fields and subfields are meant to be in force at a particular
OU.  These are generally not meant to be queried directly, in
favor of the config.ou_marc_{sub}fields() functions.

Functions:

* config.ou_marc_fields(marc_format, marc_record_type, ou)
* config.ou_marc_subfields(marc_format, marc_record_type, ou)

These functions return the authoritative set of fields or
subfields in force at a given OU.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Add F1 hotkey (checkout)

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

use preventDefault() on navbar hotkeys

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

wire the date filter into the payments grid in billing xact history

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Apply date filter range on command in billing xact history

Adds an Apply button for apply changes to the date filter to the grid.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Grid supports automatic refresh with query change

Adds a new gridControls function watchQuery(), which returns the
contents of the grid query.  When the contents change, the grid
automatically redraws.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Set refresh function so Actions trigger refresh

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

internalize jquery

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

LP#1312308 circ scripts, penalty removal release notes

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1312308 remove deprecated open-ils.penalty service

open-ils.penalty service has not been used by Evergreen since before
version 2.0.  Instead of just cleaning out the script configuration,
clean out the entire application.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1312308 script circ and deps removal

* Remove libjs and spidermonkey from prereq installer
* Remove references to spidermonkey and script runner
* Remove stock JS circ and catalog scripts

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Docs: 2.8 Patron Message Center

Documentation for Patron Message Center development in 2.8

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Docs: 2.6 Duplicate Serials Barcode Alert

Documentation for new duplicate serials barcode alert in Serials Batch Receive.

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen

New Documentation for 2.8

2.8 Display Count of Existing Copies
2.8 Duplicate PO Name Alert
2.8 Negative Balances on Patron Accounts

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

LP#1468362 fix release me syntax and update acknowledgements

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Update action trigger docs to add missing fields.

Rename heading to "Notifications / Action Trigger" so it matches
the staff client admin menu.

Signed-off-by: Josh Stompro <stomproj@larl.org>

LP#1465830: authority linker now ignores $e and $4 in bib name headings

presence of relator terms or codes in $e or $4 can prevent authority
headings linking

To test
-------
[1] Create an authority record with a name heading like this

    100 $a Example, Natalia $d 1945-

[2] Create a bib record that contains the name and a relator
    code or term, e.g.,

    100 $a Example, Natalia $d 1945- $4edt

[3] Run authority_control_fields.pl. Note that the
    heading in the bib is *NOT* linked, as evidence
    by no $0 being added to the bib heading and no
    rows added to authority.bib_linking for that bib
    and authority.

[4] Apply the patch and re-run step 3.  This time,
    the link gets established:

    100 $a Example, Natalia $d 1945- $4 edt $0 (CONS)xxx

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Docs: Change references to release 2.8.2 for server upgrade

Since 2.8.2 is released, change references in upgrade docs to this newest
release.

Signed-off-by: Ben Shum <bshum@biblio.org>

Porting 2.8.1 -> 2.8.2 SQL upgrade

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

LP#1446816 HTML-escape notes in XUL patron alert page

Avoid rendering HTML contained in patron messages, etc. in the XUL
patron stop-sign page.

Thanks to Jason Etheridge, ESI for noting this problem and contributing
the original patch fix.

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

Release notes for 2.8.2

Adding a section for 2.8.2 bug fixes in the 2.8 Release Notes.

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

Remove alpha and RC scripts for 2.3-2.4

These were in the repository from the testing phase of 2.3-2.4 during the
2.4 alpha/RC period. These script are not required and may be confusing if
run. Remove as unnecessary.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1205061: Add IF EXISTS to some DROP statements in 2.3-2.4 upgrade script

To prevent potential issues, add IF EXISTS clauses to DROP statements in this
version upgrade script.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Documentation LP1369203 - Removed formatting around image link to
re-enable the image.

Signed-off-by: Josh Stompro <stomproj@larl.org>

Docs: Updated opensearch example with new label format

The OpenSearch title now specifies the library name instead of the generic
"Evergreen OpenSearch" at every scope.

This is one of many TPAC Discoverability Enhancements included in
Evergreen 2.8.

Signed-off-by: Josh Stompro <stomproj@larl.org>

Docs: LP#1387639 - Barcode Completion Documentation

Signed-off-by: Josh Stompro <stomproj@larl.org>
Conflicts:
docs/root.txt

Signed-off-by: Josh Stompro <stomproj@larl.org>

fix live_t/08-lp1366964-libdbi-error.t

from this:

opensrf@dev141:~/git/Evergreen/Open-ILS/src/perlmods/live_t (lp902255test)$ prove 08-lp1366964-libdbi-error.t
08-lp1366964-libdbi-error.t .. # Tests libdbi transaction error reporting
08-lp1366964-libdbi-error.t .. 1/2
 #   Failed test 'CStoreEditor returns standard query error'
 #   at 08-lp1366964-libdbi-error.t line 31.
 #          got: 'DATABASE_UPDATE_FAILED'
 #     expected: 'DATABASE_QUERY_FAILED'
 # Looks like you failed 1 test of 2.
08-lp1366964-libdbi-error.t .. Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/2 subtests

Test Summary Report
-------------------
08-lp1366964-libdbi-error.t (Wstat: 256 Tests: 2 Failed: 1)
  Failed test:  2
  Non-zero exit status: 1
Files=1, Tests=2,  1 wallclock secs ( 0.02 usr  0.01 sys +  0.82 cusr  0.11 csys =  0.96 CPU)
Result: FAIL

to this:

opensrf@dev141:~/git/Evergreen/Open-ILS/src/perlmods/live_t (lp902255test)$ prove 08-lp1366964-libdbi-error.t
08-lp1366964-libdbi-error.t .. # Tests libdbi transaction error reporting
08-lp1366964-libdbi-error.t .. ok
All tests successful.
Files=1, Tests=2,  2 wallclock secs ( 0.02 usr  0.02 sys +  0.85 cusr  0.08 csys =  0.97 CPU)
Result: PASS

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP 902255: Fix upgrade script.

Run as-is, the upgrade script produces an error:

ERROR:  CREATE INDEX CONCURRENTLY cannot run inside a transaction block

Following this conversation in IRC, it was decided to remove the keyword
CONCURRENTLY from the create index statement.

http://irc.evergreen-ils.org/evergreen/2015-06-15#i_182857

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

LP 902255: Stamping upgrade script

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

LP#902255: Use simple alert box

Instead of the generic communication error dialog, we'll use
the simple yns_alert box instead.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#902255: Correct "ILS Event at check in" logic

In a classic case of "fix a bug to reveal another" it turn out that
there was a small logic bug hiding in the check in hold-capture code.
The result of this bug was that the wrong ILS Event object was being
returned when we ran into the double-scan hold-capture situation by
overwriting the correct one immediately after we construct it, which
in turn prevented the previous commit from working as designed.

We now return the correct ILS Event immediately so that we can
properly detect the situation.  Two birds with one branch.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#902255: Protect against hold double-capture

When the staff client is in Fast Scan (async) mode, it is possible
to double-scan at checkin and cause two holds to be captured by the
same item.  By creating a specially crafted unique index we ensure
that the database rejects this situation.

This protection would cause unhandled errors in the staff client, so
we also register a new string to be displayed when this particular
situation occurs, alerting staff to the possible double-scan of a
barcode at checkin IFF they are using async mode.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1463145 - Debian Jessie needs libexcel-writer-xlsx-perl

Without this package, attempts to run clark-kent.pl will fail on Debian Jessie.

Also remove old libspreadsheet-writeexcel-perl package from pre-reqs as it is
no longer required.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1365661 Found patrons with same name not opening

This patch fixes the error, while registering a new patron
and clicking the link "Found {$0} patron(s) with the same
name" did not previously work in the staff client.

To reproduce the bug, register a new patron with the name of an
existing patron.  The little widget with the message described
above will appear.  The message is a hyperlink, but nothing
appears to happen if you click it.  The JavaScript console will
contain an error ReferenceError:js2JSON is not defined.

After applying this patch, clicking the hyperlink does a search
for matching patrons as intended.  The ReferenceError also
disappears.

Signed-off-by: Adam Bowling <abowling@emeralddata.net>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1154656 MARC Expert Search "Add Rows" adds duplicate row

This patch changes the behavior of the "Add Search Row" link
(JavaScript function from simple.js "addExpertRow()") so that
we are always cloning the initial "empty" row instead of
potentially cloning a row -- by way of cloneNode() -- with user
input in place, and causing duplicate search rows.

To reproduce the bug, go to Advanced Search, fill in all three
search fields with some text, click on Add Search Row.  You wil
see that the Title search row along with the data in the text
box is duplicated and added to the search.

After this patch, the new row should be added with an empty
text box.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Rashma Kumaran <rkumaran@kcls.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

LP#1342227 - Allow EDI Ruby dependency installation on Ubuntu 14.04

Create a script for Ubuntu 14.04 ruby installation and installation
for EDI dependencies.  Remove 'rcov', which has been deprecated for
Ruby 1.9.1 and install Bill Erickson's GIR improvements.

In the future we will need to remove this dependency altogether (see
bug 1373690), but for now, at least get it working on Ubuntu 14.04.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>