bugfix: need to return something if an itemtype doesn't have an image

I noticed that on the search pages if an itemtype doesn't have an image it's information doesn't make it out the the template.  I don't understand why, but this fixes it.

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

bug fix 2590 :

seems that when itemtypeimagelocation returns undef, the hash is messed up.
put it at the end of the hash and everything is fine.

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

fix for bug 2577: Cannot delete patron in IE

Internet Explorer seems to have a problem with JS array and object constructors that have a comma at the end. There may be other JavaScript files that have this problem.

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

Remove unnecessary EXPR.

There are two problems with this EXPR:
 (1) It's unnecessary, and
 (2) It's an EXPR.

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

HTML fixes for matching-rules.tmpl

* maxnorms, maxoffset => maxlength
* no escape attribute in <input> elements

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

HTML validation adjustments to about.pl

Corrects the Tidy errors:
line 428 column 19 - Error: "tr" not finished but containing element ended
line 428 column 19 - Error: end tag for "tr" omitted, but OMITTAG NO was specified
line 426 column 25 - Info: start tag was here
line 452 column 31 - Error: end tag for "strong" omitted, but OMITTAG NO was specified
line 452 column 12 - Info: start tag was here
line 518 column 17 - Error: document type does not allow element "ul" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag
line 525 column 17 - Error: document type does not allow element "ul" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag

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

remove debug warn

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

Validation fix, include required "type" in link tag.

This removes validation warnings like:
  Warning: <link> inserting "type" attribute

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

Documentation update (test-single not single-test)

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

Bug fixing : 2516

There was a problem with names ordering in circ/circulation.pl

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

Fixing markup errors, tweaking show/hide javascript.

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

More markup corrections for better validation.

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

Putting icon sets into tabs. Makes the interface more compact, and much better when editing authorized values for which icons are inappropriate.

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

bug 2569: added the zipcode to display along with the patron address

Added the zipcode to display along with the patron address when you are viewing a Patron from the intranet.

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

fixing permission

change link to moredetail.pl. In other places it's called with itemnumber=

Change the link to moredetail.pl from item= to itemnumber=
I saw the link on boraccount.pl as itemnumber= and thought it should be that way here too.

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

Add/Change link to item details (catalogue/moredetail.pl) for circulation and patron details

This adds the link to item details to circulation.pl
and changes the link on member details to include the itemnumber and #item... in the link.

Forgot the returns.pl.  Will do that in the next patch.

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

Added link to catalogue/moredetail.pl on the returns page for each returned item

Had to add itemnumber to the hash of returned item information.

Links to moredetail.pl with itemnumber and #item in the url.

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

modification to show the library and not the username of the person at OPAC.

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

bug 2568: set default value of intranetuserjs to blank

The intranetuserjs syspref allows a Koha administrator
to insert arbitrary JavaScript into the staff page header.
Now that this syspref is functional again, this patch
changes the installation scripts so that the default
value is blank - any JavaScript we ship with Koha should
be in the templates, not in a syspref.

Documentation changes: none per se.  However, I do want to
point out that the intranetuserjs syspref is a bit of a
dangerous one - enter a value with incorrect syntax, and
you could make it difficult to access the staff interface -
and should probably be marked as an advanced option.

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

Fix for Bug 2557, "Guarantor information - error messaee when clicking borrower# link." Also correcting a couple of minor markup errors.

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

Fixing tablesorter, display of renew links

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

Spelling corrections: organisation->organization, authorise->authorize, catalogue->catalog

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.

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