koha.git
15 years agobump to 106, bug 2452, subscriptionhistory.enddate can be 0000-00-00
Paul POULAIN [Sat, 2 Aug 2008 17:22:02 +0000 (19:22 +0200)]
bump to 106, bug 2452, subscriptionhistory.enddate can be 0000-00-00

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agodealing with L' and D' in french
Paul POULAIN [Sat, 2 Aug 2008 17:22:01 +0000 (19:22 +0200)]
dealing with L' and D' in french

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agounimarc plugin fix, suggested by Zeno Tajoli
Paul POULAIN [Sat, 2 Aug 2008 14:53:14 +0000 (16:53 +0200)]
unimarc plugin fix, suggested by Zeno Tajoli

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoSerials planning: Update subscription edit to properly deal with date changes. (...
Ryan Higgins [Mon, 4 Aug 2008 01:37:51 +0000 (20:37 -0500)]
Serials planning: Update subscription edit to properly deal with date changes. ( patch 3 / 3 )

Previously subscription-add.pl allowed modification of 'firstacquidate', which changed
the subscription definition, but did not affect prediction.  This patch adds two fuctions
to Serials.pm to get/set the current expected issue date (note that all date calculations
in prediction patterns are based on the current expected date, and there's only one serial
issue per subscription in the 'expected' status at any time).  Subscription editing
now allows you to edit the next expected date, but not the first acqui date (unless you
haven't received any issues yet), thus allowing for adjustments in the prediction pattern.
This patch also updates fixes some discrepancies in irregularities / prediction display.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoUpdate serials planning javascript to properly deal with 3-level periodicity. ( Seria...
Ryan Higgins [Mon, 4 Aug 2008 01:10:15 +0000 (20:10 -0500)]
Update serials planning javascript to properly deal with 3-level periodicity. ( Serials patch 2/3 )

Minor changes to dom object names and id's to allow for proper calculation of the
base planning table from the simplified table.
Prior to this patch, the base table was populated with the incorrect value, so patterns like
Vol{X} No{Y} Issue{Z} incremented the Vol value incorrectly.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoSerials planning updates patch 1/3
Ryan Higgins [Mon, 4 Aug 2008 01:05:55 +0000 (20:05 -0500)]
Serials planning updates patch 1/3

This patch addresses usability and interface bugs in the javascript irregularity checks by
adding extra controls: 'test pattern' button (always visible) replaces the 'irregularity' link
that was previously only sometimes visible.  A 'show advanced pattern' button will display/hide the
base prediction table at any time.  A 'reset pattern' button is added.  The form may now also be submitted
even if it fails the irregularity test, as there are cases when this will result in the desired behavior.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoFixing markup error in previous commit.
Owen Leonard [Fri, 1 Aug 2008 20:24:35 +0000 (15:24 -0500)]
Fixing markup error in previous commit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 2450: expanded size of cardnumber input field
Danny Bouman [Fri, 1 Aug 2008 20:08:51 +0000 (16:08 -0400)]
bug 2450: expanded size of cardnumber input field

Expanded the size of the cardnumber input field to 20 instead of 10. This allows the full 14-digit barcode to be viewed at a glance without having to
scroll.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBugfix #2414
Frederic Demians [Tue, 29 Jul 2008 09:05:37 +0000 (11:05 +0200)]
Bugfix #2414

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBUGFIX fix on 2380 fix, dump to 102
Paul POULAIN [Wed, 30 Jul 2008 15:03:05 +0000 (17:03 +0200)]
BUGFIX fix on 2380 fix, dump to 102

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoCorrecting structure and markup.
Owen Leonard [Fri, 1 Aug 2008 16:33:11 +0000 (11:33 -0500)]
Correcting structure and markup.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoComplete sweep of help files in order to correct invalid markup and fix Bug 2444...
Owen Leonard [Fri, 1 Aug 2008 14:00:00 +0000 (09:00 -0500)]
Complete sweep of help files in order to correct invalid markup and fix Bug 2444(break up long lines in templates).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agosmall typo/html fixes as suggested by Zeno Tajoli
Paul POULAIN [Thu, 31 Jul 2008 11:21:31 +0000 (13:21 +0200)]
small typo/html fixes as suggested by Zeno Tajoli

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 2335: adding SMSSendDriver system preference - DB update to version 3.00.00.105
Andrew Moore [Wed, 30 Jul 2008 16:18:02 +0000 (11:18 -0500)]
bug 2335: adding SMSSendDriver system preference - DB update to version 3.00.00.105

Previously, the SMSSendDriver system preference was set as a local use system
preference. This patch makes it an official system preference under the "Patron" tab.
This system preference determines which SMS::Send driver is used to send SMS messages.

I have attempted to take care to not overwrite this system preference if it has already
been set.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoCleaning up tags display a little bit, adding tablesorter to user tags table.
Owen Leonard [Thu, 31 Jul 2008 17:03:52 +0000 (12:03 -0500)]
Cleaning up tags display a little bit, adding tablesorter to user tags table.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoMinor markup corrections.
Owen Leonard [Thu, 31 Jul 2008 17:03:43 +0000 (12:03 -0500)]
Minor markup corrections.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953: updating bad unit test for C4::Items::GetItemsForInventory
Andrew Moore [Thu, 31 Jul 2008 13:58:43 +0000 (08:58 -0500)]
Bug 1953: updating bad unit test for C4::Items::GetItemsForInventory

The tests I wrote for C4::Items::GetItemsForInventory confused the differences
between biblionumber and itemnumber. That wasn't uncovered on my limited test
database, but I uncovered it later.

This fixes that problem by populating a $self->{'items'} list with details of any items
added by KohaTest::add_biblios. Then, tests can probe there for the details of items
they should expect to find when searching.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoNeed to pass dateformat pref to the template so tablesorter js knows how to handle...
Owen Leonard [Thu, 31 Jul 2008 13:52:53 +0000 (08:52 -0500)]
Need to pass dateformat pref to the template so tablesorter js knows how to handle sorting date column.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoDisplay publicationdate if available, else copyrightdate.
Owen Leonard [Thu, 31 Jul 2008 13:52:44 +0000 (08:52 -0500)]
Display publicationdate if available, else copyrightdate.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoReorganizing this display for better usability and to fix bug 2290 ("Claims Should...
Owen Leonard [Thu, 31 Jul 2008 13:43:11 +0000 (08:43 -0500)]
Reorganizing this display for better usability and to fix bug 2290 ("Claims Should not Have Send Button if nothing to claim"). Adding jquery tablesorter. This also adds a message to ask the user to select a supplier if none is selected (see Bug 2338).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 2423: actually ignore already-imported records
Galen Charlton [Wed, 30 Jul 2008 23:08:02 +0000 (18:08 -0500)]
bug 2423: actually ignore already-imported records

When recommiting a partially completed MARC
record batch, records that were already imported
(or had an error status) were being processed
again, leading to duplicate bibs.  Corrected
so that these records are actually ignored.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoPartial fix for Bug 2420 ("OPAC shows broken image for some Google Jackets"). This...
Owen Leonard [Wed, 30 Jul 2008 17:45:45 +0000 (12:45 -0500)]
Partial fix for Bug 2420 ("OPAC shows broken image for some Google Jackets"). This fix only corrects for undefined img src, not for well-formed but broken img src.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agofrench updated
Paul POULAIN [Wed, 30 Jul 2008 15:03:06 +0000 (17:03 +0200)]
french updated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agofrench updated
Paul POULAIN [Wed, 30 Jul 2008 15:03:04 +0000 (17:03 +0200)]
french updated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoTweak to results display to correct punctuation and add ISBN.
Owen Leonard [Wed, 30 Jul 2008 14:27:12 +0000 (09:27 -0500)]
Tweak to results display to correct punctuation and add ISBN.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoC4::Context::import reloaded
Marc Chantreux [Wed, 30 Jul 2008 11:27:57 +0000 (13:27 +0200)]
C4::Context::import reloaded

the problem was a context was stored every time C4::Context was called.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoRemoving Billing.pl link, will not even slightly work with InnoDB tables as there...
Darrell Ulm [Mon, 28 Jul 2008 17:05:22 +0000 (13:05 -0400)]
Removing Billing.pl link, will not even slightly work with InnoDB tables as there is no fast count, will work on later

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoTag cloud implementation in jquery.
Joe Atzberger [Wed, 30 Jul 2008 01:23:33 +0000 (20:23 -0500)]
Tag cloud implementation in jquery.

Initial implementation of "cloud" display on opac-tags.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [6/6]: adding pod documentation for C4::Koha::displayServers
Andrew Moore [Tue, 29 Jul 2008 16:42:50 +0000 (11:42 -0500)]
Bug 1953 [6/6]: adding pod documentation for C4::Koha::displayServers

This method was not documented. I added some POD to it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [5/6]: whitespace changes: perltidy.
Andrew Moore [Tue, 29 Jul 2008 16:42:49 +0000 (11:42 -0500)]
Bug 1953 [5/6]: whitespace changes: perltidy.

ran perltidy on the whole method since I've rewritten most of it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [4/6]: removing dead code from C4::Koha::displayServers
Andrew Moore [Tue, 29 Jul 2008 16:42:48 +0000 (11:42 -0500)]
Bug 1953 [4/6]: removing dead code from C4::Koha::displayServers

displayServers appears to have had old code in it that was no longer used. I'm removing it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [3/6]: refactoring to remove superfluous where clause.
Andrew Moore [Tue, 29 Jul 2008 16:42:47 +0000 (11:42 -0500)]
Bug 1953 [3/6]: refactoring to remove superfluous where clause.

refactored the SQL query to remove a confusing and superfluous 'WHERE 1' clause.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [2/6]: refactoring SQL in C4::Koha::displayServers to use placeholders.
Andrew Moore [Tue, 29 Jul 2008 16:42:46 +0000 (11:42 -0500)]
Bug 1953 [2/6]: refactoring SQL in C4::Koha::displayServers to use placeholders.

The SQL call in displayServers was not using placeholders, leaving itself open
to potential SQL injection attacks. I've rewritten it to use placeholders.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [1/6]: adding test module for C4::Koha::displayServers
Andrew Moore [Tue, 29 Jul 2008 16:42:45 +0000 (11:42 -0500)]
Bug 1953 [1/6]: adding test module for C4::Koha::displayServers

This test suite tests the several different ways that we can call C4::Koha::displayServers.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 2413: remove noOPACHolds syspref (DB rev 104)
Galen Charlton [Mon, 28 Jul 2008 21:05:42 +0000 (16:05 -0500)]
bug 2413: remove noOPACHolds syspref (DB rev 104)

Preference was removed in patch for bug 2234; completing
removal in existing 3.0 installations via DB rev 104.

No documentation changes (deprecation of noOPACHolds
done in an earlier patch)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2404 This patch removes Image::Magick and adds GD
Chris Nighswonger [Mon, 28 Jul 2008 16:35:46 +0000 (11:35 -0500)]
kohabug 2404 This patch removes Image::Magick and adds GD

This patch removes Image::Magick as a dependency and replaces it with
the (much) lighter GD. Functionality of patronimage code has not changed with
this conversion.

Adding errorhandling for corrupted image file and link to return to moremember.pl when called from there

Added notes about supported image mime types.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoA couple more corrections that should have been in previous commit.
Owen Leonard [Mon, 28 Jul 2008 13:15:45 +0000 (08:15 -0500)]
A couple more corrections that should have been in previous commit.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoFix for Bug 2408 ("Note: separated from content in cgi-bin/koha/serials/subscription...
Owen Leonard [Mon, 28 Jul 2008 12:50:14 +0000 (07:50 -0500)]
Fix for Bug 2408 ("Note: separated from content in cgi-bin/koha/serials/subscription-add.pl") and other markup corrections.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoUpdate to help file
Nicole Engard [Thu, 24 Jul 2008 15:53:22 +0000 (11:53 -0400)]
Update to help file

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoSmall patch again to fix repeated col in reserveratios.tmpl
root [Fri, 25 Jul 2008 14:39:36 +0000 (10:39 -0400)]
Small patch again to fix repeated col in reserveratios.tmpl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug fixing : the link between serial and item wasnot used when editing an issue
Henri-Damien LAURENT [Fri, 4 Jul 2008 06:20:50 +0000 (08:20 +0200)]
Bug fixing : the link between serial and item wasnot used when editing an issue

table serialitems was not used when getting serialinformation.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953: refactoring C4::Koha::get_itemtypeinfos_of to eliminate potential SQL injection
Andrew Moore [Fri, 25 Jul 2008 20:31:11 +0000 (15:31 -0500)]
Bug 1953: refactoring C4::Koha::get_itemtypeinfos_of to eliminate potential SQL injection

C4::Koha::get_itemtypeinfos_of was not using plceholders, opening itself up to
potential SQL injection attacks. This patch refactors it to use placeholders to
bind parameters.
I also had to extend C4::koha::get_infos_of to allow us to pass bind parameters into it.
I'm including a test module for C4::Koha::get_itemtypeinfos_of.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2345: bad comparisons pervade reports
Jesse Weaver [Fri, 25 Jul 2008 17:32:05 +0000 (12:32 -0500)]
kohabug 2345: bad comparisons pervade reports

This patch changes all $foo eq undef's to !defined($foo). It also makes misc/spellcheck_suggest/build_spellcheck_suggest.pl have proper syntax.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [3/3]: documentation changes for C4::items::GetItemsForInventory
Andrew Moore [Fri, 25 Jul 2008 16:55:13 +0000 (11:55 -0500)]
Bug 1953 [3/3]: documentation changes for C4::items::GetItemsForInventory

This patch corrects what appears to me to be a few defficiencies in the documentation
for C4::items::GetItemsForInventory. I noticed them while writing test methods for this sub.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [2/3]: refactoring SQL in C4::Items::GetItemsForInventory to use placeholders
Andrew Moore [Fri, 25 Jul 2008 16:55:12 +0000 (11:55 -0500)]
Bug 1953 [2/3]: refactoring SQL in C4::Items::GetItemsForInventory to use placeholders

The SQL in C4::Items::GetItemsForInventory wasn't using placeholders and
bind parameters, possibly leaving itself open ot SQL injection attacks. This
patch changes that.
I've also incliuded a test module for C4::items::GetItemsForInventory.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 1953 [1/3]: test suite improvements
Andrew Moore [Fri, 25 Jul 2008 16:55:11 +0000 (11:55 -0500)]
Bug 1953 [1/3]: test suite improvements

Here are a few improvments to the test suite to make it easier to write some tests
for C4::Items

I extracted "tomorrow" and "yesterday" methods from a test module into the base class
so that they could be used by multiple test modules

Adding callnumber to items added in the test suite.

I recatored KohaTest::add_biblios a bit to remove the manual count of the number of
MARC::Fields that were added.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2379 When a borrower is added manually in Koha...
Jesse Weaver [Fri, 25 Jul 2008 16:51:34 +0000 (11:51 -0500)]
kohabug 2379 When a borrower is added manually in Koha...

This checks that the enrolmentfee for a category both exists, and is greater than 0, avoiding a spurious charge.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2224 Third installment: Further corrections for NULL fields
Chris Nighswonger [Fri, 25 Jul 2008 15:37:51 +0000 (10:37 -0500)]
kohabug 2224 Third installment: Further corrections for NULL fields

This patch adds code to clean out any NULL fields in rows returned
since NULL fields do not truly meet the filter criteria

It also corrects a bad db field reference.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2390 Setting default fund for new orders
Chris Nighswonger [Fri, 25 Jul 2008 15:37:49 +0000 (10:37 -0500)]
kohabug 2390 Setting default fund for new orders

If a new order was created and no fund selected, a db error was thrown due to the
'bookfundid' field being set to NULL. This patch sets the first fund retrieved from
available funds as the default selection in the scrolling list.

A further enhancement might be to allow the library to choose which fund is the
default fund.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2219 Corrects inconsistent use of subscription.serialsadditems column
Chris Nighswonger [Fri, 25 Jul 2008 15:37:50 +0000 (10:37 -0500)]
kohabug 2219 Corrects inconsistent use of subscription.serialsadditems column

From the bug report:

The serialsadditems syspref was ostensibly removed in DB rev 071
(http://git.koha.org/cgi-bin/gitweb.cgi?p=Koha;a=commit;h=5c41ae54e68866f9661e853376537059f4d83f70)
in favor of a new serialsadditems column in the subscription table.

However, this removal was incomplete.  It is still created for new installations by:

installer/data/mysql/en/mandatory/sysprefs.sql
installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql

and is used in

serials/serials-recieve.pl
serials/serials-edit.pl
admin/systempreferences.pl

Since the system preference was not removed from the sample data scripts, it
is necessary to add another DB rev to remove it - a user may have made a
fresh install of Koha after DB rev 071.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2380 Correcting serialitems table structure and fixing duplicate barcode...
Chris Nighswonger [Fri, 25 Jul 2008 15:44:30 +0000 (10:44 -0500)]
kohabug 2380 Correcting serialitems table structure and fixing duplicate barcode check

The current serialitems table structure does not provide for a *:1 relationship with
the serial table. This causes a problem when attempting to add multiple items to a given
serial. The db throws an error when attempting to INSERT in  serialitems due to serialid.serialitems
being a unique key. A further side effect is that the marc record is updated with the
item inspite of the error. The mods to the serialitems table structure in this patch
drop serialid.serialitems as a key and make itemnumber.serialitems the primary key
creating a *:1 relationship with the serial table. This patch also makes serialid.serialitems
a foreign key referencing serialid.serial to maintain referential integrity.

Fix for duplicate barcode check

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoPopulate items.cn_source in acq receive. However, items.cn_sort is not calculated...
Ryan Higgins [Fri, 25 Jul 2008 14:35:44 +0000 (09:35 -0500)]
Populate items.cn_source in acq receive. However, items.cn_sort is not calculated. See bug 2403.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBUG 2351 : Add duplicate barcode check prior to receiving multiple items. This patch...
Ryan Higgins [Fri, 25 Jul 2008 14:04:07 +0000 (09:04 -0500)]
BUG 2351 : Add duplicate barcode check prior to receiving multiple items. This patch adds a javascript form check for duplicates in-form, and returns an error if there are duplicates in the catalog.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoExample cronjob file.
Joe Atzberger [Thu, 24 Jul 2008 21:11:18 +0000 (16:11 -0500)]
Example cronjob file.

We have 5 or 6 different cron'd tasks for a single installation now,
so an example is in order.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoTZ - multiple timezone support
Joe Atzberger [Thu, 24 Jul 2008 21:11:17 +0000 (16:11 -0500)]
TZ - multiple timezone support

Support multiple timezones via Apache SetEnv.  See the perldoc for
admin/env_tz_test.pl on how to configure and test.  Minimal changes
to Context itself.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoSpelling corrections (Fixes Bug 2381).
Owen Leonard [Wed, 23 Jul 2008 13:28:49 +0000 (08:28 -0500)]
Spelling corrections (Fixes Bug 2381).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoThis patch fixes the OPAC view and holdability of items:
Joshua Ferraro [Fri, 25 Jul 2008 16:32:06 +0000 (11:32 -0500)]
This patch fixes the OPAC view and holdability of items:

1. restores a check to itemtype.notforloan to set the norequests flag
2. changes improper boolean OR with AND for checking conditions of setting norequests
3. displays 'Not for loan' for item-level itypes when the itemtype is set to notforloan
4. restores items.notforloan values < 0 allowing holds (ordered items for instance)

We still need a notforhold flag set at the itemtype, and items level

15 years agoIf the authorized value description doesn't exist, display the value
Joshua Ferraro [Fri, 25 Jul 2008 15:08:31 +0000 (10:08 -0500)]
If the authorized value description doesn't exist, display the value

15 years agoUpdate CalcFine to obey maxFine systempreference.
Ryan Higgins [Fri, 25 Jul 2008 07:58:01 +0000 (02:58 -0500)]
Update CalcFine to obey maxFine systempreference.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2154 Modifying form input controls to accomodate translations
Chris Nighswonger [Thu, 24 Jul 2008 23:24:41 +0000 (18:24 -0500)]
kohabug 2154 Modifying form input controls to accomodate translations

Due to the logic of the underlying picture-upload.pl depending upon the "value" of the
form input controls and this value being translated, the script then failed to function.
This patch changes the input controls so that this should not be an issue.

This issue should be kept in mind, though, so that it can be avoided in the future.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoFix 'Catalog by Itemtype' report to work with item-level itypes
Joshua Ferraro [Fri, 25 Jul 2008 13:49:45 +0000 (08:49 -0500)]
Fix 'Catalog by Itemtype' report to work with item-level itypes

15 years agoFix for bug 2398: Source of Classification - Add item or Edit item in
Joshua Ferraro [Thu, 24 Jul 2008 21:23:33 +0000 (16:23 -0500)]
Fix for bug 2398: Source of Classification - Add item or Edit item in
staff client is defaulting to the ASNCR code instead of the blank above

For some reason the loop used to build the list of values was
setting the first value to the $value variable if $value wasn't
set already, and the default_value was inside rather than outside
the loop. I've removed the setting inside the loop and placed
the default value outide the loop. It's possible I just don't
understand what the original intention was.

15 years agokohabug 2224 Second installment: Correcting NULL field display
Chris Nighswonger [Thu, 24 Jul 2008 19:14:41 +0000 (14:14 -0500)]
kohabug 2224 Second installment: Correcting NULL field display

Query references to the unused aqorderdelivery table caused a NULL field to be
displayed in the report results. This patch removes those references.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAdding a couple of IDs to the markup to enable more flexibility in custom styling.
Owen Leonard [Thu, 24 Jul 2008 18:52:30 +0000 (13:52 -0500)]
Adding a couple of IDs to the markup to enable more flexibility in custom styling.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoStyling tabs on virtual shelves page. Did I forget this, or did it get removed at...
Owen Leonard [Thu, 24 Jul 2008 18:48:23 +0000 (13:48 -0500)]
Styling tabs on virtual shelves page. Did I forget this, or did it get removed at some point?

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2139 Fixes C4::Aquisition::NewOrder so that it saves the branchcode
Chris Nighswonger [Thu, 24 Jul 2008 18:44:19 +0000 (13:44 -0500)]
kohabug 2139 Fixes C4::Aquisition::NewOrder so that it saves the branchcode

&NewOrder did not save the branchcode posted with a new order. This patch adds that param.

Added code to select the branch the order is for in the branch dropdown list on
acqui/orderreceive.pl

Updating POD and tests

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2224 Corrects display descrepancies when applying filters
Chris Nighswonger [Thu, 24 Jul 2008 15:29:45 +0000 (10:29 -0500)]
kohabug 2224 Corrects display descrepancies when applying filters

This patch corrects display descrepancies when filters are applied to the
acquisitions report. It also corrects a number of other issues with this
report.

Some documentation changes may be in order based on report filters that
were only partially implimented, but are now available.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years ago(minor) remove unconditional non-error warn
Galen Charlton [Thu, 24 Jul 2008 15:11:30 +0000 (10:11 -0500)]
(minor) remove unconditional non-error warn

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoMinor tweak to tags display.
Owen Leonard [Thu, 24 Jul 2008 14:57:37 +0000 (09:57 -0500)]
Minor tweak to tags display.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAdding js table sorter for holdings.
Owen Leonard [Thu, 24 Jul 2008 12:26:41 +0000 (07:26 -0500)]
Adding js table sorter for holdings.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2392 Changing array dereferencing syntax
Chris Nighswonger [Thu, 24 Jul 2008 02:23:35 +0000 (21:23 -0500)]
kohabug 2392 Changing array dereferencing syntax

It appears that Perl 5.10 does not like ${@$foo}[0] but rather wants $foo->[0]
The latter is also much more readable. This patch makes the change.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBUGFIX 2387 : Pass vendor id to subsequent pages in neworder search from existing...
Ryan Higgins [Thu, 24 Jul 2008 01:17:46 +0000 (20:17 -0500)]
BUGFIX 2387 : Pass vendor id to subsequent pages in neworder search from existing biblio.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoHide the payment/writeoff dropdown on lines that are credits (or forgiven).
Ryan Higgins [Thu, 24 Jul 2008 01:02:46 +0000 (20:02 -0500)]
Hide the payment/writeoff dropdown on lines that are credits (or forgiven).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoRemove 'Cash Refund' from manual invoice, as it calls a deprecated function. Change...
Ryan Higgins [Thu, 24 Jul 2008 01:02:21 +0000 (20:02 -0500)]
Remove 'Cash Refund' from manual invoice, as it calls a deprecated function. Change the code for 'forgiven' from 'F' to 'FOR', since 'F' is used for 'Fine' elsewhere in the code. Mark other subroutines as deprecated that utilize the accountoffsets table, since that functionality was lost somewhere in the 2.2 series, and half-using it breaks things.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoCombine 'publication details' and volume columns in display to work with mixed serial...
Ryan Higgins [Wed, 23 Jul 2008 20:04:37 +0000 (15:04 -0500)]
Combine 'publication details' and volume columns in display to work with mixed serial data (i.e. some issues linked to serials module, some not). Cleans up display after patch to update sorting with enumchron based on biblio.serial.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoUpdate holdings sort to sort by items.enumchron if there's no serial record and the...
Ryan Higgins [Wed, 23 Jul 2008 20:04:19 +0000 (15:04 -0500)]
Update holdings sort to sort by items.enumchron if there's no serial record and the biblio.serial flag is set.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 1953 [2/2]: whitespace and documentation changes
Andrew Moore [Wed, 23 Jul 2008 19:27:56 +0000 (14:27 -0500)]
bug 1953 [2/2]: whitespace and documentation changes

perltidy and documentation enhancements.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 1953 [1/2]: fixing SQL injection problem in C4::Context->preference
Andrew Moore [Wed, 23 Jul 2008 19:27:55 +0000 (14:27 -0500)]
bug 1953 [1/2]: fixing SQL injection problem in C4::Context->preference

C4::Context->preference was not using placeholders and was potentially vulnerable to
a SQL injectin attack. This patch refactors the method to use placeholders.

Added some tests for C4::Context.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agobug 2386: adding newline between items in overdue notices
Andrew Moore [Wed, 23 Jul 2008 18:19:57 +0000 (13:19 -0500)]
bug 2386: adding newline between items in overdue notices

The lines for each item in the overdue notices were not separated by newlines. This cause them to
all be smashed together. I'm putting a newline between them.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoUTF-8 to ASCII MAPPINGS -- WARNING: REINDEX REQUIRED
Joshua Ferraro [Wed, 23 Jul 2008 14:49:57 +0000 (09:49 -0500)]
UTF-8 to ASCII MAPPINGS -- WARNING: REINDEX REQUIRED

You've been warned :-). This patch contains a more
complete mapping of UTF-8 to ASCII. The mappings are
based on those compiled by Richard Mahoney on the
Zebra list: http://lists.indexdata.dk/pipermail/zebralist/2007-August/001707.html

Note to documentation team: we need an area in the
documentation that discusses how Koha handles searches
and indexing for words that contain diacritics, such
as E-ACUTE (vs E without an acute). If you can paste
this list of mappings from this patch directly into
the docs and it preserves the encoding that would be
great.

NOTE: I don't think this patch addresses issues of
combining vs non-combining forms, and may require
a refactor to address that.

Josh

15 years agoCorrecting C4::VirtualShelves::ShelfPossibleAction to handle the case where the staff...
Chris Nighswonger [Wed, 23 Jul 2008 04:42:19 +0000 (23:42 -0500)]
Correcting C4::VirtualShelves::ShelfPossibleAction to handle the case where the staff user is the superlibrarian ($user == 0)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoCorrecting bad session var that caused Lists link not to be displayed in anonymous...
Chris Nighswonger [Wed, 23 Jul 2008 04:29:25 +0000 (23:29 -0500)]
Correcting bad session var that caused Lists link not to be displayed in anonymous sessions

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoDB UPDATE : Patron category code was defined incorrectly in overduerules (notice...
Ryan Higgins [Wed, 23 Jul 2008 01:52:16 +0000 (20:52 -0500)]
DB UPDATE : Patron category code was defined incorrectly in overduerules (notice/status triggers) and deletedborrowers.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2374 Corrects autoBarcode behavior in additem.pl
Chris Nighswonger [Tue, 22 Jul 2008 18:43:34 +0000 (13:43 -0500)]
kohabug 2374 Corrects autoBarcode behavior in additem.pl

There should be a documenation change at least for 3.0 to the effect that on systems
not running JavaScript, 'incremental' is the only option available for autoBarcode
although other options appear. A fix for this will hopefully appear in 3.2.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2150 Correcting (somewhat) the inconsistent use of autoBarcode syspref
Chris Nighswonger [Tue, 22 Jul 2008 18:43:33 +0000 (13:43 -0500)]
kohabug 2150 Correcting (somewhat) the inconsistent use of autoBarcode syspref

This patch corrects somewhat the inconsistent use of the autoBarcode syspref in serials-edit.pl
It does not impliment all the possibilities currently available in the autoBarcode syspref as
there is currently no support for the js plug-in/drop-in feature in serials-edit.pl. So in this
sense, the autoBarcode syspref is still used inconsistantly here. The fix for this is recommended
for rel_3_2

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAdding lastmodified.virtualshelves; Bumping version to 100
Chris Nighswonger [Tue, 22 Jul 2008 18:43:31 +0000 (13:43 -0500)]
Adding lastmodified.virtualshelves; Bumping version to 100

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agokohabug 2159 Improving "Lists" button list refreshes after adding/changing lists...
Chris Nighswonger [Thu, 26 Jun 2008 04:18:07 +0000 (23:18 -0500)]
kohabug 2159 Improving "Lists" button list refreshes after adding/changing lists/shelves

Adding code to refresh "Lists" button lists display in OPAC masthead.inc

Adding javascript to force refresh of parent window to update "Lists" button menu

Adding query limits and ability to specify row count and offset in queries related to Virtualshelves.
Also added the ability to return total record counts for specified virtualshelves.

Adding C4::VirtualShelves::GetRecentShelves which returns a list of the most recently modified shelves for
a given set of parameters. This allows the user to be offered active private and open lists to add books
to in drop down menus while also allowing drop down menus to be limited to a reasonable length.
This also limits the shelves stored in the user's session to a fixed number. A further enhancement might
be to add a syspref to enable a staff member to define the limit. Currently it is hardcoded at 10 per
list type (private/public-open).

Adding pagination to list/shelf related screens

Moving refresh shelves code into C4::VirtualShelves::RefreshShelvesSummary and tidying up a bit

Correcting several inconsistancies in the shelves templates as well as handling shelf management on
the intranet side correctly.

Correcting "Add To:" drop-down list to show only lists the patron has permission to add to

Correcting a few C4::VirtualShelves::GetShelvesSummary API calls

Modifications for template consistancy

Breaking up a 1367 char line of javascript in opac-results.tmpl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAllow anyone with superlibrarian permission to edit/delete public patron lists
Chris Nighswonger [Tue, 22 Jul 2008 18:43:30 +0000 (13:43 -0500)]
Allow anyone with superlibrarian permission to edit/delete public patron lists

Edit functions here are limited to renaming and changing the type. No one can view
a list except the list owner unless it is public.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoBug 2279: TagsModeration effectiveness restored.
Joe Atzberger [Tue, 22 Jul 2008 01:28:33 +0000 (20:28 -0500)]
Bug 2279: TagsModeration effectiveness restored.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoRefine "Patrons statistics" report, fix highlight, remove CGI::scrolling_lists.
Joe Atzberger [Mon, 21 Jul 2008 23:57:45 +0000 (18:57 -0500)]
Refine "Patrons statistics" report, fix highlight, remove CGI::scrolling_lists.

At client request, I added code for a rowtitle_display and coltitle_display.  This
allows the script to substitute human-readable lables into the table instead of just
the literal hashkeys.  For this client with dozens of numerical patron categorycodes
having a row titled "29" was not very useful.

I cleaned up the templates and a lot of the report code.  It is of the same style as
the other copy/paste reports and still need further cleanup.  Added sql and debug
feedback (default styled to hidden), as I have for other reports.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoFix for Bug 2373 ("Tags missing from the left menu"). Also doing some cleanup and...
Owen Leonard [Mon, 21 Jul 2008 16:23:12 +0000 (11:23 -0500)]
Fix for Bug 2373 ("Tags missing from the left menu"). Also doing some cleanup and reformatting on review.tmpl. Fixing some capitalization--I hope that's not a translation problem.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAdding new patron form for non-javascript users (Fixes Bug 2344, No "New Member"...
Owen Leonard [Mon, 21 Jul 2008 14:28:20 +0000 (09:28 -0500)]
Adding new patron form for non-javascript users (Fixes Bug 2344, No "New Member" button w/o js).

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoRemoving duplicate column (Bug 2364, "Library column repeated")
Owen Leonard [Mon, 21 Jul 2008 13:15:28 +0000 (08:15 -0500)]
Removing duplicate column (Bug 2364, "Library column repeated")

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoCorrecting markup in intranet-main.tmpl to display "Lists" properly
Chris Nighswonger [Mon, 21 Jul 2008 02:05:40 +0000 (21:05 -0500)]
Correcting markup in intranet-main.tmpl to display "Lists" properly

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
15 years agoAdds Arabic Staff Client translation
Joshua Ferraro [Tue, 22 Jul 2008 20:38:03 +0000 (15:38 -0500)]
Adds Arabic Staff Client translation

15 years agoUpdates to the subtype limits for Audience to match the MARC21
Joshua Ferraro [Tue, 22 Jul 2008 18:44:45 +0000 (13:44 -0500)]
Updates to the subtype limits for Audience to match the MARC21
standard:

a - Preschool
b - Primary
c - Pre-adolescent
d - Adolescent
e - Adult
f - Specialized
g - General
j - Juvenile

For documentation team: please update the screenshot for
this drop-down as the options have changed.

No re-indexing should be necessary.

15 years agoFix a bug where the 008 field got replaced if you
Joshua Ferraro [Tue, 22 Jul 2008 00:38:52 +0000 (19:38 -0500)]
Fix a bug where the 008 field got replaced if you
inadvertantly clicked in the field and it wasn't empty.
This patch prevents the field from being replaced unless
it's empty.

15 years agoAdds a stylesheet for Zebra to do an on-the-fly conversion of the
Joshua Ferraro [Mon, 21 Jul 2008 23:09:27 +0000 (18:09 -0500)]
Adds a stylesheet for Zebra to do an on-the-fly conversion of the
Koha 952 item fields to the MARC21 standard 852 location fields:

Please add the following to the documentation in a section explaining
the setup of a SRU/Z39.50 server. You can also pull info from:

http://wiki.koha.org/doku.php?id=en:development:z3950sru

Modifying Output of Your Records On-the-fly

Koha supports modifying the output of your MARC records for public-facing
Z39.50 and SRU servers. You can set up custom mappings to convert fields such
as 952 (Koha items) to 852 (MARC21 location fields). For example here would
be a popular mapping:
Name: Koha Field Mapping in MARC21
---------------------------
Location:  952 $b =>  852 $b
Barcode:  952 $p =>  852 $p
Price:  952 $v =>  852 $r
Item type:  952 $y =>  852 $w
Notes: 952 $z =>  852 $z

This would make the output from the SRU server suitable for a
resource sharing network that used the above convention for location
information. To add this configuration to your Zebra, add the following
block of code in the koha-conf.xml in the <retrievalinfo> area:

<retrieval syntax="xml" name="marc21location">
         <backend syntax="usmarc" name="F">
           <marc inputformat="marc" outputformat="marcxml"
                 inputcharset="utf-8"/>
           <xslt stylesheet="/path/to/koha/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim9522852.xsl"/>
         </backend>
 </retrieval>

The retrieval set name (in this case, marc21location) is arbitrary and
should be set as required by your resource sharing network, so long as it won't
conflict with internal usage within Koha itself. It's the retrieval set name
that is used to trigger the server to respond and parse the record through the
stylesheet and apply the mappings.

In SRU, this is set as follows:

http://example.com:210/biblios?version=1.1&operation=searchRetrieve&query=it&startRecord=1&maximumRecords=20&recordSchema=marc21location

In Z39.50 it's the elementset name construct.

15 years agoFinal fix for 2335: Printers has never worked, so it's removed for 3.0
Joshua Ferraro [Sun, 20 Jul 2008 15:51:03 +0000 (10:51 -0500)]
Final fix for 2335: Printers has never worked, so it's removed for 3.0
Someone who has the time to generalize it can add it back

15 years agoRemoves the Libraries facet if singleBranchMode is ON
Joshua Ferraro [Fri, 18 Jul 2008 22:54:11 +0000 (17:54 -0500)]
Removes the Libraries facet if singleBranchMode is ON