]> git.koha-community.org Git - koha.git/log
koha.git
15 years agooffline_circ must use SQLite v2 - because Kyles PHP does
Michael Hafen [Tue, 5 May 2009 22:56:30 +0000 (16:56 -0600)]
offline_circ must use SQLite v2 - because Kyles PHP does

The Offline Circ tool by Kyle Hall uses PHP's SQLite, which is SQLite
v2.  Gnope, which Kyle links, ships with libSQLite v2.

Let's not count on libsqlite3 not being installed for perl.  If it is
installed DBD::SQLite will use it, where DBD::SQLite2 will not.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup to patch by Galen - add javascript to prevent premature submission of forms...
Michael Hafen [Thu, 13 Nov 2008 23:19:05 +0000 (16:19 -0700)]
followup to patch by Galen - add javascript to prevent premature submission of forms in Fines pages

Adds javascript to the manual invoice and manual credit pages to prevent
the premature submission of these forms when using a barcode scanner.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2897: fix English sample notices
Galen Charlton [Thu, 21 May 2009 20:03:57 +0000 (15:03 -0500)]
bug 2897: fix English sample notices

Same as Joe's prior 2897 patch, but applied to the English
sample_notices.sql - due to a Git merge and rename detection
oddity, his original submitted patch was applied to the French
sample_notices.sql.  Since the two files are
currently identical, no problem in the end.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2897 - sample notices include bogus values
Joe Atzberger [Thu, 12 Feb 2009 20:49:33 +0000 (14:49 -0600)]
Bug 2897 - sample notices include bogus values

items.content is invalid for anything but overdue notices.  It is
not processed by advance_notices.pl for DUE and PREDUE.  Also, the
odd greeting "Dear firstname title" is standardized to match the
other messages' "Dear firstname surname".

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2726: updated doc to explain auth_by_bind option
Galen Charlton [Thu, 21 May 2009 19:26:02 +0000 (14:26 -0500)]
bug 2726: updated doc to explain auth_by_bind option

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoin Auth_with_ldap.pm try binding with user password or compare
Michael Hafen [Wed, 29 Oct 2008 19:48:04 +0000 (13:48 -0600)]
in Auth_with_ldap.pm try binding with user password or compare

This changes the checkpw code in Auth_with_ldap.  Along with comparing
the user entered password against the directory attibute userPassword
you can try to bind with the users dn and password.

This is controlled by the option auth_by_bind, which, if set, causes
this code to try binding instead of comparing.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoShow "no title", when a notice doesn't have a title
Nahuel ANGELINETTI [Thu, 11 Dec 2008 12:54:05 +0000 (13:54 +0100)]
Show "no title", when a notice doesn't have a title

This patch just check if there is a title, else show "No Title".

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2287: use defaults when needed even if CSV has correct number of columns
MJ Ray [Thu, 21 May 2009 14:27:20 +0000 (15:27 +0100)]
bug 2287: use defaults when needed even if CSV has correct number of columns

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoclean up redundant test
Galen Charlton [Thu, 21 May 2009 14:00:09 +0000 (09:00 -0500)]
clean up redundant test

Credit to Joe Atzberger for cathing this.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3235) improve searchorder
Nahuel ANGELINETTI [Tue, 19 May 2009 11:54:50 +0000 (13:54 +0200)]
(bug #3235) improve searchorder

If an supplierid is provided with a ordernumber, the search should be able to return everytime a record. This patch do the job.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3149) fix searchorder
Nahuel ANGELINETTI [Tue, 19 May 2009 09:53:50 +0000 (11:53 +0200)]
(bug #3149) fix searchorder

this patch fix the searchorder function, and add to the SQL query a "AND biblionumber" statement if a biblionumber is speficied.
Sometimes C4::Acquisitions::searchorder() find more than one result, so the user is looping to the same page to choose the great record, adding a contraint to biblionumber allows to be more strict.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMore cleanup on returns.pl
Joe Atzberger [Wed, 20 May 2009 16:56:26 +0000 (11:56 -0500)]
More cleanup on returns.pl

Removed unused variable.  Corrected comments.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCleanup and enhance bookcount.pl and .tmpl
Joe Atzberger [Wed, 20 May 2009 21:55:09 +0000 (16:55 -0500)]
Cleanup and enhance bookcount.pl and .tmpl

Eliminated bug with date handling outputting an English string.
Row striping added.  Highlight for "my branch" row added.
GetBranchesLoop used instead of recreating data structure, so we get the
benefit of branches being sorted (by display name).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3120: tweak DDCN call number splitting for labels
Galen Charlton [Wed, 20 May 2009 16:20:20 +0000 (11:20 -0500)]
bug 3120: tweak DDCN call number splitting for labels

Allow a call number of the form MP3-CD F PARKER to be split as

MP3-CD
F
PARKER

instead of

MP
3
-CD
F
PARKER

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdd onlymine helper function.
Joe Atzberger [Wed, 20 May 2009 19:39:03 +0000 (14:39 -0500)]
Add onlymine helper function.

This should allow scripts to use GetBranchesLoop without having to reproduce
the check for:
(1) IndependantBranches,
(2) User logged in,
(3) User Not superlibrarian, and
(4) User branch set.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agomore permissions fixes on uk-UA installer SQL files
Galen Charlton [Thu, 21 May 2009 12:46:34 +0000 (07:46 -0500)]
more permissions fixes on uk-UA installer SQL files

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofixed permissions on ru-RU and uk-UA installer SQL files
Galen Charlton [Thu, 21 May 2009 11:52:59 +0000 (06:52 -0500)]
fixed permissions on ru-RU and uk-UA installer SQL files

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug: Importing sample data removes marcflavour variable (ru-RU)
Andrei V. Toutoukine [Thu, 21 May 2009 08:42:24 +0000 (12:42 +0400)]
Bug: Importing sample data removes marcflavour variable (ru-RU)

modified:   installer/data/mysql/ru-RU/mandatory/system_preferences_optimal.sql

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoimport of the authorized values fails on the step 3 of the web-installation (ru-RU)
Andrei V. Toutoukine [Thu, 21 May 2009 08:42:23 +0000 (12:42 +0400)]
import of the authorized values fails on the step 3 of the web-installation (ru-RU)

ERROR 1136 (21S01) at line 1: Column count doesn't match value count at row 1

modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[DAMAGED].csv.sql
modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[LOST].csv.sql
modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[NOT_LOAN].csv.sql
modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[RESTRICTED].csv.sql
modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[STACK].csv.sql
modified:   installer/data/mysql/ru-RU/authorised_values/authorised_values_[WITHDRAWN].csv.sql

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3107. Creation of `permissions` table fails during the web-installation (ru-RU)
Andrei V. Toutoukine [Thu, 21 May 2009 08:42:22 +0000 (12:42 +0400)]
Bug 3107. Creation of `permissions` table fails during the web-installation (ru-RU)

ERROR 1452 (23000) at line 3: Cannot add or update a child row: a foreign key constraint fails (`koha/permissions`,
CONSTRAINT `permissions_ibfk_1` FOREIGN KEY (`module_bit`) REFERENCES `userflags` (`bit`) ON DELETE CASCADE ON
UPDATE CASCADE)

renamed:    installer/data/mysql/ru-RU/mandatory/permissions.sql -> installer/data/mysql/ru-RU/mandatory/userpermissions.sql
renamed:    installer/data/mysql/ru-RU/mandatory/permissions.txt -> installer/data/mysql/ru-RU/mandatory/userpermissions.txt

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRussian and Ukranian SQL files
Chris Cormack [Wed, 11 Feb 2009 08:24:19 +0000 (21:24 +1300)]
Russian and Ukranian SQL files
(cherry picked from commit af505046afd9fa20d299e002645bd6517e31a971)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCleanup returns, consolidate some repeated logic.
Joe Atzberger [Tue, 19 May 2009 18:23:49 +0000 (13:23 -0500)]
Cleanup returns, consolidate some repeated logic.

Use one $userenv_branch instead of many calls to C4::Context->userenv->{'branch'}
for comparisons.  Script is not warnings safe yet, but getting closer.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBUG 2736 - added link for MARC country codes.
Nicole Engard [Mon, 18 May 2009 22:43:30 +0000 (17:43 -0500)]
BUG 2736 - added link for MARC country codes.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3236: remove disused scripts and a function
Galen Charlton [Tue, 19 May 2009 12:22:42 +0000 (07:22 -0500)]
bug 3236: remove disused scripts and a function

Removed opac/opac-dictionary.pl and catalogue/dictionary.pl,
which were not in use and not linked to from any active
template files.  According to Henri, the functionality that
these scripts implemented hasn't been working since 2.2.

Also removed C4::Search::findseealso(), which was used
only by the two scripts.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBUG #2736 Fix. Updated link to go to list of language codes.
Nicole Engard [Mon, 18 May 2009 21:42:33 +0000 (16:42 -0500)]
BUG #2736 Fix. Updated link to go to list of language codes.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1768: Calendar now retains the month on which the user was entering data.
Garry Collum [Sun, 17 May 2009 22:20:05 +0000 (18:20 -0400)]
Bug 1768: Calendar now retains the month on which the user was entering data.

If a user adds or edits a holiday, the calendar will now return to the month in which the user was working.  Also after an add or an edit, if the user selects another location, the calendar remains in the same month for which the add or edit took place.  The user sees a 'calendardate' parameter in the URL in the format that is chosen as the system preference.  If an invalid date is entered in the url, the current date is used.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2736 fix. Updated link to go to LOC instead of itsmarc.com
Nicole Engard [Mon, 18 May 2009 20:32:15 +0000 (15:32 -0500)]
Bug 2736 fix. Updated link to go to LOC instead of itsmarc.com

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3102 Fix Part 2: Fixed terminology on Transfers to Receive.
Nicole Engard [Mon, 18 May 2009 20:49:02 +0000 (15:49 -0500)]
Bug 3102 Fix Part 2: Fixed terminology on Transfers to Receive.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix permissions on new icons
Galen Charlton [Sat, 16 May 2009 01:20:28 +0000 (20:20 -0500)]
fix permissions on new icons

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding 4 additional icons to the bridge set: Software, Audio, Journal, and Realia.
Owen Leonard [Sat, 16 May 2009 00:57:16 +0000 (19:57 -0500)]
Adding 4 additional icons to the bridge set: Software, Audio, Journal, and Realia.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 1448: improve subject classification browser
Galen Charlton [Fri, 15 May 2009 14:40:00 +0000 (09:40 -0500)]
bug 1448: improve subject classification browser

* turn on warnings
* fix problem where the top level of the classification
  hiearchy wasn't displayed, instead pointing to
  the link to the Wiki page with instructions on
  how to set up the feature.
* fix XHTML validation errors

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoquell warning if format parameter isn't passed to opac-search.pl
Galen Charlton [Fri, 15 May 2009 15:53:42 +0000 (10:53 -0500)]
quell warning if format parameter isn't passed to opac-search.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix undefined variable warnings when using Syndetics
Galen Charlton [Fri, 15 May 2009 16:02:10 +0000 (11:02 -0500)]
fix undefined variable warnings when using Syndetics

Also consolidate duplicate code to a new routine,
_fetch_syndetics_content()

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRevisions to auto-generated authorities patch
Brian Harrington [Wed, 29 Apr 2009 17:37:06 +0000 (13:37 -0400)]
Revisions to auto-generated authorities patch

It turns out that the "blank" record created by MARC::Record->new()
has a leader, so now I explicitly create one in addbiblio.pl.  I
also realized I can't count. :-)

Modified per comments by Joe and Galen to keep SetMarcUnicodeFlag
and move it and the leader set to earlier in the process.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix XHTML errors in z3950_search.tmpl
Galen Charlton [Fri, 15 May 2009 14:59:49 +0000 (09:59 -0500)]
fix XHTML errors in z3950_search.tmpl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoImprove XSLT.pm performances
Frédéric Demians [Fri, 15 May 2009 08:26:33 +0000 (10:26 +0200)]
Improve XSLT.pm performances

This patch is done in the process of understanding what's
going wrong with bug #3226. It improves some processing.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1440: Show a list of holidays. Fix for the date formats.
Garry Collum [Fri, 15 May 2009 11:37:50 +0000 (07:37 -0400)]
Bug 1440: Show a list of holidays. Fix for the date formats.

This patch fixes the date formats in the holiday display tables.  They are now based on the syspref for date formats.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBUG #1277 Enhance: Z39.50 More search criteria
Darrell Ulm [Fri, 15 May 2009 14:27:17 +0000 (10:27 -0400)]
BUG #1277 Enhance: Z39.50 More search criteria

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBUG 1277 Enhance: Z39.50 More search criteria
Darrell Ulm [Fri, 15 May 2009 14:27:16 +0000 (10:27 -0400)]
BUG 1277 Enhance: Z39.50 More search criteria

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdded barcode query back to circ/ysearch.pl for patron proxies.
Daniel Sweeney [Wed, 25 Mar 2009 21:37:17 +0000 (16:37 -0500)]
Added barcode query back to circ/ysearch.pl for patron proxies.

This is to allow patron proxy relationships to be built using the patron barcode.

Signed-off-by: Daniel Sweeney <daniel.sweeney@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago@renew_failed can cause enormous performance-killing array.
Joe Atzberger [Thu, 14 May 2009 20:28:40 +0000 (15:28 -0500)]
@renew_failed can cause enormous performance-killing array.

The array was populated and values flagged with an accessor, like:
    for (@failedrenews) { $renew_failed[$_] = 1; }
But this means that an array of possibly hundreds of thousands of elements
would have to be auto-populated for high itemnumbers.  A hash is the correct
structure.  We also haven't checked the user input for validity, so we do not know
for sure that @failedrenews really does contain just itemnumbers.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoupdated JSON and JSON::XS package reqs for Debian lenny
Galen Charlton [Thu, 14 May 2009 12:53:33 +0000 (07:53 -0500)]
updated JSON and JSON::XS package reqs for Debian lenny

By default, the JSON module will automatically use
JSON::XS if available.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix small POD error
Galen Charlton [Thu, 14 May 2009 12:33:36 +0000 (07:33 -0500)]
fix small POD error

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoNew framework for AJAX services
Pianohacker [Sun, 5 Apr 2009 22:40:13 +0000 (17:40 -0500)]
New framework for AJAX services

This adds two new C4 modules, C4::Service and ::Output::JSONStream, and
makes important modifications to C4::Output. The first two are a basic
framework for JSON-based AJAX services and a simple JSON output wrapper,
respectively. C4::Output has been slightly refactored, with a new
function, output_with_http_headers, that supports different
content-types. output_html_with_http_headers still exists, and the three
pages affected by this change have been refactored to support it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3223: add 500 and 590 to MARC21 bib Note index (Zebra)
Galen Charlton [Thu, 14 May 2009 00:37:17 +0000 (19:37 -0500)]
bug 3223: add 500 and 590 to MARC21 bib Note index (Zebra)

Note: to completely apply this change, ensure that the working
copy of record.abs is updated and rebuild the bib indexes
using rebuild_zebra.pl -b -x -r

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1542, fixing filter by area in reports dictionary
Chris Cormack [Thu, 14 May 2009 06:46:56 +0000 (18:46 +1200)]
Bug 1542, fixing filter by area in reports dictionary

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3224 : inventory tool fails on DOS newlines
Ryan Higgins [Thu, 14 May 2009 07:46:45 +0000 (00:46 -0700)]
Bug 3224 : inventory tool fails on DOS newlines

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1440: Show a list of holidays. Second patch.
Garry Collum [Tue, 12 May 2009 20:53:15 +0000 (16:53 -0400)]
Bug 1440: Show a list of holidays. Second patch.

Fixed XHTML error that was introduced in the first patch. Added zebra striping to the tables. Removed an erroneous line with a date experiment that had no affect on the dates.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3169 - Comply with new Google Book branding guidelines
Frédéric Demians [Tue, 12 May 2009 12:28:27 +0000 (14:28 +0200)]
Bug 3169 - Comply with new Google Book branding guidelines

It adds pretty huge image 'Google Preview' on google
book jacket.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2505 adding use warnings and fixing warnings in opac-basket.pl
Chris Cormack [Tue, 12 May 2009 10:28:19 +0000 (22:28 +1200)]
Bug 2505 adding use warnings and fixing warnings in opac-basket.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoWhoops extra } crept in when fixing a merge conflict, removing it
Chris Cormack [Tue, 12 May 2009 10:13:10 +0000 (22:13 +1200)]
Whoops extra } crept in when fixing a merge conflict, removing it

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3079) show summary in opac results
Nahuel ANGELINETTI [Tue, 12 May 2009 09:09:49 +0000 (11:09 +0200)]
(bug #3079) show summary in opac results

This add the summary in opac search results

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFixing more warnings
Chris Cormack [Tue, 12 May 2009 07:30:56 +0000 (19:30 +1200)]
Fixing more warnings

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1440: Show a list of holidays
Garry Collum [Tue, 12 May 2009 02:51:07 +0000 (22:51 -0400)]
Bug 1440: Show a list of holidays

This patch adds displays for holidays. A table for each type of holiday.  It could be improved by adding a link for a holiday listed in one of the tables to the edit panel.  The date formats could also be changed.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1907 partial fix: prevent warnings on undefined params.
Chris Catalfo [Tue, 12 May 2009 00:35:13 +0000 (00:35 +0000)]
Bug 1907 partial fix: prevent warnings on undefined params.

This patch adds default blank values for a couple of params the script might receive which were producing warns in the error log. Also wraps GetMarcBiblio in an eval.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoScript to check for corruption of marcxml, and optionally to attempt fix.
Joe Atzberger [Mon, 11 May 2009 21:44:29 +0000 (16:44 -0500)]
Script to check for corruption of marcxml, and optionally to attempt fix.

When marcxml cannot be parsed into a MARC::Record object, the biblio is
undisplayable and it obviously breaks many features in Koha.  This script
can test to parse every marcxml, and alert on failures.  Optionally, the
marcxml can be replaced from the marc field.

See extensive perldoc for details.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #1578) change the shelf COinS support
Nahuel ANGELINETTI [Fri, 9 Jan 2009 10:36:47 +0000 (11:36 +0100)]
(bug #1578) change the shelf COinS support

this change the old COinS support, and now use C4::Biblio::GetCOinSBiblio().

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoadd subtitle to COinS for MARC21
Galen Charlton [Tue, 12 May 2009 00:03:50 +0000 (19:03 -0500)]
add subtitle to COinS for MARC21

Preserves change added in the LTFL integration
while also switching to Nahuel's COinS implementation.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #1578) fix a little error in code
Nahuel ANGELINETTI [Mon, 9 Mar 2009 13:34:27 +0000 (14:34 +0100)]
(bug #1578) fix a little error in code

This one, fix the use of "or" instead of "||", else a warning is raised.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #1578) Add the (quasi)full COinS(Zotero) support
Nahuel ANGELINETTI [Thu, 8 Jan 2009 10:41:57 +0000 (11:41 +0100)]
(bug #1578) Add the (quasi)full COinS(Zotero) support

this patch do a lot of things :
 * Add the function GetCOinSBiblio in C4::Biblio, return the COinS value that must be put in a span title
   This will be usefull for OpenURL support.

 * Replace the span in opac-detail, and put the var get from GetCOinSBiblio
 * Fix an <img> tag that wasn't autoclosed
 * Add the COinS in search result
 * Full COinS support of UNIMARC, the MARC21 support should be improved to provide all kind of document types

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoPrevent fines failure on NULL borrowernumber.
Joe Atzberger [Fri, 23 Jan 2009 22:22:40 +0000 (16:22 -0600)]
Prevent fines failure on NULL borrowernumber.

The problem is that we do not ensure that the issues table has valid
borrowernumber in each line.  This is exacerbated by Getoverdues()
returning data sorted BY borrowernumber.  So one NULL borrowernumber
in issues prevented ALL fines from being assessed.  The actual error
from fines.pl cron log is:
  No branchcode argument to new.  Should be C4::Calendar->new(branchcode => $branchcode)
  at /home/user/kohaclone/misc/cronjobs/fines.pl line 98

This patch deals only with getting fines to avoid crashing.  It does
not fix the underlying data integrity problem.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoexcised hard-coded reference to HLT
Galen Charlton [Mon, 11 May 2009 21:22:04 +0000 (16:22 -0500)]
excised hard-coded reference to HLT

Note to Chris Cormack: I suspect the current ability
to override most circulation conditions is sufficient,
but if HLT still needs to establish a patron category
that can ignore issuingimpossible limitations, needs
to be done in a more flexible way.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix references to $session in circ/circulation.pl
Galen Charlton [Mon, 11 May 2009 21:19:20 +0000 (16:19 -0500)]
fix references to $session in circ/circulation.pl

Now that sticky due date processing uses the session,
script will always need it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMake stickyduedate remember for session a little more permanent
Michael Hafen [Mon, 11 May 2009 20:52:35 +0000 (14:52 -0600)]
Make stickyduedate remember for session a little more permanent

Store the stickyduedate in the session.  So "Remember for Session" means
for session rather than "while on this page".

Includes some updates since initial submission of patch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix to CanBookBeIssued()
Galen Charlton [Mon, 11 May 2009 21:10:37 +0000 (16:10 -0500)]
fix to CanBookBeIssued()

Bail out if item does not exist - otherwise
subsequent checks can fail for lack of an
item branchcode.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2770: add RenewalPeriodBase syspref for 3.2 (DB rev 032)
Galen Charlton [Mon, 11 May 2009 20:25:14 +0000 (15:25 -0500)]
bug 2770: add RenewalPeriodBase syspref for 3.2 (DB rev 032)

Reconciliation patch to add the RenewalPeriodBase system
preference, which per code by Nahuel Angelinetti changes
the process of calculating the due date after a renewal:

If RenewalPeriodBase is set to 'now', the due date is
set to the current date + the loan period.  This is the
current behavior.

If RenewalPeriodBase is set to 'date_due', the due date
is set to the due date before the renewal + the loan period.

Unless like the corresonding database update in the 3.0.x branch
(3.00.01.007), the default value of RenewalPeriodBase is set to
'now' so as not to unexpectedly change the calculation.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix the previous patch to add a syspref that permit to choose the renewal period...
Nahuel Angelinetti [Fri, 6 Mar 2009 10:38:04 +0000 (11:38 +0100)]
fix the previous patch to add a syspref that permit to choose the renewal period(day or date_due).

This patch just check the syspref and use it as base date of renew.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agochange the date calc, to set the date_due to date_due+loanlength.
Nahuel Angelinetti [Fri, 6 Mar 2009 10:38:03 +0000 (11:38 +0100)]
change the date calc, to set the date_due to date_due+loanlength.

At the moment when you renew a document the date_due calculated is now+loanlength, but it's a bad thing, It should be
date_due+loanlength.
This patch do change Circulation.pm to use the date_due as renew base and add the loanlength to get the new date_due after
renewal.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug # 3048 changed Descriptions label to Title Notes
brendan [Tue, 31 Mar 2009 23:20:52 +0000 (23:20 +0000)]
Bug # 3048 changed Descriptions label to Title Notes

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 988 followup - tweak wording
Galen Charlton [Mon, 11 May 2009 19:26:23 +0000 (14:26 -0500)]
bug 988 followup - tweak wording

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix for #988
Paul Poulain [Sat, 18 Apr 2009 20:51:41 +0000 (22:51 +0200)]
fix for #988

When a borrower can't issue any book (issuingrules=0), the message should be
"this user can't issue book" and not "too many books already issued"wrong message

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdd a date field in news edition, and permit to set the "timestamp" field, that is...
Nahuel Angelinetti [Fri, 6 Mar 2009 10:38:00 +0000 (11:38 +0100)]
Add a date field in news edition, and permit to set the "timestamp" field, that is the publication date and modify the GetNewsToDisplay to return only news that timestamp is < to current date that permit to publish news later without any action.

Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2872: fix width of columns in branch_transfer_limits (DB rev 031)
Galen Charlton [Mon, 11 May 2009 17:22:46 +0000 (12:22 -0500)]
bug 2872: fix width of columns in branch_transfer_limits (DB rev 031)

change itemtype, tobranch, and frombranch from varchar(4)
to varchar(10)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agotweak removal of whitespace from barcodes
Galen Charlton [Mon, 11 May 2009 16:39:19 +0000 (11:39 -0500)]
tweak removal of whitespace from barcodes

Per suggestion from Joe Atzberger, match on \s
instead of [ \t] for stripping leading and
trailing whitespace from barcode lookups.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix to remove leading/trailing whitespace from entered barcode.
Mason James [Thu, 19 Mar 2009 22:52:26 +0000 (11:52 +1300)]
fix to remove leading/trailing whitespace from entered barcode.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobugfix stickyduedate more sticky - couple more forms in circulation.tmpl
Michael Hafen [Wed, 6 May 2009 22:15:08 +0000 (16:15 -0600)]
bugfix stickyduedate more sticky - couple more forms in circulation.tmpl

There are a couple more forms in circulation.tmpl that need the
stickyduedate and duedatespec hidden inputs.  The 'Don't Check Out' form
of the needsconfirmation block, and the findborrower form.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobugfix stickyduedate more sticky - also patron search in quick search block
Michael Hafen [Wed, 6 May 2009 22:34:47 +0000 (16:34 -0600)]
bugfix stickyduedate more sticky - also patron search in quick search block

Forgot about the patron search for check out form in the quick search
block

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoadd windows quote to ignored chars
Nahuel ANGELINETTI [Mon, 11 May 2009 14:51:46 +0000 (16:51 +0200)]
add windows quote to ignored chars

This patch add the CP1252 "quote" (’) in ignored chars in zebra configuration.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agochange DOS newlines to Unix newlines in word-phrase-utf.chr
Galen Charlton [Mon, 11 May 2009 15:48:14 +0000 (10:48 -0500)]
change DOS newlines to Unix newlines in word-phrase-utf.chr

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding some null words and mappings for Œ œ
Henri-Damien LAURENT [Wed, 28 Jan 2009 16:39:10 +0000 (17:39 +0100)]
Adding some null words and mappings for Œ œ
(cherry picked from commit e6e27bc80a7be792c228d2fef2e981539191ec36)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBugfix 3176 - Browser selected languages are not use by OPAC
Frédéric Demians [Thu, 30 Apr 2009 15:23:17 +0000 (17:23 +0200)]
Bugfix 3176 - Browser selected languages are not use by OPAC

This patch uses HTTP_ACCEPT_LANGUAGE web browser variable
to select OPAC available language.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTweak ceilingDueDate and CalcDateDue
Michael Hafen [Wed, 29 Apr 2009 22:02:11 +0000 (16:02 -0600)]
Tweak ceilingDueDate and CalcDateDue

This patch moves ceilingDueDate and ReturnBeforeExpiry checking into
CalcDateDue(), so the renewal due date is also calculated with those.

Also restores a check in CanBookBeIssued.  If the due date isn't given
call CalcDateDue and make sure it's not in the past.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3196) fix itemlost.pl
Nahuel ANGELINETTI [Thu, 7 May 2009 09:48:44 +0000 (11:48 +0200)]
(bug #3196) fix itemlost.pl

This patch fix 3 things:
* fix order by
* take care of item_level-itype
* use join in sql query

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 1984 - added Javascript in the header to open, print, and close the slip window
Liz Rea [Fri, 1 May 2009 20:26:41 +0000 (15:26 -0500)]
Bug 1984 - added Javascript in the header to open, print, and close the slip window

This is for printing circulation slips.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 1907 Partial fix: prevent bad records from truncating output.
Chris Catalfo [Fri, 1 May 2009 23:34:43 +0000 (23:34 +0000)]
bug 1907 Partial fix: prevent bad records from truncating output.

This patch starts improving export.pl by adding warnings, skipping invalid records, and not trying to access fields which don't exist.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3212: MARC21 bib leader plugin sets Leader/09 to 'a'
Galen Charlton [Fri, 8 May 2009 20:10:15 +0000 (15:10 -0500)]
bug 3212: MARC21 bib leader plugin sets Leader/09 to 'a'

Changed the plugin so that the form for editing the MARC21
leader now sets postion 9 (character coding) to 'a' (UCS/Unicode)
and no longer gives the cataloger the option to change it.  Koha
uses UTF-8 exclusively for MARC records, and a number of bugs
can be triggered by storing MARC21 bib and authority record
whose Leader/09 is anything other than 'a'.  Since internal
routines are increasingly ensuring that the MARC21 Leader/09 is
set to 'a', it doesn't make sense to give catalogers the idea
that it's possible (or a good idea) to set the Leader/09 to any
other value.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3211: fix XHTML validation errors in MAR21 bib leader plugin
Galen Charlton [Fri, 8 May 2009 20:10:14 +0000 (15:10 -0500)]
bug 3211: fix XHTML validation errors in MAR21 bib leader plugin

Note that as a result of this patch, not only will the
page use valid XHTML, but the intranet stylesheets are
now used, IMO improving the display of the form.

This patch also fixes the descriptions of the Leader/09
and Leader/17.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3209: add 'p' (mixed materials) to MARC21 Leader/06 dropdown
Galen Charlton [Fri, 8 May 2009 20:10:13 +0000 (15:10 -0500)]
bug 3209: add 'p' (mixed materials) to MARC21 Leader/06 dropdown

Also made the following updates to the MARC21 leader plugin

[1] Changed description of Leader/08 to 'Type of control'
[2] Fixed typo in description value for 'z' in Leader/17
[3] Updated Leader/19, which is now 'Multipart resource record level'

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3120: tweak splitting of Dewey call number
Galen Charlton [Fri, 8 May 2009 22:31:14 +0000 (17:31 -0500)]
bug 3120: tweak splitting of Dewey call number

If a call number prefix contains a hyphen, try not to
split on the hyphen.  For example, rather than
splitting the call number 'CD-ROM 787.87 EAS'
like this:

CD
-ROM
787.87
EAS

it should be

CD-ROM
787.87
EAS

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3198) fix serial receive with autobarcode
Nahuel ANGELINETTI [Mon, 11 May 2009 08:46:58 +0000 (10:46 +0200)]
(bug #3198) fix serial receive with autobarcode

this add an arg to GetMarcFromKohaField, else it will always return (0,0).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding the irc meeting and maintenance tree to the history
Chris Cormack [Sun, 10 May 2009 09:49:23 +0000 (21:49 +1200)]
Adding the irc meeting and maintenance tree to the history

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3213 - UNIMARC/MARC21 selection
Frédéric Demians [Mon, 11 May 2009 11:41:49 +0000 (13:41 +0200)]
Bug 3213 - UNIMARC/MARC21 selection

Modify how XSLT stylesheet are choosen
depending on 'marcflavour' syspref.

DOCUMENTATION: XSLTResultsDisplay and XSLTDetailsDisplay
sysprefs aren't MARC21-only anymore.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3213 - Add UNIMARC XSLT stylesheet
Frédéric Demians [Mon, 11 May 2009 11:41:50 +0000 (13:41 +0200)]
Bug 3213 - Add UNIMARC XSLT stylesheet

With those stylesheets result and detail pages
render better UNIMARC biblio record than normal
views. But works still have to be done in order
to reach UNIMARC-ISBD compliant display.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMore fixes for Bug 2704, 440 Display Issues
Owen Leonard [Thu, 30 Apr 2009 16:36:19 +0000 (11:36 -0500)]
More fixes for Bug 2704, 440 Display Issues

Multiple series now display along with other detailed bibliographic information. Series title is displayed along with volume information. The template no longer displays information from biblioitems like volume and volumedesc in favor of information pulled directly from the MARC record.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3205: fix another error in the OAI-PMH Identify response
Galen Charlton [Fri, 8 May 2009 16:54:29 +0000 (11:54 -0500)]
bug 3205: fix another error in the OAI-PMH Identify response

The optional description element of an Identify response
can't just be a string.  Identify.description is a container
for one or more elements; see http://www.openarchives.org/OAI/2.0/guidelines.htm

For now, simply commenting it out.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBugfix 3205 - Fix OAI server resonse to Identify request
Frédéric Demians [Fri, 8 May 2009 14:45:45 +0000 (16:45 +0200)]
Bugfix 3205 - Fix OAI server resonse to Identify request

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3195 : Item Transfers and Holds disrupted.
Ryan Higgins [Thu, 7 May 2009 18:19:05 +0000 (11:19 -0700)]
bug 3195 : Item Transfers and Holds disrupted.

This follows revert of commit 257a439d (bug 3025).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRevert "(bug #3025) multiple check-in of a multiple times reserved item go wrong"
Ryan Higgins [Thu, 7 May 2009 18:19:04 +0000 (11:19 -0700)]
Revert "(bug #3025) multiple check-in of a multiple times reserved item go wrong"

This reverts commit 257a439dafa33d0c642f1ec7badedb5414027e64.
This patch causes Bug 3195 .

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3190) allow right truncation in lot of records
Nahuel ANGELINETTI [Wed, 6 May 2009 08:50:51 +0000 (10:50 +0200)]
(bug #3190) allow right truncation in lot of records

This patch just add an option to zebra-biblios.cfg that allow to make right truncate requests on a huge request.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3173 Place Hold link visible in OPAC even if syspref turned off
Jane Wagner [Thu, 30 Apr 2009 14:58:39 +0000 (10:58 -0400)]
Bug 3173 Place Hold link visible in OPAC even if syspref turned off

If the syspref RequestOnOpac is turned off, the link to place a hold does not
show in the detail page.  However, the link is still present in the results
list as well as in the basket and in lists.  Modified templates to check
for the syspref and to obey it.

NOTE:  This does not turn off the display of the Holds tab in the OPAC
user account.  I think there are situations where library staff may be
allowed to place holds but users are not allowed to do so through the
OPAC.  Leaving the Holds tab visible shows users what they have on hold,
but does not allow them to place holds.  It does allow users to cancel
holds.  If there is a need to hide this tab, that would probably be
best accomplished by a separate syspref.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>