Merge branch 'bug2505_patches' of git://git.catalyst.net.nz/koha into to-push

* 'bug2505_patches' of git://git.catalyst.net.nz/koha: (24 commits)
  Bug 2505 - use strict and warnings in sax_parser_test
  Bug 2505 - enable warnings for link_bibs_to_authorities
  Bug 2505 - add strict and warnings to perlmodule_ls
  Bug 2505 - add strict and warnings to check_sysprefs
  Bug 2505 - Add commented use warnings where missing in *.t
  Bug 2505 - Add commented use warnings where missing in *.pm
  Bug 2505 - Add commented use warnings where missing in the cataloguing/ directory
  Bug 2505 - Add commented use warnings where missing in the misc/ directory
  Bug 2505 - Add commented use warnings where missing in the tools/ directory
  Bug 2505 - Add commented use warnings where missing in the installer/ directory
  Bug 2505 - Add commented use warnings where missing in the rotating_collections/ directory
  Bug 2505 - Add commented use warnings where missing in the C4/ directory
  Bug 2505 - Add commented use warnings where missing in the serials/ directory
  Bug 2505 - Add commented use warnings where missing in the catalogue/ directory
  Bug 2505 - Add commented use warnings where missing in the sms/ directory
  Bug 2505 - Add commented use warnings where missing in the opac/ directory
  Bug 2505 - Add commented use warnings where missing in the virtualshelves/ directory
  Bug 2505 - Add commented use warnings where missing in the suggestion/ directory
  Bug 2505 - Add commented use warnings where missing in the admin/ directory
  Bug 2505 - Add commented use warnings where missing in the circ/ directory
  ...

Conflicts:
C4/Auth_with_cas.pm
acqui/supplier.pl

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

Fix for Bug 4302, shouldn't have to scroll left to right on z search results

Adds inline pop-up menu for each row of Z39.50 search results, similar to the
one recently added to the additem screen. This allows the user to click
anywhere in the row to display the menu and access the preview/import links.

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

removing warn logs

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Followup : d31ac1 reintroducing submit buttons

this reintroduces translation of submit values

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

unconditional warning changed to if $DEBUG

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

follow-up : remove perl error if the librarian choose no budget

the librarian can enter the suggestion without selecting any budget.
Searching the budget name in this case resulted in a Perl error

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3168 : Suggestion budget management

budget were not saved and not correctly displayed

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

(MT3174) acqui/neworderempty.pl: incrementing the quantity when adding an item

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

budget limit not working

on a new order, the branch limits for the budgets where not managed properly
if a budget had a limit on branch, it was not respected : the list displayed all budgets, including other branches than mine

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT3186: Remove warnings from Auth_with_cas

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

A fix for Bug 4208, Many submit buttons are not translatable in 3.2

Not a fix for a submit button problem, but for a problem with
TMPL_IFs nested inside tags, a practice which will break the
translation script.

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

(BUG #4388) aqbudgets.pl: add a column in the table for display the total spent of level and sublevels

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

(BUG #4360) aqbudgets.pl: Adding the hierarchy of budgets in the tooltip

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

Bug 4399 Untranslated link 'Export selected items data' in claims.tmpl

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

bug 4391 fix a series of minor text styles and

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

Adding new committer to the history

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

Fix a warning in log

Using a hash as a reference is deprecated

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

wr69607 - fixed conditions for display of tax details

Previously, if the tax rate for a particular seller was '0.000' it
couldn't be edited. Now, the tax information always is shown if there
is a default system one, or one for that seller.

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

wr69959 Pagination on neworderbiblio was missing baskteno, caused a server error when trying to add an order on any page other than page one

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

Fix ampersand problem in non-XSLT Intranet search results

Quick fix for ampersand display issue in non-xslt intranet search results,
related to bug 3326.

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

update news links to point to official koha site

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

When purging, remove /var/log/koha.

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

Remove generated /etc/koha/koha-conf.xml when package is purged.

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

Remove now-unnecessary debian-specific patch.

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

bug 4391 - fix minor typos

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

Bug 2505 - use strict and warnings in sax_parser_test

Bug 2505 - enable warnings for link_bibs_to_authorities

Bug 2505 - add strict and warnings to perlmodule_ls

Bug 2505 - add strict and warnings to check_sysprefs

Bug 2505 - Add commented use warnings where missing in *.t

Bug 2505 - Add commented use warnings where missing in *.pm

Bug 2505 - Add commented use warnings where missing in the cataloguing/ directory

Bug 2505 - Add commented use warnings where missing in the misc/ directory

Bug 2505 - Add commented use warnings where missing in the tools/ directory

Bug 2505 - Add commented use warnings where missing in the installer/ directory

Bug 2505 - Add commented use warnings where missing in the rotating_collections/ directory

Bug 2505 - Add commented use warnings where missing in the C4/ directory

Bug 2505 - Add commented use warnings where missing in the serials/ directory

Bug 2505 - Add commented use warnings where missing in the catalogue/ directory

Bug 2505 - Add commented use warnings where missing in the sms/ directory

Bug 2505 - Add commented use warnings where missing in the opac/ directory

Bug 2505 - Add commented use warnings where missing in the virtualshelves/ directory

Bug 2505 - Add commented use warnings where missing in the suggestion/ directory

Bug 2505 - Add commented use warnings where missing in the admin/ directory

Bug 2505 - Add commented use warnings where missing in the circ/ directory

Bug 2505 - Add commented use warnings where missing in the reports/ directory

Bug 2505 - Add commented use warnings where missing in the members/ directory

Bug 2505 - Add commented use warnings where missing in the acqui/ directory

Bug 2505 - Add commented use warnings where missing in the errors/ directory

Replace branch codes in on returns screen with branch names.

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

Bug 3293, changes to the xslt files to fix the wrong type/icon showing for music scores. Courtesy of Bernardo Gonzalez Kriegel

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

(bug #4318) remove the confirmation step in validating a form

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

(bug #4339) fix catalog stats

This fix the catalog stats to count items that have no barcode.

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

Allow zebra search for Accelerated Reading Point in field 526$d

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

Allow zebra search for Accelerated Reading Level in field 526$c

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

Allow UNIMARC item callnumber splitting

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

Bug 4382 Export batch label to CSV

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

History updated - 4 New developers !

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

Change #! line for cronjob to refer to sh using canonical path.

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

MT3160 : Fix plugin callback when subfields are cloned

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Replace old, non-functional Debian packaging with a new one.

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

Fix file permissions: if it is not a script, it should not be executable.

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

Bug 4368 - Avoid reseting UNIMARC 100 tag when importing biblio records

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

bug 4233 HidePatronName values swapped

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

bug 4378 Missing option prompt for IntranetBiblioDefaultView syspref

labeled_marc ioption was omitted (spelling police?)
options available now match those in code

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

Merge branch 'serials_mods' of git://github.com/colinsc/koha into to-push

Description by Colin Campbell:

A number of commits, subsequent to enabling warnings in C4::Serials and
eradicating errors and removing dead code.
These are extracted from an install branch running at UN FAO in Rome.
Commits have been rebased on current 3.2 HEAD.
These are not major new functionality, but  a series of fixes to
eradicate some randomness caused by undefined values. Includes a commit
to fix Licenses in the serials directory.

* 'serials_mods' of git://github.com/colinsc/koha:
  Check value is defined before splitting it
  Fix FSF Address in directory serials/
  Bug 4380 Use syspref to resolve which bib display is used
  Remove bottleneck making claims page unuseable
  Clean up Pod Documentation
  add warnings to Serials.pm
  Add Text to claim date column's header
  Basic refactoring of serials-edit.pm
  Remove unused or unnecessary variables in claims processing

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

Fix for Bug 4235, Date picker broken if placing multiple holds

Also includes minor style change.

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

Fix for Bug 4262, changing matching rule add big message box

Fix includes a change to the format of the page to consolidate
matching rule settings with matching rule controls. This
reduces potential confusion from the placement of the message
box. Assuming that matching rule settings and controls were
displayed separately for clarity and control, I've added
JavaScript which allows the user to change their mind after
changing a selection (but before submitting). It's a new
interaction which I hope is useful.

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

Fix for Bug 4121, Cataloging record modification should obey IntranetBiblioDefaultView pref

- Re-used redirect logic found in search.pl

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

Fix for Bug 4251, click on show more in facet drops limit=branch in query

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

Fix for Bug 4251 in the staff client

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

Fix for Bug 4250, cart missing print button

- Print button was hidden by CSS for some reason (git blame me)
- Print formatting was off because of column mismatch
- Table style modified in print view for clarity
- Redirect from print view corrected by adding missing bib_list
  and verbose parameters

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

Fix for Bug 4214, "Your Cart is Currently Empty" popup displays underneath News header

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

Adding Calyx to the about page and the history

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

Check value is defined before splitting it

splitting an undef generates a warning

Fix FSF Address in directory serials/

Add License statement to those files lacking it

Bug 4380 Use syspref to resolve which bib display is used

Be consistent with elsewhere in cataloguing
When switching there from serials

TODO Duplicates Code Should be in a common routine

Remove bottleneck making claims page unuseable

While the idea of showing the number of late serials against the
vendor name was nice it does not scale and on large sites selecting claims
was just timing out. Improved the speed of the initial query but have removed
the big query for each user just to get a count.
Check for 0000-00-00 dates so that C4::Dates does not log error
Removed a variable that was never set and the bit of template used
if the impossible happened

Clean up Pod Documentation

Ensured returns were documented
Fixed some language issues
Removed the host of podchecker warnings about missing lists
added a couple of explicit returns when subroutines
'fell off the end' to remove any ambiguity

add warnings to Serials.pm

Change obvious warning generators:
     ( use of string comparisons in numeric comparison)
     ( declaration of variable in comnditional )
also some errors caused by undefined values:
    abouttoexpire was not checking for undef values
    Pass a valid planneddate in generation of next expected
    (undef here causes odd results)
Add a basic test script
    test is minimal but I fell over a bug this would have caught

Add Text to claim date column's header

Basic refactoring of serials-edit.pm

Removed some useless counts from serials-edit / Serials interfaces
Removed old commented out code. Unrequired variables
Reformatted some code so that improving logic can be done
more easily

Remove unused or unnecessary variables in claims processing

Also cleaned the interface to the claims related functions
in C4::Serials so they do not return an extra count variable
moved generation of dropdown to template instead of inline code

Bug 2505 - remove unnecessary -w and replace with use warnings;

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

MT2663 : Adds an itemcallnumber plugin

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

MT2502 : Reports Editor's 200$b to biblio's 210$c within unimarc_field_210c plugin

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Added plugin to automatically add brackets around the value in marc21 field 245h

This plugin will automatically add brackets around the value typed in at the time the field's focus is removed ( onblur ).
If the text in the field is already enclosed in brackets, it will do nothing.

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

Bug 4270 Instructions for loading patron attributes are wrong

Corrects the on-screen instructions for loading multiple patron attribute fields

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

Hide damaged loop in item form if empty

Follow-up : removing itemlost if empty as well

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

Hide damaged loop in item form if empty

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Removed the language limit displaying twice in the OPAC advanced search

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

Fix for Bug 4167, Tag link styled incorrectly on Cart popup

- Removing DOM creation of tag and hold links: if the Cart is working,
  the user must have JavaScript enabled. Therefore we can assume
  tagging and holds will work
- Conditionally displaying some JavaScript based on whether tagging
  or placing holds is allowed

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

bug 4294 use 'required' class on required fields on funds page

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

Fix for acquisitions - Series title was not saving when adding new record

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

(BUG #4346) replaces Basket grouping by Basket groups in acquisitions-toolbar.inc

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

Refactor for Human Readability

Fixes  generation of empty index page when total_parcels is a multiple of pagesize
Remove some calculations whose results were not used
Make variable names meaningful (results and count arnt)
Fix errors in pod syntax
Make what code does clearer to readers who are not computers
Use a subroutine

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

Bug 4331 Remove cruft from acqui-home script

Remove unnecessary db calls and processing from acqui-home.pl
Do not duplicate processing of fields
Declare variables closer to use
Remove unnecessary holding vars
Replace some anonymous var names by something more meaningful
Do not generate a toggle variable use Template's LOOP vars

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

Bug 4331 Index aqorders by budget_id

Calculation of budget spent and encumbered is done by
queries on aqorders by budget_id
Index budget_id to speed these queries

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

Fix for ampersand display problem in OPAC search results

This is a quick fix to XSLTParse4Display to make ampersands display
correctly in OPAC search results.

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

MT3204 : Fix unstranslatable strings in JavaScript

For an explanation, see:

http://wiki.koha.org/doku.php?id=en:development:codingguidelines#translatable_text

This bug blocks entirely serials module. It isn't possible
to create/edit a subscription. Brocken JavaScript prevents
Search for a vendor and Search for Biblio dialog box
to pop-up.

Problem with new strings and new html tags inside the strings
this patch removes them

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>