Removes red notice about emailed queue complete notice on LMOR.

Signed-off-by: Tiffany Little <tlittle@georgialibraries.org>

LP#1812698 - Fix broken pubdate display

reporter.simple_record, which is now used more
extensively for bib display, had a copy-paste error
originating in 2014 that now gets corrected.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

Returns Activate PO ticky to LMOR

Signed-off-by: Tiffany Little <tlittle@georgialibraries.org>

typo correction

typo correction

typo correction

LP1783421 - Make Copy Alerts permission not global

For some reason copy alerts require global permissions.  Instead
update the IDL for asset.active_copy_alert and asset.copy_alert to
require {CREATE,VIEW,UPDATE,DELETE}_COPY_ALERT perms only as far as
they are scoped for the circ lib of the copy.

Signed-off-by: James Fournie <jfournie@vcc.ca>

Revert "LP#1746360 Record Bucket Missing Deleted and Edition Columns"

This reverts commit 3730fc8f0146cbb44ce8722defec2d8a5af19d86.

add copy alert perms to LocalAdmin and LibraryManager

Adds documentation links to acqadmin menus

Adds links to appropriate PINES wiki pages for remaining Dojo Acquisition administration menus. Additionally also marks required fields on Load MARC Order Records page.

Removes checkin date column

Removes Circ ID and fixes publisher

Removes edition, due date, floating and holds count

Removed auto-populate of column options in Item Bucket, individually outlined column options in alphabetical order.

LP#1737800 - add delete action to pending patrons

This adds a way to remove a pending/staged user from
the pending users grid.

Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>

removing spine label printing feature

LP#1726568: expose circ lib and owning lib fields to spine/pocket labels

This patch fleshes the item circulation library and owning library
fields so that spine/pocket labels can include the short name or
full name of the OU rather than just the ID.  As a consequence, the
following fields are now available for label templates:

* copy['call_number.owning_lib.id']
* copy['call_number.owning_lib.shortname']
* copy['call_number.owning_lib.name']
* copy['circ_lib.id']
* copy['circ_lib.shortname']
* copy['circ_lib.name']

If, for some reason, existing label templates intentionally use
copy['call_number.owning_lib'] or copy['circ_lib'] to include the numeric
ID of the OU on the label, they will need to be replaced with
copy['call_number.owning_lib.id'] or copy['circ_lib.id'] respectively.

To test
-------
[1] Apply the patch.
[2] Open the print labels window for an item and try including one
    or more of the new fields listed above; verify that the current
    value is displayed on the label preview.

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

switch around order, update already existing setting

adding several other post-upgrade tasks to script

LocalAdmin already has COPY_DELETE_WARNING.override

Cat1 already has COPY_DELETE_WARNING.override

Staff already have CREATE_DUPLICATE_HOLDS perm

more typo corrections

LocalAdmin already has EMERGENCY_CLOSING

typo corrections

Lp 1803734: Only push on-order purchase orders in edi_order_pusher.pl.

Add a check for the state of purchase orders being on-order when
gathering them in edi_order_pusher.pl.

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

adding perms and settings for post-upgrade

adding refressh of reporter.materialized_simple_record

LP#1746360 Record Bucket Missing Deleted and Edition Columns

Added deleted and edition columns to the record bucket grid.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

tweak parent/guardian data update

PINES Fix to Search Results Title

Fixes problem where Search Results page title was overlapping
filter boxes.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

LP1765179 - fix issue with pending/staged user reg

After completing a staged user registration, the controller
in regctl.js was refreshing to the same route, and
trying to fetch the same staged user, but never finding it
as it was no longer staged/pending. This addresses that bug
and adds a bChannel refresh to the pending grid, so once
the user is registered in the child tab, it closes the tab and
the pending user grid is updated.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>

adding file to combine rbdigital installation

adding parent/guardian data

Removes "Activate PO" ticky box on the Load MARC Order Records page.

LP#1714070 Patron parent/guardian field

Adds a new dedicated patron parent/guardian field.  This field is
editable in the patron edit interface, displays in the patron summary
side bar on the browser client, and is search-able from the patron
search interface in the browser client.

Adds patron editor "show" and "suggest" library settings.

Also adds a new library setting
'ui.patron.edit.guardian_required_for_juv' ("GUI: Juvenile account
requires parent/guardian").  When this setting is applied, a value
will be required in the patron editor when the juvenile flag is active.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

PINES Cust: Make offline interface alerts bolder

This overwrites the bootstrap CSS just for the offline
circulation page alerts boxes.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

PINES Custom: Redisplay print and email links on item record page

These were hidden in a previous patch because the functionality
wasn't there, but now that the functionality is working, adding
them back in.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1779467: Fix 'Dicard' typo

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

LP#1737800 - add delete action to pending patrons

This adds a way to remove a pending/staged user from
the pending users grid.

PINES Customization: Patron Summary Link to Library

Turn patron home library name on the patron summary bar into
a link to the library's contact info page.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1810296: Fix selfcheck patron password setting check

Previously, the selfcheck ignored this setting if it was
set to "False", since the setting value is stored as text
and the truth test always succeeded.  Instead, explicitly
check that the setting is "True".

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

PINES Custom Hold Shelf Slip

Updated to include new preferred name at top of slip (when
present).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

adding perms for new copy alert features

PINES CSS Customizations

Restores previous CSS customizations that got knocked out
somewhere along the way.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1743783 Web Client Bill Full Detail Display Issues

Fleshes out retrieved data to be able to display the billing
location for both circulation and grocery bills on the full
details page as well as the owning library for circulation bills.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

LP#1662535 Patron Bill Grid Columns

1) Adding missing columns, including:

- Call Number
- Owning Library
- Shelving Location
- Last Billing Note
- Last Billing Timestamp
- Last Payment Timestamp
- Last payment Note
- Last Payment Type

2) Corrected path for Type

3) Changed column name for Location to 'Billing Location'
and the former Billing Location to 'Grocery Billing Location' for clarity

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Offline Interface: Display text for non-cataloged items

When a non-cataloged item is checked out, display preview
text instead of a blank barcode.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

PINES Customization: Alert when no hold notification options

Present an alert if no notification options are selected in
both the OPAC and web staff client.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

PINES Custom: Adjust top menu bar in OPAC for baskets

- Adjust spacing so that the basket doesn't go off the edge of the
menu bar.
- Add responsive code as well.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

PINES Custom Offline Checkout and Renew Receipts

- Correct due date to not show due time.
- Show 'Non-cataloged Item' text when there is no barcode.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

PINES Custom fix for Patron Self Registration Form DOB Validation

This moved away from master at some point. This patch brings it back
in line along with correcting a dob format checking error.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

adding new post-upgrade script

LP 1779467: Fix Error When Marking Item on Hold as Discard/Weed

Fix the following error when marking an on-hold item as Discard/Weed:

Can't use an undefined value as an ARRAY reference at
/usr/local/share/perl/5.18.2/OpenILS/Application/Circ.pm line 1393.

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

LP 1779467: Fix typo in circ/renew/app.js.

Had copyies for copies in one location and this prevented marking an
item as Discard/Weed from working on the renew item interface.

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

LP 1779467: Fix SyntaxError: missing ) after argument list

Missing parenthesis in Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
caused the titular error when searching the catalog.

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

LP 1779467: Enhance Mark Items Functionality

A new option to "Mark Item as Discard/Weed" is added to many actions
menus in the staff client.  This command is connected to the back end
function open-ils.circ.mark_item_discard.

The back end functionality for the open-ils.circ.mark_item_* family of
functions is altered to provide more consistent behavior and to avoid
some strange situations that have come up in the past, such as items
with the Missing status having active transits or open circulations.

The code for "Mark Item as Damaged" and "Mark Item as Missing" are
altered to take advantage of the back end changes.  NB: These changes
do not affect the "Mark Item as Missing Pieces" function, as that is
handled by different back end code.

Perl live tests are added for the backend functionality changes to
test that certain conditions works.  Like most of our tests these
could be expanded to cover more potential situations.

See the release notes for more detail on changes in functionality.

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

PINES Cust: Fix layout for Patron Edit Form

Readjustments to previous customizations that no longer work
properly with 3.2. Also added links to relevant documentation.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

PINES OPAC fix to My Lists

Removes incomplete IF statement that caused page not to load.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

remove post-upgrade script

PINES doesn't use LoC bib source

drop triggers first

create index only if not exists

LP#793802 Billing/payment purge improvements

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

LP#793802 Aged billings/payments release notes

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

LP#793802 Aged billings/payments PGTAP tests

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

LP#793802 Add money.all_[payments|billings] views

Similar to money.all_circulation_slim, add views to collect data from
both active and aged versions of the payment and billing tables.

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

LP#793802 Aged billings and payments

Creates tables money.aged_billing and money.aged_payment.  When a
circulation is aged, billings/payments linked to the circ are migrated
to the aged_* table.

Upgrade script includes a migration script for migrating existing
billings and payments.

Includes IDL entries for reporting and pcrud access just in case it's
needed.  Note the 'xact' column on these new tables links directly to
the aged circulation instead of the non-existent money.billable_xact.

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

Add fix for PG segfaults during reingest.

Also add inline docs.

fix eg version in post upgrade script

remove z3950 update for LoC

fix out-of-sync id sequence

drop reporter.copy_statistics_view for upgrade to work

LP#1672346 - Copy Statistics View

Certain third-party products such as collection development
management providers require copy statistics that are not
readily available in a single report.  This view adds those,
which will also benefit library staff reports generally.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

add pre-3.2-upgrade script

adding 3.1-3.2 upgrade script

move reingests to the end

LP#1772062: prevent copy template apply failure on null stat cats and non-null statcat filter

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>

add iOS link to OPAC footer

LP#1738249: further fix to checkout workstation

Since accs.checkout_workstation contains the workstation associated
with the initial loan (and not any subsequent renewals), use that.

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

LP#1738249 - Fix checkout_workstation display issue

Addresses the issue with checkout workstation, by
attaching as a object property to the flatCopy hash.
Also makes sure the pcrud call uses combcirc.

Signed-off by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>

LP#1788260 - Break out in-house-use non-cat circulations.

Previously, all non-cataloged circulations were grouped
together, causing problems for libraries who need to
exclude in-house circulations from circ counts.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP#1785887 edi_order_pusher.pl sets message account

Apply the edi_message.account value when sending EDI order messages via
edi_order_pusher.pl.  This solves a problem where EDI ORDERS message
were not visible in the interface, because there was no way to complete
the permission check.

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

LP#1743650: Bib vis testing needs different handling

For bib-level visibility testing, we can only use the source helper for
positive inclusion, and have to deal with LURIs on a case-by-case basis.
This is because, unlike the copy visibility cache, the LURI cache is pre-
composed in a single value for all LURIs on a record, not separate ones
for each.  This is fine, as we just need to find the effectively visible
org units and test for that subset of the relevant orgs. This commit provides
a sub to test for that, and handles LURI test composition at the perl
level rather than in the database.

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

fixing typo in PINES documentation link

LP#1773417 Revamp item and call number tranfers

This commit attempts to achieve the goals of both simplification and
feature completeness/flexibility.  In brief, limit the number of
marking and transfer options, then have the code decide the right
action to take given the circumstances.

There are now just two "marking" actions, one at the record level,
one at the holdings level.  The holdings level mark will automatically
mark the destination as specifically as possible from the selected row,
which means either to the library or call number (vol) level.

We are also now down to two transfer options: transfer the selected
item, or transfer the selected call number.  Either option will use
as much given context as possible, then fill in any blanks with
reasonable defaults and actions.

As part of the change, a number of functions and variables are also
renamed.  This is all done for clarification, and in most cases is
due to the variable or function now being used more generally (i.e.
it is used in both the item and vol context, so it is confusing to
be named 'volume_transfer_target', etc.).

Finally, clear up a fair bit of now redundant and unused code.

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

LP#1737812: Simplify holdings tranfser options

Instead of providing direct actions for transfer of library, or record, or
both, have just one volume transfer option that Does the Right Thing (tm).

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

PINES Syndetics Customization

This attempt leaves the Syndetics perl code in place to retrieve images,
but ignores it in favor of hardcoding in the Added Content button and
modal code on the item details page.

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>

Revert "PINES Customization for Syndetics"

This reverts commit 329d5d9dbd51d8bf2c7ab3b2e447e047ddd7e3a0.

LP#1721139 User Buckets - View Batch Changes date/time display not wrapping.

This fixes a wrapping issue in Firefox for the date/time display in the 'View Batch Changes' modal in user buckets.  The css change was made to style.css.tt2 because the class can be used for other tables with text wrapping issues.

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. Apply patch.
4. Delete browser cache, and then log back in and 'View Batch Edits' again.

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

PINES Customization for Syndetics
(This is attempt 3 - ignore previous attempts.)

- Eliminates unnecessary calls in the perlmod for all types of content.
- Creates new modal popup instead of creating tabbed links.
- Copies syndetics url & id to eg_vhost.conf for easier access by tt2.
- Updates extras.tt2 to always expand added content section if content.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

PINES does not use patron credit, so remove it from view

PINES: Further customizations to new books feed

Hid availability because it didn't differentiate between in process
and checked out. Added copydepth to url because it wasn't working on
production without it.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

LP 1735539: Now with streaming action.

Modify item service selectedHoldingsCopyDelete method one more time to
use streaming response instead of getting all results in a single,
atomic call.  This should better handle large numbers of selected
items.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>

LP 1735539: Simplify item service selectedHoldingsCopyDelete.

Remove the var copy_objects and the unnecessary push of the results
into var copy_objects.

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

Lp 1735539: Fix Item Status ability to delete multiple copies.

Set atomic to true for the PCRUD search in the item service's
selectedHoldingsCopyDelete method, and do an angular.forEach over the
results to properly fill the copy_objects array so that all copies end
up in it.  As it was only 1 copy was getting into the array.

To reproduce the bug:
   1. Open Item Status in the web staff client.
   2. Load multiple items, perhaps by uploading a file of barcodes.
   3. Select more than 1 item from the list.
   4. Choose "Delete Items" from the Actions menu.
   5. Note that the dialog indicates that you will delete 1 copy.
   6. Click the Cancel button in the dialog.

To test, after applying the fix branch:
   1. Open Item Status in the web staff client.
   2. Load multiple items, perhaps by uploading a file of barcodes.
   3. Select more than 1 item from the list.
   4. Choose "Delete Items" from the Actions menu.
   5. Note that that the dialog indicates you will delete X copies,
      where X is the number of items selected.
   6. Click OK and confirm that all copies were deleted.
   7. You can confirm 6 by reloading the interface and uploading the
      file of barcodes again.

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

KPAC Art and Features for SRP 2018

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

Customization to default print template for Hold Shelf Slip

Fixed alignment of notification options.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

PINES Customization to Self-check Wording

Signed-off-by: McCanna <tmccanna@georgialibraries.org>

PINES Customization for Default Patron Data Print Template

Added formatting and signature block.

Signed-off-by: McCanna <tmccanna@georgialibraries.org>