Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Fix authority linking script typo

Problem reported by Ditty Mulder <dmu@iisg.nl>

Signed-off-by: Dan Scott <dan@coffeecode.net>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Merge branch 'master' of git.evergreen-ils.org:working/Evergreen into user/dbs/foo13498-29838-ws

When auto-versioning, use last piece of branch

Specifically, last / delimited piece.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into user/dbs/lp-787150-ws

trival whitespace cleanup

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

Add --nonum to make-db-patch

Makes XXXX.* files.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Trivial line-wrapping patch for call number class defaults

Signed-off-by: Dan Scott <dan@coffeecode.net>

Make label_class on any new call numbers default to org_unit setting

Evergreen 2.0 added a label_class column to the call number table with
one major purpose being the generation of correct sort keys. You can
also specify a default label class as an org unit setting. However:

1) There are no interface elements for setting the label class of an
individual call number.

2) The default setting from the actor.org_unit_setting
'cat.default_classification_scheme' value, if set, is not consulted
(that is, not set in the call number table) when new call numbers are
created.

These two facts together greatly reduce the utility of this very
valuable feature, as all new call numbers end up in the 'Generic' class.

While #1 has been addressed in 2.1+, there is still work to be done in
setting this class on import. Also, which interface parts (if any) make
it back to 2.0 is subject to debate.

This commit addresses #2.

See lp #787150.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Bring serial.unit in line with asset.copy

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

remove restriction on resetting pw to random number when Use Phone Password setting is active.  uEditMakeRandomPw will still get called for new users, but should get overwritten once a phone number is entered

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

wrong identifier and lingering copy/paste-o in code for Item Status -> Add Items

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Merge branch 'master' of git.evergreen-ils.org:Evergreen

Revert "Wrap proximity refreshing in a transaction to avoid timespans where there is no such information"

This reverts commit fbfb74639cc2628fb90e56f7dbd0d364e0b7e386.

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

Apply Apache file expiration to the current official Javascript content type

At least on Ubuntu 10.04, Javascript files are generally delivered with a
Content-Type header of 'application/javascript', not
'application/x-javascript'.  This change applies the much shorter 18 hour
expiration to the affected files, rather than allowing them to default to
1 month.

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Fix bad seed data in an A/T template

The action_trigger event def template for printing acq invoices was
syntatically broken.  The collapse filter isn't important per se, but
it balances the extra [% END %] at the bottom of the template, and it
was there originally, and I see no reason not to put it back.

Incidentally, maybe the database upgrade script in the commit will be
the first in master to successfully use the new
evergreen.upgrade_deps_block_check() function ?

Reported by Sally Fortin.

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

Restore the Replace Barcode option in Holdings Maintenance, as the Edit Item menu will only serve the same duty if the unified volume/copy editor is activated via org setting (otherwise, it's just the Item Attribute Editor).  klussier++ for the catch

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen

Fix advanced receive bug in manage_items.js
Signed-off-by: Dan Wells <dbw2@calvin.edu>

align the batch edit menus in the vol creator.  mostly aesthetic, but can save horizontal real estate, so usability as well for horizontal vol/copy editor.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

usability tweak for controlling screen real estate, in unified vol/copy editor don't embed the the Record Summary in the vol editor, but in the main wrapping UI.  Also move CN batch edit bar into groupbox

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

usability tweak, pull the Create button out of the embedded interfaces and into the wrapping UI for the horizontal variant of the unified vol/copy editor

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

css hooks so we can hide class, affix, and part columns in volume creator/editor.  can help with screen real estate with the horizontal vol/copy editor

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

usability tweak, spread the template bar across the top of both panes for the horizontal unified vol/copy creator/editor

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Fix a problem with open-ils.circ.holds.retrieve_all_from_title

Anoop Atre spotted this bug that's triggered by selecting "View Holds"
from "Actions for Selected Record" in the catalog seen through the staff
client.

Said method would error out like this:

[2011-05-20 10:04:03] open-ils.circ [ERR
:31866:CStoreEditor.pm:109:13058333633194115] editor[0|1] request error
open-ils.cstore.direct.serial.issuance.id_list.atomic :
{"subscription":[]} : Exception:
OpenSRF::DomainObject::oilsMethodException 2011-05-20T10:04:03
OpenILS::Utils::CStoreEditor
/openils/lib/perl5/OpenILS/Utils/CStoreEditor.pm:412 <500>  Severe query
error -- see error log for more details

because when trying to gather issuance-type holds, it didn't properly test
whether there actually were any subscriptions and issuances related to
the bib record in question.

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

Revert "When placing holds on a bib with monographic part, require selection of a specific part"

This reverts commit 3e63a898c53f6100d678f263e5dde5b9f678692d.

We must not force P-type holds as not all copies may not use parts.  Imagine a situation where
only one library breaks up a season of Seinfeld by using a part per DVD, and all others leave
them bound and therefore need no part designations.

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

When placing holds on a bib with monographic part, require selection of a specific part

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

order any monograph parts by sort key when placing holds

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

CN prefix/suffix columns in xul-based hold interfaces

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

label consistency in vol/item creator

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

add missing upgrade step to create staging schema

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

The term "foreign" has undesirable implied meaning, so we will dub these "conjoined"

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

Merge branch 'user/senator/lp-784062' of git.evergreen-ils.org:working/Evergreen

Make DATE_FORMAT macro check inputs

Specifically, if no date and no format (empty replaces?) replace with empty string.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Add Shelf Expire Time to hold slip, date formatter

Date formatter would work like so:
%DATE_FORMAT(date,format)%

For example:
%DATE_FORMAT(%shelf_expire_time%,%m/%d)%

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Add permission checking for updating and deleting volumes.

This addresses LP #784062 reported by Ben Shum, and I think others?

Creating volumes was already covered.  The ability to delete volumes
without permission would be less often an issue in practice since you
would need permission to delete the volume's copies before you could
delete the volume itself.

Anyway, this should square things.

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

trival whitespace changes

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

provide hint to sysadmin if memcached issue is preventing auth

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Make Perl 'make check' work without 'make all'

Factor the build of Build from Build.PL into its own
make target and then make that target a prereq for the
other make targets so that we make Build from Build.PL
when we build Makefile from Makefile.am and run 'make check'

Clear?

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

We need to update config.upgrade_log

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

Record the current upgrade version number in the baseline schema

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

Fix the definition of some of the tracking functions

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

avoid error when generating fines on checkin

Prevents "No active transaction to commit" error when checking
in overdue loans.

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

Let's have this upgrade script not use that upgrade_deps_block_check()...

...function quite yet

use correct name of upgrade check function

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

More make-db-patch goodness: wrap existing

--wrap option will take an existing file and wrap it properly.
When --wrap is specified --from is ignored.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

fix CN prefix/suffix display in XUL interfaces so we can handle any affix we come across

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Include call number prefix/suffix in display and sorting of holds for Print Full Pull List (Alternate strategy).  Also fix things so that we're sorting on the label_sortkey for call number, and not the label.  So, for this interface, the hard-wired sort order is Copy Location Order (as defined by Admin -> Local Administration -> Copy Location Order), Call Number Prefix, Call Number Label, Call Number Suffix, and Hold Request time.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

have the call number checkbox for pocket labels include prefix/suffix.  A %call_number% macro is still available for the bare call number label without affixes

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Can't use encodeURI and decodeURI for everything with persist_helper (in particular, something like "%macro1% %macro2%" as a custom entry in the label interface), so try/catch them and fall back to the original behavior.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

only use the label to initialize call number browse

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

Table name thinko

add a bit of context to upgrade SQL hints

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Improvements to make-db-patch.pl

Grab all changes to base sql files into the upgrade script
Based on HEAD (uncommitted changes) or --from=refspec

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Conflicts:
Open-ILS/src/sql/Pg/020.schema.functions.sql

Convert/move SQL script for Lazy Circ

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

Lazy Circ (AKA partial barcode lookup)

Database tables/access functions for prefix/suffix info

    Table: config.barcode_completion

    Function: evergreen.get_barcodes
        Takes org unit, context, and input barcode
        Context is a string and can contain:
            asset - asset.copy barcodes
            serial - serial.unit barcodes
            actor - actor.usr (via actor.card) barcodes
            booking - booking.resource barcodes
        Special case: asset and serial both in context
            Returns non-serial asset entries as asset
            Returns serial entries as serial
    Type for function return: evergreen.barcode_set

    With editing interface: Admin->Local Admin->Barcode Completion

    OpenSRF Call: open-ils.actor.get_barcodes
        Basically a passthrough to the database function
        Checks for permissions:
            STAFF_LOGIN - To do anything
            VIEW_USER - At home_ou of the user owning the returned
                        barcode when in actor context

Add get_barcode to menu.js and to xulG in a number of places
    Takes a window handle, a context, and an input barcode
    Passes the current OU, the context, and the input barcode to the db
    If multiple results come back it pops up a dialog
    Returns boolean false on no results
    Returns "user_false" on dialog cancel
    Returns an object with type, id, barcode on success
    If dialog was brought up, returns data used to populate button

Add option for looking up actors at checkout
    Org unit setting: Load patron from Checkout. Default: Don't.

Call sites for get_barcode:
    Lookup Patron By Barcode (actor only)
    OPAC's Staff Client Place Hold (actor only)
    Checkout (asset only by default, with actor if above option set)
    Checkin (asset only)
    Item Status (asset only)

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

LP#783486: Improve authority lookup performance

Testing showed that the two indexes added herein improve
authority lookup performance radically; on a 16 GB RAM
database server with 1.1M records, lookup times decreased
from 5500 ms to 0.223 ms. Recommended, would EXPLAIN ANALYZE
again.

Signed-off-by: Dan Scott <dan@coffeecode.net>

stop hold permit check from crashing if age protection in effect

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Ignore "deleted" authority records when using the tag-validation method

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

Merge branch 'authority-control-sets' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Borrow on_or_after() from MFHD/Caption.pm in MFHD/Date.pm

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

SQL typo cleanup

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen

standardize name of 2.0-2.1 upgrade script

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

Missed }

persist partial changes to new copies in vol/copy editor has new copies are gathered.

In other words, you can do something like enter 5 in the "# of Copies" field, enter the first barcode (say b1), edit an attribute (say, set Circulate = No), and then enter a second barcode (b2).  Previously, the changes made to b1 would be lost as b2 was entered (or as other movement around the barcode entry textboxes happened) and both copies would get a Circulate = Yes, but now in this scenario, you would see 1 copy set to No and 1 copy set to Yes for the Circulate attribute.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

simpler language, and the behavior of this action is configurable

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Merge branch 'authority-control-sets' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets

first pass at 2.0 to 2.1 upgrade script

Tested so far on an upgrade from 2.0.1; goes up
to DB patch 0533.

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Merge branch 'authority-control-sets' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

usability tweak

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

default the label class according to library setting during vol creation

Signed-off-by: Jason Etheridge <jason@esilibrary.com>

remove this comment, testing the repo & email

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

Nuke one more Trunk reference: windowssetup.nsi

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into authority-control-sets

More SVN to Git rigging

Staff client Makefile can no longer build based on SVN.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

remove references to SVN now that we're using Git

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

Trivial whitespace change

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

Merge branch 'master' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets

add missing update script for change to circ age stored procedure

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@20475 dcc99617-32d9-48b4-a31d-7c20da2025e4

Merge branch 'master' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets

Patch from Ben Ostrowsky (LP#782268) to simplify seed/example permission setup and move one step towards being able to rid ourselves of permission.perm_list.id

git-svn-id: svn://svn.open-ils.org/ILS/trunk@20473 dcc99617-32d9-48b4-a31d-7c20da2025e4

Merge branch 'master' of git+ssh://yeti.esilibrary.com/home/evergreen/evergreen-equinox into authority-control-sets