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

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

LP#740466 Sort report folders by name

Add order_by 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>

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>

Revert "f424dd8" for bad commit message

This reverts commit 997eef7b50acf10ec2afcb8d8933b75317e0410f.

f424dd8

Missing db upgrade number bump plus backport fix

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>

LP#739444 Incrementing opac.dtd copyright date

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

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>
(cherry picked from commit bbc874ddb42ac05130ccfad92fc55ccebd6af882)

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>

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>

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>

upgrade_deps_block_check() is not in the branches at this point

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

Use the correct upgrade log format for rel_2_1

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

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 'rel_2_1' of git.evergreen-ils.org:Evergreen into rel_2_1

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

This reverts commit 2a16bb01ebf782a248155bee9c2d37d5b3184e0c.

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 'rel_2_1' of git.evergreen-ils.org:Evergreen into rel_2_1

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 69395e8c55db3a0993b2a48eb930ab55402d5c8d.

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>

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>

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>

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>

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>

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

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

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

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

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>

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>

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>
(cherry picked from commit 473bc4784377977cc16a4f61cd8961be0ffe40df)

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/branches/rel_2_1@20476 dcc99617-32d9-48b4-a31d-7c20da2025e4

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/branches/rel_2_1@20474 dcc99617-32d9-48b4-a31d-7c20da2025e4

A bevy of MFHD bug fixes (and test cases)

1) Remove hard-coded 'gap' text from format_part()
2) Honor $ypm## for chron_to_date() for annuals
3) Removed overly simple check on $w as digit
4) Allow for combined chronologies other than 2nd level
5) Make calendar changes trump $u counts
6) Honor issue restarts at calendar changes
7) Rudimentary support of $u counts for non-restarting issue numbering
8) Special interpretation of $yps for early winters (i.e. winter is the first season of the calendar year)
9) Teach chron_to_date() about early winters

The last two points deserve some special attention.  A large number of journals consider 'Winter' to be the first season of the year, so Winter 2011 means (roughly) January 2011, not December 2011.  However, there is no explicit way to indicate this in the standard, and also the code relies in many ways on the fact that 24 (chronology code for winter) is greater than 21 (code for spring).  To work around this, we are interpreting a certain ambiguous pattern in a particular way.  Specifically, if you set $w to a digit, then set $y to 'ps24,21,22,23' (or some variation with '24' as the first value), you will get this winter-first behavior.  Also, the season is treated internally as '20' then converted to '24' when processing is completed.

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

fix query in various copy count functions

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

for upgrade, let prefix and suffix in audit be null

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

add affix columns to acn auditor table

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

Patch from James Fournie addressing LP#781803: Multiple renewal fails on MAX_FINES circs

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

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

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

Address LP#779975; Status of Available (0) not honored because 0==false -- use ternary op with defined() instead

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

Add missing permission checks to Serial.pm

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

Misc updates to installer rigging

Fewer spurious errors (extras.nsi, MUI_TEXT)
More checks (makensis, unzip, branding)
Slightly improved Welcome screen

Author: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20447 dcc99617-32d9-48b4-a31d-7c20da2025e4

Properly tag the branch for the staff client build as suggested by tsbere

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

bump xulrunner version

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

Use valid JSON in Dojo i18n for Searcher.js

Dojo i18n processing scripts require valid JSON (quoted keys in hashes),
even though Dojo itself is more relaxed and accepts unquoted keys. So,
do the i18n dance here.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20438 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix typo introduced in r20426

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

Patch from Niles Ingalls to make patron phone number searching more matchy.
With assistance from Mike Rylander on pointing out the needed indexes to keep
search fast, and from Mike Peters on arranging a last minute DCO :-)

Type numbers into a patron phone number field and get a match regardless of
the punctuation actually used in the database.

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

Patch from Thomas Berezansky to do the following:
    Add more "click to copy" fields

    User's Name
    All three phone numbers

    Also fix issue where tooltip didn't clear when re-opening search form

Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20428 dcc99617-32d9-48b4-a31d-7c20da2025e4

Add a feature authored by Joscha Filius for the International Institue of
Social History: a diacritical palette for the MARC editor!

Afelonne Doek of IISH wants the community to benefit from this code.  To try
it out, press control S in the marc editor (reguarly non-flat-text view).

You can insert the characters you see, and you can right click on entries in
your palette to customize.  Customizations /should/ persist through staff client
restart (untested by me so far).

This should make cataloging in multiple (Western-ish) languages at once easier.

Below I copy and paste some of Joscha's message to the list where he introduced
this code:

> *Bugs:*
> A known bug is that is you switch marc edit styles back and forth the
> listener trigger won't work anymore. Also for a far as I know there is at
> least another bug concerning layout which is as far as I know not the result
> of my doing.
>
> *Here comes a possibly nasty part*, to ensure that the diacriticals remain
> intact throughout the persist I've changed global_util.js to URL encode and
> URL decode all stored values. This might cause backwards incompatibility
> with values that are already stored for other purposes without encoding.
> This needs to be checked.

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

Remove redundant plpgsql install step from README

PostgreSQL 9.0 includes plpgsql by default, and running the
createlang plpgsql step throws an error that could concern
the installer. Drop the step from the README.

Signed-off-by: Dan Scott <dan@coffeecode.net>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20424 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from James Fournie improving due-date-editor usability by auto-checking the check box; LP#70956965

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

Patch from Jason Boyer swapping the order of report and template name in emailed notices; LP#777273

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

Fix some Serial Control text boxes which were too small to use effectively.

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

Only pass the label to the CN browse search API, that is all we care about for the key

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

Collaboration patch from Joseph Lewis and Michael Peters addressing LP#758007, missing patron merge OU settings.

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

Addressing LP#732681 at upgrade time -- make authority records useful for controlling bibs

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

Initial fix from Michael Peters addressing small-screen UI issues: LP#767507

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

Change an ERROR to a DEBUG log message in mod_idlchunk

Reported by James Fournie at Sitka, this message was logged
as an error when it is simply a debug message; we drop the
level accordingly so that it will not appear in the Apache
logs during the normal course of operations.

Signed-off-by: Dan Scott <dan@coffeecode.net>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20398 dcc99617-32d9-48b4-a31d-7c20da2025e4

Patch from Michael Peters to protect cut-in-line holds from staff without the appropriate permission

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

Patch from Ben Ostrowsky addressing button order: LP#754660

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

Prevent uninitialized var warnings in OpenILS::WWW::SuperCat::unapi

Given a tag URI like tag:localhost,2011:biblio-record_entry/1 with
no format, Evergreen was throwing uninitialized variable warnings
and attempting to invoke "open-ils.supercat..formats" due to the
unitialized $type variable, which resulted in an Apache error.

On the TT OPAC, this enables Zotero to successfully offer import
of all entries on the search results page. On the AjaxPAC, it seems
that Zotero parses the page before the unAPI hrefs have been supplied
by JavaScript.

Signed-off-by: Dan Scott <dan@coffeecode.net>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20391 dcc99617-32d9-48b4-a31d-7c20da2025e4

Fix typo in fm_IDL.xml

"oils_persist" namespace was missing an "s" in one instance, preventing
autogen.sh from running.

Bug reported by Thomas Berezansky.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20389 dcc99617-32d9-48b4-a31d-7c20da2025e4

Avoid data loss by setting MARC::Charset->assume_unicode(1)

When using MARC::File::XML, MARC::Charset is used to perform character
conversions; however, MARC::File::XML does not tell MARC::Charset that it is
handling Unicode data. If we do not tell MARC::Charset that it is handling
Unicode data, it can return an error which results in the loss of data
(typically a subfield containing one or more characters which MARC::Charset
does not have an equivalent mapping outside of Unicode).

This problem could be reproduced in authority_control_fields.pl with a
subfield like "von Hans-Christian Müơller" - when this subfield was encountered
without assume_unicode(1), a null string was returned for that subfield, and
if the record was written back to the database due to an authority match being
found in a different field, the only recourse was to restore the record from
auditor.biblio_record_entry_history. The same sort of problems could occur
for any other script or function that modifies the data being handed to it
using MARC::File::XML and BinaryEncoding => UTF8.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_1@20386 dcc99617-32d9-48b4-a31d-7c20da2025e4