Asciidoc edit for My Lists documentation

Change to asciidoc structure in opac/my_lists.txt

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

Acq: Be fuzzy about case when retrieving EDI documents

The edi_fetcher relies on a routine in OpenILS::Acq::EDI to, among
other things, try to avoid fetching the same EDI document multiple times
when many rows in acq.edi_account refer to the same host and login
credentials.

Since in practice most vendors seem to run FTP servers for EDI on
Windows, not UNIX, and pathnames are therefore case-insensitive, that
test for other occurrences ought also to be case-insensitive.

If I were to look at this as a purist, I could argue that vendor servers
might sometimes by run on UNIX, and that for some reason it is possible
that different vendor-to-buyer EDI documents (order responses or
invoices) could have pathnames that differ only in the case of some
characters.  But that seems wildly unlikely.  If anyone does take this
possibility seriously, perhaps acq.edi_account needs a Boolean column to
indicate the remote host's case [in]sensitivity.

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

Fedora RPM for bzip2 headers is bzip2-devel

There was an attempt to add the right RPM for the bzip2 headers in
Makefile.install, but that failed on Fedora 19. Swap in bzip2-devel
instead to correct that.

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

Asciidoc fix for documentation

Fixed faulty asciidoc in docs/admin/lsa-library_settings_editor.

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

LP#1053074: Editimg MARC Fixed Fields jumps cursor to marc record

Put the cursor back to the current fixed field after reload
the MARC record table.

Signed-off-by: Simon Hieu Mai <hieu.mai@mnsu.edu>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

LP1183340 selectivly apply editable funds sorting

Avoid sorting editable funds to the front of the fund list when there
are no org units at which the user has the ADMIN_ACQ_FUND permission.
Otherwise, we are sorting on an empty 'IN' list in pcrud, which results
in an error.

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

Documentation for preset tabs and auto login

Missed the actual documentation in the last commit for preset tabs and
auto login docs. Here it is.

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

Pre-set tabs and auto login

Adding staff client documentation for pre-set tabs and auto login.

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

Acquisitions Admin updates

Updates to the chapter on acquisitions administration, including 1) the
reintroduction of the Providers section, which appears to have dropped out
of the docs after 2.1 2) the new default # copies and improvements to
distributions formulas introduced in 2.4 and 3) minor copy editing.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>

Documentation for Org Unit Hiding in TPAC

Added Org Unit Hiding in TPAC documentation to admin/lsa-library_settings_editor.txt.

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

Documentation for Call Numbers in My Lists

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

LP1081576 - fix utf8 characters in searches using portal page

Replace escape() with encodeURIComponent() on the portal page to allow utf8
characters to be passed more appropriately to the subsequent search in catalog.

Thanks Dan Scott for pointing this out.

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

Documentation updates for acquisitions invoices

Updates to acquisitions invoices documentation, including new 2.4
features. Also moves up some 2.1 documentation that is still relevant.

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

LP1168186 - Add tag 111 and remove subfield 4 from author pool

Add meeting/conference main entry (111) to TPAC author pool.

Remove subfield 4 from the author data because the information contained
refers to relator and may not be helpful at this time.

Signed-off-by: Ben Shum <bshum@biblio.org>
Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>

Add corporate author (110) to TPAC author pool

As requested in LP#1168186, here is a simple attempt to add corporate
authors to the author 'pool' in TPAC.

We may also want to consider adding 111 (meeting or conference as
main entry), but that isn't totally clear cut.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

Documentation for Best Hold Selection Sort Order

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

Updated root

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

Org Unit Proximity Adjustment documentation

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

LP1183357 Pass locale to bib search (tpac)

Pass the TPAC locale value, which is collected at each page load, to the
bib search API call.  This allows us to take advantage of language-based
relevance bumps.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

TPAC: Death date missing from authors in record details

The trailing period prescribed by ISBD punctuation for x00 $d subfields
in certain circumstances confounds the regex that was expected the $d
subfield to match ^\s*\d{4}-\d{4}\s*$. As a result, death dates were
often being omitted from the record detail display. Relaxing the regex
to match against a trailing .* instead of \s* resolves the issue.

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

Fix Launchpad Bug 1183418.

Clicking on an author with a dash or dates in the name on the search
results page comes up with 0 results. It looks like the dash is stripped
from the date and/or name.

Clicking on such a link from the record view actually works because
the dash is replaced with a space.

This branch changes table.tt2 to replace special characters in author
output with a space in the same way that authors.tt2 does.

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

LP#1182360: typo fix

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Database upgrade script for 2.2.9

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

2.3.6 -> 2.3.7 DB upgrade script

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

Correct subfield format for authority lookup

The Authority Control Set code expects the format of subfields
that it will be used to build a MARC.Field object to be of the
form: [[code,value],...]  This commit makes that true.

Additionally, dojo.filter over a list of lists can cause the
nested array set to be flattened.  So, instead, we loop directly.

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

Repair faulty MARC editor authority validation

Repair how the JS inspects the results of
open-ils.search.authority.simple_heading.from_xml.batch.atomic, which is
used to look up matches for authority validation.  It returns a more
complex structure in the brave new world of authority control sets.

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

Apply all 2.2 upgrades prior to running 2.3.0 upgrade

If users are expected to run all of the 2.2 upgrade scripts, then let's
make it easy for them. As PostgreSQL 9.1 does not include the \ir
"include relative path" command, we have to pause at the beginning of
the upgrade process to ensure that the user is running the script from
the right directory.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP#118066: fix searches that failed because punctuation was a token in query

When building the text array literal to pass to evergreen.rel_bump,
make sure that tokens that get {search_}normalize{d} away
don't result in syntax errors.

With this patch, searches like the following will work again:

keyword:Take me to the river / Al Green and Mabon

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

LP1179614 - fix Export All Records action

In the record buckets, the button for 'Export All Records' would work up
to the point where you chose where to save the file. After clicking where
to save, it would not show any saved files or prompts indicating it completed
the export process.

This fix changes the location of where to find the export tool and properly
exports the bibs as the file format selected.

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

Serials: MFHD::get_compressed_holdings() can reach infinite loop

Even controlled serials holdings involve the internal creation of MFHD
fields, upon which caculations are performed for such purposes as the
display of holdings summaries in the OPAC.

There are too many ways that incorrect MFHD (or MFHD that our code just
can't yet handle) can lead our MFHD routines to crash. We can't address
all these possibilities in a single bug fix. But we can avoid this
infinite loop.

A subroutine within open-ils.serial (_summarize_contents()) relies on
MFHD::get_compressed_holdings(). When the latter went into an infinite
loop the result would be an open-il.serial drone process consuming CPU
time indefinitely and, depending on the data that provoked the loop,
potentially writing repeating messages to stderr indefinitely.

End users will still see the item receiving fail in these cases, and be
obliged to work around the issue as before until more robust
holdings summarization code can be written, but at least the overall
condition of the running Evergreen system won't be affected, and there
will be better information in the error logs.

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

Serials: Test method to identify holdings that lead to summarization bugs

This adds a simple test method that allows testing of problem holdings.
See the next commit.

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

LP#1176206: improve deadlock handling in 2.3-2.4-supplemental.sh

- make sure that it doesn't attempt to create more than one
  index at a time for a given table
- if deadlocks occur (because of a PostgreSQL bug whose fix
  isn't slated for release until after April 2013), find
  and recreate the affected indexes.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Exports are broken

Proposed fix for lp1175816, tested with 2.3.2 / 2.4.0.

Signed-off-by: Mark Cooper <markchristophercooper@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Add support for CAS intro in tpac for rel_2_4_mergery

These changes allow a CAS intro to be put in place before
a patron is sent off to the CAS service from the tpac. CAS
logins can be a jarring experience since the interface is
controlled by the validating organization.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Merge remote-tracking branch 'conifer/feature/tpac_show_details_config_2_4' into rel_2_4_mergery

TPAC: Make "Show more details" in results optional

Academic sites typically handle a lot of electronic content and want to
bring most details about entries, such as availability, status, and
URLs, on a results screen to the front.

This adds a 'show_more_details.default' option in 'config.tt2' that can
have the values 'true', 'false' and 'hide'.  The 'true' and 'false'
values set the default for the button to more and less respectively, and
'hide' suppresses the button from showing at all.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Merge remote-tracking branch 'conifer/feature/ldap_osul_2_4' into rel_2_4_mergery

Add an ou_host_name parameter for TPAC login forms

If set, and the incoming username does not already include an '@' symbol
(a very simple attempt to detect if we're already dealing with an email
address), then append '@' + the ou_host_name value to the username for
authentication purposes.

The rationale is that in a large consortium, you might want to enable
users to log in with short usernames (like 'fred'), but you also want to
avoid conflicts between short usernames at different organizational
units. Thus, create the users with the email equivalent of their
usernames, like 'fred@br1.example.com' and 'fred@br4.example.com', and
let the templates for the TPAC in br1 and br4 contain a hidden input
field to append the appropriate email hostname.

Truth be told, this is probably most appropriate for a large consortium
containing two or more academic institutions that hope to use LDAP
authentication rather than native authentication, and therefore have
LDAP CNs that map to email addresses of CN@hostname that can then be
mapped to actor.usr.usrname (and actor.usr.email, of course).

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

Very rough LDAP authentication enablement for OSUL

Currently requires that the user enters their email address at the
username prompt; we then just use the local-part of the email address to
authenticate against the LDAP server.

In the VirtualHost sections of Apache, we can use SetEnv to force the
physical_loc to match the org unit ID(s) specified in opensrf.xml; for
example:

SetEnv physical_loc 103

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

Merge remote-tracking branch 'conifer/feature/hold_slip_format_due_date' into rel_2_4_mergery

Merge remote-tracking branch 'conifer/feature/CAS_auth_dynamic_links_2_4' into rel_2_4_mergery

Merge remote-tracking branch 'conifer/bugfix/recall_not_more_than_loan_2_4' into rel_2_4_mergery

Merge remote-tracking branch 'conifer/feature/callno_tpac_template_2_4' into rel_2_4_mergery

Merge branch 'feature/barcode_generator_2_4' into rel_2_4_mergery

Merge remote-tracking branch 'conifer/feature/patron-privacy_2_4' into rel_2_4_mergery

Merge branch 'feature/move_to_storage_2_4' into rel_2_4_mergery

Add production "moveto" options

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

Move the moveto to the new moveto template place

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

Add "Move to storage" quick UI for bulk location moves

This simplistic interface accepts a barcode and immediately returns
some bibliographic information if the barcode was found - and the
item is moved immediately to the specified location. Goal was to
minimize clicking, this pretty much requires just scan scan scan.

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

Merge remote-tracking branch 'conifer/feature/marctemplates_2_4' into rel_2_4_mergery

Merge remote-tracking branch 'conifer/feature/fts-polish-l_2_4' into rel_2_4_mergery

Merge branch 'feature/circ_scripts_2_4' into rel_2_4_mergery

MEDIACEN doesn't charge anything on AV-EQUIP items

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

Add 3-day loan rule for AV-EQUIP for media centre

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

Prevent renew if item already is targeted for a hold

From http://biblio.laurentian.ca/tickets/conifer/wiki/devHolds
apparently I missed this in moving our customizations from SVN (or it
was only ever local?)

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

OSBO: Accidental copying of the "DVD VIDEO" rule

Meant to remove the "DVD" from the "DVD VIDEO" part of the circ modifier
the second time around.

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

OSBO really did want $0.25/hour for all reserve periods

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

Add Reserve circ rules for Boreal

Still need some clarification on how these rules are to be applied, but
better to get the durations working right now and fix up the fines
later.

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

Add LEDL MISC to the list of allowed circ modifiers for Leddy

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

Add ILL-BOOK to the list of allowed circ modifiers for Leddy

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

Add LEDL MISC 1 year 0 renew rule

This had been working in production, but apparently never got
checked into git?

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

Add Boreal circulation rules

Still need to add their limits based on user profile & item type

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

Add ILL-BOOK circ rule for OWA

Per ticket #683:

Loan period of 3 weeks
NO RENEWALS

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

Track Conifer JavaScript circulation rules

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

Generic patron barcode generation (OpenSRF and DB)

Laurentian University needed the ability to generate barcodes as part of
its LDAP integration work, and the first generation (so to speak) of the
was specific to LU - including hard-coded prefixes and database
functions that include the "lu" name.

This commit makes the functionality much more generic and thus more
likely to be able to be adopted by other institutions. The principle
components are:

Database functions:

evergreen.actor_generate_barcode([prefix TEXT]) - returns a 14-digit
  barcode from the evergreen.actor_barcode_seq sequence with a prefix of
  'AUTOBC' or the specific prefix of up to 6 characters. If the
  resulting barcode is all digits, then the 14th character will be a
  mod10 check digit; otherwise the 14th digit will be '0'.

evergreen.actor_update_barcode(usr_id INTEGER[, prefix TEXT]) -
  generates a new barcode for the specified user, with the optional
  barcode prefix.

evergreen.mod10(barcode TEXT) - given a barcode, generates a mod10
  check digit and returns the barcode with the appended check digit

OpenSRF method:

open-ils.actor.generate_patron_barcode([usr_id INT[, prefix TEXT]]) -
  generates a new barcode for the patron

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

Add and use a patron search that overrides opt-in invisibility

Something like this is required for API calls that need to operate
against a number of libraries in a given instance that are using opt-in;
otherwise, attempts to search for users will fail and you may end up
creating near-duplicates etc.

The implementation adds an
open-ils.actor.search.patron.advanced.opt_in_override method to
open-ils.actor, which, if invoked, checks to see if the caller has the
OPT_IN_OVERRIDE permission. If so, then the crazy_search ignores the
normal opt-in limits and searches all pertinent users in the database.

As a global permission, OPT_IN_OVERRIDE is a blunt instrument. Others
might want to put together a more refined version that uses OU depths to
define boundaries.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/actor.pm

Remove extraneous spaces from bib email template

Various email servers refuse to send email that begins with blank lines,
so use TT's [%- and -%] to eat up whitespace. Also, remove the extra
linefeeds (_ "\n") as the linefeed is already generated by virtue of
being on its own line.

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

Add line return to URL entry

I missed this in the URL rendering but lines need a line return
for the e-mail option.

Add option for host value in URLs

The ability to link back to a bib record for print and e-mail displays
from tpac is very useful, esp. for e-mail where there is a good chance
that the mail system will make it an active link. It looks possible to
wire in environmental variables with the template toolkit but that's
probably as complicated as it is to modify the template directly.

Add call number and location info to TPAC e-mail and print option

I haven't figured out how to get the hostname properly, I am guessing
it needs to be passed as a parameter somehow. I also need to test
URIs but this is a start.

Make Date Addition for Hold Receipt Configurable on Client

As per Dan Scott's suggestion, this changes the print_win.js
function for formatting the holds date receipt to allow
the number of days to be specified with the "add_days" attribute.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>

Hold Slip Formatting for Windsor

The Leddy Library used some custom javascript for adding a due date
to the Hold Slip in a previous version of Evergreen. This uses the
far cleaner print_win.js setup to bring this back.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>

Fix CAS to handle dynamic auth links

CAS uses a "service" URL to redirect a user back to where an application wants
them to go if authentication is passed. This works well for simple URLs but
becomes problematic for complex URLs, which can get mangled or, worse,
truncated. This doesn't seem to be unique to Windsor's implmentation of CAS.

I tried a few tricks to encode the URLs in strange ways to get the same
URL back from CAS, but I think a better approach is to stuff the URL into
a cookie, and invoke when the request comes back from CAS. Any other approach
seems to result in horrid URLs and there is always the chance that some
character will break the scheme.

One flaw in my approach is that if a TPAC user selects "email" or "place hold"
and invokes the logon screen, the cookie gets set for the "redirect" URL. If,
for some reason, a user decides to do another search and chooses to log in to
their account from a different screen, the CAS URL can be invoked. This
would only happen for CAS, and the cookie itself is only set for 10
minutes, so I don't think this is a major concession. I also try to
invalidate the cookie wherever it is possible to know that the authentication
has not been invoked.

The assumption is that there is a CAS link added to the login form (login/form.tt2),
for example:

 <a href="[% ctx.cas.url %]"
    class="opac-button opac-button-header" id="home_myopac_link_uwin">
    [% l('Log in to Your Account (UWind ID)') %]
 </a>

I had become so used to testing CAS by logging in first, that I totally missed
the links that support authentication at the time of need, e.g. the "email"
or "place hold" links that are displayed prior to authentication. Hopefully,
this branch will address what is probably a common scenario.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Conflicts:
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader.pm

Check to see if existing loan date is less than recall date

It seems possible that a recall can be issued that actually extends
the loan period for an item with semester or other generous loan
periods. This adds a check for the current loan period and uses it
if the calculated threshold date is later.

Signed-off-by: Art Rhyno <art632000@yahoo.ca>
Signed-off-by: Dan Scott <dscott@laurentian.ca>

Remove display of patron birthdate, internet access level, etc

These fields in the staff client are either not applicable or
potentially privacy-invasive or both. If the data somehow does
get into the system, we don't want it to show up on the staff
client anyway - and we don't want annoying warnings about unset
dates of birth, etc.

This is the hardcore way of doing things, we really should make
it yet another configuration setting. Oh well.

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

Add Media Centre MARC template

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

Flag the UTF8 encoding for MARC templates

The UTF8 encoding is flagged by LDR09 being 'a'. If this is not set, it
greatly confuses Evergreen, which tries to convert what it thinks are
MARC8 characters to UTF8 before saving the record.

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

Add Conifer's custom MARC templates

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

Treat a Polish l as an ASCII l in full-text search

Is this code path even hit these days? Would have expected
naco_normalize to be poking around here if that was the case...

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

Restore the "Local Administration" link to the splash page

From Local Admin we can get to Library Hours, and thence satisfaction.

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

Robin's interface for managing library hours

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

Add params support to show in opac function

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

LP#1175287: make sure tsearch2 is removed, regardless of its origin

A sufficiently old Evergreen database might still have tsearch2
exist as an unpackaged contrib, so convert it to an extension first.

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

adjust release notes references to refer to 2.4.0

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

update upgrade instructions for 2.4.0 release

- specify minimum supported PostgreSQL version as 9.1
- specify minimum supported OpenSRF version as 2.2.0
- update schema upgrade instructions and miscellaneous
  references to 2.3

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

revisions to installation instructions

- Remove references to PostgreSQL 9.0; 9.1 is recommended
  for Evergreen 2.4.
- Clarify that the latest version of OpenSRF now means 2.2
  or later.
- Increase chances that somebody following the instructions
  precisely doesn't run into a problem during the configure step.

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

Make sure the "running" indicator goes away

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

Add a $cache_timeout-based backstop for infinite loops

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

"Queue Compression" -- let one do the work for all identical, concurrent searches

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

LP#1174359: follow-up on dropping the reporter.classic_item_list view

Add note for the benefit of any Evergreen admin who may
need to recreate that view.  Also, update the point schema
update script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

LP#1174359 Need to drop reporter.classic_item_list

ERROR: cannot drop view metabib.full_rec because other objects depend on it
DETAIL: view reporter.classic_item_list depends on view metabib.full_rec
HINT: Use DROP ... CASCADE to drop the dependent objects too.

Beginning at line 55 of the upgrade script, several views and functions are dropped
so metabib.full_rec can be dropped. One of them is missing.

We need to drop reporter.classic_item_list as well, for the script to work.

Signed-off-by: Michael Peters <mpeters@emeralddata.net>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Improved AC socket fix

This provides better coverage for socket shutdown needs, but intends
to fix the same issue as the previous commit.

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

Shutdown added content sockets to prevent hung connections.

We were getting several thousand sockets hanging around in a
CLOSE_WAIT state on our server.  The connections were from our
server and to our server.  Further investigation showed it to be
caused by the stage2 added content lookup not closing the request
sockets when it was done reading data.  This branch fixes that.

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

LP1076411 Return updated circ object from checkin

Retrieve an updated copy of the circulation object to return to the
caller during checkin.  This ensures the data in the circ object, in
particular date fields, are consistent with the database.

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

When adding errors from failed processing to acq.edi_message, set error_time

This has obvious benefits in facilitating log analysis after a problem.

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

Add accepting_usr to IDL to fix reporter field

In the reporter interface, the view 'Payments: Brick-and-mortar' is
missing the 'Accepting User' field, which is present in the similar view
'Payments: Desk'. This field is already present in the database view and
is simply missing from the IDL.  Since it is needed for certain kinds of
reports, this commit adds the missing IDL fields. It also adds the
missing 'reporter:datatype' to an existing instance of the IDL field.

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

LP1170861 Remove unused lineitem alert field

Remove extraneous field from the IDL field list for acq lineitem alerts.
The extra field prevented new alerts from getting saved.

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

LP1104823 - OPAC search results "Limit to available items" toggles
off "Show more details"

Added a hidden input to the form when "Show more details" is selected
so that when the form is reloaded, "Show more details" will be still
selected.

Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
Signed-off-by: Ben Shum <bshum@biblio.org>

LP#1172893: fix query for retrieving available holds

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

Add 2.4 release notes for Novelist Select changes.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>