pin a version number

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

Unwrapped upgrade script for transit checkin interval setting/perm

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

Transit minimum checkin interval setting

OK, here's the situation, a copy went away on a week's vacation...

Support for preventing checkin of a copy that was put into transit "too
recently".  The scenario that prompted the development is this:

Copy at Branch A is checked in and put into transit to fill a hold at
Branch B.  The copy is then accidentally checked in at Branch B.
(Imagine a crate of items with a superbarcode gets checked in at
Branch B and the copy in question was presumed to be in that crate).
This puts the item on the holds shelf (assuming a hold transit) at
Branch B and notifies the patrom the item is ready for pickup.
However, the copy is still just embarking from Branch A.

This new setting allows libraries to configure a minimum transit time.
If a copy is put into transit and checked in during the configured
interval, the checkin will return an (overridable) event that prevents
checkin.  If the setting is not configured, this has no affect on
checkin.

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

Extend hold shelf expire date beyond closed dates

If the shelf_expire_time on a hold would land on a closed date for the
pickup library, push the expire time out to just beyond the close date
range, similar to the circulation due date overlap logic.

In the current iteration, the time component of the date is pushed out
to the end of the day (23:59:59).  If this is not generally the desired
behavior, an org setting could be added to bypass this step so that the
time matches "now" (i.e. capture time) instead.

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

honor ui.unified_volume_copy_editor setting with Add Volumes in Holdings Maintenance

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

make unified vol/copy editor honor required stat cats

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

Add i18n support for "Retrieving title..." tab name

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Integration of unified vol/copy editor and fast  add

* Wire up unified vol/copy editor for marc editor fast item add function
* Robustify callnumber vivication for unified vol/copy editor from marc editor
* Load opac after unified vol/copy with marc editor fast add.
* Various bug fixes

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

Prevent future dates of birth in patron editor

The patron editor form allowed staff to enter a date of birth that takes
place in the future. While it is theoretically conceivable (hah) that
a library might register patrons that have yet to be born, it is
unlikely enough that we will not even assign this behavior to an
org-unit setting, but we will take the bold move of using a regular
expression to prevent a future dates of birth at all.

Note that if a library system travels into the past, this will cause all
sorts of problems in the patron editor interface (not to mention
overdues). Please do not mix Evergreen and time travel.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Fix false MFHD editing cues (LP #791974)

The in-DB serial holdings use the same general display code as
the MFHD records, but were falsely triggering MFHD administration
widgets when in fact no MFHD record exists.  These holdings were
already being marked, so this fix honors that mark.

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

Tweak MFHD scoping

Three changes:
1) Add a closure for 'entryNum' value (currently used for some display
placement and menu-entry correlation)
2) Remove depth-climb to simplify code and better approximate copy scoping
3) Move 'here' assignment out of the foreach

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into dbs/fix-mfhd-render

LP 791546: advanced search ISBN/ISSN in .staff mode

In 1.6, advanced search ISBN/ISSN searches always operated in .staff
mode, returning results whether the results should have been visible to
the user or not. This confused patrons who saw records to which they had
no access.

In 2.0, this behaviour changed so that advanced search ISBN/ISSN
searches never operated in .staff mode. This confused staff who were
used to retrieving records via the ISBN/ISSN search when they wanted to
add holdings for their own library to the records.

The pattern for addressing this problem and satisfying both use cases is
the same - use the multiclass.query.staff method if we invoke the ISBN
or ISSN searches with the .staff method name.

One could easily refactor many of the search method bodies in this
module to use the exact same logic, keying off the method name to
identify the identifier field and the .staff portion of the method name
to determine whether to invoke .staff mode or not. For now we just
address the ISBN and ISSN entry points.

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

fix View Holds by preventing an empty IN list with cstore, and stubbing out part_holds

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>

for the Circulate as Type column in xul lists, display the actual citm value instead of the code

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

Fix Circulate as Type in the item editor.

The citm retrieval method was changed to use open-ils.fielder, which
doesn't return actual fieldmapper objects, so when data.js went to
convert the retrieved list to a hash, it broke.

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

Allow util.functional.convert_object_list_to_hash to handle more than fieldmapper-like objects

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

PO activation date display improvement

Use the smarter Util.js date parsing code to prevent small time offsets
in PO activation date display.

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

tweak function name

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

Refactor the "show copy notes button" functionality

Collapse the duplicated code for showing / hiding the "show copy notes
button" into a single method. Might make maintenance a bit easier some
day.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

fix the Copy Notes button in the unified vol/copy editor

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Clean up a few stock perm mistakes, part 2

0547 did not completely apply to rel_2_1, so the correction of errant inserts
from 0545 has been split off into this upgrade instead, for clarity.

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

Clean up a few stock perm mistakes

This upgrade file does cleanup for three errors:
1) deletes errant inserts from 0545 (group 4 is not the circulation admin group)
2) accounts for spelling errors (Admin != Administrator)
3) adds a group missed in 0542, for upgraders

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

Fixed and extended upgrade file 0542 (stock permissions)

This upgrade file did not work on a multi-upgraded DB, as a bunch of new
permissions and groups never made it into any upgrade.

The changes here are intended to both fix the upgrade and rectify the
missing pieces, where needed.

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

Wrapped upgrade script

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

Upgrade script, needs wrapping.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Sip Statcats - Actor/Asset Statcats via SIP2

Stat cats can now have a sip_field and sip_format.
sip_field is the field identifier code, sip_format is one of the following:

1 - Null/Empty, places stat cat value in as-is
2 - A plain string, placed in value as-is when stat cat has value
3 - A plain string with a %s, 1 and 2 combined, where the stat cat value replaces the %s
4 - A regular expression surrounded by | characters (ex, |([0-9]*) -|):
    If the regular expression does not match the value, nothing
    If the regular expression matches and has a capture group, the captured group
    If the regular expression matches and does not have a capture group, the entire match

The | was used because it would otherwise be stripped from the final result anyway.

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Merge ... transit_abort_perms from git.evergreen-ils.org:evergreen/equinox.git into master

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

Permission to allow transit abort on lost/missing

This patch introduces a permission that allows staff to abort transits
for items in the lost/missing status.  Previous, this was disallowed by
a hard-coded block.

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

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

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

Whitespace cleanup in aisle 3 (O:A:Reporter)

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

LP#740466 add order by name method to Reporter.pm to sort report folders by name Signed-off-by: Michael Peters <mrpeters@library.in.gov>

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

Collections exemption user setting

Adds a new user setting "circ.collections.exempt" which, when set to
true for a user, means the user will not be considered for collections.
It does this by ignoring the user when calculating the list of users of
interest in the "open-ils.collections.users_of_interest.retrieve" API
call.

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

Serial holdings display consistency fix

When holdings are sourced from both MFHD and the DB, display may be
inconsistent due to a hard-coded ', ' in Serial.pm.  This patch works around
the problem by storing generated_coverage as a JSON array instead of a string,
leaving display formatting to the OPAC.

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

Missing db upgrade number bump

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

Remove specific admin perms, reorganize seed file

The various administrators already have 'EVERYTHING' at their
respective levels, so no need to assign new perms there.  The
seed file is now more consistently organized.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Change a few permission levels and assignments

In an attempt to be consistent with existing perms, a few were moved to
levels lower than 'Consortium', and ADMIN_SERIAL_ITEM is only in the
'Serial' group.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

Push the grp perm mappings into the seed data

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

wrong group ids for system admin and global admin

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

put CREATE_PURCHASE_REQUEST into Users, not Staff

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

map these perms to stock perm groups

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

added other permissions missing in rel_2_1

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

some recent cataloging permissions, but I'm unsure about the new upgrade/ script regime

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>

Make database schema patch dependencies work more better

Per a suggestion from Bill Erickson, move evergreen_patch into the
evergreen schema (where we can simply call it evergreen.patch).

Also, as we seem doomed to repeat ourselves, add a little bit of schema
adjustment to the 0526 upgrade script so it can be applied
retroactively.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Make patch-checking function able to raise a notice

There's probably a more elegant way to do this, but basic
testing with the following at least didn't throw an error:

SELECT evergreen.upgrade_deps_block_check('XXXX', NULL);

Also note that the CREATE OR REPLACE function calls will
fail on an existing database due to the changed return
type of the functions - so if this gets moved into its
own upgrade script, DROP FUNCTION calls will be
required.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Bill Erickson <berick@esilibrary.com>

Merge branch 'master' of git.evergreen-ils.org:Evergreen into dbs/fix-mfhd-render

Repaired typo on schema and upgrade SQL

ARRAY_ACUM is not a defined function.  Per Dan Scott, replaced with
ARRAY_AGG instead of the correctly spelled ARRAY_ACCUM, since we'll be
using that function in place of ARRAY_ACCUM going forward.

Thanks, Dan.

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

Merge branch 'master' of git.evergreen-ils.org:Evergreen into dbs/fix-mfhd-render

LP#739444 Incrementing opac.dtd copyright date

Signed-off-by: Michael Peters <mrpeters@library.in.gov>
Signed-off-by: Dan Scott <dan@coffeecode.net>

Fix rendering of MFHD records when ownership is out of scope

Dojo would die with a null reference error when an MFHD record
was retrieved that was outside of the current OU search scope.
For example, if a given bib record had one linked MFHD record
for BR1, and one linked MFHD record for BR4, the details page
would choke trying to draw the MFHD holdings in a scope that
only included BR1 and end up drawing nothing. Ungood.

Moving the ownership check and invoking _holdingsDrawMFHD()
only after ensuring that the record is within our scope resolves
the problem.

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

indicate what date format to use for offline patron registration

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

Fix Hold Capture checkin mode

oils_persist was trumping hardcoded values.
Also, change how checked is set/unset for reliability.

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

Patch suggested by Jason Boyer for LP#788601

The <script> tag for the chrome frame message has a misplaced semicolon,
which causes an odd looking display issue in Internet Explorer. I
realize that ideally very few people will be using IE, but even if
they're not interested in a quality browsing experience we should
probably put our best face forward.

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

Distinguish between 'Show Groups' checkboxes in Serial Control

Two different checkboxes ended up with the same ID, which, as one might
expect, kinda worked and kinda didn't.

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

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

Checking for 'twisty' leads to false negatives in Serial Control

Since a tree can be opened with a double-click, we can't rely on 'twisty'
being set to load the child data.

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

Don't rely on call number column being rendered before prefix/suffix/class columns.  Also, remove prefix/suffix/class from the check-in interface, since those column definitions can make extra network calls and we want that interface to be as fast as possible.

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

wrong target for parts holds in open-ils.circ.holds.retrieve_all_from_title

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

browser.js debugging / robustification.  Replace some magic numbers with proper constants.

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

debugging aid, take the "thread trace" for a given page and make it stand out more

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

debugging aid.  split dump output produced by replacement dump function in custom.js.example

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

missed this way back when.  fix multiple event override handling (for ilsevent lacking events), re: lp#757634.  Also add some missing typeof's

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

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: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

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>

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>

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>

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>

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>

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>