LP#1053397 MR holds seed data

* move seed data to separate file
* use subset of icon_format coded values for MR hold format

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

Grab uncontrolled attributes as well as controlled for mmr_mra

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

unapi.mmr thinko repair

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

defaulting to icon_format attr for testing

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

Upgrade updates

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

Support functions for unAPI metarecord support

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

Fix metarecord copy counting

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

LP#1053397 MR TPAC filters; editing

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

LP#1053397 MR TPAC filters; editing

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

MR targeting thinko

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into collab/miker/lp1053397-tpac-metarecords-plus-composite

LP#1053397 MR holds; more formats / lang display bits

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

LP#1053397 MR formats record attr seed data

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

Revert "Eliminate an annoying and useless warning in the JavaScript Console."

This reverts commit bb4a797010201d6f26156c9e921d78690f265e35.

My fault for picking this in without more testing. This broke things like noted
in LP1275074.

remove old, dupe unapi-mmr upgrade to avoid confusion

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

LP#1053397 collectin MR hold filter bits into API calls

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

constituent record filtering methods: syntax, thinkos

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

constituent record filtering methods: syntax, thinkos

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

LP#1053397 MR holds edit, initial.

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

New constituent record filtering method for hold targeting, also exposed to the world through open-ils.circ.holds

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

LP#1053397 MR holds placement cont.

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

LP#1053397 MR holds placement; almost

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

LP#1053397 MR holds filter display

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

Convert the holdable_formats field on unfilled MR holds to the new style

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

We must return null instead of an empty string, which is not a valid query_int

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

We need to actually decode the json...

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

Supporting SQL changes

Supply MR-friendly versions of ranked_volumes() and located_uris()
Split composite attribute compiler into a standalone complier and config-id wrapper for reuse by the hold targeter

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

LP#1053397 MR holds available formats; initial extraction

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

LP#1053397 global flag for MR holds format attr

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

LP#1053397 metabib.record_attr_flat IDL class

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

Merge branch 'lp1053397-tpac-metarecords' into lp1053397-tpac-metarecords-plus-composite

LP#1053397 more MR display / filter bits

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

LP#1053397 basic MR holds support; todo: formats

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

LP#1053397 MR search and results display

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

LP#1053397 support unapi.mmr retrieval from tpac

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

Revert "LP#1053397 initial metarecord detail page"

Rolling back MR detail page for now

This reverts commit a594b9062d512a4309f48caa8c9a1012e67258c1.

Revert "LP#1053397 remove deprecated AC key lookup"

Rolling back MR detail page for now

This reverts commit 5bf50a33a02e3758ea6e2483f8dccdcc2f4dbb41.

Revert "LP#1053397 MR detail page additions"

Rolling back MR detail page for now..

This reverts commit 3bae24efbce8663df574aeea43b96b67a8f41fb1.

LP#1053397 MR detail page additions

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

LP#1053397 remove deprecated AC key lookup

Remove the extra, unnecessary DB call to find the record's added content
key, since the lookups are now based on the record ID instead of the
ISBN, etc.

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

LP#1053397 initial metarecord detail page

* support for MR-focused copy query
* MR-focused hold / copy summary counts

Still much to do

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

LP#1053397 unapi.mmr() gets new slimit->bre limit

Support limiting the number of constituent records to flesh into a
metarecord.  This carries particular weight when fleshing holdings,
since a large number of constituent records could produce very large
documents.

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

LP#1053397 unapi.mmr() repairs

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

LP#1053397 initial unapi.mmr() support

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

LP#1269911 composite attributes admin UI

New interface for managing composite record attribute definitions:

/eg/conify/global/config/composite_attr_entry_definition/<id>

The UI for a coded value map is accessed from an existing coded value
via a new "Manage" link column in the CCVM table.  The UI allows staff
to build tree-shaped boolean composite definitions for CCVMs in terms
of existing CCVMs.

Additionally, the record attribute definition UI now has a link from
each definition to the coded value map page for the attribute.

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

Teach unapi about composite attributes

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

Need to protect against null values for sorters

Signed-off-by: Mike Rylander <miker@esilibrary.com>

Provisional upgrade script for composite attributes

Various baseline fixes including syntax repairs, use of
JSON::XS for debian squeez support, correction of plperlu
SPI usage.

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

Inform the IDL and confiy of composite attributes

IDL link from ccvm => ccraed via composite_def field

IDL selector attribute for ccvm class

ccraed gets CUD actions

In conify/global/config/coded_value_map, if the selected attr type is
composite=true, show a link from each coded value to manage the
composite definition.

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

Schema elements and ingest logic for composite attributes

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

Teach unapi about multi-valued attributes

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

Need to distinguish between controlled and uncontrolled attributes when finding value IDs

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

Fix constructed method call syntax and a logic inversion

Signed-off-by: Mike Rylander <miker@esilibrary.com>

Perferred language query fix

Signed-off-by: Mike Rylander <miker@esilibrary.com>

Make extra sure there is no empty data

Signed-off-by: Mike Rylander <miker@esilibrary.com>

Schema bug fixes from testing

Signed-off-by: Mike Rylander <miker@esilibrary.com>

Teach the upgrade script to convert old data

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

Provisional upgrade script for MVF

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

Always replace sorter values

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

Add a useful index

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

Teach QueryParser new tricks

QP Needs to be made aware of several new structures in the database.

First, we have added a new sort-supporting table called metabib.record_sorter
which holds values extracted by crad.sorter=true attrs.  This is used instead
of the mrd.attrs->"something" hstore composite.

Next, we teach QP how to convert from a list of user-supplied values across
many dynamic filters (based on crad) into an intarray query of ids extracted
from config.coded_value_map (in the case of controlled attributes) or
metabib.uncontrolled_record_attr_value (in the case of, you guessed it,
uncontrolled attributes).  This query is applied against the vlist column
of metabib.record_attr_vector_list, which is GIN indexed for speed.

Finally, metabib.record_attr is now a view over metabib.record_attr_vector_list
and is consequently going to be slow for general use.  We restrict
its inclusion in the core query to only the case of a during() filter
which requires access to the value of a bib's Date2 field.  For the
other common case, requiring access to the Date1 field, we instead
use the pubdate sort value now stored in metabib.record_sorter.  We
might consider making the specific sorter attribute used configurable
so that we can change the definition of pubdate down the road, but it
starts out (and generally stays) defined as equivalent to Date1.

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

Teach the IDL about MVF-related structures

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

New intarray-focused attribute extraction

We rewrite the record attribute extraction to capture all the
record-supplied values for each attribute (where multi is true)
and store that list in the new (fkey-corrected) metabib.record_attr_vector_list
table.  Only filters make it into this table.

We also insert a parameter after the record id to accept a list of
record attributes we want to rewrite. This defaults to NULL to rewrite
all of them.

Sorters are stored in a new, separate table built specifically for them.

metabib.record_attr becomes a vew atop metabib.record_attr_vector_list
which expands the intarray stored therein into an hstore. For
multi=true attributes, only one will be returned through this view,
as is HSTORE's way, and which will be returned is undefined.  However
this view is only provided for the purpose of backward compat with
reports or other locally defined logic.

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

Move current record attr logic into its own function

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

Add a multi flag for record attrs

We pre-coordinate which record attrs are allowed to be multi-valued
with this new bool.  Most can be, we set the default to true and
adjust the seed data for those that should be false (sorters and
fields in the leader).

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

Hidy hole in which to stick "uncontrolled" values

In order to make use of the massive speed increases provided by
intarray indexing, we need to use (you guessed it) integers.  But
uncontrolled record attributes are not necessarily (or even very
often) numbers.  We will store them in a table of unique (per
attribute) values, and use the id from that table in our intarray
indexing.  That id comes from a DECREMENTING serial that starts
at -1 and counts downward.  This avoids collision with the other
set of integers (the id from config.coded_value_map) that we will
use for controlled record attribute values.

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

Add intarray extension

We need intarray for GIN index support of integer arrays, which is
how we'll be storing the in-use record attribute value identifiers.

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

Teach vandelay.marc21_physical_characteristics() to see all 007's

We use vandelay.marc21_physical_characteristics() to extract fixed
field data that lives in the 007.  Before this change, it would
only look at the first 007 in the record.  Now it will look at
all of them in turn, supporting configurations such as DVD+BluRay.

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

Include RFID docs with full path

Ben Shum noticed that the RFID docs weren't showing up in the documentation; we
need to include the full relative path to the RFID docs.

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

Fix copy_info variables for result and record view

Because we yanked out the code that generates the link into a separate INCLUDE
file, the function was looking for something for copy_info. The one for result
was missing entirely and the one for record was backwards.

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

KPAC library page: give the kids some style

Just the bare minimum: some padding around the body, and a font colour and size
that's reasonably consistent with the rest of the look of the KPAC for h2 and
h3 elements.

Yes, I know there is text-transform:uppercase and h2 is sized at 48px but that
seems to be used for "You saved it!" and "Get it!" banner things. On the
library info page, we're using headings for headings.

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

Clean up the breadcrumb trail in the KPAC

Links to the library pages don't maintain GET context, so we can't feed search
or "Get It" links in the breadcrumb trail, currently. We can revisit this,
of course.

Wasn't there some fairy tale about kids and a breadcrumb trail... once upon a
time?

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

KPAC root, OPAC root: we all live under one root

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

Begin enabling KPAC-branded library info page

This needs work and is going to get squashed like a bug when we're through.

Yeesh.

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

KPAC: Won't somebody think of the children's record details?

Factor out the library name link logic into a separate template that can be
included by TPAC search results, TPAC record details, and KPAC record details.

Notice along the way that the KPAC is devoid of copy-level structured data,
but save that for later.

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

TPAC: Link library in results page to library page

Dan Wells noticed that the links on results pages went to a different location
(potentially, depending on the "prefer external library uri" OUS) than the
links on record pages. Make things consistent.

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

Forward port 2.5.2 upgrade script

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>

Stamping 0854: data.min_max_prices

Stamping 0854 and resolving permission ID conflicts due to drift.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

Release notes for lp1207903: new lost billing min/max setting

Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Jeff Godin <jgodin@tadl.org>

Add new option for max/min lost item pricing

In addition to a 'default' price when price is missing or zero,
we can also accommodate a range of prices by saying it should be
at least 'X' and not more than 'Y'.  This also allows you to
effectively set a fixed price for all lost items by setting min
and max to an equal amount.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jeff Godin <jgodin@tadl.org>

LP#842991 Reports needs to error when deleting fails

When attempting to delete a report template, the reports interface
incorrectly shows "Action Succeeded" even when it partly fails. If the
user performing the delete does not own all the dependent reports and
outputs, then the perl function returns 1 to indicate that some were
deleted, but not all. However, the javascript that calls that function
does not check for a return value of 1. This commit adds that check, and
also improves the formatting of the related error messages.

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

Serials: improve routing slips per Dan Wells' suggestions on LP #1229349

  - Don't repeat the same address for the same user
  - Label mailing vs. billing
  - Border to separate mailing and billing addresses
  - [Also] move some inline styles to a CSS file

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

Stamping 0853: 'Prefer external URL' OUS

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

Release notes for the TPAC library web pages

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Add upgrade script for lib.prefer_external_url OUS

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Copy details link to Evergreen library page by default

Add a new library setting, 'lib.prefer_external_url', to force library names in
copy details pages to link to their associated 'lib.info_url' setting rather
than the Evergreen-generated page. In the absence of a 'lib.info_url' setting,
the name will fall back to being linked to the Evergreen-generated page.

Per discussion with Ben Shum and Kathy Lussier.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Improve label for library's external web site

Normal humans will find "Library web site" more meaningful than "External web
site"; thanks to Kathy Lussier for the suggestion!

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

%l instead of %H if we're using AM/PM for default time formats

Thanks again to the indefatiguable Ben Shum for this catch!

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Use the 'format.time' OUS to format library hours

Sparked by a concern that Ben Shum had over the display of library hours in 24
hour format, I eventually figured out that we could use the Template::Toolkit
Date module to format the times according to library preferences. And given
that we already have a format.time OUS, let's use it.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

TPAC Library schema info - add some caching

Rather than hitting the database every time, cache our requests for a
period of time. It's unlikely that hours of operation or addresses are
going to change very often...

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Link from copies to library info page

In the absence of a specific lib.info_url org unit setting, link to the library
info page.

If we _do_ have a lib.info_url org unit setting, link from the library info
page to to the external web site.

Rather than wrapping everything in the table cell as the seller, focus
on the <a> and give it an explicit Library typeof attribute; then we can
offer up the name of the library as the name attribute, should the
crawler be too lazy to follow the link.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

First inklings of schema.org support for library pages

Adds a TPAC page where the information for a given library can be publicly
exposed; in addition, that information is marked up with schema.org structured
data so that, for example, a search engine can offer up the hours of operation
and contact information for a given branch right on the search results page.

We give the content-wrapper a main-content for indentation, to be consistent
with other pages in the TPAC.

We also add the cargo-cult common-full-pad div for a bottom margin.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Fixed title/author display at checkout for non-pre-cat items.

Fixed a display problem on checkout where items that were formely pre-cats
were still displaying the pre-cat data. Now it will make sure the item is
still a pre-cat before using the dummy data.

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

LP969312: No warning for Delete All from Catalog in Copy Buckets

Add a confirmation dialog popup.

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

TPAC: Display authors using inline-block

Rather than letting author credits wrap willy-nilly, use inline-block mode so
that each line will begin with a new author. Devices with constrained screens
in particular benefit from this, as the list of authors is much easier to scan,
and wider screens can still fit multiple authors on a single line.

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

TPAC: Use indexed subfields only in author search links

Addresses LP# 1267231 in which we found that the titles of works in the added
author field (such as subfield t) were showing up in the link ahead of the
author's birth and death date (if applicable). Now we reserve the link for
only the indexed author subfields (depends on whether the name is personal,
corporate, or conference, but generally subfields a/b/c/d/e/n/q), then the
extra subfields go after the name + dates, then we finally put the relationship
into the parentheses after everything else.

We also simplify the markup so that each name is contained in a single <span
class="rdetail_author_div"> element to make it easier to control the layout.

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

Serials: Batch Receive interface Copy Location widget was inert

Choosing a value from the Copy Location widget wasn't working.  You'd
get a default copy location (or the one supplied by the Receive Unit
Copy Template) no matter what you set that control too.

This is fixed.  One does not read from XUL menulist element by using its
'value' property.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Remington Steed <rjs7@calvin.edu>

LP#882586: In Serial Control View, show copy templates owned by ancestors

When creating/editing distributions, dropdowns for Receive Unit Template
and Bind Unit Template now show copy templates owned not only at the
distribution's holding lib, but at any of the holding lib's ancestors.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>