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>

Bug 3604: Fixes invalid XHTML in manage-marc-import.tmpl.

Also adds zebra striping to <table>

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

Bug 3605: Fixes XHTML error in z3950_search.tmpl

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

Bug 3606: fixes 'last 50 items only' link in members/readingrec.tmpl.

If a staff member clicked on 'Show all items' in a patron's reading record, and
then clicked on 'show last 50 items only,' all of the records would still be
shown.  The link for the 'last 50' items contains a switch for 'limit=full'
instead of 'limit=50'.

Also fixes some XHTML errors.

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

Avoid potentially unnecessary database call in Auth

Auth, after verifying the session/logging in the user, calls to the database
to get the users borrowernumber.  This call is probably unnecessary because
borrowernumber is part of userenv.  Check userenv before going to the database.

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

BUG 3625: untranslatable strings in opac-shelves.tmpl

select all, clear all, tag, place hold

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

Follow-up for Bug 3524, adding itemtype text description

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

Bugs: 1172 3579 3596 fixes to patron detail in opac

This patch makes it so that the patron can see their
details even if the OPACPatronDetails sys pref is OFF.

It also removed the messaging options from the patron
details and leaves them on the messaging tab.

Finally it adds in address fields that were missing
previously on the update patron form.

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

Handle null-or-empty to Charset::StripNonXmlChars

When rebuild_zebra.pl is run from cron, there is an occasional error
of the form:

Use of uninitialized value $str in substitution (s///) at /home/ebpl/kohaclone/C4/Charset.pm line 304.

This error is occuring when the string that is fed to Charset::StripNonXmlChars
is null or undefined, for some reason.

This fix will handle the null-or-empty condition, and thus suppress the error.

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

Bug 2505: Enabled warnings in tools-home.pl, holidays.pl, exceptionHolidays.pl, and newHolidays.pl

Tested for warnings after addition of 'use warnings.' No warnings were generated.

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

Bug 3601: Cleanup of holiday.tmpl

Changes the word 'location' to 'library.'
Expands the description of 'exception.'

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

Remove js debugging which caused failure of "uncheck all."

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

Merge commit 'origin/master' into sopac

Fix for typo in previous history patch

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

2 new committers for the history doc

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

Bug 3567: Display Alternate Graphic Representation (MARC21 880) in OPAC

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

Changes to make checkouts/holds display the same on circulation and patron detail screens (Bug 3106)

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

Display description, avoid broken itemtype image on holds page (Bug 3524)

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

Adding strings to translate in tags.js

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