Bug 4305 - prevent errors when handling invalid ISBN13 values

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge branch 'translation' of git://git.workbuffer.org/git/koha into to-push

Translations

Bug 4472 Fix &nbsp; handling for UNIMARC intranet XSL

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 3164: fix OPAC holds in SingleBranchMode

Account for case where no branch is explicitly supplied.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Add one new speech, bugfix a few other history lines

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(MT #3697) Fixed GetAuthorityRecords

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

[MT3696] Fixed ILSDI GetAvailability call
(cherry picked from commit 96552d64727dc6fc2525cc9c0c6ece0c718b999d)

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(MT #3698) Fixed ILSDI GetRecords
(cherry picked from commit f69061357a75a5e024fe6379a1a48f8755a2b6db)

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

remove scratch files from the it-IT SQL files

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

add new lenny dependency libmodule-install-perl to the apt-get-debian-lenny.sh script

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

add Italian installer SQL to test cases

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Italian installer SQL

Created by Zeno Tajoli <tajoli@cilea.it>.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix step to install Indexdata apt key

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Updates to INSTALL.debian documentation for 3.2.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Adding a temporary workaround to keep template tests from borking on very large templates

NOTE: This should be properly fixed at some point in 3.4

This may help:

http://www.perlmonks.org/?node_id=150748

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5288 - Update package dependencies, control source details

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

mark 3.2.0 release candidate - DBrev 3.01.00.999

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

defer rotating collections

Disabling rotating collections for 3.2.0 release; after more testing,
candidate for 3.4 as well as backport into 3.2.1.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

translator hack to avoid failure on opac-opensearch.tmpl

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

further correction to the invocation of unzip

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Tighten system call to unzip

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

add explicit type to <input> elements missing it

Quells a warning from xt/author/translatable-templates.t

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 4472: in a hackish fashion, quiet warnings about DOCTYPE in the XSLT

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge branch 'translation' of git://git.workbuffer.org/git/koha into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Updated spanish files

Another fix for Bug 2704, 440 Display Issues

Removing the check for 490 @ind1=0 since the
alternative 8xx tag will not display anyway

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge remote branch 'koha-fbc/k_bug_5284' into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5284 - Generic report covering various problems with the labels/patron card tests

This patch removes a very incomplete test script which was really not even formed into
a test routine.

Note: In all probability, the labels tests could be adapted to test patroncards as well.

Merge remote branch 'koha-fbc/k_bug_5284' into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5284 - Generic report covering various problems with the labels/patron card tests

This patch fixes the C4::Labels::Template tests

Bug 5284 - Generic report covering various problems with the labels/patron card tests

This patch fixes the C4::Labels::Profile tests

Bug 4472 Fix how &nbsp; are injected into XSL files

tmpl_process is patched to handle properly specific XML directives.
UNIMARC XSL files are modified to gain knowledge of HTML &nbsp; entity
which isn't the case by default. It may be necessary to do the same
thing for MARC21 XSL.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5284 - Generic report covering various problems with the labels/patron card tests

This patch fixes the C4::Labels::Layout tests

Followup to Bug 4305: multiple ISBNS in biblioitems.isbn breaks patron

This patch adds a scrub for ' | ' when the ISBN comes from the
biblioitems table, in the event that Koha has attempted to store
multiple ISBNs in the field.  The ISBN presented to Business::ISBN will
now be something it can handle.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix invalid invocation of GetBranches in test case

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5284 - Generic report covering various problems with the labels/patron card tests

This patch fixes the C4::Labels::Batch tests

Bug 5013: Advanced patron search category pull down broken

The pull down for category in patron's advanced search was only showing any and no patron categories.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

more test case fixes

SQLHelper.t - clean up after itself better
Context.t - fix reporting of test plan

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

cleanup of DB-dependent fund test cases

* remove test that was adding a second active budget period,
  which was throwing off later tests
* updated to reflect current API

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix typo/thinko in list test

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

various improvements to t/db_dependent/VirtualShelves.t

* updated to reflect recent (and old) changes to the API
* corrected test plan
* adopted Test::More
* correctly handle and test functionality to not add duplicate
  bibs to a list

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

removed AddToShelfFromBiblio from C4::VirtualShelves

Back in the day, lists could contain item records instead or in addition
to bibs, but that hasn't been the case for a while.  Therefore, removed
AddToShelfFromBiblio, which does exactly the same thing that AddToShelf does.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix test plan and false test errors in Search.t

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fix test plan for SQLHelper.t

Also removed incorrect comments about test case numbers - we should
just let Test::More number them.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 2377 - Review tags: format date according to system preference

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge branch 'test_cases' of /home/koha/koha1/dev into to-push

fix typo in INSTALL.debian

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

use Test::More for t/Koha.t

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5100, Incorrect link to System Preferences

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5156 - JavaScript error when adding list

JavaScript referenced nonexistent form name

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

fixes bug 4385

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5264 (Retain additional bibliographic subfields when merging authorities) [for master only]

The current code in AuthoritiesMarc.pm rewrites all subfields from the report
tag in the authority record to fields in related bibliographic records when
merging authorities. Additional subfields in the biblio records (e.g. relator
code or term) are lost in the process.

This patch retains the original algorithm but restores the additional subfields
from the original biblio record when rewriting the linked marc fields.

The problem also exists in 3.0. Two separate patches have been submitted.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixing non placeholder sql in UpdateGuarantees

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5089: Add German translation of borrowerRelationship and BorrowersTitles values

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5093 - JavaScript error in IE8 when choosing authority record

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5092: No results when searching by LC Call Number: in Z39.50 Server

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 2947, value_builder scripts using authnotrequired

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 3630 Impossible to perform Scan Indexes search

This is a quick fix. The way scan indexes performs should be improved in
3.4. There are several issues:

  - No paging
  - The interface is the same as for biblio records search result and so
    is unusable: for example you have a button to place a hold or you
    can sort by Popularity which is irrelevant for index terms.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Avoid to add a warning in log

... each time GetMemberDetails function is called

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4849 - Cannot delete records from Labeled MARC view

The template checks for a "count" variable (item count) to see
whether it should allow deletion of the biblio record. For some
reason labeledMARCdetail.pl hard-coded this as 1, which meant
the template always thought was 1 item.

This patch also adds an item count check on the display of
"Delete all items," disabling it if there are no items.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5054 - allow the resulting .deb location to be specified

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5139 - packages now do email and other cron jobs

By default the packages now set up the cron jobs to handle things like
overdues and email etc. By default, email is off, 'koha-email-enable'
and 'koha-email-disable' can manage this.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5116 – Duplicate and missing entries in "apt-get-debian-lenny.sh"

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5116

In the Debian installer shell script for Debian Lenny -
"apt-get-debian-lenny.sh"  - there were 2 missing lines for packages
that exist for Debian Lenny and that have versions that satisfy the
requirements of the Koha Installer, listed in the "PerlDependencies.pm"
file - located at "C4/Installer/PerlDependencies.pm".

The two missing lines are for "libnet-server-perl" (for the
"Net::Server" module) and "libpdf-table-perl" (for the "PDF::Table"
module).

I also noticed, in that same "apt-get-debian-lenny.sh" script, that
there were some duplicate (repeated) entries / lines for other modules,
namely: libdate-calc-perl, libdbd-mysql-perl, libemail-date-perl,
libgd-barcode-perl, libhtml-scrubber-perl, liblingua-stem-perl,
libmail-sendmail-perl, libmime-lite-perl, libnet-ldap-perl
and libpoe-perl.

So I'm adding, in this patch, the 2 missing lines
and removing the duplicate lines / entries.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Update VOKAL icon set

The Green Mountain Library Consortium has updated the 'vokal' item type
image set to include new and updated icons, including:
Puzzles and Games
BLURAY
Hardware
Educational Media
Fiction, Nonfiction, New and Holiday books
ILL materials

Smaller versions of all the icons (32 px) are included as well.

Sorting of the icons has been included in this patch to group
large and small versions of these icons together, as well as make large
icon sets easier to visually scan (the 'bridge' set benefits from this,
as well) .

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 4922 - Item display tries to show serial info for titles with series tags

Removing check for "is this a serial record" when displaying enumchron
information. This check evaluates as true for all biblio records which
have a series title, so it's an inaccurate determiner of whether
enumchron information should be displayed. The script already
checks for the presence of enumchron data, so that will display
anyway if it exists.

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Update VOKAL icon set

follow up patch to Ian's August 20th patch.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5217 : package/CPAN updates for Ubuntu Lucid install

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5207 - Details on editing SAX Parser ini file.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5200 - ubuntu.packages file installs CVS Removed 'install cvs' line from ubuntu.packages

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Removing outdated note about z3950search in the help file, no longer applicable

Signed-off-by: Robin Sheat <robin@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5077 - ensure rebuild_zebra will run somewhere it can read

This prevents it leaving files lying around in /tmp

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Shifting the tests that need user input to their own directory, they cant be used by the automated tester

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixing the VirtualShelves test

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixing C4::Boolean tests, in August 2009 we changed the die to a warn, tests didn't reflect this. Now we should have 100% of C4::Boolean

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5032 Remove duplicate code

Two duplicate calls to GetSerials appear to have been merged in
to no purpose other than to generate warnings

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5047 html option not in -help output

Add -html option to list in pod so that it shows up
in output of help

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5160 Stop words should be stop words not strings

Stop words in the default zebra config were being defined as
initial strings not as words causing then to truncate legitimate
headings.

This patch corrects that behaviour. It does not address the
question of what should be in the default file

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug 5086: fix setting claim date

Also removed a locus for SQL injection.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5101, Creators tab isn't highlighted when selected

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge remote branch 'koha-fbc/k_bug_5215' into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge remote branch 'koha-fbc/k_bug_5247' into to-push

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge remote branch 'koha-fbc/k_bug_5249' into to-push

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

bug_5245 Fix SQL syntax in GetItems

Routine never executed query due to syntax error
and returned undef
made the call to fetchrow_array explicit as the old fetchrow alias
is no longer documented in perldoc DBI

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Merge remote branch 'koha-fbc/k_bug_5203' into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fixes bug 5231: bad call to GetAuthType in BiblioAddAuthorities()

If systempreference BiblioAddsAuthorities is on, this could lead to error messages when trying to add records
to a basket from an external source.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix compile time warning in member.pl

variable to declared twice

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Missed adding the beta release

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Updating history doc

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5090: New order from empty record does not save publication year and ISBN

- make publication year save for MARC21
- make ISBN save
- fix small template issue with label for vendor note

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Revert "Bug 5081 - let the ISBN be saved on new manual orders"

Reverting in favor of a more comprehensive patch.

This reverts commit 4bd749251893e680a166b7cf31cd6e0d24b443fc.

Merge branch 'translation' of git://git.workbuffer.org/git/koha into master

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug: 4904: Problem with printing fines in overdue notices

my $formatted_fine = currency_format("$1", "$fine", FMT_SYMBOL);
is already utf-8.

Resend with additional change, removing 2 lines (no strict, use strict) from the code.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 2664: Add item should preselect logged in branch

An earlier fix to stop branch values being overwritten in
edit items was causing the branch value to be unset when adding items

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5265, error message on lists is misleading

The check to see if an item exists always evaluates as true.
Correcting this lets the script handle the error as it
was designed to.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5081 - let the ISBN be saved on new manual orders

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5201 Patron surname dropped

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Fix for Bug 5222 - 600 t not showing with xslt on

Original commit message was for 650 field, but fix works for 600.  See bug report.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 4305 Get Amazon book covers for ISBN13

The way Koha get book cover from Amazon doesn't work from biblio records
having ISBN13 because Amazon expect an ISBN10. This patch transforms
ISBN13 into ISBN10 before sending it to Amazon.

Warning! On some devel installations, you will have to install by hand a
new CPAN depency:

  Business::ISBN

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Bug 5187: Show place of publication (MARC21, 260$c), XSLT

Show place of puplication in
- OPAC result list
- OPAC detail
- staff detail

The normal display does already show the place of publication, but it's missing from the XSLT controlled displays.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>