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>

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>

ran ILS-DI code through perltidy

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

bug 2505: enable strict and warnings in ILS-DI code

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

added missing module import to C4::ILSDI::Utility

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

Changed the way to get borrower's branchcode in HoldItem and HoldTitle

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

I forgot to export functions from Utility.pm

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

This fixes the encoding of ampersand entities in GetRecords

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

Added comments to ILS-DI code

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

Added ILS-DI module

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.

[15/40] Initial work on label batch edit interface.

[14/40] Work on batch management interface.

[13/40] Work on profile editor interface.

[12/40] Work on label managment interface.

[11/40] Work on layout editor interface.

[10/40] Work on Label Tool Homepage.

[9/40] Work on template interface.

[8/40] Adding new layout management page

This moves the layouts off of the labels tool start page to bring it into
conformity with the other parts of this tool.

[7/40] Adding C4::Labels::Lib module

Library of often used label related linear functions

[6/40] Adding C4::Labels::Label Module

Creates and manipulates Label objects