perltidy finishreceive.pl and two comments added

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

Bug 3090 - Add perldoc details

This is half of the fix for 3090.

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

parcel.pl cleanup

It looks like a lot of change, but it isn't much:
perltidy, removed unused "toggle" code, and corrected comments.

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

orderreceive cleanup

Convert to GetBranchesLoop, remove unused variables.
Pull out params that are assigned the same value in each loop and just assign them once.
Pull out params that are assigned the same value in each half of the conditional, and just assign them unconditionally.
Remove useless call of $template->param($count).

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

acceptorreject.pl cleanup

Perltidy the ugly block (still ugly).
Use elsif where appropriate.
Added FIXME.

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

Factor common element out of both sides of conditional.

Add placeholder complaint about misuse of TMPL EXPR statements.

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

Remove unused TMPL variable.

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

Fix for Bug 3095, Add to lists page doesn't redirect with no javascript

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

Adding new "tag" icon to tag link in lists interface. Also removing an unwanted ID from opac-shelves.tmpl markup.

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

More fixes for Bug 3044

Hiding the results page's "Add to Cart" links with CSS and revealing them with javascript; Writing the "Add to Cart" link to the opac-detail page with js using the biblionumber embedded in the Export form.

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

(bug #3053) extract ISBD view generator, and permit to display valuecode in ISBD view

this patch extract the generator of ISBD view in a C4 function, because it's used in intranet AND in OPAC but with 2 separate codes.
Now it's more reusable.

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

add the possibility to replace, in sysprefs, the code by it's valuecode with {{code}} ex : {{995c}}

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Add warning for unrecognized sort order.

This may help elucidate problems we are having with sorts that
do not seem to work.  I.E., if the inteface is requesting
unrecognized sort orders, we need to know about it.

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

remove incomplete bib bulk editing code

Per discussion I had with Henri, removing experimental
bulk editing from the staff search results code, as
feature is incomplete and can be dangerous if
one tries to use it on a large search result
set.

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

Writing Cart button to DOM with inline script. Fixes Bug 3044 without reintroducing Bug 2887 ("Cart is not translated").

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

3040 : seriestitle mistaken as collection title

If series title was previously taken as collectiontitle
then you should launch
misc/maintenance/UNIMARC_fix_collectiontitle.pl
misc/batchRebuildBiblioTables.pl

[RM note: applies only to UNIMARC users - MARC21
users should not run the batch jobs.]

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

New styles for bulk hold and bulk tag inputs on search results page.

This change gives the buttons a link style like is used on the lists page. My hope is that the similarity will help the user recognize the function, and the differentiation from the "Save" button will improve the clarity over having three similar buttons.

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

Another try at fixing Bug 2949.

This fix is in circulation.tmpl instead of the search box include. That eliminates the clash with returns.pl. Focus works correctly when checking out, when circulation.pl is at its "empty" state, and when patron selection form is displayed.

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

restored tab-delimited report download

Previous patch broke this, always providing the
guided report download in CSV format regardless of
whether CSV or tab-delimited was requested.

Also, default name of downloaded file is reportresults.csv
for CSV output and reportresults.tab for tab-delimited.

Also removed a couple subroutine prototypes that
were leading to unnecessary compile-time warnings
under use warnings.

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

Bug 3066 - Overhaul guided reports

execute_query now refactored, returns reliable results, does
zero presentation-layer crap.  Arguments reduced, client scripts
adapted to new API and performance improved.  Text::CSV now used
to generate CSV output, ensuring portability, encoding and accuracy.

Replaced tools/runreport.pl with misc/cronjobs/runreport.pl:
    ~ security fixed
    ~ documentation improved
    ~ TODO: finish sendmail option.

Bug 3077 also fixed.

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

Condense inarticulate construction of TMPL structures.

Use map to replace multiple temporary hash constructions.
Also a minor POD fix.

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

runreport.pl and prep work on Guided.pm

The old location of runreport.pl was under tools, leaving it exposed
to web requests.  This is a security flaw since it does NOT check any
Auth and allows the user to request any Saved Report be run.  This is
not a problem under misc/crontab/ and it suggests the more appropriate use.

Guided.pm is not fixed here (see bug 3066), but it is prepared to be fixed
and made compatible with runreport as detailed in the perldoc.

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

Serial IF statements on same $variable should be ELSIFs.

Small POD update also.

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

patch for bug-3017

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

(bug #2862) improve sql reports

Clear all non-alphanum chars at the end of an sql query.
Use ESCAPE=HTML to escape double quotes in csv export

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

Internationalize Amazon support

With this patch:

  * On Amazon book cover, 'search inside' text is not displayed
    anymore because it's not translatable.
  * Link the image to local Amazon web site 'search inside' service.
    Based on AmazonLocale syspref.

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

Updated INSTALL.debian

Modified INSTALL.debian to update install and web
configuration instructions.

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

Changes to installed indexing.

These changes tidy up ISBN and ISSN indexing, per Michele Maenpaa.  It's being
set up manually on many new installations, and probably ought to become part of the default
Koha installation.

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

Changes to bulkmarcimport.pl

Adds three new switches:

-idmap <filename> - optional output file of
                    map of source record ID numbers
                    to Koha biblionumber
-x                - if idmap is supplied, MARC tag
                    to get source record ID from
-y                - if idmap is supplied, MARC subfield
                    to get source record ID from

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

Modifications to C4/Matcher for colon and ending punctuation

Recommended by Michele Maenpaa, this adds handling colon and end-punctuation stripping
in subroutine _normalize, and fixes length testing in subroutine _get_match_keys

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

Added ISBN to RSS results.

Added ISBN tag to RSS results, right after the title.

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

tweek css for staff search results item type images

This add a clear:left; to the #searchresults ul li selector.  Otherwise
it's possible an item type image will be offset by an image above it and
not float all the way to the left.  This makes sure the image is clear
of the above image.

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

DDCN callnumber splitting with test.

Similar to previous patch for LCCN splitting, this patch incorporates
changes to split_ddcn and supplies a test file for verifying proper
operation.  Note that the only previously documented example for intended
operation is included as one of the tests.

This regexps are created to be rather forgiving.  For example, the function
will not choke if two spaces were included where the "spec" (such as it is)
expects one.  Obviously this is because for CN splitting purposes, it doesn't
matter, we're not going to ever split in the middle of whitespace.

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

Corrected: Hiding renew link in the OPAC for items which cannot be renewed. Fixes bug 3083.

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

(bug #3084) in baskets, change the isbn of column

this patch add a new column and put the isbn in it.

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

Adding details to OPAC search results RSS, described in Bug 2973.

This includes a change to Search.pm to add a variable, subtitle_nospan, in order to display subtitles in the feed without search term highlighting HTML. The modified template attempts to display Amazon, Syndetics, or Baker & Taylor book cover images. Google images are not included because they require Javascript.

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

bug 3088: fix crash in opac-ISBDdetail.pl

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

fix type of new serials display system preferences

OPACSerialIssueDisplayCount and StaffSerialIssueDisplayCount
should be integers, not free.

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

Cleared a trailing warn, sorry.

Signed-off-by: Daniel Sweeney <daniel.sweeney@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Adds a 'New Subscription' option for selection from the detail page of a record.

NOTE TO DOCUMENTATION WRITERS: Due to the limitations in the serials module search
interface, some users may find it easier to use the standard catalog search to locate
an item on which to create a subscription record, and then use this new button to
create a subscription from the detail page.

Signed-off-by: Daniel Sweeney <daniel.sweeney@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for Bug 424: Serials display: installer does not make database changes for display parameters, so can't save new subscriptions

Signed-off-by: Daniel Sweeney <daniel.sweeney@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Serials Display Enhancement

Allows specification of how many issues of a subscription to display
at a global and subscription-specific level. Also adds a link to the
detail page to a specific subscription.

Signed-off-by: Daniel Sweeney <daniel.sweeney@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

fix syntax error in tags JavaScript

FF will tolerate a trailing comma in an
array literal, but Internet Explorer does not.

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

bug 3034: Tag multiple items at once.

Added a "Tag" button in the toolbar area of the results
page.  This activates an inline form that allows the user
to enter a tag that can then be applied to all currently
selected items.

In addition, a "Tag" link has been added to both the List and
Cart pages that provides the same functionality in those contexts.
In these two cases, the a per-item states appears below the
bibliographic information for each item.

Status that applies to the overall operation, such as failing to
provide a tag, will be shown in an alert dialog box.

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

added dependencies for new OAI-PMH server

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

Add a UNIMARC to Dublin Core XSL

This XSL transforms XML UNIMARC biblio record into
its DC representation. It is the equivalent of
MARC21slimOAIDC.xsl.

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

Add another OAI server

This OAI server has those characteristics:

   * It doesn't use C4::OAI anymore.
   * It uses HTTP::OAI library which is easily expandable.
   * XML result is not build anymore directly but using
     XML::SAX::Writer. It should solve encoding and XML related issues.
   * It returns two formats: oai_dc and marcxml.
   * DC is generated from biblioitems.marcxml by applying XSLT
     (MARC21slim2OAIDC.xsl or UNIMARCslim2OAIDC.xsl).
   * It uses the same tokens as previous OAI server (thanks Philippe
     Jaillon).
   * There is no more sets (it wasn't operational in previous version).

Add two new dependencies:

   * HTTP::OAI
   * XML::SAX::Writer
     available as a Debian package: libxml-sax-writer-perl

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

add } that slipped out because of bad merge

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

Bug #3071 - Z39.50 search result highlights are mixed

This patch modifies z3950 search result page in order to
use jquery to highlight rows rather than an explicit
css class.

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

Add Babeltheque OPAC enhanced content

www.babeltheque.com is a LibraryThings for Libraries equivalent.
This patch enables this service on OPAC.

  * Replace previous patch in order to apply onto last master HEAD
    and solve conflicts induced by last pushed pateches:
    1bb9f76e66fd962930007964a2f918665dc3a956
  * Modify kohaversion.pl
  * It uses v3.01.00.016 slot.

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

Fix Amazon cover for UNIMARC

In UNIMARC, 010 tag contains ISBN. In UNIMARC, ISBN with '-'
are valid. This patch modify ISBN normalization in order to
automaticaly suppress '-' before sending ISBN to Amazon
to get book cover.

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

Reformatting receipt template and CSS to be more readable.

A narrow printable area is unsuitable for a three-column table of information. Changing to paragraph sections for each item. Also correcting terminology: issue -> checkout

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

BUG-1503: fixes broken matching for 'hidden' values.

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

(bug #2913) show "--" on repeated fields

This patch just transform the script to set the field title as "--" if it was already shown.

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

fix to ignore 600$2 'Source of heading' fields for MARC21 subject display

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

Adding patron address to select patron form.

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

(bug #3057) mandatory subfield edition

modify the "name" of mandatory input field.

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

Convert opac-reserve to use GetBranchesLoop.

Removed irrelevant and unused variables dealing with branches.
Also pulled default branch from userenv when not specified as a param.

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

Improve performance and fault tolerance for sync_items...

Pulled the GetMarcFromKohaField outside the loop, since we only need to
check it once.  Same for $item_sth.  Added safety checks for success of
delete_field and GetMarcItem, with warnings on failure.

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

cleanup some of value_builder MARC21 plugins

Tons of unnecessary dependencies removed.

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

perltidy some ugly value_builder plugins

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

whitespace cleanup and remove editor comments

No functionality affected.

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

(bug #3063) fix unimarc 4xx plugin with cloned field

this patch fix the unimarc plugin 4xx to work with cloned fields, the id of the div wasn't replaced when the field was cloned.

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

Instructions and packages required for Debian Lenny Installation

INSTALL.debian-lenny
    Instructions for installing Koha on Debian Lenny.
    Based on INSTALL.debian. Almost all Perl modules are now available
    via Debian packages. MySQL installation is simplified.

debian-lenny.packages
    List of all Lenny packages required by Koha.

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

Bug #3036 Avoid command-line installer blocking on opensolaris plateform

Koha installation/upgrade on OpenSolaris 'seems' to work.

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

fixes error below...

 bookcount.pl: Illegal Date 'Item has no branch transfers record' does not match 'iso' format: yyyy-mm-dd at /home/mason/git/kap-k3-2/circ/bookcount.pl line 88

tested on 3.0.x

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

fixes many 'Use of uninitialized value in string eq at ...' warnings.. at admin/branches.pl line 242.

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

Changing "Waiting to be pulled" to "Pending" as per Bug 3033 'Holds "Waiting to be pulled" falsely implies that items are available'

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

Minor bug fix Documentation update : 3039

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

Hiding labels for add to list options when user isn't logged in or options don't exist. Fixes Bug 3043.

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

quietens a noisy debug pop-up, when debugging

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

Bug 2958 - notforloan (8008) is a numeric field.

Update index to allow searching for "On Order" items, -1 value.
Example:
http://atz.dev.kohalibrary.com/cgi-bin/koha/opac-search.pl?q=ccl=notforloan:-1

Without this patch, search for "-1" is normalized to "1".

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

Invalid field 'expiry' referenced.

This caused a warning from C4::Dates on every execution, since the value
passed to format_date was undefined.  Now we reference the correct field
and check for a populated value before trying to reformat it.

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

Bug Fix : weight field was broken in UNIMARC

Title-cover was not defined in record.abs
So the relevance ranking was broken.
This patch corrects that
For UNIMARC people, please reindex

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

Fix for Bug 3062, "place hold on lists different font"

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

convert '&' to '&amp;' in Syndetics URLs

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

DB rev 014 -> 015

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

Changed syndetics base URI from syndetics.com to syndetics.com

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

Added upc to content_identifier_exists criteria.

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

Added content_identifier_exists test to opac-detail.tmpl to prevent broken images in Syndetics content for some titles.

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

Fix for LL Bug 405: Syndetics Enhanced Content: some cover imags are being generated as IMGs but show as broken images

Adds new template variable to test for enhanced content identifiers

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

Fix for LL Bug 404: Syndetics Enhanced Content: Excerpts show with embedded HTML tags and entity references

XMLout needed the NoEscape option set

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

Syndetics Enhancement: adds upc/oclc to the functions to retrive non-image data

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

Syndetics and Amazon bugfix enhancements

This patch resolves a number of problems related to Enhanced Content:

1. Previously, there was no syspref for controlling whether or not to display
Amazon.com cover images apart from other content. This had the side effect
of preventing use of amazon.com content alongside use of another provider's
images. This patch introduces AmazonCoverImages and OPACAmazonCoverImages,
and changes the name of AmazonContent to AmazonEnabled.

So, for instance, you can now enable OPACAmazonSimilarItems yet utilize
SyndeticsCoverImages for displaying those similar items.

NOTE TO DOCUMENTATION TEAM: please update references to AmazonContent
and OPACAmazonContent to comply with the above.

2. Fixes some semantically incorrect uses of AmazonContent (now OPACAmazonEnabled)
on the OPAC side.

3. Resolves once and for all, the normalization of ISBN,UPC,EAN and OCLC numbers
for all enhanced content elements; These elements can be normalized using the
new functions in C4::Koha; I've replaced use of the various previously used
variables $xisbn,$norm_isbn,$clean_isbn, etc with $isbn, and the template
variable normalized_isbn.

We finally have a single, consistant place to retrieve normalize values for
these fields given a particular record.

4. Adds Syndetics attribution statements to display of all Syndetics content
'enhanced content provide by Syndetics' or 'Enhanced Description from Syndetics'

5. Adds an option to view the large cover image on the detail page on the OPAC
when using SyndeticsCoverImages; this option is controlled by a new system
preference: SyndeticsCoverImageSize which has two values: MC (medium) LC (large)

6. Adds UPC and OCLC numbers for Syndetics enhanced content queries especially
helpful for finding enhanced content for DVD and Music materials

7. Adds capability to display Syndetics images to opac-user for checkouts and overdues

8. Updates to systempreferences.sql, and updatedatabase.pl database revision 015

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

Second patch to address LL Bug 367 Titles without ISBNs display "no cover image available" while titles with ISBNS that don't match a Syndetics cover display nothing

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

Add Syndetics AuthorNotes, Awards, Series

This patch also includes a new function to retrieve the index
page from Syndetics and parse it for available content prior to
retrieving content; this is done to speed up syndetics content
by not retrieving content that doesn't exist for an item. However,
Syndetics continues to be a very slow service compared to Amazon.com
and other enhanced content services

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

Fix for LL bug 367 Titles without ISBNs display "no cover image available"

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

Adds Syndetics Reviews

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

Adds support for Syndetics Excerpts and Editions

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

Add Syndetics Summaries and TOC

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

Adding Syndetics Cover Images to OPAC Detail Page, including shelf browser

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

Adds Syndetics Cover Images to Results Page on OPAC

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

Add System Prefs for Syndetics to Auth

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

Move Amazon.pm to External/ directory

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

Database Revisions for Syndetics Solutions Enhanced Content

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

Fix for Bug 3031, Standardize OPAC login form labels.

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

fix error introduced in a previous commit

Change to GetReservesFromBiblionumber() had effect
of causing all requests of constrainttype other
than 'o' to not be included in result.

while ($foo) {
    $bar or next;
    # do stuff
    # do other stuff
}

is not equivalent to

while ($foo) {
    if ($bar) {
        # do stuff
    }
    # do other stuff

}

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

bug 3028: remove reservefix.pl cronjob

Removed cronjob, which was written to work around
a bug in 2.2 that no longer applies and is specific
to a single library in any event.

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

Test for LCCN splitting.

This test establishes what LCCN splitting is required to do
to be considered successful.  See Bug 2691.

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

Correcting delete patron button to use TMPL_VAR instead of DOM query. Fixes Bug 3014, Can't delete restricted patron from circulation screen.

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