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>
Conflicts:

C4/Koha.pm
catalogue/search.pl
opac/opac-search.pl

Removing strange dot file

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

bug 2503: tests for C4::Circulation

I'm adding some tests for C4::Circulation methods that I'm altering
to allow the offline circulation tool to use C4::Circulation to upload
its data. These test a bit of the old functionality and try to show
that the new functionality does what I think it does.

C4::Circ::Addissue to tests issuedate
 these also test AddRenewal.
tests for C4::Circ::MarkIssueReturned

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

bug 2503: adding dates to some C4:Circulation methods to allow offline circulation tool to specify dates other than today

When uploading an offline circulation file, some actions may need to take place
in the past. This patch adds date parameters to some C4::Circulation methods
to allow us to set dates on them

added issuedate to C4::Circulation::AddIssue
adding issuedate to C4::Circulation::AddRenewal
adding returdate to C4::Circulation::MarkIssueReturned

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

bug 2530 : create new date object in javascript function instead of modifying passed-in reference.

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

bug 2526: removing DEBUG flag from C4::Search

The DEBUG flag was left turned on.

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>

bug 2523: add columns to holds queue report

* added author, ccode, location, and item enumchron
* sorted by collection, locatoin, call number, author, title

As part of this patch, commented out the JavaScript tablesorter,
which could cause the browerer to lock up if the picklist
contains more than a couple hundred items to pull.
Some sort of pagination is clearly required, but since
it is necessary for customers to be able to print out
the picklist, there still needs to be an option to
see the complete list for a branch.

This patch includes the final component of the fix
for bug 2331.

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

Corrections to manage marc import interface, including fix for inaccurate progress bar.

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

bug 2341: OPAC title-level holds permitted when items are on-order

Changed so that the presence of an on-order item (items.notforloan =- -1)
permits a title-level hold request to be made.  Prior to this patch,
if all of the items linked to the bib were on-order, no title-level
request was permitted.

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

bug 2520: don't change item-level requests to bib-level

Prior to this fix, changing the priority of any hold requests
in the queue of requests for a bib would convert
all item-level requests to title-level requests.

As part of this fix, improved display of the holds queue
so that item-level and bib-level requests can be more
clearly distinguished.

Documentation note: new screenshot of list of requests for
a bib in the staff interface.

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

bug 2521: distinguish types of hold requests

Title-level and item-level requests are now more
clearly distinguished in the patron details page
and the patron checkout page.

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

bug 2518: remove useless patron and branch lookups

AddIssue does not use the results of the lookup
of the patron whose hold request was preempted
by the issue.  This patch now leaves a no-op in
the '$restype eq "Waiting"' test, but I'm not going to
refactor the hold cancellation logic now.

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

bug 2519: clear waitingdate if request is moved off waiting status

* Changed C4::Reserves::ModReserve so that if using it to take an
  item off the hold shelf, reserves.waitingdate is cleared.
* Improved POD.

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

Adding javascript to prevent premature submission of forms when using a barcode scanner (or by hitting the enter key). Addresses Bug 2591

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

Reduce logging from rebuild_zebra.pl with a command line option

This reduces the output of the script and zebraidx, and creates a -v
command line switch which will increase the logging to their former
states.

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

Eliminate left-column margin on print view

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

BUGFIX missing div

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

fix date formatting in overdue_notices.pl

This makes overdue_notices.pl format dates in fields that both start and end with 'date'.

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

Bugfix #2630 2nd attempt

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

Markup and formatting improvements to Manage Staged Records interface.

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

Bugfix #2633

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

bug 2615: remove unneeded 'require Exporter'

Most Perl scripts (as opposed to modules) do
not need to require Exporter.

No user-visible or documentation changes.

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

bug 2609: display correct requested title

In databases where biblionumber is not necessarily
the same as biblioitemnumber, the title of
certain hold requests displayed by moremember.pl
could be drawn from the wrong record.

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

Bug 2365 : Inner counter not properly set for serials subscriptions not starting on first issue.

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

fix for bug 2556: Overlaying records sometimes creates new record

This makes "Try another search" in the Z39.50 search tool pass the biblionumber along, so that records are overlaid correctly.

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

Corrections to fix javascript error in Internet Explorer.

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

Altering system log search form to conform with other similar forms; escaping output of "info" variable.

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

moredetail.pl is not an Exporter.

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

Remove acquisitions permission requirement on catalogue/showmarc.pl

This script is linked to from z39.50 search as well as acquisitions.
cataloguing/z3950_search.pl requires only the 'catalogue' flag, so
requiring only that permission here.  A user without acquisitions permissions
would get a login instead of the record display without this change.

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

Sort libraries by name in holds queue's select input.

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

Change missing from previous commit re:Bug 2600.

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

Markup correction; Fix for Bug 2600, "HTML tags in titles not escaped in list of returned items."

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

Refining keyboard shortcuts javascript.

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

Some formatting fixes for bookcount.tmpl and issuehistory.tmpl. Linking to moredetail.pl from issuehistory.pl (just as barcode on detail.pl links to moredetail.pl).

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

bug 2596: remove URL escaping added by previous patch

If URL in an href comes from a MARC record, it should not
be URL-escaped - otherwise, link is interpreted as relative
to MARCdetail.pl.

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

Fix for Bug 2596, "restore ability to hyperlink URL fields in MARC detail display"

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

Fix for Bug 2592, "hyphens being added to marc view isbn." Also removing truncation of subfield descriptions.

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

bug 2588: fixed several links to open in new window when OPACURLOpenInNewWindow is ON

Fixed several links on the OPAC Detail page to open in a new window when the OPACURLOpenInNewWindow system preference is ON.

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

Can't escape title variable as HTML because of search term highlighting.

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

Trying again: Standardizing styling of error messages; other markup corrections.

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

fix extra <td> introduced in previous patch

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

commenting out debug warns

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

Correction to previous commit, removing debugging.

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

Formatting itemtype icon section with tabs to match authorized value interface.

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

Remove double conditional, fix one FIXME.

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

Minor cleanup, updated POD.

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

A few more en-US spelling corrections; Markup corrections; Standardizing on "email" instead of "e-mail"; Copying isbn-formatting from opac-detail.pl to detail.pl;

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

bug 2585: fix search links for names (XSLT OPAC display)

Fix malformed OPAC search syntax that caused this bug:

When using the default stylesheet for MARC21 XSLT OPAC details display, the
search links for name headings (from 100, 700, 710, 710, 111, and 711) that are
linked to authority records don't work.  When the user clicks on such a link,
they are directed to the advanced search form instead of the search results.

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

Correct application of noissues flag in SIP.

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

bug2580: category name and home library will now correctly display on all patron tabs

This patch fixes several improperly named variables and includes the home library variable on the pages that were missing it. The category
description was added to the GetMemberDetails function since a couple of the pages using that function required the variable.

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

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>