LP#1979071: add release notes

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

LP#1979071: (follow-up) fix various DB schema and update issues

* Incorporate the delayed reification option for search dictionary
  updates derived from authority records (see bug 1997485)
* Fixed an issue that prevented propagation of changes to main
  headings
* Fixed an issue introduced in bug 1977485 that prevented
  authority headings from being deleted
* Remove now-disused authority.indexing_ingest_or_delete()
* Remove an older version of authority.propagate_changes()
  from the base schema; this would be an issue in new databases
  installed from scratch.

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

LP#1979071: (follow-up) install ingest_ctl by default

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

LP1979071 Fix column/value name ambiguity

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

LP#1979071: Queued Ingest

This feature allows for the separation of bib and authority record
updates and the search (and other) indexing that occurs when a record is
modified in some way.

The Queued Ingest mechanism consists of several parts working together:
 * A set of configuration flags that control when ingest should be
   performed immediately, and when it can be deferred until after the
   transaction commits and control is returned to the user.
 * Refactoring of the in-database ingest triggers to separate deciding
   what should happen to a record given a data modification event, and
   when/how that process should take place.
 * A set of queuing tables used to track which records are to be
   processed and in what ways, when that processing was requested, and
   the ability to group processing requests into named queues that can
   report who made a processing request and for what purpose.
 * A Queued Ingest Coordinator that runs in the background monitoring
   the queuing tables for activity and processes records as they are
   enqueued.  This can run on any server that can connect to the
   database and has the OpenSRF Perl modules installed.
 * A command line tool to be usedby administrators to enqueue records
   for Queued Ingest processing, to create named queues, and to process
   enqueued records either in one queue or all outstanding enqueued
   entries.  This tool can also report on the status of requested Queued
   Ingest processing, whether pending, ongoing, or complete, either for
   all time or since a particular date and time.

The queuing tables added here are not yet included in the IDL for
reporting or Staff interface construction.  They can be added as the
need arises.

To test Queued Ingest:
 * Upgrade the database schema using the appropriate upgrade script.
 * Install OpenILS/src/support-scripts/ingest_ctl in the usual OpenSRF
   bin directory.
 * Start the Queued Ingest Coordinator by using the --coordinator
   parameter to ingest_ctl along with the relevant database parameters.
   Use the --help parameter for more details.
 * Enable the Global Flag called 'ingest.queued.all'
 * Edit, import, and delete some bib and authority records, and use the
   --stats parameter to ingest_ctl to see entries being processed.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Updating fonts and sounds settings

No LP, DIG revamp project.

Changing to Disable Sounds

Fixing typo.

Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP1615707 ARIA landmarks for staff interface

Adds the following ARIA landmarks and roles to the Angular staff
interface:

* <main> and role="main" for the content container
* <nav> and role="navigation" for the navbar
* role="form" for the catalog search form (which lacks a <form> tag)
* role="search" for the search tab panel inside the form
* type="search" and role="searchbox" for the search term input field
* a custom region for the bib record summary box
* <aside> and role="complementary" for the facet sidebar

To construct the ARIA label for the bib record summary, I've added an ID
to the header row of the bib summary component so the screen reader can
use the localized text.

Since the staff interface doesn't really have a header section other
than the navigation, and there is no footer, I've left out these
landmarks.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxOLI.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP#1977554: (follow-up) fix delayed dictionary reification

This patch fixes a regression introduced by the authority heading
update patches in this branch and ensures that updates to
authority records participate in the option to delay
suggestion dictionary reification.

Co-authored-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1977554: (follow-up) include release notes entry

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

LP#1977554: (follow-up) tag new strings for translations

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

LP#1977554: (follow-up) style tweaks

Deal with issues introduced by a merge conflict resolution
error and the Bootstrap 5 upgrade.

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

LP#1977554: (follow-up) remove "switch" role

... not needed for these buttons

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1977554 - Add Password visibility toggle on login screens

Changed the <span> tags to <button> tags so it can be reached via
keyboard navigation.

Added a few more attributes to the password input tag.
autocapitalize="none"
spellcheck="false"
aria-description so screen readers can describe to the user what
the current state is.

Added aria-checked to the button for screen readers

Added a new css file for the login.component with a little styling.

Signed-off-by: Scott Angel <scottangel@mobiusconsortium.org>
Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1977554 - Add Password visibility toggle on login screens

Added an icon to each login input of a closed eye when input type is set to password.
If you click the icon it switches to an open eye and the input type is set to text.
It's a pretty standard way to allow people to view their password before submitting it.

Signed-off-by: Scott <scottangel@mobiusconsortium.org>
Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1977554 - Add Password visibility toggle on login screens

Added an icon to each login input of a closed eye when input type is set to password.
If you click the icon it switches to an open eye and the input type is set to text.
It's a pretty standard way to allow people to view their password before submitting it.

Signed-off-by: Scott Angel <scottangel@mobiusconsortium.org>
Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1967328: add release notes entry

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

LP#1967328: (follow-up) tweak layout

The modal to add permission maps is now wider and uses
a bit less vertical space, allowing the user to see more
of the pending mappings at once.

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

LP#1967328 Add multiple new permission group mappings at once

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: Susan Morrison <smorrison@georgialibraries.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1997485: stamp database update

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

LP#1997485: (follow-up) flesh out reingest intructions

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

LP#1997485: (follow-up) fixes to the authority suggestions updates

* Fix a long-standing issue where a stored procedure used in
  search suggestion dictionary generation was not included in
  baseline.
* Fix an error in direct reingest query.

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

LP#1997485: (follow-up) fix propagation of settings

Ensures if the search suggestions library settings have not been
set, that the attempt to copy them over to the search classes
will not display error messages.

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

LP#1997485: (follow-up) update release notes

* Reflect change to how authority-derived suggestions
  are generated.
* Advise Evergreen administrator to review the search
  class settings.

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

LP#1997485: Include authority headings in symspell dict

This patch adjusts the generation of search suggestions
based on authority headings by:

* Including all indexed authority headings (1XX, 4XX, and 5XX) from
  authority records for author, title, and subject headings in the
  search suggestion dictionary.
* As a consequence, suggestions can be based entirely on headings
  found in authority 4XX and 5XX fields without those headings
  also needing to be directly in at least one bibliographic record.

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

LP#1997485: Release notes for multi-term Did You Mean

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

LP#1997485: Reify baseline schema

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

LP#1997485: Multi-term Did You Mean

Expanding on the previous single-class, single-term search suggestion
development, this feature provides suggestions for single-class searches
with multiple terms.

 * The Library Settings that were previously used to control the global
behavior of search suggestions have been moved to search class
configuration fields.  This was done because the data in each search
class benefits from different setting values.

 * If a patron's search brings back a suggestion that matches an authority
variant heading, the system will provide the main heading as a suggestion
as well, along with spelling-corrected suggestions.

 * Quoated phrases in user input require strict term order and adjacency
for the phrase portion of the suggestion generated for the phrase(s),
whereas unquoted input (or the portion that is not quoted) does not.

MARC Search/Facet Class (config.metabib_class) field additions:

 * variant_authority_suggestion   Whether this class should attempt variant authority suggestions based on search-class/browse-axis mapping
 * symspell_transfer_case         Whether suggestions should retain user-supplied letter case
 * symspell_skip_correct          Only supply suggestions to misspelled words
 * symspell_suggestion_verbosity  Setting that controls the amount of effort, and therefore time, spent on suggestion generation
 * max_phrase_edit_distance       Maximum average per-word edit distance when evaluating suggestions
 * suggestion_word_option_count   Maximum alternate suggestions per word
 * max_suggestions                Maximum suggstions to present
 * low_result_threshold           Maximum hit count beyond which suggestions are not provided
 * min_suggestion_use_threshold   Minimum number of times a suggestion must exist in the corpus
 * pg_trgm_weight                 Weight of the trigram similarity metric; 0 avoids calculation costs
 * soundex_weight                 Weight of the soundex similarity metric; 0 avoids calculation costs
 * keyboard_distance_weight       Weight of the keyboard distance similarity metric; 0 avoids calculation costs

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ruth Frasur <rfrasur@library.in.gov>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#2000482: add a brief release notes entry

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

LP#2000482: (follow-up) couple more layout tweaks

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

LP#2000482: (follow-up) update Angular cheat sheet

... since `ng build --prod` no longer works.

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

LP2000482: Get tests passing

* Update karma config
* Update Patron Barcode Validator directive due to a breaking change in rxjs single()
* Update test assertion to match new ar-JO localization

Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP2000482: update tsconfig options to satisfy compiler warning

Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP2000482 Bootstrap 5 continued

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP2000482 Angular 15 and Bootstrap 5 upgrade

Includes numerous markup and CSS adjustments to cope with Bootstrap 5.

This patch is a squash of patches authored by Bill Erickson
and Stephanie Leary.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1791791: remove a regression regression on bug 1923225

This was introduced on the Bootstrap side by the patch for
bug 1955403.

To test:

[0] This applies to the Bootstrap OPAC skin.
[1] Perform a search on ISBN.
[2] Note that on the record page, the ISBN (under more details)
    is not properly highlighted.
[3] Apply the patch and repeat step 1. This time, the ISBN
    should be highlighted.

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

LP#1791791: prevent regression on bug 1923225

Ensures that when doing an ISBN search, the ISBN on the record
page is properly highlighted.

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

LP1791791: Google book previews not displaying from a bib linked from a search.

The javascript that queries Google Books Preview looks for ISBN(s) wrapped in
a rdetail_value class.  There was a line of code in which the isbn variable
 was not enclosed in this class.  This patch just wraps that stray variable
into a <span> with the class.

To test:
0. Use the TPAC skin - this bug does not affect the Bootstrap skin
1. Perform a keyword search for an bib that should have a google book's link.
   As of March 2023, ISBN 9780786496570 has previews enabled.
2. Go to the bib display and notice that there is no link.
3. Remove the ';query=something' text from the url and notice that the
google book information displays.
4. Apply the patch.
5. Perform the same search.  The google book information should now display.

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

LP#1863387: multi-select now allows filtering shelving locations by owner

The Angular multi-select component now has a special case for
shelving locations: when the IDL class of "acpl" is selected,
rather than just displaying a combobox, the item-location-select
component followed by an org selector is displayed and checkbox.

The org selector defaults to workstation OU and is used to restrict
the list of shelving locations displayed in the shelving location
combobox to the context org unit and its ancestors. If the checkbox
is also selected, descendants of the context OU are included as well.

The effect of this is to allow large consortial to more efficiently
select the shelving locations to be used by a carousel.

To test
-------
[1] Apply the patch.
[2] Create or edit carousel definitions. Verify that the widget
    for the carousel's shelving locations now displays both a
    combobox for the location selector as well as one for the
    location owning library. Further verify that when the OU
    selector for the owning library is changed, that the list
    of available shelving locations reflects the locations available
    at the ancestors of the filter OU. Also verify that the
    "Include descendants?" checkbox updates the list of available
    locations as well.

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

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

Docs: updates to record buckets

Signed-off-by: Britta Dorsey <bdorsey@library.in.gov>
Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxinitiative.org>

Docs: adding Angular Staff View docs for 3.11

Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxinitiative.org>

LP#1970946: stamp schema update

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

LP1970946 Adjust color contrast in result highlights

Lowers the contrast on search result match highlights in the staff
catalog, and changes the highlight tag from <b> (PostgreSQL's default),
to <mark> (new in HTML5).

As an additional consequence, the color contrast for highlighted
search terms in the Bootstrap OPAC (using default styles) improves
from 4.38 to 17.12.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP2009865 Revised search result headings & source order

Revises the hierarchy of headings in the search results. In combination
with the H1 provided in in bug #1994711, the result is now:

h1. Staff Catalog
h2. Search Results (N)
(repeat for each result:)
h3. [Title] [Author]
h2. Facets
(repeat for each facet box:)
h3. [Facet title]

The source order of the results list and facet sidebar have been
swapped. The facets still appear on the left visually, but now fall
after the results in the document.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1994711 Restore <eg-title> to staff banner

Restores <eg-title> to staff-banner.component.html for pages that don't
have their titles set by any other method.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1994711 Semantic H1 headings for page titles

Patch alters the markup within <eg-staff-banner> to include H1 headings
for page titles.

The patch also includes a new CSS file for the course page component,
which appears to be the only component taking advantage of the custom
classes and icons for page titles. I've made archived course titles gray
and italicized.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1929593 stamp upgrade script

Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1929593 (follow-up) address make livecheck warning

Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1929593 UPDATE_COPY_BARCODE permission

This adds the permission UPDATE_COPY_BARCODE and a new API call,

  open-ils.cat.update_copy_barcode

which explicitly tests for both UPDATE_COPY_BARCODE and UPDATE_COPY,
with either being sufficient for allowing a barcode change.  Existing
Replace Barcode UI's in both Angular and AngularJS have been modified
to use this API call instead of the pcrud service.  One side-effect of
this has been better surfacing of errors, as errors in pcrud were
uncaught and bypassing the normal error handling.  This addresses
LP1951469.

The upgrade script gives any permission groups that already have the
UPDATE_COPY permission the new UPDATE_COPY_BARCODE permission at the
same depth, though it's technically not needed.

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Ruth Frasur <rfrasur@library.in.gov>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1996651: treat empty string as null for preferred name/alias in wide_hold_data

This prevents the patron name from appearing blank in the hold shelf
"User Display Name" or "User Alias or Display Name" columns when the
alias or preferred name fields are empty strings.

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>
Signed-off-by: Susan Morrison <smorrison@georgialibraries.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP2016337 Refresh grids in booking return tabs

Loads missing tab contents in booking return by patron screens.

Co-authored-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP2016341 Fix duplicate ID in staff catalog record

Gives the actions button area a different ID than the navigation.

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxoli.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1889916 SMS Carrier does not include region in borrower edit screen

Displays the region next to the carrier name and in the borrower edit and
create screens.  Also sorts the carrier names alphabetically.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1716479: (follow-up) fix handling of onSave callbacks for AngularJS MARC edit

This patch fixes an issue that has been present for a while but
clarified by Beth Wills in the course of testing the base patch
for this bug.

Specifically, the routine to process onSave callbacks was
not bound to the scope properly, meaning that the wrong onSave
callbacks could be run when dealing with mutiple active
egMarcEditRecords.

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

LP1716479: Make sure authority linker works in embedded MARC editors

1) Perform a z39.50 search
2) Select a record
3) Click Edit then Import
4) Click on the link next to any authorizable field.
5) You will see a mysteriously data-free heading: {{bibField.tag}} {{bibField.ind1}}{{bibField.ind2}}
6) Clicking on the "Immediately" and "Create and edit" buttons doesn't work.
7) Apply this patch and repeat steps 1-6.
8) Note that the heading is now correct, and the buttons work.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Jennifer Weston <jennifer.weston@equinoxOLI.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Docs: More README Cleanup

Fix "Installing PostgreSQL server packages" heading.  It was showing
up as a list entry.

Drop the text about installing additional packages for the database,
since there are none to install at this time.  (We could replace that
with a subsection on the packages required for a standalong databse
server without all of Evergreen installed on it.)

Add a "Create the Evergreen PostgreSQL user" heading above the
instructions to creat the evergreen user in the database.

Co-authored-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>

Docs: Remove extraneous blank line from README introduced by LP1947728

When processing the README with asciidoc the following message appears:

asciidoc: WARNING: README: line 82: list item index: expected 1 got 3

This is caused by a blank line on line 74 introduced in the
removal of the Debian Stretch prerequisites.

Co-authored-by: Ben Shum <ben@evergreener.net>
Signed-off-by: Jason Stephenson <jason@sigio.com>

docs: (follow-up) tweak OpenAthens doc

- fix a warning for a NOTE block
- link to the navigation

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

Add files via upload

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

Docs: Configuring sign-on to OpenAthens

Adding new page for OpenAthens configuration. Used documentation from 3.10 release notes, with the addition of a couple screenshots.

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

LP1970476: Restore Localization Function Wrapper

The previous commit removed the l() function wrapper from two bits of
text when it probably should not have.  This commiet restores it.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1970476 Where filter in Bootstrap Catalog Produces Unwanted Results

Prior to this change, the Advanced Search page split the library selector
and the "Where" scope selector into two separate dropdown lists that
produced unexpected results when both were used.

This change uses the same approach as the combined dropdown on the Basic
search page instead.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1965797: Add Acq Lineitem Detail Audit Table

Add SQL to create an audit table for acq.lineitem_detail in the base
schema creation.

The table is not added during a database upgrade.  The release notes
include instructions on creating the new table.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1981774: (follow-up) fix a straggling "opac-button"

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

LP#1981774: use Bootstrap CSS classes for buttons instead of opac-button

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1981774: fix button CSS on curbside in Bootstrap OPAC

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1752334: (follow-up) adjust to allow "0" as a note

Also ignore the additional note parameter if all it has
are zero or more whitespace characters.

It's unlikely that a client would want to set the additional
note to "0", but it's nonetheless a good idea to distinguish
between difference versions of Perl's false value.

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

LP#1752334 - BadContact, use additional note if provided.

If an additional note is provided when marking a contact invalid,
then that note should be appended to the penalty note field.

This change adds that functionality.  This doesn't change the results
for staff that are using the invalidate buttons in the client, since
there isn't a chance to add an additional note from that interface.

But if someone is calling open-ils.actor.invalidate.email via some other
method, then you can specify the additional note.

I would like to be able to add extra notes when automatically processing
bounced emails, to give staff a clue as to why the email failed.  Such as
"<email> Account over quota for past 3 weeks, ask patron to check their account."

Includes a new perl live test to check the new behavior, along with all
the open-ils.actor.invalidate.* call options.

Testing plan:

Before fix.
1. Choose a patron with an email address and note their patron id number.
2. Run an srfsh shell.
3. Login to get an auth token 'login username password'
4. Run the invalidate email command to mark the users email invalid, including
   an addition note value.
  srfsh# request open-ils.actor open-ils.actor.invalidate.email "<authtoken>" "<patronid>" "Testing Additional Note"
5. Take a look at the patron account and see that the penalty message only
   includes a copy of the email address that was marked invalid.

After fix.
1. Restart the opensrf actor service if needed to re-read the changed perl module.
  osrf_control -l --service open-ils.actor --restart
2. Remove the penalty and re-add email address to test patron's account.
3. Run an srfsh shell.
4. Login to get an auth token 'login username password'
5. Run the invalidate email command to mark the users email invalid,
   including an addition note value.
  srfsh# request open-ils.actor open-ils.actor.invalidate.email "<authtoken>" "<patronid>" "Testing Additional Note"
6. Take a look at the patron account and see that the penalty message note
   now includes the additional note.

Run the live test at Evergreen/Open-ILS/src/perlmods/live_t
perl ./36-lp1752334-badcontact.t

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP1879517: Surveys shouldn't end before they begin

To test:
1. Go to Admin > Local > Surveys.
2. Create a new survey.
3. Attempt to create a survey where the end date
comes before the start date.  Without this patch, you will get
no notice that this is invalid, and you can save the invalid
record.
4. Edit an existing survey.
5. Repeat step 3 while editing the existing survey.
6. Apply the patch.
7. Repeat steps 1-5.  Note that you now get a notice and
cannot save if the end date is before the start date.

This commit generalizes a validator already present in the booking
module, and corrects several small bugs related to the datetime-select
component.

Signed-off-by: Jane Sandberg <sandbergja@gmail.com>
Signed-off-by: Susan Morrison <smorrison@georgialibraries.org>
Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

LP#1990968: add release notes

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

LP1990968: Move Strict Barcode Checkbox in Angular Staff Client

Move the Strict Barcode Checkbox to the right of the barcode input
from the bottom of the page for the Checkout, Checkin, and Renew
interfaces.

This change differs from that for AngularJS as the latter moves the
checkbox to below the barcode input.

Signed-off-by: Jason Stephenson <jstephenson@cwmars.org>
Signed-off-by: Elizabeth Thomsen <et@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1990968: Move strict barcode checkbox

Moved the 'Strict Barcode' checkbox next to the item barcode input box
for a more streamlined and workflow page layout. This affected three
Evergreen interfaces: Patron checkout, Item Renew, Item Checkin

Signed-off-by: Scott Angel <scottangel@mobiusconsortium.org>
Signed-off-by: Jason Stephenson <jstephenson@cwmars.org>
Signed-off-by: Elizabeth Thomsen <et@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#2007073: add release notes

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

LP#2007073: stamp DB update

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

LP#2007073 Staff Should Only Select Manual Hold Cancel Causes

Adds a field to action.hold_request_cancel_cause to indicate
if they are manual cancel causes.

Filters on this field in cancel hold dialogs to only display
manual cancel causes.

Adds two new cancel causes, Patron via email and Patron via SMS.

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: Susan Morrison <smorrison@georgialibraries.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1964986: encourage distinct results when querying the ahopl IDL source

This patch addresses a now-theoretical issue where a direct PCRUD
query of ahopl could return duplicate rows in cases where the patron
linked to the hold request has multiple penalties attached.

The significance of this change is likely minor nowadays, as ahopl
was directly used by Evergreen only by the Dojo and AngularJS hold
pull lists, and as of 3.8 the Evergreen Angular pull list uses a
different approach for querying the pull list. However, until such
time as ahopl is deprecated and removed, this change may help
local integrations.

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Jennifer Weston <jennifer.weston@equinoxOLI.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#19801356 Making 'username' consistent in the OPAC

Signed-off-by: Gina Monti <gmonti@biblio.org>
Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

LP#1959010: (follow-up) break up long lines in release notes

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

lp1959010 release notes

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Mary Llewellyn <mllewell@biblio.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

lp1959010 have the staff display component check search context for the active search OU

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Mary Llewellyn <mllewell@biblio.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

lp1959010 toward Staff View tab

* link searches for listed genres
* s/Abstract:/Summary:/
* link to View all Formats and Editions
* remove border from staff view
* tweaking what goes where
* add metabib record count next to View all Formats and Editions link
* remove links for what boils down to marc 508 and 511
* made the count off-by-one trying to not make it off-by-one :)
* remove the creator field
* keep just Subject:
* remove the Record is or is not holdable information as it doesn't present the whole truth
* refresh interface on tab change

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Mary Llewellyn <mllewell@biblio.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

lp1959010 CSS and layout tweaks

* More layout and CSS tweaking
* list style fix
* left margin fix
* UL margin fix

Signed-off-by: Stephanie Leary <stephanie.leary@equinoxOLI.org>
Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Mary Llewellyn <mllewell@biblio.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

lp1959010 toward Staff View tab

* make our Staff View tab with the new BibStaffView widget
* seed our BibStaffView with BibSummary
* first attempt at changing some identifiers with our seed code
* experimenting with layout and stripping courses functionality
* simplifying the layout for now and adding display fields
* moar data
* layout tweaks and propagate metabib attributes for Formats and Editions
* don't cross the streams with metabib related
* field names for the record and metabib variants of the catalog_summary call.  This would otherwise break some links in search results involving bibs with metarecords.
* hyperlinks for the formats and editions in the catalog staff view
* fix catalog_summary vs catalog_summary.staff bug
* searchOrg depth was being passed where a boolean was expected
* This seems like a failure of TypeScript to detect.. type.  What was going on here?
* layout and css tweaking
* fixing toc and relocating it and some other fields to column 1
* label fix

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Mary Llewellyn <mllewell@biblio.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1972738: stamp schema update

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

LP#1972738: add an ANALYZE after the index gets created

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

LP1972738: Add index on hopeless date

Add an index on action.hold_request hopeless_date column to speed up
hopeless holds processing.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1369345: de-duplicate ILS events and clean up whitespace

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

remove release notes entries made moot by security point releases

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

forward-port 3.10 upgrade scripts

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

forward-port version upgrade scripts

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

fix title of the 3.10 release notes document

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

Docs: 3.10.1 release notes - edited

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Docs: 3.9.2 release notes - edited

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

update 3.8.3 release notes to include security fix information

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

update 3.9.2 release notes to include security fix information

Also fixes a typo

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

update 3.10.1 release notes to include security fix

Also fixes a typo.

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

Docs: 3.10.1 release notes

Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Docs: 3.9.2 release notes

Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Docs: 3.8.3 release notes

Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

LP#1361782: (follow-up) fix the default search limit in seed data

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

LP#1361782: stamp DB update

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