]> git.koha-community.org Git - koha.git/log
koha.git
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>
15 years ago(bug #3187) authority search pagination doesn't work
Nahuel ANGELINETTI [Tue, 5 May 2009 09:53:30 +0000 (11:53 +0200)]
(bug #3187) authority search pagination doesn't work

this patch just set a marclist argument value on search authority, else the pagination won't work.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3174) show the member name in reports
Nahuel ANGELINETTI [Mon, 4 May 2009 15:16:45 +0000 (17:16 +0200)]
(bug #3174) show the member name in reports

this patch show the member firstname and surname instead of only the borrowernumber in the guided reports page

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTweak offline_circ - honor itemBarcodeInputFilter and add to Makefile.PL
Michael Hafen [Tue, 5 May 2009 21:01:09 +0000 (15:01 -0600)]
Tweak offline_circ - honor itemBarcodeInputFilter and add to Makefile.PL

Add the ofline_circ directory to Makefile.PL as needing to be installed.
Add calls to barcodedecode() on issueing and returning.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2287: handle case where autoMemberNum is off
Galen Charlton [Fri, 8 May 2009 13:44:03 +0000 (08:44 -0500)]
bug 2287: handle case where autoMemberNum is off

When importing a patron file where new records
do not have a card number *and* autoMemberNum is
off, make sure that bororwers.cardnumber is set to NULL
for those new patrons rather than '' - otherwise, only
one patron with an empty barcode can be created, as there is
a uniqueness constraint on borrowers.cardnumber

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAllow autogenerated cardnumbers again to solve http://bugs.koha.org/cgi-bin/bugzilla3...
MJ Ray [Tue, 5 May 2009 15:52:47 +0000 (16:52 +0100)]
Allow autogenerated cardnumbers again to solve http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2287

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3183) fix the SetMarcUnicodeFlag function
Nahuel ANGELINETTI [Mon, 4 May 2009 09:22:15 +0000 (11:22 +0200)]
(bug #3183) fix the SetMarcUnicodeFlag function

This patch fix the funciton SetMarcUnicodeFlag for UNIMARC support, now the function will fix the length of the field, and set encoding as "50  " instead of "5050".

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoOptimize XSLT.pm
Frédéric Demians [Thu, 7 May 2009 17:28:33 +0000 (19:28 +0200)]
Optimize XSLT.pm

Avoid reading marc_tag_structure for each biblo record to
transform by XSLT. Not totaly useless on OPAC result page.
Should be done via application variables rather than
request variable, but this is another story.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdd system preference search to administration home page (Bug 3181)
Owen Leonard [Sat, 2 May 2009 18:35:14 +0000 (13:35 -0500)]
Add system preference search to administration home page (Bug 3181)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBugfix 3199 - batchRebuildBiblioTables.pl failed due to missing argument [3.0.x]
Frédéric Demians [Thu, 7 May 2009 16:57:26 +0000 (18:57 +0200)]
Bugfix 3199 - batchRebuildBiblioTables.pl failed due to missing argument [3.0.x]

Thanks G. Henry

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix all templates errors reported by xt/author/valid-templates.t
Galen Charlton [Fri, 1 May 2009 22:59:04 +0000 (17:59 -0500)]
fix all templates errors reported by xt/author/valid-templates.t

In process, removed four disused HTML templates and one
disused script, serials/receipt.pl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2944 fix courtesy of Amr Denni
Chris Cormack [Sun, 3 May 2009 05:02:22 +0000 (17:02 +1200)]
Bug 2944 fix courtesy of Amr Denni

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3197) Subfield code box was not present on new subfield tab in Admin/Authority
Colin Campbell [Thu, 7 May 2009 13:59:25 +0000 (14:59 +0100)]
(bug #3197) Subfield code box was not present on new subfield tab in Admin/Authority

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoSpelling correction
Owen Leonard [Sat, 2 May 2009 16:40:50 +0000 (11:40 -0500)]
Spelling correction

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding David Birmingham as a developer
Chris Cormack [Fri, 1 May 2009 23:50:16 +0000 (11:50 +1200)]
Adding David Birmingham as a developer

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMoved some variable definitions out of conditionals
Colin Campbell [Tue, 5 May 2009 09:36:41 +0000 (10:36 +0100)]
Moved some variable definitions out of conditionals

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3081 - The category code in the patron personal details page in opac showed the...
Garry Collum [Sun, 3 May 2009 04:23:14 +0000 (00:23 -0400)]
Bug 3081 - The category code in the patron personal details page in opac showed the category code and not the description. This patch makes the description appear instead. Looking at both opac-userupdate.pl and opac-userupdate.tmpl, the actual value of this text box is not retrieved for any purpose.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3193) fix the comtd value in acqui-home.pl
Nahuel ANGELINETTI [Wed, 6 May 2009 10:26:55 +0000 (12:26 +0200)]
(bug #3193) fix the comtd value in acqui-home.pl

this patch fix the value of the comtd value which is not the same as in bookfund.pl, clicking on the value from acqui-home.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRemove duplicate assignments for $template->param(script_name)
Joe Atzberger [Mon, 4 May 2009 20:16:01 +0000 (15:16 -0500)]
Remove duplicate assignments for $template->param(script_name)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoConvert shelves from using "toggle" code
Joe Atzberger [Mon, 4 May 2009 20:16:03 +0000 (15:16 -0500)]
Convert shelves from using "toggle" code

And one ELSIF.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3157: IE compatibility for label display
Joe Atzberger [Wed, 6 May 2009 19:41:51 +0000 (14:41 -0500)]
Bug 3157: IE compatibility for label display

Also fixed validation errors where label elements were "for" non-existant ids.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding Colin Campbell to the history doc
Chris Cormack [Fri, 1 May 2009 22:37:45 +0000 (10:37 +1200)]
Adding Colin Campbell to the history doc

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoremoved extra /TMPL_IF introduce in previous Syndetics patch
Galen Charlton [Fri, 1 May 2009 22:27:04 +0000 (17:27 -0500)]
removed extra /TMPL_IF introduce in previous Syndetics patch

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug fix to OPAC shelf browsing query statement
David Birmingham [Thu, 30 Apr 2009 16:54:48 +0000 (12:54 -0400)]
Bug fix to OPAC shelf browsing query statement

If the items.location field was NULL, then the current
SQL query would produce no results.  I have turned this
into a conditional block that removes the location condition
in the query if the location is not specified.

In addition, there was a small change to opac-detail.tmpl
that changed Library to Shelves when the shelf browser was
open.  This removes a potential redundant Library Library
display if Library is contained in the starting_homebranch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding warnings, fixing a bug that meant the title of an item that had a fine was...
Chris Cormack [Fri, 1 May 2009 21:39:25 +0000 (09:39 +1200)]
Adding warnings, fixing a bug that meant the title of an item that had a fine was never printed

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug #2974 Adding category code to BorrowerMandatoryField produced a "The following...
Garry Collum [Fri, 1 May 2009 01:54:31 +0000 (21:54 -0400)]
Bug #2974 Adding category code to BorrowerMandatoryField produced a "The following fields are mandatory: categorycode"

If you made a column that is implemented by a select box mandatory in
BorrowerMandatoryField, Members.js would throw an error on the first
element.  The original intent of the function was probably to catch
the first option for a drop-down list such as "Salutation" where the
text is blank.  Added code to check for an empty string in the first
element.

You can test this by making salutation (title) mandatory.

Also fixed some typos in some of the comments.  Why?  Because I'm a Librarian.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #3174) fill borrower number, and show author and creation date in saved reports
Nahuel ANGELINETTI [Thu, 30 Apr 2009 09:36:52 +0000 (11:36 +0200)]
(bug #3174) fill borrower number, and show author and creation date in saved reports

This patch add 2 columns in "Use Saved" page whith creation date, and the creator id.
It fix the creation of a report, to save the borrowernumber in reports table(to show it after).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agonew author test for syntax errors in HTML templates
Galen Charlton [Wed, 29 Apr 2009 21:51:11 +0000 (16:51 -0500)]
new author test for syntax errors in HTML templates

Added a new author test to look for syntax errors
in the HTML templates.  Run by doing

prove xt/author/valid-templates.t

from the root of a Koha source tree.

This test will help catch the following errors:

* unbalanced TMPL_IF, TMPL_UNLESS, and TMPL_LOOP contructs, e.g.,
  cases where a TMPL_IF is not closed by a /TMPL_IF.
* references to nonexistant include files
* syntax errors within a HTML::Template::Pro tag

This test currently ignores errors related to TMPL_IF EXPR, which
is currently deprecated for use in Koha -- this may be made
stricter in the future.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix XHTML validation error introduced in previous patch
Galen Charlton [Fri, 1 May 2009 21:17:26 +0000 (16:17 -0500)]
fix XHTML validation error introduced in previous patch

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoEnhancements/fixes for OPAC advanced search screen
Jane Wagner [Wed, 29 Apr 2009 18:14:56 +0000 (14:14 -0400)]
Enhancements/fixes for OPAC advanced search screen

Fixed opac-search.pl for correct test of expanded options and
to pass variable for MARC21 to advanced search screen.

Added Publication Date Range search to advanced search screen

Removed obsolete item type code section from advanced search screen
(wasn't displaying anyway because expanded options wasn't working,
superceded by site-specific item type searches).

Changed wording on Additional Content Types pulldown to make clear
it's limited to Books/Printed Materials -- this is basing off the
008 positions 24-27, but the options (and those positions) are only
valid for Book material types. Other material types have different
008 settings in those positions.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRemove unused $toggle code
Joe Atzberger [Thu, 30 Apr 2009 17:15:12 +0000 (12:15 -0500)]
Remove unused $toggle code

Some trivial other cleanups also.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding the creation of sql library on the wiki
Chris Cormack [Thu, 30 Apr 2009 21:41:08 +0000 (09:41 +1200)]
Adding the creation of sql library on the wiki

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoSyndetics: add support for AVSUMMARY and AVPROFILE
Galen Charlton [Thu, 30 Apr 2009 20:09:42 +0000 (15:09 -0500)]
Syndetics: add support for AVSUMMARY and AVPROFILE

Added support for the Syndetics AV summary and AV profile
data elements.  The AV summary is displayed in the description
tab if present, while the AV profile is displayed as
a link off of the OPAC bib details page.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoSyndetics: start switch to XML::LibXML to parse results
Galen Charlton [Thu, 30 Apr 2009 19:10:34 +0000 (14:10 -0500)]
Syndetics: start switch to XML::LibXML to parse results

XML::Simple doesn't handle mixed content, which is a
problem because reviews provided by Syndetics sometimes
contain HTML tags.  Furthermore, it's often easier to write
XPath to extract bits of an XML document than trying
to play with XML::Simple's options.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoSyndetics: add fiction notes/title profile link
Galen Charlton [Thu, 30 Apr 2009 19:29:54 +0000 (14:29 -0500)]
Syndetics: add fiction notes/title profile link

If a Syndetics title profile (AKA fiction notes page) exists
and is avaailable to the user, add a link to to in the OPAC
detail page.

TODO: add a system preference

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCorrect comparison which was using an octal number
Colin Campbell [Fri, 1 May 2009 08:32:27 +0000 (09:32 +0100)]
Correct comparison which was using an octal number

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding Liz Rea to the history.txt
Chris Cormack [Thu, 30 Apr 2009 03:33:13 +0000 (15:33 +1200)]
Adding Liz Rea to the history.txt

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoSyndetics: handle variations in reviews XML output
Galen Charlton [Thu, 30 Apr 2009 17:25:27 +0000 (12:25 -0500)]
Syndetics: handle variations in reviews XML output

For some reason different Syndetics users get
slightly different results when requesting
an XML-formatted review, so it's necessary to
handle both observed variations:

Here's one variation:

...
<Notes>
    <Fld520 I1="BLANK" I2="BLANK">
    <a>This is a review

And another:
...
<Notes>
    <Fld520 I1="BLANK" I2="BLANK">
        This is a review

This patch fixes a bug where some users the
source of a review but not the review itself.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix to C4::Koha::_isbn_cleanup
Galen Charlton [Thu, 30 Apr 2009 18:06:24 +0000 (13:06 -0500)]
fix to C4::Koha::_isbn_cleanup

Both valid ISBN-10s and -13s can use 'X' as a checkdigit
value.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoVerbiage clarification in intranet advsearch.tmpl
root [Thu, 30 Apr 2009 16:42:23 +0000 (11:42 -0500)]
Verbiage clarification in intranet advsearch.tmpl

Changed verbiage from Limit type to: match any of the following to Limit to any of the following:

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFix for Bug 2767, cart and lists buttons missing on opac-search.pl
Owen Leonard [Thu, 30 Apr 2009 16:40:14 +0000 (11:40 -0500)]
Fix for Bug 2767, cart and lists buttons missing on opac-search.pl

This fix adds alternate markup to be included on the advanced search page in order to mimic the placement of the cart and lists buttons on other opac pages.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoLibraryspeak extinction in OPAC: Verbiage Change opac-advsearch.tmpl
Liz Rea [Thu, 30 Apr 2009 16:30:26 +0000 (11:30 -0500)]
Libraryspeak extinction in OPAC: Verbiage Change opac-advsearch.tmpl

Changed verbiage from Limit type to: match any of the following to Limit to any of the following:

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3177 - haspermission offers bogus option
Joe Atzberger [Thu, 30 Apr 2009 15:04:38 +0000 (10:04 -0500)]
Bug 3177 - haspermission offers bogus option

$intflags was never used or returned if hashref instead of userid was passed.
Also cleaned up needless passing of $dbh.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3179: Typo in C4::Circulation function call causes fatal error
Ryan Higgins [Thu, 30 Apr 2009 21:35:10 +0000 (14:35 -0700)]
Bug 3179: Typo in C4::Circulation function call causes fatal error

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoWhoops I typo'd Garry's name - fixed
Chris Cormack [Wed, 29 Apr 2009 21:52:13 +0000 (09:52 +1200)]
Whoops I typo'd Garry's name - fixed

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoUsability change: widened the search type dropdown from 9em to 10em so that it doesn...
root [Wed, 29 Apr 2009 18:14:45 +0000 (13:14 -0500)]
Usability change: widened the search type dropdown from 9em to 10em so that it doesn't look cramped.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoUpdating history docs for new developer
Chris Cormack [Wed, 29 Apr 2009 20:24:19 +0000 (08:24 +1200)]
Updating history docs for new developer

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoUsability change: Added example of 5.00 to fines/manual credit and fines/manual invoi...
Liz Rea [Wed, 29 Apr 2009 16:22:53 +0000 (11:22 -0500)]
Usability change: Added example of 5.00 to fines/manual credit and fines/manual invoice to remind folks that they do not have to add the dollar sign (or other international money symbol) before the amount.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoOPAC Advanced Search - have user's branch selected by default
Michael Hafen [Tue, 28 Apr 2009 20:16:31 +0000 (14:16 -0600)]
OPAC Advanced Search - have user's branch selected by default

If SearchMyLibraryFirst is on, and the user is logged in, then have
their branch selected in the Location And Availability drop-down.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCheckout page - Check Out's table sortable on barcode
Michael Hafen [Tue, 28 Apr 2009 20:24:17 +0000 (14:24 -0600)]
Checkout page - Check Out's table sortable on barcode

Add the barcode column of the Check Out's table to the tablesorter
setup.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agominPasswordLength not returning user back to systempreferences.pl?tab=Patrons. Incorr...
Garry Collum [Wed, 29 Apr 2009 02:55:21 +0000 (22:55 -0400)]
minPasswordLength not returning user back to systempreferences.pl?tab=Patrons. Incorrect reference MinPasswordLength fixed. Could not find bug report.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3126: remove comma for IE7
Joe Atzberger [Tue, 28 Apr 2009 17:08:31 +0000 (12:08 -0500)]
Bug 3126: remove comma for IE7

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3130 and general cleanup
Joe Atzberger [Tue, 28 Apr 2009 23:23:36 +0000 (18:23 -0500)]
Bug 3130 and general cleanup

Javascript error in FF and IE when loading page and no data matches criteria.
This was because it was passing more holds than necessary to the template and
letting a template conditional inside the loop control whether to display or not.
That doesn't make sense, and it forces the table to be displayed even when all
rows FAIL the conditional.  jquery.tablesorter was confused trying to be added
on top of an empty table with nothing to sort.

There were several other errors addressed by this patch:
 ~ 13 unused variables deleted.
 ~ regexps run on user-supplied $var before checking $var defined
 ~ decimal ratio prohibited and silently replaced with "3"

I also added the hold ratio to a column display, with a jquery tweak to put it in
the ratio input box on click.  Hidden .sql div now contains the actual query run,
like the other wizard reports.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2505 adding warnings to opac-authorities-home and fixing warnings generated
Chris Cormack [Tue, 28 Apr 2009 23:32:27 +0000 (11:32 +1200)]
Bug 2505 adding warnings to opac-authorities-home and fixing warnings generated

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBugs 2822, 2551, 3163 fixes to XSLT
Jane Wagner [Tue, 28 Apr 2009 12:01:25 +0000 (08:01 -0400)]
Bugs 2822, 2551, 3163 fixes to XSLT

Resent with additional comments.

Bug 2822 -- for leader6 types of i (nonmusical) or j (musical), the hitlist
uses an icon and label of Sound while the record display page uses
Music.  Modified record display to use Sound.

Bug 2551 -- for leader7 type of i says it's type Serial which is incorrect.
The bug report suggested Integrating Resource" but the latest install's
hitlist page uses Continuing Resource.  Changed the record display page
to also use Continuing Resource.

Bug 3163 -- if you have a 110 or 710 with a subfield a and a subfield b,
there's no space between them (on either hitlist or record display).
The two headings run together in the display.  Modified the utils file
to force a space.

The XSLT files referenced two icons which did not exist (globe.png and
drive_remote.png. Replaced those with icons from the same subject area
(world.png and drive_web.png).  Also, corrected spelling of icon name
from pencile.png to pencil.png.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix DOS newlines in MARC21slimUtils.xsl
Galen Charlton [Tue, 28 Apr 2009 13:58:05 +0000 (08:58 -0500)]
fix DOS newlines in MARC21slimUtils.xsl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3156 followup Fix display of LTFL content when not displaying it in tabs.
Chris Catalfo [Tue, 28 Apr 2009 12:36:29 +0000 (12:36 +0000)]
bug 3156 followup Fix display of LTFL content when not displaying it in tabs.

LTFL content needs to appear in div rather than span elements.  This patch simply replaces the LTFL span elements with divs to fix Bug 3156: LTFL content not displaying if LibraryThingForLibrariesTabbedView sys pref is Off.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix to remove blank newlines from labels
Mason James [Fri, 16 Jan 2009 08:16:27 +0000 (21:16 +1300)]
fix to remove blank newlines from labels

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agodon't try to resolve if WEBSERVER_IP is set and now use WEBSERVER_HOST.
Marc Chantreux [Thu, 8 Jan 2009 00:52:44 +0000 (00:52 +0000)]
don't try to resolve if WEBSERVER_IP is set and now use WEBSERVER_HOST.

as it crashed the installer when DNS doesn't respond as expected.

it will now use the WEBSERVER_HOST described in the doc but unused in the script.

TODO:

- lot of koha-httpd.conf generation is useless if you plan to use vhosts strategy
- on some recent linux distributions, $(getent hosts `hostname`) will give us
  127.0.0.1. this is to not mess up the MTA when there is no network. Don't
  you think there would be an error message in this case?
- default domain is localdomain .. this is not a good idea but i haven't a better one.
  can a network guru fix that ?

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2750) Cannot return to another branch if the independantbranch option is set...
Nahuel ANGELINETTI [Tue, 23 Dec 2008 14:14:27 +0000 (15:14 +0100)]
(bug #2750) Cannot return to another branch if the independantbranch option is set as ON

The code was modified to change the holding branch ONLY if the return to the right branch is done. If you set IndependantBranch to on, you must return the document to the HomeBranch.
And verify the return is done before show the message that allow to transfert the document to another branch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2961) change the workflow of the new issue
Nahuel ANGELINETTI [Mon, 27 Apr 2009 15:13:08 +0000 (17:13 +0200)]
(bug #2961) change the workflow of the new issue

This patch change the operations, it add a new expected if no one expected exists, else it create new excepted, and set the others as "late"

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2915 followup - small efficiency improvement
Galen Charlton [Mon, 27 Apr 2009 15:31:03 +0000 (10:31 -0500)]
bug 2915 followup - small efficiency improvement

Following up on a suggestion by Joe Atzberger, save
a database query by not counting the subscriptions
to delete; instead, simply fetch a list of them.

No user-visible change.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2893: tweaks to AllowNotForLoanOverride (DB rev 030)
Galen Charlton [Mon, 27 Apr 2009 19:39:36 +0000 (14:39 -0500)]
bug 2893: tweaks to AllowNotForLoanOverride (DB rev 030)

* moved to DB rev 030
* assigned AllowNotForLoanOverride to circulation syspref category
* added a hint to the circulation operator: when the
  AllowNotForLoanOverride is ON, if the operator attempts
  to check out an item that is normally not for loan due to
  its item type or because the item is explicitly marked as
  not-for-loan, the operator is given the chance to check
  out the item anyway.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2893) Allow loan forcing if a syspref is set
Nahuel ANGELINETTI [Thu, 8 Jan 2009 13:15:31 +0000 (14:15 +0100)]
(bug #2893) Allow loan forcing if a syspref is set

If the syspref 'AllowNotForLoanOverride'(YESNO) is set to YES, the librarian is able to force a loan on an item set as "not for
loan".
If the item is not for loan and the syspref is set to YES, koha will ask to the librarian if he really want to check-out it, else
do nothing.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago_set_defaults_for_add easier to read
Marc Chantreux [Fri, 16 Jan 2009 09:23:27 +0000 (09:23 +0000)]
_set_defaults_for_add easier to read

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2920 Display AmazonReviews syspref in the enhanced content category
Frédéric Demians [Mon, 27 Apr 2009 14:17:36 +0000 (16:17 +0200)]
Bug 2920 Display AmazonReviews syspref in the enhanced content category

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2559: fix search limit on Spanish language (DB rev 029)
Galen Charlton [Mon, 27 Apr 2009 12:29:05 +0000 (07:29 -0500)]
bug 2559: fix search limit on Spanish language (DB rev 029)

Fixes bug where searching with the language limit set
to Spanish would instead retrieve Russian records.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFollowup (1 of 2) to: Correct iso639_2_code for Spanish is "spa" not "rus".
Joe Atzberger [Thu, 23 Oct 2008 20:56:22 +0000 (15:56 -0500)]
Followup (1 of 2) to: Correct iso639_2_code for Spanish is "spa" not "rus".

updatedatabase change deferred for now.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCorrect iso639_2_code for Spanish is "spa" not "rus".
Joe Atzberger [Mon, 13 Oct 2008 16:46:14 +0000 (11:46 -0500)]
Correct iso639_2_code for Spanish is "spa" not "rus".

The result of this apparent copy/paste error is that adv. search
limits by Language=Spanish would return results for Russian!

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3114 - IE7 spacing needs width: 100% to avoid line collapse.
Joe Atzberger [Fri, 24 Apr 2009 22:31:53 +0000 (17:31 -0500)]
Bug 3114 - IE7 spacing needs width: 100% to avoid line collapse.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 3161: don't display blank language chooser
Galen Charlton [Sun, 26 Apr 2009 16:45:24 +0000 (11:45 -0500)]
bug 3161: don't display blank language chooser

If more than one translation is installed but only one is active, a blank div
is displayed at the bottom of all pages in the intranet (and in the OPAC if the
opacchangelanguages system preference is ON), taking a bit of vertical real
estate needlessly.

Changed so that <div> for the language chooser is output
only if there is more than one enabled language.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoWhoops missed Brendan's commit, fixed the history now
Chris Cormack [Mon, 27 Apr 2009 00:17:21 +0000 (12:17 +1200)]
Whoops missed Brendan's commit, fixed the history now

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505 fixing warnings, I think we have them all for opac-search.pl now
Chris Cormack [Mon, 27 Apr 2009 04:08:21 +0000 (16:08 +1200)]
bug 2505 fixing warnings, I think we have them all for opac-search.pl now

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2920 DB Rev 028 AmazonReviews syspref
Frédéric Demians [Mon, 27 Apr 2009 05:49:58 +0000 (07:49 +0200)]
Bug 2920 DB Rev 028 AmazonReviews syspref

Add AmazonReviews syspref.

DOCUMENTATION:

  AmazonReviews is OPACAmazonReviews counterpart,
  except that in staff interface Editorial Review
  is displayed alone, without reader reviews.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2920 Fixes Amazon content use in pro interface
Frédéric Demians [Mon, 27 Apr 2009 05:49:57 +0000 (07:49 +0200)]
Bug 2920 Fixes Amazon content use in pro interface

This patch fixes Amazon content use in pro interface :

* Control Amazon Editorial Review by a new syspref:
  AmazonReviews.
* /catalogue/detail.tmpl doesn't display Amazon reader reviews.
  It displays only Editorial review in Description tab.
  This patch disable request for AWS reader reviews.
* Use Amazon TLD for linking book cover to local
  Amazon Search Inside.
* Minor clean up on C4::External::Amazon.pm

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBugfix #2920 Avoid doing unecessary calls to Amazon Web Services
Frédéric Demians [Sat, 25 Apr 2009 07:24:54 +0000 (09:24 +0200)]
Bugfix #2920 Avoid doing unecessary calls to Amazon Web Services

This patch modifies how AWS is called. AWS is now called
depending on syspref. It works completly for OPAC; it
has to be refined for intranet.

For OPAC:

  * If OPACAmazonReviews is set, AWS EditorialReview and
    Reviews (users) are grabed.
  * If OPACAmazonSimilarItems is set, AWS Similarities info
    are grabed.
  * If nothing is asked, AWS is not called anymore,
    sparing server bandwidth.

For intranet:

  It works as it used to work. AWS is called if AmazonEnabled
  is set whatever how other syspref are set.

  TODO:

    * Add a AmazonReviews syspref
    * Request Amazon content depending on AmazonSimilarities
      and AmazonReviews syspref

DOCUMENTATION:

  It should be explained that Amazon services related syspref
  have two levels:

  * AmazonEnable / OPACAmazonEnable
  * Other: OPACAmazonReviews (new), OPACAmazonSimilarProduct,
    OPACAmazonCover

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: fix more warnings in opac-search.pl
Galen Charlton [Sun, 26 Apr 2009 15:23:55 +0000 (10:23 -0500)]
bug 2505: fix more warnings in opac-search.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2505 Adding use warnings, and fixing the warnings generated
Chris Cormack [Sat, 25 Apr 2009 04:49:48 +0000 (16:49 +1200)]
Bug 2505 Adding use warnings, and fixing the warnings generated

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 3148 : Database revision to make zebraqueue.biblio_auth_number as big as auth_hea...
Ryan Higgins [Sat, 25 Apr 2009 17:29:01 +0000 (13:29 -0400)]
Bug 3148 : Database revision to make zebraqueue.biblio_auth_number as big as auth_header.authid.

If you've never truncated your zebraqueue table, you might want to
update your index and truncate it before applying this patch and running updatedatabase.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix warning in C4::AuthoritiesMarc::merge()
Galen Charlton [Sun, 26 Apr 2009 14:48:42 +0000 (09:48 -0500)]
fix warning in C4::AuthoritiesMarc::merge()

"my" variable @X masks earlier declaration in same scope at C4/AuthoritiesMarc.pm line 1200

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoimproved POD for C4::Serials::HasSubscriptionExpired()
Galen Charlton [Sun, 26 Apr 2009 14:48:41 +0000 (09:48 -0500)]
improved POD for C4::Serials::HasSubscriptionExpired()

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFix :Fatal Errors occurs when date is not a regular date
Henri-Damien LAURENT [Fri, 6 Mar 2009 11:03:21 +0000 (12:03 +0100)]
Fix :Fatal Errors occurs when date is not a regular date

When the start date or the "current date" is not a regular date
the transformation and checkdate fails because there is no
Year/Month/Date
This patch makes it fail more 'silenty' for the users and raises an
error that could be coped with.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2937) use check_date in C4::Serials::hassubscriptionexpired
Nahuel ANGELINETTI [Fri, 30 Jan 2009 15:48:02 +0000 (16:48 +0100)]
(bug #2937) use check_date in C4::Serials::hassubscriptionexpired

This patch change the date check in hassubscriptionexpired to use Date_Calc::check_date

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRemoving map calls in void context
Henri-Damien LAURENT [Sat, 22 Nov 2008 21:23:28 +0000 (22:23 +0100)]
Removing map calls in void context

Thanks Joe.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoproblem saving authority with ? and other special characters [resend]
Henri-Damien LAURENT [Wed, 19 Nov 2008 14:38:37 +0000 (15:38 +0100)]
problem saving authority with ? and other special characters [resend]

Filtering those signs in order not to have them interpreted.
Removing some warns

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug Fix : 2494
Henri-Damien LAURENT [Wed, 19 Nov 2008 13:58:24 +0000 (14:58 +0100)]
Bug Fix : 2494

Adds some rewriting
bug fix for merging
AddAuthority had some problem with updating existing data.
ModAuthority adds a new Syspref called MergeAuthoritiesOnUpdate which is used to launch or disable biblios update when updating an authority

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