Bugfix LDAP config to play nice with Zebra.

The section in KOHA_CONF was moved inside <config> and renamed
<ldapserver>.  Perldoc updated to correspond.  Tested with local
LDAP and existing test (t/db_dependent/Auth_with_ldap.t).
Patch can be applied with confidence based on comparison of 2 dumps:
perl -e 'use C4::Context; use Data::Dumper; my $context=C4::Context->new("./old_config.xml"); print Dumper ($context->{server}->{ldapserver}),"\n";'
 and
perl -e 'use C4::Context; use Data::Dumper; my $context=C4::Context->new("./new_config.xml"); print Dumper (C4::Context->config("ldapserver")),"\n";'
These dumps reflect the way Auth_with_ldap accesses configuration info before and after.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix 1978 - on circ, check date specified output is later than now.

This resolves all remaining significant parts of the bug.
Note that this may have to be revisited for hourly or partial-day circ functionality
for 3.2.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix for 2174: s/langauge/language/

Resolution for issues raised in Bug 2076:

OPACItemHolds
OPACShelfBrowser

have been added to updatedatabase.

The xslt sysprefs have also been added with a note stating they
only work with MARC21.

In OPAC, verticaly align on top biblio records and book cover image

Without this patch, on OPAC result page, depending whether book cover
image is available or not, biblio records and book cover are badly
verticaly aligned. The information is in the middle of the cell.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add a kinda config file for Makefile.PL

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bypass corrupt bibs when linking headings

During link_bibs_to_authorities.pl run, a bib was
observed that had both biblioitems.marc and biblioitems.marcxml
set to blank or NULL.  Added check to verify that a
defined value is returned from GetMarcBiblio(); may follow
up with additional patches once it's determined how
the corrupt bib came to be loaded.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug 2247: don't crash linking bib heading with invalid subfield

Prevent following crash when linking a bib heading to an
authority record if the bib heading has a subfield whose code
is ')' or any other regular expression metacharacter:

Unmatched ) in regex; marked by <-- HERE in m/) <-- HERE / at
/home/koha-pro/kohaclone/C4/Heading/MARC21.pm line 186.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Spelling correction in template.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix upgrade-child: could not update child unless guarantor was defined. Also, limit upgrade path to C -> A .

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Change order of checks, and avoid warnings if args undef (i.e. anonymous login).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

(UNIMARC) index only 101$a as language

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2112 - add indicators to MARC display

In any MARC record display in the OPAC or staff client
that displays the MARC tag numbers, the indicators are
now displayed as well, following the tag number.  If an
indicator is a blank, it is displayed as '#'.

Add a function to C4::Koha, display_marc_indicators(), to
generate this display form of the indicators.

Refactoring note: the four scripts changed in this commit
have a lot of duplicate code that could be merged into
a MARC displayer class.

Documentation notes: screenshots of tagged MARC record
displays should be updated.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2207 - improve indicator input in MARC editor

Instead of having one input field for both indicators
of a variable field, the bib and authority MARC editor
now has an input field for each indicator.  This has
two main advantages:

* it is easier to tell what the indicator values are,
  even when the first indicator is a space
* it is easier to set the first indicator to blank
  and the second indicator to non-blank.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

code cleanup - remove unused find_values() copy and paste

The find_values() function in authorities/authorities.pl
and authorities/detail.pl is not used; appears to be copied
from additem.pl.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix column misalignment in opac-reserve .

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug 2235: fix various data-corrupting bugs in MARC editor

1. Non-ASCII characters in the 942$h (biblioitems.cn_class)
   would cause an edit to fail and lead to a corrupt MARC record in
   biblioitems.marc and biblioitems.marcxml.  Fixed by
   turning on Perl's UTF-8 flag for MARC tag parameters
   processed by C4::Biblio::TransformHtmlToMarc

2. Do not add empty 006 and 007 fields if the corresponding
   input fields are blank.

3. Make it possible to save '0' as a valid subfield
   value.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixes bug #2229

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Typo bugfix: sterling is GBP not GPB

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Reverts AnonSuggestions syspref to correct type, and translates into FR

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fixes bug #2231

In OPAC, when cart is sent by email, the email contains an attachement:
an ISO2709 file with cart biblio records. Most users don't have a clue
what to do with this file.

This patch modifies opac-sendbasket.tmpl to clarify what the attachment is.
It also fix a bug in opac-sendbasket.pl which wasn't reading the template
section containing information about the basket:

  Here is your cart, sent from the Koha Online Catalog... foo

It may be necessary to verify that template text is in 'valid' English...

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix authentication problem in previous commit; userenv->{} should not be used before check_auth is called.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix: show (cleaned) comment back on detail page.

Also added minor debug feedback to review page.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bug 2098: do not apply stemming if search term has digits

If a search contains a digit, it is almost certainly an identifier, and should
not be stemmed.  This is particularly relevant for ISBNs and ISSNs, which
can contain the letter "X" - for example, C4::Search::_build_stemmend_operand
would reduce "014100018X" to "x ", which for a MARC21 database would bring
up irrelevant results (e.g., "23 x 29 cm." from the 300$c).

With this patch, supplying a search term that contains one or more digits
followed by "X" will no longer retrieve irrelevant results.  This applies
to catalogs using Zebra and query stemming.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Revert "Gives a type YesNo to AnonSuggestions syspref"
As pointed out by Joe, this wasn't correct, it's not supposed
to be a YesNo type.
This reverts commit 0c512c7dd41599a18543b3791c89c4c03b5352aa.

Revert "Bug fixing : get_user_permissions SQL Fixing"
From Galen: because code is part of the join, there is in fact no ambiguity

This reverts commit 3a05ae40f6d03a8399e4653da1761f5dde7e23ea.

Bug fixing : get_user_permissions SQL Fixing

user_sub_permissions join introduced ambivalence for permissions field

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix : generating session id on some systems was not defined

Explicitly naming the method to generate id is required on some systems
(SUN 4)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add | separator between authors repetitions in OPAC detail page

Without this patch authors display on OPAC is weird.
You get for example:
  Author: Freud--Sigmund Lacan--Jacques
And you finish to read an author 'Simund Lacan'

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Small script to identify syspref differences between languages (3). See

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add missing variables to default fr-FR systempreferences

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Gives a type YesNo to AnonSuggestions syspref

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2225 - fix resource leak in SimpleSearch

Explicitly destroy ZOOM Query and ResultSet objects created
in C4::Search::SimpleSearch() - for long-running users
of SimpleSearch, such as link_bibs_to_authorities.pl or
the matcher used by the staging MARC import, the failure
to do this causes a memory leak in both the client Perl
script and (in the case of ZOOM ResultSet objects) a
corresponding resource leak in zebrasrv for the life
of the Z39.50 connection.

With this change, link_bibs_to_authorities.pl will be
be able to process large bib datasets without leaking
memory due to the Z39.50 lookups it does.

Similar changes are indicated for all uses of
ZOOM that could last longer than a single CGI
query.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Display only approved tags, change simple form to GET, supply "strata" for eventual cloud building.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 1993 - task scheduler improvements

[1] Removed hardcoded path to original developer's
    Koha configuration file; now checks KOHA_CONF.
[2] Now attempts to figure out if a task was successfully
    added to the task queue; if it wasn't, the failure
    is now reported in the  task scheduler page.
[3] When traveling to the task scheduler page from a
    saved report, the report in question is now
    selected by default.
[4] Removed the button to edit a queued task; this is
    not supported.

This patch is essentially a bandage; I've added a BUGS
section and some FIXMEs to the POD for C4::Scheduler
detailing some issues - a refactoring of the task
scheduler is in order at some point.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 1874 Adding variable assignment so that the confirm button is properly displayed in both Staff and OPAC

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correcting "Open" to read "Free" on shelves.pl in order to match pull-down selections

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX 2106 - repeatable holidays always on sunday. Also removing an old, unused syspref, and correcting a branch limitation for ind.branches.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

replace #!/usr/bin/perl, lost on a recent commit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

default results per page for OPAC authority search now 20

Previously was 19, which is an odd number.  20
is consistent with the staff authority search.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

removed 'AddStatement' op from authorities search

The AddStatement op is an archaism from 2.2 that
has not been used in the templates for at least a
year.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bugfix - don't allow deletion of authority records from OPAC

Because of some copy-and-paste work, it was possible to
construct an OPAC URL that would attempt to call DelAuthority
on a selected authority record.  This is perhaps taking
patron contributations to library metadata a bit too far.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2205 - fix 404 when traversing pagination link in OPAC authorities search

Currently, the heading sort order is a required parameter, but was
not supplied to the template or used in the search results.  As
a result of this patch, clicking a link to view another page
of the authority search results will now work.

Outstanding issue not addressed by this patch: when searching
authorities in both the staff interface and the OPAC, choosing
sort order "none" results in a crash.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Fix date filter on issues_stats.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove vestigial checkbox for label-batch creation on MARC import

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Update issues_stats to use authorised_values instead of distinct(sortn); add col/row headers as Descriptions instead of codes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Update issues_stats to show auth_vals as row & column headers

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add Collection code limit to issues_stats report.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Update issues_stats: add shelving location limit, replace cgi::scrolling_list, update terminology.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2192 - set tabs for four system preferences

OPACItemHolds => OPAC
AutoEmailOpacUser => Patrons
AutoEmailPrimaryAddress => Patrons
serialsadditems => Acquisitions

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

bugfix - HTML escape syspref explanation

This applies to the list of system preferences in
the syspref editor - the OPACUserCSS syspref had
angle brackets in its description.

Used template change to accomplish this.  Note that
syspref value is escaped by the script, note the template.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fix to display itemcallnumber on opac brief display

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

adding some missing packages to about.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

few changes in UNIMARC record.abs

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

BUGFIX: the language list was calculated on opaclanguage, whatever the interface

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Interactive debugging aides for SIP abstraction layer.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Major SIP components reworked.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Expanded SIP config file.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Object oriented module is not an Exporter.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add debugging around readline.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

SIP tests for renew and renew_all.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Debugging util. script for SIP config XML.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Tweak verify_cksum.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

SIP abstraction layer implementation for ILS, Items and Patrons.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Abstraction layer implementation for Transactions.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add check to resensitize sub.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Object oriented module is not an Exporter. 2 subs added, also.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correct package name.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Object Oriented module is not an Exporter.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add do_renew_all function, necessary for implementation.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Whitespace cleanup and one commented-out function added.

Verify with: git diff -w  C4/SIP/Sip/Configuration.pm

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Trivial POD revision.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

SIP/t - test files adapted, supplemented, expanded. Note: requires matching data to be loaded.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

SIP/koha_test - remove duplicative tests

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Convert to using loop_context_vars for rows instead of highlight/toggle/whatever.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Remove bad use of map in void context, exclusively for side effects!

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add Debug for warns and remove unnecessary middleman variables from returns.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Show copy/volume info on STAFF detail page.

This change helps the staff side catch up w/ the OPAC display and be
more useful to librarians dealing with serials.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bugfix is_approved() to handle the 3-state nature of approval.

That is, a term is either approved, or rejected, or neither.
Without an external dictionary, most terms will fall into the
latter category.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

minor changes in french translation

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

french translation updated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

re fixing opac-suggestion < misplaced

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

html fix (detected by translator)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

small fixes in french translation

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

fix to have the branch search on OPAC being really a branch search, not kw

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

display itemnote on check-out (added, was not here) & check-in (red-ed, was already here)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Add MARC preview to cataloguing search, and add a link to add holdings. Once the edititems permission patch is accepted, this page should be updated to require edititems instead of editcatalogue.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

add single-transaction flag to mysqldump in example backup script.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Restrict OPAC tags display to approved=>1.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Correction to: Comment out confirmation popups (on successful events).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Comment out confirmation popups (on successful events).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - settle on application/xml Content-type for feeds

Required by IE6 to at least get a feed to display as XML
in the browser instead of being downloaded.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - send correct Content-type for search feeds

OPAC search RSS and ATOM feeds now have the correct
Content-type sent - "application/rss+xml" and "application/atom+xml",
respectively.

As part of this patch, added an optional fourth parameter
to C4::Output::output_html_with_http_headers to specify
the content type.  If that parameter is now supplied, or if
the value of the parameter does not contain at least a "/",
the default type of "text/html" is returned.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - removed ersatz ESCAPE="html" added in prior patch

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - fixed channel links in Atom and RSS

query_cgi already includes the "q=" bit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

OPACBaseURL - do not tack on :443 if using SSL

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2078 - fix feed validation errors

Fix several validation errors in RSS and Atom feeds
generated from the OPAC:

- add missing guid to RSS elements
- add missing feed ID and element ID to Atom elements
- reflect OPACBaseURL changes
- fix atom:link self links
- add HTML escaping to fields comming from bib record
- set default timestamp for Atom updated elements

Issues identified but not solved in this patch:

- setting Atom updated element for each bib (presumably
  from biblio.timestamp)
- possible problems performing paging of Atom feed

Based on successful validation of feeds by feedvalidatory.org,
it is expected that feeds should now work in IE7 and IE6.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

calculate OPACBaseURL instead of using syspref

Calculate OPACBaseURL from the
CGI environment variables instead of using the
system preference.  As a result of this patch,
it should be possible to change the hostname,
protocol, or port of the OPAC without having to
reset a system preference.

Also added a FIXME to opac/unapi - the URL of
the SRU/W server has no necessary relationship
to the URL of the OPAC.

Once this patch is confirmed, the syspref can
be removed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

kohabug 2180 - improve display when fines block checkouts

If a patron's fine balance is over the limit set by the
noissuecharges syspref, checkouts are blocked.  However,
this was not made very clear in the interface.

Improved the display by doing the following:

[1] If the fine balance is over the noissuecharges limit, use the "blocker"
styling (i.e., make all of the text red).
[2] Add wording to explicitly signal whether or not the fine balance is
blocking charges.

Documentation changes: new screenshots of blocked checkouts.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Apply item-level_itypes syspref to MARC export.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>