Commit Graph

12796 Commits

Author SHA1 Message Date
Owen Leonard 77b824a00d Fix for Bug 4173 - Statuses not appearing in the OPAC
This adds display of "Use restrictions" authorized values
to the OPAC and the staff client for available and
not-for-loan items.

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bdf4dd13af)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-13 22:34:30 -05:00
Chris Nighswonger f8230678e0 Bug 5391 - can't add multiples to batch
This was due to the use of an OR which seems to have caused the
param method to be called in a scalar context thus returning only
the first of multiple repeated cgi params.

This patch retains the intent of the OR by replacing it with a
conditional.

Signed-off-by: Koustubha Kale <kmkale@anantcorp.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-13 22:34:02 -05:00
Chris Nighswonger ecd2cefea6 Updating Version Number to 3.02.01.002 2010-12-13 22:16:12 -05:00
Katrin Fischer 00f560980c Bug 5423: Deleted Patrons are no longer saved in table deletedborrowers
Tested with HEAD
> Should also apply to 3.2.x
( Not tested with 3.2.x)

Signed-off-by: Koustubha Kale <kmkale@anantcorp.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-13 22:10:18 -05:00
Chris Cormack 5acb8a2c60 Adding another developer to the Koha history, 120 now!
(cherry picked from commit 4fc888572b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:27:54 -05:00
Marcel de Rooy fdc9905e35 bug5455 (Fix uninitialized-warnings on authorities.pl)
Fix warnings for uninitialized authtypecode on lines 646, 653.

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 434c1d6483)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:27:53 -05:00
Robin Sheat 0fc64a62ef Bug 5479 - Update package building scripts
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 8b3a2ecebd)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:27:53 -05:00
Liz Rea d4ccf67231 Bug 4935: fix for allowing authorized values to be "unset"
Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 61957d07ce)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:27:53 -05:00
Chris Cormack 0a44de48d8 Updating test for systprefs to give more useful output
(cherry picked from commit 423138606e)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:27:53 -05:00
Owen Leonard 5665a3d4b0 Fix for Bug 2808, Hold queue not alphabetizing
White space in the biblio-default-view include and in the page
template caused sorting on title or patron to be incorrect.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit e29a123a9b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:26:53 -05:00
Frédéric Demians e1cf24e89c Bug 5475 Wrong messages date formating on Check Out page
On the Check Out page, messages related to the borrower are displayed on the
right side. Message text is preceded by a date and library code. The date is
displayed in US format (mm/dd/YYY). Is should be displayed formated depending
on 'dateformat' syspref.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 37a17d8fad)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:26:53 -05:00
Jean-André Santoni 8ad55cbdcf Ergonomy improvement in smart rule management
Added a jQuery table filter, usefull when you have a lot of rows. This filter works on the entire row and allows you for exemple to type "Adul Perio 5".
Indentation cleaning.
Added tabs to filter colums on 4 axes: All, Issues, Fines and Reserves.
Moved the table filter to the right of the tabs.

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit fefd158f42)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:26:52 -05:00
Robin Sheat ddac69c1e8 Bug 4500 - remove commas from price formatting
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 692786b519)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:26:52 -05:00
Colin Campbell 5ce340e8ba Bug 5022 Supplements were not saved correctly
No subscription id was set
Also entered a default arrived date on the supplement
And removed errors due to bad dates returned from GetSerials

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit aa6ce934d9)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-12 14:26:51 -05:00
Owen Leonard 02f21b98fc Fix for 4832 - Re-editing a suggestion changes the item type
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit c9f3f2872f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-11 21:22:18 -05:00
Katrin Fischer 9c2231b935 Bug 5467: Kits shown wrong as visual material in XSLT displays
- reused icon for mixed materials for kits
- fixed missing icon for mixed materials

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 5633b423e3)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-11 21:22:18 -05:00
f.demians at tamil.fr 00dd132acd Bug 5446 Item creation in Acquisition module doesn't control mandatory field
Add some JS validing form, so saving an order is blocked if any item mandatory
field is empty.

Signed-off-by: Koustubha Kale <kmkale@anantcorp.com>
(cherry picked from commit 76264638cc)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-11 21:22:17 -05:00
Katrin Fischer ff14e53c02 Add BSZ to Koha's About page
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 27828231d4)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-11 21:22:17 -05:00
Owen Leonard d941d85712 Incremental fix for Bug 3285 - Location on Add Subscription should be pull down
- Adds location dropdown to subscription edit form
- Pulls authorised value for display on subscription detail page
- Adds function for pulling authorised value description based
  on category and value

This patch does NOT implement automatic preselection of the shelving
location form field on the serial add item screen. This must be
worked out in order for the bug to be closed.

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit e54169ba0f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-06 17:19:13 -05:00
Ian Walls 7f8130e433 Bug 5460: AutoEmailPrimaryAddress set to invalid value in sysprefs.sql
Fixes de-DE, en, fr-FR, pl-PL, ru-RU and uk-UA default install values for this system preference (changing from '0'
to 'OFF', the preferred value).

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit b112ec8ff4)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-06 17:19:13 -05:00
Ian Walls f86e6f5d24 Bug 5457: gather_print_notices.pl requires explicit stylesheet
Adds a conditional around the opening and closing of STYLESHEET, testing on whether the $stylesheet variable is set
or not.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 31622bc6b3)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-06 17:19:12 -05:00
Galen Charlton ce763e58ff bug 4908: rename temlate variable to branchname
Using branchname is more accurate and could avoid
a bug if/when the patron update function is made more
sophisticated.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bb032138e7)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-02 19:51:29 -05:00
Owen Leonard 4e0f420eed Fix for Bug 4908 - OPAC patron details page doesn't show patron's home library
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d86c4aa894)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-02 19:49:01 -05:00
Galen Charlton 1a3ae8f0d3 bug 5397: display renew count consistently
Display the renewal count for loans consistently; in particular,
for loans that have never been renewed, display 0.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 02524213b3)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-02 19:10:35 -05:00
Owen Leonard 0935ee139e Fix for Bug 4983 - Add edit record / edit item links to search results
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 05dd542ceb)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-02 19:10:18 -05:00
Owen Leonard adf93917bd Fix for Bug 4851 - Batch delete shouldn't show delete button if no items were found
- Adding template flag to indicate valid results (hidden or not)
 - Clarifying warning messages based on various outcomes

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit e6ebf10bba)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-02 19:09:07 -05:00
Owen Leonard 157ee4096b Fix for Bug 5048, Error in menu/submenu for select language in intranet
Patch for HEAD. This issue was apparently solved quite a while
ago in the OPAC. This patch copies that solution over to the staff client.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 1024eceec7)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 17:00:46 -05:00
Owen Leonard 1c3214b6e6 Fix for Bug 5030 - Improve handling of duplicate patrons
- Making duplicate patron warning dialog simpler but more explicit
- Pointing duplicate patron preview window to new brief version
  of moremember.pl to display more patron details than the previous
  version.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 9f0495238d)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 17:00:28 -05:00
Frédéric Demians 3f9b09fa0a Bug 5444 Fix misc/translate script for 'standard' Koha install
[3.2] Should apply

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit be65390d06)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:59:53 -05:00
Ian Walls 90bb50959d Bug 4449: AllowHoldPolicyOverride does not override all hold policies
This patch provides a universal override to any hold policy if the AllowHoldPolicyOverride is set.
Amongst other overrides, this patch allows missing, non-checked out items to be placed on hold even with AllowShelfHolds turned off.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 5e8b078651)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:59:37 -05:00
Owen Leonard 777ab9f7d1 Fix for Bug 5029 - Update patron deletion error page
- Styling error message with "dialog alert" class
- Adding sidebar patron details to match other patron pages

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit f7b3b0c75b)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:57:47 -05:00
Galen Charlton 2651340809 bug 5405: remove disused script catalogue/suggest.pl
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 9753368626)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:57:33 -05:00
Owen Leonard b358815bfb Further fixes for Bug 5142 - Untranslatable strings in tag review template
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 470dc1d37c)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:55:44 -05:00
Chris Cormack be3d9f6c13 Adding 3.2.1 release to the history
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit a1aa1eea16)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:55:44 -05:00
Chris Cormack 55bb361248 Fixing formatting and links in the history document
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 5a71ffa0d3)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:55:44 -05:00
Colin Campbell 288cbb1f57 Fix some compile time errors reported in test suite
redeclaration of itemnumber in RotatingCollections
A couple of errors caused by retaining commas in qw()

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit bc3dcdd733)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:55:44 -05:00
Colin Campbell ae0387e21e Bug 5448: Refactor Boolean.pm
Remove unnecesssary export
Use Exporter according to best practices
Use Carp not warn so we know where the error string came from
Don't create warnings manipulating undefined input
Replace package variables by lexicals

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit f08b33a912)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:54:52 -05:00
Colin Campbell c99ce7f3fd Bug 5448 Use Test::More in Boolean.t
Use Test::More's structure as more readable
Replaced a test of an used function with a use_ok test
of the module

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 78be08c73e)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:54:52 -05:00
Ian Walls faead6cbd2 Fixes bug 5232: Shelfname won't display on high-numbered lists
This bugfix adds an explicit GetShelf for the specific shelfnumber when viewing that individual shelfnumber.  This captures the
shelfname, which was originally pulled from shelflist->{$shelfnumber}, which may not have been populated if the List in question was
beyond the 20th private or public list in the system.

Also adds a more explicit sort: the sort from the CGI takes top priority, then it falls back to the default sort for the list, and
ultimate falls to 'title' if neither CGI nor default sort exist.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 31439fc51a)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-12-01 16:54:51 -05:00
Colin Campbell bd5e6e9d89 Bug 5450 Avoid a name clash in ILSDI modules
C4::ILSDI::Utility exports a subroutine CanBookBeReserved
Both ILSDI were also importing a subroutine of that name from
C4::Reserves Remove conflict by listing subroutines
imported from C4::Reserves explicitly

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit b9ccc51841)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-30 16:24:42 -05:00
Owen Leonard dad8b9c270 Fix for Bug 5402, "add to cart" shown on checkbox hover
This patch removes the title hint altogether. The checkbox might
be used for the cart, lists, holds, or tags, and expressing this
in a title hint seems overly complicated.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit e7321e1a7f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-30 16:24:42 -05:00
Galen Charlton 373cc1b42e bug 4451 followup - don't need /g for this replace
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit d056b2f62a)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 20:35:06 -05:00
Owen Leonard 82e19e06e9 Fix for Bug 4451 Batch item tool can't process file with Windows line endings
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 3f5d9537db)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 20:35:05 -05:00
Owen Leonard dd79498768 Fix for Bug 4981, Tweak style of item listing on MARC display
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 781b4e5fae)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:11:57 -05:00
Owen Leonard ebb6e40ccd Fix for Bug 5051, Renewal due date doesn't always show on patron Checkout tab
Display of the renewal date footer in the template was conditional upon
the existence of checkouts from today.

In order to fix this bug it was necessary to repeat the markup for the
table footer twice in the template, so I created an include file for it.
The same include file can now be used in moremember.tmpl.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 083d3b1a92)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:11:57 -05:00
Owen Leonard 92b4ccdbaf Fix for Bug 5269 - link to patron edit form when editing turned off
Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit 16c891092f)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:11:57 -05:00
Owen Leonard 3e0ffc7d19 Fix for Bug 5281, "Check in" then "Renew" checkboxes checked in the same time
This fix corrects the behavior of the checkboxes whether the user clicks the
checkbox itself or the <td> around it. Tested on a PC in Firefox 3.6, Chrome,
Safari 5, and IE 8.

Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
(cherry picked from commit cc57c2688d)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:08:28 -05:00
Galen Charlton a65d4188b2 Updating Version Number to 3.02.01.001
bug 5127: database update to add the suggestion notification templates

Being especially careful to not scribble over any ones
that the library might have already defined.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:05:31 -05:00
Galen Charlton 9273f354a9 bug 5127: sample notice templates for suggestions
* supply default suggestion status change email
  templates.
* remove the deprecated mail_suggestion_*.tmpl files

There are a couple suggestion statuses referred to
in the code and suggestion management page that currently
do not have notice templates defined: ASK and CHECKED.  These
seem to be internal, so perhaps default message templates
are not needed for these.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit 7fd45b9861)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:01:37 -05:00
Galen Charlton 19e494bcd5 bug 4211: ensure that suggestion emails have full details
Because of how the ModFoo routines work, if you are only
changing selected columns but are also using the change
as a trigger to place a message in the queue, you need
to fetch the full row before generating the message.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
(cherry picked from commit d3df54d052)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
2010-11-29 15:01:37 -05:00