Adding some error proof on GetMarcRecord

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

Claims management enhancement

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

Adding enddate to Subscriptions

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

C4::Search :Enhancements

Stemming is now available for more languages

Adding also getDistinctValues which gets distinct values for a zebra index or for a Mysql field

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

C4::Output adding getlanguagecookie + other enhancements

required for language stemming management
Adding also some filters for NSB NSE characters
Now, get_template can be used even when query->cookie not set

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

Adding updates for Suggestions

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

Editing C4::Suggestions.pm

Enhancing C4::SearchSuggestion
And changing its API
Changing C4::Suggestions API taking a hashref for NewSuggestion and ModSuggestion

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

defining a value which could be optionnal

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

Modifying Members : Add Mod and GetMember

This update the way Member is added and editing so that import and Edition
 could be best automatized
GetMember evolves and allow ppl to serach on a hash of data

Adding SQLHelper A new package to deal with INSERT UPDATE and SELECT

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

Adding Merge of two biblios from a virtualshelf

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

Adding download for Cart and Shelf at OPAC

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

Adding notices.pl which enables staff to see what notices have been sent to user

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

Move Item from one biblio to another

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

Delete All items in one click

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

Adding RIS And BibTex Export Followup

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

Adding Batch edit for items

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

Enhancing Inventory.pl

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

Adding RIS and bibtex export

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

Adding Some new functions

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

Adding GetSupportList and GetSupportName

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

Adding OpacSearchHistory system preference

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

Adding Opac-SearchHistory feature

Enables ppl to store their search history and delete the whole history

Adding Storable required by Opac-Search-History

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

Authorities bulkmarcimport

Adding some new options to bulkmarcimport :
-k idtagsubfield in order to store the id of the file record into another field
-match tagsubfield,index
-a to import authorities
-l logfilename to store logs

Bug Fixing : C4/Charset.pm
Charset was incorrect for UNIMARC Authorities

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

Adding management for multiple $9 in MARCSubjects

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

UpdateDataBase for smart-rules modification

Members.pm :
Adding IsMemberBlocked
Circulation.pm :
Using IsMemberBlocked in order to implement finedays

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

Smart Rules enhancements

Adding finedays and reservesallowed and renewalsallowed management in smart rules
Adding Clone Rules
Adding CanBookBeReserved and CanItemBeReserved in C4::Reserves
Manage Reservesallowed in opac and staff interface
Manage renewalsallowed in C4/Circulation.pm
Update Database follow

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

Merge commit 'koha-public/biblibre-sopac' into to-master

[Merge the ILS-DI web services used for integration with SOPAC]

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

Adding one template file missing

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

Merge commit 'origin/master' into bl-sopac

Delete unused function

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

Bug 3661 XSLT fixes

The XSLT results list (MARC21slim2OPACResults.xsl) did not have any separators between multiple 246 fields, so additional titles all ran together.  I added the standard semicolon or period as in other repeating fields.

The display for personal and corporate authors was also confusing.  The period after the last personal author (100/700) is very hard to see, so the display looked like the first corporate author only belonged to/was associated with the last personal author, rather than being a separate author entry.  I've added a dash at the beginning of any 110/710 entries to set off the corporate authors a bit in the display.

I added the same code to insert a dash before any 111/711 (Meeting author) as well.

This patch also adds missing text labels for text_list_bullets.png
(bibliography) and remote.png (available online).

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

Minor markup tweak and correction

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

Minor interface tweaks and markup corrections.

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

Interface tweaks and markup corrections.

Includes JavaScript to automatically change field labels based on which unit of measurement has been chosen.

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

Interface tweaks and markup corrections.

Includes JavaScript to automatically change field labels based on which unit of measurement has been chosen.

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

Minor markup and interface tweaks fo "Edit Label Layout"

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

Markup corrections and other minor changes.

- Clarifying error message
- Hiding batch action buttons when batch is empty

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

Markup tweaks and corrections for label search window.

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

Corrections to ensure message list appears in the proper box (Bug 3668).

The patch sets a "flagged" variable for displaying the messages correctly, bringing it in line with the way other messages are displayed. The patch also includes some markup tweaks and a change to make display of the "add message" link javascript-dependent.

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

Implementation of OPAC Message formatting

This patch implements the suggestions about the new and old message system made by Owen Leonard.

Includes type fix pointed out by Owen.

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

Correcting references in the OPAC to files in intranet-tmpl (Bug 3574).

Adds greybox library to OPAC files, corrects reference to icons in intranet-tmpl.

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

Updated help files

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

Updated help files.

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

Silence extraneous messages when in batch mode on build_browser_and_cloud.pl

The script already has a param -b for batch mode, which should silence informational
messages, but it missed a couple.  This patch fixes that.

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

Bug 2505: Enabled warnings in members/messaging.pl and readingrec.pl.

Fixed any warnings that appeared in the logs.

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

Bug 2505: Enabled warnings in members/patronimage.pl and setdebar.pl.

Fixed resulting warnings after warnings was enabled.

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

Bug 2505: Added warnings to members/members-home.pl and members/setstatus.pl

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

Bug 2505: Enables warnings in members/mancredit.pl and maninvoice.pl

Fixes resulting warnings.

Also corrects a couple of minor typos.

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

Bug 2505: Enables warnings in member-flags.pl, member-password.pl, and member-picupload.pl.

No warnings were generated in testing after warnings were enabled.

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

Merge commit 'workbuffer.org-koha/translation' into to-push

[3.2 translation work]

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

BUG 3675: Untranslatable strings in opac-basket.tmpl

Makes 'Place Hold' and 'Tag' translatable.

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

BUG 3380 Show serial enumeration on opac holds screen

Shows column Vol Info if serial enumeration is used for one of the items.
Replaces first patch.

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

German updates

Fixing typo in MARC21slim2OPACResults.xsl

German updates for 3.2

Last of the language updates

Continuing on with updates

Continuing updating po files

Updating po files for 3.2

More new language files for 3.2

Getting languages ready for 3.2

German updates for 3.2

Updates for 3.2.x - French OPAC

Maori updated for 3.2.x

German Opac updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

English NZ variant translations for 3.2

German Intranet template updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

German Translations for 3.2 updated

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

Fixing a syntax error in german opac .po file

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

German translation updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

Bug-3098-Bug-3108-Bug-3545 revision to XSLT syspref patch

The patch I sent earlier repeated a line having to do with the count of items, which mean the count was now doubling (2 items instead of 1, 4 instead of 2, etc.).  This patch removes the first, now superflous line of $xmlrecord =~ s/\<\/record\>/$itemsxml\<\/record\>/;

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

Bug 3416: Fixes filter on parcels.tmpl.

Changes the action of the form on parcels.tmpl from "post" to "get."  Otherwise the filter is not functional.

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

Bug 3415: Adds highlighting to tables in parcel.tmpl

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

Bug 3414: Cleanup and xhthml correction of parcels.tmpl

Fixes several xhtml in parcels.tmpl and enables highlighting within its data table.  Removes unnecessary variable from parcels.pl.

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

bug 3436: tweak cache settings for patron images

Per suggestion by Joe Atzberger.

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

DB rev 061: added ShowPatronImageInWebBasedSelfCheck (bug 3436)

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

bug 3436: improve error messages in self-check

Improved the error messages displayed to users
by web-based self-checkout when a problem with
a loan occurs.

This changed was sponsored by the Plano Independent
School District.

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

bug 3436: enhance sorting of loans (self-check)

[1] Change the default sort order of checked out
    items to have the most recent loans displayed
    first.

[2] Add the ability (via jQuery tablesorter) for
    the user to sort the list of loans.

Also fixed the formatting of the due date.

This change was sponsored by the Plano Independent School
District.

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

bug 3436: add patron images to web self-check

Adds a new system preference, ShowPatronImageInWebBasedSelfCheck;
if this preference is ON, a patron's image is displayed
if available when using web-based self-check.

Note: a patch for updatedatabase.pl will be made when this
change is ready to push.

This change is sponsored by the Plano Independent School
District.

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

Add predefined fee types

Ability in system administration to define additional fee and fine options that appear in dropdown box
on the create manual invoice tab in patron fines.

To use create authorised values with the category MANUAL_INV,
Authorized Value is the description, and Description is the default fee, if any.

This work sponsered by East Brunswick Public Library, East Brunswick, NJ, USA

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>

Ensure op is defined in serials scripts

Set $op to an empty string if no value passed
otherwise the error-log fills with warnings on comparison

DB rev 059->060

Bumped for new patron messaging feature.

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

avoid error message since messages table does not exist yet

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

New Messaging System.

This system is indended to replace the old opacnotes
  and borrowernotes fields in the borrowers table.
  This system allows an unlimited number of Libraran and OPAC notes to be
  attached to a borrower. Each note has a message, a message type,
  the data it was created, and which library created it.
  Each message can only be deleted by the library that created it unless the syspref
  AllowAllMessageDeletion has been set.

  This system may be used simultaneously with the old notes system and does not affect it in any way.

  A new database table (messages) was added for this feature.

  The System also allows for pre-defined notes for Borrower records

  To use these, just create authorised values with the category BOR_NOTES
  where the Authorized Value is the short description shown in the pulldown,
  and the description is the text that should be in the note.

  Original Author: PTFS Contractor <dbavousett@ptfs.com>

  This work co-sponsered by Middletown Township Public Library, Middletown, NJ, USA
     and East Brunswick Public Library, East Brunswick, NJ, USA

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>

DB rev 058->059 - add DisplayOPACiconsXSLT syspref

Pushing new new system preference DisplayOPACiconsXSLT created per bug 3545
as DB rev 059

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

Bug 3098, Bug 3108, Bug 3545 -- Make MARC21 XSLT obey sysprefs in OPAC

The MARC21slim2OPACDetail.xsl and MARC21slim2OPACResults.xsl XSLT
stylesheets do not obey any of the OPAC sysprefs, OPACURLOpenInNewWindow,
URLLinkText, or a new one added for Bug 3545, DisplayOPACiconsXSLT.
Modified C4/XSLT.pm based on code written by Fréric Demians (thanks!) to
pass the values of the sysprefs to the .xsl files.  Modified the .xsl
files to implement them.  If the respective sysprefs are set, the OPAC
results and display pages will/will not display the format/material
type/audience icons, will/will not open URLs in new browser windows,
and will/will not use the text entered in URLLinkText as generic link
text when there is no value in the 856 subfield y, 3, or z.  (Note that
if subfields y, 3, or z do contain text, the first match is what will be displayed.)

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

Fix for license statement on about page

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

Update some DBI interactions

remove unnecessary calls to finish
let DBI return an array instead of looping to create one
make some SQL oneliners more succinct

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

Bug 2500 Correcting incorrect splitting of cutter numbers

This patch does two things to improve the call number splitting algorithms:

1. It makes changes to ensure that cutter numbers are split correctly in ddcns

2. It moves custom/fiction/biography call number splitting to a separate algorithm.
    Before they were incorrectly placed in ddcns.

This patch also modifies the call number splitting tests to accept call numbers from the
command line to allow quick testing of any give call number against a particular algorithm.

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

Avoid unnecessary warnings in reserve/modrequest.pl

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

bug 3614 followup

* renamed default_ind_to_space to _default_ind_to_space
  as it is an internal function
* added calls to _default_ind_to_space in other places

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

bug 3614 Fix incorrect setting of indicator

Pull the code into its own subr so that the dangers of
inaccurate copy & paste are avoided in future

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

updated DB-dependent tests to reflect labels rewrite

* removed lib/KohaTest/Labels.pm, which is now completely outdated
* updated list of database tables to truncate in lib/KohaTest.pm

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

bumped up required version of Test::More

Now requiring at least 0.80, the oldest version available
from CPAN, because use_ok tests were not being properly
accounted for in the test plan before 0.73.  This led
to a situation where a test script containing a use_ok test
that passes running under an earlier version of Test::More
would fail under a more recent one.

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

(bug #3558) fix build_browser_and_cloud

This patch, permit to take care of similar entries, if there is some diacritics or not.
(cherry picked from commit 776c177e3debedaf08fec65fbf8111675ccc93e7)

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

include grace period when calculating overdue fine

Before the grace period was not included in the number of days overdue when
calculating the fine.  My librarians seem to think the number of days overdue
should include the grace period.

Fine is not assessed before the grace period is up though.

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

Enable warnings in serial scripts

Remove some superfluous variables
Simplify some loops over lists
Fix generated warnings
Strip whitespace from line endings

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

Moving css to correct location

See comments in commit b1197477d476ddaedec43392bdd92396b94b0852

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

Bug 3602: Fixes a log error for opac-search.pl if expanded_options is not defined.

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

Bug 3603: Fixes XHMTL errors in catalogue/showmarc.tmpl

Missing <title> tags, missing </head> and <body>, extra </div>

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