LP#1269574 - make EDI also cancel acq-created copies

Before this change, acquisitions EDI did not attempt to
delete "real" copies attached to lineitem details. We
check for copies and delete them.

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

Add fix for PG segfaults during reingest.

Also add inline docs.

LP#1764542 - Update older sites to MODS 3.3

MODS 3.3 support was introduced in March 2009, but this
change was never included in an upgrade script.  Older
sites on 3.1+ will need to apply this update to avoid scary
segmentation faults, and it should be harmless for newer
sites and desirable for those on 3.0 or older.

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

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

Fix lingering git merge messages in upgrade docs.

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

lp1739286 default search box in Z39.50

This change tweaks the layout of search fields in the web client Z39.50
interface and adds radio buttons for the default search field to auto-focus.
The behavior is "sticky" without the need for an explicit Save action.  If the
current default field is not present due to a given combination of selected
services, then no field will be focused by default.

Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

LP1010635: Make bre dates populate in Item Status grid

To test:
1) Enable the Date Record Created and Date Record Last Edited columns
in Item Status.
2) Scan an item barcode into Item Status.  Note that the two columns
you enabled are empty.
3) Apply this commit.
4) Reload the Item Status screen, and scan your barcode again.  The
dates should now be populated.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
Signed-off-by: Garry Collum <gcollum@gmail.com>

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#1642344: XUL report cloning

This patch allows report templates of version 4 and earlier
(which would have been created by the XUL staff client) to
be upgraded to template version 5 (used by the web staff
client) when the user clones a report. This closes a limitation
introduced in the reporter module for the web staff client.

Note that during the conversion, inner joins are used by
default when IDL classes have a 'has_a' relation rather than
defaulting to left outer joins, improving correctness of the
generated SQL query.

To test
-------
[0] Apply the patch.
[1] In a test database that has reporter templates created
    by (or for) the XUL staff client, create clones of
    those templates.
[2] Verify that the output of reports created using the new
    templates matches output of reports from the original
    templates.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Terran McCanna <tmccanna@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#1736419: Bib visibility tests get OR'd

Either the source or the existance of a Located URI will trigger record
visibility, so we will OR them together.

NOTE: This branch is dependant on the branch supplied on LP#1730758. Without
that fix, the bib-level visibility attributes are not updated properly when
Located URIs are added or changed.

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

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>

Revert "LP#1764542 - Update older sites to MODS 3.3"

This reverts commit 1f4123fbcfc1285f2914ea7ec8bafa34accc572a.

LP#1764542 - Update older sites to MODS 3.3

MODS 3.3 support was introduced in March 2009, but this
change was never included in an upgrade script.  Older
sites will need to apply this update to avoid scary
segmentation faults, and it shouls be harmless for newer
sites.

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

PINES Local Customization: Hide Set Default View Button

This hides the Set Default View button that is in the web staff client
on the catalog item pages. Clicking the button has caused white screen
problems that have not been resolved. (There is also a button on the Admin >
Workstation page, but that was hidden in a different local customization
patch.)

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

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

Revert "temp fix for broken library locator"

This reverts commit a1ab46766e8fcb313bc039ccf46313fcc8054334.

temp fix for broken library locator

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 Temp Fix: Hide Adv Search Default Pane

Hide section that sets advanced search default pane
in web client because it causes problems in Chrome.
Can un-hide when bug 1731272 is fixed.

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>

Re-add GALILEO features.

lp1693036 Patron Editor Print Data
- Add Print button to Patron Editor.
- Add Patron Data print template.
- Add additional print template dummy data needed for Patron
Data template.
- Implement print functionality for Patron registration/edit
data.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/src/templates/staff/admin/workstation/t_print_templates.tt2
modified:   Open-ILS/src/templates/staff/circ/patron/reg_actions.tt2
new file:   Open-ILS/src/templates/staff/share/print_templates/t_patron_data.tt2
modified:   Open-ILS/web/js/ui/default/staff/admin/workstation/app.js
modified:   Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js

Modify custom default hold transit slip to include note
that item is needed for a hold. (This was at the request
of several libraries who prioritize their sorting based
on whether items are needed for holds or not.)

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

Updated PINES Map Graphic including LOPL

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

LP#1747542 - Repair item/user circulation test

The action.item_user_circ_test function was previously
ignoring lost or longoverdue and paid-for circulations.
Adding a check for whether the circulation is closed
fixes this.

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

KPAC Fix to broken image links

Removes cache info from image links in KPAC because
it was causing links on Dewey page to break.

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

fix IP redirection by pulling the X-Real-IP header

LP#1691269: (follow-up) include new cust in seed data

Also s/Copier/Copy/.

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

LP1691269: Webstaff Copy Editor Templates

This branch will translate and transfer your XUL client templates
the first time you load the web client copy editor. From that point
on the two diverge with no further connection to each other.

This branch also allows your copy templates to travel with you
between browsers, machines, and so on, while still allowing users
that share a browser but not usernames to have their own templates.

Signed-off-by: Jason Boyer <JBoyer@library.in.gov>
Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Revert "PINES Cust: Add shortname to alerts"

This reverts commit b814b414a79e98d5be790d1ba52d5d1056928426.

PINES Customization: Hide Claims Never Checked Out Action

On the patron account's Items Out screen, hide the
Mark Claims Never Checked Out action since we do not use
it in PINES and there is no associated permission to prevent
it from working.

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

Temp Fix for Patron Account Expiring Soon Message

Marked as temp fix for testing before packaging for submission
to launchpad bug 1726918.

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

PINES Cust: Add shortname to alerts

Add the short name of the library who created an alert to
the alert line in the patron summary bar.

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

PINES Customization: Set default damaged billing type

This sets the default billing type when marking an item damaged to
'Damaged Materials' which is ID 106 in our database.

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

PINES Customization: Require patron identification

Requires something to be entered in the ID value field

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

PINES Customization: Require Parent/Guardian Field

This shows a warning on screen if a juvenile patron doesn't have
the parent/guardian field filled in (although it doesn't actually
require it).

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

New Books List - Change to Wording
When no copies available, change wording from 'on order' to
'not available - ask at desk'

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

PINES Cust: Damaged popup button colors

Changed Charge / No Charge buttons to different color in response
to staff expressing confusion about which button would submit.

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

PINES Cust: Reset Patron Search Form with Defaults

Change code to refresh page instead of clearing individual fields,
which resets fields to the defaults we want (inactive checked and
org unit to consortium).

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

My Account Bills Screen

This fixes column header conflict between old changes and new.

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

PINES Customization: Hide Barred Checkbox on New Patron Reg

Hide the barred checkbox when creating a new patron account because
it is unnecessary. Display it only when editing accounts.

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

Hide Patron Search Button on Place Hold Screen

The functionality is currently broken (LP Bug 1724052) so hiding
the button until it is fixed.

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

Fix for PINES-Specific Holdings Sort Order

See Tom Berezansky's original fix at:
http://git.evergreen-ils.org/?p=evergreen/pines.git;a=commit;h=adc566e152d10796b73c992a16ae46e69cc3ab77

Some changes to the base code required this fix to be updated.

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

remove offline blocked list download button

Revert "Re-apply fix for Fix for org unit order in OPAC holdings list"

This reverts commit 83aef4b5a3714c87080de52082b9327036271ba7.

Re-apply fix for Fix for org unit order in OPAC holdings list

Revert "Fix for org unit order in OPAC holdings list (adapted from"

This reverts commit 02778825199abf50891b4c247cb65526fbe6824b.

PINES OPAC Fix: Holdings Navigation

Add in OPAC code from Master to calculate the number of copies.

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

adding PINES PO file

PINES Receipt Template Customizations for Web Client

Additional receipt template default changes.

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

PINES Search Results Customization: Display List Link

Display Add to List link even though it isn't completely
functional.

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

PINES Receipt Template Customizations for Web Client

Corrected branch error

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

Staff Client Customization: Browser Tab Naming

Changed tab names from Evergreen Staff XXX to PINES: XXX

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

Staff Client Customization: Backdate Checkin Alert

Makes onscreen checkin alert more obvious.

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

Staff Client Customization: Change Wording on PreCat Form

Change 'ISBN' to 'Call Number' on field label.

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

Staff Client Customization: Hide Delete Patrons Button

Hide Delete Patrons button from user buckets (function was
not allowed by permissions, so no need to even show it).

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

Staff Client Customization: Search Menu Wording

Changed Basic Search to Advanced Search to more accurately
reflect the screen it goes to.

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

Staff Client Customization: Patron Registration Form Tweaks

- Fixed alignment of checkboxes.
- Hid Claims returned on new patron registration form.
- Hid Claims never checked out.
- Made survey drop 'up' instead of dropdown so it
  doesn't disappear off of page.

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

Staff Client Customization: Hide Booking Menu

Hid Booking as a menu option since we don't use it in PINES.

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

KPAC Config File: Feature Page Reset

Fresh version of kpac.xml file ready for upgrade to 3.0.

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

OPAC Customization: My Lists

Reset customizations to master and reapplied PINES help tips.

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

OPAC Customization: Place Hold Screen

Code cleanup in preparation for 3.0 upgrade.

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

OPAC Customization: Login Screen

Fixed layout issues in preparation for 3.0 upgrade.

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

OPAC Customization: Item Details Page

Upgrade fixes to OPAC item details page - force Novelist content to stay
open, re-show List links in staff client, fix problems with duplicate
holdings tables and column adjustments, fixed e-resource link within the
staff client, removed print/email link.

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

OPAC Customization: Search Results Page

Add space around covers and adjust colors of filter bar.

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

OPAC Customization: Advanced Search Group Box

Remove duplicate group checkbox that was leftover when I moved the
position of the group checkbox before.

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

OPAC Customization: Home Page Feature Icons

Adjusted layout of feature icons so they wrap properly.

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

PINES Customization: Patron Reg Form - Parent Guardian Field Label

Relabel "Secondary Identification" field as "Parent/Guardian"
to be consistent with usage and xul client.

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

PINES Customization: Advanced Search Filter Results

Added line break before filter display to improve the way
they were wrapping.

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

PINES Customization: Group Records Checkbox

Had moved checkbox to a different spot on page in previous
release. This release added it back, which caused a duplicate
checkbox. This patch removes the duplicate.

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

PINES Staff Splash Screen for Web Client
Adjusted order of icons so that Reports is before Quick Reports

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

PINES Customization: Patron Registration Field Tips

Added examples for dob and GA DL format.

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

LP#1528647 Self-check only accepts user name value if regex
for barcode not set up

This fix allows a patron to log into the self-check interface with either
a barcode or a user name.

Written by: Bob Wicksall <bwicksall@pls-net.org>

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

PINES Customization: OPAC header sizing

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

Default Print Templates - PINES Customizations

Customized default print templates for PINES so that each library
will have a good starting point to work from.

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