koha.git
9 years agoBug 6448 - carp on invalid barcode
Dobrica Pavlinusic [Fri, 3 Aug 2012 22:03:11 +0000 (00:03 +0200)]
Bug 6448 - carp on invalid barcode

This is consistent with implementation of other barcode types
which all carp instead of die on invalid barcodes.

9 years agoBug 6448 follow-up, DBrev number
Paul Poulain [Thu, 2 Aug 2012 16:49:59 +0000 (18:49 +0200)]
Bug 6448 follow-up, DBrev number

9 years agoBug 6448 [2/3] Barcodes::EAN13 autoBarcode
Dobrica Pavlinusic [Wed, 28 Mar 2012 12:43:35 +0000 (14:43 +0200)]
Bug 6448 [2/3] Barcodes::EAN13 autoBarcode

Implement auto-incrementing EAN-13 barcodes

To make this work, C4::Barcodes::next was modified to call process_tail with
new incremented value so that process_tail can generate correct checksum.
Since process_tail is currenlty not used by any barcodes, this change is safe.

C4::Barcodes is used by addbiblio.pl when adding multiple records, while value_builder
is used in all other cases.

Test scenario:

1. prove t/Barcodes_EAN13.t

2. KOHA_CONF=/etc/koha/sites/fer/koha-conf.xml prove t/db_dependent/Barcodes.t
   this will check C4::Barcode implementataion

3. in systempreference change autoBarcode to incremental EAN-13 barcode

4. edit two items of any biblio assigning barcodes and verify that numbers
   are increasing. Have in mind that last digit is check digit, and it
   doesn't increment, but is calculated from barcode itself. Example with
   checksum in brackets: 000000086275[2], 000000086276[9], 000000086277[6]

5. Add Item and verify that it gets assigned next barcode

6. Add & Duplicate item and verify barcode increase

7. Add Multiple Copies and verify that barcode increase for each copy

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 6448 [1/3] EAN-13/UPC-A itemBarcodeInputFilter
Dobrica Pavlinusic [Tue, 27 Mar 2012 22:38:34 +0000 (00:38 +0200)]
Bug 6448 [1/3] EAN-13/UPC-A itemBarcodeInputFilter

This filter which check validity of EAN-13 barcode and padd it with zeros
up to full 13 digit number. This will also expand 12 digit UPC-A barcodes
to EAN-13 automatically which is useful for older barcode readers which tend
to ignore first zero in EAN-13 if they have just UPC-A support.

It should be noted that EAN-13 or UPC-A product codes printed on books are not
good choice for barcodes in Koha since each item has to have unique barcode.

Test scenario:

1. prove t/Circulation_barcodedecode.t
   this checks expansion of 12 digit UPC-A to 13 digit EAN-13 and zero padding

2. in systempreferences search for itemBarcodeInputFilter and select EAN-13

3. edit one item and assign it valid EAN-13 barcode, eg. 0000000695152, check it out

4. test checkin with just 695152 to test leading zero expansion

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
9 years agoBug 6448 [3/3] EAN-13 barcode labels
Dobrica Pavlinusic [Sat, 4 Jun 2011 12:23:20 +0000 (14:23 +0200)]
Bug 6448 [3/3] EAN-13 barcode labels

Current implementation doesn't scale barcodes because low-price
CCD barcode readers are very sensitive about size

Test scenario:

1. in Tools > Labels create or edit Layout and select EAN13 as barcode
   type

2. export one of existing batches using EAN13 layout and verify that
   generated pdf file contains barcodes

3. print pdf file and test it with barcode reader

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
9 years agobug 8477 follow-up for [HTML4] coding guideline
Paul Poulain [Thu, 2 Aug 2012 15:40:07 +0000 (17:40 +0200)]
bug 8477 follow-up for [HTML4] coding guideline

9 years agoBug 8477: Displays home/holding branches on return page
Adrien Saurat [Thu, 19 Jul 2012 13:39:41 +0000 (15:39 +0200)]
Bug 8477: Displays home/holding branches on return page

Signed-off-by: Mirko Tietgen <5p4m@gmx.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8455 - Followup - perltidy renewscript.pl
Kyle M Hall [Wed, 18 Jul 2012 14:10:59 +0000 (10:10 -0400)]
Bug 8455 - Followup - perltidy renewscript.pl

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8455 - Check ins processed through "Check Out" tab of the Patron Record ignore...
David Cook [Tue, 17 Jul 2012 08:30:13 +0000 (18:30 +1000)]
Bug 8455 - Check ins processed through "Check Out" tab of the Patron Record ignore Circulation System Preferences

If you check-in an item through the "Check Out" tab of the Patron Record, the return/check-in ignores the "InProcessingToShelvingCart" and "ReturnToShelvingCart" circulation system preferences. Or rather, there just isn't any logic to handle them in the renewscript.pl file.

I'm pretty much just bringing the code straight across from returns.pl, although some minor changes need to be made due to context.

Both system preferences are important to include, since you can check-out an in processing item to a patron. While a regular check-in will clear the in processing status, a check-in through the Patron module will cause the item to have its status returned to "In Processing" rather than going to "Shelving Cart", if the preference is set, of course.

To test the scenario, turn on one of (or both of) these settings and try checking books in through the Circulation module and the Patron module.

Before the patch, you will see that only the Circulation module will change the shelving location.

After the patch, you should see identical behaviour.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
http://bugs.koha-community.org/show_bug.cgi?id=7189

9 years agoBug 8463 - Koha/SearchEngine/Solr/Search.pm fails perlcritic test
Mason James [Wed, 18 Jul 2012 02:04:50 +0000 (14:04 +1200)]
Bug 8463 - Koha/SearchEngine/Solr/Search.pm fails perlcritic test

modified:   Koha/SearchEngine/Solr/Search.pm

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 4198 - Followup - PerlTidy authorities-home.pl
Kyle M Hall [Thu, 5 Jul 2012 14:36:39 +0000 (10:36 -0400)]
Bug 4198 - Followup - PerlTidy authorities-home.pl

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 4198 - deleting an authority refreshes the page
Kyle M Hall [Thu, 5 Jul 2012 14:33:40 +0000 (10:33 -0400)]
Bug 4198 - deleting an authority refreshes the page

Makes clicking a Delete link refresh the search that
was performed so that the results are still on the screen.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Works as advertised. Very usefull. Still the issue that indexing being
not in real time, the search result displays the just deleted authority.
But there is no way to do better.

9 years agoMerge remote-tracking branch 'origin/new/bug_6720'
Paul Poulain [Thu, 2 Aug 2012 13:40:31 +0000 (15:40 +0200)]
Merge remote-tracking branch 'origin/new/bug_6720'

9 years agoBug 6720 follow-up: fix error when auth has been deleted
Jared Camins-Esakov [Thu, 2 Aug 2012 13:18:39 +0000 (09:18 -0400)]
Bug 6720 follow-up: fix error when auth has been deleted

Thanks to Paul Poulain for finding the problem and proposing a solution.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8520: fix authority display in staff client
Jared Camins-Esakov [Mon, 30 Jul 2012 13:25:30 +0000 (09:25 -0400)]
Bug 8520: fix authority display in staff client

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoAdded a URI filter to encode a cardnumber for borrowers as to not break patronimage.pl
Elliott Davis [Wed, 11 Jul 2012 10:52:53 +0000 (05:52 -0500)]
Added a URI filter to encode a cardnumber for borrowers as to not break patronimage.pl

To Test:

Add a + (or any char that would need to be encoded) to a card number of a patron with an image attached.
Go to member/moremember.pl.
If the patrons image is there, you are golden.

http://bugs.koha-community.org/show_bug.cgi?id=8421
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7470: Followup: Changes mailto links
Jonathan Druart [Fri, 30 Mar 2012 13:50:03 +0000 (15:50 +0200)]
Bug 7470: Followup: Changes mailto links

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7470 follow-up: adding comments to avoid removal of this file
Paul Poulain [Thu, 2 Aug 2012 12:14:26 +0000 (14:14 +0200)]
Bug 7470 follow-up: adding comments to avoid removal of this file

it is unused by Koha itself, but must be here for babeltheque customers !

9 years agoBug 7470: Followup Move Babeltheque css into a specific file
Jonathan Druart [Mon, 23 Apr 2012 07:30:48 +0000 (09:30 +0200)]
Bug 7470: Followup Move Babeltheque css into a specific file

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8427 - Patron search results not consistent
Kyle M Hall [Thu, 12 Jul 2012 11:55:06 +0000 (07:55 -0400)]
Bug 8427 - Patron search results not consistent

Simple fix, removed the directive to order by surname, and the system
now orders by surname, then firstname.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8432 follow-up, fixing error in T::T variable scope
Paul Poulain [Thu, 2 Aug 2012 11:25:24 +0000 (13:25 +0200)]
Bug 8432 follow-up, fixing error in T::T variable scope

9 years agoBug 8432: Change serials results display
Jonathan Druart [Fri, 13 Jul 2012 09:26:36 +0000 (11:26 +0200)]
Bug 8432: Change serials results display

The original author of this patch is : Julian Maurice

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 6655 - Sorting order of serial issues in OPAC
Meenakshi.R [Wed, 23 May 2012 05:07:19 +0000 (10:37 +0530)]
Bug 6655 - Sorting order of serial issues in OPAC

Works as advertised.

Signed-off-by: Elliott Davis <elliott@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7839 follow-up fixing HTML4 coding guideline error
Paul Poulain [Thu, 2 Aug 2012 10:06:05 +0000 (12:06 +0200)]
Bug 7839 follow-up fixing HTML4 coding guideline error

(Capital cases only on 1st word)

9 years agoBug 7839 [ENH] : Add tab in patron record to show patron's routing lists
David Cook [Mon, 2 Jul 2012 00:16:46 +0000 (10:16 +1000)]
Bug 7839 [ENH] : Add tab in patron record to show patron's routing lists

dcook: fixed SQL error and removed blank targets

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Thanks. Works as described. Valuable addition on patron information
page.

Ideas of enhancement:

- For some library, and some patron, the routing lists table could be
  very long, and a paged table would be a plus.
- In the table, a new Vendor column.
- The possibility to sort the table entries by Title and Vendor column.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8405 : add IntranetBiblioDefaultView and BiblioDefaultView in XSLT.pm
Gaetan Boisson [Tue, 10 Jul 2012 09:20:04 +0000 (11:20 +0200)]
Bug 8405 : add IntranetBiblioDefaultView and BiblioDefaultView in XSLT.pm

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8408 - Odd OPAC message when OPAC Renewals are disabled and OPACFineNoRenewals...
Kyle M Hall [Tue, 10 Jul 2012 14:57:46 +0000 (10:57 -0400)]
Bug 8408 - Odd OPAC message when OPAC Renewals are disabled and OPACFineNoRenewals is set to blank

If OpacRenewalAllowed is set to "Don't allow" and OPACFineNoRenewals is set to blank ( i.e. disabled ).
A user who owes any fines, when logged in to the OPAC will receive the following message:

    Please note: You have more than 0.00 in fines.

This is a rather odd message, and is not very sensible. The message should either be modified or removed altogether.

This patch modifies the message to the following:

    Please note: You currently owe $fines in fines

where $fines in the formatted amount the borrower currently owes
in fines.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8518 - Self checkout does not display debt amount if syspref AllowFineOverride...
Marc Veron [Sun, 29 Jul 2012 07:30:54 +0000 (09:30 +0200)]
Bug 8518 - Self checkout does not display debt amount if syspref AllowFineOverride is set to allow

If the patron has depts that prevent issuing and the system preference 'AllowFineOverride' is set to 'allow', the amount is not displayed in the user message.

Additionally, patch adds currency symbol to amount.

Test plan:
Do self checkout with patron who has debts that are over the limit.
Test with and without preference 'AllowFineOverride' set.

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoAdd Street Address searchfields set to patrons search header widget
MJ Ray [Fri, 27 Jul 2012 15:57:38 +0000 (16:57 +0100)]
Add Street Address searchfields set to patrons search header widget

http://bugs.koha-community.org/show_bug.cgi?id=8516
Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8503: Software error in edit items with EasyAnalyticalRecords
Fridolyn SOMERS [Wed, 25 Jul 2012 12:11:08 +0000 (14:11 +0200)]
Bug 8503: Software error in edit items with EasyAnalyticalRecords

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Solves software error when editing items for record with invalid
analytic.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_7955'
Paul Poulain [Wed, 1 Aug 2012 14:07:08 +0000 (16:07 +0200)]
Merge remote-tracking branch 'origin/new/bug_7955'

9 years agoBug 7955: Followup : Check the syspref value (avoid sql injection)
Jonathan Druart [Tue, 10 Jul 2012 14:00:54 +0000 (16:00 +0200)]
Bug 7955: Followup : Check the syspref value (avoid sql injection)

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Works as expected. Fields with disallowed characters do not show up.
Added 'if $debug' to an pseudo-unconditional warn.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8207 follow-up DBrev
Paul Poulain [Wed, 1 Aug 2012 13:46:38 +0000 (15:46 +0200)]
Bug 8207 follow-up DBrev

9 years agoBug 8207: Allow see also fields in auths to link to thesauri
Jared Camins-Esakov [Thu, 28 Jun 2012 22:29:12 +0000 (18:29 -0400)]
Bug 8207: Allow see also fields in auths to link to thesauri

Right now if you were to link a field in an authority to a thesaurus, it
would not work properly. The authority type frameworks require some
adjustments to allow see also headings to be linked to thesauri (such as
adding subfield $9).

This patch adds subfield $9 and thesaurus links to existing installs
for those 5xx fields (under MARC21 and NORMARC only) which do not yet
have any subfield $9 or thesauri configured, as well as adding that to
new English language installs. There will be a follow-up patch adding
the subfields to new non-English language install files later, once
it is clear that the changes proposed and demonstrated in the English
files are accepable to the community.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested frameworks install cleanly on a new installation.

Tested database update and made sure, that configuration and subfields
of a field were not changed if
- a $9 subfield already existed for a 5xx field
- a $9 subfield and a link to another thesaurus in $a existed
- no $9 but a link to another thesaurus in $a existed

This seems safe to me.

9 years agoMerge remote-tracking branch 'origin/new/bug_8204'
Paul Poulain [Wed, 1 Aug 2012 12:59:23 +0000 (14:59 +0200)]
Merge remote-tracking branch 'origin/new/bug_8204'

9 years agoBug 8204 follow-up: quiet warn
Jared Camins-Esakov [Fri, 27 Jul 2012 11:53:06 +0000 (07:53 -0400)]
Bug 8204 follow-up: quiet warn

Also removes unnecessary variable declaration. Thanks to Jonathan Druart
for spotting it.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
9 years agoBug 7143: Updating history and about file
Chris Cormack [Thu, 26 Jul 2012 11:00:18 +0000 (23:00 +1200)]
Bug 7143: Updating history and about file

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_8233'
Paul Poulain [Wed, 1 Aug 2012 11:53:43 +0000 (13:53 +0200)]
Merge remote-tracking branch 'origin/new/bug_8233'

9 years agoBug 8233: We can't load test on Koha::SearchEngine::*
Jonathan Druart [Wed, 25 Jul 2012 12:19:48 +0000 (14:19 +0200)]
Bug 8233: We can't load test on Koha::SearchEngine::*

As a temporary measure, don't try to load the search engine modules
in t/00-load.t.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_8486'
Paul Poulain [Wed, 1 Aug 2012 10:38:23 +0000 (12:38 +0200)]
Merge remote-tracking branch 'origin/new/bug_8486'

9 years agoBug 8486 - Follow up making the tests db independent
Chris Cormack [Sat, 28 Jul 2012 08:07:36 +0000 (20:07 +1200)]
Bug 8486 - Follow up making the tests db independent

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Makes the test db independant by specifying the date format. Thus, there is no call to ->preference() that retrieve a systempreference

9 years agoBug 8204: fix UNIMARC display
Jared Camins-Esakov [Thu, 26 Jul 2012 15:49:36 +0000 (11:49 -0400)]
Bug 8204: fix UNIMARC display

UNIMARC display of authorities in the OPAC wasn't working due to
changes in the BuildSummary function since the code was originally
developed.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8204: Add user friendly authority view to OPAC
Jared Camins-Esakov [Thu, 31 May 2012 11:24:29 +0000 (07:24 -0400)]
Bug 8204: Add user friendly authority view to OPAC

Up until now, the only authority view in the OPAC was a
rather-unfriendly expanded MARC view. This patch adds a user-friendly
view similar to the biblio details view.

Specific features to be aware of:
* Right-to-left text in the MARC21 880 field will show up in the
  appropriate location with the appropriate alignment and wrapping
* There is very little CSS styling. Any suggestions for how to make
  the display more attractive would be gratefully received.

To test:
1) Do a search for an authority in the OPAC.
2) Choose an authority record to view.
3) Observe that the view is more user-friendly and polished.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works very nicely now. The old view is still available, but the
user is presented with a nicer non-MARC view first.

9 years agoBug 8208 follow-up: move Clear and Create new buttons out of results
Jared Camins-Esakov [Mon, 9 Jul 2012 12:56:27 +0000 (08:56 -0400)]
Bug 8208 follow-up: move Clear and Create new buttons out of results

Turns "Clear" and "Create new" links into buttons that are visible
before a search is performed, so that the user does not have to do
a search just to clear the field or create a new authority. This
commit also changes the formatting from textual links to buttons.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Nicer and more logical display.
Allows creating of a new authority without doing a search first
or when no results were found.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8208: Add fast-add link to auth finder plugin
Jared Camins-Esakov [Mon, 2 Jul 2012 16:51:08 +0000 (12:51 -0400)]
Bug 8208: Add fast-add link to auth finder plugin

When performing subject analysis, it is not uncommon to find that an
authority record you need does not exist. This commit adds a link in
the search results to enable the cataloger to create an authority on
the fly and have the heading populated by the new authority.

To test:
1. Use the auth finder plugin the cataloguing module to search for a
   heading you do not have an authority record for.
2. Click the "Create new" link.
3. Fill in your authority and save it.
4. Note that the heading field was populated with the data you entered
   in the authority record.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
1) Added a new authority using the plugin
2) Used an existing authority using the plugin
3) Created a new authority in the authority module

All worked nicely.

9 years agoBug 7153 - Show Open Library as Search Target in "More Searches" in OPAC detail page
Savitra sirohi [Sat, 5 Nov 2011 04:21:03 +0000 (00:21 -0400)]
Bug 7153 - Show Open Library as Search Target in "More Searches" in OPAC detail page

Update syspref OPACSearchForTitleIn to include Open Library as a target in More Searches in the OPAC detail page

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8470: remove depreciated H:T:P test
Mason James [Wed, 18 Jul 2012 05:24:29 +0000 (17:24 +1200)]
Bug 8470: remove depreciated H:T:P test

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7420 tiny bugfix
Paul Poulain [Wed, 25 Jul 2012 16:31:50 +0000 (18:31 +0200)]
Bug 7420 tiny bugfix

The printing at the end, in verbose mode was not displaying variables properly because the print was '' and not ""

9 years agoBug 7420 follow-up DBRev number
Paul Poulain [Wed, 25 Jul 2012 16:31:10 +0000 (18:31 +0200)]
Bug 7420 follow-up DBRev number

9 years ago[PATCH] bug_7420: Added overduefinescap to issuingrules
Elliott Davis [Wed, 11 Jul 2012 03:38:28 +0000 (22:38 -0500)]
[PATCH] bug_7420: Added overduefinescap to issuingrules

Replaced existing MaxFine syspref logic with overduefinescap.
Repurposed MaxFine to be the overall overdue limit for all items
overdue. Implemented new MaxFine logic in UpdateFine().

Signed-off-by: Elliott Davis <elliott@bywatersolutions.com>
Tested according to Srdjan's test plan and everything worked like he said it would.  I set fined equal to $2 and max fine equal to $1.      When I ran the fines script for overdue items fines assessed were only $1.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agobug 8486 tiny follow-up, removing `
Paul Poulain [Wed, 25 Jul 2012 14:55:30 +0000 (16:55 +0200)]
bug 8486 tiny follow-up, removing `

9 years agoBug 8486 Merge Calendar test files
Colin Campbell [Wed, 25 Jul 2012 09:55:56 +0000 (10:55 +0100)]
Bug 8486 Merge Calendar test files

Moves the tests previously in t/Kalendar.t to t/Calendar.t
and removes the old file.
Note the existing minimal test of C4/Calendar still called in
t/Calendar

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8486 Correct calculation of days_between
Colin Campbell [Tue, 24 Jul 2012 14:16:10 +0000 (15:16 +0100)]
Bug 8486 Correct calculation of days_between

Thee were errors in the calculation used for days_between
which caused incorrect values to be returned
Added tests to validate calculation

NB Tests still need to be provided for the hourly
calculations

9 years agoMerge remote-tracking branch 'origin/new/bug_7889'
Paul Poulain [Wed, 25 Jul 2012 13:08:50 +0000 (15:08 +0200)]
Merge remote-tracking branch 'origin/new/bug_7889'

9 years agoBug 7889 - Use standard style for errors and messages
Owen Leonard [Fri, 18 May 2012 12:57:15 +0000 (08:57 -0400)]
Bug 7889 - Use standard style for errors and messages

Correcting the markup of error messages in three places:
catalogue merge, CSV profile editing, and opac comments.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years ago7368 Adjustments for Perl dependencies
Marcel de Rooy [Wed, 18 Jul 2012 13:03:36 +0000 (15:03 +0200)]
7368 Adjustments for Perl dependencies

HTML::FormatText is used by Template::Plugin::HtmlToText which is called in the template for opac-sendbasket.pl.
This is not a core Perl 5.10 module. I had to install it additionally on a test server. So I am adding it to PerlDependencies.
I just picked an old version 1.23 that I still could find as min_ver.

Changed the version information for Test::Strict too (not required, only for tests).

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7302 tiny follow-up removing </td><td>
Paul Poulain [Tue, 24 Jul 2012 16:34:52 +0000 (18:34 +0200)]
Bug 7302 tiny follow-up removing </td><td>

to have the Export as CSV in the same cell as other buttons

9 years agoBug 7302: Followup: rename "Client number" column with "Account number"
Jonathan Druart [Mon, 11 Jun 2012 08:29:23 +0000 (10:29 +0200)]
Bug 7302: Followup: rename "Client number" column with "Account number"

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7302: Export basketgroup as CSV
Jonathan Druart [Thu, 16 Feb 2012 13:32:03 +0000 (14:32 +0100)]
Bug 7302: Export basketgroup as CSV

Adds new action export for basketgroup.
This action is available only if your basketgroup is closed.
This export generates a csv file with order informations.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested together with bug 5356.

9 years agoBug 8376: New script to export borrowers misc/export_borrowers.pl
Julian Maurice [Fri, 6 Jul 2012 10:31:48 +0000 (12:31 +0200)]
Bug 8376: New script to export borrowers misc/export_borrowers.pl

This script prints to standard output what is returned by
GetMemberDetails in CSV format.
Exported fields can be specified with option -f. If no -f option is
specified, all fields are exported.

Signed-off-by: Gaetan Boisson <gaetan.boisson@biblibre.com>
Signed-off-by: Robin Sheat <robin@catalyst.net.nz>
Amended with some code to better handle bad data.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8434 - Notice generation fails for Advanced Notices, Item Due, and Overdues when...
David Cook [Fri, 13 Jul 2012 01:50:04 +0000 (11:50 +1000)]
Bug 8434 - Notice generation fails for Advanced Notices, Item Due, and Overdues when run in shell (due to error in Letters.pm)

Patch introduces a check for C4::Context->userenv in Letters.pm, so that script doesn't fail when it calls to C4::Context->userenv->{branch}, when run from shell.

Without the check, Advanced Notices, Item Due, and Overdues fail to generate.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7367 - One "the" too many
Magnus Enger [Mon, 23 Jul 2012 11:26:19 +0000 (13:26 +0200)]
Bug 7367 - One "the" too many

On line 123 it says "of the the last". One "the" is probably enough.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_8062'
Paul Poulain [Tue, 24 Jul 2012 15:31:07 +0000 (17:31 +0200)]
Merge remote-tracking branch 'origin/new/bug_8062'

9 years ago8062 Followup for HTML::FormatText
Marcel de Rooy [Wed, 18 Jul 2012 13:58:37 +0000 (15:58 +0200)]
8062 Followup for HTML::FormatText

The dependency is not used in opac-sendbasket. But there is a require of HTML::FormatText in the code of Template::Plugin::HTMLToText.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
9 years agoBug 8392: Avoid having an untranslatable age range
Frédéric Demians [Mon, 9 Jul 2012 15:45:11 +0000 (17:45 +0200)]
Bug 8392: Avoid having an untranslatable age range

Currently, error message reports an age range in English. For example '0 to
17'. With this patch, the 'to' is not in the .pl file anymore.

Ruth could you sign-off this fix?

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8392: Category age ranges not being enforced
D Ruth Bavousett [Mon, 9 Jul 2012 14:18:03 +0000 (10:18 -0400)]
Bug 8392: Category age ranges not being enforced

This patch works around the redefinition of an array that was causing the
enforcement of age ranges to not-happen.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
I can confirm the bug and the solution.

9 years agoBug 8440: Dates does not appear in suggestions management
Fridolyn SOMERS [Fri, 13 Jul 2012 14:23:02 +0000 (16:23 +0200)]
Bug 8440: Dates does not appear in suggestions management

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8414 - Intranet header toplinks display white rather than blue in < IE8
David Cook [Wed, 11 Jul 2012 23:13:10 +0000 (09:13 +1000)]
Bug 8414 - Intranet header toplinks display white rather than blue in < IE8

Removes IE7 hash hack which caused head toplinks to display white rather than dark blue against light blue background

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8448 - Holds Awaiting Pickup : Cancelling a hold on a waiting item with multiple...
David Cook [Mon, 16 Jul 2012 04:13:31 +0000 (14:13 +1000)]
Bug 8448 - Holds Awaiting Pickup : Cancelling a hold on a waiting item with multiple holds displays a blank screen instead of a warning prompt

When you cancel a hold in the "Hold Waiting" tab of Holds Awaiting Pickup, you should be prompted to retain or transfer the hold for the next patron in the queue. However, instead of seeing this prompt, you are just shown white screen that fails to alert you to the fact that there are more holds on that item!

There is a missing FOREACH reference in waitingreserves.tt. The template tries to directly reference the hash within the cancel_results array, but we need to access the array before we can access the hash.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8442 - labels creator fixes for plack
Dobrica Pavlinusic [Thu, 19 Jul 2012 17:12:13 +0000 (19:12 +0200)]
Bug 8442 - labels creator fixes for plack

- remove all exit(1) and replace them with __END__ which benefits CGI
- insert correct =cut markers at end of POD (required for CGI::Compile
  under plack to correctly parse source code and exeute it)
- scope variables with our which are used inside sub for plack

Signed-off-by: Mirko Tietgen <5p4m@gmx.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBugFix - ReturnToShelvingCart
Kyle M Hall [Thu, 8 Oct 2009 12:13:13 +0000 (12:13 +0000)]
BugFix - ReturnToShelvingCart

If the ReturnToShelvingCart syspref is on, and something needs to go in transit,
the shelving cart setting is overriding the transit.

What seems to be happening is this:
  * Item is checked in, and flagged as needing to go in transit from A to B.
  * Item is immediately flagged as shelving cart, though.  Current display says both shelving cart and in transit.
  * After the cart-to-shelf script runs, item comes out of transit and
    shows as being available at the destination library, although it hasn't
    arrived yet and no one has manually checked it in.

http://bugs.koha-community.org/show_bug.cgi?id=3701

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8439 - Printing basketgroup does not work on plack
Dobrica Pavlinusic [Mon, 16 Jul 2012 10:43:24 +0000 (12:43 +0200)]
Bug 8439 - Printing basketgroup does not work on plack

scope $input CGI object with our for plack

Signed-off-by: Mirko Tietgen <5p4m@gmx.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8476: Little bug in OPAC XSLT on OPACURLOpenInNewWindow
Fridolyn SOMERS [Thu, 19 Jul 2012 13:29:12 +0000 (15:29 +0200)]
Bug 8476: Little bug in OPAC XSLT on OPACURLOpenInNewWindow

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
I know where it comes from: vim cut-and-paste...

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 6716 tiny follow-up, mysql want a space after the --
Paul Poulain [Mon, 23 Jul 2012 16:26:21 +0000 (18:26 +0200)]
Bug 6716 tiny follow-up, mysql want a space after the --

9 years agoBug 6716: Update z3950servers documentation
Nicole C. Engard [Thu, 19 Jul 2012 02:08:27 +0000 (22:08 -0400)]
Bug 6716: Update z3950servers documentation

The new timeout field was not documented.

9 years agoBug 6716: Document search_history table
Nicole C. Engard [Thu, 19 Jul 2012 02:06:10 +0000 (22:06 -0400)]
Bug 6716: Document search_history table

9 years agoBug 6716: Document saved_sql table
Nicole C. Engard [Thu, 19 Jul 2012 02:04:34 +0000 (22:04 -0400)]
Bug 6716: Document saved_sql table

9 years agoBug 6716: Update reserves table
Nicole C. Engard [Thu, 19 Jul 2012 02:01:13 +0000 (22:01 -0400)]
Bug 6716: Update reserves table

This table had one field that I hadn't documented
before and two new fields for hold suspensions. All
of these are now updated.

9 years agoBug 6716: Update documentation for the letter table
Nicole C. Engard [Thu, 19 Jul 2012 01:59:13 +0000 (21:59 -0400)]
Bug 6716: Update documentation for the letter table

The letter table is now for slips and notices, and
had other changes necessary.

9 years agoBug 6716: document import_record_matches table
Nicole C. Engard [Wed, 18 Jul 2012 14:11:34 +0000 (10:11 -0400)]
Bug 6716: document import_record_matches table

9 years agoBug 6716: Document import_batches table
Nicole C. Engard [Wed, 18 Jul 2012 14:07:37 +0000 (10:07 -0400)]
Bug 6716: Document import_batches table

9 years agoBug 6716: Document tags tables
Nicole C. Engard [Wed, 18 Jul 2012 13:34:08 +0000 (09:34 -0400)]
Bug 6716: Document tags tables

I did not document the 'tags' table in this patch
because it looks like this table is unused, so I
need confirmation of that before marking it as
'unused.'

9 years agoBug 6716: Document patronimage table
Nicole C. Engard [Fri, 13 Jul 2012 18:36:08 +0000 (14:36 -0400)]
Bug 6716: Document patronimage table

9 years agoBug 6716: document issuingrules table
Nicole C. Engard [Fri, 13 Jul 2012 18:32:34 +0000 (14:32 -0400)]
Bug 6716: document issuingrules table

9 years agoBug 6716: Document branch_borrower_circ_rules table
Nicole C. Engard [Fri, 13 Jul 2012 16:12:22 +0000 (12:12 -0400)]
Bug 6716: Document branch_borrower_circ_rules table

9 years agoBug 6716: Document branch_item_rules table
Nicole C. Engard [Thu, 12 Jul 2012 13:20:26 +0000 (09:20 -0400)]
Bug 6716: Document branch_item_rules table

9 years agoBug 6716: Document the biblio_framework table
Nicole C. Engard [Thu, 12 Jul 2012 13:13:53 +0000 (09:13 -0400)]
Bug 6716: Document the biblio_framework table

9 years agoBug 6716: Document aqbasket table
Nicole C. Engard [Thu, 12 Jul 2012 03:32:47 +0000 (23:32 -0400)]
Bug 6716: Document aqbasket table

9 years agoBug 6716: Document action_logs table
Nicole C. Engard [Thu, 12 Jul 2012 03:24:50 +0000 (23:24 -0400)]
Bug 6716: Document action_logs table

9 years agoBug 6716: Update ratings table documentation
Nicole C. Engard [Thu, 12 Jul 2012 03:20:28 +0000 (23:20 -0400)]
Bug 6716: Update ratings table documentation

This patch adds a comment to the table and updates the field
descriptions with a bit more information.

9 years agoBug 6716: Document branchtransfers table
Nicole C. Engard [Thu, 12 Jul 2012 03:17:38 +0000 (23:17 -0400)]
Bug 6716: Document branchtransfers table

9 years agoBug 6716: Document aqorders_items table
Nicole C. Engard [Thu, 12 Jul 2012 03:13:27 +0000 (23:13 -0400)]
Bug 6716: Document aqorders_items table

9 years agoBug 6716: Document the aqorders table
Nicole C. Engard [Thu, 12 Jul 2012 03:05:31 +0000 (23:05 -0400)]
Bug 6716: Document the aqorders table

This table has a lot of fields that I never see any values in.
All have been noted in the documentation and I welcome any
additional info about how these fields might be used.

9 years agoBug 6716: Document aqbudgets and aqbudgetperiods tables
Nicole C. Engard [Thu, 12 Jul 2012 01:58:21 +0000 (21:58 -0400)]
Bug 6716: Document aqbudgets and aqbudgetperiods tables

9 years agoMerge remote-tracking branch 'origin/new/bug_8130'
Paul Poulain [Mon, 23 Jul 2012 16:06:50 +0000 (18:06 +0200)]
Merge remote-tracking branch 'origin/new/bug_8130'

9 years agoBug 8130: Followup: errors must be a hashref
Jonathan Druart [Thu, 19 Jul 2012 11:50:08 +0000 (13:50 +0200)]
Bug 8130: Followup: errors must be a hashref

http://bugs.koha-community.org/show_bug.cgi?id=8130
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
9 years agoBug 8130: [SIGNED-OFF] Followup - Empty file upload error never displays.
Kyle M Hall [Thu, 19 Jul 2012 12:41:39 +0000 (08:41 -0400)]
Bug 8130: [SIGNED-OFF] Followup - Empty file upload error never displays.

http://bugs.koha-community.org/show_bug.cgi?id=8130
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
9 years agoBug 8130 [FOLLOW-UP] attach PDF files to a patron record
Owen Leonard [Tue, 10 Jul 2012 17:56:26 +0000 (13:56 -0400)]
Bug 8130 [FOLLOW-UP] attach PDF files to a patron record

Markup corrections:

- Hide table of files if no files are attached
- Move error messages out of table (invalid)
- Add <tr> tags to table (invalid)
- Apply standard form structure to upload form
- Add proper labels and ids to upload form
- Correct unescaped ampersands

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
9 years agoBug 8083 follow-up DBRev number
Paul Poulain [Mon, 23 Jul 2012 08:51:04 +0000 (10:51 +0200)]
Bug 8083 follow-up DBRev number

9 years agobug_8083: Explanation of update query
Srdjan [Wed, 18 Jul 2012 05:53:31 +0000 (17:53 +1200)]
bug_8083: Explanation of update query

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>