sitka/evergreen.git
4 years ago(sitka-tpac) force metarecord display when MR has one constituent result
Jeff Davis [Fri, 13 Jun 2014 20:36:00 +0000 (13:36 -0700)]
(sitka-tpac) force metarecord display when MR has one constituent result

When Group Formats & Editions is selected, records are grouped by
metarecord; the MR title is followed by a count of constituent records
(i.e. all records for that MR which are relevant to your search), and
clicking on the title displays all the constituent records.  However,
when there is only one constituent record, clicking the MR title takes
you directly to the master record.  This is problematic if the
constituent record is NOT the master record: for example, if all your
search library's holdings are attached to a sub-record, that sub-record
is the only constituent record, but the holdings will not show in the
display for the master record.

As a workaround, this commit disables the special handling for
metarecords with a single constituent record.  Clicking a metarecord
search result will always take you to the list of constituent records
(even if there is only one such record), never directly to the master
record.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) Apply max width to Recent searches
Jeff Davis [Fri, 13 Jun 2014 21:33:50 +0000 (14:33 -0700)]
(sitka-tpac) Apply max width to Recent searches

4 years ago(sitka-tpac) Add browse button to search results page
Jeff Davis [Fri, 13 Jun 2014 19:43:00 +0000 (12:43 -0700)]
(sitka-tpac) Add browse button to search results page

4 years ago(sitka-tpac) cleanup search header in client
Jeff Davis [Fri, 13 Jun 2014 19:39:56 +0000 (12:39 -0700)]
(sitka-tpac) cleanup search header in client

  - update searchbar_staff.tt2 to more closely approximate upstream
    searchbar.tt2
  - CSS and layout improvements

Conflicts:

Open-ILS/src/templates/opac/results.tt2

(sitka-tpac) RT21640/RT21647: fix search/sort on search results

Previous TPAC customizations inadvertently broke the search form and
sort/limit/group formats & editions options on the search results page.
This commit repairs the damage.

(sitka-tpac) force white background on staff searchbar

(sitka-tpac) avoid duplicate maint-message in client

(sitka-tpac) RT21259: adjust size of input box in staff searchbar

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka-tpac) [RT24384] Expert search sort fixed

Currently, when sorting results form the Expert Search feature, the
results come back with no items.  The old code that handled this never
made it into our 2.6 branch.

I have updated this file with the working code.

4 years ago(sitka-tpac) unify numeric search form
Jeff Davis [Fri, 13 Jun 2014 19:38:22 +0000 (12:38 -0700)]
(sitka-tpac) unify numeric search form

Earlier Sitka customizations introduced a separate TT2 file,
numeric_staff.tt2, for displaying numeric search in the staff client.
But having two separate numeric search forms creates maintainability
headaches.  This commit removes numeric_staff.tt2, makes some staff
client-friendly adjustments to the main numeric.tt2, and updates other
TT2 files to use numeric.tt2 instead of numeric_staff.tt2.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) Tab background should be black when mouse over so that white text is...
Mark Bucholtz [Wed, 11 Jun 2014 17:28:42 +0000 (10:28 -0700)]
(sitka-tpac) Tab background should be black when mouse over so that white text is visible

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) RT21237: want_dojo on advanced search so that shelving location filter...
Jeff Davis [Fri, 6 Jun 2014 20:39:53 +0000 (13:39 -0700)]
(sitka-tpac) RT21237: want_dojo on advanced search so that shelving location filter works

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) rt21069 vertical align results screen place hold/reviews
Jeff Davis [Fri, 6 Jun 2014 19:58:51 +0000 (12:58 -0700)]
(sitka-tpac) rt21069 vertical align results screen place hold/reviews
vertically align Place Holds, Add to my List, Reviews & More
on results screen

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka-tpac) [SQUASH] reduce width of Place Hold etc to prevent alignment issues

4 years ago(sitka-tpac) make 2.6 Basic search page look like production 2.4
Mark Bucholtz [Fri, 6 Jun 2014 19:44:27 +0000 (12:44 -0700)]
(sitka-tpac) make 2.6 Basic search page look like production 2.4

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) center footer text in style css rec
Mark Bucholtz [Fri, 6 Jun 2014 02:21:44 +0000 (19:21 -0700)]
(sitka-tpac) center footer text in style css rec

4 years ago(sitka-tpac) rt21259 change Catalog to Catalogue
Mark Bucholtz [Thu, 5 Jun 2014 18:07:31 +0000 (11:07 -0700)]
(sitka-tpac) rt21259 change Catalog to Catalogue

4 years ago(sitka-tpac) rt21069 there is no background colour for My List
Mark Bucholtz [Thu, 5 Jun 2014 11:49:59 +0000 (04:49 -0700)]
(sitka-tpac) rt21069 there is no background colour for My List

which makes it hard to read because the list overlaps
with Reviews and More.

Just added back class="popUpDropDown" to ul line
in bookbag_actions.tt2 and added back

popUpDropDown {
    z-index:50;
    background-color: [% css_colors.footer_bg %];
    width: 100%;
    white-space: normal;
}
to css/style.css.tt2

4 years ago(sitka-tpac) Rewrite Place Hold link for Overdrive records in TPAC
Jeff Davis [Thu, 5 Jun 2014 23:21:20 +0000 (16:21 -0700)]
(sitka-tpac) Rewrite Place Hold link for Overdrive records in TPAC

If a record's bib source corresponds to an Overdrive source in
config.bib_source, replace the Place Hold link with a link to the
Overdrive page for that record, and change the link text to include the
item format (specifically, the link text uses the value of format_label,
which should be a patron-friendly string -- see misc_util.tt2 for
details).

The code assumes that Overdrive records will not be grouped with other
types of records when "Group Formats and Editions" is enabled.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka-tpac) [SQUASH] RT21190: fix metarecord Place Hold link

In 2.4, when the Place Hold link in search results is constructed in
result/table.tt2, the hold_type param is hard-coded to 'T'.  In 2.6,
metarecord holds are introduced, and the hold_type param uses the value
of the hold_type variable instead.  However, when merging commit e942e5a
(an unrelated Sitka customization), we introduced a regression whereby
the hold_type param was hard-coded to 'T' as in 2.4; this broke the
ability to place metarecord holds.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) [RT20195] Added return to record link
Liam Whalen [Fri, 30 May 2014 16:00:23 +0000 (09:00 -0700)]
(sitka-tpac) [RT20195] Added return to record link

When clicking on a library name within a record detail page, the user is
taken to the details for a library as those details are defined within
the Staff Client.

However, there is no link present that will return the user to the
record details page.  This forces the user to use the back button, which
is not desirable on touch screen interfaces.

To fix this, a link was added with the class of 'referer-link', and a
css entry for referer-link was addes in styles.css.tt2.

We use class instead of an id because referer-link may occur on more
than one page, so class is the correct selector to use.

Removed the Return to Record Details link from the Staff Client view of
a library's contact information.  The staff can use the Go Back button
if they need to return to the record details.

LP#1441755 Removed check for is_staff added check for record detail

Thanks to a suggestion from Mike Rylander, this commit has been
improved. There will no longer be a need for a check for is_staff once
the web staff client is released, so, in anticipation of that day, the
is_staff check has been removed.

As well, Mike pointed out that a return to record link should actually
be checked to ensure the referrer is returning to a record detail page.
So, this check has been added, and it has been moved to outside the div
instead of only surrounding the link because we do not want to include
an empty div if the link is not a referrer.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka-tpac) rt21072 add Browse the Catalogue option after Advanced Search option...
Mark Bucholtz [Fri, 30 May 2014 17:00:04 +0000 (10:00 -0700)]
(sitka-tpac) rt21072 add Browse the Catalogue option after Advanced Search option in sc

add Browse the Catalogue option after Advanced Search option and
before Refine Your Original Search option in staff client in
searchbar_staff.tt2 record

example

            class="adv_search_font">[% l('Advanced Search') %]</span></a>
            <span class="browse_the_catalog_lbl"><a href="[% mkurl(ctx.opac_root _ '/browse', {}, ['fi:has_browse_entry']) %]">[% l('Browse the Catalogue') %]</a></span>

and give 'Refine Your Original Search' it's own css label

.refine_original_search_lbl {
    font-size: [% css_fonts.size_bigger %];
}
in order to make the size adjustable to match the existing browse catalog
and advanced search labels fonts

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) make staff searchbar use new search_format filters
Mark Bucholtz [Wed, 28 May 2014 18:14:16 +0000 (11:14 -0700)]
(sitka-tpac) make staff searchbar use new search_format filters

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) adjust layout of advanced search screen
Jeff Davis [Wed, 28 May 2014 19:41:56 +0000 (12:41 -0700)]
(sitka-tpac) adjust layout of advanced search screen

Move Search Library, Limit to Available, Sort Results, and Group Formats
and Editions filters to a new "global filters" section of the Search
Input area instead of the Search Filters area.

As currently implemented, this involves duplicating some code and means
that these filters can't be selectively disabled on a per-skin basis.
It would be better to add configuration options for the global filters
in config.tt2 and move the original Search Filters loop to an included
file.

Also, in default config.tt2, replace Item Form and Item Type filters
with a single Format filter displaying search formats, and move shelving
location selector to end of filter block.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka-tpac) [SQUASH] rt21237 change copy loc drop box to 4 lines from 3

(sitka-tpac) [SQUASH] advanced search filters: include item format, adjust layout

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka-tpac) [SQUASH] RT#38288: move Exclude Electronic Resources checkbox on advanced search

The "Exclude Electronic Resources" checkbox really belongs with the
"Limit to Available" and "Group Formats and Editions" checkboxes, which
we have moved higher up on the Advanced Search screen (above the other,
less-used search filters).

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) RT16562 use correct search boxes on Temporary List
Mark Bucholtz [Tue, 2 Jul 2013 12:53:41 +0000 (05:53 -0700)]
(sitka-tpac) RT16562 use correct search boxes on Temporary List

Do search. Click Add to My List and choose Temporary List.  The header
on the screen shows the search box used on OPAC (a single search box).
Use the double search boxes we used on the Staff Client.

Conflicts:

Open-ILS/src/templates/opac/temp_warn.tt2

4 years ago(sitka-tpac) prevent numeric search text from displaying in keyword search results...
Mark Bucholtz [Fri, 28 Jun 2013 19:12:30 +0000 (12:12 -0700)]
(sitka-tpac) prevent numeric search text from displaying in keyword search results box in sc

4 years ago(sitka-tpac) numeric search changes
Mark Bucholtz [Wed, 26 Jun 2013 16:24:18 +0000 (09:24 -0700)]
(sitka-tpac) numeric search changes

Conflicts:

Open-ILS/src/templates/opac/home.tt2
Open-ILS/src/templates/opac/parts/myopac/base.tt2
Open-ILS/src/templates/opac/place_hold.tt2
Open-ILS/src/templates/opac/record.tt2
Open-ILS/src/templates/opac/results.tt2

4 years ago(sitka-tpac) new records for STAFF numeric, searchbar and topnac tt2 records
Mark Bucholtz [Fri, 14 Jun 2013 21:18:51 +0000 (14:18 -0700)]
(sitka-tpac) new records for STAFF numeric, searchbar and topnac tt2 records

(sitka-tpac) sticky search location on result page in staff client

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) RT16624: hover background color on active advanced search tab
Jeff Davis [Wed, 21 May 2014 20:28:33 +0000 (13:28 -0700)]
(sitka-tpac) RT16624: hover background color on active advanced search tab

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) change wording of message on barcode entry form
Mark Bucholtz [Wed, 21 May 2014 20:25:34 +0000 (13:25 -0700)]
(sitka-tpac) change wording of message on barcode entry form

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) add my account option if ctx.is_staff
Mark Bucholtz [Mon, 17 Jun 2013 14:31:38 +0000 (07:31 -0700)]
(sitka-tpac) add my account option if ctx.is_staff

Conflicts:

Open-ILS/src/templates/opac/advanced.tt2

4 years ago(sitka-tpac) adding piwik tracking to the catalogue
kreed [Wed, 11 Dec 2013 21:53:41 +0000 (13:53 -0800)]
(sitka-tpac) adding piwik tracking to the catalogue

4 years ago(sitka-tpac) RT17079 applied stop fines value to display instead of only showing...
kreed [Wed, 18 Sep 2013 03:13:52 +0000 (20:13 -0700)]
(sitka-tpac) RT17079  applied stop fines value to display instead of only showing "fines accruing"

4 years ago(sitka-tpac) RT17079 fallback to dummy title/author if title/author are black so...
kreed [Thu, 5 Sep 2013 19:47:10 +0000 (12:47 -0700)]
(sitka-tpac) RT17079 fallback to dummy title/author if title/author are black so that pre-cats display properly

4 years ago(sitka-tpac) [RT:17173] use locg instead of physical_loc for scoping
Jeff Davis [Wed, 28 Aug 2013 19:40:54 +0000 (12:40 -0700)]
(sitka-tpac) [RT:17173] use locg instead of physical_loc for scoping

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

Open-ILS/src/templates/opac/parts/topnav_secondary.tt2

4 years ago(sitka-tpac) RT16963: don't show publication date after author in results
Jeff Davis [Fri, 16 May 2014 21:45:28 +0000 (14:45 -0700)]
(sitka-tpac) RT16963: don't show publication date after author in results

Since we show record details (including publisher info) by default in
search results, putting the pub date after the author's name is
redundant.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) [rt16749] Remove 'NO COPIES' in Staff Client
Liam Whalen [Wed, 17 Jul 2013 00:23:57 +0000 (17:23 -0700)]
(sitka-tpac) [rt16749] Remove 'NO COPIES' in Staff Client

When a record has only OPAC invisible items attached to it, the TPAC
displays a 'NO COPIES' message in the staff client.

This fix skips the check for opac visible items when the staff client is
being used.  This can result in incomplete information being displayed in
the summary of the available items because opac invisible items are
often incomplete and do not have shelving locations or libraries
assigned to them.

(sitka-tpac) [rt16749] Fix the exclusion of item statuses

The previous commit removed the No Copies message, but it also removed
all item information such as Library, Shelving Location, Call Number,
and Status.

This fix moves the search of ctx.is_staff into a number of lines instead
of enclosing a group of lines in an if statement.  This results in
holdings being excluded only if the copy is not visible and the
ctx.is_staff variable is not set.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka-tpac) comment out display of call# on bib results page
Mark Bucholtz [Wed, 26 Jun 2013 19:47:40 +0000 (12:47 -0700)]
(sitka-tpac) comment out display of call# on bib results page

Conflicts:

Open-ILS/src/templates/opac/parts/result/table.tt2

4 years ago(sitka-tpac) RT16621: TPAC improvements to reduce loadtime
Jeff Davis [Fri, 16 May 2014 20:46:06 +0000 (13:46 -0700)]
(sitka-tpac) RT16621: TPAC improvements to reduce loadtime

Calls to org_within_scope and org_within_hiding_scope in
org_selector.tt2 did not include the "ctx." prefix, which (according to
Galen Charlton) "for an as-yet unknown reason appears to cause Template
Toolkit to spin a bit before it locates the subroutine."  This commit
adds that prefix.

Initial testing suggests that these changes improve loadtimes on TPAC
pages by several seconds.

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

Open-ILS/src/templates/opac/parts/misc_util.tt2

4 years ago(sitka-tpac) RT16450 adding hold pick up location filter function
kreed [Fri, 21 Jun 2013 23:10:01 +0000 (16:10 -0700)]
(sitka-tpac) RT16450 adding hold pick up location filter function

4 years ago(sitka-tpac) RT16451 bringing search scope restrictions into 2.4
kreed [Fri, 21 Jun 2013 22:59:23 +0000 (15:59 -0700)]
(sitka-tpac) RT16451 bringing search scope restrictions into 2.4

4 years ago(sitka-tpac) add 856u map to misc_util.tt2 for use in results.tt2
Mark Bucholtz [Thu, 20 Jun 2013 13:00:32 +0000 (06:00 -0700)]
(sitka-tpac) add 856u map to misc_util.tt2 for use in results.tt2

4 years ago(sitka-tpac) add 2 new css elements in style.ccs.tt2 for no copies and electronic...
Mark Bucholtz [Wed, 19 Jun 2013 21:13:50 +0000 (14:13 -0700)]
(sitka-tpac) add 2 new css elements in style.ccs.tt2 for no copies and electronic resources

4 years ago(sitka-tpac) turn off some search limit options
Mark Bucholtz [Mon, 17 Jun 2013 13:53:25 +0000 (06:53 -0700)]
(sitka-tpac) turn off some search limit options

Conflicts:

Open-ILS/src/templates/opac/parts/config.tt2

4 years ago(sitka-tpac) rt15915 adding clear form button to searchbar
kreed [Fri, 31 May 2013 22:06:05 +0000 (15:06 -0700)]
(sitka-tpac) rt15915 adding clear form button to searchbar

Conflicts:

Open-ILS/src/templates/opac/parts/searchbar.tt2

4 years ago(sitka-tpac) changing facet lables and what ones are shown to sitka BFG choices
kreed [Thu, 30 May 2013 20:28:40 +0000 (13:28 -0700)]
(sitka-tpac) changing facet lables and what ones are shown to sitka BFG choices

4 years ago(sitka-tpac) turn on 500a in full bib detail
Jeff Davis [Wed, 29 Oct 2014 22:10:21 +0000 (15:10 -0700)]
(sitka-tpac) turn on 500a in full bib detail

Conflicts:

Open-ILS/src/templates/opac/parts/misc_util.tt2

Conflicts:

Open-ILS/src/templates/opac/parts/record/summary.tt2

4 years ago(sitka-tpac) add 580 to bib detail screen
Jeff Davis [Wed, 29 Oct 2014 22:08:52 +0000 (15:08 -0700)]
(sitka-tpac) add 580 to bib detail screen

Conflicts:

Open-ILS/src/templates/opac/parts/misc_util.tt2

Conflicts:

Open-ILS/src/templates/opac/parts/record/summary.tt2

4 years ago(sitka-tpac) change the wording of the password hint
Mark Bucholtz [Fri, 10 May 2013 10:43:17 +0000 (03:43 -0700)]
(sitka-tpac) change the wording of the password hint

Conflicts:

Open-ILS/src/templates/opac/parts/login/password_hint.tt2

Conflicts:

Open-ILS/src/templates/opac/parts/login/password_hint.tt2

(sitka-tpac) rt17270 adding alternat password help text based on library setting

Conflicts:

Open-ILS/src/templates/opac/parts/login/password_hint.tt2

4 years ago(sitka-tpac) [RT:15674] TPAC: preferred search location and holdings display
Jeff Davis [Tue, 7 May 2013 01:32:23 +0000 (18:32 -0700)]
(sitka-tpac) [RT:15674] TPAC: preferred search location and holdings display

- Hide "Show preferred library" link to avoid patron confusion.
- Fix scoping issues with linked searches from My Holds page, so that
  current physical location's holdings are shown in holdings display and
  hits from libraries other than the current location and preferred
  search library are not displayed in holdings summary.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-tpac) map 264 rda subfields for display
Jeff Davis [Wed, 29 Oct 2014 22:05:44 +0000 (15:05 -0700)]
(sitka-tpac) map 264 rda subfields for display

4 years ago(sitka-tpac) use Sitka logo for tpac
Mark Bucholtz [Fri, 5 Apr 2013 11:25:54 +0000 (04:25 -0700)]
(sitka-tpac) use Sitka logo for tpac

4 years ago(sitka-tpac) RT15384 Adding volume level holds for all users
kreed [Wed, 20 Mar 2013 19:51:47 +0000 (12:51 -0700)]
(sitka-tpac) RT15384 Adding volume level holds for all users

Conflicts:

Open-ILS/src/templates/opac/parts/record/copy_table.tt2

4 years ago(sitka-tpac) [RT:15425] adding and clairifing hold error messages for tpac
kreed [Tue, 5 Mar 2013 05:35:23 +0000 (21:35 -0800)]
(sitka-tpac) [RT:15425] adding and clairifing hold error messages for tpac

4 years ago(sitka-tpac) Disable the "awards" aka "Awards, Reviews, Suggested Reads"
James Fournie [Wed, 27 Feb 2013 22:35:06 +0000 (14:35 -0800)]
(sitka-tpac) Disable the "awards" aka "Awards, Reviews, Suggested Reads"
extras area in the TPAC.  This never contains content.  This may need
to be enabled for Novelist users.

(sitka-tpac) RT15500 adding the awards record extra for sites with novelist set up

4 years ago(sitka-tpac) Don't display Help/FAQ on login form as it would be annoying to maintain...
James Fournie [Mon, 17 Sep 2012 00:06:30 +0000 (17:06 -0700)]
(sitka-tpac) Don't display Help/FAQ on login form as it would be annoying to maintain these
links and restyle these buttons for everyone

4 years ago(sitka-backport) [RT45943] LP#1686194 Account for adjustments when generating fines
Jeff Davis [Fri, 28 Apr 2017 16:30:44 +0000 (09:30 -0700)]
(sitka-backport) [RT45943] LP#1686194 Account for adjustments when generating fines

Fine generation stops at a predetermined 'max fines' amount, but this
does not account for possible adjustments which lower the effective
fine amount.  This situation is common in cases where lost items are
returned, and the library wants to generate additional overdues upon
that event.

With this commit, fine generation will pull in any adjustments applied
to overdue fines, and use those to offset the current total fine when
checking for max fines.

Also includes test for this condition.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Conflicts:
Open-ILS/src/perlmods/live_t/09-lp1198465_neg_balances.t

4 years ago(sitka-backport) LP#1681466: move text to body of confirm dialog and add title
Galen Charlton [Mon, 24 Apr 2017 17:16:21 +0000 (13:16 -0400)]
(sitka-backport) LP#1681466: move text to body of confirm dialog and add title

This patch tweaks the opt-in confirm dialog to add a
title and move the text to the body of the dialog, making it
consistent with other dialogs that ask long questions.

Test case for the entire bug fix
--------------------------------
[1] Ensure that <opt_in> is set to true in opensrf.xml.
[2] In the web staff client, open the checkout page
    and do a barcode search for a patron is *not* the
    same system as the workstation that the staff user
    is registered to.
[3] Verify that a dialog box is presented asking the
    staff user to confirm that personal information can be
    shared with the workstation's library.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years ago(sitka-backport) LP#1681466: Text for web client opt-in dialog
Jeff Davis [Wed, 19 Apr 2017 16:42:38 +0000 (09:42 -0700)]
(sitka-backport) LP#1681466: Text for web client opt-in dialog

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-backport) [RT45509] LP#1257915 Repair receive-when-cancel query
Bill Erickson [Wed, 12 Apr 2017 20:54:25 +0000 (16:54 -0400)]
(sitka-backport) [RT45509] LP#1257915 Repair receive-when-cancel query

Repaire the json_query used to determine if a PO is ready to be marked
received.

A PO is non-receiveable if it has any lineitems that are not in the
received/cancelled [sic] state OR any that are canceled with a
keep_debits=true cancel reason.

Prior to this change, simply having a state of "cancelled" was enough to
prevent receiving.

* Replace tabs w/ spaces
* Remove unnecessary "or return 0" clause which was causing Perl
  precedence warnings.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years ago(sitka-backport) [RT45509] LP#1257915 - Also check whether to mark the PO received...
Chris Sharp [Fri, 7 Apr 2017 12:45:06 +0000 (08:45 -0400)]
(sitka-backport) [RT45509] LP#1257915 - Also check whether to mark the PO received when canceling.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
4 years ago(sitka-backport) [RT45509] LP#1257915 - Mark POs received when all lineitems are...
Chris Sharp [Wed, 5 Apr 2017 13:52:50 +0000 (09:52 -0400)]
(sitka-backport) [RT45509] LP#1257915 - Mark POs received when all lineitems are received or canceled.

Previously, Evergreen only considered lineitems not in a "received" status
when deciding whether to consider a purchase order to be "received".  Now
items with cancel reasons that do not keep debits (e.g. not backordered)
are considered "done".

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
4 years ago(sitka) 2.12 SQL: [RT45345] new perms and org settings
Jeff Davis [Wed, 5 Apr 2017 21:04:24 +0000 (14:04 -0700)]
(sitka) 2.12 SQL: [RT45345] new perms and org settings

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: [RT45174] switch to upstream subject browse indexes
Jeff Davis [Mon, 3 Apr 2017 22:28:47 +0000 (15:28 -0700)]
(sitka) 2.12 SQL: [RT45174] switch to upstream subject browse indexes

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: set "Is Available" flag for Reserves, On Display, and Onsite Consul...
Jeff Davis [Mon, 3 Apr 2017 18:19:01 +0000 (11:19 -0700)]
(sitka) 2.12 SQL: set "Is Available" flag for Reserves, On Display, and Onsite Consultation statuses

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: move long-running 2.12 updates to post
Jeff Davis [Tue, 28 Mar 2017 22:53:37 +0000 (15:53 -0700)]
(sitka) 2.12 SQL: move long-running 2.12 updates to post

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: purge old user activity
Jeff Davis [Tue, 28 Mar 2017 22:30:19 +0000 (15:30 -0700)]
(sitka) 2.12 SQL: purge old user activity

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: set Is Available flag for custom On Display status
Jeff Davis [Tue, 28 Mar 2017 22:17:59 +0000 (15:17 -0700)]
(sitka) 2.12 SQL: set Is Available flag for custom On Display status

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: skipping bib source update for all MARC records
Jeff Davis [Tue, 28 Mar 2017 22:12:06 +0000 (15:12 -0700)]
(sitka) 2.12 SQL: skipping bib source update for all MARC records

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) 2.12 SQL: include our modifications to query parser
Jeff Davis [Tue, 28 Mar 2017 21:51:35 +0000 (14:51 -0700)]
(sitka) 2.12 SQL: include our modifications to query parser

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) modify hard-coded install path in web client
Jeff Davis [Tue, 7 Mar 2017 17:30:53 +0000 (09:30 -0800)]
(sitka) modify hard-coded install path in web client

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT44082] use library currency as patron currency instead of always using USD
Jeff Davis [Tue, 21 Feb 2017 01:20:26 +0000 (17:20 -0800)]
(sitka) [RT44082] use library currency as patron currency instead of always using USD

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [SQUASH] [RT42059] fix leader in sitka_object.xml MARC template
Jeff Davis [Thu, 3 Nov 2016 04:37:40 +0000 (21:37 -0700)]
(sitka) [SQUASH] [RT42059] fix leader in sitka_object.xml MARC template

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT42179] display full library name, not shortname, in patron summary
Jeff Davis [Wed, 26 Oct 2016 21:34:39 +0000 (14:34 -0700)]
(sitka) [RT42179] display full library name, not shortname, in patron summary

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT38630] OPAC format icons for Online and E-serial
Jeff Davis [Tue, 28 Jun 2016 18:43:08 +0000 (11:43 -0700)]
(sitka) [RT38630] OPAC format icons for Online and E-serial

These "new" images for our custom Online and E-serial OPAC format icons
are modified copies of mattype/t.png and mattype/media_onlinejournal.png,
respectively.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-backport) [RT39537] Revert SIP portions of LP#1499123 from commit 831a808
Jeff Godin [Sun, 22 May 2016 18:38:39 +0000 (14:38 -0400)]
(sitka-backport) [RT39537] Revert SIP portions of LP#1499123 from commit 831a808

SIP Patron Information and Patron Status requests fail when the patron
has standing penalties, causing the SIP worker to crash.  This interim
fix reverts the changes from LP#1499123 which caused such requests to
fail.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [SQUASH] [RT16923] [RT38648] LP#1429317: tweak handling of ORDER BY clause...
Jeff Davis [Fri, 29 Apr 2016 22:57:28 +0000 (15:57 -0700)]
(sitka) [SQUASH] [RT16923] [RT38648] LP#1429317: tweak handling of ORDER BY clause for serial holdings retrieval

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [SQUASH] [RT16923] [RT38648] LP#1429317: skip sort by monograph part label

For some reason, ordering by bmp.label_sortkey causes an error when
doing Sitka-style sorting of serial holdings.  We barely use monograph
parts, so it's easiest just to leave out that field when sorting.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [SQUASH] [RT33838] fix copy filter for serials in mk_copy_query
Jeff Davis [Fri, 29 Apr 2016 22:56:06 +0000 (15:56 -0700)]
(sitka) [SQUASH] [RT33838] fix copy filter for serials in mk_copy_query

4 years ago(sitka) [RT16923] [RT38648] LP#1429317 Modify sort of holdings in record view
Liam Whalen [Fri, 11 Oct 2013 19:31:55 +0000 (12:31 -0700)]
(sitka) [RT16923] [RT38648] LP#1429317 Modify sort of holdings in record view

Currenlty, holdings in the record and search resutls are ordered via
their call number.  When searching at a higher org unit, the holdings
are first ordered within org units and then ordered according to call
number.

This commit adds the Library Setting opac.holding_sort_type.  This
setting has three posible options.  The first and default if no correct
option is chosen is 'call', which sorts by call number.  The other two
are 'asc' and 'desc'.  'asc' sorts from the oldest item to the newest
item.  'desc' sorts from the newest item to the oldest item.

In order to make this YAOS work with the Staff Client, the commit adds a
value to the TPAC ctx variable.  The value added is ctx.ws_ou.  This
ensures that the correct setting for the Staff Client's workstation is
used rather than the setting for the org unit being searched.

Move MFHD_SUMMARIZED_SUBFIELDS to Const.pm.  The
MFHD_SUMMARIZED_SUBFIELDS constant is defined in
OpenILS/perl/lib/Application/Serial/OPAC.pm.  It is used in the process
of populating the Issues Held drop down for serial's module records.
This commit also uses this variable with its
rank_serial_copy_by_issuance plpgsql function.  In order to minimize
errors where one version of the variable is updated while another is
left as it currently is, I moved the variable to Const.pm.  The code to
dereference the constant hash reference requires two lines.  It would
be nice if it could be done in one, but I could not figure out the
syntax to dereference a constant.

Added rank_ou_root_search_with_pref_lib SQL function.
This SQL function ranks OUs at the same level unless they belong the
the preferred library entered.  It is meant to be used with a search at
the root level of the org tree, so that all org units will be ordered
alphabetically except the pref_lib and its descendants.

This function is needed to sort holdings correctly when searching at the
Consortial level.  As the code is now, when a search is perforemd at the
Consortial level, the org units are sorted according to their distance
from the Consortial OU.  This results in a confusing listing unless the
reader is aware of the org unit tree layout.

In order to fix this, the commit sorts only by org unit name
(alphabetically) when a consortial search is performed.  However, if the user
has a preferred library set, then the sort needs to rank the preferred
OU and its decendants higher than all others, so this new function,
rank_ou_root_search_with_pref_lib does this.  It is a modified version
of rank_ou, with all but the pref_lib logic cut out.

The sorting of items according to the LS opac.holding_sort_type is done
in a number of ways.  First, records are put into two groups.  Pure
serial module records (a record with assets added only via the serials
module) and all other records.  In some cases, a library may put only
serials module items on a record while another library may put regular
items on a record.  This results in a hybrid record.  Hybrid records are
ordered as if they were regular records instead of serials records.

Pure serials module records are ordered via the data in
serial.materialized_holding_code if either 'asc' or 'desc' are chosen.
All other records are ordered by active date then asset.copy id when
'asc' or 'desc' are chosen.  The copy id is used to maintain order when
items have identical active dates.  This can happen with serials module
records. The copy id is needed to keep the holdings of hybrid records
ordered properly.  Because active_date and copy id are used to order
items in 'asc' and 'desc' orders the previous order which ordered items
within call numbers via their copy statuses no longer works.  Items are
ordered strictly by their age within a collection.

If call is chosen for the LS then all records are ordered in the same
manner with their call number determining their order within their org
unit.  In this case, copy status still applies to the order.  In effect,
using the LS of 'call' retains the current functionality of the TPAC.

The ordering of pure serials module's holdings is done with a plpgsql
function named rank_serial_copy_by_issuance.  This function creates a
number that is used by the SQL copy query to maintain the proper
chronological or enumerrative order.

Finally, there is a small bug in the curent TPAC when browsing through
items in the record view.  If the number of items returned is evenly
divisible by the number of copies per page, then the final page of
items will have a Next link that will take the user to a page with no
items and no links to navigate back to the previous items.  This commit
increases the number of items returned by the query for copies by 1, so
the TPAC can determine if there are more items waiting to be displayed,
and if there are then it shows the Next link.  Otherwise, no Next link
is displayed.

The serial.rank_serial_copy_by_issuance function ranks serials added via
the Serials moduel according to the values in
serial.materialized_holding_code.  It currently causes an internal
server error if an issue is encountered that has a combined month e.g
Jan/Feb or Jun/Jul. In these cases the materialized_holding_code is a
value of 01/02 or 06/07 in the above examples.  The function is looking
for an integer value, and the / in the materialized_holding_code is
causing a Postgresql SQL error, which in turn causes the internal server
error in Evergreen.

This fix looks for the first occurance of a string of digits in
materialized_holding_code and uses that as the value to rank the
issuance.  This should preseve the correct order because Jan/Feb will
use 01, while the next issue if it is March will use 03 and still be
placed after Jan/Feb or before it if the order is descending.  The same
thing will occur with enumerative order.

The only unknown at this point is the possibility of a
materialized_holding_code without a numeric value.  This code takes that
into accout, but uses a default value of 0, which will not help order
the issue, but it will stop internal server errors from occuring.

The possiblity of non-numeric materialized_holding_code values needs to
be investigated.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Conflicts:
Open-ILS/src/sql/Pg/950.data.seed-values.sql
Open-ILS/src/templates/opac/parts/result/table.tt2

4 years ago(sitka) modifications to 2.10 SQL version upgrade scripts
Jeff Davis [Tue, 8 Mar 2016 20:34:27 +0000 (12:34 -0800)]
(sitka) modifications to 2.10 SQL version upgrade scripts

(sitka) 2.10 SQL: preserve local changes to acq.rollover_funds_by_org_tree

(sitka) 2.10 SQL: skip some updates that have already been applied on prod

(sitka) 2.10 SQL: grab 2.10 beta upgrade script (which is not yet in the upstream 2.10 branch)

(sitka) 2.10 SQL: skip creation of stock 655 index, since we already have a custom index for this

(sitka) 2.10 SQL: skip org.restrict_opt_to_depth setting, which we already have on prod

(sitka) 2.10 SQL: move Sitka customizations to main 2.10 upgrade script, dropping the beta version

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) 2.10 SQL: move some long running updates to post

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) 2.10 SQL: Do not use default reingest commands

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) 2.10 SQL: [RT38284] Update org settings and permissions

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) 2.10 SQL: [RT38723] enable E-video format filter

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) 2.10 SQL: [SQUASH] [RT38723] change search label for e-video

4 years ago(sitka) [RT36524] [RT18711] opt-in check has more than two possible values
Jeff Davis [Wed, 27 Jan 2016 20:37:54 +0000 (12:37 -0800)]
(sitka) [RT36524] [RT18711] opt-in check has more than two possible values

The open-ils.actor.user.org_unit_opt_in.check method's return values
have been augmented: it now returns 2 if opt-in is disallowed by the
patron's home library.  This commit fixes a now-outdated assumption that
the opt-in check will only return 0 or 1.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [SQUASH] [RT36524] [RT18711] bugfix for restricted opt-in

A bug in a previous commit caused the opt-in warning to appear
incorrectly on patron retrieval, even when opt-in is not required.
This commit fixes the problem by being more strict about checking return
values.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT34656] Removed CREATE_PICKLIST perm
Liam Whalen [Wed, 2 Dec 2015 22:50:11 +0000 (14:50 -0800)]
(sitka) [RT34656] Removed CREATE_PICKLIST perm

The creation of widgets for the purchase order and selection list does
not handle multple permissions easily.  This commit reduces the perm
needed to create the owner and other widgets to a single perm.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka) [RT34078] Pull KPacConfigFile from environment
Liam Whalen [Mon, 23 Nov 2015 21:41:51 +0000 (13:41 -0800)]
(sitka) [RT34078] Pull KPacConfigFile from environment

The puppet apache configs use SetEnvIfNoCase to set values for the
Apache state.  The old method used PerlSetVar and PerlAddVar in the
VirtualHost defininition.  PerlSetVar and PerlAddVar require the values
be retrieved from self->apache->dir_config.  However, now that we set
things via SetEnvIfNoCase self->apache->dir_config will be empty so we
need to check $ENV instead.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm

4 years ago(sitka) [RT33838] Speed up bookbag retrieval by making acn the core class in basic_op...
Jeff Davis [Wed, 16 Sep 2015 20:02:06 +0000 (13:02 -0700)]
(sitka) [RT33838] Speed up bookbag retrieval by making acn the core class in basic_opac_copy_query

Thanks to Mike Rylander for assistance with this.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [SQUASH] [RT33838] [RT35553] fix copy filter in basic_opac_copy_query

4 years ago(sitka) enable load time stats in OPAC
Jeff Davis [Wed, 4 Nov 2015 17:09:19 +0000 (09:09 -0800)]
(sitka) enable load time stats in OPAC

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT35168] temporary workaround for non-matching cover image (1/2)
Jeff Davis [Wed, 14 Oct 2015 16:53:35 +0000 (09:53 -0700)]
(sitka) [RT35168] temporary workaround for non-matching cover image (1/2)

4 years ago(sitka) [RT32858] Fix Batch update of Funds in Selection Lists
Liam Whalen [Sat, 19 Sep 2015 03:24:14 +0000 (20:24 -0700)]
(sitka) [RT32858] Fix Batch update of Funds in Selection Lists

Selection lists are currently unable to batch update funds.  The problem
stems from the fact that the Fund selection list is not populated in the
copy rows.  New code used to limit the number of funds, in the selection
list, to those funds available to the user was added for purchase
orders.  However, these changes were not added to Selection Lists.  This
code adds the new call to initBatchUpdater to the selection list code.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka) grab OILSWebTemplatePath from $ENV if set
Jeff Davis [Sun, 12 Jul 2015 18:14:12 +0000 (11:14 -0700)]
(sitka) grab OILSWebTemplatePath from $ENV if set

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) simplified EDI install for Ubuntu 14.04
Jeff Davis [Fri, 10 Jul 2015 19:17:25 +0000 (12:17 -0700)]
(sitka) simplified EDI install for Ubuntu 14.04

Previously, the install.sh script for EDI had some ugly workarounds: the
latest rubygems was installed from SVN, and gems that are now packaged
in Ubuntu were downloaded using gem install.  This commit omits these
workarounds and uses Ubuntu packages wherever possible.  For testing
purposes, we also omit rspec and rcov, which are probably not really
needed.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [RT33242] Modified installation of openils-mapper Ruby gem.

The openils-mapper gem installed by default does not contain the changes
made by Bill Erickson or by myself.  This code clones the BC Libraries
Cooperative's GitHub repo of openils-mapper and installs it from there
instead.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka) selfcheck: show skin logo and "Self Check" header
Mark Bucholtz [Tue, 30 Jun 2015 22:05:51 +0000 (15:05 -0700)]
(sitka) selfcheck: show skin logo and "Self Check" header

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) selfcheck: UI improvements
Mark Bucholtz [Tue, 30 Jun 2015 22:03:16 +0000 (15:03 -0700)]
(sitka) selfcheck: UI improvements

    - change text of hold status to green if it's 'Ready for pickup'
    - [RT19097] change text for success logins to green, red for failure
    - better wording for various messages

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) selfcheck: if opac.barcode_regex is not set, assume a barcode has been entered
Mark Bucholtz [Tue, 30 Jun 2015 21:37:44 +0000 (14:37 -0700)]
(sitka) selfcheck: if opac.barcode_regex is not set, assume a barcode has been entered

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) selfcheck: [RT19020/RT19096] check whether patron is opted-in and/or expired
Mark Bucholtz [Tue, 30 Jun 2015 21:04:53 +0000 (14:04 -0700)]
(sitka) selfcheck: [RT19020/RT19096] check whether patron is opted-in and/or expired

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) selfcheck: turn printing on or off using org settings
Mark Bucholtz [Tue, 30 Jun 2015 20:59:17 +0000 (13:59 -0700)]
(sitka) selfcheck: turn printing on or off using org settings

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) selfcheck: [RT17771] fix receipt printing
Mark Bucholtz [Tue, 30 Jun 2015 20:33:17 +0000 (13:33 -0700)]
(sitka) selfcheck: [RT17771] fix receipt printing

Clicking on the print option on the hold display page does nothing.
console.log displays:

'Uncaught TypeError: Cannot read property 'length' of undefined'.

The error can be traced to the printHoldsReceipt() function where
a data object called  'holds' is being called but there is nothing in
it.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) Fix RT17771 Self Check - Receipt Printing

Event handlers for click links are bound by a for loop in the Javascript
code. Within the loop, the DOM element of the link is found by ID name
and then the element is 'connected' to the relevant handler.

However, if no DOM element is found for an ID name, because for example,
the template has been customized by commenting it out, the code proceeds
to bind a handler to an undefined element, which results in the handler
being triggered by the user clicking any element.

The fix is to test for the existence of the DOM element before binding
the handler.

Signed-off-by: Steven Chan <schan@sitka.bclibraries.ca>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) SQL upgrade changes for 2.6-2.8
Jeff Davis [Wed, 13 May 2015 23:47:50 +0000 (16:47 -0700)]
(sitka) SQL upgrade changes for 2.6-2.8

(sitka) [RT32056] SQL upgrade scripts for scoping and display of e-resource records

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) SQL: cannot create actor.org_unit_full_path() unless actor.org_unit_ancestor_fenced_path() exists

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) SQL: assign org settings and perms

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [RT31757] SQL: opt-in setting for email notifications

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
(sitka) [RT32764] SQL: create and assign COPY_STATUS_LOST_AND_PAID.override perm

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka-backport) LP1353643: fix CREATE FUNCTION order for new located URI scoping...
Liam Whalen [Wed, 26 Aug 2015 22:00:59 +0000 (15:00 -0700)]
(sitka-backport) LP1353643: fix CREATE FUNCTION order for new located URI scoping functions

The function actor.org_unit_ancestor_fenced_path needed to exist
earlier in the 020 script.  Its current position caused an earlier
 CREATE OR REPLACE to fail because it referenced
actor.org_unit_ancestor_fenced_path before it existed.

4 years ago(sitka-backport) LP1353643 Located URIs search scoping and display
Jeff Davis [Wed, 10 Jun 2015 18:15:49 +0000 (11:15 -0700)]
(sitka-backport) LP1353643 Located URIs search scoping and display

The current system returns all URIs attached to a record.  This is too
much information when a record has a large number of 856 $u attached to
it.

This code adds new SQL functions to control the search scope of records
with URIs attached to them and the display of URIs once records have
been retrieved.

To do this 7 new libray settings have been created.

opac.luri_anc_vis_depth, which controls the depth of the ancestors
searched when retrieving URIs for a record.

opac.luri_des_vis_depth, which controls the depth of the descentants
searched when retrieving URIs for a record.

opac.luri_ancestor_record_visibility_depth, which controls the depth of
ancestors searched for relevant URIs when searching records containing
856 $u values.

opac.luri_descendants_record_visibility_depth, which controls the depth
of descendants searched for relevant URIs when searching records
containing 856 $u values.

opac.luri_pref_ancestor_record_visibility_depth, which controls the
depth of the preferred library's ancestors searched for relevant URIs
when searching records containing 856 $u values.

opac.luri_pref_descendants_record_visibility_depth, which controls the
depth of the preferred library's descendants searched for relevant URIs
when searching records containing 856 $u values.

These last two settings are needed to allow search for URIs and search
for physical copies to behave in the same mannner regarding the
preferred library.  When search for copies, the preferred library
returns copies that are attached to records belonging to the search
library, but it does not return records that belong only to the
preferred library.  Using these settings, a library can ensure that
records with 856 $u behave in the same manner.

Lastly opac.luri_results_uri_limit determines how many URIs to display
in the search results screen.  This stops too many links from being
displayed for a single result in the search results, which causes the
search results page to be too long.

At Sitka, we set the opac.luri_descendants_record_visibility_depth to
100 at the Consortia level.  This allows the search to return all
records with 856 $u. But, we set the opac.luri_des_vis_depth to 0, which
stops the search results from displaying any links at the Consortia
level. All other settings are 0 for the Consortia level.

At one branch we set the opac.luri_descendants_record_visibility_depth to
100.  But the opac.luri_desc_vis_depth is set to 4. At the same branch
the opac.luri_ancestor_record_visibility_depth is set to 2, which causes
records belonging to the system to be returned, but not records
belonging to the Consortia or the Federation.  As well,
opac.luri_anc_vis_depth is set to 2, to ensure URIs are
retrieved for any records that could be retrieved.

Added link to More results

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Removed extra More...

Conflicts:

Open-ILS/src/templates/opac/parts/result/table.tt2

Ensure More... Link is not displayed when # links = limit

When the number of links to display on a record is equal to the setting
opac.luri_results_uri_limit a More... link was being displayed.  This is
inaccurate because all the links are displayed on the results page.

This commit adds a check to determine if the number of uris on the record is
greater than the limit of uris being displayed.  If there are more URIs
to be displayed than the limit, then a More... link is show.  Otherwise,
no link is displayed.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Added comment referencing SQL function used to return URIs

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Conflicts:
Open-ILS/src/templates/opac/parts/record/summary.tt2
Open-ILS/src/templates/opac/parts/result/table.tt2

Conflicts:
Open-ILS/src/sql/Pg/300.schema.staged_search.sql

4 years ago(sitka) [RT31170] added content handler should always pass a value to provider
Jeff Davis [Fri, 1 May 2015 18:38:31 +0000 (11:38 -0700)]
(sitka) [RT31170] added content handler should always pass a value to provider

Some records don't have an ISBN.  For these records, the added content
handler doesn't have a value to pass to the provider.  Unfortunately,
Content Cafe doesn't return a 404 in this scenario; it returns a
plain-text error message with a 200 HTTP status code, which means our
Apache error handling in eg_vhost.conf doesn't work.

This commit ensures that the AC handler always passes a value -- the
ISBN if there is one, otherwise UPC, otherwise a dummy value.  (We
formerly implemented this approach directly in the TPAC, but that
prevented us from using locally-hosted cover images, which require the
r/rec_id URL format.)  This obviously won't work for AC providers that
don't accept UPCs as an identifier.  Providers that allow us to specify
the identifier type directly in the request, such as Syndetics (or
Content Cafe in EG2.7+), are unaffected by this change.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT31578] Changed retrieve perm for Holdings Import Profile
Liam Whalen [Mon, 4 May 2015 01:55:45 +0000 (18:55 -0700)]
(sitka) [RT31578] Changed retrieve perm for Holdings Import Profile

When retreiving the holdings import profile, the perms needed are the
same perms needed to update the profile.  We only want Sitka Staff to be
able to update, but we need library staff to be able to view, so when
importing marc records for shelf ready items, they can choose a Holdings
Import Profile to use.

This commit adds a new perm. VIEW_IMPORT_ITME_ATTR_DEF, which replaces
the current perms in the fm_IDL.xml file for retriveing the Holdings
Import Profile details.

A new sql/PG/upgrade file is added to INSERT the new permission.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
(sitka) [RT31578] Removed extraneous comma

(sitka) [RT31578] Added proper BEGIN COMMIT and closing ;

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
4 years ago(sitka) [RT20865] LP#1411819: org setting to override PATRON_EXCEEDS_FINES penalty...
Jeff Davis [Thu, 23 Apr 2015 14:58:00 +0000 (07:58 -0700)]
(sitka) [RT20865] LP#1411819: org setting to override PATRON_EXCEEDS_FINES penalty on renewals

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) [RT31287] Added edit_date modification to Vandelay import
Liam Whalen [Thu, 23 Apr 2015 06:11:24 +0000 (23:11 -0700)]
(sitka) [RT31287] Added edit_date modification to Vandelay import

When importing records via Vandelay, the current code does not modify
the edit_date or note which user imported the records.

This causes a problem with Bibliocommons.  Because the edit_date does
not change, Bibliocommons cannot detect that a new bib record exists, so
the old brief records are left in the Bibliocommons database.  This fix
updates the edit_date with the current time and notes which user
imported the record(s) via Vandelay.

4 years ago(sitka) Customize staff client splash page
Jeff Davis [Tue, 14 Apr 2015 22:59:50 +0000 (15:59 -0700)]
(sitka) Customize staff client splash page

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) RT26732: Fix for creating active event on marking circ as lost
Jeff Davis [Mon, 29 Dec 2014 22:35:28 +0000 (14:35 -0800)]
(sitka) RT26732: Fix for creating active event on marking circ as lost

When marking an item as lost from a patron's record in the staff client,
if there is no stop_fines_time value when the circ is marked as lost,
the circ object passed to open-ils.trigger.event.autocreate will have a
stop_fines_time of "now" (not a date string).  Now, suppose you have an
action trigger event definition where hook = 'lost' and delay_field =
'stop_fines_time'.  When the item is marked as lost, the date-handling
code in Trigger.pm's create_active_events_for_object function will choke
on that "now" value in the delay_field, and no event will be created.

This fix checks for a delay_field value of "now", and uses a string
representation of the current time instead for the purposes of event
creation.

Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
4 years ago(sitka) add sitka_archive.xml template rt27304
Mark Bucholtz [Wed, 24 Dec 2014 20:10:54 +0000 (12:10 -0800)]
(sitka) add sitka_archive.xml template rt27304

4 years ago(sitka) [RT19284] Liberalized the check for taxes
Liam Whalen [Wed, 29 Oct 2014 18:31:22 +0000 (11:31 -0700)]
(sitka) [RT19284] Liberalized the check for taxes

The amount regex used to check for taxes needed to be liberalized, so
that it could find Ingram's amount sent in EDI messages.  They use an
MOA+120 field, where as we were checking for MOA+124.

4 years ago(sitka) [RT26171] Examine all matching bibs for Onorder Overlay
Liam Whalen [Thu, 6 Nov 2014 00:25:44 +0000 (16:25 -0800)]
(sitka) [RT26171] Examine all matching bibs for Onorder Overlay

Currently, the code only looks at the bib with the highest match score
to try and determine if that bib should be overlaid based on copies
attached to it.

When looking to overlay a bib based on the copies attached to that bib
record, look at all matching bibs not just the bib with the highest
match score.