missing )

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX : bump to 056 was wrong when marcflavour = UNIMARC

The 652 field don't exist in UNIMARC. So linking the new enumchron to a 952 field automatically should be done only for marc21

For UNIMARC, i map it to 995$v, which has been introduced in the last release of the "recommandation 995"
adding the 995$v as well for installer

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2076 - add browser if necessary (DB rev 078)

Ensure that the browser table is added during an upgrade
from 2.2.9.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2076 - add authorised_values.imageurl

The column was added in updatedatabase.pl rev 075
but not added to kohastructure.sql.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2076 - remove three tables (DB rev 077)

Removed the categorytable, mediatypetable, and subcategorytable
tables.  These three tables appear to be from an incomplete
patch from pre-git that defined these tables to implement
additional search limits, and are not present in the 2.2.9
package.  In 3.0, these tables did nothing - the only code that
referred to them are incomplete admin editors that lack HTML
templates and are not referred to anywhere.

The updatedatabase.pl logic to remove these tables is conservative;
the tables are dropped only if they exist and are all empty.  Effectively,
this means that they'll be dropped if you've been running a fresh install
of 3.0 from git or the alphas or betas; if you've upgraded your database
from 2.2.9, these tables likely weren't in your database to begin with.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

added three tables to kohastructure.sql

tags_all, tags_approval, tags_index; for those tracking
git HEAD, these were already added via updatedatabase.pl
DB rev 073, so there's no DB bump for this patch.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Some minor markup changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding more details to cart's verbose view. Adding auto-print to cart print view (Addresses Bug 2070)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixing bug 1629: RSS Feeds of Search Results in OPAC

Changing label from Overlay? to Match? since overlay is no longer
the only operation

kohabug 2026 - HTML-escape comments

This is a partial, perhaps temporary fix.  "<", ">",
and "&" characters in patron comments (AKA reviews)
are converted to "&lt;", "&gt;", and "&amp;" to avoid
certain attacks, e.g., a user entering a <script> tag
in a comment.

A more permanent fix should scrub all (or perhaps just
unsafe) tags from submitted comments entirely.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

staging import - enhance record overlay behavior

Enhanced the ability of catalogers to specify how
bib and item records should be added, replaced, or
ignored during a staging import.

When an import batch of bib records is staged and commit,
the user can now explicitly specify what should occur
when an incoming bib record has, or does not have, a match
with a record already in the database.  The options are:

if match found (overlay_action):
  create_new (just add the incoming record)
  replace (replace the matched record with the incoming one)
  use_template (option not implemented)
  ignore (do nothing with the incoming bib; however, the
          items attached to it may still be processed
           based on the item action)

if no match is found (nomatch_action):
  create_new (just add the incoming record)
  ignore (do nothing with the incoming bib; in this
          case, any items attached to it will be
          ignored since there will be nothing to
          attach them to)

The following options for handling items embedded in the
bib record are now available:

  always_add (add the items to the new or replaced bib)
  add_only_if_match (add the items only if the incoming bib
                     matches an existing bib)
  add_only_if_add (add the items only if the incoming bib
                   does *not* match an existing bib)
  ignore (ignore the items entirely)

With these changes, it is now possible to support the following use cases:

[1] A library joining an existing Koha database wishes to add their
    items to existing bib records if they match, but does not want
    to overlay the bib records themselves.
[2] A library wants to load a file of records, but only handle
    the new ones, not ones that are already in the database.
[3] A library wants to load a file of records, but only
    handle the ones that match existing records (e.g., if
    the records are coming back from an authority control vendor).

Documentation changes:

* See description above; also, screenshots of the 'stage MARC records
for import' and 'manage staged MARC records' should be updated.

Test cases:

* Added test cases to exercise staging and committing import batches.

UI changes:

* The pages for staging and managing import batches now have
  controls for setting the overlay action, action if no match,
  and item action separately.
* in the manage import batch tool, user is notified when they
  change overlay action, no-match action, and item action
* HTML for manage import batch tool now uses fieldsets

Database changes (DB rev 076):

* added import_batches.item_action
* added import_batches.nomatch_action
* added 'ignore' as a valid value for import_batches.overlay_action
* added 'ignored' as a valid value for import_records.status
* added 'status' as a valid value for import_items.status

API changes:

* new accessor routines for C4::ImportBatch

    GetImportBatchNoMatchAction
    SetImportBatchNoMatchAction
    GetImportBatchItemAction
    SetImportBatchItemAction

* new internal functions for C4::ImportBatch to
  determine how a given bib and item are to be
  processed, based on overlay_action, nomatch_action,
  and item_action:

    _get_commit_action
    _get_revert_action

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

test framework - two improvements

[1] When running the database-dependent tests (cd t, make test),
    all tables in the test database are dropped prior to running
    the installer and test cases.  This means that the test
    database will start with a clean slate.
[2] It is now possible to specify a single test class to run,
    to avoid having to run all of them:

    cd t
    make test TEST_CLASS=Search

    To run all DB-dependent tests, just do the usual

    cd t
    make test

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

added reindexing routine to KohaTest

KohaTest->reindex_marc() does a full Zebra reindexing of
bib and authority records and clears zebraqueue.  It is
meant for use in test classes that do a lot of bib, authority,
and/or item record changes that would be slowed down by
waiting for zebraqueue_daemon.

Also adjusted list of tables that are automatically truncated.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding authorised value icons display in the search results
pages.

Bug 2047: adding images to arbitrary authorized values

I've refactored the subs I added in the previous commit so that they make a little
more sense and are in better places in the code base. I was really hoping to make use
of existing subs, but they all seemed so specific to particular uses.

The icons now show up on the OPAC item details page.

TODO: The icons still don't show up in the OPAC search results page.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug 2047: adding icons to authorized values

This patch adds the ability to associate icon images with authorized values.
They're not displayed anywhere yet, though. There's also a database update
in here. I took version "3.00.00.075", but I can change that later if it gets
taken before I commit these patches.

DOCUMENTATION CHANGES: When you add or edit authorized values, you can now
associate image icons with them. These icons come from the same icon sets
that you can have with your itemtypes, such as media type. They don't show
up anywhere yet, but that's coming. Keep an eye on
http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=2047
for more updates.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug 2047: refactoring icon manipulation logic

I'm extracting some of the icon manipulation logic so that I can get to it from the authorized values pages.

There should be no functionality or documentation changes with this commit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding Fedora installation documentation

Re-structuring shelf browser markup for improved robustness; Including GoogleJackets js only if pref is turned on.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixes for Bug 2067 (terminology consistency)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Making label toolbars more consistent

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX labels - Adding labels was failing because of faulty "type" selection.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixes for interface bugs 2064, 2065, 2066

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

More labels repair: FIXME's added, warns $debug-ified.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX - get_batches argument counterproductive. NONE of the Labels code

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX: $item is reference to hashref, not reference to hash.

In fact, the double reference is useless overkill for this and another arg.
This patch relieves the error seen in logs as:
label-print-pdf.pl: Not a HASH reference at /home/atz/koha/production/intranet/modules/C4/Labels.pm line 1007., referer: http://staff-atz.dev.kohalibrary.com/cgi-bin/koha/labels/label-manager.pl?type=labels

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add a new Debian package and GoogleJacket on OPAC detail page

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixed langauge typo

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixed langauge typo

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Minor change to improve compatibility with IE7

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Some fixes for bug Bug 2056, No Claim Letter Note Should Stand Out, and other markup corrections.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Changes to improve compatibility with Internet Explorer 6 and 7

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Removing duplicate NPL itemtype image icons from the staff client

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Another OPAC popup page requiring ID

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding a couple missing variables for opac login screen (opacheader,OpacUserCSS); Minor tweak to OPAC tags on opac-detail

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Normalizing s/On loan/Checked out/

bug 2058: removing spurious warnings from C4::Items::AddItem

just remove a couple unnecessary warn statements.
no documentation or functionality changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Split Location and Call Number on OPAC detail page

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add Debian packaged Image::Magick Perl module to Debian install

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

name error "opac-bottom.inc" in template

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Removing duplicate itemtype icons.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding show/hide function to add tag form; fixing bug in series search link (Thanks Tim!)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tweaking returns interface and jquery-ing inline javascript.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Enabling OPACURLOpenInNewWindow preference for "Search for this title in..." links

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for bug 2043 (Language Issues - OPAC)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for Bug 2042 (Language Issues - Administration (pt 2)). Also adding tablesorter.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Minor markup changes, validation corrections.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding IDs to popups to add flexibility to customization.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixing anonymous permissions access to zebra via Z39.50
and SRU

bug 2044: adding database change for itemtype icon change

This is the database update change to take into account the rearrangement of the
itemtype icons directory.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bug 2044: allowing multiuple icon sets

I've rearranged the two directories where media type icons live.
I've also added two more collections of icons, one from
http://apps.carleton.edu/campus/library/bridge_icons/
and one from liblime (Tina). The first has a license restriction that I added
to the "Licenses" tab on the "About" page.

Then, I've adjusted the Item Types Administration page so that it can deal
with multiple collections of icons.

I also added a test script to verify that the two identical icon directories
are actually identical.

DOCUMENTATION CHANGE: It's possible that we need to add something to the administration
documentation to indicate how you can add more sets of icons if you want. You simply add
directory to koha-tmpl/intranet-tmpl/prog/img/itemtypeimg/ and to
koha-tmpl/opac-tmpl/prog/itemtypeimg. The icons should start showing up.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Minor indexing bug : Title series indexed as Title

410$t is now indexed as Title-series UNIMARC

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

unlocking table when adding/updating authorities is finished.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add MARC21 (English) to fr-FR installation

With this addition, it's possible to install Koha in French (fr-FR) and
having the English MARC21 framework. It would be desirable if someone
(a French Canadian?) translate this into French.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix default value for TagsExternalDictionary to be null,

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Revised suggestions management interface, allows setting of one reason for all suggestions in a particular category (pending/accepted/rejected).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Some style changes following up on recent fix for Bug 2019. Hopefully improves readability. Converted up.png from 24-bit to 8-bit for IE6-friendliness (doesn't require alpha transparency).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Validation of opac results and detail pages, normalization
of XSL stylesheets and display

Improvements to the results display in the OPAC, merging style
between the XSL and non-XSL stuff. Refining style of tags display,
adding back links to place hold, add to list, and add to cart on
results page

Cleanup after merge of labels improvements

Adding TagsEnabled to list of known sysprefs, adding 'my tabs'
to the OPAC user portal

Nomenclature cleanup for supplemental issues in serials

Improving dialog for linking to a vendor, allow choice between
OK and Cancel for whether to link to a vendor rather than requiring
a vendor be added.

Nomenclature cleanup for subscription adding page

Revert "2 FixPriority calls were broken. FixPriority itself may be broken too,"

This reverts commit ea9e0804cd05ee0b0ff386afe28561a2d3cf23e0.

Moving Tags tab into Enhanced Content to reduce the number
of tabs in sysprefs

NOTE TO DOC TEAM: Moved FRBR, Amazon.com and Baker and Taylor
tabs in system preferences over to a new tab in sysprefs called
Enhanced Content

Remove duplicate "use C4::Context". Once is enough, thanks.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Hack RSS to work. Just a temporary fix before XSL takes over.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

2 FixPriority calls were broken. FixPriority itself may be broken too,

but we can't test that withtout calling it correctly.
Note FIXME for questionable use of @-.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Copy skin.css from intranet-tmpl, resolve 404 in OPAC.

We should consider using a single yui directory w/ apache redirects to it.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix #1624 - remove problem w/ EXPR in template, minor cleanup of module.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Pretty sure that "S" is needed. Significant typo.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove extraneous code. Note: this is another example of unchecked input, yet unfixed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove redundant zip file.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Cleanup code formatting for readability. No functional change.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

opac-search - CORRECTION: remove diagnostic Dumper lines

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

stats.screen.pl - cleanup, conditionalize warns w/ Debug

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

smart-rules.pl - tighten regexp, Debug for warn statements, readable layout.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

smart-rules.tmpl - cleanup display, remove units for zero/empty values

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

DB and syspref update for Tags and BakerTaylor

Feel free to apply this before the other related Tags/B&T patches.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tags - module, script and template support for user tagging in OPAC.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Baker and Taylor OPAC integration, plus some logic for Tags

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

C4::Search - just cleanup, 1 FIXME added

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

C4/External/BakerTaylor.pm - Back end for B&T content.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

opac.css - add styling for tag classes

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

opac-tags_subject.pl - touch up: this script is does not require Exporter.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BakerTaylor.pm - Initial implementation of B+T jacket cover and bookstore links.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tagging - add basic template slot in search for tags and script responsiveness to tagging sysprefs

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 1776 - try to locate Zebra during install

If running Zebra, try to locate zebrasrv and zebraidx
so that koha-zebra-ctl.sh can point to a functioning
zebrasrv.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

added RUN_DATABASE_TESTS to install log

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fixing version numbering from 71 to 72

Add dropbox mode to return.pl. Small API change to allow setting

returndate other than now().  Overdues will be handled in a forthcoming commit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Small template cleanup: display library name and cat description instead of codes

in circ-menu include.  Also, change 'Default' on main login page to 'My Library'.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

DB BUMP 71 : add formatstring to labels_conf for specifying labels content

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Labels update- print any marc field and add csv output.

Make any MARC data available to labels module;
Also add a csv output option, allowing export of relevant
biblio data do a 3rd party label layout application.
Note: This patch REQUIRES a forthcoming updatedatabase patch !

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Adding breadcrumbs (fixes Bug 2031, "No way back home from about page")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

More cleanup and corrections for circulation reports.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for Bug 2018 ("Incorrect focus")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>