web staff: after-action report, cont.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: after-action report, cont.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: initial after-action report

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: FF autofil login form support

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : lower breakpoint size of CSS grids

Replace use of col-lg-* with col-md-*.  This prevents the UI from
switching into "mobile" layout too aggressively.  We may even want
to consider col-sm-* instead, since col-xs is still available for the
smallest screens (phones, etc.).

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : remove debug line

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff; more css repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: log minification

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: separate circ/patron.css file

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : cleanup

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : show no-items message on empty holds/items-out tables

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : record bucket; bibs link to catalog

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into web-staff-proto

web staff: noncat checkouts

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Documentation for 2.5

asciidoc fix

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

Revive and update Action Triggers docs for 2.5

This commit revives the missing Action Triggers docs section, last seen
in the 2.0 version of the docs. Also, several small corrections and
wording edits were made, as well as a few actual feature changes since
version 2.0.  The most explicit of these feature changes is the addition
of a basic description for a new Event Definition field in 2.5: "Event
Repeatability Delay".

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

web staff: checkout form; make style consistent

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : load js within body, not after; bump verions.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: log - unit tests

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: unit test structure; sample tests

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Fix LP1180140, View Holds not working for a serial with subscription and no issuances

A request is made to
open-ils.circ.holds.retrieve_all_from_title,
which is the function Circ/Holds.pm/all_rec_holds().

In the function, when checking for issuance holds, there is a missing @
to dereference the list of issuance IDs, so that the ahr table is search
with badly formed input, which results in returning an undefined list
instead of an empty list.  The error will occur for any record with a
serial.subscription entry but with no related entries in
serial.issuance.

We also recode the initialization of the response object in order to
clarify its expected structure, a hash of array references.

Signed-off-by: Steven Chan <schan@sitka.bclibraries.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

web staff: recover patron summary groups; more pronounced alt rows

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : egweb stop-at-index supports /-ended paths

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web log : more apache config improvements

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : dynamic apache index delivery

Generic apache config wasn't working, since different apps create
arbitrarily nested URL parameters, which means each app would need its
own Apache rewrite configuration to sort them all out.

This creates a new OILSWebStopAtIndex EGWeb.pm option which tells the
code to check for the presence of an index file at the current directory
as it's looking up the directory tree for the requested template.

Example w/ stop-at-index enabled.

The code will look for these templates in this order:

URL -> /eg/staff/circ/patron/142/holds

1. staff/circ/patron/142/holds.tt2
2. staff/circ/patron/142/index.tt2
3. staff/circ/patron/142.tt2
4. staff/circ/patron/index.tt2 -- file exists, all done.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web log : patron search, use bootstrap grid/form for layout

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web log : patron search css repairs

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : log update for apache config

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : repair apache configuration

Location per app is no longer necessary.  LocationMatch now supports
arbitrary nesting.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: more patron saerch

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : remaining patron search fields

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Documentation for Sorting Billing Columns

Docs for sorting money columns in billing interface.  In ersions 2.3-2.5.

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

New documentation for 2.5

Documentation for:
-Z39.50 Search Enhancements
-Catalog Browse
-Alternative Hold Pick-up Location
-Marking Items Long Overdue
-Lost and Long Overdue Bill Colors
-Vandelay Item Load

Signed-off-by: Angela Kilsdonk <akilsdonk@esilibrary.com>

web staff : log templates / dynamic strings

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : log templates / dynamic strings

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : dynamic alert/confirm dialogs

Create new services egAlertDialog and egConfirmDialog, which each load
a TT-based template on demand.  This allows us to re-use the templates
instead of having to include one template per alert type, which will
grow indefinitely.

Experimenting with embedding dynamic strings within the TT template, so
that they can be translated alongside other template strings.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Always show Email Address label to staff

When placing holds staff commonly change out patrons, and pointing staff
towards the "My Account" area makes no sense to begin with.

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

web staff : log templates (in progress)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Documentation typo in Authorities chapter subheading
Addresses this bugs reported in launchpad:
https://bugs.launchpad.net/evergreen/+bug/1248726

I also browsed through the rest of the docs/admin
directory, and made some (very non-consequential)
changes.

Signed-off-by: Elliot Voris <evoris@slcconline.edu>
Signed-off-by: Robert Soulliere <robert.soulliere@mohawkcollege.ca>

web staff : log templates (in progress)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: hide locale selector when only one locale is available

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff : locale selector

leverage the existing set_eg_locale URL parameter for applying the
selected locale.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Pushing upgrade scripts forward

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

Forward port 2.3.11->2.3.12 DB upgrade

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Pull in 2.5 version upgrade files

For 2.5.0 and 2.5.1

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

web staff / log tidying

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff / log : promises

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: cleaning up remainder of inline style

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: extracting some inline css

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: drive css from templates (like tpac)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Fix a broken link in the holds docs

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into web-staff-proto

web staff: log on workstations/printing

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Docs: integrate holds docs from EG 2.1

The small amount of Holds docs present in master seemed to fit well at
the end of the older Holds documentation, so this commit appends them
to the new holds.txt file, deletes the previous holds_management.txt file,
and makes the appropriate change to root.txt.

This commit also includes several small formatting improvements, wording
changes, and bug fixes.

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

web staff: applying some render-time data sanity checks

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: services for route persistence

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: route to hold shelf dialog

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff / minor code cleanup

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff / list supports auto class columns

list.addColumnsForClass(classname, basepath)

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: checkin copy alert msg dialog

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: checkin transit dialog

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: checkin continued

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: initial checkin; egOrg.settings()

Signed-off-by: Bill Erickson <berick@esilibrary.com>

web staff: style overdue circs in items-out

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into web-staff-proto

Fix boolean lists; Better atom regex; Caching

First, we didn't need to make $last_type local, and it broke explicit
grouping anyway.  That's removed, and we now reset that (and a few more
like it) at calls to the top level parse() method.  This introduces a
situation where a long list of booleans could cause query plan problems,
so we limit the plan depth to 40 (20 ||'d conditions).

Second, we are smarter about finding the boundary of atoms.  Previous
to this commit, and curly brace could send the parser into a tailspin
from which it would not recover.  Now we use alternation instead of
a character class, which is much safer with the default multi-character
float syntax specifier.

Third, as a catch-all, if we can't parse the remained of a query we
now simply say so (when in debug mode) and go away, instead of risking
an infinite loop.  We do this via a final, unqualified "else" clause
in decompose().

Finally, instead of building 10+ regexp objects on each query parse,
cache them per QP subclass and reuse them.

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

LP#1238015 hide patron credit summary when disabled

When the circ.disable_patron_credit / "Disable Patron Credit" org unit
setting is disabled, also hide the patron credit summary information in
the bills interface.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

web staff / log -> getting started w/ testing

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Re-apply the changes provided by  0802 for backport

Preserve record order of subfields for authority heading extraction

When extracting headings from authority records we currently read the
subfields of a tag in configuration order.  We should, instead, read
them in record order, to preserve the desired sorting properties that
the cataloger has encoded in the record.

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

web staff / log : install and hacking

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Angular web staff - initial import

Batch import of staff client prototype work in progress.

Signed-off-by: Bill Erickson <berick@esilibrary.com>

Fix copy_table HTML markup errors

TD elements take a "headers" attribute, not a "header" attribute. Also,
we have a few typos in identifying the header attributes.

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

LP#1254146 gracefully handle custom trees sans entries

Avoid references to custom tree entries when no tree entries for a
custom tree exist.  Prevents:

egweb: template error: undef error - Can't call method "org_unit" on an
undefined value at
/usr/local/share/perl/5.10.1/OpenILS/WWW/EGCatLoader/Util.pm line 185.\n

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

Improve facet styling

1) Facets currently display very poorly when the number reaches 3-4
digits.  Facet data is more or less a two column table, so let's style
it that way to make it more robust.

2) Rearrange some styles to reduce redundancy, and allow overflow
where possible.

3) Change the selected highlight to look a little better.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

Pulling in new version upgrade

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

LP#1251774 exit and alert on multiple payments per xact

If more than one payment for a transaction is received, exit the API
call early and return a MULTIPLE_PAYMENTS_FOR_XACT event.

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

Enforce one-payment-per-xact-per-call

There is no legitimate reason for a transaction to receive more than
one payment per call to open-ils.circ open-ils.circ.money.payment,
but we have seen the staff client generate such a data structure. This
leads to seemingly duplicate payments, and is bad all around.  So,
we will enforce the restriction by taking only the first payment per
xact in the list of payments.

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

Replace erroneous calls to $e->retrieve_authority_record($rec_id).

On lines 153 and 175 of OpenILS::Application::Cat::Authority calls
like the above should be $e->retrieve_authority_record_entry($rec_id).
This branch replaces the two misspelled calls with the proper ones.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP1093856 fix Fast Item Add with Z39.50 import

The first time I tried this fix I ran afoul of
https://bugs.launchpad.net/evergreen/+bug/787561

But a second attempt worked.  So I didn't kill
two birds with one stone, but this should take
care of 1093856.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

LP#1037171 Removed Expert Search paramters from subject links

After performing an Expert Search, if a user visits a record and then
clicks on one of the author, subject, or series search links within that
record, then they are given the results of the Expert Search again
instead of the relevant subject or author search.

This is happening because the Expert Search parameters are retained in
the URLs via the TT2 MACRO mkurl.  In particular, the tag parameter is
used to identify an Expert Search, so if it is present in a URL, than an
Expert Search will be attempted regardless of which other paramaters are
in the URL.

To fix this, I created two varialbes within header.tt2 above the
definition of mkurl.  These variables group common URL paramters for
Expert Searches and General Search parameters together.  These variables
can be used wherever mkurl is used to strip out unneeded search
paramters.

There is a third variable that is already present within many of these
pages named stop_parms.  While I initially tried to modify this
variable, I found that there are some instances where the general search
parameters should be retained while the Expert Search parameters must go.

This commit uses these three variables togehter as input into the third
arugment of the mkurl MACRO.  The third arugment specifies which
paramtes to remove from the previous URL before making a new URL based
on the pervious URL.

I also added the query parameter to the Place Hold's hold_source_page
URL variable.  I did this, so that the query terms would be preserved
once the user is done placing the hold or decides to cancle the placing
of the hold.

As well, I used the new variables to remove Expert Search paramters
from the Advanced Search links on the results and record pages.

Conflicts:

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

Signed-off-by: Liam Whalen <whalen.ld@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Docs: Remove ref to missing included file

This file was moved into the 2.5 release notes. If others believe it
belongs in the main documentation, it can easily be added in again. For
now, we will remove the reference because it only generates an asciidoc
warning.

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

Docs: Fix leveloffset bug, raise REL NOTES level

We should restrict our use of the asciidoc leveloffset command to the
root.txt file, since it is a global setting and cannot be set relative
to an existing offset.

This commit also raises the Release Notes section to the highest level
in order to prevent the lowest level subsections within it from being
lost. This is in response to asciidoc errors:

    WARNING: RELEASE_NOTES_2_5.txt: line 950: missing section: [sect5]

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

Document how to load concerto sample data

We might as well make it as easy as possible for people to use the
sample dataset that we created!

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

Add basic docs for testing with pgTAP

This can be expanded to describe how to create pgTAP tests later. For
now, let's help people actually set up their environment and test.

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

Fix untranslatable strings in several dijits:

AutoGrid, EditPane, FlattenerGrid, GridColumnPicker and HoldingCode.

[LFW: One string amended in conflict resolution; commit message edited
to wrap; one line of code moved (this.nls assignment in
GridColumnPicker.js).]

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP1251424 - Fix for submit button when placing holds in staff client

I've moved the initializing of the cur_hold_barcode variable
to within the function rather than floating outside of it.
This seemed to fix an intermittent issue where the submit
button would not become activatable when first going to the
place hold screen via the staff client. To create the hold,
ths user had to cycle through the "Place hold by patron barcode"
option and then back to "Place hold for me" option.

It looked like behind the scenes for certain org units that
variable would not be set and the button will not open until
it at least has something. This causes it to be set on the initial
loading of the screen.

Signed-off-by: Steven Callender <stevecallender@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

lp1012237 HOLD_ITEM_CHECKED_OUT display

Added a definition to the HOLD_ITEM_CHECKED_OUT event in ils_events.xml.
Trying to be generic between the opac and the staff client the message
reads, "The item you have attempted to place on hold is already checked
out to the requestor."

[LFW: I made a tiny spelling change to be consistent with other
appearances in Evergreen of the term "requestor".]

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Update version of release notes in 2.5 docs

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

Acq: Improvements to account-matching incoming EDI messages

The way the EDI fetcher works gives us a problem. That process iterates
over EDI accounts for which it has FTP host and credential information,
downloads documents from each of those sites, and files the messages
within those documents under the EDI account from which the login
credentials came. The problem is that in practice the exact same host
and login information is used by multiple accounts under the same
vendor, and files relating to these sub-accounts are commingled, so that
you can't make the decision about which messages should be filed under
which accounts based on the name of the document or its location. You
have to make that decision later, based on its contents.

We are already incompletely doing this, distinguishing between
sub-accounts under which we could file our messages when the vendor
specifies the buyer's SAN next to the specific sub-account number *and*
those sub-accounts belong to different Evergreen org units. We still
need ways to distinguish in other cases.

This will do what is natural for at least one vendor, and match the
message content against the vendacct field of the acq.edi_account table.

*Also,*

We were re-retrieving the working acq.edi_message row from the database
before writing it, throwing away possible changes to the object in hand
made by O::A::Acq::EDI::process_parsed_msg(). We should only do that in
the case where that function has raised an exception.

We were doing the same kind of thing in another place actually inside
process_parsed_msg() where we set the edi_message's purchase_order field
based on the first lineitem encountered if the message itself didn't
specify a valid PO identifier.

This supports making account-correction work for ORDRSP messages in
addition to INVOIC messages.

We also propagate that same correction to the provider and shipper
fields of any invoices that get created from said edi_message.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>

TPAC: Remove fixed height from 'View My List' button

The TPAC button styles have been largely consolidated, but this little
outlier got missed.  It doesn't do anything of noticeable value in
recent browsers, and in fact hurts the display by causing the "Sort by"
widget to "hang" when resizing the window to certain intermediate
sizes.  Remove it.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Ben Shum <bshum@biblio.org>

Forward-port upgrade scripts from recent releases

In accordance with LP #894052.

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

Update OpenSRF minimum version in install docs

Evergreen now requires OpenSRF 2.2.1 or later, so it makes sense to be
clear about it.

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

Update upgrade instructions for 2.5.0 release

- Bump up minimum OpenSRF to 2.2.1
- Replace all 2.5-beta1 markers (and few stray 2.4 markers) with 2.5.0

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

Translation updates - newpot

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

Translation updates - po files

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

Don't auto-logout TPAC in staff client

Wrap the meta refresh in TPAC within a check for ctx.is_staff, so
that idle TPAC tabs will not log out when running within the staff
client.

This is intended to resolve the primary cause of symptoms mentioned
in bug 1036318 "OPAC timeout within the client"

Signed-off-by: Jeff Godin <jgodin@tadl.org>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

Fix typos in 2.5 release notes

These typos were fixed in the separate (now deleted) files, but only
after the collective file had been built.  This commit transfers those
fixes to the built file.

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

Delete separate NEXT release notes

These files have been carefully woven together into a single document,
so it is time to part with these bits and pieces.

Also, add a DONOTREMOVE file to preserve the SIP folder in the same
fashion as the others.

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

Remove manual TPAC locale applications

OpenILS::Utils::CStoreEditor::default_locale is no longer set, since the
opensrf client locale is now global.  Remove all references to this
variable.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Default per-request TPAC locale

Set the OpenSRF default client locale for each TPAC request to ensure
all outbound OpenSRF calls use the requested locale.  When done, reset
the locale to the OpenSRF default.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>