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>

Fix Bug 5261 broken link in Local Use sysprefs

This patch eliminates the sysprefs-menu.inc include file and changes the
systempreferences.pl and systempreferences.tmpl files to work with the
prefs-menu.inc instead. This will centralize the syspref tabs and make it
easier to modify tabs in the future if necessary.

This commit also changes the default tab to Acquisitions, since the Local Use
tab does not work with preferences.pl at present.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Add 3 more links to Ohloh to about page

- Markus Enger (asked for permission)
- Jared Camins-Esakov (asked for permission)
- Katrin Fischer

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

Bug 5194 - Patron notification on new issues not working

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

Fix for syntax error in the spanish opac file

Bug 4867 - cannot clone subfields in marc editor, other clone/unclone weirdness

When the clone/unclone plus/minus were replaced with images the <span> tags were inadvertantly deleted
causing the js to loose its ability to identify the clone/unclone links and properly udpate the indexes.
The resulting incorrect indexes caused various weirdnesses to result when one attempted to clone/unclone
subfields.

This patch simply adds classes to the two links and updates the js to identify the links based on class.

Thanks to owen for his help in chasing this down.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Language updates

Bug 5191 Patron notification was being reset to none

The value of the letter variable was not being correctly passed
to the edit template instead it was reset to none
Also compressed some verbage from letter_loop generation
(if you want a scalar use one)
removed an unused $count variable (?!)
and put a couple of lone )s back on the line they came from

Resend to remove wrong utf-9 header.

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

More language updates

Bug 4310 Map orders to new budget structure

Map orders to newly created budgets
creates a temporary file fundmapping
which is not dropped so that conversion can be
checked subsequently

As acquisitions historically let you do some odd things
with dates and funds. This probably needs close checking
if you are converting active data

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

Bug 5205 Fix compile time warnings in tools/batchMod.pl

$dbh was declared twice
prototypes used in a subroutine that was not declared before use

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

bug 4084: remove \r from header line for sake of consistency

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

(bug #4084) fix offline circ

this add the function C4::Circulation::GetOpenIssue and delete \r chars when processing koc files.

Cherry picked from 3.0.x commit 5af80ab

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

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

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

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

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

Bug 5249 - Guide boxes do not print when selected for a layout

Bug 5247 - Text does not wrap properly on labels

The apparent fix is to have the text wrap alogrithm to calculate based
on a label width reduced by approx. 3/16" when there is no left text margin.

This is probably not the proper fix as it appears that this algorithm is being
affected by adjustments in the label width for label creep and perhaps offset
as well. But this does fix the symptom and is all the time available for this
problem at present.

Documentation Changes:

It might be good to add a note to the effect that if the user does not supply
a left text margin in the layout, a 3/16" (13.5 point) left text margin will
apply by default.

Bug 5203 - Creating a label template causes fatal error if description field is empty

This is due to the underlying script not having any defaults hard coded which results
in skewed hash elements when the parameter array is coerced into a hash at a later point.

This also occurs in layouts and profiles. This patch adds defaults to all three.

No documentation changes are necessary.

Bug 5183 Incorrect hash dereferencing syntax causes an error to be thrown when attempting to export patron cards

That says it all.

No documentation changes, etc. needed

Bug 5182 - Attempting to export a patron card with no text causes an error to be thrown

Can't use an undefined value as an ARRAY reference at /home/koha/koha.prod/C4/Patroncards/Patroncard.pm line 86.

This is caused by testing for text at line 87 rather than line 85. This patch moves the test to line 85.

Bug 5215 - Barcode field not cleared when using "Add and Duplicate" option on additem.pl

The barcode field is not cleared/incremented properly when using the "Add and
Duplicate" option. Rather it is pre-populated with the previously used barcode
which causes additem.pl to return a duplicate barcode error if the cataloger
does not catch the mistake.

The problem is caused by the fact that the current item record is simply
duplicated and re-loaded into the form.

The solution is to add code which either pre-populates with the next barcode
number if the incremental pattern is set or clears the p subfield if one of the
various patterns using the javascript plugin is selected.

In reality C4::Barcodes should be implimented here along with all necessary changes elsewhere.

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

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

Lots of lanugage updates

Finished EN-NZ

English NZ and Portuguese updates

German, English NZ, and Romanian updates

Language updates for intranet

Danish and NZ english updates

Updating files for 3.2

Preference updates

Tidying up and english nz updates

English intranet updates

Finished the en-NZ system preferences

Continuing updates for 3.2

Greek and NZ updates

Greek and NZ english updates

More spam removal

More spam tidy ups

Spam removal

Updates and spam removal

Removing spam from Amharic

Removing spam from albanian

Amharic; Ethiopian updates getting ready for 3.2

Intranet .po updates

Lots of opac .po file updates

Language updates from pootle

Updating german prefs

Language updates from pootle

Bug 5046 - fix the zebra authorities configuration

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

Merge branch 'origin/new/bug3756_reenable_local_sysprefs' of git://git.tamil.fr/git/koha into to-push

* 'origin/new/bug3756_reenable_local_sysprefs' of git://git.tamil.fr/git/koha:
  Bug 3756 Don't display as local AutoSelfCheckID syspref and others

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

Bug 3756 Don't display as local AutoSelfCheckID syspref and others

Improve how sysprefs are extracted from .tmpl file in order not to miss
some of them.

bug 4989: set DB rev to 145

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

(bug 4989) add index on guarantorid

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

Fix for Bug 4996 - Untranslatable strings in budget/fund toolbar

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

Fix typo in z3950 search in acquisitions

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

Bug 5023 Remove unused Serials Modules

Modules possibly added in error
Neither is called elsewhere in the system

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

Follow-up fix for Bug 5013, "New patron" menu button broken

Menu was working on members-home.pl but not on members.pl

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

bug 5007: fixing the syspref for IntranetBiblioDefaultView

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

Bug 5034 - update the maintainer details, correctly

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

Correcting URL typo in link back to labels home page

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

Bug 5021 Formatting of undefined serials dates

Routine was checking undefinedness but such dates were
normalized on retrieval to 0000-00-00
Also planned/received date should also be checked

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

Fix for Bug 4999, OPAC error pages have incorrect markup structure

No strings have been harmed in the making of this patch.

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

Minor markup correction to send Cart form

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

Markup corrections

- Changing input "button" to "submit" so that confirmation
  dialog redirects the user as expected
- Changing non-standard message box styles to standard ones

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

Bug 3756 Difference between original and local sysprefs

Amend Chris N. patch in order to display local sysprefs as the
difference between DB sysprefs and .pref files sysprefs.

As a side effect, local sysprefs editor displays also deprecated
sysprefs which is a feature from my point of view. For example,
GranularPermissions is displayed as a local syspref because it has been
removed recently from admin.pref but hasn't been yet removed from DB
(could be done later with a DB update).

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

Bug 3756: Adding the ability to have localuse systempreferences

Squashed commit of the following:

commit d42198bfadff20f40c38b3d9bc7f210ecc372e7c
Author: Chris Nighswonger <cnighswonger@foundations.edu>
Date:   Mon Jul 12 16:48:16 2010 -0400

    Bug 3756 [4/?] new sys prefs - no way to add a new local use preference

        Fixing Global Administration link

commit 060a52bc8d9da1d83387455967ca05292e5987b9
Author: Chris Nighswonger <cnighswonger@foundations.edu>
Date:   Mon Jul 12 16:45:07 2010 -0400

    Bug 3756 [3/?] new sys prefs - no way to add a new local use preference

        Hiding non-local sysprefs

commit d63858e13037fa91f18bc248ab770d757e31153c
Author: Chris Nighswonger <cnighswonger@foundations.edu>
Date:   Mon Jul 12 16:44:40 2010 -0400

    Bug 3756 [2/?]new sys prefs - no way to add a new local use preference

        Fixing a typo

commit 5b23a615018e049b1e11dbc10b131ecb30dfc6ab
Author: Chris Nighswonger <cnighswonger@foundations.edu>
Date:   Mon Jul 12 15:52:04 2010 -0400

    Bug 3756 new sys prefs - no way to add a new local use preference

        A quick and dirty fix

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

Fix for Bug 5013 "New patron" menu button broken on patron select screen

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

Bug 5011 install debian graphicsmagick not imagemagick

Scripts in install-misc were installing Image Magick
not Graphics::Magick the actual Koha dependency

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

wr72054 fixing the z3950 search for acquisitions

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

Bug 2505: Quieting warns in acqui/z3950_search.pl

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

Bugfix 3419: fixes typo in inequality.

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

Fixes Bug 4978: longoverdues.pl claims undefined subroutine ModItem

Adds an explicit C4::Items:: prefix to the ModItem call on line 323

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

Fix for Bug 4484, Rss tag is not escaped in search result

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

newordersuggestion would not filter by status

All the suggestions would be displayed without any consideration of their status.

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