Changing version

little change on french translation

Bug Fixing: Amazon elements would not display at OPAC

Owed to a change in systempreferences name in templates,
amazon elements would not display
This patch creates OPACAmazonContent value for templates.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Add errorproof to GetCOinSBiblio

When no MARC record is returned by GetMarcBiblio for one record,
it makes search result go error 500

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

German intranet updates

Updating the translation credits for Spanish

Followup : French translation file for staff

updating translation file + bug fixing javascript string in
subscription
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Updating french translations for staff

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

minor improvement french translation

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

(bug #2904) this patch is a fix from previous

in the previous patch the field used for link "name" was 856$z, but the unimarc doesn't specify this, it say to use the 856$2 field.
This patch make koha to use the 856$2.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Minor fix in french opac translation.
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Updated circulation page in order to make things appear the right way

Strings integrated to js could not be translated.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

bug 2952 followup

The feature in question is described in bug 2952; to
summarize the enhancement, which the earlier patch
description did not do, the list of bibs in an
import record batch now includes a column linking
each import record to the bib that was actually created
or updated when the import batch was committed.

The improvements in this patch are:

* If bib in import batch has not been committed, it
  has not been linked to a matching new or updated bib.
  In that case, do not create a link to a guaranteed
  404 (/cgi-bin/koha/catalogue/detail.pl?biblionumber=)

* When reverting an import batch, set matched_biblionumber
  to NULL for affected records - otherwise, the Bib
  column will include links to bibs that may no longer
  exist.

* Fixed a minor HTML validation error.

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

bug 2952: 1 char typo

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

patch for feature 2952 - v3

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

(bug #2956) fix another results calculation

this patch fix more about 4XX plugin, for results display and calculation.

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

Thai Opac updates

Turkish intranet, Marathi Opac and Thai Opac

Updated thai opac

Updating credits

Fixed turkish file

kohabug 2955 (2/2) GetMarcFromKohaField now requires $framework

Failure to pass the $framework parameter by dateaccessioned.pl causes
GetMarcFromKohaField to return 0,0 which breaks the value builder. The
patch
submitted corrects this.

This patch is build against the current 3.1.x HEAD, however, this bug
also
affects the 3.0.x codebase and should be applied there as well.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Followup previous patch bug 2955 :

Some calls to GetMarcFromKohaField would not tell which framework to
use.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

kohabug 2955 GetMarcFromKohaField now requires $framework

Failure to pass the $framework parameter by barcode.pl causes
GetMarcFromKohaField to return 0,0 which breaks the value builder. The
patch
submitted corrects this.

This patch is build against the current 3.1.x HEAD, however, this bug
also
affects the 3.0.x codebase and should be applied there as well.

fbcbug 4

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

Bug Fixing : Modifying a patron would not show his information

Improving translation : some js strings were not taken into po files

French opac updates

Russian and Ukranian SQL files

Updated translation notes

Russian, Ukranian and Turkish translations

Followup : updating circulation.tmpl

summary checkouts were displayed even with no borrowers.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

"Cart" translation updated

"Cart" translation in french is replaced with Panier

Bug Fixing : updating circulation.tmpl

problem with displaying circulation page

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>

More norwegian updates

Norwegian updates

NZ English intranet updates

Updated Credits and lots of updates to the po files

French intranet

po updates

Adding credits to OpenNCIP and nots to release notes about platforms

Changing nongnu.org to lists.koha.org

Updating translation Minor correction to  Release notes

Adding a translation note.

Bug 2940 - private shelf (list) display error on 2nd and subsequent pages.

The before/after URL for the 2nd page of list elements:
http://atz.dev.kohalibrary.com/cgi-bin/koha/opac-shelves.pl?display=privateshelves?viewshelf=120&itemoff=2
http://atz.dev.kohalibrary.com/cgi-bin/koha/opac-shelves.pl?viewshelf=120&display=privateshelves&itemoff=2

The change in query string arguments is incidental, the prevention of multiple "?" is essential.

German updates

Followup: Adding AllowNotForLoanOverride

an orphan bracket was lost in the file.

Adding release notes for 3.0.1

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Followup : Adding System Preference AllowNotForLoanOverride

this system preference was created but not added to system preferences
Changing the kohaversion in kohaversion script

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Updating French translations

Staff trnalstaions update

Adding System Preference AllowNotForLoanOverride

this system preference was created but not added to system preferences

Updated translations for French, German and Croation

(bug #2937) use check_date in C4::Serials::hassubscriptionexpired

This patch change the date check in hassubscriptionexpired to use Date_Calc::check_date

Needed authorised values list for french frameworks.

Adding some null words and mappings for Œ œ

Bug Fixing duplicate = in labels management.

Revert "more readable version of the init and new sub"

This reverts commit 5a74b4fa0cf79163755560daed640ead85fa0b71.

Revert "Bug Fixing : Sub Output was mistakenly removed."

This reverts commit cf355ab195e19c7d9ac37f6fc563141b5190a231.

Bug Fixing : Sub Output was mistakenly removed.

French OPAC translation Update

Reverting from MARC::File::XML parser forcing

Authtypecode return an warning even if subfield is ignored

this patch just add a condition to the sql request to return a warning
only if authtypecode is not in an ignored subfield.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Bugfix #2921 OPACAmazonContent syspref not used

In opac-results.tmpl and opac-detail.tmpl, AmazonContent is used
rather than OPACAmazonContent to enable/disable Amazon book jacket.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Minor rework of template js.

Lines explicitly ended with semicolons.  Allow some better behavior if TMPL_VAR's
are undefined or zero.

Note FIXME added.  I believe this is part of the fragmentary integration of
Katipo code for GST.  Would be obliged if someone with access there can confirm.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Trivial comment typos corrected.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Basic cleanup of fixup_cardnumber

Avoid warning on syspref "checkdigit" undefined.
Functionality essentially unchanged.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Prevent js failure if syspref "gist" is undefined.

The javascript is malformed and therefore fails the onsubmit check if
gist does not receive a value.  This patch allows failover to default 0.
Also removed an unused dbh.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Block warning on detail.pl.

If Amazon doesn't have an avg. rating number defined, then we cannot multiply it
by 20 without the following warning:

    detail.pl: Use of uninitialized value in multiplication (*)
    at /home/user/kohaclone/catalogue/detail.pl line 228.

So the important part of this patch is || 0 on the $average_rating assignment.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #2917) editing authority marc field structure doesn't workl

This patch add a field in the ->execute() statement which was missing.
The sql request have 8 args, and only 7 was passed.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Autoselect marcflavour in marc import

This patch is writted by Paul Poulain.
It auto-select the marcflavour from syspref in the stage marc import.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Correct capitalization on CGI::Session::Serialize::yaml

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Fixes command-line 'number' arg in bulkauthimport.pl.

for HEAD and 3.0.x

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

fix to remove blank newlines from labels

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

_set_defaults_for_add easier to read

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

more readable version of the init and new sub

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #2914) improve the deletion

this patch disable the button to delete all of items if there is no item attached to the marc record.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #2914) delete all items of a notice

This patch allow the librarian to delete all items attached to a notice in one click.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(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: Henri-Damien LAURENT <henridamien@koha-fr.org>

Format marc notes

this patch add a filter to notes, it transform \n in <br />

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Correction: Greybox js should be loaded only if XSLTDetailsDisplay is on (not only if GoogleJackets is on).

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Basic reformat of section in Search.pm

In for loops, replace all-inclusive conditional blocks like:
    for (@foo) {
        if (test) {
            block;
        }
    }

with conditional use of "next":
    for (@foo) {
        test or next;
        block;
    }

This reduces the number of unnecessarily nested blocks, and the
overall number of lines.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #1578) change the shelf COinS support

this change the old COinS support, and now use C4::Biblio::GetCOinSBiblio().

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Add mc-loc definition for ccl.properties.

Advances search limit by shelving location doesn't work due to
missing ccl definition in default installation.  Once updated,
the zebradb will need to be reindexed.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

bug 2505: enable warnings for labels/label-print-pdf.pl

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #2893) Allow loan forcing if a syspref is set

If the syspref 'AllowNotForLoanForcing'(YESNO) is set to YES, the librarian is able to force a loan on an item set as "not for
loan".
If the item is not for loan and the syspref is set to YES, koha will ask to the librarian if he really want to check-out it,
else do nothing.

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

(bug #1578) Add the (quasi)full COinS(Zotero) support

this patch do a lot of things :
 * Add the function GetCOinSBiblio in C4::Biblio, return the COinS value that must be put in a span title
   This will be usefull for OpenURL support.

 * Replace the span in opac-detail, and put the var get from GetCOinSBiblio
 * Fix an <img> tag that wasn't autoclosed
 * Add the COinS in search result
 * Full COinS support of UNIMARC, the MARC21 support should be improved to provide all kind of document types

Signed-off-by: Henri-Damien LAURENT <henridamien@koha-fr.org>

Updating updatedatabse in order to work in any installs

(bug #2909) fix the attachment base64 encoding

this patch change the attachment encoding, else if the file is too large, koha throws an error.
This solution was found on koha list in a Joshua's e-mail

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

Unescape some double quotes, that make a validation error.

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

(bug #2904) support of 856$u and 856$z in UNIMARC

This patch change the linktext as the label of the link(in 856$z), and set the url from the 856$u.

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

Bug 2902 Public shelves do not display when user not logged in.

An ending </div> was not placed correctly and encompassed both
privateshelves and publicshelves

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

Prevent multiple warnings per notice filling cron logs.

If a value being substituted in was NULL, then warnings like this
would be emailed to the crontab owner (or MAILTO):

Use of uninitialized value in substitution (s///) at /home/user/kohaclone/C4/Letters.pm line 508.

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

Conflicts:

C4/Letters.pm

bug 2505: more warnings fixes in C4/Context.pm

* more portable way to determine uname for
  crash messages
* C4::Context->userenv now returns undef instead
  of 0 if no userenv has been set yet; returning
  undef when necessary is almost always better
  than returning a scalar when a hashref is
  normally expected.

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

Enable use warnings;, clean up complainy functions, fix a few tab/space formatting issues.

Refactoring KOHAVERSION to be less crufty, enabling use warnings, change formatting slightly in places.

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

WEBSERVER_HOST and WEBSERVER_HOST added

comments added too

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

followup to patch for bug 2900

Previous patch broke date comparison that sets
overdue flag - now that ISO-formatted date
strings are being compared, must use "it"
instead of "<".

"2009-01-02" lt "2009-01-08" is true
"2009-01-02" <  "2009-01-08" is *false*

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

Bug 2900: fix GetPendingIssues.

GetPendingIssues did several bad things:
~ select * on a 4 table join,
~ including multiple namespace collisions,
~ including large fields marc and marcxml from biblioitems,
~ return ($count, \@array_being_counted).

Not everything is fixed here (see FIXMEs), but the situation is
improved considerably, with bug 2900 resolved.  The "timestamp"
namespace collision in query should be resolved by separate patch.

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

bug 2747: avoid warning if items.itype is NULL

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

Bug 2747 for staff interface itemtype display.

Prefer accuracy over graphical icon.

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

GetAuthValCode calls were positioned inside the loop,

despite the same values being supplied each time.  Then
the conditional assignments would repeat the same calls again!
That means execution was liable to query the DB at least once
and as many as four times per item.  With a large number of items
this is an unnecessary burden.  By moving the calls outside
the loop, we can guarantee that we never have to call the DB for
that info more than twice (once for lost, once for damaged).

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