If itemnumber is sent to catalogue/moredetail.pl use it

If itemnumber is part of the url for catalogue/moredetail.pl
put it to use.
This happens in members/boraccount.pl if a fine is attached to a
barcode.

This patch makes moredetail.pl show only the item specified by
itemnumber.

Implements some changes to the patch suggested on
patches@koha.org
give visual cue on UI and a link out.
More friendly pruning of the other items.

Thanks to Galen Charlton and Joe Atzberger

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Revert "If itemnumber is sent to catalogue/moredetail.pl use it"

Applied wrong version of patch.  Oops.

This reverts commit 9eb1465b2874a8a65195065bed083d26614ca287.

Fines fixes: apparent problems with fines prevent processing.

CalcFine returned values that mismatched expectations in fines.pl.

fines.pl refactored: added debugging, prevent needless recreation of
Calendar objects by storing them in hash by branch.
Still outstanding problems with fines, including the output of a field
that has no other references in Koha (so is always undef) and the
incorrect description of FinesMode.

Calendar exported "new" erroneously.  I also cleaned up the queries to
avoid needlessly compiling additional statement handles.

Please test and consider application to 3.0 maintenance.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

If itemnumber is sent to catalogue/moredetail.pl use it

If itemnumber is part of the url for catalogue/moredetail.pl put it to use.
This happens in members/boraccount.pl if a fine is attached to a barcode.

This patch makes moredetail.pl show only the item specified by itemnumber.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2466: regression test

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2466: fix clearing item field

When editing an item in the cataloging and serials
item editors, clearing a field now works.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Usage option correction (-h not -n)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2419: require Exporter

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2552: correct issue history display

The loan history for a bib can be incorrect if the bib's
biblioitemnumber is not the same as its biblionumber.
This was due to a bad join in GetBiblioIssues().

Credit to Michael Hafen for patching a similar bug,
which inspired me to check for other bad joins.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Update INSTALL.debian to mention a2dissite default

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Style correction in member entry form; Enabling intranetuserjs in doc-head-close.inc.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix number two for bug 1204: Lists of issues and overdues on opac-user.pl

This makes opac-user.pl sort the output from GetPendingIssues itself, by descend
ing due date.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug: bookcount.pl pulls wrong title. Problem is sql joins on wrong columns

the sql seems to assume that biblionumber and biblioitemnumber are always the same, which in my case they aren't.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for Bug 2424: Altering layout of advanced search screen to make it compatible with Internet Explorer 6. Documentarians: this will require an updated screenshot.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for Bug 2548, "Links to Reports that No Longer Exist in Side Menu." Some links were left behind after removal from reports-home.tmpl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Spelling correction. Fixes Bug 2501.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2545: fix XSLTDetailsDisplay

Corrected so that when XSLTDetailsDisplay is ON but
XSLTResultsDisplay is OFF, the portion of the OPAC bib
details page that's derived from the MARC bib record
is no longer missing.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix itemtype image problem in moremember.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2509: fix file permissions

Following suggestion by Vincent Danjean for Debian
packaging, 0755 -> 0644 for non-executable
files.

Also removed shebang from a few modules in C4.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2540: allow download of report

$error return C4::Reports::execute_query is
always defined.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix for bug 1101: Renew All for the OPAC

This adds a Renew All form to the OPAC, and updates opac-renew.pl to take multiple item= parameters.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix for bug 1771: Template errors with remote itemtype image

This adds a new function, getitemtypeimagelocation, that returns the image
unmodified for absolute urls and returns the proper intranet or opac path
otherwise. It also updates all of the relevant files to use that function.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix for bug 2488: OPACItemsResultsDisplay/singlebranchmode

This patch changes the template slighly so that these two system preferences do not conflict.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for minor error in shelfbrowser queries

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix for bug 2477: Improve Speed of the Shelf Browser

This patch updates the queries for the shelf browser in opac-detail.pl for a
slight performance gain. It also adds a param for dateformat to C4::Auth to
fix a warning.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Still fixing permission and first line of scripts

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

No 'BOM' marker for a script file

'#!' must be the first characters

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Shebang line must be the first of the file

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Add support for DESTDIR

Distributions (such as Debian) needs a staging area when building the package

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2503: documentation improvements in C4::Circulation

These are some documentation changes that I made to C4::Circulation as
I was writing tests for it.

moved around the docs for CanBookBeIssued. It was in the wrong spot in the file.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix for bug 2528: installation fails with confusing error message

This makes the dns check happen earlier, and fail humanely if there is a problem.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug #2429, bad HTML from unclosed <head>.

Also some cleanup of opac-showmarc script including removing bogus
dependencies (DBI, CGI methods).  Should consider using C4::XSLT if
applicable.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fixing a couple of minor markup issues.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2524: improving the submitted test reports

switching from 'prove' to 'perl' to run the test scripts when we're preparing
them to be submitted to smolder. Prove adds an additional summary at the end
of the test run which confuses the TAP parser in smolder. It thinks that an
additional test has been run, and it fails it.

Then, I added a '-' before the command in the Makefile. THat's so that even
if 'perl' returns a failing code (because some tess failed), the rest of the
rules are run and a report gets submitted. Othewise, only passing reports
get sent.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2525: fixing tests for misc/cronjobs/longoverdue.pl

This patch fixes the tests so that they include the new required parameters for longoverdues.pl.

This patch also doesn't include a test script that accidently got in the last one.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2524: Adding script to submit test reports to smolder server

This patch adds a target in the t/Makefile and a script in t/smolder_smoke_signal
to submit the results of test runs to a smolder server. It will allow users to submit
their own test results to a central location by running "make submit" from the t/
directory.

Smolder is a web-based application that aggregates and presents the output of
tests. We have an instance running at:
http://arwen.metavore.com:8000/app/public_projects/smoke_reports/1

More information about using smolder for Koha is available at:
http://wiki.koha.org/doku.php?id=en:development:smoke_reports

I intend to have the test suite run each night automatically and the results
sent to the smolder server. This will help us keep track of the current state
of the the code and possibly catch bugs introduced by changes. I encourge others
to submit their own test results if they want, especially if they are on an unusual
platform or are with a non-standard configuration.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for bug 2155: duplicate checkin box on returns screen

This changes some css and markup to make the return box at the top not show when one is on the checkin screen. It also moves the markup for the autocompleting search box to its own file.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Prevent compilation errors from complex C4 import/export.

SIP code uses UNIVERSAL modules that do not tolerate the circular deps in
C4, specifically, in Auth.pm.  Changing the order so that UNIVERSAL follows
the other imports is important to avoiding failures like:
  perl -w -e 'use UNIVERSAL qw(can); use C4::Auth;'
    "MarkIssueReturned" is not exported by the C4::Circulation module

This patch should resolve the problem earlier seen with Shelves too.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

bug 2502: correctly check patron account expiration

When placing a hold request from the staff interface,
now correctly indicate if the patron account has expired.
Prior to this patch, would warn that the account
had expired even when the expiry date is in the future,
and vice versa.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Have zebrasrv log less

Better to fill the hard drive with biblio than with zebra logs ;)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

kohabug 2507 Fixing labels search to indicate keyword rather than title search

Documentation may require an updated screen shot

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

masthead - realign terminology for Tag Cloud

Tag Cloud should link to user-generated tags, in a cloud display.
What used to be called "Tag Cloud" (ISBD subject links for UNIMARC libraries)
should be called a "Subject ...".  AFAICT it doesn't currently use a "cloudy"
display, so some other term might be more appropriate.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Working on preconfiguring for the package

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bumping patch number, new package release

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Starting work on the postinst file

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

More work on packaging

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fixing some bugs in the package, updating dependencies

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bumping patch number

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Still working

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Keeping on working on the packaging

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

More packaging work

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Files needed for packaging

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [18/18]: fixing pod syntax in C4/SIP/Sip/MsgType.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [17/18]: fixing pod syntax in C4/SIP/ILS/Item.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [16/18]: fixing pod syntax in C4/SIP/ILS/Patron.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [15/18]: fixing pod syntax in C4/External/BakerTaylor.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [14/18]: fixing pod syntax in C4/Barcodes/incremental.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [13/18]: fixing pod syntax in C4/Barcodes/hbyymincr.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [12/18]: fixing pod syntax in C4/NewsChannels.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [11/18]: fixing pod syntax in C4/Labels.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [10/18]: fixing pod syntax in C4/ImportBatch.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [9/18]: fixing pod syntax in C4/Overdues.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [8/18]: fixing pod syntax in C4/Koha.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [7/18]: fixing pod syntax in C4/Barcodes.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [6/18]: fixing pod syntax in C4/Serials.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [5/18]: fixing pod syntax in C4/SMS.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [4/18]: fixing pod syntax in C4/Reports.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [3/18]: fixing pod syntax in C4/Log.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [2/18]: fixing pod syntax in C4/Scheduler.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Bug 2400 [1/18]: fixing pod syntax in C4/Acquisitions.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Nomenclature cleanup prior to release

Changing nomenclature prior to release, removing -stable

Update labels formatstring parse to pick the correct item tag from MARC data.

This patch allows use of "952a" e.g. in labels format string, getting the item-level
data from the correct MARC tag.  Previously it just grabbed the data from the first item tag.

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

Update longoverdue.pl cron job to manage setting lost values.

This patch deprecates the NoReturnSetLost system preference, which, as it
turns out, was not implemented anyway.  New longoverdue script allows one to
specify on the commandline system-wide delays for changing items to different
lost statuses, and optionally charge for the item.

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

Adding some missing folks to the About page

Adding three statistics files that NPL uses, they don't
work 100% with stock 3.0 yet but are good examples of
custom scripts that libraries might want to write and
schedule periodically

Updates to the example crontab

Remove biblio.* from sample notice templates and
replace with <<items.content>>

Fix for 2492: Mail::Sendmail chokes on utf-8 [ process_message_queue.pl ]

Update fines cronjob: remove redundant scripts, remove some superfluous code in remaining fines.pl.

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

Minor update to README for RM

Updates to the subtag registry for new translations
Updates to the RElease Manager's notes
Fix Danish langauge from dk to da

Translation updates prior to 3.0-stable release

Reverting an earlier change to overdue.pl (still broken) and
fixing a translation problem with borrowers_stats.ptmpl:

xgettext.pl: Warning: borrowers_stats.tmpl: line 28: SGML "closed start tag" notation: <p<!-- TMPL_IF NAME="debug"--> class="debug"<

Release Notes for 3.0-stable

Final cleanup of labels for 3.0

This patch adds the callnum_split and text_justify options to the templates,
fixes bad javascript to switch between 'formatstring' and fixed-field means of specifying labels content,
fixes csv output when fixed-fields specifiers are used, and adds some help text for the formatstring case.

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

fix for 2382: Self Checkout Won't Work for Staff with Circ

Prevent borrower import from overwriting data that it shouldn't, and polish off a couple error conditions.

When an imported patron matches an existing one, keep data in the original record unless our import file
includes that column.  Also, check for branch & categorycodes, and produce error if invalid.  Finally,
auto-calculate expiry date if not given.

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

kohabug 2475 [3/?] Adding splitting algorithm for fiction call numbers

This patch also looks for the call number type in items.cn_source and if
that is NULL, falls back to the DefaultClassificationSource syspref.

Splitting algorithms need to be written for other known/supported call
number systems (ie. sudocs, udc, generic, etc.)

Documentation Notes:

For now, if a call number falls through the regex checking for formal lccn or ddcn,
it is passed through the regex for a fiction call number; if it falls through that,
it is returned unsplit.

If a call number does not split, please update enhancement request 2500 with the
requested information.

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=2500

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

2433: Cannot apply manual credits with an item number

This patch changes the behavior so that you can enter in
a barcode instead of an itemnumber for the manual credit
and invoice screens. Also, the borrower's account page now
displays a link back to the item record if an item was
linked to the charge

2487: Collection code select boxes in Adv search show both code and description

commented out unconditional warns

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

Bug fix : use delete_field/insert_grouped_field rather than replace_with

replace_with maight have unexpected behaviour.
Has to test merge_authority.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

Bug Fixing merge_authority.pl

merge works on the fly now.
But for an obscure reason, merge_authority.pl fails to update database when lanched on command line.
Adding one table to LOCK for noZebra UPDATE in Biblio.pm
You should remove C4::Search from merg_authority.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>

2482: Default Letter on Notice Triggers Causes Error

Changing 'Default' to 'No notice' as per solution
suggested in bug report

kohabug 2475 [2/2] Porting LCCN splitting code to Labels.pm

This patch corrects the csv field list processing alogrithm in two areas:

1. It adjusts the regexp to handle quoted fields with embedded spaces.
2. It adds descrimination for individual 952 subfields.

Documentation needs to be written for the label layout editor.

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

kohabug 2463 & 2464 Removing unlinked fields from label-edit-layout.pl

This patch removes unlinked fields from the 'Choose Order Of Text Fields to Print'
option on label-edit-layout.pl

It also adds code to make the 'Itemtype' field respect the item-level_itypes syspref.

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

kohabug 2475 [1/2] Porting LCCN splitting code to Labels.pm

This patch ports LCCN splitting code from Koha 2.2.9 to Koha 3.0
This algorithm has been ported just as it appears on some production
systems. LCCNs that do not split correctly should have a bug opened
and include an exact example so that the regexp's can be adjusted.

This patch also adds code to split DDCNs using the *loosest* possible
interpretation of DDCN rules. On the simple end, the DDCN split
algorithm will handle being passed just a Dewey call number.
However, there may be some unusually complex DDCNs that will not
split properly. These will need to have a bug submitted for them
including a specific example so that the regexp's can be adjusted.

The correct choice of splitting alogrithm is determimed by the
item level classification source (items.cn_source).

Documentation should be updated to reflect these changes. Please include
the bit about complex call numbers and the need of a bug report.

[LL Bug 26]

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

Bug 2410: Can't use string ("0") as an ARRAY ref while "strict refs" in use at detail.pl line 219

Thanks to HDL for your patch, I couldn't get it to apply so I did it manually

No change to documentation

2425: Spelling Error - Permissions