wip/koha-chris_n.git
9 years agoMerge remote branch 'kc/new/pending_qa/enh/bug_3644' into kcmaster
Chris Cormack [Sat, 9 Apr 2011 07:16:45 +0000 (19:16 +1200)]
Merge remote branch 'kc/new/pending_qa/enh/bug_3644' into kcmaster

9 years agoUpdating database number
Chris Cormack [Sat, 9 Apr 2011 00:06:32 +0000 (12:06 +1200)]
Updating database number

9 years agoBug 5431: Separate state field for library addresses
Katrin Fischer [Thu, 7 Apr 2011 14:28:38 +0000 (16:28 +0200)]
Bug 5431: Separate state field for library addresses

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBumping database version
Chris Cormack [Fri, 8 Apr 2011 23:56:17 +0000 (11:56 +1200)]
Bumping database version

9 years agoBug 5422: Add missing fields to deletedborrowers and change datatype to text
Katrin Fischer [Thu, 7 Apr 2011 09:41:36 +0000 (11:41 +0200)]
Bug 5422: Add missing fields to deletedborrowers and change datatype to text

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5422: Small layout correction to borrower search result list
Katrin Fischer [Thu, 7 Apr 2011 09:41:35 +0000 (11:41 +0200)]
Bug 5422: Small layout correction to borrower search result list

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoAdding state to output of patron search results screen
Owen Leonard [Thu, 7 Apr 2011 09:41:34 +0000 (11:41 +0200)]
Adding state to output of patron search results screen

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5422: Add missing fields to deletedborrowers and change datatype to text
Katrin Fischer [Thu, 7 Apr 2011 09:41:33 +0000 (11:41 +0200)]
Bug 5422: Add missing fields to deletedborrowers and change datatype to text

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoAdditional fixes for Bug 5422, Separate state field for patron's adresses
Owen Leonard [Thu, 7 Apr 2011 09:41:32 +0000 (11:41 +0200)]
Additional fixes for Bug 5422, Separate state field for patron's adresses

Adding state variable to various circ and member pages so that
sidebar display of patron details includes state.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5422: Separate state field for patron's addresses
Katrin Fischer [Thu, 7 Apr 2011 09:41:31 +0000 (11:41 +0200)]
Bug 5422: Separate state field for patron's addresses

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5422: Separate state field for patron's addresses - database update
Katrin Fischer [Thu, 7 Apr 2011 09:41:30 +0000 (11:41 +0200)]
Bug 5422: Separate state field for patron's addresses - database update

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 4394 Adding preference definition change to opac.pref
Owen Leonard [Tue, 5 Apr 2011 15:28:15 +0000 (17:28 +0200)]
Bug 4394 Adding preference definition change to opac.pref

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 4394 REVISED Allow opaccolorstylesheet syspref to use an external URL
Jane Wagner [Tue, 5 Apr 2011 15:27:55 +0000 (17:27 +0200)]
Bug 4394 REVISED Allow opaccolorstylesheet syspref to use an external URL

Previous code would only allow for an auxiliary stylesheet on the same server.  This checks for
http in the opaccolorstylesheet syspref.  If it's found, sets a different variable pointing to the
remote server location.  If not found, retains the previous css directory path.

Also modifies the description of opaccolorstylesheet in the installer files to reflect current
usage as an auxiliary stylesheet.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/enh/bug_4449' into kcmaster
Chris Cormack [Fri, 8 Apr 2011 23:40:47 +0000 (11:40 +1200)]
Merge remote branch 'kc/new/enh/bug_4449' into kcmaster

9 years agoFix for Bug 4449: AllowHoldPolicyOverride cannot over all policies
Ian Walls [Thu, 9 Sep 2010 20:19:10 +0000 (16:19 -0400)]
Fix for Bug 4449: AllowHoldPolicyOverride cannot over all policies

AllowHoldPolicyOverride should be able to override AllowOnShelfHolds, as well as other possible reasons for blocking a hold. Before, this only
allowed override of branch policies set in Circ Rules.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6119 - Add biblioitems.itemtype in columns for items.content
Alex Arnaud [Thu, 7 Apr 2011 15:37:03 +0000 (17:37 +0200)]
Bug #6119 - Add biblioitems.itemtype in columns for items.content

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agobug #6054 - Add a filter by active budget on acqui-home
Alex Arnaud [Tue, 5 Apr 2011 11:40:55 +0000 (13:40 +0200)]
bug #6054 - Add a filter by active budget on acqui-home

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6049 - Add a filter by date in overdue.pl
Alex Arnaud [Wed, 6 Apr 2011 08:20:59 +0000 (10:20 +0200)]
Bug #6049 - Add a filter by date in overdue.pl

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agobug #6053 Improvement of letter HOLD_PRINT
Sophie Meynieux [Tue, 5 Apr 2011 09:31:00 +0000 (11:31 +0200)]
bug #6053 Improvement of letter HOLD_PRINT

Modify title template for letter HOLD_PRINT in installer sample data

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBZ6087: remove duplicate budget link
Paul Poulain [Fri, 8 Apr 2011 08:04:44 +0000 (10:04 +0200)]
BZ6087: remove duplicate budget link

this feature doesn't exist yet, so the link is useless

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoFix for Bug 5950 - changing frameworks doesn't change anymore
Owen Leonard [Fri, 8 Apr 2011 10:16:08 +0000 (12:16 +0200)]
Fix for Bug 5950 - changing frameworks doesn't change anymore

Selecting a framework from the dropdown menu has never (AFAICT)
auto-submitted the form to change the framework. This patch
reformats the form in order to make it clearer that this menu
 is part of the search function.

Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6108 - Quicksearch in member.pl does not display paging correctly
Stéphane Delaune [Fri, 8 Apr 2011 13:07:08 +0000 (15:07 +0200)]
Bug 6108 - Quicksearch in member.pl does not display paging correctly

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6052 - BranchTransfer Limit :Adds a default value
Alex Arnaud [Fri, 8 Apr 2011 08:10:16 +0000 (10:10 +0200)]
Bug #6052 - BranchTransfer Limit :Adds a default value

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6130 - Letters: Fixing Subject encoding
Alex Arnaud [Fri, 8 Apr 2011 09:49:20 +0000 (11:49 +0200)]
Bug #6130 - Letters: Fixing Subject encoding

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6118 - Title-host missing in Search.pm
Paul Poulain [Fri, 8 Apr 2011 07:25:16 +0000 (09:25 +0200)]
Bug 6118 - Title-host missing in Search.pm

in Search.pm, in the list of available indexes, Title-host is missing

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_5234' into kcmaster
Chris Cormack [Fri, 8 Apr 2011 22:22:46 +0000 (10:22 +1200)]
Merge remote branch 'kc/new/bug_5234' into kcmaster

9 years agoFix for Bug 5234 - Remove unused CSS files
Owen Leonard [Thu, 7 Apr 2011 13:34:07 +0000 (15:34 +0200)]
Fix for Bug 5234 - Remove unused CSS files

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/enh/bug_5921' into kcmaster
Chris Cormack [Fri, 8 Apr 2011 02:25:08 +0000 (14:25 +1200)]
Merge remote branch 'kc/new/enh/bug_5921' into kcmaster

9 years agoBug 5921 follow-up: fix text wrapping
Jared Camins-Esakov [Sun, 3 Apr 2011 14:40:12 +0000 (10:40 -0400)]
Bug 5921 follow-up: fix text wrapping

On records with hundreds of items, the "Serial enumeration/chronology" label
wraps onto a second line, which is very unattractive. Changing the label to
"Serial enumeration" fixes this.

Signed-off-by: Claire Hernandez <claire.hernandez@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6017 : MT3157 : editing an attribute lead to Error500
Henri-Damien LAURENT [Wed, 6 Apr 2011 14:51:00 +0000 (16:51 +0200)]
Bug 6017 : MT3157 : editing an attribute lead to Error500

Editing an extended attribute in the extended attribute tab would lead to an Error500
This patch adds some checks for values before UPDATE in SQLHelper.pm
And Adds some checks also befaore calling ModMember in members/memberentry

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 4340 - patron name search does not handle punctuated names
J. David Bavousett [Tue, 5 Apr 2011 14:26:41 +0000 (16:26 +0200)]
Bug 4340 - patron name search does not handle punctuated names

This patch will enable C4::Members::SearchMember to handle searching for punctuated
names (e.g. the "Jones" in "Smith-Jones" or the "Angelo" in "D'Angelo")

It is possible to add a bunch of LIKE clauses, but that adds to the search time in
a rather dramatic way.  REGEXP, by itself, is also a performance killer, but I found
a suggestion of using a more-general LIKE ANDed with the specific REGEXP, and this
gives excellent performance over other approaches.

Signed-off-by: Guillaume Hatt <guillaume.hatt@enc.sorbonne.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5251 : Enhancement: [3.2] (Resubmission) Adding code to support using gmail as...
Chris Nighswonger [Thu, 7 Apr 2011 15:54:17 +0000 (17:54 +0200)]
Bug 5251 : Enhancement: [3.2] (Resubmission) Adding code to support using gmail as an SMTP server

This patch adds support for using a gmail account as an SMTP server.
It includes a basic HOWTO.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5415 Let calls of SimpleSearch utilize considtent interface
Colin Campbell [Wed, 6 Apr 2011 09:42:41 +0000 (11:42 +0200)]
Bug 5415 Let calls of SimpleSearch utilize considtent interface

Remove some unnecessary checks when check of error is
sufficient. Make the order in some cases more logical
Should remove some possibilities of runtime warning noise.
Although some calls belong to the 'Nothing could
ever go wrong' school have added some warnings

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5415 More consistent returns from SimpleSearch
Colin Campbell [Wed, 6 Apr 2011 09:42:40 +0000 (11:42 +0200)]
Bug 5415 More consistent returns from SimpleSearch

SimpleSearch returns a 3 element array the first pf which is an error
indication. If an error is returned the other elements are
undefined. If error is undef the other elements are defined
This restores the consistency of the interface as it was before
the addition of zebra

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 3803 Show ISSN in orderline of basket display
Colin Campbell [Wed, 6 Apr 2011 08:14:04 +0000 (10:14 +0200)]
Bug 3803 Show ISSN in orderline of basket display

Where an item possesses an ISSN is may be of equal or greater
importance to acquisitions staff as the ISBN so include it in
the display

Sponsered by UNFAO, Rome

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6024 : Show "Run report" link after sql update as wel
Srdjan Jankovic [Thu, 7 Apr 2011 14:45:58 +0000 (10:45 -0400)]
Bug 6024 : Show "Run report" link after sql update as wel

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6024 : add "Run report" link on the new report creation success page
Srdjan Jankovic [Thu, 7 Apr 2011 14:45:41 +0000 (10:45 -0400)]
Bug 6024 : add "Run report" link on the new report creation success page

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5895: Translation error in German fast add framework
Katrin Fischer [Fri, 18 Mar 2011 20:21:54 +0000 (21:21 +0100)]
Bug 5895: Translation error in German fast add framework

Fixes description of 952$1.

Signed-off-by: Mark Gavillet <mark.gavillet@ptfs-europe.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6095: Add UTF8 encoding in template
Alex Arnaud [Thu, 7 Apr 2011 09:12:40 +0000 (11:12 +0200)]
Bug 6095: Add UTF8 encoding in template

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_5949' into kcmaster
Chris Cormack [Fri, 8 Apr 2011 01:22:54 +0000 (13:22 +1200)]
Merge remote branch 'kc/new/bug_5949' into kcmaster

9 years agoBug 5949: Hide 'Edit record' link when creating new order
Katrin Fischer [Thu, 7 Apr 2011 10:56:12 +0000 (10:56 +0000)]
Bug 5949: Hide 'Edit record' link when creating new order

The 'Edit record' link only makes sense, when you want to modify
an order after creating it. Before there is no biblionumber the link
can link to. Patch adds check for existing biblionumber so link is
only shown when there is an existing record to be edited.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 4189 : Searching z39.50 without selecting any servers results in error message
Tomas Cohen Arazi [Wed, 30 Mar 2011 20:08:57 +0000 (17:08 -0300)]
Bug 4189 : Searching z39.50 without selecting any servers results in error message

This patch makes z3950_search.pl handle the situation properly, and z3950_search.tmpl
show a proper message instead of a program error. All this, in case the javascript
checks that alert the user (that at least one target has to be chosen)  are by-passed.

Regards
To+

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Fixed small translation problem in javascript alert message.
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6088: remove Perl warning
Paul Poulain [Thu, 7 Apr 2011 11:09:27 +0000 (11:09 +0000)]
Bug 6088: remove Perl warning

caused by use warnings;

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6072: fixing permission inconsistencies MT5306
Paul Poulain [Wed, 6 Apr 2011 08:54:38 +0000 (10:54 +0200)]
Bug 6072: fixing permission inconsistencies MT5306

In large libraries, some librarian may have permission only
to recieve shipments This patch fixes some permission :
* booksellers page = accessible to anyone that has at least 1 acq permission
* parcels = accessible to anyone with order_recieve
* supplier detail = accessible to anyone that has at least 1 acq permission,
  but modifying accessible only if vendor_manage

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoFix for Bug 6109, Hey!
Owen Leonard [Thu, 7 Apr 2011 13:44:28 +0000 (15:44 +0200)]
Fix for Bug 6109, Hey!

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_4912' into kcmaster
Chris Cormack [Fri, 8 Apr 2011 01:02:55 +0000 (13:02 +1200)]
Merge remote branch 'kc/new/bug_4912' into kcmaster

9 years agoBug 4912: Follow-up fix for Bug 4912, redirect user to correct lists
Owen Leonard [Thu, 7 Apr 2011 08:04:23 +0000 (10:04 +0200)]
Bug 4912: Follow-up fix for Bug 4912, redirect user to correct lists

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6112 - Missing fields in overdues csv file
Alex Arnaud [Thu, 7 Apr 2011 14:30:13 +0000 (16:30 +0200)]
Bug #6112 - Missing fields in overdues csv file

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6107 - Problems when duplicate a patron on memberentry.pl
Alex Arnaud [Thu, 7 Apr 2011 10:05:27 +0000 (12:05 +0200)]
Bug #6107 - Problems when duplicate a patron on memberentry.pl

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6099: error in report pagination when GROUP BY
Jonathan Druart [Thu, 7 Apr 2011 14:14:29 +0000 (16:14 +0200)]
Bug 6099: error in report pagination when GROUP BY

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5903 : French authorised values fixed
Paul Poulain [Thu, 7 Apr 2011 09:53:37 +0000 (11:53 +0200)]
Bug 5903 : French authorised values fixed

country codes must be in capital letters

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5275: enable switching back to default biblio framework
Paul Poulain [Thu, 7 Apr 2011 14:15:54 +0000 (16:15 +0200)]
Bug 5275: enable switching back to default biblio framework

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoFix for Bug 6114 - Conflicting warnings about Koha to MARC links
Owen Leonard [Thu, 7 Apr 2011 14:48:56 +0000 (16:48 +0200)]
Fix for Bug 6114 - Conflicting warnings about Koha to MARC links

Removing incorrect message from marc_subfields_structure.tmpl and
moving the correct message from marctagstucture.tmpl into its place.

Also correcting the help file for marc_subfields_structure.mpl

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoFix for Bug 6037, Invalid markup, missing breadcrumbs on Keyword to MARC Mapping...
Stéphane Delaune [Thu, 7 Apr 2011 13:35:13 +0000 (15:35 +0200)]
Fix for Bug 6037, Invalid markup, missing breadcrumbs on Keyword to MARC Mapping page

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_3072' into kcmaster
Chris Cormack [Thu, 7 Apr 2011 23:22:57 +0000 (11:22 +1200)]
Merge remote branch 'kc/new/bug_3072' into kcmaster

9 years agoBug 3072: Search for Heading-Main in authorities
Marcel de Rooy [Thu, 7 Apr 2011 09:42:30 +0000 (11:42 +0200)]
Bug 3072: Search for Heading-Main in authorities

Includes Heading-Main in authorities bib1 attribute set.
Also includes Any in authorities bib1 (was still missing).
Also removes temporary fix in AuthoritiesMarc.pm.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Rebased after pushed fix for #5924.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6096 Correctly return arrayref from GetAllShelves
Colin Campbell [Thu, 7 Apr 2011 15:30:17 +0000 (17:30 +0200)]
Bug 6096 Correctly return arrayref from GetAllShelves

Follow on from bug 5529 GetAllShelves was also vaguely stuffing
an arrayref into an array
Some signs of cut and paste (unnecessary @params variable)
Calling code now does not have to shuffle the return about
to get the reference. Also removed a debug warn

Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5529 Absence or Presence of lists not being reliably returned
Colin Campbell [Thu, 7 Apr 2011 14:25:27 +0000 (16:25 +0200)]
Bug 5529 Absence or Presence of lists not being reliably returned

C4::VirtualShelves::GetRecentShelves contained some rather confused
code The contents of the requested list are returned in an arrayref
which was in its turn being wrapped needlessly in an array
As a result the returned array always consisted of a single element
irrespective of the number of lists.
Made the routine return the arrayref, which can now be tested directly
Unfortunately rather than fixing this we had previously coded around it
assuming it to be a "design" decision. Have amended other calls of
the subroutine resulting in some hopefully less obscure code

Fixed logic error in the results template which displayed new list
within a test for the presence of lists

Removed the offset parameter from the sql in the routine as it was hardcoded
to 0 i.e. the default value

Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6082 - On aqbudgets.pl bug on the Statistic's combobox
Alex Arnaud [Thu, 7 Apr 2011 09:59:22 +0000 (11:59 +0200)]
Bug #6082 - On aqbudgets.pl bug on the Statistic's combobox

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6106: add item not respecting framework in acq The PrepareItemrecordDisplay sub...
Paul Poulain [Thu, 7 Apr 2011 15:15:53 +0000 (17:15 +0200)]
Bug 6106: add item not respecting framework in acq The PrepareItemrecordDisplay sub builds an item framework for later use in ACQ Some cases were missing : item subfield connected to a plugin, hidden subfield, long size subfield.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6110 - fix Callnumber index mistake
Alex Arnaud [Thu, 7 Apr 2011 12:59:53 +0000 (14:59 +0200)]
Bug #6110 - fix Callnumber index mistake

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoNew developer to the history
Chris Cormack [Thu, 7 Apr 2011 20:43:06 +0000 (08:43 +1200)]
New developer to the history

9 years agoMerge remote branch 'kc/new/enh/bug_5547' into kcmaster
Chris Cormack [Thu, 7 Apr 2011 20:39:14 +0000 (08:39 +1200)]
Merge remote branch 'kc/new/enh/bug_5547' into kcmaster

9 years agoBug: 5547 - corrected insert to database
Mark Gavillet [Wed, 6 Apr 2011 12:16:15 +0000 (14:16 +0200)]
Bug: 5547 - corrected insert to database

Inserting a new category failed because only 10 bind variables were present and 11 were required.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6084: Follow up - fix more translation markers
Tomas Cohen Arazi [Thu, 7 Apr 2011 08:46:16 +0000 (10:46 +0200)]
Bug 6084: Follow up - fix more translation markers

grep -R 'alert("' *

was not enough. I needed

grep -R "alert('"

too!.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6084: Missing translation markers in javascript alert messages
Tomas Cohen Arazi [Thu, 7 Apr 2011 08:46:15 +0000 (10:46 +0200)]
Bug 6084: Missing translation markers in javascript alert messages

I ommited commented lines and "if $debug" lines too.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6105 - Fix a problem when cancel a reserve from moremember.pl
Alex Arnaud [Thu, 7 Apr 2011 09:24:34 +0000 (11:24 +0200)]
Bug 6105 - Fix a problem when cancel a reserve from moremember.pl

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agobug 4903: enable OAI-DC output when install done in standard mode
Galen Charlton [Thu, 7 Apr 2011 07:57:45 +0000 (09:57 +0200)]
bug 4903: enable OAI-DC output when install done in standard mode

Based on patch by Tomás Cohen Arazi <tomascohen@gmail.com>,
revised to work regardless of the installation mode.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agobug6064 adding forgotten news version in kohaversion
Stéphane Delaune [Thu, 7 Apr 2011 08:21:17 +0000 (10:21 +0200)]
bug6064 adding forgotten news version in kohaversion

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug #6064 Improvement of default value usage in overdue
Sophie Meynieux [Thu, 7 Apr 2011 08:21:16 +0000 (10:21 +0200)]
Bug #6064 Improvement of default value usage in overdue

Change default value in overduerules table for delayn (NULL instead of 0) and fix GetBranchcodesWithOverdueRules function

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6062: ordering basketgroups
Paul Poulain [Tue, 5 Apr 2011 12:43:23 +0000 (14:43 +0200)]
Bug 6062: ordering basketgroups

When you have many basketgroups, you get the oldest one on top. Usually, you
have something to do on recent ones, so it's better to have basket groups
ordered DESC

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5951 : consistency change: s/planning/statistic/
Paul POULAIN [Tue, 5 Apr 2011 14:26:16 +0000 (16:26 +0200)]
Bug 5951 : consistency change: s/planning/statistic/

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5915 : C4::AuthoritiesMarc AddAuthority
Paul Poulain [Tue, 5 Apr 2011 09:04:42 +0000 (11:04 +0200)]
Bug 5915 : C4::AuthoritiesMarc AddAuthority

Fixing 100$a simple error: Sometimes, language is encoded fre50 and not
frey50.
This patch adds check for that problem

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBumping database version
Chris Cormack [Thu, 7 Apr 2011 04:31:48 +0000 (16:31 +1200)]
Bumping database version

9 years ago5936: Remove some obsolete db columns from aqbooksellers and aqbudgets if needed
Marcel de Rooy [Wed, 6 Apr 2011 11:35:37 +0000 (11:35 +0000)]
5936: Remove some obsolete db columns from aqbooksellers and aqbudgets if needed

Removes obsolete columns from aqbooksellers and aqbudgets if needed.
aqbooksellers: deliverydays followupdays followupscancel invoicedisc nocalc
specialty
aqbudgets: budget_description

These columns should not exist in a fresh install from kohastructure.sql, but
originate from older 3.0 versions.
One line in acqui/updatesupplier.pl referring to one of these columns is
removed too.

This patch therefore does not change functionality, but only keeps your db in
shape.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Rebased April 6.
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5923 : Removed link from variable $summary
Julian Maurice [Tue, 5 Apr 2011 08:40:15 +0000 (10:40 +0200)]
Bug 5923 : Removed link from variable $summary

Variable $summary contained <a> and <b> tag. They have been deleted.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5923 : Authorities list :removing link on Summary
Henri-Damien LAURENT [Tue, 5 Apr 2011 08:40:14 +0000 (10:40 +0200)]
Bug 5923 : Authorities list :removing link on Summary

removing the link to authority details on authority List directly from summary
And adding a generic link to details

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_5957' into kcmaster
Chris Cormack [Thu, 7 Apr 2011 04:07:58 +0000 (16:07 +1200)]
Merge remote branch 'kc/new/bug_5957' into kcmaster

9 years agoBug 5957 follow-up: use comparison operator in if
Jared Camins-Esakov [Tue, 5 Apr 2011 15:49:35 +0000 (17:49 +0200)]
Bug 5957 follow-up: use comparison operator in if

The patch for bug 5957 used an assignment operator instead of a comparison
operator. Replace the assignment operator with the string comparison that was
desired.

Signed-off-by: Claire Hernandez <claire.hernandez@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoRemoving conflict markers
Chris Cormack [Thu, 7 Apr 2011 03:55:33 +0000 (15:55 +1200)]
Removing conflict markers

9 years ago(Bug 4049) Searching itemtypes returning noise
Henri-Damien LAURENT [Tue, 5 Apr 2011 12:13:51 +0000 (14:13 +0200)]
(Bug 4049) Searching itemtypes returning noise

Fix the problem of searching on partial itemtypes strings
for instance, itemtype="ARTICLE DE PERIODIQUE", itemtype="PERIODIQUE"
searching on itemtype="PERIODIQUE" would also get "ARTICLE DE PERIODIQUE"

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5213 - Suffix number sequence not resetting properly in hmyymmincr barcode autoge...
Chris Nighswonger [Tue, 5 Apr 2011 09:50:48 +0000 (11:50 +0200)]
Bug 5213 - Suffix number sequence not resetting properly in hmyymmincr barcode autogen pattern

Two things are happening.

First, the SELECT does not account for a branchcode greater than 2 chars.

This is fixed by just selecting the final four digits representing the
incremental suffix from the barcode field.

Second, the incremental suffix (4 digits) is not rolling back over to zero when
reaching 9999.

This pattern probably needs help. It should allow for cataloging of 10000 items
per month as is. I would not recommend doing barcode numbering this way, but
need to support it since it is in use in at least one library.

This patch also impliments strict and warnings per Bug 2505

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6077: qty not incremented for AcqCreateItem=recieve
Paul Poulain [Wed, 6 Apr 2011 08:02:31 +0000 (10:02 +0200)]
Bug 6077: qty not incremented for AcqCreateItem=recieve

When you want to create items on order recieve, the qty was not incremented
when the librarian adds an item with the +

This was because the js was expecting the qty field to be id=quantity

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5946 : (MT1177) aqplan.pl, fix the problem of calculating the total remaining.
Alex Arnaud [Fri, 14 May 2010 15:07:55 +0000 (17:07 +0200)]
Bug 5946 : (MT1177) aqplan.pl, fix the problem of calculating the total remaining.

Bug 5946 followup : (MT1236) aqplan.pl - Fixe the problem of unrounded sums

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoFix for Bug 5156, JavaScript error when adding list
Owen Leonard [Wed, 6 Apr 2011 09:27:11 +0000 (11:27 +0200)]
Fix for Bug 5156, JavaScript error when adding list

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6028 : MT3815: Quickfix for CSV export (9999 rows limitation)
Matthias Meusburger [Wed, 6 Apr 2011 14:39:36 +0000 (16:39 +0200)]
Bug 6028 : MT3815: Quickfix for CSV export (9999 rows limitation)

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoThis patch fixes the broken pagination in members-search.pl
Koustubha Kale [Sat, 2 Apr 2011 16:53:06 +0000 (22:23 +0530)]
This patch fixes the broken pagination in members-search.pl

Small patch to fix Error 404 which comes up when we click on pagination links after searching in Patron Card creator.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6100: request.pl should check maxreserves exists
Jared Camins-Esakov [Wed, 6 Apr 2011 18:00:47 +0000 (14:00 -0400)]
Bug 6100: request.pl should check maxreserves exists

Adds a check to confirm that the maxreserves syspref actually has a value before
using it. Also fixes an off-by-one error in the maxreserves calculation that
would allow librarians to place maxreserves + 1 holds for patrons.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6032: Increase zebra biblio register stores
Robin Sheat [Wed, 6 Apr 2011 23:18:32 +0000 (11:18 +1200)]
Bug 6032: Increase zebra biblio register stores

This keeps the package versions of the zebra configuration in sync with
the makefile-installed ones.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6032: Give biblio register and shadow sane default max sizes
Jared Camins-Esakov [Wed, 6 Apr 2011 18:28:49 +0000 (14:28 -0400)]
Bug 6032: Give biblio register and shadow sane default max sizes

The out-of-the-box maximum size for the biblio register and shadow stores is
4GB, which is much too small for even medium sized libraries. This makes the
default maximum 20GB, which should be enough for even most large libraries.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 4374 Improve biblio data entry form regarding hidden/mandatory fields
Frédéric Demians [Wed, 6 Apr 2011 08:14:29 +0000 (10:14 +0200)]
Bug 4374 Improve biblio data entry form regarding hidden/mandatory fields

Biblio framework has 3 entry points controlling if input box are
displayed by default in data entry form:

  - 'mandatory' flag at field level
  - 'mandatory' flag at subfield level
  - 'hidden' flag at subfield level

It doesn't work as it should (?):

  - All subfields are expanded (shown) if their field is mandatory
  - A subfield containing anything is always expanded whatever it contains
  - A mandatory subfield is always expanded
  - An hidden subfield isn't expanded.

Default framework having 0 in all marc_subfield_structure.hidden, by default
all subfields will be expanded.

TO BE INTEGRATED IN RELEASE NOTES:

  Koha sysadmin should have to edit its frameworks to add hidden=1 in
  appropriate subfields.

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 6014 : Removing a leftover warn
Chris Cormack [Thu, 7 Apr 2011 01:16:26 +0000 (13:16 +1200)]
Bug 6014 : Removing a leftover warn

9 years agoBug 6014 : MT3138 Adding some permission management to viewlog
Henri-Damien LAURENT [Wed, 6 Apr 2011 15:28:45 +0000 (17:28 +0200)]
Bug 6014 : MT3138 Adding some permission management to viewlog

If user can not access reports, then form to search the logs is not displayed.
It also improves the presentation.

C4/Log.pm
- adds the fact that we can enter multiple actions
- fixes the fact that user information was truncated

circ-menu.inc:
Fixing information passed to the viewlog from circ-menu

Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_4276' into kcmaster
Chris Cormack [Wed, 6 Apr 2011 22:41:37 +0000 (10:41 +1200)]
Merge remote branch 'kc/new/bug_4276' into kcmaster

9 years agoBug 4276 follow-up: quote strings with curly braces
Jared Camins-Esakov [Wed, 6 Apr 2011 20:00:43 +0000 (16:00 -0400)]
Bug 4276 follow-up: quote strings with curly braces

Previous fix used the wrong characters for quoting. I should have used curly
braces instead of double quotes. Some searches using double quotes will fail
(the problem seems to be punctuation, although there may be other circumstances
where double quotes fail as well).

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoPartial fix for Bug 4870, Damaged status does not prevent items from being selected...
Owen Leonard [Wed, 6 Apr 2011 07:51:45 +0000 (09:51 +0200)]
Partial fix for Bug 4870, Damaged status does not prevent items from being selected for reserves

This patch changes the holds queue build process in order to
require that items not be damaged in order to appear in the
holds queue report.

Revision adds a check for the AllowHoldsOnDamagedItems preference to
determine whether a damaged item should be included in the holds
queue report.

Signed-off-by: Christophe Croullebois <christophe.croullebois@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5868: Subject indexes do not search 6xx fields
Jared Camins-Esakov [Tue, 5 Apr 2011 09:19:01 +0000 (11:19 +0200)]
Bug 5868: Subject indexes do not search 6xx fields

Prior to this change, the named Zebra Subject phrase index did not include the
600, 610, 611, 630, 654, 655, 656, and 657 fields. This could be seen in
searches such as: "su:Bible and su:N.T. and su:Matthew and su:Textbooks", which
should return a result with the following field:
=630  00$aBible.$pN.T.$pMatthew$vTextbooks.
This did not formerly work.

This patch modifies etc/zebradb/marc_defs/marc21/biblios/records.abs. This file
must be installed and biblios reindexed in Zebra for the changes made here to
affect records already in the catalog.

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoMerge remote branch 'kc/new/bug_5555' into kcmaster
Chris Cormack [Wed, 6 Apr 2011 07:49:55 +0000 (19:49 +1200)]
Merge remote branch 'kc/new/bug_5555' into kcmaster

9 years agoBug5555 Corrected search for ISBN / ISSN
Janusz Kaczmarek [Wed, 6 Apr 2011 07:36:22 +0000 (09:36 +0200)]
Bug5555 Corrected search for ISBN / ISSN

Signed-off-by: Stéphane Delaune <stephane.delaune@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 5924 : C4::AuthoritiesMarc.pm search enhancement
Henri-Damien LAURENT [Tue, 5 Apr 2011 12:18:16 +0000 (14:18 +0200)]
Bug 5924 : C4::AuthoritiesMarc.pm search enhancement

Heading Main is defined in unimarc record.abs
re enabling that feature

Signed-off-by: fdurand <frederic.durand@univ-lyon2.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>