koha.git
9 years agoBug 7977: QOTD uploader to enable uploading csv files containing quotes
Chris Nighswonger [Fri, 20 Apr 2012 20:12:26 +0000 (16:12 -0400)]
Bug 7977: QOTD uploader to enable uploading csv files containing quotes

This series will add a DataTable's based upload/editor with which
to upload csv files containing quotes to be used by the QOTD
feature.

The file should be formatted thusly:

"source","text-of-quote"
"source","text-of-quote"
...

Note: This work serves as a good example of potential improvements
in all other "editor" and file upload areas of Koha.

This patch is a squash of the following work:

--Adding code to parse CSV file contents and push it into a DataTable
--Adding in jEditable to enable table editing
--Adding ajax to post data back to the server to be saved
--Fixing edit and adding delete functionality
--Adding some missing css as well as server feedback on save
--Fixing a bug which limited the number of quotes which could be uploaded
--Also fixing a minor bug with fnCSVToArray and doing some style cleanup.
--Adding sanity checks to verify file type and size
--Implements YUI button widget/toolbar
--Improved handling of hiding uploader UI
--Adds row selectability
--Adds multi-delete capability
--Adds YUI button/toolbar widget
--Fixing capitalization in quote uploader
--Implements improvments suggested by jcamins and oleonard

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years agoBug 7977: Quote-of-the-day (QOTD) Feature for OPAC
Chris Nighswonger [Mon, 9 Apr 2012 20:35:06 +0000 (16:35 -0400)]
Bug 7977: Quote-of-the-day (QOTD) Feature for OPAC

This feature will add the option of displaying a select quote
for the day on the OPAC homepage. It will include the addition
of a QOTD editor in the tools section of the staff interface
which will allow the addition, editing, and deletion of quotes.
A single system preference will enable/disable the display of
the QOTD on the OPAC homepage. A new granular permission will
also be added to conrol user access to the QOTD editor tool.

Possible extentions to this would be code to allow alternate
quote selection algorithms to be added to vary how the currently
displayed quote is selected from the table of quotes.

This patch is a squash of the following work:

--Adding quotes table to kohastructure.sql and updatedatabase.pl

Note: This patch is intended for a MySQL based installation only.
That means that it includes backticks because that is what is required
at this point in history to install Koha correctly on a MySQL based
installation.

Feel free to port this over to the Pg stuff in the data/Pg directory.

--Adding system preference to control display of QOTD on OPAC main page
--Adding sample quote data

These quotes are taken from various US presidents. I'm not sure these
are applicable to the rest of the world, so I'm leaving it to translators
and others to add sample quote data for other languages.

--Adding edit_quotes user permissions
--Squash with other db related patches in this series
--Adding quotes editor pages to tools
--Adding QOTD editor link to tools homepage
--Integration of DataTables into QOTD Editor
--Impleminting jEditable into the quote editor
--Adding delete and add option to editor
--Fixing OPAC display so the QOTD div does not show if no quote is returned
--Also removing useless code from tools/quotes.pl
--Adding spans around QOTD foo in opac-main.tt
--Also fixing quote selection logic to accomodate the possibility of
    a single quote in the db a litte better.
--Changing timestamp column header to a more user-friendly 'Last Displayed'
--Fixing bug in quote selection logic. This bug caused a situation where
    when the table had only a single quote with an id greater than 1 in it,
    no quote would be selected.
--Fixing up sorting in the quote editor table. --jcamins
--Adding span element to quote separator --jcamins
--Overriding the default empty table message supplied by datatables-strings.inc
--Adds missing page heading
--Adds beginning of click-for-help elements.
--Refactors delete functionality to allow selecting of quotes to delete,
    enabling multi-delete.
--Refactors saving added quote functionality so that striking <Enter>
    saves the new quote.
--Refactors canceling aded quote functionality so that striking <Esc>
    cancels the new quote.
--Removing debug console.log statements
--Implementng the YUI button widget/toolbar used on other pages.
--Adds a 5px radius to all YUI buttons to bring them into
    conformity with the general trend toward rounded corners
--Fixes capitalization in quote editor
--Implements improvements suggested by jcamins and oleonard
--Adds DataTables Plugin dataTables.fnReloadAjax.js

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years agoBug 7977: Adding DateTime::Format::MySQL as a core dependency
Chris Nighswonger [Thu, 19 Apr 2012 14:50:21 +0000 (10:50 -0400)]
Bug 7977: Adding DateTime::Format::MySQL as a core dependency

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years agoBug 7977: Adding jEditable minified version
Chris Nighswonger [Sat, 14 Apr 2012 04:37:16 +0000 (00:37 -0400)]
Bug 7977: Adding jEditable minified version

jEditable is necessary to create DataTables with editable fields

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years agoBug 7977: Updating datatables.css to reflect changes from v1.8.1 to v1.9.1
Chris Nighswonger [Tue, 17 Apr 2012 15:06:57 +0000 (11:06 -0400)]
Bug 7977: Updating datatables.css to reflect changes from v1.8.1 to v1.9.1

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years agoBug 7977: Upgrading DataTables library to 1.9.1
Chris Nighswonger [Tue, 17 Apr 2012 14:48:02 +0000 (10:48 -0400)]
Bug 7977: Upgrading DataTables library to 1.9.1

Previous versions of the DataTables library did not support listeners
for custom DataTabels events. These listeners are necessary to the
"add quote" feature of the QOD editor.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Mason James <mtj@kohaaloha.com>
9 years ago[PATCH 1/1] Bug 7757 - Followup - Move IF outside of html tag
Paul Poulain [Thu, 24 May 2012 11:36:47 +0000 (13:36 +0200)]
[PATCH 1/1] Bug 7757 - Followup - Move IF outside of html tag
(patch written by kyle@bywatersolutions.com, see http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9588

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7757 - Edit basket vendor after it has been created
Kyle M Hall [Fri, 23 Mar 2012 16:43:25 +0000 (12:43 -0400)]
Bug 7757 - Edit basket vendor after it has been created

Signed-off-by: mveron <veron@veron.ch>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 3521 - Items table in catalogue/detail.pl and cataloguing/additem.pl is sorted...
Kyle M Hall [Thu, 29 Mar 2012 13:54:38 +0000 (09:54 -0400)]
Bug 3521 - Items table in catalogue/detail.pl and cataloguing/additem.pl is sorted nonsensically

Based on patch submitted by Ian Walls

Items were sorted by branch, then date accessioned, in GetItemsInfo.  While this can be
helpful in some circumstances, more often it is useful for items to be sorted first by
their enumchron (volume/issue, if applicable) then by their copy number.  This patch changes the
sort on GetItemsInfo to branch, enumcrhon, copynumber then date accessioned.

As sorting on copynumber will be incorrect based on standard sorting ( e.g. 1, 10, 2, 20 ),
the copynumber is now padded with leading zeros to correct the sorting ( e.g. 01, 02, 10 20 ).
This function appears to be a standard SQL function and not a mysql-ism. I have verfified
that is available in MySQL, Postgres, and Oracle.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Passed t xt

Item sorts appear to be correct to me based on the rubric of branch, enumchron, copynumber, then date accessioned.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7948 - Printing transfer slip loses barcode field focus
Kyle M Hall [Thu, 12 Apr 2012 15:16:20 +0000 (11:16 -0400)]
Bug 7948 - Printing transfer slip loses barcode field focus

  * Refocus on barcode field after running Dopop()
  * Cleaned up javascript, removed tabs

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
passes t xt
works as advertised.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8057 - Error when adding a patron with email address
Kyle M Hall [Tue, 8 May 2012 19:49:10 +0000 (15:49 -0400)]
Bug 8057 - Error when adding a patron with email address

Caused by code assuming a the variable $err is a hashref
when it is only a hashref if there is an error.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7788: [SIGNED-OFF] Followup: GetShelf call in addbybiblio script corrected
Marcel de Rooy [Thu, 22 Mar 2012 10:03:19 +0000 (11:03 +0100)]
Bug 7788: [SIGNED-OFF] Followup: GetShelf call in addbybiblio script corrected

Patch resolves incorrect call to GetShelf in opac and staff. It was and is harmless, because the singlecategory variable was not really used (the category$x param was ignored in the template). Just a cleanup operation.

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7788: [SIGNED-OFF] Remove two unused calls to GetShelf
Marcel de Rooy [Thu, 22 Mar 2012 09:36:41 +0000 (10:36 +0100)]
Bug 7788: [SIGNED-OFF] Remove two unused calls to GetShelf

Removed two unused calls. Initialized $output in downloadshelf. Corrected some confusing indentation in downloadhself too.

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>

9 years agoBug 7704 -Fix bug that prevent items to be returned to home branch, when independentb...
Meenakshi.R [Thu, 5 Apr 2012 06:45:37 +0000 (12:15 +0530)]
Bug 7704 -Fix bug that prevent items to be returned to home branch, when independentbranches and canreservefromotherbranch sysprefs are both turned on.

Before this fix, when attempt to checkin the item at the other branch is made, system disallows checkin. Now (with this fix) the item is checked in and an automatic transfer to the home branch is setup.

How to test:
Case 1
Set independentbranches to "Prevent" and canreservefromotherbranch to "allow". Set up an item in branch A and a member in branch A and another member in branch B.

Checkout the item to member in branch A.
Place a reserve on the item on behalf of member in branch B.
Return the item in branch A. Check if a transfer is setup to branch B.
Checkout the item to the member in branch B and check it back in. Checkin should be allowed and at this point an automatic return to branch A (the home library) should be setup.

Case 2
Set independentbranches to "Prevent" and canreservefromotherbranch to "not allow". In this case item checkin should not be allowed in branch B.

Other cases
Set independentbranches to "Not Prevent". Here items should be returned automatically to home branch, the value in canreservefromotherbranch has no impact.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8106: DataTables integration in ACQ: addorderiso2709.tt
Jonathan Druart [Fri, 17 Feb 2012 14:55:38 +0000 (15:55 +0100)]
Bug 8106: DataTables integration in ACQ: addorderiso2709.tt

addorderiso2709.tt

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Only affects the list of imports, not the list of records

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7604 - Link on basket group name for closed basket groups is broken
Katrin Fischer [Sat, 14 Apr 2012 12:43:32 +0000 (14:43 +0200)]
Bug 7604 - Link on basket group name for closed basket groups is broken

Fixes the intended behaviour of the link on the basket group name
for closed basket groups by adding a missing '='.

Clicking the link will reopen the basket group.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8077 - overdues with fines won't run
Kyle M Hall [Mon, 14 May 2012 12:49:57 +0000 (08:49 -0400)]
Bug 8077 - overdues with fines won't run

When I try to run the overdues with fines report I get the following:
Software error:
Undefined subroutine &main::dt_from_string called at
kohaclone/circ/branchoverdues.pl line 116.

Fixed by add a use line for Koha::DateUtils to branchoverdues.pl

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7175: Allow to choose which items to receive
Julian Maurice [Fri, 13 Jan 2012 15:55:10 +0000 (16:55 +0100)]
Bug 7175: Allow to choose which items to receive

If AcqCreateItem=ordering, when you receive an order, you now have a
list of all created items and checkboxes that permit you to choose which
items you want to receive.
A 'Edit' link open additem.pl page in a popup to allow you edit the
items before receiving them (popup is automatically closed after
modification, and items table is automatically updated)
If quantity is set manually in the text box, the appropriate number of
checkbox are checked from top to bottom and a warning shows up if
quantity is greater than order quantity

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7175: orderreceive.pl cosmetic changes
Julian Maurice [Fri, 13 Jan 2012 14:15:41 +0000 (15:15 +0100)]
Bug 7175: orderreceive.pl cosmetic changes

- use warnings
- replace @$results[0] by $order when $count == 1

9 years agoBug 7248 follow-up (alternative)
Paul Poulain [Mon, 14 May 2012 15:45:33 +0000 (17:45 +0200)]
Bug 7248 follow-up (alternative)

This patch introduces some new features for caching system:
* the type of caching is retrieved from ENV variable in httpd configuration if not forced when the ->new() is called.
* if $ENV{DEBUG} is ON, you'll have feedback when something is set or read from cache
* the Koha::Cache->is_cache_active is now available and will return 1 if there is a caching system available. It's a replacement for C4::Context->ismemcached
* fixes the mistake for debug/compress_threshold and expire_time parameters

The 2 report web service modules have been updated to use this new API

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoBug 7248 Added caching support and and moved Caching into Koha namespace
Chris Hall [Sun, 20 Nov 2011 22:38:23 +0000 (11:38 +1300)]
Bug 7248 Added caching support and and moved Caching into Koha namespace

Unit test for Koha/Cache.pm (which includes Koha/Cache/Memcached.pm)

Note that in order to test Koha::Cache you must export the environment variable
MEMCACHED_SERVERS. For example:
$ export MEMCACHED_SERVERS=127.0.0.1:11211

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
9 years agoBug 5742 follow-up DB rev number
Paul Poulain [Tue, 15 May 2012 16:02:05 +0000 (18:02 +0200)]
Bug 5742 follow-up DB rev number

9 years agoBug 5742 follow-up permissions for RU and UK languages
Paul Poulain [Tue, 15 May 2012 16:01:49 +0000 (18:01 +0200)]
Bug 5742 follow-up permissions for RU and UK languages

9 years agoBug 5742: Followup: sort* fields can to be filled with an input text
Jonathan Druart [Thu, 5 Apr 2012 08:13:41 +0000 (10:13 +0200)]
Bug 5742: Followup: sort* fields can to be filled with an input text

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBut 5742: Batch edit patrons
Jonathan Druart [Wed, 28 Mar 2012 15:43:13 +0000 (17:43 +0200)]
But 5742: Batch edit patrons

new permission : edit_patrons

Plan test:
- Go on the page: tools > Patrons modification (modborrowers.pl)
- Enter a list of cardnumbers (or use a file)
- Modify one or more patron's fields (you can modify surname, firstname,
branchname, category, sort1, sort2, date of enrollment, date of expiry,
debarred date, debarred comment and borrower note)
- Save
- Check on the result page (or in database for non-displayed fields) if
modifications have been correctly made.
- re test with different attributes. For each attributes filled with an
authorized value category, you can select a value in a drop-down list.
Else it's a free input text. If your attribute is filled with a patron
category, the modification performs only on patrons belonging to this
category
- Save
- Verify on the result page

Signed-off-by: Mathilde Formery <mathilde.formery@ville-nimes.fr>
Signed-off-by: Stéphane Delaye <stephane.delaye@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8041: lost items are no longer available in search
Stéphane Delaune [Thu, 3 May 2012 11:46:51 +0000 (13:46 +0200)]
Bug 8041: lost items are no longer available in search

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8001: [SIGNED-OFF] improve styling potential of the tag list
Robin Sheat [Tue, 24 Apr 2012 04:43:56 +0000 (16:43 +1200)]
Bug 8001: [SIGNED-OFF] improve styling potential of the tag list

This adds the class name "tagterm" to the terms in the list of a user's
tags in the OPAC, allowing them to be seperately styled.

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7722 - Insidious problem with searching
Kyle M Hall [Thu, 15 Mar 2012 20:27:07 +0000 (16:27 -0400)]
Bug 7722 - Insidious problem with searching

I cannot find the root cause of this issue, but multiple libraries that I am aware of have problems searching on particular search terms ( and never the same terms at the same library ). The error they get when they trigger this problem is:

Tag "" is not a valid tag. at /home/koha/kohaclone/C4/Biblio.pm line 1849

Something somewhere is adding empty keys to C4::Context->marcfromkohafield, I think it may have something to do with the analytics feature that was added.

In the while loop for TransformKohaToMarc, there is a line

next unless my $dtm = $db_to_marc->{''}->{$name};

I don't think it's working.
If I dump $dtm, for each search, I see the dump twice.
It looks like this:
$VAR1 = [
           '952',
           'w'
         ];
 $VAR1 = [];
I think the second time, when it is empty is what's breaking this.
The next never fails because even though it is empty, it is still a valid arrayref.

The solution I have some up with is to skip over the elements where the arrayref is empty.

Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7641 - Followup - Suspend Until not set on by suspend button.
Kyle M Hall [Fri, 13 Apr 2012 14:11:00 +0000 (10:11 -0400)]
Bug 7641 - Followup - Suspend Until not set on by suspend button.

For request.pl, there are two ways to suspend a reserve, either
by using the 'suspend' button for an individual reserve, or by
using the 'Update hold(s)' button with suspend until dates set.

If the 'suspend' button is used, any date in the 'suspend until'
field is ignored. This commit fixes this issue.

  * Add suspend_until date to suspend button link via jquery
  * Add optional date to ToggleSuspend()
  * Add KohaDates plugin where necessary

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
passes tests, tested:
* suspend all holds from circ.pl
* suspend one hold from circ.pl
* suspend all holds from moremember.pl
* suspend one hold from moremember.pl
   --- NOTE: clicking suspend all holds without setting a date will mean the holds must be manually unsuspended. I'm not sure this is intentional?
* suspend a specific hold using the in-table button on reserves
* suspend a specific hold using the "update hold" button

500 error is gone.

http://bugs.koha-community.org/show_bug.cgi?id=8084
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years ago7368 typo: MARC -> Card
Marcel de Rooy [Thu, 10 May 2012 08:01:12 +0000 (10:01 +0200)]
7368 typo: MARC -> Card

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_7213'
Paul Poulain [Tue, 15 May 2012 13:44:57 +0000 (15:44 +0200)]
Merge remote-tracking branch 'origin/new/bug_7213'

9 years agoBug 7213 : Follow up fixing license information
Chris Cormack [Mon, 14 May 2012 21:15:43 +0000 (09:15 +1200)]
Bug 7213 : Follow up fixing license information

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8019 - Preserve language choice between Browser sessions
Marc Veron [Fri, 27 Apr 2012 22:48:28 +0000 (00:48 +0200)]
Bug 8019 - Preserve language choice between Browser sessions

Language choice in Opac an Staff client was not preserved between Browser sessions.
Changed expiry date of cookie 'KohaOpacLanguage' from '' (= expire after session) to '+3y' (3 years from now)

To test:
- In a multilingual Koha istallation, choose a language.
- Log out, close all Browser instances. Go to login screen. Login should show up in same language as before.
- Repeat with other languages.

modified:   C4/Templates.pm

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7968 : Create meta-packages to simplify installation of koha
root [Wed, 18 Apr 2012 14:07:31 +0000 (16:07 +0200)]
Bug 7968 : Create meta-packages to simplify installation of koha

Minor change in the koha packages : that is having perldeps and third party softwares installable as stand alone packages
in other words, create meta-packages to simplify installation of koha.
2 packages was created : koha-deps and koha-perldeps which install general dependencies of koha.
I modified the file control.in and executed the update-control script to generate the file control. Whith the control file,
I launched the build-git-snapshot script (see http://wiki.koha-community.org/wiki/Building_Debian_Packages_-_The_Easy_Way)
to generate packages.

Signed-off-by: Robin Sheat <robin@catalyst.net.nz>
I've removed the 'Version' and 'Maintainer' lines, as they don't belong
in package sections. They apply to the source declaration only.
Otherwise, it seems fine and useful.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7213 - simple /svc/ HTTP example
Dobrica Pavlinusic [Thu, 10 Nov 2011 20:39:55 +0000 (21:39 +0100)]
Bug 7213 - simple /svc/ HTTP example

Simple command-line client which can authorize itself to Koha,
get MARC XML record based on biblio number and update record

This script can also be used as module using require "koha-svc.pl"
from other scripts which can implement MARC XML creation or parsing.

This is follow up version which now uses Content-type: text/xml
header when using POST method to be in sync with documentation at
http://wiki.koha-community.org/wiki/Koha_/svc/_HTTP_API

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 5549 followup: fix suspension in days further
Ian Walls [Thu, 22 Mar 2012 21:06:21 +0000 (17:06 -0400)]
Bug 5549 followup: fix suspension in days further

Grace periods are now measured in the same unit as the loan.

Suspension in days works explicitly in days, independent of unit.  Since
the 'deltadays' duration is actually the duration, possibly in hours, between
when the material was due and when it's returned, we need to take the finedays
factor (how many days per unit overdue) and multiple it by 24 if the unit is
hourly.

Example: a 1 hour loan material with 1 hour grace period and finedays = 2,
returned 2 hours late would give 4 days suspension

To test:
1.  create some circ rules:
  A) an hourly loan rule with suspension in days
  B) an hourly loan rule with suspension in days and a grace period
  C) a daily loan rule with suspension in days
  D) a daily loan rule with suspension in days and a grace period
2.  Check out 2 materials under each rule.  We'll call these A1-D1
and A2-D2 (to associate with the rule)
3.  Return A1-D1 late, but within the grace period for B1 and D1 as appropriate
4.  Return A2-D2 late, after the grace periods

You should get:

A1: suspended for (#hours late) * (#suspensionindays) days
B1: not suspended
C1: suspended for (#days late) * (#suspensionindays) days
D1: not suspended

A2: suspended for (#hours late) * (#suspensionindays) days
B2: suspended for (#hours late) * (#suspensionindays) days
C2: suspended for (#days late) * (#suspensionindays) days
D2: suspended for (#days late) * (#suspensionindays) days

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_8022'
Paul Poulain [Mon, 14 May 2012 16:14:10 +0000 (18:14 +0200)]
Merge remote-tracking branch 'origin/new/bug_8022'

9 years agobug 8022 follow-up, missing permission in spanish
Paul Poulain [Mon, 14 May 2012 16:13:41 +0000 (18:13 +0200)]
bug 8022 follow-up, missing permission in spanish

This patch should quiet jenkins

9 years agoMerge remote-tracking branch 'origin/new/bug_7178'
Paul Poulain [Mon, 14 May 2012 14:18:46 +0000 (16:18 +0200)]
Merge remote-tracking branch 'origin/new/bug_7178'

9 years agoFollowup 7178: Remove warning on uninitialized value in sprintf for neworderempty
Marcel de Rooy [Fri, 11 May 2012 14:38:50 +0000 (14:38 +0000)]
Followup 7178: Remove warning on uninitialized value in sprintf for neworderempty

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7178: Follow-up Improve order item creation
Julian Maurice [Wed, 28 Mar 2012 15:01:07 +0000 (17:01 +0200)]
Bug 7178: Follow-up Improve order item creation

Move SQL code from Perl script to Perl module
Replace SHOW COLUMNS by $dbh->column_info()
Update total on neworderempty.pl when adding or deleting items

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested ok for ordering and receiving items.
Total updated correctly.

Note: There are lots of errors in the logs before and after applying
the patch. A follow up is needed.

9 years agoBug 7998: Don't make the news all bold
Jared Camins-Esakov [Sun, 13 May 2012 15:06:03 +0000 (11:06 -0400)]
Bug 7998: Don't make the news all bold

Signed-off-by: Gaetan Boisson <gaetan.boisson@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7998 - CSS fixes for new interface
Liz Rea [Mon, 23 Apr 2012 16:07:51 +0000 (11:07 -0500)]
Bug 7998 - CSS fixes for new interface

Omnibus of changes thus far:

adds slight transparency for news so logo shows through on mainpage..
Fixes purple header gradient in Chrome-based browsers.
remove list from returns.tt options so checkboxes do not have bullets.
fix missing gradient class on returns.tt.
reverse colors of menu div - blue for inactive, grey for active.

turns searchheader blue, rounds corners, improves spacing for sort form.

Adds padding, rounded corners, and a 1px border to the now-blue toolbar.

increase width of intranet nav div to width 40%

add a bit of padding to #searchheader

fieldset.action changes - removed background, added a little padding to make it look better in all of the uses I could find of it.

Bug 7998 - followup - make facets header background blue

bug 7998 - followup - fixing headers on search.pl to be blue, rounded.

bug 7998 - followup - consistency tweaks

match menu borders to the search header tabs (green border)
hover menus a very light grey instead of #eee.
make fieldset.brief have a consistent border with the rest of the fieldsets.

bug 7998 - followup - more tabs/borders updating to fit in new look

boraccount.pl

bug 7998 - followup - add gradient div to prefs-admin-search.inc

Bug 7998 - Change toolbar to be lighter, with barely discernible border

Will need to be applied after the other patch.

Bug 7998 - add gradient to roadtype admin panel

Bug 7998 - adds gradient to patrons-admin-search.tt

Bug 7998 - add gradient to budgets-admin-search.inc

bug 7998 - add gradient to z3950-admin-search.inc

Bug 7998 - add gradient to cities-admin-search.inc

bug 7998 - active tab on checkout table now has green border like side menu

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_5345'
Paul Poulain [Mon, 14 May 2012 13:42:47 +0000 (15:42 +0200)]
Merge remote-tracking branch 'origin/new/bug_5345'

9 years agoBug 5345: DataTables integration in acquisition module [6]
Jonathan Druart [Fri, 17 Feb 2012 14:56:51 +0000 (15:56 +0100)]
Bug 5345: DataTables integration in acquisition module [6]

histsearch.tt

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 5345: DataTables integration in acquisition module [10]
Jonathan Druart [Fri, 17 Feb 2012 14:58:20 +0000 (15:58 +0100)]
Bug 5345: DataTables integration in acquisition module [10]

ordered.tt

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
9 years agoBug 5345: DataTables integration in acquisition module [13]
Jonathan Druart [Fri, 17 Feb 2012 15:00:53 +0000 (16:00 +0100)]
Bug 5345: DataTables integration in acquisition module [13]

spent.tt

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
9 years agoBug 5345: DataTables integration in acquisition module [15]
Jonathan Druart [Fri, 17 Feb 2012 15:01:28 +0000 (16:01 +0100)]
Bug 5345: DataTables integration in acquisition module [15]

uncertainprice.tt

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
9 years agoBug 5345: DataTables integration in acquisition module [9]
Jonathan Druart [Fri, 17 Feb 2012 14:57:57 +0000 (15:57 +0100)]
Bug 5345: DataTables integration in acquisition module [9]

newordersuggestion.tt

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
9 years agobug_7647: Table sorter on issue history
Srdjan [Thu, 29 Mar 2012 00:38:16 +0000 (13:38 +1300)]
bug_7647: Table sorter on issue history

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7982: [SIGNED-OFF] Typo in moremember-receipt.tt
Magnus Enger [Thu, 19 Apr 2012 08:12:18 +0000 (10:12 +0200)]
Bug 7982: [SIGNED-OFF] Typo in moremember-receipt.tt

Changes "Checked uut today" to "Checked out today".

To test:
Um, just looking at the patch should be enough, I think?

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8022: Permissions test doesn't check all languages
Jared Camins-Esakov [Sat, 28 Apr 2012 14:04:51 +0000 (10:04 -0400)]
Bug 8022: Permissions test doesn't check all languages

Add the permissions files for es-ES and nb-NO to the permissions test.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Running xt/permissions.t before the patch leaves out nb-NO and es-ES.
After the patch they are picked up and the test fails correctly
because of the missing permissions described in Bug 7984.
As far as I can tall, all permission files are now picked up by
this test.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8045: fixes "date due" when checking in
Adrien Saurat [Mon, 7 May 2012 14:11:11 +0000 (16:11 +0200)]
Bug 8045: fixes "date due" when checking in

Added cloning of object parameters in "days_between" method.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Checked in items that were overdue, due in the future and not
checked out and all showed the right info in the check in
table.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7984 [SIGNED-OFF] - Fix the upload_local_cover_images permission
Magnus Enger [Thu, 19 Apr 2012 19:22:01 +0000 (21:22 +0200)]
Bug 7984 [SIGNED-OFF] - Fix the upload_local_cover_images permission

upload_local_cover_images is needed to use the new, cool local covers
feature

This patch
- adds upload_local_cover_images to
  installer/data/mysql/nb-NO/1-Obligatorisk/userpermissions.sql
  from whch it was missing, making it impossible to use the local
  covers feature in an nb-NO installation
- translates the text of the preference for de-DE (thanks to cait,
  drojf and mveron for the translation)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Marc Veron <veron@veron.ch>
I did no fresh installs, however I
- had a look at the files before and after applying the patch
- carefully checked the syntax
- tested with MySQL by removing the entry in the table 'permissions'
  double checked that now the entry in Staff client was one
  and then did  an insert with the sql snippets.
  In DE-de the translated entry translation appeared as expected.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7924 - Fix handling of command line arguments in koha-remove
Magnus Enger [Mon, 30 Apr 2012 11:49:38 +0000 (13:49 +0200)]
Bug 7924 - Fix handling of command line arguments in koha-remove

koha-remove would fail with a message like this:
Removing Koha instance --
ERROR 1396 (HY000) at line 1: Operation DROP USER failed for 'koha_--'@'%'

After fixing this, the --keep-mysql option made the script exit without doing
anything.

To test:

Alt 1
- Build package with build-git-snapshot
- Install package

Alt 2
- Replicate the changes in the patch in an existing installation

Then:
- Add instance1 and instance2
- koha-remove instance1
-- Check that script completes and the databse is gone
- koha-remove --keep-mysql instance2
-- Check that the script completes but the database is not removed

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8035 - bibs with comments show an error in opac
Kyle M Hall [Wed, 9 May 2012 12:37:40 +0000 (08:37 -0400)]
Bug 8035 - bibs with comments show an error in opac

If a record has any approved comments, viewing the record
details in the opac will error out with the following
message:

Undefined subroutine &main::format_date called at
/home/load02/kohaclone/opac/opac-detail.pl line 654.

This is caused by a use of C4::Dates::format_date
without adding a use call to the head of the file.

Removing the call is the only change necessary. The
dates are already being filtered by $KohaDates.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8072: [SIGNED-OFF] Changes in capitalisation had broken guided reports
Chris Cormack [Fri, 11 May 2012 00:03:00 +0000 (12:03 +1200)]
Bug 8072: [SIGNED-OFF] Changes in capitalisation had broken guided reports

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7877 - note on expiration in the opac worded wrongish
Owen Leonard [Wed, 9 May 2012 18:29:55 +0000 (14:29 -0400)]
Bug 7877 - note on expiration in the opac worded wrongish

Correction following Nicole's suggestion.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7842 - Inconsistencies in Notices interface
Owen Leonard [Wed, 9 May 2012 19:41:15 +0000 (15:41 -0400)]
Bug 7842 - Inconsistencies in Notices interface

- Putting "New" button in toolbar <div>
- Moving "Copy" function and making label more explicit
- Adding page headings
- Correcting table header sort options

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7739 follow-up for capital cases
Paul Poulain [Fri, 11 May 2012 10:20:14 +0000 (12:20 +0200)]
Bug 7739 follow-up for capital cases

9 years agoBug 7739 - Batch edit items of a title
Amit Gupta [Wed, 18 Apr 2012 11:11:28 +0000 (16:41 +0530)]
Bug 7739 - Batch edit items of a title

A new option - Edit items in batch - is made available under the menu - Edit - in the catalog detail page. When this option is selected the tool batch item modification is called with all items under this biblio passed as parameters.

User can then edit all items of this biblio in batch. Once changes are saved the "done" link takes the user back to the catalog detail page of this biblio.

To test:
    Navigate to catalog detail page of any biblio with one or more item records. Use the option "Edit items in batch" from the "Edit" menu. Modify the items in batch, say the collction code or lost status. Press "Save", verify that changes have been effected in the confirmation screen. Use the "Done" link to navigate back to the catalog detail page.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8048 - better spacing on categories for attributes
Owen Leonard [Wed, 9 May 2012 16:28:58 +0000 (12:28 -0400)]
Bug 8048 - better spacing on categories for attributes

 - Convert <h4> to <caption> and adding comfortable padding.
 - Add quotes to attribute for validity
 - Add padding to patron messaging note too since it has
   the same problem.

To test you must have at least one value added to the PA_CLASS
authorised value category. At least one patron attribute must
be categorized using that value.

On the patron edit screen that patron attribute category should
display with comfortable padding.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Looks much better!

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_5345'
Paul Poulain [Fri, 11 May 2012 09:33:51 +0000 (11:33 +0200)]
Merge remote-tracking branch 'origin/new/bug_5345'

9 years agoBug 5345: DataTables integration in acquisition module [16]
Jonathan Druart [Fri, 17 Feb 2012 15:01:47 +0000 (16:01 +0100)]
Bug 5345: DataTables integration in acquisition module [16]

z3950_search.tt

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
9 years agoWelcoming Lyon 3 team as a committer, and adding 3.8.0 release
Paul Poulain [Fri, 11 May 2012 09:28:25 +0000 (11:28 +0200)]
Welcoming Lyon 3 team as a committer, and adding 3.8.0 release

Also fixing 3 space/tab mistakes

9 years agoReplace itemnumber by barcode in links of patron modification log
Lyon3 Team [Mon, 2 Apr 2012 09:53:16 +0000 (11:53 +0200)]
Replace itemnumber by barcode in links of patron modification log

in the ISSUE and RETURN links to item

http://bugs.koha-community.org/show_bug.cgi?id=7870
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Passes tests, works as expected. Perhaps for followup - items that have been deleted are linked, but do not work. If an itemnumber doesn't exist, it should not be linked.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_5184'
Paul Poulain [Fri, 11 May 2012 09:23:55 +0000 (11:23 +0200)]
Merge remote-tracking branch 'origin/new/bug_5184'

9 years agoBug 5184 [REVISED] Upgrade jQuery to the latest version (OPAC)
Owen Leonard [Tue, 17 Apr 2012 14:53:41 +0000 (10:53 -0400)]
Bug 5184 [REVISED] Upgrade jQuery to the latest version (OPAC)

Upgrade jQuery in the OPAC.

To test: Try any JavaScript-dependent interaction
you can think of in the OPAC and watch for errors.

Revision updates the one syntax I've found so far in testing
which requires an update. Where we used to do this to
uncheck a checkbox:

$(foo).attr("checked","");

We should now use:

$(foo).removeAttr("checked");

As far as I can tell the old way will not generate an error
but neither will it work.

Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
9 years agoBug 6335 Set branch in sip session
Colin Campbell [Thu, 2 Jun 2011 11:26:09 +0000 (12:26 +0100)]
Bug 6335 Set branch in sip session

The server returns its branch as the institution in the config file
However in transactions it will default to the homebranch of the
sip user which may not be consistent with that. Needs to set branch
in the session if there is a value in config otherwise statistics and
behaviour can be wrong (esp in discharge)

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7820: Followup: Add missing Debian packages
Jonathan Druart [Wed, 28 Mar 2012 13:27:50 +0000 (15:27 +0200)]
Bug 7820: Followup: Add missing Debian packages

Adds DateTime::Set used in Koha/Calendar.pm

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7820: Add missing Debian packages
Ian Walls [Sat, 24 Mar 2012 13:50:29 +0000 (09:50 -0400)]
Bug 7820: Add missing Debian packages

Adds DateTime::Format::DateParse, Readonly, Readonly::XS and Modern::Perl
Debian packages to install_misc/debian.packages

Signed-off-by: Marc Balmer <marc@msys.ch>
Signed-off-by: Ian Walls <koha.sekjal@gmail.com>
Sign off for Marc Balmer added manually by Ian Walls on Mar 24, 2012 by
permission (see IRC chat log)

9 years agoBug 5184 - Upgrade jQuery to the latest version (staff client)
Owen Leonard [Tue, 17 Apr 2012 15:21:14 +0000 (11:21 -0400)]
Bug 5184 - Upgrade jQuery to the latest version (staff client)

Upgrade jQuery in the staff client. Besides the change
to the jQuery file syntax has been changed wherever this
syntax was written:

$(foo).attr("checked","");

The new correct way to un-check a checkbox:

$(foo).removeAttr("checked");

I also removed some JavaScript altogether from
branch-transfer-limits.tt which used the old syntax but
which wasn't used on the page at all anymore.

Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7249 BDRev number
Paul Poulain [Fri, 11 May 2012 08:58:43 +0000 (10:58 +0200)]
Bug 7249 BDRev number

9 years agobug 7249 follow-up, online help
Paul Poulain [Fri, 11 May 2012 08:58:33 +0000 (10:58 +0200)]
bug 7249 follow-up, online help

9 years agoBug 7249: Syspref to control number of rows in web service results
Jared Camins-Esakov [Sun, 29 Apr 2012 19:41:30 +0000 (15:41 -0400)]
Bug 7249: Syspref to control number of rows in web service results

The original patches for bug 7249 had a hardcoded limit of ten rows in the
results returned by the web service. This patch adds a SvcMaxReportRows syspref
which allows the librarian to choose how many rows should be returned by the
web service.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7249 : Allow reports to be called through a restful interface
Chris Hall [Mon, 21 Nov 2011 03:09:19 +0000 (16:09 +1300)]
Bug 7249 : Allow reports to be called through a restful interface

Hands back JSON, and supports caching

Squashed commit of the following:

commit 5c93506079738b9a13139417d3a0734289a4b007
Author: Chris Hall <chrish@catalyst.net.nz>
Date:   Mon Nov 21 15:41:14 2011 +1300

    Fixed bugs when creating reports, added more user friendly input and
    feedback

commit c62430bb358ee4af6ee5331b4d3a2ed67f723032
Author: Chris Hall <chrish@catalyst.net.nz>
Date:   Mon Nov 21 12:00:27 2011 +1300

    Added caching to intranet report webservices, added cache expiry and public
    options to reports

commit 2d89f0777d95b26bf08635782070b6367d0698f3
Author: Chris Cormack <chris@bigballofwax.co.nz>
Date:   Wed Nov 9 20:29:26 2011 +1300

    Working on caching for services

commit e0511f180cebd81747858ad776433fe3a1cf6854
Author: Chris Cormack <chris@bigballofwax.co.nz>
Date:   Wed Nov 9 08:28:26 2011 +1300

    Starting work on webservices

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Patch includes database update adding 2 new fields to the saved_sql table.

1) Checked that adding/deleting/editing of sql reports still works and new
fields are correctly saved to the database.

2) The saved reports page now lists the new fields correctly.

3) Checking URLs with JSON output
- if report is not public http://localhost/cgi-bin/koha/svc/report?id=2 fails,
but http://localhost:8080/cgi-bin/koha/svc/report?id=2 works
- if report is public, both links work

4) Checking results are cached
Created a report listing my borrowers, setting caching to 60 seconds and
added new borrowers before doing following tests.
- URL doesn't show new borrower immediately - correct
- Turning off the usecache system preference updates data - correct
- Restarting memcached updates data '/etc/init.d/memcached restart' - correct
- Waiting until cache time runs out updates data - correct

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Retested the feature and renewing my sign-off.

There are some things that should be noted:
- currently there is a hardcoded limit of 10 results for the reports webservice
- reports with runtime parameters are currently not supported

9 years agoBug 7500 - Use CSS Sprites for faster page loading
Owen Leonard [Wed, 11 Apr 2012 16:55:48 +0000 (12:55 -0400)]
Bug 7500 - Use CSS Sprites for faster page loading

This patch moves *most* small interface icons and
the Koha log into a single 10K sprite image. CSS
is used to position the sprite correctly for each
icon instance.

Not covered by this patch: XSLT icons, star ratings
icons, table sort icons (the latter which I don't
think can be done using this technique).

A follow-up patch should remove the individual images
which have been replaced by the sprite.

Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7903 follow-up, wrong capital cases
Paul Poulain [Mon, 7 May 2012 21:26:16 +0000 (23:26 +0200)]
Bug 7903 follow-up, wrong capital cases

Also fixing a tiny indentation problem

9 years agoAdd ordernumber column in orders history table
Lyon3 Team [Thu, 5 Apr 2012 13:14:39 +0000 (15:14 +0200)]
Add ordernumber column in orders history table

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
There was a conflict (whitespace changes causing a conflict)

http://bugs.koha-community.org/show_bug.cgi?id=7903
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7949 - a different favicon for the staff client
Gaetan Boisson [Thu, 12 Apr 2012 16:24:05 +0000 (18:24 +0200)]
Bug 7949 - a different favicon for the staff client

different favicon proposal

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agobug 7990 - styl= in aqplan.tt
Stéphane Delaune [Fri, 20 Apr 2012 13:32:10 +0000 (15:32 +0200)]
bug 7990 - styl= in aqplan.tt

Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7953 - add opacuserjs to the sco template
Liz Rea [Thu, 12 Apr 2012 21:54:28 +0000 (16:54 -0500)]
Bug 7953 - add opacuserjs to the sco template

To test:
add some jquery to opacuserjs that would change the screen
a la: $(".checkout label[for='patronid']").css("border","3px solid red");
verify you see a red box on the sco-main page.

Also adds the ability to use opacusercss to style sco-main.tt

to test:
Add some css to opacusercss that would change the screen:
.checkout { background: grey; }
verify you see a grey box around the checkout div.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
http://bugs.koha-community.org/show_bug.cgi?id=7959
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8008 - Fix typo in man page for koha-create
Magnus Enger [Wed, 25 Apr 2012 12:28:32 +0000 (14:28 +0200)]
Bug 8008 - Fix typo in man page for koha-create

Line 121 had "/etc/koha/koha-site.conf" instead of "/etc/koha/koha-sites.conf"

To test:
I don't think this needs any actual testing, just read the patch and check
that it looks sensible...

Signed-off-by: Marc Veron <veron@veron.ch>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agobug 6372: moved background-job.inc to background-job-progressbar.js
Srdjan [Fri, 23 Mar 2012 00:08:54 +0000 (13:08 +1300)]
bug 6372: moved background-job.inc to background-job-progressbar.js

Check for div 0 when calculating percentage

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
div0 no longer appears on 0 size jobs.
Passes t xt

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7929 - Duplicate id "availability" on XSLT result page
Katrin Fischer [Wed, 11 Apr 2012 06:06:00 +0000 (08:06 +0200)]
Bug 7929 - Duplicate id "availability" on XSLT result page

Changes id="availability" to class="availability" for XHTML valididy
(ids have to be unique on a page).

Signed-off-by: mveron <veron@veron.ch>
Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 7987 [SIGNED-OFF] Fix typo in circulation template
Colin Campbell [Fri, 20 Apr 2012 11:16:13 +0000 (12:16 +0100)]
Bug 7987 [SIGNED-OFF] Fix typo in circulation template

li was not properly closed after loss of a character

Signed-off-by: Marc Veron <veron@veron.ch>
- Checked koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt before / after applying patch
- Line is changed as expected, ends now with </li>

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 2399: Follow up removing MySQLisms from ru-RU and uk-UA sample files
Katrin Fischer [Tue, 3 Apr 2012 18:23:47 +0000 (20:23 +0200)]
Bug 2399: Follow up removing MySQLisms from ru-RU and uk-UA sample files

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 2399: All status fields in the item edit interface offer two blank/null entries...
Katrin Fischer [Mon, 2 Apr 2012 17:28:37 +0000 (19:28 +0200)]
Bug 2399: All status fields in the item edit interface offer two blank/null entries per dropdown instead of one

Removes the '0' values without description from the sample date for
authorised values.

To test:
1) Verify that SQL files load correctly
- truncate authorised_values; /* empty authorised values table */
- load sample data using one of the SQL files
2) Check cataloging interface - all status fields should only show 1 empty line

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
9 years agoBug 3413: Checkboxes for new subfields not saving correctly
Katrin Fischer [Mon, 9 Apr 2012 09:44:06 +0000 (11:44 +0200)]
Bug 3413: Checkboxes for new subfields not saving correctly

The bug was in the numbering of the fields and subfields in the form.
The form elements for the 'new subfield' got the same numbered names and ids as
the form elements for the second subfield. (example: repeatable1)
The patch fixes the numbering to continue to the new subfield.

To test:
- Create some new subfields and check all checkbox fields save correctly
  (mandatory, repeatable, is url)
- Check that other subfields' checkboxes are not changed
- Check existing subfields and newly created subfields can be edited

Signed-off-by: mveron <veron@veron.ch>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8020 [3.8.x/MASTER] - update git build script to 3.9
Robin Sheat [Sat, 28 Apr 2012 01:02:26 +0000 (13:02 +1200)]
Bug 8020 [3.8.x/MASTER] - update git build script to 3.9

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8020 - [3.8.x] Update changelog for 3.8.0-1
Robin Sheat [Sat, 28 Apr 2012 00:40:09 +0000 (12:40 +1200)]
Bug 8020 - [3.8.x] Update changelog for 3.8.0-1

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoBug 8020 [3.8.x/MASTER] - update changelogs to get to 3.8
Robin Sheat [Wed, 26 Oct 2011 03:02:32 +0000 (16:02 +1300)]
Bug 8020 [3.8.x/MASTER] - update changelogs to get to 3.8

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoBug 8025: corrects the return of a zero value for authorised values
Adrien Saurat [Mon, 30 Apr 2012 08:05:07 +0000 (10:05 +0200)]
Bug 8025: corrects the return of a zero value for authorised values

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoBug 8027: fix param order for z3950 INSERT
Adrien Saurat [Mon, 30 Apr 2012 10:15:13 +0000 (12:15 +0200)]
Bug 8027: fix param order for z3950 INSERT

In the z3950servers.pl order of columns which are inserted
for new server is wrong, which results in empty timeout
and encoding values when new server is added.

Test scenario:

1. Open Z39.50 server administration
2. add new server with encoding and timeout
3. save server and verify that it has encoding and timeout

Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_3969'
Paul Poulain [Mon, 7 May 2012 19:49:42 +0000 (21:49 +0200)]
Merge remote-tracking branch 'origin/new/bug_3969'

9 years agoBug 3969: Budget search in acquisition doesn't work (string changes)
Katrin Fischer [Sat, 14 Apr 2012 16:12:42 +0000 (18:12 +0200)]
Bug 3969: Budget search in acquisition doesn't work (string changes)

String changes:
- Correcting tab name from "Search budgets" to "Seach funds"
- Changing search option from "Name" to "Fund code"

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Strings look a-ok.

9 years agoBug 8002 follow-up to use 3.09 and MODIFY instead of 3.08 and CHANGE
Paul Poulain [Mon, 30 Apr 2012 12:36:08 +0000 (14:36 +0200)]
Bug 8002 follow-up to use 3.09 and MODIFY instead of 3.08 and CHANGE

see Ian comment 4 (http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8002)

9 years agoBug 8002 [SIGNED-OFF][REVISED] Can't add patron attribute type in newer installation
Owen Leonard [Tue, 24 Apr 2012 15:20:42 +0000 (11:20 -0400)]
Bug 8002 [SIGNED-OFF][REVISED] Can't add patron attribute type in newer installation

Correcting patron_attributes_types definition in kohastructure.sql
to allow NULL entries for category_code, matching the definition
specified in updatedatabase.pl. Adds an entry to updatedatabase.pl
to correct existing installations with the incorrect table structure.

After running the updated updatedatabase.pl the category_code
column in borrower_attribute_types should show that it allows
NULL entries. Adding a new patron attribute type with no
patron category specified should work correctly.

To test the update to kohastructure.sql: Back up your Koha
database, drop it, and recreate it. Load the staff interface
and go through the web installer. After recreating the database
enable ExtendedPatronAttributes and try adding a new patron
attribute type with no patron category specified. The operation
should be successful.

signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
a new update for installations affected.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
9 years agoMerge remote-tracking branch 'origin/new/bug_7849'
Paul Poulain [Mon, 30 Apr 2012 12:15:23 +0000 (14:15 +0200)]
Merge remote-tracking branch 'origin/new/bug_7849'

9 years agoFix typo in patch for bug 7849
Colin Campbell [Tue, 24 Apr 2012 13:27:58 +0000 (14:27 +0100)]
Fix typo in patch for bug 7849

$type declared twice effectively throwing away the returned value

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