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>

Fix for Bug 4884, opac-showmarc.pl can't find compact.xsl

This patch implements Fridolyn SOMERS' suggested change
to the .xsl file path but copies compact.xsl from the
intranet template dir and points to that instead.

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

Fix for Bug 4859, Formatting cleanup for merge biblio record interface

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

Fix for Bug 4869 - Non-staff patrons logging into the OPAC don't have option to place holds from Public Lists

The template was originally designed so that form controls were
hidden from users who didn't have permission to modify a particular
shelf  because the form controls were only used for deleting from
lists. Now that users can place holds and add tags via the check-
boxes, the controls must be available to all users.

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

Bug3916 : Batch Modify tool overwrites branches fields

This was owed to the Item modification construction and the change in GetBranchesLoop behaviour which selects the user branch by default.
Adding a void branch selected
Changing default value to ""

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

Bug 4474 swap options for sys pref singleBranchMode

Resend with corrected spelling.

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

Bug 4902: Add missing b_phone and b_email to borrower details

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

bug 4900: add in missing value for usedaysmode pref

The usedaysmode preference was missing the option to
calculate due date by finding the next open date on the
calendar instead of ignoring all days the library is
closed.

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

Bug 4897: Invalid XHTML in opac-userupdate.tmpl.

Removes several duplicate name tags.

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

fixes to xt/sysprefs.t test case

* use root of source tree as base directory
* ignore case when comparing system preference codes;
  at present this is also case-insenstive in the database
* added German sysprefs file
* corrected paths to Russian and Ukrainian system preference files

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

fixed permissions for item batch deletion and modification for French

Solves bug where granular permissions for batch deletion and modification
would not work correction for a fresh installation using the French
translation.

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

fixed problems in xt/permissions.t test case

* now looks for permissions SQL files in the current
  source tree, not in intranetdir; tests in xt/ and
  t/ are meant to be run without requiring a full Koha
  installation
* added path to German
* fixed paths to permissions SQL for Russian and Ukrainian
* handles case where SQL script contains more than just
  inserts into to permissions

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

Bug 4895: Add missing granular permissions

Add missing granular permissions for serials to userpermissons.sql
de-DE, pl-PL

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

set DBversion to 139

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

Bug 3682: change message_attributes.message_name from varchar(20) to varchar(40)

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

Fix for Bug 4868 - Improve controls for cloning and deleting MARC subfields

Changes "+" and "-" controls to actively linked images giving
better feedback about their clickability and an increased click
target area.

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

Page structure correction, helps Bug 3850 (Page Need Design Work)

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

Fixes bug 3619: _send_message_by_email not respecting AutoEmailPrimaryAddress = 'OFF'

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

BUG 4883: Staff - remove \n from strings for translation

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

Bug 4833: OPAC - remove \n from strings for translation

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

Bug 4895: Add missing granular permissions to ru-RU, uk-UA

Files are different from en files, combining userpermissions.sql and userflags.sql.

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

Bug 4895: Add missing granular permissions to de-DE, pl-PL

Added missing permissions, files are translations of en.

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

But 4890: Fixes invalid XHTML in basket.tmpl

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