overdue_notices.pl various bugs in HEAD 3.2

Those bugs must have been introduced by merge?

- Overdue to all libraries with overdue rules doesn't work
- Overdue to a specific library doesn't work also

Adding inline edit links for additem table. Suggested for Bug 2534

Correcting array element reference syntax which caused unnecessary warns

Also adding conditional to check ENV VAR definition to avoid unnecssary warns.

BZ4026 : Fix record's leader when merging records

updated Tools help files

BZ 4028 : OPAC search history available to non-logged-in users

The "search history" link and the "delete history" link are now correctly displayed after the first search.
Moreover, an additional "delete history" link as been added on the "search history" page itself.

Bug 3945 NULL is not a valid Supplier id

Get SuppliersWithLateIssues should return a list of suppliers
If you have subscriptions with no supplier it also returned a
invalid supplier with a null id and null name.
This patch ignores the non-supplier supplier

Bug 766 Remove scrolling list from serials claims

Let templating system do what its designed for
pass an array ref with the suppliers

Fix some code issues in circulation/returns

Fix obvious warning generators
 use of string comparison on numeric values
 use of capture variables without testing comparison
 reuse of variable names in same lexical scope
Tidy some layout issues
 remove commented out code
 remove unused variables
 remove tabs from mixed space tab layouts
 rewrite a couple of expressions where code flow obscured

Bug 3764: Made city/town admin form clear

Bug 3654: Resubmit: Fixes issues with authorized values

#1 Moves Authorized Values under Basic on the Admin page & menu
 #2 Add defaults for PROC & CART in LOC
 #3 Add MANUAL_INV default
 #4 Add BOR_Notes default
 #5 Fix descriptions and language on authorized values page related
    to these changes

Interface corrections and improvements to OPAC lists

- improving behavior for users with no javascript
- style improvements to list "toolbar"
- adding new toolbar icons
- adding display of subtitle to lists

Moving some list functions into the standard toolbar.

- Adding download functions to menu button
- Some markup corrections

BZ4019 : Lists Merge Issue

Removes deprecated virtualshelves/merge.pl as well as the link to it.
Records merging is done by selecting two records from a list's content.

(bug #4018) fix coins support in opac detail and xslt

bug 4017 fixed delete label to read just delete

Fix for Bug 4008, footer floating up next to purchase suggestion in OPAC

bug 3765: move road type after address

I have fixed 2 road type issues.

changing title of help back to Circulation Statistics for consistency

Bug 3096: re-adding note field to suggestion management.

Fix for Bug 4005, Inventory Tool w/ Blank Options

- Using some TMPL_IF logic to hide empty lines
 - Other markup and layout tweaks

bug 4011 remove blank row on stats form

bug 4009 & 4010 add required fields

Bug 4009: New list w/out title fails silently
  Fixed so Title is required when adding a list

Bug 4010:  Purchase Suggestions Should Require Title
  Fixed text so that it indicates that title is required
  Added check to make sure Title is filled in

For both bugs I added a required style to the CSS

Fix for Bug 2743, Z39.50, not z39.50

bug 4006: change categorycode to Patron Category

For usability reasons, I have changed the categorycode label to read 'Patron Cateory' on the Patron page.

Fix for Bug 3991, Circulation search submit button wrapping in WebKit browsers

If CircAutocompl is ON, the circ search form has a hidden <div> for use in
displaying live search results. For some reason WebKit browsers (Safari and
Chrome) treat this hidden element as a something that should create a line
break like any other block-level element. Moving the submit button to
before that hidden <div> fixes the problem without disrupting the display
of the live search results.

Authtypecode return an warning even if subfield is ignored

this patch just add a condition to the sql request to return a warning
only if authtypecode is not in an ignored subfield.

Fixing typo in the ils-di page

Updates and clarifies the issues_stats (Checkout Statistics) help file. This patch created by Sharon Moreland (smoreland@nekls.org)

Bugs: 3965, 3979, 3981 - GST changes in acquisitions

This patch fixes three bugs:

Bug 3965: gist sys pref not taken into account
 changed it so that GST does not show on basket if gist pref set to 0
 changed it so that GST fields don't appear on vendor record if gist pref=0

Bug 3979: gst options not lining up
 fixed layout on vendor add/edit form so GST radio buttons line up right

Bug 3981: GST confusing in en-US
 there is no GST in the US and so GST changed to 'Tax' in templates

Also, a small spacing fix was made to the path on the basket grouping
template.

bug 3978 fix column display on vendor search results

bug 3875: fix language inconsistency on lists

Bugs 3854 & 3855 Minor title and text fixes

bug 3845 fix minor capitalization and spelling errors on suggestion page

Bug 3839: update admin menus to include links to both new acq admin areas

Bug 3833: additional patch to fix spelling error introduces by first patch

replacement for bug 3817 added patron permissions for tag tool

Bug 3835: Add link to CSV Profiles on Tools menu

Bug 3833: Fix titles for Batchmod files

Bug 3830: Fix spelling error in patron permissions

Bug 3809: minor language change on import patrons tool

Adding pl-PL directory to installer/data/mysql.

Fix for Bug 3271, Missing message when adding to a list with no items selected

The shelf js was trying to access a variable set by the Cart-related js. With
the cart disabled, the variable was missing. The patch sets this variable
for cases when the Cart is turned off.

Fix for Bug 3229, The "Sort By" feature for lists does not work.

White space in table cells can often break the javascript tablesorter.
The previous patch to tie the js tablesorter to the shelf sort
parameters should work now.

Bug 2662: make rand password consider minPasswordLength pref

Update Debian Lenny installation procedure

- Install Debian Lenny packages required by Koha directly with apt-get
  rather than with dselect.
- Update documentation about Zebra indexer configuration via
  rebuild_zebra.pl rather than zebraqueue_daemon.pl script.

Bug 3952 changed checkout stats to read circulation stats

bug 3834 put names on patron page last, first

Fix for Bug 3944, new order template formatting and markup corrections

Fix for Bug 3940, "Notes on add patron form need formatting"

Fix for Bug 3687, Empty "Add to" with cart off and non-logged-in user.

Markup, breadcrumbs, and language corrections.

Also adding a link back to smart-rules.pl from clone-rules.pl results page.

Bug 3930 link to biblio was missing in manage staged marc import

Fix for Bug 3926 (suggestion edit form combines data)

Breaking patron information variable into separate surname, firstname, branch, category, and borrowernumber variables and using those variables to tweak the display.

(bug #3925) fix build_holds_queue.pl

This fix sql query when item-level_itype is set to "off".

Remove diagnostic output from t/Members_Attributes.t

While this might be useful for developers, is mostly pointless
otherwise. Should be replaced with a true test of these four
functions.

Bug 3890: Nothing in Type Pull Down on Purchase Suggestions.

This patch adds a description of the itype next to the image that i suppose to appear in the selection list.

Fixed the path of the image in opac-suggestions.pl, but I'm not sure if you can actually display an image in a <SELECT> list option.

Also removed an extra </label> closing tag.

Bug 3903 - Members names not showing on serial routing list

bug 3894 moved pull down to the proper place in the form

bug 3899 moved serial enumeration to the proper subfield

Bug 3909 don't show basket link if no basket exists

Puts focus in search box on page load.

Removed tab indexes in additem.pl to enable tabbing to buttons in order. Previously, tabbing through this form would skip the submit buttons.

Fixing bug with CanBookBeReserved ... rules for default borrower codes were being ignored

Removing global database handler, breaking tests and dangerous if we ever use mod_perl - All tests now passing

C4::Bookfund no longer exists, removing the test

C4::Boolean has had its' exception changed to warns, the test was erroring falsely

Bugfix for various bugs in mandatory and sample data for webinstaller

NOTE: This patch removes the sample budgets data which was obsoleted by the new
aquisitions module. The changes introduced with this new module will require
a completely new set of sample data to be written.

Bugfix: Missing line terminator at the end of an INSERT

Removing redundant coma in updatedatabase

Bug 2505 enabling warnings

Bug 1807, can now search with : in the box if you want to use ccl do ccl: title=fish

Bug 1807 - Commenting out flawed nested queries implementation, now users can search using ()

Markup corrections, removing table sorting from some columns.

Markup cleanup on subscription catalog search form and results page

Style change to fix Bug 3898, Vendor Search Box Moved on Serials

Patch also includes markup fixes and cleanup for subscription-add.pl

Fix for Bug 3905 (Serial routing list add patron popup pagination broken)

This patch also includes formatting fixes and style updates. The patron search filter form is incorporated now within serials/member-search.tmpl in order to have a markup structure more suited to the pop-up.

(bug #3897) hold policies and circcontrol

This patch fix hold policies way to do, make it using circcontrol syspref to know the branch the rule must be based.

Fixing suggestions management page, Authorised values werent being passed through

updatedatabase change : set CAS auth OFF by default

ON by default will confuse most libraries, and it won't work as the CAS server won't be defined

[MT2343] Removed useless select in the planning

Cleaned some code too.

[MT2343] Display branchcodes instead of branchnames in columns

[MT2343] Modified display of Budget Parent to show budget code

[MT2343] Renamed "Budget for period" into "Root Budget"

[MT2343] Removed the hint for Amount and display Amount as mandatory

[MT2343] Added help in the budget adding form

[MT2343] Select in budget add forum

Sorted the branches select.
Modified display to "branchcode - branchname"
Fixed a bug, selection was inverted.

[MT2435] Fixed the SQL syntax error in GetLateOrders

[MT2343] Makes action column not sortable in root budgets

[MT2343] Renamed Quit button Cancel, and removed the table in delete confirm

[MT2343] Prevents Koha to format total amount in budget period add/edit form

+ Forgot a "my"

[2343] Hide "duplicate root budget" if no root budget selected

[MT2343] Hide "Edit budget period" in the menu if no root budget selected

[MT2339] Added acquisition links in the administration left menu

[MT2338] Prevents users to check 'active' with a currency different from 1

[MT2338] Cancel is now a button

[MT2338] Fixed the missing content in the "active" column

[MT2338] Deleted the Hint comment in currencies management

Bug 3964: Fixes several XHMTL errors in aqplan.tmpl.

aqplan.tmpl still contains some errors after this patch, mainly id elements that start with an numeric value.  But I need to examine the code more throughly before these are fixed.

Bug 3962: Additional patch to fix XHTML errors in aqbudgets.tmpl.