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>

javascript redirect when selecting a patron on circulation.pl Patron Selection Screen

This adds a little javascript to the Patron Selection form input after searching for a patron to check out to.
On click it redirects to checking out to the clicked patron.

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

Put patron Privilege 'updatecharges' to use on fines pages

This changes the flags required for the fines pages to include the
updatecharges flag.

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

a more readable way to test an array size.

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

bug 2505: enable warnings for admin/authorized_values.pl

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

authorized_values cleanup

Moving towards being able to enable warnings.  Use one dbh, and stop
redeclaring it in each conditional chunk.  ($sth still need to be cleaned.)
Toggle in script removed (tmpl should use loop context var __odd__).
$sth->finish calls removed where unnecessary.

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

REFACTOR: correcting indirect object method syntax

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

Bug 2505: turning on "warnings" in catalogue/detail.pl

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

Providing options for adding to order when existing record search has failed.

I've duplicated the "Add to Order" block on the search results page to display after a user has searched for an existing record to add to an order. This block is displayed whether or not results were found on the assumption that a non-empty result set may not contain the desired result. Changes to neworderbiblio.pl allow supplier name to appear in breadcrumb nav.

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

patch followup: restore test in GetMarcItem

Change made to GetMarcItem to quell a warning
changed sense of a test; defined($foo) is *not*
the same as $foo ne ''.

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

bug 2505: more warnings fixes to C4/Biblio.pm

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

Fix for Bug 2700, Keeping Data When Adding Multiple Items

When cloning the set of inputs, the new js
increments the IDs of each form field (to keep them unique) and automatically
selects the option that was selected in the cloned group.

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

Bug 2505: adding warnings to C4/Biblio.pm

I added 'use warnings' to C4::Biblio and made a handful of changes to
reduce the number of warnings emitted.

One notable spot is the change in the regex in
C4::Biblio::GetNoZebraIndexes. I have replaced the parens with a character
class. The parens change the way 'split' works, making it return elements
for each delimiter. We did not want those elements returned, and they
only resulted in "'' => undef" being added to the final hash. They also
resulted in two "undefined" warnings for each pass through the loop. I've
included a simple test for this function.

There may be a few warnings still emitted in spots, but either I haven't
seen them yet, or I have chosen to not fix them yet because they require
too much change.

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

barcodedecode() did not always return barcode

This patch amends the function to return barcode, in particular when
filter is not defined.  It also adds an optional 2nd argument to
allow the filter to be specified by caller, enabling testing.

Non-DB-dependent test script included.  Note: T-prefix style
barcode filter is not documented, and drops the first nonzero
digit after the T.  This seems mistaken, but is not corrected here
to avoid any surprises.

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

Allow AddIssue to log SIP transactions distinct from others.

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

remove TMPL_IF EXPR in member.tmpl

Quells following error message in Apache log:

EXPR:at pos 8: non-initialized variable overdues,
referer: http://example.edu/cgi-bin/koha/members/members-home.pl

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

perltidy lables/label-item-search.pl

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

bug 2505: turn on warnings in labels/label-item-search.pl

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

followup to Mason's labels patch

* fix XHTML errors
* fix problems with paging through results list

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

The current searching in labels is a bit minimal, and current only does keyword searching.

there is some old 2.2 code for more complete labels-searching, thats been commented out

ive logged a 'bug' for this - 2777

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

this fix...
  - enables the index-searching functionality
  - adds date-ranges
  - uses the numSearchResults syspref, not a hardcoded '20'
  - rewrote old 2.2 page-num code, (didnt work for 3.x)
  - some indent and whitepages tidys.
  - unused 2.2 search-code removed.

Mason

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

(bug #2887) Change the "Cart" box from js to tmpl

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

Adds the ability to reference itemcallnumber within XSLT

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

Fixes bug #2875

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

Minor updates to minor cronjob script.

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

crontab.example - add back some useful comments

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

Expand crontab.example

Default overdues to "triggered" mode (-t).

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

Revert "Updates to the example crontab"

This reverts commit def09f5a21b47431ff107d9a530900cd5a4fb93d.

As I emailed to the patches list Oct 06, 2008:

I suggest we need to revert Josh' commit def09f5a21b47431ff107d9a530900cd5a4fb93d.

The effect on the crontab example is to invalidate the lines being executed.  The lines were apparently copied in from a cron source, not crontab, despite the header describing it NOT being an example for cron.  It also runs longoverdue twice, instead of fines.pl.

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

Fixes bugs #2875-2879

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

Add 'use warnings' and correct uninitialized value warnings.

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

bug 2505: further warnings fixes to memberentry.pl

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

Added 'warnings' support, fixed bug 2870.

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

Test suite: C4::Members::GetMemberDetails

Tests for C4::Members::GetMemberDetails.  Validates both calls
using either borrowernumber or cardnumber.

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

A script generating keyword clouds from Zebra Indexes

See embedded perldoc to see how it works.

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

test suite: C4::ImportBatch

Here are some tests for functions in C4::ImportBatch.

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

bug 2824: improving error messages from test suite errors

This patch adds two warning messages to places in the test suite that
may fail. I don't think it changes any functionality, but it sure makes
it easier to figure out what has gone wrong.

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

fixed "too many opened files" error during translation

The problem was that the TmplTokenizer create global symbols to store file
handlers so the objects aren't destroyed by the garbage collector.

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

bug 2864 [2/2]: move rss/* to misc/cronjobs/rss/*

As rss.pl is not a CGI script, moved it to join the
other cronjobs.  Full documentation of the script
is in misc/cronjobs/rss/rss.pl, but to summarize:

[1] rss.pl is run on the command line to produce
    an RSS XML document.  The output should be
    placed in a directory accessible to the OPAC
    (or staff) web interface so that users can download
    the RSS feed.  An example of usage:

    misc/cronjobs/rss.pl lastAcquired.conf

    Normally rss.pl should be run periodically (e.g., daily)
    to keep the feed up-to-date.

[2] The configuration file (e.g., lastAcquired.conf) lists

    * name of the template file to use
    * path of output file
    * SQL query

    rss.pl runs the SQL query, then feeds the output of the
    query through the template to produce the output file.

[3] The template file (e.g., lastAcquired.tmpl) uses
    HTML::Template syntax like any of the HTML
    templates for the web interface.

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

bug 2864 [1/2]: restore rss/rss.pl

Restored basic functionality of rss/rss.pl:

[1] Reverted "Add support for itemtypes. Still need to hardcode 'reservable'"
    Revert commit e7575b00edd6098a869425a69856916108e84914, which had
    replace the script with one of its templates.

[2] Minor fixes to update it, including using HTML::Template::Pro instead
    of HTML::Template, perltidy, and turning on warnings.

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

Add default classification sources and filing rules to fr-FR

fr-FR install wasn't populating classication rules tables:

  * class_sort_rules
  * class_sources

This patch add default values for those tables
mandatory during installation process.

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