LP1950166 TPac: Self-registration form won't submit with blank DOB

Adds a check for a blank value in the DOB validation script. Note
that the issue is present in TPAC, not the Bootsstrap OPAC.

To test:
1. Confirm "Require dob field on patron registration" is not set or set to False.
2. Confirm "Show dob field on patron registration" is true.
3. Confirm "Allow Patron Self-Registration" is set to true.
4. Try to register a patron in the TPAC with DOB blank.
5. Apply patch.
6. Try to register a patron in the TPAC with the DOB blank.
7. Try to register a patron with Invalid DOB input.
8. Try to register a patron with valid DOB input.

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

LP1950514: TPac hold groups table displays incorrect column names on small screens.

This fixes the incorrect display of column names in the hold groups table
in the tpac. Previously the table would display names for the hold history
table.

To test:
1. create an opac visible hold group and assign a patron to the group.
2. login to the TPac as the patron assigned to the group.
3. view hold groups and make the browser screen as small as possible.
4. notice on the smallest screen the headers for the table are incorrect.
5. apply the patch and view the headers.

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

LP#1942647: (follow-up) fix lint

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

LP#1851884: eg-fm-record-editor: avoid fetching all rows from linked table

This patch ensures that Angular FM record editor dialogs do
not attempt to fetch all rows from target table when constructing
a combobox for a linked field.

In particular, it makes the following changes:

* If a custom template for a field is supplied, use that by
  default rather than _also_ creating a data source for
  a combobox.
* Rather than creating its own data source for a linked field,
  the FM record editor now uses the IDL mode of eg-combobox. By doing
  this, we use eg-combobox's default data source, which limits
  record retrievals to 100 rows max. Also, empty-click is now
  enabled by default.
* When attempting to identify a selector for an IDL class, if
  the class doesn't define a selector and doesn't have a field named
  'name', but its primary key is a text field, use the primary
  key as the selector.

To test
-------
[1] Create a few thousand empty bib record buckets in your test database.
[2] Edit a carousel under Local Administration; note that it can
    take some time for the dialog to load.
[3] Apply the patch and repeat step 2. This time, the carousel edit modal
    should open instantly.
[4] Test various Angular record editor modals to confirm that they behave
    as expected. For example:

    * The allocate to fund dialog in Fund Administration, in particular
      to verify that only active funds show up.
    * The assign user to course modal, to verify that searching is done
      by course number, not name.
    * Filter Dialog Set editor under /eg2/en-US/staff/admin/local/config/filter_dialog_filter_set
      to verify that the drop down for the creator doesn't fetch
      all patrons. (Note that this is an artificial example).

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Tiffany Little <tlittle@georgialibraries.org>

LP#1945385: use eg-admin-page for circ limit sets admin UI

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Jennifer Pringle <jennifer.pringle@bc.libraries.coop>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1932203: serialize requests on Edit Due Date in Items Out tab

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>
Signed-off-by: Jason Stephenson <jason@sigio.com>

lp1942647 Provide Warning when deleting Term linked to Courses

- Separates out Course Term Grid to its own component
- Adds retrieval service function for Course/Term Maps based on Term ID
- Displays Confirmation dialog when deleting terms on the term grid if term is mapped to a course

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-list.component.html
modified:   Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-reserves.module.ts
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-term-grid.component.html
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/course-reserves/course-term-grid.component.ts
modified:   Open-ILS/src/eg2/src/app/staff/share/course.service.ts

Signed-off-by: Beth Willis <willis@noblenet.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

Docs: fix display error

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

Hold Management page update

Boopac screenshots

OPAC update

Boopac screenshots for hold notifications

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

LP1950345-Format the Current Hold Groups table in bootstrap opac

This patch styles the Current Hold Groups table in bootstrap to be consistent
with other tables in the bootstrap catalog.  It fixes overlappling text
in cells.  It also fixes the column headings for small screens.  Previously
the headings on small screens were pulling from the css rules for the
holds history table.

To test:
1. create a couple of hold groups and add a patron to each group.  Make the
groups opac visible.
2. login to the opac as the patron to view the holds groups to which the
patron belongs.  Notice the table style with not margins or padding.
3. make your browser as small as possible to see the incorrect headings.
4. apply the patch.
5. repeat 2 and 3 to view the fix.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Gina Monti <gmonti@biblio.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP#1930617: reduce parallel requests initiated by AngularJS holdings editor

This patch decreases the number of parallel requests spawned by the
AngularJS holdings editor. This can reduce load on open-ils.pcrud when
the editor is used to handle a large number of items in one batch.

This patch takes the following approaches:

- the fetching of call number affixes now only makes one request
  per relevant org unit
- item alerts are now retrieved via fleshing when the batch of items
  is requested (as opposed to doing a PCRUD request for each copy)
- fetching monograph parts is now done serially

To test
-------
[1] Apply the patch.
[2] Create a bucket with a few hundred items and edit all of them
    in the AngularJS holdings editor (which is what you get when
    you edit items from the bucket interface).
[3] Verify that the editing loads itself without causing errors
    or PCRUD drone spikes.
[4] Verify that item alerts and monograph parts are loaded
[5] Verify that the call number prefix and suffix drop-downs
    are correctly populated.
[6] Verify that copy alerts can be attached to newly-created items
    and saved.

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

Docs: Alternate Notice Templates documentation

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

Docs: Cover Image Upload documentation

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

LP#1974195: fix memory leak when performing fleshed Fieldmapper search

This patch fixes a memory leak that occurs when performing
a fleshed search of an IDL class via open-ils.cstore, open-ils.pcrud,
or open-ils.rstore. It removes a debug-level log entry that didn't
free a temporary string.

To test
-------
[1] Perform a PCRUD search that includes fleshing and retrieves
    a large responses. One example is a search on bre that fleshes
    call numbers and copies and returns a few hundred bibs.
[2] Observe that the PCRUD drone that serviced the request has
    increased its memory usage significantly.
[3] Apply the patch and repeat step 1. This time, the drone's memory
    usage should not (persistently) increase as much.

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

Docs: correction to Staff Portal Page docs

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

Docs: Staff Portal Page documentation

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

fix typo in 3.8.1 release notes

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

Forward Port 3.8.1 Upgrade Script

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

Forward Port 3.7.3 Upgrade Script

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

Update 3.8 Point Release Notes

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

Update 3.7 Point Release Notes

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

Docs: 3.8.1 Release Notes

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

Docs: 3.7.3 Release Notes

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

LP#1857910: (follow-up) fix styling of mandatory class and field comboboxes

This patch uses the 'mandatory' synonym of 'required' in the custom comboboxes
for the IDL class and field.

There remains an issue: the fact that those fields are mandatory does
not propagate to the overall form validation.

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

LP#1857910: (follow-up) enhance IDL for field doc

This patch adds human-readable labels to the IDL for
field doc. It also marks required fields in the IDL, removing
the need for the Angular code to explicitly specify
the set of required fields.

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

LP#1857910: (follow-up) stamp DB update

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

LP#1857910: (follow-up) clean up lint

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

LP1857910 release note blurb for field doc port

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1857910 add org family selector to field doc UI

If we want this to default to workstation org or something, we can do that, though I was having trouble
getting that to work.  Right now it'll include all orgs in its default unselected state.  I'm also not
sure about i18n-labelText and whether that works with eg-org-family-select.

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1857910 make field doc grid sortable/pageable

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1857910 ws setting type for field doc grid

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1857910 Field doc UI Angular 10 updates

Modify the lazy loader to use the new Ang10 style loading syntax.

Avoid using template variables as output variables for other components,
since template variables (those declared with let-foo="bar" in
<ng-template/>) are read-only, possibly as of Ang 10, so the page was
not loading at first.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

lp1857910 Field Documentation Port

- Port grid UI allowing the creation and editing of Field
Documentation from DOJO to Angular

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/eg2/src/app/staff/admin/local/admin-local-splash.component.html
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/field-documentation/field-documentation.component.html
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/field-documentation/field-documentation.component.ts
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/field-documentation/field-documentation.module.ts
new file:   Open-ILS/src/eg2/src/app/staff/admin/local/field-documentation/routing.module.ts
modified:   Open-ILS/src/eg2/src/app/staff/admin/local/routing.module.ts

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1946531: (follow-up) show progress bar when adding users to bucket

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

LP#1946531: use batch method to add users to bucket via User Buckets UI

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

lp1950507 Providers Admin Screenreader form fix

- Move Search and Reset Form buttons to bottom of form when expanded.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
Signed-off-by: Tiffany Little <tlittle@georgialibraries.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1968082: (follow-up) show progress bar when adding/removing items from bucket

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

LP#1968082: (follow-up) tidy up whitespace

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

LP#1968082: Use the batch API call to add and remove items from item buckets.

In particular, when adding/removing items while on the item
bucket interface itself.

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

fix a syntax warning in the 3.9 release notes

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

Docs: further updates to 3.9 release notes

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

Docs: OAI-PMH data provider support

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

Docs: adding rel_3_9 menu item

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

Docs: Simple Reports docs

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

Docs: updates to 3.9 release notes

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

LP1930740 EDI order writer attribute check repair

Repairs mistaken references to $self->{compiled}->{edi_attr}...  The
hash key we need is {edi_attrs}.

This issue prevented these attribute types from getting applied:

LINEITEM_IDENT_VENDOR_NUMBER
COPY_SPEC_CODES
INCLUDE_EMPTY_LI_NOTE

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jessica Woolford <jwoolford@biblio.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

clear miscellaneous.adoc for next release

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

LP#1965579: stop gap for negative bills in OPAC

(and zero bills).

If a patron has zero-dollar or negative bills present, the OPAC will
disallow payment and point the patron to their library to resolve.

Cribbed from Terran's rel_3_8_0_temp_fix_for_stripe_negative_bill_bug @ evergreen/pines.git

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

Forward port 3.9 upgrade script

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

Base 3.9 release notes

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

Translation updates - po files

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

Translation updates - newpot

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

Correct jacket uploader upgrade script

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

LP#1970486: fix SuperCat crash on serial units with stat cats

This patch fixes a bug where retrieving a title via SuperCat (in
particular, with holdings XML) can crash of the title has a serial
unit (i.e., barcoded serial item) attached that also has a statistical
category.

To test
-------
[1] Create a title with a barcoded serial item.
[2] Attempt to fetch the title via SuperCat:

https://EGSERVER/opac/extras/supercat/retrieve/marcxml-full/record/BIBID

[3] Note that an internal server error is returned.
[4] Apply the patch and repeat step 2. This time, an XML
    document should be returned that includes the serial item
    and its stat cat.

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

LP1907863 Bootstrap Opac: My Lists formatting

This patch moves the buttons for each list row underneath the title and descriptions fields.
It makes the create list form collapsible using a button.
If there are lists, the create form is collapsed.
The create form is expanded on load, if the user selects move bucket to new list, or add rec to new list from a menu or button from another page.
The "Move contents of basket to this list?" in the creation form now defaults to 'yes' if an option to move a basket to a new list is chosen from a menu.
It adds localization functions to the button texts and removes a dangling </div> in anon-list.tt2
It moves the create form from a <table> design and uses Bootstrap elements instead.

To Test.

1. After applying the patch view the list page and notice that the form is expanded.
2. Create a list.  The form is now collapse.
3. Perform a search and add some of the titles to a basket.
4. From one of the result rows drop-down the Add to My List menu and select 'Add to New List'.
   The form is expanded and the "Move contents of basket to this list" should be 'yes'.
5. From the basket screen or from the lists screen click on the 'Add to New List' button.  The results should be the same as step 4.
6. After creating a few lists, notice that the buttons for each list are now displayed under the list title and description and
   each row is distinguished by a bottom border.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

Test Followup: Fix Jacket Dir in autogen.sh

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

LP#1965317 Barcode Completion on Traditional Cat Staff Holds

This allows the completed barcode to populate in the barcode
input when the hold group interface isn't rendered.

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP1956626 Copy editor loads all needed copy locations

The item-location-select component now allows the caller to pass in a
set of context org unit IDs (in lieu of just a single id) for loading
copy locations across different areas of the org unit tree (without
having to load *all* copy locations).

Changes applied to the Angular copy location editor to pass the needed
context org units to the item location select component.

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

LP#1950468: make Replace Barcode from Item Status detect duplicate barcodes

This patch makes the Replace Barcode action when invoked from
Item Status properly detect and warn if the replacement barcode
is a duplicate of one already found in the system. This is a complement
to the fix for bug 1890498.

To test
-------

[1] Enter an item barcode into Item Status.
[2] Try 'Replace Barcode' from detail view and set the
    replacement barcode to one that is already active in
    Evergreen. Note that the dialog closes without
    updating the barcode or warning about the duplicate and
    that a error is logged in the browser console.
[3] Repeat step 2, but this time from list view. Note
    the problem remains the same.
[4] Apply the patch and repeat steps 2 and 3. Note that
    now the dialog will remain open and will display an
    error message complaining about the duplicate barcode.

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Signed-off-by: Erica Rohlfs <erica.rohlfs@equinoxOLI.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP1969232 Use fonts-only material icon fonts package

This replaces the larger package which caused 'npm install' to take a
very long time to complete.

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

LP#1967770: Followup fix for cover image directories

In the initial fix we checked for the top level cover image location and
error out if it's not there, but this is unnecessary since it will be
created if it doesn't yet exist.

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

Stamp upgrade script for MADS update

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

LP1800871: small correction to upgrade script

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

LP#1800871 - MARC21slim2MADS 2.15 update

This update fixes an error when trying to process a 755 tag in
and authority record that has a source set.

See the discussion on the listserv at:
https://georgialibraries.markmail.org/thread/rmwcxkwjzv2qczmu

To test, try to import the sample authority record provided
by Linda Jansova (https://markmail.org/thread/2ay3j4wg7fecymhz)
in evergreen, and see the log files for the import errors.

It is also possible to test from the command line.

In /openils/var/xsl, copy the example autority marc xml file to
that directory, name it americ.xml, then run the command
xsltproc ./MARC21slim2MADS.xsl ./americ.xml

This will error out.

After patch has been applied, try importing the sample authority
record and see that it imports successfully.

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>

LP#1942240 Insufficient color contrast in boostrap forms

Overrides the bootstrap form-control border color. Also changes the
border_standard color in colors.tt2 which is used for the border
around some form elements.

After this patch is applied the color ratio is 3.54:1

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1966802: add Matomo support to Bootstrap OPAC

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1958163 Bootstrap Opac: the 'Request a Card' link.

This fixes the 'Request a Card' link which displays in the bootstrap opac
 regardless of the 'Allow Patron Self-Registration' library setting in both
the login page and the login modal.

To Test:
1. Click on My Account in the Bootstrap Opac to open the login modal.
   Notice the Request a Card link.  Click Login without adding a barcode
   or pin# to see the login page with its link.
2. Set the Allow Patron Self_Registration library setting to false for the
   Consortium.
3. Repeat 1, and notice that the link still displays.
4. Apply the patch.
5. Now the link should not be visible in both forms.
6. Set the library setting to true, to verify that the link does display.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1915816 follow-up: one more .gitignore addition

Suggestion from Shula Link.  Thanks, Shula!

Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Co-authored-by: Shula Link <slink@gchrl.org>

LP1915816 follow-up: update package-lock.json

LP1915816: Bonus commit - Remove Old .gitignore Entries

There are a few things that were being ignored that could no longer
really exist anyway, so we may as well clear some out.

Signed-off-by: Jason Boyer <JBoyer@EquinoxInitiative.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1915816: Add test output to .gitignore and track package-lock.json

Just what it says on the tin.

Signed-off-by: Jason Boyer <JBoyer@EquinoxInitiative.org>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP1944205 Advanced Authority Documentation

Added a new document authorities_advanced.adoc

Integrated to the nav

Signed-off-by: blake <blake@mobiusconsortium.org>
Signed-off-by: Lynn Floyd <alynn2671@gmail.com>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1778783 - Circulate.pm fix log_me correct barcode var

The log_me sub wasn't using the correct variable for showing the entered
barcode.  It should be $self->copy_barcode instead of $self->barcode.

Testing Notes:

Before fix:
1. Watch the logs with something like
  tail -f osrfsys.log | fgrep 'circulator: do_permit()'
2. Perform a checkout of a non-existant barcode.
3. Notice that the entered barcode is missing after 'copy='

After fix:
1. Restart the circ openils service
  osrf_control -l --service open-ils.circ --restart
2. Watch the logs.
3. Perform a checkout of a non-existant barcode.
4. Notice that the barcode is now shown after 'copy='

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

LP1958265 Holds grid barcode print/csv fix

Tweak the barcode print/csv logic to populate the current_item and
requested_item values using the same logic as the grid.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP1958265 Angular Holds Grids Not Printing Barcode

This adds the ability to print (or download CSV) the Current Item
and Requested Item barcodes from the Holds Pull List and from the
Holds tab of the bib record.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP1959904: Fix Angular Profile Select Component

The Angular profile select component has a bug when permission group
tree display entries are used.  It will loop forever if the grp and
parent have the same value in permission.grp_tree_display_entry
because it is comparing permission group tree entries to permission
group tree display entries in the groups filter of the grpLabel
function.

This patch adds a ternary operator to compare display entries to
display entries and group tree entries to group tree entries in a
manner identical to how the parent value is determined a few lines
above.

See the bug description for more information:

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

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP1950826 Return error result on contact invalidation

If penalty creation fails, return the error/event response to the caller
instead of the editor's last event.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Christine Burns <christine.burns@bc.libraries.coop>

LP1950826 fix invalidate email action

and other invalidate contact actions

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

lp1913340 - List All Courses in OPAC

- Removed handling preventing an empty query from providing results
- Strip * from query to ensure expected behavior

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
Signed-off-by: Christine Burns <christine.burns@bc.libraries.coop>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>

LP#1954923: Fix current date issue in Boostrap OPAC Circ History CSV export

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

LP1920039-Bootstrap Opac: More Details button doesn't change on use.

The 'More Details' button in a bib record in the Bootstrap Opac should change
to say 'Less Details' when pressed.  This patch fixes that behavior.

To Test.
1. Go to a bib record in the boostrap opac.
2. Press "More Details"
3. Notice the button still displays "More Details"
4. Apply the patch and repeat.
5. The button should now toggle between "More Details" and "Less Details"

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1967770: Fix test for cover image upload

The cover image upload test assumes that it will be run from exactly one
location, but that's not necessarily true.  This commit uses FindBin to
provide the right filesystem context for the test to find the files it
needs.

[Code by Galen, commit message by Mike]

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

LP#1967770: Make sure cover image dirs exist

This commit teaches autogen.sh, which should be run after each upgrade,
how to check for and create the required locations for cover image
upload.

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

LP1930747: Add MARC_NAMESPACE to Const.pm

Now that we have 3 separate $MARC_NAMESPACE definitions it's time
to just move it into Const.pm and call it done.

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

Update billing.adoc

updating link

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

Updating billing.adoc

for the 2022 DIG project

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

LP#1919500 - Tweak to Checkout Staff display

Displays Checkout Staff's usrname instead of id and adds a link to the
record.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1919500 - Add Checkout Workstation and Checkout Staff to Item Status -> Circ History List

Add two more data fields to the Circ History List in Item Status, so that
it is possible to see them after an item is renewed.

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP1865062 Add Credit Card Approval Code to Payment Receipt

Makes approval code available to Bill Payment print template (when relevant).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Llewellyn Marshall <lbmarshallv.ncdcr@gmail.com>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

LP#1965432: follow-up also update package-lock.json

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

LP#1965432: Upgrade karma to latest version.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jane Sandberg <sandbergja@gmail.com>

LP 1964963: Improve Customization of BOOPAC topnav logo

Move the anchor for the topnav logo from topnav_links.tts to
topnav_logo.tt2.  This minor change means that those who wish to move
or remove the topnav logo will have fewer files to change and those
changes will be less likely to conflict with future changes to BOOPAC.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>

Fix ng lint errors

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

Stamping upgrade script for localized notifications

This feature supplies the ability to create alternate templates for Action Triggers
that will generate locale specific out for Action Triggers.  If you send notices in
multiple languages, we recommend putting some words to that effect in your notice
templates.  The template, message and message title can all be localized.  To use the
feature the following new UI elements have been added:

- When you double-click on an Event Definition under Notifications / Action Triggers
  to edit it there will be a tab option for Edit Alternate Template if the reactor is
  ProcessTemplate, SendEmail or SendSMS.
- In the Patron Registration and Patron Editor screens staff members may now select a
  locale for a patron and edit it in the Patron Preferred Language field.
- Patrons may set their own locale in the My Account interface off the OPAC by going to
  Preferences -> Personal Information and setting the Preferred Language field.

The templates used on the Edit Definition tab are the defaults that are used if there are
no alternate templates available that match the preferred language.  If alternate templates
are available the system will use a locale that is an exact match and then if failing that
use one where the language code matches and then fall back to the default one.

For example, if a patron has a locale of fr-CA and there are templates for both fr-CA and
fr-FR it will use the fr-CA.  If the fr-CA template was deleted it would fall back on using
the fr-FR for the patron since it at least shares the same base language.

Valid locales are the codes defined in the i18n_locale table in the config schema.

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

Stamping upgrade script for OAI-PMH

LP#1729620: Rebase and remove array_accum uses

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

LP#1729620: Polish up sets and visibility tests

This commit improves the logic needed to control record inclusion in
OAI Sets.  Sets for bib records are currently based on copy visibility,
Located URI visibility, or bib source.  Generally useful combinations
are generated for item, Located URI, and transcendant [sic] bib sources.

Sets for authority records are based on the browse axis of the heading
for the record.

All generated Sets for bib records currently test for effective search
visibilty on the three described components.

In order to retrieve item data from a specific branch of the org tree,
a harvester must use a Set that contains COPIES:. The hierarchial
shortnames of the org units follow the colon, so in a freshly installed
system a Set with the setSpec of COPIES:CONS:SYS2:BR3 will retrieve all
bib records that have visible copies at BR3, and include only the item
data for that org unit.

To retrieve records with Located URIs, which behave in the heirarchical
reverse of copies by limiting display to those org units "inside" the
branch of the tree where the licensing "lives", a harvester should
supply a setSpec with an org unit shortname that would display the
Located URI. For example: LURIS:CONS:SYS2:BR3

Note: the LURI: org unit is also used to restrict any item data that may
be available for LURI-filtered records.

To retrive records are in a transcendant [sic] bib source, a harvester
can used a setSpec starting with SOURCES: followed by the source name.

To harvest all records, including deleted records, simply omit the
setSpec from the request.  All item data will be included.

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

LP1729620: Enable org unit OAI sets

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP1729620: Fix opensrf.xml syntax

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Mike Rylander <mrylander@gmail.com>