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>

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>

Bug Fixing 3568 : Untranslated strings

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

DB rev 054->058

055: changed OPACSearchForTitleIn per requests in bug 1934
056: Bug 1172 : Add OPACPatronDetails syspref
057: Bug 2576 : Add OPACFinesTab syspref
058: Added primary keys to language tables

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

Bug 1934: Final Patch - Fixed Default Values

The default values had classes they didn't need assigned
to them.

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

Bug 2559 Added primary keys to tables per comments

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

Bug 2576: Added OPACFinesTab Syspref

This preference allows library staff to decide whether to show
the fines tab in the OPAC for logged in patrons.

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

Bug 1172: Added OPACPatronDetails system preference

This patch allows library staff to disable the patron details
tab on the OPAC.

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

Bug 1934: Changed variables per request in bug report

System preference now looking for brackets around variables.
ex. {TITLE} instead of TITLE

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

bug 1003: tweak GetBibliosShelves

* correct POD - returns *public* lists that bib
  belongs to
* use fetchall_arrayref to return results in
  one fell swoop
* corrected indentation

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

Bug 1003 Added lists to bib details pages

This patch adds links to all public lists that a bib
record belongs to to the detail page in the OPAC and
the Inranet. If the item is not in any lists nothing
shows on the detail page.  The same goes if virtualshelves
is turned OFF.

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

Bug 3582: Missing usage information for -h / --help switch for rebuild_nozebra.pl

Fix for Bug 3582:  Missing usage information for -h / --help switch for rebuild_nozebra.pl

http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3582

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

comment out styles in gb_styles.css

These were added in the labels rework, but do not belong here, since
gb_styles.css should references only CSS classes responsible for
displaying the greybox itself, not the content inside it.  In particular,
the generic table styles were causing the icons added by tablesorter
to be tiled in the opac-detail template.

To finish this work, the styles should be taken out entirely and
moved to whichever lables template they were supposed to affect

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

Bug 2713 Make Marc & ISBD View match Normal template

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

DB rev 053->054: add altcontactcountry and B_address2 to borrowers and deletedborrowers

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

Bug 3464: Added altcontactcountry and B_address2 to borrowers and deletedborrowers

- added altcontactcountry and B_address2 to tables borrowers and deletedborrowers so that all addresses offer the same fields
- changed B_country and country to work with syspref BorrowerMandatoryField
- changed display of Alternate Address on borrower detail page, added labels, lines and edit button to match the other sections

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

Get rid of a few warnings in the bulkmarcimport script: C4/Biblio.pm, hunks #1, #2: a warning occurring in NoZebra configurations. C4/Biblio.pm hunk #3: warning occurring in Unimarc MARC flavour. misc/migration_tools/bulkmarcimport.pl hunk #1: warning occurring when no default format is specified on command-line with -m switch.

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

bug 3581: fix staff permission for item circ alerts

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

Revamp interface of branch transfer limits to give finer conrol of bulk selections.

Adds selection by row and by column, all/none/toggle for individual ccode/itype sets, and a tabbed interface for a more compact display.

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

Tighten up layout of item circulation alerts interface, putting checkout and checkin table in tabs.

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

Modules for caching - C4::Cache is the base class, C4::Cache::Memcached and C4::Cache::FastMemcached are subclasses

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

Starting on OO module for memoizing our functions

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

bugfix breeding results of cataloging search has wrong id

The breeding id, rather than coming from the database, was set by a counter.
Not so good links in the search results that way.

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

smart-rules load users library instead of default

If the user isn't a super librarian load their library by default instead of
the default settings.

This would eliminate some confusion amoung my librarians.

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

Bug 2505: Enabled warnings in admin-home.pl and roadtype.pl.

Fixed warnings generated by roadtype.pl and tightened up the code (referred to cities.pl).

No warnings were generated by admin-home.pl

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

cataloging search results from catalog don't show on later pages

In the cataloging search results from the catalog, the result set would
decrease by one per page.  Past page 19 there was an empty table.  The reason
is that SimpleSearch gives the result set ( 20 records here ), and
searchResults expects to trim the result set to just the 20 records we want.
This changes the call to searchResults() to have a 0 offset to the result set.

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

More history updates

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

Updating history

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

Return barcode as Sip hold_patron_id

Behave consistently with other patron_ids as many sip clients
use this in a subsequent patron information or patron status request

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

Merge branch 'labels_recon' into to-push

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

Merge commit 'origin/master' into labels_recon

tweak label batch template for new label batches

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

debugify a warn

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

fixed references to template includes

Errors found by the xt/author/valid-templates.t test.

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

fixed translatibility error in label-edit-layout.tmpl

Error found by xt/author/translatable-templates.t test.

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

remove unnecesary import of HTM:::Template::pro

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

replace syslog with warns

As discussed with Chris Nighswonger on #koha, this patch
removes the calls to syslog and replaces them with warns
so that error messages generated by the labels code
are sent to the Apache error log.  This avoids splitting
this sort of logging across multiple files and is consistent
with current practice in most of the rest of Koha.

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

Nicer error message if a branch is not set

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

Adding a missing module to labels search

This causes an error to be thrown if no items are retrieved and the
code goes to call &syslog

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

Fixing warns (tnx to Chris Cormack for identifying these)

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

bug 2505: remove additional warning messages

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

enable warnings in overduerules.pl Call GetBranchesLoop instead of duplicating code make countletters a count as it says check for definedness before doing string comparisons do as not to generate warnings

UNIMARC XSLT enhancement

Improve UNIMARC XSL for result and detail page:

  * Add fields
  * Consolidation
  * Support RTL language display using $7 UNIMARC standard
    for Arabic and Hebrew
  * Fix bug #2773 via XSL at field level, removing NSB and
    NSE characters in biblio records, and avoiding to do
    it at application level in C4::Output.pm

Note for RM: There will be some tweaking coming next week.
If I see you don't push this patch, I will resubmit a
completed version later.

Sponsored by BULAC, www.bulac.fr

Bug 2505: Enabled warnings in boraccount.pl and pay.pl.

Enabled warnings in members/boraccount.pl and members/pay.pl.  Fixed resulting warnings.

[40/40] Removing depricated files

[39/40] Removeing depricated functions from label module.

[38/40] Misc bugfixs

[37/40] Updating sample labels data

This update takes care of the EN and FR data only. Sorry.

[36/40] Bugfix for profile update on profile/template association change.

[35/40] Work on C4::Labels tests and various bugfixs resulting

This patch also moves the Labels tests into their own sub directory.

Due to a squash mistake this patch also includes the following:

Fixing up POD for C4::Labels modules

Also a minor bugfix and code refactoring.

[34/40] Work on syncing db schema on both new and upgrade installs

Labels creator data migration script
Updating updatedatabase.pl and bumping db version

[33/40] Porting call number splitting improvements from HEAD

[32/40] Work on patron card creator module

This is simply cut&paste from the older labels module code in an effort to preserve the existing patron card code for use
in the patron card creator module. This is NOT functioning code at the moment.

[31/40] Misc bugfixes and cosmetic cleanup.

[30/40] Work on labels module home page.

[29/40] Work on label module general tool bar.

[28/40] Updating tools homepage to reflect label module rework.

[27/40] Fixing up the labels batches toolbar functionality.

[26/40] Refactoring auto label batch generation feature in bulk import tool and adding error trapping/handling.

[25/40] More work on label export coded and interface. Adding icons for all export formats & general dressing up.

[24/40] Adding single/multiple label printing to label export code and interface.

This adds single/multiple label printing by item number along with new CSV and XML export code.

[23/40] Initial work on label export interface.

[22/40] General code cleanup of new labels module code.

[21/40] Adding de-duplicating method and associated label edit batch code.

[20/40] Work on label printing code.

[19/40] Various tweaks to the interface cosmetics, form validation scripts, css, etc.

[18/40] Work on label item search.

[17/40] Work on C4::Labels::Lib::html_table api to add link field option

[16/40] Improvements to label management interface code.