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>

Bug 4889: Fixes minor XHTML error in booksellers.tmpl.

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

Bug 4827: Fixes library drop-down list in aqbudgets.tmpl

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

bug 4896: granular permissions now always on (DB rev 138)

Per the following koha-devel thread, the use of
staff user subpermissions, AKA granular permissions, is
now the default behavior in Koha.  This patch removes
the GranularPermissions system preference.

[1] http://lists.koha-community.org/pipermail/koha-devel/2010-February/033670.html

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

Follow up patch, removing missed conflict marker

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

Cherry-picked onto master, fixing conflicts: MT 1816: Granular permissions for the serials module

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

Conflicts:

installer/data/mysql/en/mandatory/userpermissions.sql
installer/data/mysql/fr-FR/1-Obligatoire/userpermissions.sql
installer/data/mysql/updatedatabase30.pl
koha-tmpl/intranet-tmpl/prog/en/includes/header.inc
koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tmpl
serials/member-search.pl
serials/serials-edit.pl
serials/subscription-add.pl
serials/subscription-bib-search.pl
serials/subscription-detail.pl
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>

Missing indexes and a missing column

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

Missing tables in the kohastructure.sql table.

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

Changing <p> to <span> per gmcharlt's suggestion re: Adding <p class="noholdstext"> tag around the "No Holds Allowed" text to allow for jquery selection/replacement of the text.

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

Adding <p class="noholdstext"> tag around the "No Holds Allowed" text to allow for jquery selection/replacement of the text.

A later patch might come that changes the text entirely, as it does not match the OPAC, and is often confusing for staff.

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

Bug 4472 - Missing / in img tags breaking xslt (and other img tags)

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

Updated history file, 2 more committers

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

Fix depreciation warnings in acqui/z3950_search.pl

Perl generates depreciation warnings on compilation
caused by use of ambiguous syntax. Seemed that an
underlying confusion as to row's type

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

Bug 2789 Fix UNIMAC leader plugin

The plugin was returning a 25 characters length leader rather than 24
characters. So leader was recreated automatically, removing user update,
before writing biblio record.

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

(bug #4853) change rights needed to renew loans

Use "circulate" right, instead of "borrower".

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

Bug 2505: Enabled warnings in patron-attr-types.pl

Fixed all resulting  errors.

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

Final batch of POD cleanups.

Should fix any remaining warnings with 'podchecker'

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

POD Cleanups

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

More POD cleanups

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

POD Cleanups - Few minor ones this time

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

Another batch of POD cleanups

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

The start of some POD cleanups in C4/*.pm

If these are acceptable I'll continue - target is to get rid of all warnings
and errors with podcheck and make sure that any man and html formatted pod
docs look 'OK'

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

applied paranoia

RM note: we've had our fun; next patch will be to trim
the comment to something that makes sense in context.

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

Build with pdebuild, not debuild.

This checks build dependency declarations more thoroughly.

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

Remove dependencies on perl-base.

perl-base is a required package, so it is always installed on a Debian
system, and Debian packages should not depend on it, unless they need
it a specific version, and I don't think we do.

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

Bug #4864 Add SIPServer Perl Module Dependencies

SIPServer requires Net::Server and UNIVERSAL::require

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

set DBrev to 136 for previous patch to add edit_items

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

MT2938 : Adds a permission for editing items

This patch adds a permission on edit_items

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

Fix for Bug 3081, Url's contain spaces

My investigation finds that the problem in Internet
Explorer was not with the spaces in the URL but
with the spaces in the "name" attribute of the pop-up
window. Converting spaces to underscores allows
the plugin links to work correctly.

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

Fix for Bug 3770, 'Add to list' page only allows adding to private lists

This patch is a reformat of chad@pennmanor.net's submission to
the bug report on 2010-02-16.

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

Fix for Bug 3722 - Branch deletion results in incorrect message

A fix was submitted by Nahuel on 23 Nov 2009 and pushed, but
it combined messages for existing patrons and existing items.
This patch defines separate messages for the various cases and
corrects a typo in the original fix which left existing patrons
unreported.

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

Fix for Bug 3926, inadvertantly resurrected by me.

The template-side fix for Bug 3926 was overwritten by
another template change I submitted later. My bad.

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

Fix for Bug 3992 - New Sys Prefs Branch - Local Use Tab not Highlighted

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

bug 4205 remove extra 'plan by' option

There wasn't a conditional in place to stop the planning
pull down from showing a blank 'plan by' option.

This patch also makes the other 'plan by' options look
cleaner by removing the all caps.

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

Bug 4844 Remove a circular dependency in koha_perl_deps.pl

Remove 'use Installer' which had external dependencies we're trying to
report on. Instead call Installer::PerlModules directly

added use warnings and strict
updated FSF address in License statement

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

Fix for Bug 4504, Confirmation messages in opac account not translated

Strings within JavaScript onclick attributes are not getting picked
up by the translator:

onclick="return confirmDelete(_('Are you sure you want to cancel this
hold?'));"

This patch converts those to use variables defined in the main
script block:

onclick="return confirmDelete(MSG_CONFIRM_DELETE_HOLD);"

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

bug 4845 change language from reserves to holds

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

German web installer files, including translation of MARC21 frameworks

Translation of MARC21 frameworks according to http://www.d-nb.de/standardisierung/formate/marc21.htm

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

bug 4445: Upping the daily limit for XISBN

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

bug 4445 update OCLC text tips on preferences

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

bug 4834 split joined preferences

these prefs : FilterBeforeOverdueReport & DisplayClearScreenButton shouldn't be
lumped together - they do two very different things and have nothing to do with
each other.

this patch splits them into two individual preferences.

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

bug 4027 fix testStatus typo to textStatus

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

Disable a new site created with --request-db.

This way, the various cron jobs will not try to do things to it.
They can't, since it is not yet really created.

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

Update FSF address in debian/copyright.

This makes lintian be a bit more quiet.

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

Upgrade Koha instance database schema when package is upgraded.

If the new version of Koha requires a newer version of the database
schema, the Koha instances will work immediately after the package
is upgraded, rather than after the sysadmin has run koha-upgrade-schema
manually.

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

Fix bug 4818: Add note to koha README.Debian to maybe disable default vhost.

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

Add YAML as a dependency, since C4/Record.pm uses it.

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

Add missing build-dependencies (python, python-debian).

Also, debian/control gets re-generated, from debian/control.in and
PerlDependencies.pm.

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

Add tools to update debian/control's build dependencies.

The canonical list of Perl module dependencies are in
C4::Installer::PerlDependencies::PERL_DEPS now. Add a script
(debian/list-deps) to turn that into a list of Debian package names.

Because that is a slow process, and the output rarely changes, do
not do that at build time. Also, doing it at build time would require
modifying debian/control in evil ways.

Instead, add another tool, debian/update-control, which reads the
new file debian/control.in, and adds the output of debian/list-deps to
Build-Depends and creates a new debian/control.

debian/control.in is the master file. If changes are needed, that should
be edited. For performance and convenience reasons, the output is also kept
in git, but don't edit debian/control directly, please. Such changes
might get lost by the next commit by someone else.

Whenever PERL_DEPS changes, debian/update-control should be run as well
and the result committed to git.

This is not quite as automatic as it might be, but should be good enough.
It avoids keeping the list of Perl modules in two places.

Note that since it seems impossible to automatically figure out the Debian
package version that corresponds to a Perl module version, I have not
tried to do that at all.

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

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

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

System Preference updates

Raft of language updates

Language updates

Bug 4817: Point to README.Debian in koha package description, postinst output.

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

Whitespace removal, removal of U+FEFF character

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