LP1816679 Match Set Type selector options

Provide a canned set of options for Vandelay => Create/Modify Match
Set => Match Set Type of 'biblio', 'autority', and 'serial'.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

LP1693580 Authority create/update API repairs

Fixes issues with the authority record create and update open-ils.cat
API calls, where internal function calls were not passed the correct
values.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Pearl <dpearl@cwmars.org>

LP1693580 Marc editor notify and API changes

1. Indicate in the interface when a bib/auth MARC record edit failed or
succeeded.

2. Handle permission failures on MARC edit by displaying the permission
override dialog.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Pearl <dpearl@cwmars.org>

LP1735754 - WGAC Compliance for Advanced Search

In compliance with WGAC standards, the html valign attributes have been
removed from Open-ILS/src/templates/opac/parts/advanced/search.tt2

Signed-off-by: Sam Link <slink@LIBPC002>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

Docs: fix uneven lengths of code delimiter blocks

This continues the work that Dan Scott started, fixing more unmatching
start/end lines around code blocks, as revealed by the sad-looking HTML
output of asciidoctor.

This commit also fixes a few heading underlines of the wrong length.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#1029601: Prevent duplicate holds from double clicks

Double clicking on hold submit buttons causes multiple holds to be
generated.  This commits adds code to disable the Submit button(s)
after the first click, so subsequent clicks don't "work" in unintended
ways.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1774707 Quick fix-up: add missing comma

We probably should move this on up with the others, but as a quick
fix...

Signed-off-by: Dan Wells <dbw2@calvin.edu>

lp1538678 MARC edit warning prompt translateable strings

- Define "There is unsaved data in this record." as a translateable string.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/cat/share/marcedit_strings.tt2
modified:   Open-ILS/web/js/ui/default/staff/cat/services/marcedit.js

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

lp1538678 Apply Warning Prompt when leaving dirty MARC editor

- Check for unsaved user input when leaving the Create New
MARC Record UI, and warn if it exists.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   services/marcedit.js

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

LP1817601 MARC Flat Text Editor Monospace Font

Use a fixed width font in the MARC flat text editor, consistent with the
fonts used for the non-flat-text editor.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>

LP1774707 Stamping DB update: group members grid

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

LP#1774707: Add Seed Data for worstation setting for group member details

Adds the config.workstation_setting_type for patron group member details
to seed data and includes an upgrade script.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1774707: Allow saving Group Member Details grid settings

For a web staff eg-grid to save and retrieve its settings, it needs a unique
persist-key. This commit adds one for the Group Member Details grid (which is
within the patron record view).

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP#1669120: Make scrollable dropdown height match column picker

To make scrollable dropdowns more usable, let's make them the same
height as the grid column picker dropdown. This is a quick, simple
improvement, and it follows precident with a frequently used feature
(column picker, which has no known complaints).

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

Docs: Add bugfix release notes for 3.3.1

This commit also relabels the original (now redundant) headings.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Add bugfix release notes for 3.2.6

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Add bugfix release notes for 3.1.12

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP1813633 TCN search can find deleted records

As with the XUL client, when performing a bib record TCN search
(Cataloging -> Retrieve Record By TCN), first look for non-deleted
records with the requested TCN.  When none are found, perform a
secondary search for deleted records with the requested TCN.

To test in concerto:

[1] Navigate to Cataloging -> Retrieve Record By TCN
[2] Search for TCN value "10", which is deleted by default in Concerto.
[3] Confirm the deleted record is loaded.

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

LP#1788680 prevent copy template apply failure on null stat cats

Under some circumstances, the current code can result in a JS failure:

Cannot read property 'filter' of undefined

When this happens, the copy template may apply partially or not at all.
This change simply makes sure cp.stat_cat_entries() exists before
attempting to filter it.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1823982 Vandelay Match Set new tree repair

When creating a new Match Set expression, provide a default root
'AND' node upon which new nodes may be added.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

Stamping upgrade script for Vandelay on PG10

Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP1820339: Vandelay Imports on Pg 10

The previous commit used a function that only exists in Pg 10, so it
would require everyone to upgrade PostgreSQL.

After some investigation, Ben Shum and I found a method that works on
Pg 9.4 through Pg 10 by putting the existing function into a
subselect as suggested by PostgreSQL documentation.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP1820339: Vandelay Imports on Pg 10

In vandelay.flatten_marc_hstore there are a couple
instances of set-returning functions used inside a
CASE statement, which Pg10 is unhappy about. This
branch changes these regexp_matches calls to use
regexp_match instead which avoids this error.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP1806394: Placing Holds in Item Buckets

A missing return prevents a list of item ids from
being built so no holds can be placed from item
buckets. This branch returns the return to its
proper place.

To test
-------
[1] Add some items to an item bucket, ensuring that they are
    all holdable.
[2] From the item bucket interface, select the items and choose
    the Request Selected Items action.
[3] Confirm placing the requests.
[4] Note that the requests are not actually created.
[5] Apply the patch and repeat steps #1-#4. This time, the requests
    should be placed.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Millissa Macomber <millissam@burlingtonwa.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1826890: set page title for Statistical Popularity Badges admin page

To test
-------
[1] In the browser client, navigate to Local Administration -> Statistical
    Popularity Badges
[2] Note that the web browser's tab header doesn't display a title for the
    page, just the URL.
[3] Apply the patch and reload the page. This time, a page title should
    be dispayed in the browser's tab header.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1792188: Record -> View Holds fails to refresh on Next/Prev record

Here we clean up and add back a call to refresh the holds list when navigating
between records in a result set.

To test
-------
[1] In the web staff client, perform a catalog search, bring up a record,
    and change to the View Holds tab.
[2] Click the "Next" button to navigate to the next hold. Note that
    the list of holds doesn't change.
[3] Apply the patch, refresh, and repeat steps #1 and #2. This time,
    the list of holds should get updated as you navigate between
    records in the results list.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: John Yorio <jyorio@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1803729 action_trigger_aggregator.pl help indicates the wrong bash argument for remote-acct

Fixed the help message action_trigger_aggregator.pl to agree with the code

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1782014 Fix grid path for Title in patron items out

When the Items Out grid was updated to use display fields, the
displayed title was updated (along with the fleshed data in the
javascript) but the grid "path" was not. This commit fixes the path,
which fixes title sorting. (Title sorting broke because the
simple_record.title was no longer being fleshed in via the JS code, so
the grid was attempting to sort on an empty value.)

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

Lp 1752968: Select content when focusing on the barcode field.

To reproduce the reported behavior:

 1. Find a title in the staff client catalog.
 2. Click on Add Holdings.
 3. See that the Barcode field does not have cursor focus.
 4. Repeat step 1.
 5. Click on edit next to one of the barcoded items in the list of
    holdings.
 6. Click Show Holdings Details and see that the Barcode field does
     not have focus and the content is not selected.

To test this branch:

 1. Find a title in the staff client catalog.
 2. Click on Add Holdings.
 3. See that the Barcode field has cursor focus, i.e. the cursor is in
    the field and blinking.
 4. Repeat step 1.
 5. Click on edit next to one of the barcoded items in the list of
    holdings.
 6. Click Show Holdings Details and see that the Barcode field has
     cursor focus and the content is selected/highlighted.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Janet Schrader <jschrader@cwmars.org>

Lp 1752968: Focus Barcode Field in Volumes/Copy Editor

Always focus the barcode field when adding or editing volumes or
copies.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Janet Schrader <jschrader@cwmars.org>

LP 1333893: Keep Empty Bib on Volume Transfer

Volume transfer did not honor the cat.bib_keep_on_empty organizational
unit setting when removing empty bib records.  This commit changes it
so that tranfer honors the setting in the same manner that deleting
copies and volumes does.

To test this fix, set cat.bib_keep_on_empty for your workstation
org. unit to true, and then transfer all of the volumes from one bib
record to another.  The empty bib record should not get deleted.

You can also test with the setting on false to make sure that the
empty bib gets deleted.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Beth Willis <willis@noblenet.org>

LP#1814943: (follow-up) remove ersatz ctx.page_type line

Guessing that the original bug was due to "page_type" being
entered when "page_title" was meant.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1814943 Closed Dates Editor Tab Missing Title

Small fix to add missing page title.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Rogan Hamby <rogan.hamby@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1823387: avoid double slashes in booking URLs, which will not work with recent versions of Apache

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1642036 Group member detail grid query repair

Teach the grid to automatically use the current version of the grid
query in group member details lookup, since the query changes as
patron's change and is sometimes applied after the grid has already
absorbed the setQuery function.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

Docs: Fix TIP block and list formatting

This improves the formatting of the docs section that Katie Martin just
improved.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Updated checkin modifiers

Updated checkin modifiers to include Manual Float Active and Update
Inventory and their descriptions. For 3.3 only.

Signed-off-by: Katie G. Martin <bcl@blaircountylibraries.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP1793005 Stamping upgrade / print settings as workstation settings

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

LP#1793005 Angular6 Hatch support / printer settings

Adds support for Hatch print communication to the Angular(6) app.
Migrates the print preferences settings (eg.print.config.*) from
in-Hatch settings to server-stored workstation settings.

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

LP#1778940: stamp schema update

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1778940: (follow-up) create indexes concurrently during upgrade

This reduces potential for locking when run in a production
database, particularly if done outside of a full version upgrade.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1778940: Add Indexes to ate.*_output

When there are many rows in action_trigger.event_output
simply ensuring referencial integrity can take an
unacceptably long time as Postgres has to make sure
that the output being deleted isn't referenced in
any of these 3 fields. Adding these indexes keeps
these checks always fast.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1825578: reverse direction of sort-order arrows in eg-grid

This patch adjusts the arrow icons used to indicate if a column
in the Angular eg-grid is sorted to use the following convention:

  sort order ascending == display _up_ arrow
  sort order descending == display _descending_ arrow

To test
-------
[1] Apply the patch.
[2] In a grid with a sortable column (as can be found in /eg2/en-US/staff/sandbox),
    click on the heading to sort it. Verify that ascending sort displays
    the upwards arrow and descending sort the downwards.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

Trivial change to file header comment

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Stamping upgrade script for holds prefetch setting

Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1818288 WS Option to pre-fetch record holds

Adds a workstation setting allowing staff to decide whether to pre-fetch
all holds on the record detail holds tab, to perform sorting paging in
the client, or to leave the sorting/paging on the server.

Improves client-side sorting in the grid.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1818288 Grid checkboxes emit events

Modify the grid toolbar checkbox onClick handler to emit events instead
of requesting a reference to a function.  This is more consistent with
other event-handling code.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1818288 Release notes - record holds tab

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1818288 Ang staff catalog record detail holds tab/actions

Adds support for the Holds tab in the record detail view of the Angular
staff catalog.  Includes grid and hold-related actions.

* Holds grid built from the new wide-holds API.
* batch cancel holds
* batch retarget holds
* batch edit holds
** Unified form to modify notify options, dates, etc.
* hold detail page (menu and row double-click)
* batch mark items damaged
* batch mark items missing
* show last few circulations
* retrieve patron
* support for indented menu groups a la angjs grids for grouping the
  hold actions menu.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Docs: fix uneven lengths of code delimiter blocks

The lengths of the starting and ending sets of hyphens are supposed to be the
same. Asciidoc tools like asciidoctor can be less forgiving than the original
asciidoc processor, with less than hilarious results.

It would probably make sense to try to standardize on something like 60 hyphens
to easily flag this problem for deviations from the norm, but for now here's
the result of running the docs through asciidoctor and manually adjusting the
output.

Signed-off-by: Dan Scott <dan@coffeecode.net>

LP1821196: Remove arrow function from item status

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

Forward port 3.3.0 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Docs: Point to release notes for new version 3.3

Signed-off-by: Remington Steed <rjs7@calvin.edu>

lp1145213 avoid race condition in bib merge test

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

LP1744386: Alphabetize stat cats in volcopy attr editor

To test:
1. Go to the volume copy editor.  Confirm that the stat
cats do not display in order of owning library or stat cat
name.
2. Apply this commit.
3. Go to the volume copy editor.  Confirm that the stat
cats now display in alphabetical order by owning library,
then stat cat name.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

LP#1666987: Include a user-agent to build the relator map

loc.gov started using Cloudflare protection to block HTTP access from default
user agents, so identify ourselves as an Evergreen user agent to restore our
access.

Long term we should evolve to using the linked open data access method
via id.loc.gov but let's get this working first.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Forward-port 3.1.11 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Translation updates - po files

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Translation updates - newpot

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Forward-port 3.2.5 upgrade script

Signed-off-by: Dan Wells <dbw2@calvin.edu>

Coerce numbers for bib IDs in Angular staff catalog

Under some circumstances, the search results screen in the new
experimental Angular staff catalog did not display any results (though
it did show the pager, facets, etc.).

This was traced back to the results array attempting to be populated
with ID values of '-1' instead of the real IDs.  Coercing the incoming
IDs to numbers appears to resolve the issue.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Fix Asciidoc levels in 3.3 release notes

Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1821067 Angular i18n uses XMB; cs-CZ examples

Includes documentation updates and cs-CZ example using export cs-CZ .xtb
translation bundle.

Also remove xmliffmerge configuration options since we are not using
xliff files.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1821067 Angular Czech translation bundle

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Sandbox editor handles dismissals

Teach the sandbox FM editor example to log a useful message on dialog
dismissal instead of throwing an error (as a result of the uncaught
rejection).

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Allow Combobox to default to field id

If a combobox field is provided without a label, use the id value as the
display value.  This helps avoid ugly JS errors while trying to format
a null string.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Admin grids preload combobox values

Adds a global option to the fieldmapper editor component to preload
linked selector values by default.  Update the basic admin pages to use
the new flag to preload comboboxes.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Basic admin page readonlyFields repair

Fix a bug in the basic admin page component that prevented the readOnly
fields from successfully propagating to the fieldmapper editor.

To Test:
1. Navigate to /eg2/en-US/staff/admin/booking/resource_type
2. Click "New Resource Type"
3. Confirm the "Bibliographic Record" field is disabled.

Includes ng-lint repair as well.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Angular fm-editor uses combobox

* Linked field options traditionally rendered via <select> are now
  rendered with an eg-combobox.
* Caller has option to force a combobox to preload values or rely solely
  on typehead.
* Caller has option to provide a canned list of combobox values.
* Caller has option to set / override which field on the linked class
  should searched by the typeahead
* General improvements to fm-editor form building and field options
  management.
* Includes Angular7 rxjs import repairs

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1811288 Combobox support entrylist+async / id labels

Allow the caller to pass a seed entrylist value for async comboboxes.
This is useful when a value should be applied to the box on load instead
of waiting for user input for typeahead loading.

Allow combobox entries to default to using the 'id' field for the label
if no label is provided.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1821409 Ang admin editor clears fields on new

The Angular admin pages re-use an FmEditor component for creating and
updating objects.  When creating a new object after editing an existing
object, reset the editor so it won't load the previously edited object.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1797934: follow-up: make the reservations tab in MyOpac optional

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Docs: finalizing 3.3 release notes and related documentation

LP1819179 PCRUD selector fleshing handles maps

Teach the PCUD selector fleshing code to handle selector fields on
mapped classes, where an intermediate object flesh is performed by pcrud
at flesh time.

Adds a 'selector' column on metabib.metarecord in the IDL so that we can
have a functioning example of this to use in the sandbox test code.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1819179 IDL2js includes 'map' attribute data

Patch from Mike Rylander to teach the IDL2js generator to include 'map'
attributes, which allow JS clients to step through mapped, linked
fields, particuarly useful for auto-fleshing these fields.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1819179: Angular value formatter gets link smarts

Teach the FormatService to display selector values for linked objects
when the requested field is a link field and it contains an object value
instead of just a id/key value.

Improve IDL data passing from the grid to the IDL service so it can
better determine which fields are avialble for link selector display.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1812670 Angular grid shows selector labels

* Teach PcrudService how to flesh link fields when a selector is defined
  on the linked class. This uses a new search/retrieve API flag
  {fleshSelectors:true}.

* Teach the grid how to render selector values when configured to do so
  via a new grid component attribute [showLinkSelectors]="true".

* Teach the Angular staff admin page to request linked selectors from
  pcrud and tell its grid to expect them.

* Adds utility function to IdlServer for finding the selector for a
  given class + field.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1807461 Admin page avoid errors on dialog dismissal

Add support to the base DialogComponent class for passing information to
the caller via the reject handler about whether a dialog was dismissed
via user interface interaction (body click, esc key, cross click, cancel
button) or for some other reason, presumably an error.

Teach the generic admin page to avoid toasting errors when an edit or
create dialog is dismissed via UI.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1807458 Eg2 grid Edit option lint repairs

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1807461 FM-editor show danger toasts when create/update fails

Display a danger toast when update or create attempts fail in the
Angular fieldmapper editor dialog.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1807458 Angular admin grid Edit Selected option

Adds a new "Edit Selected" action to the "Actions for Selected Items"
menu in the general purpose admin grid.  This only visibly affects the
ACQ admin grids at time of writing, but applies to all auto-generated
admin grids.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Docs: more improvements to 3.3 release notes and related docs

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

Docs: Revising 3.3 release notes and related docs

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

docs: Statcat updates

Signed-off-by: abneiman <abneiman@equinoxinitiative.org>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>

Docs: Add bugfix release notes for 3.2.5

For the acknowledgements section, I opted to include anyone who had
signed off on a commit in 3.2.5. I'm not sure what the criteria were
for pervious point release notes.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

Docs: Add bugfix release notes for 3.1.11

For the acknowledgements section, I opted to include anyone who had
signed off on a commit in 3.1.11. Not sure if this is the same as past
point release notes.

Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP1776736 Record merge marc edit repairs

1. Fixes issues where edits to MARC records would fail to save during
   merge.

2. Provide options to select flat or non-flat text editor in edit mode
   for record merges.

3. Allow record editor action buttons/selectors to flow down
   horizontally when they don't fit in a narrow space.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP#1780639: fix error made when stamping DB update

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1709698: fix error made when stamping DB update

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1749502 - Holds Pull List Print Order

- Expose call number affix sortkey data.
- Use sortkey data to sort default pull list template.
- Combine all call number info into one field in default pull list template.
- Sorty by shelf location position if it exists, then by shelf location name if it doesn't
- Only grab copy info if copy is assigned, to not break patron holds grid.
- Remove some web console logging only needed for testing.

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: John Amundson <jamundson@cwmars.org>

LP 1749502: Add Call Number Prefix & Suffix to pull list print template.

We add the call number prefix and suffix as separate columns to the
holds pull list print template.  We also add the suffix to the end of
the orderBy so that we also sort by the suffix.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>

lp1749502 Prefix patch

- Ensure prefix information is properly set

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/share/print_templates/t_hold_pull_list.tt2
modified:   Open-ILS/web/js/ui/default/staff/circ/services/holds.js

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>

lp1749502 Holds Pull List Print Order

- Ensure copy shelf location position is available for each hold on
holds shelf.
- Add _copy_location_position to hold_data while printing.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/share/print_templates/t_hold_pull_list.tt2
modified:   Open-ILS/web/js/ui/default/staff/circ/services/holds.js

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>

LP1739293 Record merged edit-in-place avoid wrap

When using the inline editor, prevent text wrapping and instead allow
the text to overflow x and y and use scrolls to navigate.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Elaine Hardy <ehardy@georgialibraries.org>

LP1739293 Record merge fits container

Allow the text contents of each merged MARC record to wrap so the
records may fit horizontally on the screen.  A single vertical scroll
allows the user to scroll down to see any parts of the record that
require more vertical space.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Elaine Hardy <ehardy@georgialibraries.org>

LP1739293 Record merge horizontal; optional holdings

* Tile bib records horizontally in record bucket => merge records
  interface.

* Add a "Toggle Holdings View" button for displaying holdings and leave
  holdings un-displayed by default.

* Duplicate the Merge and Cancel buttons along the top of the dialog.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Elaine Hardy <ehardy@georgialibraries.org>

LP 1819796: Fix method call on undefined value in generate_fines

The below message repeats in the open-ils.storage_stderr.log:

Caught error from 'run' method: Can't call method "search_where" on an
undefined value at
/usr/local/share/perl/5.26.1/OpenILS/Application/Storage/Publisher/action.pm
line 1014.

This commit removes an errant ->search_where on a CStoreEditor call to
make that message go away.

Also, delete some lines of code that were commented out at the time the
code was converted to use CStoreEditor.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1793196 Fix console error when none selected

If you open the dropdown and close it without selecting anything, the
console shows an error complaining that $scope.selected is undefined.
This commit checks that it exists in that case before using it.

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1793196 Volume batch editor offers MARC call numbers

Provide call numbers extracted from the MARC record as options in the
volume batch editor.  Call numbers to extract are based on the load-time
classification scheme, defaulting to Generic ('1') if no default scheme
is applied.

Patch includes additional sanity checks on volume editor utility
functions to avoid console errors referencing undefined values.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1665534: Make pickup library in bib record holds view sticky

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>