LP#1190508: Rename certain image files to avoid Ad Block Plus's blacklist

Ad Block Plus was blocking images with /images/adv_ in their pathname. It is
presumed that "adv" was short for "advertisement".  I have renamed the files
and the references to avoid the ABP blacklist.

Signed-off-by: Dan Pearl <dpearl@cwmars.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 grid print CSV repair; all cols repair

* repaired grid Print CSV action, which had not been updated with the
  latest print API.

* repair grid show/hide all columns actions.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Move new color note to release notes 2.7

Signed-off-by: Ben Shum <bshum@biblio.org>

lp1261791 Adjustment to background color for mobile search links

Adding an id for the div surrounding mobile search links so that the
background color matches the color of the rest of the header.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Release notes entry for new mobile-header-text-color

Adding a release notes entry for the addition of mobile-header-text-color
in colors.tt2.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP 1261791 - Mobile Search links for my account screens

Added links for Basic, Advanced and Browse Searching from all my Account pages in a less than 600px view.:

Added a new color for the mobile search links, so users can modify based on their background color.

Signed-off-by: Suzanne Paterno <paterno@noblenet.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Docs: Fixed small typo in docs/reports/reporter_daemon.txt

Signed-off-by: Josh Stompro <github@stompro.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: Updated usage of autogen.sh to not use param 'u'

Signed-off-by: Josh Stompro <github@stompro.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Docs: update to 'Address Alerts' feature content

Signed-off-by: Josh Stompro <github@stompro.org>
Signed-off-by: Yamil Suarez <yamil@yamil.com>

Release notes cleanup

Cleanup to the 2.7 release notes, to include spelling corrections, moving
some content to the official docs, and adding acknowledgements at the end.

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

Docs: Fix filenames to match references in the docs

These image files were not displaying in the docs because the filenames
and the references used different capitalization. Web URLs are
case-sensitive, so this commit changes the image filenames to be all
lowercase.

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

LP#1350042 add missing ui-bootstrap dep to Gruntfile

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 Browser client templates/scripts (phase 1)

 * Templates in Open-ILS/src/templates/staff/
 * JS files in Open-ILS/web/js/ui/default/staff/
 * Also includes a few TPAC modifications for embedded mode
 * Resurrects the pre-XUL user permission editor

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 make_release -c for building browser client

Adds support for fetching JS dependencies, minification, and
building of the experimental browser client via new -c option.
Browser client building is disabled by default.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 link opensrf websockets JS scripts

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 browser client apache config additions

* Location configuratoin for /eg/staff templates
* Support for "stop-at-index" in EGWeb, so that navigating to "/eg/foo"
  can result in /eg/index.tt2 getting served.
* Cache and compress additions for /js files.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 detect is-staff mode for browser client

The browser client does not pass an OILS-Wrapper header to the embedded
catalog (etc.) so consider the presence of a workstation an indication
that the TPAC should operate in staff mode.

NOTE: this may require some additional thought, since this means all
TPACs within an authenticated browser will run in staff mode, which may
not be desired.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 streaming holds pull list API

Adds an API name-based option to the standard holds pull list API to
respond with a stream of fleshed users, so the caller is not forced to
make additional fetch-by-ID calls to collect the holds data.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 streaming patron search API

Adds an API name-based option to the standard patron search API to
respond with a stream of fleshed users, so the caller is not forced to
make additional fetch-by-ID calls to collect the user data.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1350042 PCRUD access for various IDL classes

These changes are mostly data retrieval and help support
PCRUD/auto-grid-style retrieval of certain patron data.

Classes affected:

actor::workstation
money::billable_transaction_summary
actor::usr_note
money::credit_card_payment
money::open_billable_transaction_summary
money::cash_payment
money::forgive_payment
action::hold_notification
money::credit_payment
config::non_cataloged_type
money::billable_transaction
money::work_payment
money::goods_payment
money::check_payment
money::payment
money::billing

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Docs: added Address Alert documentation for local admin menu feature

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Fix broken prereq installer for debian wheezy

Problem found by live-tester scripts. Yay for live tests!

Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP#1282783: Use patron hold notification defaults for KPAC

Since the KPAC interface doesn't display a patron's email/phone notification
preferences, we had initially assumed it was pulling in the patron's default
notification info, but we've found that it does not.

Emails are not sent to patrons who have their notification preferences set to
email, and hold slips print out without phone numbers or email addresses. This
causes extra work for Circ staff in processing incoming holds as they need to
look up each account number to get the patron's contact information.

This patch pulls in the patron's email & phone notification preferences and
related values in hidden fields and allows the patron to choose their pickup
library at time of hold placement (defaults to their preferred pickup location).

Signed-off-by: Terran McCanna <tmccanna@georgialibraries.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp949101 Hold Columns for Item Status

Fixes display of hold columns in Item Status Holds/Transit tab.

Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1362210: Install PostgreSQL packages where we can

Rather than relying on purely manual installation, carve out a common
set of packages that are needed by both the Evergreen server and the
standalone PostgreSQL server and add them as dependencies to both
scenarios. In the worst case, the package manager will see a package
that it has already installed and skip it (if the database server is on
the same OS instance as the Evergreen server).

It sure beats upgrading your database server and wondering why things
like the addition of call numbers fails with database query errors when
it turns out that you're missing the Library::CallNumber::LC Perl module
:)

It also makes the README shorter and requires fewer steps for users to
follow to install successfully. Win win win?

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1306675 TPAC maketext default handler

Fall back to an unitialized maketext handler, which defaults to the
template language, when OpenILS::WWW::EGWeb::I18N->get_handle fails to
return a response.

Note that why get_handle() returns undef is not yet known.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1233757 Correct Spelling Error in Checkout Override Exception

Correcting a few misspellings of "overridden" in the staff client
override exception.

Signed-off-by: Elliot Voris <evoris@slcconline.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1010027: tweak patron columns in patron search interface

specifically, remove the Mailing Address and Billing Address columns that come
off the actor table, since they are redundant with the Mailing Addr: Address ID
and Billing Addr: Address ID columns that come off the address table.

Also, alphabetize the columns coming directly off of the user table. The columns
coming off the library card and address tables remain unsorted; I believe their
order is as listed in the fm_IDL.xml file for their corresponding fieldmapper
definitions.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1010027: more internal options for xul lists

specifically, give fm_columns a sort_headers flag for alphabetizing the columns
returned by label; and delete_me flag for omitting a column definition entirely

Not all xul lists are using fm_columns yet

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1189556: stamping upgrade script for URL_VERIFY description typo fix

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1189556: Fix typo in URL_VERIFY permission description

The description for the URL_VERIFY permission contained a typo,
fixed here.

Added an upgrade sql to update the text.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1322341: Count part holds on records

Also fix a small mistake in metarecord matching

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1277551: Limit simplified pull list to available copies

Copies that are targeted by a hold but are not in available or reshelving
statuses should not show up on a pull list.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Michele Morgan <mmorgan@noblenet.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1314370: Enable dojo all the time in the catalog

Because we rely on the dojo framework for the shelving location filter in the
advanced search, we should choose to use dojo everywhere and not just when
autosuggest, google books, or novelist are enabled.

This is a quick change to toggle the flag for want_dojo to be set all the time.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>

LP1277556 Fast Item Add no longer opens record after copy is created

Altered save_attempt to handle undefined and true cases for replace_on_complete
When fastItemAdd is not used, replace_on_complete is undefined
When it is used, and successful, replace_on_complete will be set to true
then we need it to execute result.on_complete()

Signed-off-by: Blake Henderson <blake@mobiusconsortium.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP1084269, LP1050043 Expert Search Fixes

Sorting Expert Search corrected

LP1050043 was issued because attempting to sort the MARC expert search
results, in the staff client, caused the staff client to return to the
basic search page rather than sort the results.

To fix the sorting I had to modify two perl files.

metabib.pm had to be changed in a number of ways.

1. The way the sort works currently assumes the user is doing a full
text search.  So, it passes in the values of titlesort and authorsort as
the sort variable in the URL.  However, in the expert search, it is
looking for the values 'title' and 'author'.  This resulted in the title
sort and author sort never being executed and returning the relevance
sort instead. I modified the search to use a regex instead of eq.

2. If a record has no value associated with it e.g. no 260 c for
pubdate, no 245 $a for title, no 1xx $a for author, then the SQL is
supposed to replace those values with either 'AAAA' or 'zzzz' depending
on the order of the sort.  This should cause records without the
relevant sort values to be pushed to the bottom of the sort results.

But, the PgSQL function COALESCE, which is supposed to do the
replacing, does not do anything if no rows are returned for a record's
relevant value.  To work around this, I surrounded the relevant values
with the MAX() function call, which returns NULL if no row is returned.

Because author sort works on any 1xx value, it sorts those values in
numerical order when there is more than one 1xx in a record.  This
requires the use of an ORDER BY statement, which seems to nullify
the effect of MAX().  So, I had to move the MAX() into a second SELECT
statement in the case of author sort.

3. I added the variables number_default_sort and string_default_sort
to the metabib code because the current code hard codes values of
'9999' and 'zzzzzzzz' into the sort.

Search.pm had to be changed to pass the sort values from the TPAC
into the relevant sort subroutines.  This was done with the addition of
an $arghash variable that checks the cgi parameters for the sort
variable.

As well, Expert search puts the tag and _special params in the
search form.  This causes all other searches to perform Expert searches
afterwords.  To fix this, Search.pm is modified to look for an empty
query value before it conducts an Expert search.  If it is empty and
the Expert search variables are present, then the Expert search is
performed.  Otherwise, a normal search is done.

Finally, to fix the problem reported by LP1050043, I had to modify
searchbar.tt2.  Currently, searchbar.tt2 does not have the values from
the Expert search entered into the <form> where the sort functionality
is located.  This means, when a sort option is chosen, the TPAC no
longer knows which values were used for the Expert search.

This is not obvious at first because after choosing to sort, the
user is returned to a page where the URL has the Expert search values
in it.  This happens because the sort is submitting a form without a
query value, so the TPAC redirects the user to the referring page,
which is the initial Expert search.

To fix this, I added the Expert search values as hidden variables
into the <form> where the sort functionality is located.  I also
removed the use of the "_adv" hidden variable from the <form> when
not in the advanced search.

[LP1094269] Retrive past 100 records in Expert Search

Currently, when paging through the results of the Expert search, the
system will not return more than 100 records.  Going past page 10
causes a no results screen.

This is due to the offset and limit values not being passed into the
argument hash used to create the SQL for the search.  Once the SQL
returns, it returns record ids within the range of the offset and
the limit. So, if the offset is 100 and the limit is 10, the record
ids for search results 100 to 109 are returned.

But, because offset and limit are not being passed into the arghash,
they default to 0 and 100 respectively.  This means the first 100
records are always returned regardless of where in the search pages
you are viewing.  These 100 results were then pruned to the relevant
10 results in Biblio.pm.  However, if the offset starts beyond 100
then no results are returned because the results passed 100 are not
returned by metabib.pm.

This commit passes the offset and limit as arguments in the search
hash.  It also removes offset and limit as parameters of the
search.biblio.marc call. Because offset and limit are needed in the
arghash to limit the number of records returned, it is better to
pass them only via the arghash to ensure that no more than the
necessary number of records are returned.

Offset and limit could be kept as parameters, but it would require
modifying metabib as well to look for those parameters, so I think
forcing the Expert search to pass them as argument parameters is the
better option.

As well, this commit removes the pruning in Biblio.pm that was based
on the offset and limit because that is handled in metabib.pm now
that offset and limit are being supplied in the argument hash.

I have checked the source code and the marc_search function in
Biblio.pm is only called via the Expert search in
WWW/EGCatLoader/Search.pm, so modifying Biblio.pm in this manner
will only affect the Expert search and nothing else.

Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Signed-off-by: Sarah Childs <sarahc@zionsvillelibrary.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1182605 Add sort value for holds interfaces

Holds interfaces were still sorting incorrectly, so let's add a sort value
there.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

lp1182605 Abstracting out common functionality from code that sorts call numbers by label_sortkey

Code had been added to insure that LC call numbers are sorting as they should by asset.call_number.label_sortkey.
Code was duplicated in the 'call_number column' in the 'columns' group and in the 'hold_columns'group.

I abstracted the logic out of 'sort_value' and 'render' in the 'call_number' column in the 'columns' group and from
'render' in the 'call_number' column in the 'hold_columns' group to a utility function  called
'sort_call_numbers_by_label_sortkey'.

Signed-off-by: Victoria Lewis <vcamklewis@gmail.com>
modified:   xul/staff_client/server/circ/util.js
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1182605 : LC call numbers sort improperly in tabular displays

The issue was that the column was sorting on the label (call number)
and not label_sortkey.  An attribute 'sort_value' was added that used
label_sortkey.

Signed-off-by: Kyle Tomita <tomitakyle@gmail.com>
modified:   Open-ILS/xul/staff_client/server/circ/util.js
(cherry picked from commit 696ce9e6c36bffd1d8c884baab2f5d36687e741b)

Signed-off-by: Victoria Lewis <vcamklewis@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Fix currently harmless but still confusing sigil error

get_org_descendants() returns an array ref, not a list, so it doesn't
make sense to assign it to an '@orgs' variable.  Lucky for us, our
where-parser treats this single element list containing the ref in the
same manner as the proper ref itself, but at best, it is still very
confusing to see.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#1078787 Fix serial scoping in TPAC

If you have generated holdings, but are not using the "compressed"
display, your holdings show up regardless of scope.  This change
limits the display to only sdists with a holding_lib in your current
search scope.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

LP#800478: Stamping upgrade script for fund transfer fix

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

LP#800478 repair logic error in acq fund transfer

The source data for the amount applied to each allocation and transfer
(in certain circumstances) was incorrect.

The transfer amount should not be the total amount allocated from the
source to the fund, but instead the amount of the funding source credit
in cases where we have to pull money from multiple funding source
credits.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Liam Whalen <liam.whalen@bc.libraries.coop>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1359934: prevent error opening patron search form

As a result of the patch for OpenSRF bug 1316245, JSON2js
in the OpenSRF library is now stricter -- in particular, passing
an undefined value to it results in an exception being thrown.

This patch runs with the increased strictness and fixes an
issue where the patron search form was passing an undefined value
to JSON2js.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Docs: Removed use of deprecated file suffix for eg_db_config in install docs

Signed-off-by: Yamil Suarez <yamil@yamil.com>

Compile release notes for Evergreen 2.7.0

Signed-off-by: Ben Shum <bshum@biblio.org>

Pre-2.7 Release notes cleanup

1) Fixing up formatting issues with the asciidoc
2) Remove extra words "New feature" from certain notes

Signed-off-by: Ben Shum <bshum@biblio.org>

Fixes for create_release_notes.sh

1) Teach it to ignore the sample RELEASE_NOTES_NEXT.txt file
2) Add a proper section for "New Features" heading, instead of
   defaulting everything under the "Upgrade notes" heading.

Note: Will still require manually moving any upgrade notes to the
      right place post-compile.

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 text input repairs

* Correctly propagate values for text widget "in list" values when editing
  reports
* After entering a value for "in list" text widgets, clear the text
  input so the user can enter subsequent values w/o manually clearing.

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

LP#1329503 report edit scheduling repairs

* ensure default values for scheduling and recurrence are applied with each
  edited report, including today's date in the scheduler.
* If the most recent report run run_time is in the past, deafult to "run
  now"
* fix bug with propagating months recurrence
* If recurrence interval is in days and divisible by 7, present as weeks
* make date regexes easier on the eyes
* fix bug with rendering sets of org units
* fixed bug with modifying report to be "run now" from scheduled.

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

LP1355319: Add missing Parse::RecDescent perl dependency.

Add the appropriate deb or rpm entry to the various Makefiles.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

Release note repairs and additions

A handful of repairs and additions to the 2.7 release notes, including:
  * moved copy_edit_links to cataloging
  * making no-fixed-field-defaults-for-ingest an upgrade note
  * adding release note entries for hold targeting of precats and improved
acq fund rendering.

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

Secondary permission groups release note

Release notes entry for secondary permission groups as well as some fixes
to the docs to get the images to display.

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

LP#1292705 Fix broken is*() methods in fm_IDL2js.xsl

A thinko meant that we were no longer incrementing the field variable,
leading to breakage in interfaces that used isnew() and the like.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Secondary Permissions Documentation and Screenshots.

Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Add multiple permission groups editor to user registration form

This makes use of permission.usr_grp_perm_map which grants secondary
permission groups.  It does not affect the 'profile' ie: circulation policy.

Signed-off-by: James Fournie <jfournie@sitka.bclibraries.ca>
Signed-off-by: Jeff Davis <jdavis@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

Update ContentCafe Added Content Module

Now with keyhash support, but losing "No Image" image support.

Also added a "fetch all" function for possible "cache everything" calls.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 avoid run now + later combined schedule

When editing a report, ensure only one of "now" vs. "schedule for later"
options is selected by default.  Similarly, if the schedule option is
activated by default, ensure the date fields are enabled.

If the run time of the most recent report is in the past, default to the
current date for the schedule instead of the no longer valid date.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 propagate 'in list' values for editing

For example, ensure a previously selected list of copy locations are
propagated into the selected values list during report editing.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 report editing additional repairs

* correctly propagate values for the "between" widget
* propagate pivot data and label when present
* propagate report run time
* improvements to report email propagation

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 report editing misc. repairs

* Fixed some faultly widget rendering on report edit

* Avoid showing "save as new" for new reports.

* Repair fault report edit Exit/Cancel buttons

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 View/Edit Report Release Notes

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 View Edit Reports

Alongside each report in the Reports folder view there are two new
links, one for viewing (read-only) and one for editing.  After changing
a report, the user has the option to save the modified report or create
a new report with the new values, in effect cloning the original report.

When saving a changed report, if a pending report run exists, the user
will be warned of this and asked if they would prefer to modify the
scheduled report or to instead save the changed values as a new report,
leaving the original report intact.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1329503 Reporter report.fleshed.retrieve API

API for fetching fleshed reports, including the most recent
reporter.schedule.  Useful for modifying existing reports.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1332138 hide doc URL in report editor when no URL present

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1332138 Report Doc/Hints Release Notes

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1332138 Report template / field docs repairs

* Various sanity checks and misc. repairs
* Add missing strings
* oils_rpt_editor.xhtml XML repair; XMLENT is strict
* links to external resources open XUL tabs (to avoid HTTP dialogs)
* avoid displaying bare HTML in labels
* show hints for display fields

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1332138 Report template / field documentation

* Report templates support a new External URL field, which may
  contain a link to template-specific, local documentation.

 ** When set, a link to the external documentation will be
    displayed in a new column in the template list and within
    the report editor.

 ** URL's are set in the 'Documentation URL' entry.

* Template display fields and filters support a new Field Hint
  value.  When set, hints are displayed in the report editor.

 ** Values are set via the 'Change Field Hint' option along
    the bottom of the template editor.

------ from commits ----

* Initial template-level support for inline template and field documentation
* Use external documenation url in clark output
* Show the docs link and strings in the report editor
* Set the href of a link, instead of the text content of a span
* Attempt to pop up a new window (may require more pushups in the XUL staff client)
* Add calculated column support, use it for external docs url
* Realign field names

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1346421 TPAC copy view/edit links release notes

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1346421 TPAC copy view / edit links

Adds "view" and "edit" links next to each copy in the TPAC copy grid
when in embedded mode.  The "edit" link only appears when the user has
permission to edit the specified copy.

Via the XUL volume_item_creator, the ui.unified_volume_copy_editor org
unit setting is inspected to determine which UI to load.  Also, the data
needed to load the subsequent UI is fetched within the
volume_item_creator, so the catalog doesn't have to.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1346421 TPAC permission check function.

Support permission checks against the authenticated TPAC user.

[% IF ctx.has_perm('UPDATE_COPY', copy.circ_lib) %] ... [% END %]

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1304462: Release notes about expanded RDA tag 264 support

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP#1304462: Use other 264 types in search results if publisher is not available

In the event that the publisher information is not part of the record, use
other types of 264 tags for producer, distributor, or manufacturer.

Also employ the copyright if there is no available publication date.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP#1304462: Stop using combined pubinfo for search results

Remove one of the two places we define the pubinfo variable in misc_util.tt2
and then change the way search results pull in publisher data.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

LP#1304462: Add schema.org copyrightYear, manufacturer for RDA 264 fields

It looks like the best we can do is to define a "manufacturer" property (and
corresponding Organization entity) when 264 ind2="3"; and we can provide a
copyrightYear when we have 264 ind2="4".

Even then, MARC / RDA cataloging practices end up giving us copyright notices
like '@2012' instead of actual dates, so we have to regex the year out of the
notice to satisfy the Integer range for copyrightYear. Theoretically we'll be
in trouble if the copyright notice includes some other 4-digit number (for
example, a multipart work catalogued in a single record that has a date range),
but what are you gonna do? It's MARC / RDA.

Same for manufacturer name, which seems likely (if the LoC examples that Yamil
provided are any guide) to include statements like "Manufactured by Foobar"
instead of cleanly structured data. The RDA fields were added to MARC21 in
2011, you would have thought that the benefits of providing clean data would
have been obvious by then. Grumble grumble grumble. Yes, this is meant to be
a transcription field, and theoretically one could use a 7xx to record the
publisher/distributor/manufacturer/producer in a more cleanly stated factual
way that would lend itself to actual linked data (hey, maybe even providing a
link to an external resource that could act as an identifier), but NOBODY DOES
THAT.

So we do the best we can and soldier on.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1304462: Add additional RDA information to TTOPAC record summary.

This pulls information from the 264 tag where the indicator 2 has
the following values and displays the information below the Publisher
info:

0 - Producer (a,b,c)
2 - Distributor (a, b, c)
3 - Manufacturer (a, b, c)
4 - Copyright (c)

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP# 1308768 Make RDA bibs loadable for testing purposes

Squash the RDA records into XML loading format via the magic
of 'yaz-marcdump -i marc -o marcxml' and some judicious regular
expressions.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP# 1308768 Second set of test RDA records from LC - 6 total

Here is another  batch of test records from LC from this site:

http://catalog2.loc.gov/vwebv/searchKeyword
http://catalog.loc.gov/vwebv/searchKeyword

Here are links to the MARCXML versions of the files. Search for files with

XML records with 264 and ind2 set to _1 and _4
http://lccn.loc.gov/2014340080/marcxml - DVD - Jorge Mautner
http://lccn.loc.gov/2013335283/marcxml - CD - Claudia
http://lccn.loc.gov/2013493195/marcxml - SCORE - Wiegenlied

XML with 264 and ind2 set to _1 and _2
http://lccn.loc.gov/2013565186/marcxml - SCORE - Arias for bass
http://lccn.loc.gov/2013565190/marcxml - SCORE-CD - Arias for soprano

XML with 264 and ind2 set to _1 _2 and _4
http://lccn.loc.gov/2013568403/marcxml - SCORE - Intermediate_studies

Ind2 values key:
0 - Production
1 - Publication
2 - Distribution
3 - Manufacture
4 - Copyright notice date

I added both MARCXML and UTF8 MARC files for each record.

RDA_record_DVD_Jorge_Mautner
RDA_record_CD_Claudia
RDA_record_SCORE_Wiegenlied
RDA_record_SCORE_Arias_for_bass
RDA_record_SCORE-CD_Arias_for_soprano
RDA_record_SCORE_Intermediate_studies

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP# 1308768 Add 4 sample RDA records from LoC

At the suggestion of Dan Scott, I went to this LoC site:

http://catalog.loc.gov/cgi-bin/Pwebrecon.cgi?DB=local&PAGE=First

I used their "expert search"

with a query of "040e rda"

then used various "search limits" of "type"

Here are the 4 record filenames, plus the available LoC permalinks to them:

RDA_record_MAP_2012_Enchanted_Circle.mrc
http://lccn.loc.gov/2013585177

RDA_record_DVD-BLUE-RAY-MOVIE_Pineapple_express.mrc
http://lccn.loc.gov/2010600022

RDA_record_PHOTO_Srid_pa_ho.mrc
http://lccn.loc.gov/2011432135

RDA_record_SERIAL_IEEE_International_conference.mrc

I originally found more than 4 RDA records, but it was pointed out that they were
using older/outdated versions of RDA. For example, the records used the 260
tag instead of using the 264 tag.

Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1292705 trim unneeded fields from IDL2js

Avoid sending fields in the web-published IDL (IDL2js) that have no
value, as the keys in the JSON take up a lot of space.  Also avoid
repeating fields (selector) and default values (e.g. label defaults
to name) where the client can reasonably apply the defaults locally.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1347793: TPAC added content by ID release notes

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1347793: Teach TPac to load AC by Bib ID

So that we may get more than just jackets

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1246745: Remove OK message from SIP2 messages

Because having clients constantly say there is a message of "OK" gets annoying
very quickly, especially if it is displayed in a popup message instead of via
a "messages" button.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1351317 ACQ fund selector filters cont.

To improve load times, limit the funds we attempt to retrieve for
various fund selectors to those the user has permission to use.

Interfaces affected include the following:

* Purchase order copy grid
* Purchase order "New Charge" interface
* Invoice "Add charge..." interface
* Distribution formula configuration interface.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Docs: various AsciiDoc formatting fixes for docs

Fixed the 'bad first character' error, the bad apostrophe error. Removed various UTF-8 chars.
Fixed some numbered list formatting, to take out redundant manual numerals…

for example:

. 1 item
. 2 item
. 3 item

became:

. item
. item
. item

also added "=" to wrap around breaks in numbered lists

for example:

. Right-click on the value control for the fixed field that needs to be edited.

 image::media/ffrc1.jpg[]

 . Select the appropriate value for the fixed field from the menu options.

 became:

  . Right-click on the value control for the fixed field that needs to be edited.

  +
  image::media/ffrc1.jpg[]
  +
  . Select the appropriate value for the fixed field from the menu options.

Signed-off-by: Yamil Suarez <yamil@yamil.com>

LP#1351355: Add release note on change to XLSX format for Excel report output

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1351355: Use libexcel-writer-xlsx-perl package for Ubuntu Trusty

Since Ubuntu 14.04 Trusty has a more recent packaged version of
libexcel-writer-xlsx-perl, use that instead of installing from CPAN
for Excel::Writer::XLSX.

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1351355: Switch to XLSX format for Excel report output

We've always used Spreadsheet::WriteExcel::Big to generate Excel output
from reports. That module has a limitation of 64k rows, imposed by the
binary Excel format. Recently, the Excel::Writer::XLSX module has become
feature-complete enough that we can use that instead, and the XML format
files that it writes are now supported by the majority of installed
versions of Excel (and OpenOffice) -- the format arrived with Excel 2007,
released in 2006 (8 years ago). So, let's switch to that!

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 - stamping upgrade scripts for acq cancel display

Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 remove redundant lineitem cancel display

Remove the cancel reason label displayed in the lineitem list pages
since the cancel reason is now displayed as the lineitem status for
cancelled lineitems.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 support canceling delayed copies

In the PO copy grid, allow cancelled-as-delayed copies to be canceled
again, so that true cancelation can be achieved as needed.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 sort ACQ cancel reason selectors

Sort lineitem and copy cancel reasons selectors by label.  In addition
to being generally useful, this helps to quick differentiate between
delayed and canceled items in the list.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Release notes

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Show delay/cancel reason print PO / worksheets

Display the cancel reason label in the print PO and lineitem worksheets.

Note: this only applies to templates which have not been locally
modified.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Support canceling already-canceled lineitems

If a lineitem is canceled and the cancel_reason has keep_debits == true
(i.e. the lineitem is "delayed"), allow the lineitem to be canceled
again, up until the point the lineitem cancel_reason has keep_debits ==
false, at which point, no more cancelations are allowed.

Similarly, when a copy is canceled with a keep_debits==true reason,
allow the copy to be canceled when its parent lineitem is canceled,
following the rules above.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Show cancel reason labels for lineitems/copies

In the standard lineitem list page, display lineitem states as
translatable strings instead of displaying the raw lineitem state code.
Additionally, when the state is 'cancelled', display the cancel_reason
label instead of "cancelled" to help differentiate between canceled and
delayed items.

Similarly, display the cancel reason label for canceled copies in the
copy edit grid.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 improve stock acq cancel reason labels

Add "Delayed:" and "Canceled:" prefixes to stock labels based on their
keep_debits flag to better differentiate between delayed and canceled
items in the UI.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Show delay count in ACQ lineitem summary

This affects the canned lineitem summary displayed in the lineitem
copies page and the invoice UI.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1270289 Split canceled into delayed vs canceled

The acq.lineitem_summary DB view (and related
acq::lineitem_summary_invoiceable IDL view) now differentitate between
canceled items and delayed items by inspecting the keep_debits flag on
the cancel reason.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

OpenSRF Services Update

Updating list of Evergreen-specific OpenSRF services as of 2.6. Many
thanks to Galen Charlton for providing me with an updated list.

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