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>

Added "dialog alert" class to "No Results Found" message on the patron
search so the alert stands out.  Previously no css was applied to it so
it was just plain text.  Librarians weren't noticing it.

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

(MT #2565) fix aqplan csv export, and turn off debug

This fix the csv export that doesn't export correctly the budget name, it's empty column.
And it turn off some SQL debug that slow down the result of the page and the csv export.

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

bug 4508: fix crash when editing patron attributes or message prefs

When editing patron attributes or messaging preferences on the
patron editor, Koha would crash with the following error:

Software error:
Can't use an undefined value as an ARRAY reference at
/home/koha/kohaclone/C4/SQLHelper.pm line 186.

This would occur only when editing the specific attribute
or message preferences section, not when editing the entire
patron record.

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

Set upload target to squeeze-dev.

Since we're uploading a git snapshot, this is not meant for
production, and thus it should not be uploaded to squeeze,
but for squeeze-dev.

Documentation for the pockets will appear in the apt
repository or the wiki once it is ready.

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

Update README.Debian for koha-common with post-install setup.

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

Update README.Debian for koha.deb package.

One bug fix (three steps, not two, are automated in koha-post-install-setup).
Also add a note about the need to possibly edit ports.conf.

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

Use a native Debian package.

Since the packaging is part of the upstream source, there's no point
in having a Debian version.

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

Add dependency on apache2-mpm-itk for koha-common.

The way things are set up, each Koha instance runs under its own
user account. At least for now, this requires apache2-mpm-itk.
Later we can investigate other possibilities.

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

Make sure the temporary version number comes before 3.2.

This way, upgrades from the temporary ones to the final release
will happen automatically.

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

bug 4816: require authentication for placerequest.pl

Otherwise, it was possible to construct a URL to place
hold requests without authorization.

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

bug 4386: fix email hold filled notifications

Call to GetMembers() was using the old parameter style.

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

removed needless imports of the YAML module

Removed instances of 'use YAML' that were either completely
unnecessary or which were used only in debug code.  Also
removed a needless import of Data::Dumper.

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

Bug 4199: Adds ability to print routing slip. Patch 2.

Previous patch added an invalid XHTML element.  This patch fixes it.

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

Bug 4199: Adds ability to print routing slip from serials-collection.

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

Bug 4805: Fixes multiple subscriptionid's being passed to serials-collection.

This is a replacement patch for my previous patch.

This patch insures that only unique subscriptionids are defined in the serials-edit form.  Multiple ids are passed to serials-collection, but not multiple copies of the same id.

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

fix another link to koha-community.org

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

fix a common "developement" typo

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

bug 4377: ensure HOLD_PRINT letter added during installation and upgrade (DBrev 135)

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

bug 4311: respect OPACXSLTResultsDisplay

Check OPACXSLTResultsDisplay instead of XSLTResultsDisplay when
determining whether to use the XSLT bib results stylesheet for
OPAC search results.

In the process, added a new $search_context parameter to
C4::Search::searchResults() to specify whether results
are to be served up for the staff interface or for the
OPAC.

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

bug 4018: remove duplicate unAPI link when XSLT bib details display on

The unAPI link is now added only in opac-detail.tmpl.  More than
one unAPI link in the same page was causing a bit of indigestion
for Zotero, which interprets multiple unAPI links as specifying
more than one resource on a given page.

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

Install the admin scripts.

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

Make koha-create be able to handle manual database creation.

This is necessary if we do not have access to DB server with sufficient
permissions. The DB server is used for other things, that may well be
the case.

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

Add support for a remote mysql server.

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

Use /etc/mysql/koha-common.cnf instead of the Debian one.

koha-common.postinst now creates a symlink to the Debian one.
The sysadmin may replace that manually with something that allows
Koha to access a remote mysql server.

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

Add koha-foreach, a script to run a command for each Koha instance.

Use script to enable the build_Holds_queue.pl cron job.

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

Centralize creation of Koha instance dirs.

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