Fix error in handle_mark_item_event switch.

Bad string constant name.

Still syntax error at line 721.

Syntax error in holds.js line 721.

Revert "Try making mark_damaged always refresh the list."

It makes things worse, in my opinion.  Refreshing damaged items is
bugged, and deserves its own fix.

This reverts commit 8c85a539069aa73c592be81c1924c6b718f81da1.

Try making mark_damaged always refresh the list.

Missing parenthesis.

Fix typo.

Modify mark_damaged to use handle_mark_item_event and more.

The final fixup. (I hope.)

Try fixing checkin and renew

Fix up the other places.

Use my changes with mark missing.

Fix another error cause by a typo.

ReferenceError: egcore is not defined
    at handle_event (circ.js:1385)

Fix missing close paren on the same line as previous error.

Fix circ_strings.tt2

egweb: template error: file error - parse error - staff/circ/share/circ_strings.tt2 line 37-38: unexpected token (])

Finishing ti up?

Small attempt to resynchronize it.

I think I need to go back to defering the status look up if this
doesn't work.

Try rearranging the code to look up new copy status only once.

Try actually marking the copies now.

Really fix the previous error.

Things are never as simple as they seem.

Remove extraneous return in circ.mark_discard.

Fix TypeError: egCirc.mark_discard(...).thef is not a function

Add missing ".result" in previous commit.

Bring initial confirmation dialog.

Also edit the strings a bit.

Trying something slightly different.

Revert "Trying something...."

This reverts commit 24f32f1fcc7a23ad74e4563d263864b9bdcb7cae.

Trying something....

TYPOE!!!!!!!!

Fix pcrud search syntax.

Grrr...

Forgot to change the argument name of the get_status inner function.

Trying pcrud search and deferred.notify().

Just trying to lookup copy status.

Revert "Attempt to get flesh_copy_status to work."

This reverts commit 7248fc97f04c645409c6c244b1de5cc1937fa267.

Attempt to get flesh_copy_status to work.

Alter constant strings for dialogs.

Revert "Try using $q.defer();"

This reverts commit 0c79043b2fb5b80eda0195d9736adc74acc67267.

Try using $q.defer();

Remove doRefresh from circ.mark_discard

Change previous commit to use a then().

Add copy barcode to list that works.

Fix typo.

Fix access to copy status.

Update UI code for backend changes.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Add comments to test file.

Small improvement.

Remove Data::Dumper from tests.

Fix bug in mark_item resrict copy delete check.

Fix tests for new events code.

Comment fix.

More back end improvements.

It should actually work.  Though I didn't try the previous commit, I
realized that the transaction code was wrong.  I also decided to make
a request to open-ils.circ.transit.abort rather that do all that code
"by hand."

More back end work for marking items.

This work unifies the back end behavior more for the different
statuses.

The tests will have to be reworked.

Add Open-ILS/src/perlmods/live_t/zz-lp1779467-mark-item-discard.t

Add Perl live tests for minor back end change to marking items as discard/weed.

Try checkin before marking item discard/weed.

Modify Open-ILS/src/perlmods/lib/OpenILS/Application/Circ.pm to call
try_checkin in mark_item when the api includes discard under the same
conditions as when marking an item damaged.

Make a potential fix to mark item discard.

Wire up basic mark discard/weed with back end.

Wire up the JS wrapper functions for discard.

Add missing 's in previous commit.

LP 1779467: Add Mark Item(s) Discard/Weed to grid menus.

Forward Port Release Notes for 3.0.13.

Signed-off-by: Jason Stephenson <jason@sigio.com>

Forward-port 3.1.7 upgrade script

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

Forward-port 3.2.1 upgrade script

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

Docs: adding another entry to 3.2.1 release notes

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

Docs: adding another entry to 3.1.7 release notes

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

LP#1797923 Browser client iframe initial loading page

Apply a default value to the browser client iframe URL so the iframe
does not attempt to load a copy of the current page.  The default value
points to a simple loading page, displying "Loading..." plus our
standard spinny gif.

Progress gif copied from web/opac/images/ to web/images/ to avoid
dependence on the old opac directory.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

Docs: Adding release notes for 3.2.1

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

Docs: adding release notes for 3.1.7

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

Forward port 3.1.5-3.2.0-upgrade-db.sql.

Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1746536: Remove input-group-addon class from Add Call Number button

The button no longer looks like a button with this class in place.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1746536: Restrict CN addition but allow CN edits...

... in the top half of the Vol/Copy editor when there are multiple bib records
represented in the UI.

Also, restrict the size of the org selector using a bootstrap row.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Conflicts:
Open-ILS/src/templates/staff/cat/volcopy/t_edit.tt2
Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js

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

LP#1746536: Allow batch-apply to volume fields in multi-record context

I can't see a reason to restrict Batch Apply for volume fields in a multi-
record context, as all the data is gathered based on the volume owners, not
record information.  This commit removes that restriction.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1746536 - enable volcopy editor for multi-bib but only volumes

This removes the disabling of the volcopy editor caused by
passing it a record id of 0. Allowing the Item Status grid's
Edit>Volumes to function as a way to batch edit volumes of
more than 1 bib record.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP1746536: cannot edit vol/call number in item status

spawnHoldingsEdit was disabling record summary by passing record_id: 0
this change will provided the record_id if exactly one record is selected.
behavior is unchanged if multiple records are selected.

Signed-off-by: a. bellenir <ab@grpl.org>
Signed-off-by: cesar velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Conflicts:
Open-ILS/web/js/ui/default/staff/circ/services/item.js

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

LP#1796978 Realign working copy refresh with proper condition

The "working copy" grid needs to update whenever the copy data above
updates.  I *believe* this aligns the refresh with its intended
condition.

This appears to have come about via bug #1732761, so retesting of
that fix may be in order.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>

LP#1796971 Wait for call number and copy before loading locations

In cases where add a copy to a call number, we fetch the original
call number via async, then attach the new copy to it.  Before this
commit, however, we were not waiting for the call number fetch
promise to resolve, so the copy wasn't there to use for limiting the
location load.

Since the async request is internal to a loop, one way out is to
capture that promise, then make sure it resolves before loading the
locations.

(We previously returned the copies from the internal function, but that
value wasn't being used anyway, so we don't worry about unwrapping the
$q.all() when it resolves.)

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>

Docs: correcting inaccurate statement in action trigger docs

Docs: Update release notes version in consolidated manual

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

Docs: adding info about testing action triggers

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

Docs: adding action/triggers to staff client admin docs

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

Docs: re-adding missing anchor to install instructions

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

LP#1792658 Angular Evergreen favicon

Use the Evergreen favicon.ico in the Angular(6) application.

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

LP#1785681 Patron address copy to clipboard

Support copying formatted addresses to the clipboard in the browser
staff client, similar to the XUL client.  Action is performed via a new
link in the addresses section (next to "print") in the patron summary
sidebar.

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

LP#1721139: apply proper date format in certain places

This patch ensures that the correct date format is used
to display dates in certain contexts in the web staff
client:

- patron bucket summary
- record bucket summary
- patron bucket 'View batch edits'

This has the side effect of fixing a problem where the batch
edit timestamps were not wrapped correctly in Firefox.

To test
-------
1. Using user buckets, make some batch changes on some patrons and roll
   back some of those changes.
2. Using Firefox, click on the 'View Batch Edits' button.
3. Note that the times dispayed are formatted as raw
   database timestamps
4. Note that the summary for the user bucket displays the
   create time as a raw database timestamp.
5. Navigate to a record bucket and note that its create
   time is displayed as a raw timestamp as well.
3. Apply patch.
4. Repeat steps 2-5; this time, the times should be displayed
   using the correct date and time format.

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#1778083 Location, Floating, and Age Protect missing from Items Out

Adds columns for Location (Copy), Floating Group, and Age-based Hold
Protection to the items out grid.

To test.
1. View an items out screen to see that the Location, Floating Group,
   and Age-based Hold protection columns are not available as options.
2. Apply the patch.
3. Edit some items and add definitions for Floating Group, and
   Age-based hold protection.
4. Check out these items and other items that do not have a floating
   group or age-based protection.
5. Display the Location, Floating Group, and Age-based Hold Protection
   columns.

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

LP#1770246: (follow-up) display the open-in-new-window/tab icon

Since the title links added by the previous patch intentionally open
the record in a new tab, display the corresponding icon to be
consistent with other interfaces.

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

LP#1770246 Catalog linked title column in query and pending buckets

To mirror the Record Bucket View grid, this patch adds a title column
in the query and pending bucket tabs that links to the catalog record
of the title displayed. The catalog record is opened in a new tab, so
the query and pending data is not lost.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

Docs: adding basic info about printing spine labels

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

Docs: Adding info about copy number

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

Forward-port 3.1.6 upgrade script

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

Forward-port 3.0.12 upgrade script

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

Docs: adding another fix to the 3.1.6 release notes

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

Docs: adding another fix to the 3.0.12 release notes

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

LP#1786987: Locale-less org tree cache object becomes stale

The autogen.sh script calls a function that flushes all org tree objects
cached by locale, but does not flush the one used when the locale is
unkown.  This commit forces that flushing to occur.

berick: Definition of my $cache pushed up above first use.

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

Docs: Adding 3.1.6 release notes

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

Docs: Adding release notes for 3.0.12

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

Docs: Clean up installation docs

- Remove redundant NodeJS section
- Relabel sections to distinguish between AngularJS and Angular steps
- Fix a few typos

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

Translation updates - newpot

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

Translation updates - po files

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

Docs: Remove old release notes files

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

Docs: Add Upgrade Notes to release notes

This commit copies relevant sections of the release notes into the
Upgrade Notes section.

A few paragraphs are also reformatted into lines of appropriate length.

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

Update Release Notes for 3.2-rc

Add new release notes. Also, slight reorganization to the
acknowledgements section (move organizations to appropriate section, fix
sorting of COOL, use consistent name for BC Libraries).

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