now ready for testing

small fixes, seems to generate working opac and intranet

updated 'conversion' script, now seems to catch all cases within both opac and intranet, still some errors (13) but no 'missed' cases

More work on 'conversion' script for HTML::Toolkit to Template::Tolkit

Draft script to 'convert' HTML:Toolkit syntax in templates to Template::Toolkit. Will need human to go through most if not all templates (basically those with a loop) and fix up. Makes no attempt to fix horrendous formatting. Branch: html-template-to-template-toolkit

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

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>

error 500 when record cannot be loaded

On a basket or a basket download, or csv export, if Koha cannot get a biblio, then it failed with error 500.
This patch fixes that behaviour skipping the faulty record in order to present the user with the biblios which are not causing trouble.

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

UNIMARC coded field search : No results

ccl.properties had to be adapted in order to be able to request the good index

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

(bug #3772) show place hold button in isbd view

This fix the isbd view to show the "place hold" button.

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

(bug #4954) allow date expiry modification only on step 0 and 3

This add a verification, to allow automatic modification of expiry date, only if we are on a page that allow modification of expiry date.

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

Update 3.0 to 3.2 budgets

FOREIGN_KEY_CHECKS had to be set to 0
in order to make the upgrade of aqbudgets table work.

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

bug 4953 - make the debian package copy in YUI images

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

bug 4952 - make the koha-remove script stop zebra for the instance

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

bug 4139: set DBrev to 143

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

Bug 4139 - Adding some indexes to deal with a performance issue with overdue reports

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

Bug 4188 Valid PDF file name when printing basket group with no name

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

Bug 2725: Fixes uppercasesurnames preference.

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

bug 3902: item class source can now be set during serials receiving

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

bug 4808: allow limited-permission circ operators to print receipts

Staff users now only need circulation privileges to print
checkout receipts and slips; the 'borrowers' permission, which
allows editing patron records, is no longer required.

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

bug 3916: set another "no change" dropdwon in batch item modification

Item type drop-down in the batch item modification tool
now has its default value set to the 'no change' value.

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

(BUG #4804) fix following bug: in neworderempty.pl, Click on minus sign in a item part don't decrements quantity.

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

bug 3916: set final "no change" dropdwon in batch item modification

Classification source drop-down in the batch item modification tool
now has its default value set to the 'no change' value.

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

Fix for Bug 4934 - Image missing for linked subfields in MARC display

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

Fix for Bug 4932, Serial Enumeration / chronology not populated on receive

- Correcting JavaScript so that serial enumeration / chronology is always
  prepopulated in the add item form when receiving a new issue.
- Altering the behavior of the "click to add item" link so that the
  "Arrived" option is automatically selected. This makes the link's
  behavior mirror the <select>'s behavior so that Arrived -> Add item
  and Add item -> Arrived.

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

bug 4422: add AR (article) format icon to staff interface

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

bug 4422: Show article icon with DisplayOPACiconsXSLT

Show article (open book) icon on results and details screen when DisplayOPACiconsXSLT is enabled.

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

Fix for bug 4928, problematic 78x display in OPAC

- Add <xsl:if> wrappers around the display of subfield 'n' for 76x-78x fields
  This eliminates the issue in Firefox that caused the font size to grow
  progressively smaller on the OPAC Details page for records with multiple
  76x-78x fields

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

add EUC-KR to list of support Z39.50 character encodings

EUC-KR (an older character encoding for Korean) is used by the
Z39.50 server of the National Assembly Library of the Republic
of Korea.

Note that this requires that a suitable version of iconv,
one that supports mapping from EUC-KR to UTF-8,
be present on the Koha server.

Thanks to a person from the Korea Society for reporting the
issue on #koha.

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

Bug 4854: Show same subfields for 245 and 880$6245

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

remove a bunch of unconditional debug warns

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

bug 4188: add note about hiding the OrderPdfTemplate syspref

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

bug 4188: move PDF templates for printed purchase orders

The PDF templates for purchase orders now live in the
directory koha-tmpl/intranet-tmpl/prog/pdf to separate
them from code.  As no automatic translation system for
PDF files is currently implemented, I'm keeping them out
of the language subdirectories for now.

At present, there are two choices of templates for printing
purchase orders (AKA basket groups), controlled by setting
the OrderPdfFormat syspref:

pdfformat::layout2pages (two-page layout)
pdfformat::layout3pages (three-page layout)

To customize a template or to add a new one requires two steps:

* modifying the PDF; note that OpenOffice source files
  are included as a way of generating the PDF
* modifying the code in acqui/pdfformat/layout*.pm to
  position the order text onto the template

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

remove extraneous warns

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

Bug 4188 Allow acq basket group printing into PDF

Since new acquisition module workflow and specifications is not
documented, this fix is more an hypothesis than a patch.

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

bug4929 - amended the instructions

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

bug 2434: set DBrev to 142

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

bug 2434 followup patch to remove EVENT from message_transports

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

bug4926 - Improves the ID fields when sending lists and carts

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

Bug 4915: Remove renewalsallowed from sample_itemtypes.sql

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

Correct typo in German sample news

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

Fix for Bug 3683 - 'Sort by' column in list interface is not translatable

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

Fix for Bug 3372 - OPAC Reading History sorting breaks limit

Some other cosmetic and functional changes:

- "Show all/Show 50" links had been styled as tabs at some time
in the past. A CSS change somewhere broke this. Reimplementing it.

- Moving sort control to a <select> similar to the one used on
search results. Since javascript-based table sorting from the
table header is used elsewhere in the OPAC I felt the <select>
method better reflected the behavior of this re-sort.

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

Fix for Bug 4909 - extra > in path on contracts pages

The logic for displaying a useful title did not take all
cases into account. This patch adds the missing case.

The patch also fixes problems with the "Cancel" link
on update and delete operations.

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

Bug 4911 Remove duplicated code

Two duplicated subroutine definitions removed

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

remove byte order mark (BOM) from installer SQL files

The precence of a BOM can prevent an SQL script from being
run.

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

bug 2434: assign to DBrev 141

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

bug 2434 remove upcoming events for all languages

this patch removed the upcoming events messaging option
and the upcoming event message from the system for all
languages since it was never implemented and confuses
librarians and patrons alike

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

(MT #3651) Fix problems on search filters.

removing of useless value "no tabs" in organize by combo.
removing of useless value "Sort 1" in organize by combo.
Adding combodate for suggesteddate, manageddate, accepteddate.
Adding research on dates in query.

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

(MT #3651) Fills "book fund" combobox with bugdets and fix some bugs of research by filter.

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

replace references to defunct info email address

Now links to Koha project website.

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

remove old revision history notes from a couple files

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

bug 4312: DBrev = 140

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

bug 4312 change default for tagsmoderation to 0 from NULL

resending with fixes

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

(BUG #4857) aqplan.pl: consideration of Planning categories with authorised values

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

MT3652 : Unifying the search of neworder with Search

This patch unifies the search in neworder so that the same biblios should be sent back when searching from catalogue search or from box.

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

(BUG #4811) suggestion.pl: display borrowers name in suggestion information filters (Suggested By, Managed by, Accepted by)

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

(BUG #4810) parcel.pl: Fix a bug with applying a filter on pending orders displaying

When we apply a filter and click on "Click here to show x items" link, there are duplicates orders which are display.

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

(bug #4523) possibility to show / hide the filters menu

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

Bug 3217 Impossible to change biblio record FW to Default

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

(MT2371) basket.tmpl, in basket details, change Open on => Opened on

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

(BUG #4356) Basket.pl: adding a link to the basketgroup

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

(BUG #4521) aqbudgets.pl - Transform undefined budget spent value to 0.00 value

This patch tranform the undefined value of budget spent to a money value which equals 0.00.

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

Select in budget add forum

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

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

Bug 4905 Runtime errors in about.pl

Calls to VERSION were generating errors when called on a string.
Use a bit of sleight of hand to avoid this
Added a couple of tests on Installer/PerlModules
Module needs better coverage testing (esp on newer perls)
Sorted to list of modules to make koha_perl_deps.pl more
human-friendly

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

fix two broken calls to _FixPriority

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

bug 3344: display in-transit status on holds priority list

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

bug 4224: explicitly mark hold requests as being handled by item in transit

This patch defines a new values of reserves.found, 'T', to indicate that
an item has been allocated to fill a hold request and is currently
in transit from one branch to another.  This explicit value is meant
to filter out hold requests from processing by holds queue batch
job.

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

Fix for Bug 4532, Use include file for bibliodefaultview logic

Note that this new include builds only the first half of the
anchor tag (<a>). Because title and subtitle are output differently
on some pages, rendering of the second half of the tag is left
to individual pages.

Includes fixes for staff client templates.

Change to basket.pl fixes a bug wherein the OPAC's bibliodefaultview
preference was being checked instead of the staff client's.

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

Fix for Bug 4821, With multiple 5XX fields, the font display gets progressively smaller

Prevent the generation of empty label spans for various
bib details fields.  A construct like

<span class="results_summary"><span class="label" />contents note</span>

is consistently misinterpreted in many web browsers, leading to
CSS oddities.  See the bug for more details.

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

Partial fix for Bug 4423, Staff Client XSLT is just a copy of the OPAC one

This patch improves the template logic specifying a layout
based on whether the display is XSLT or normal, and whether
Amazon Content is on or off. Previously XSLT view did not
display Amazon covers when enabled.

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

More fixes for Bug 4423, Staff Client XSLT is just a copy of the OPAC one

- Adding material type icons from the OPAC side
- Adding OPAC View link
- Copying revised handling of MU from the OPAC
- Removing OPAC-specific markup

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

Followup fix for Bug 4453: Removing invalid template fragment.

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

Fix for Bug 4278, canceling vendor add refreshes wrong

Patch corrected to follow HDL's suggestion that link check
for existing vendor id and conditionally redirect based
on that.

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

Fix for Bug 3895 - Menu on left of Contracts is for Admin Pages

- Also adding cancel link to the add/edit view
- Also adding message when there are no contracts to list

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

Fix for Bug 4886, missing 404 redirection on wrong biblionumber for MARC and ISBD details

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