LP#1729934 Webstaff penalty create org/depth repair

Apply correct org unit to patron penalties created in the web staff
client.  Specifically, be sure the standing penalty depth value is
honored when set.

Commit replaces manual pcrud creation with the XUL-tested API, which
calculates the correct org unit by depth for us.

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

LP#1746300: Fix circulation counts in Item Status Details

Currently, the "Total Circs -Current Year" and "Total Circs -
Prev Year" numbers are seemingly randomly incorrect.  This is
caused by pulling the numbers from the "circbyyr" fieldmapper
object, which results in an array of 2 rows (one for renewals
and one for new circs).  The current JS only displays the count
for the first item in the array, ignoring the other.

This branch totals the two, resulting in the right result for each
year's circulations.

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

LP#1758360: handle DOB display logic in egPatronSummary correctly

This patch ensures that the patron DOB will be displayed or
obscured correctly by the standalone egPatronSummary directive
in the same way that the patron app does.

To test
-------
[1] In the web staff client, do a patron search, select two patrons,
    then click the Merge Patrons button.
[2] Note that the patron DOB is not displayed in the modal.
[3] Apply the patch and repeat step #1.
[4] This time, the patron DOB will be displayed (if the
    circ.obscure_dob library setting is not enabled) or the
    the field label will be clickable (and clicking it will
    cause the DOB to be displayed).

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.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>

LP#1361258 Patron reg repair user settings saving/loading

Avoid caching user setting values at the service (app) level in the user
editor.  In other words, always fetch a fresh copy of the current user's
setting with each instance of the editor page.

This solves a number problems where stale versions of the cached data
were referenced and acted on, both when displaying an updating user
settings.

Additionally, ensure that clone user and stage user data is refreshed
on all page instances as the URL params that affect these values can
change per instance.

Other data that is common to all instances (e.g. the set of profile
groups) is still cached as before.

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

LP1721143: making permission group dropdown scrollable in patron create/edit interface

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

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

LP#1749992 Disable payment button during payment

Since the display is not refreshed until the payment finishes, it
is possible to fire off a second payment by double-clicking.

This commit is a simple way to disable the button while a payment is
in process, then enable it again after the display refreshes.

Since the 'sendPayment()' function is wrapped, we may need to move
this logic up, but this being a single promise is simpler, and it
solved the problem in testing.

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

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>

LP#1743262 CC external payment requires app. code

1. Indicate in the CC form that the approval_code field is required,
   styled consistently with the patron edit app.

2. Prevent the form dialog from closing via Submit when no approval code
   is set.

3. Handle payment failures better by preventing the post-API code from
   running (i.e. receipt printing) when the API fails.

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

LP#1739460: Be more careful when gathering shelving locations

Specifically, make sure we're getting the IDs, and comparing them as numbers.

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

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#1743608 Replace barcode discards NULL/accidental entries

When replacing a patron's barcode in the patron editor, discard any
previous replacement attempts done in the current edit session.  This
prevents cases where secondary, etc. replacement cards are accidentally
created or NULL cards that result in ugly error messages.

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

LP#1743819: Use combcirc rather than circ to get circ data

This addresses a merge regression.

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

LP#1734963: Teach copy template converter about older templates.

The XUL-to-web-client copy template converter created to fix
bug 1691269 needs a little help when the value stored is a
number rather than a string.

Thanks to Jeff Godin for troubleshooting help and Mike
Rylander for the suggested solution.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jeff Godin <jgodin@tadl.org>

Stamping upgrade script for xul-to-web copy template translation

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

LP#1691269: (follow-up) fix whitespace to match local style

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

LP#1691269: include volume fields in converted copy templates

This patch insures that call number prefix, suffix, and classification
scheme from copy templates created in the XUL unified copy editor are
brought over to web staff copy templates.

Note that in order to successfully test the call number portion of
converted copy templates in the web staff client, the volume/copy editor
default "Allow Call Number attributes in Copy Templates" should be
turned on.

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

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>

lp1742194 - Print Current Bills only printing circ bills

- Check xact for Circulation before assigning circulation data to
xact for printing.

Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
 Changes to be committed:
modified:   Open-ILS/web/js/ui/default/staff/circ/patron/bills.js

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>

LP1712646: Require a Billing Type to Bill User

Previously billing a user without choosing a billing type
would either fail silently or default to misc. This patch
requires staff to choose a billing type before the new
bill window can be dismissed.

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

LP1741072: Global String<->Num Directives for ngModel

To increase flexibility the stringToNumber directive
has been separated into int and float versions and
aditional directives are added to convert in the other
direction. The existing use of stringToNumber in the
volume/copy editor are also converted to strToFloat.

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

LP#1691861 - make Item Status edit items in batch in volcopy

This makes the Item Status grid action of "Edit Items" work
like the Copy Buckets grid "Edit Selected Items", in that the
copies open in the volcopy editor in batch, with no record summary.

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

Signed-off-by: Kathy Lussier <klussier@masslnc.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: Set default patron permission group

This should only affect new patron accounts.

Signed-off-by: Terran 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>

adding new CONTAINER_BATCH_UPDATE perm to appropriate groups

LP#1735807 Webstaff holdings owning lib shortname

Display org unit shortnames in the web staff bib record Holdings View
-> Owning Library column, since displaying the full names requires
significant horizontal space.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Jason Etheridge <jason@EquinoxInitiative.org>

LP#1736763: Wait until the org tree is there before using it

In offline mode, we need to wait for the org tree to be fetched or recreated
before we set the working location dropdown.  This commit adds a function
to loop and wait for that to happen.

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

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.

correct schema for trigger function

LP#1729922: correctly display most recent transit in item status

This patch ensures that the 'Most Recent Transit' display in
the web staff client's item status page shows the latest transit,
not the oldest one. This also fixes a bug where the transit
receive time never got displayed.  As a side effect, this patch
also ensures that only the transit in question is fetched from
the database, not all transits on the item.

To test
-------
[1] Set up an item with some closed transits and display in in
    detail view in the Item Status page. Note that the
    /oldest/ transit is displayed and that the transit
    receive time field is blank.
[2] Apply the patch and repeat step 1.
[3] This time, the most recent transit should be displayed, and
    if that transit was received, the receive time should be
    displayed.

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

PINES already has 1077

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 Customization: Offline Transactions Alert

Make text larger so that it is more obvious and easier to read.

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

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>

lp#1724321 Web Client -- existence of record with duplicate TCN does not prevent import

This routes the MARC import execution back to the z39.50 code instead of using the generic
MARC editor. This will make use of open-ils.cat.biblio.record.xml.import instead of pcrud.

Test
1. Edit global flag variable "Cat: Use Internal ID for TCN Value" = false
2. Perform a z39.50 search, click a result and click "import"
3. Step two should be successful, now import it again and you should get an error
4. Click the same result but use "Edit then import"
5. From the MARC editor, make no changes and click "Import"
6. It won't complain about duplicate TCNs
7. Apply the patch
8. Perform steps 4 and 5
9. Notice that the editor does not allow you to import the record

Signed-off-by: blake <blake@mobiusconsortium.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>

Web Client: Make Patron Email Clickable

The patch is twofold. First, "mailto" has been added to
$compileProvider.aHrefSanitizationWhitelist() throughout the project.
Second, the patron email in the patron summary has been wrapped in an
anchor with ng-href="mailto:{{patron().emaio()}}" to enable the desired
functionality.

Signed-off-by: Sam Link <slink@columbiacountyga.gov>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>

2.12-3.0: move auth reingest to separate file

Drop view before adding/replacing it.

PINES already has 1058

2.12-3.0: prevent pain by dropping biblio.record_entry triggers

adding 2.12-3.0 pre-upgrade script

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>

Revert "LP#1672346 - Copy Statistics View"

This reverts commit f4bb098fcff54de26eb64610749096f5329e15fb.

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>

PINES Customization: Hide FAST Subject Headings

See help desk ticket 4655.

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

PINES Customization: Mobile view for fines payment form

Modified table so that the form design doesn't go off the side
of the screen in mobile view.

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

Temp fix for Broken Hold Links

Temporary fix for https://bugs.launchpad.net/evergreen/+bug/1687319

Not an ideal fix because it requires javascript, but it'll do for now.

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

PINES Customization: Self-check - add note about online catalog

Added note to screen referring people to online catalog if they
wish to perform functions not available in self-check such as
paying fines and handling renewals.

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

PINES Customization: Self-check text changes

Modified login text to make it more user friendly.

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

PINES Customization: Add IMLS logo to footer

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

updating PINES survey display

PINES Customization: Adding menu link to Android App download.

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