]> git.koha-community.org Git - koha.git/log
koha.git
15 years agoRevert "Updates to the example crontab"
Joe Atzberger [Tue, 23 Dec 2008 18:34:57 +0000 (12:34 -0600)]
Revert "Updates to the example crontab"

This reverts commit def09f5a21b47431ff107d9a530900cd5a4fb93d.

As I emailed to the patches list Oct 06, 2008:

I suggest we need to revert Josh' commit def09f5a21b47431ff107d9a530900cd5a4fb93d.

The effect on the crontab example is to invalidate the lines being executed.  The lines were apparently copied in from a cron source, not crontab, despite the header describing it NOT being an example for cron.  It also runs longoverdue twice, instead of fines.pl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFixes bugs #2875-2879
Frederic Demians [Tue, 23 Dec 2008 07:57:15 +0000 (08:57 +0100)]
Fixes bugs #2875-2879

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdd 'use warnings' and correct uninitialized value warnings.
Stan Brinkerhoff [Wed, 24 Dec 2008 12:41:56 +0000 (07:41 -0500)]
Add 'use warnings' and correct uninitialized value warnings.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRemoving ':' from table headers.
Stan Brinkerhoff [Wed, 24 Dec 2008 12:13:16 +0000 (07:13 -0500)]
Removing ':' from table headers.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdded 'dbd::sqlite' as appears as an optional module during latest installers.
Stan Brinkerhoff [Wed, 24 Dec 2008 11:52:30 +0000 (06:52 -0500)]
Added 'dbd::sqlite' as appears as an optional module during latest installers.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: further warnings fixes to memberentry.pl
Galen Charlton [Tue, 30 Dec 2008 01:42:45 +0000 (19:42 -0600)]
bug 2505: further warnings fixes to memberentry.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdded 'warnings' support, fixed bug 2870.
stan [Wed, 24 Dec 2008 03:06:10 +0000 (22:06 -0500)]
Added 'warnings' support, fixed bug 2870.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2856 followup - display form correctly when modifying patron
Galen Charlton [Tue, 30 Dec 2008 01:19:30 +0000 (19:19 -0600)]
bug 2856 followup - display form correctly when modifying patron

Upon further testing, original patch for 2856 had effect
of displaying form with blank values when modifying
an existing patron record.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2856 followup: remove unconditional warn
Galen Charlton [Mon, 29 Dec 2008 19:53:11 +0000 (13:53 -0600)]
bug 2856 followup: remove unconditional warn

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2856) Activate the duplicate patrons detection and check birthdate only if...
Nahuel ANGELINETTI [Thu, 11 Dec 2008 09:08:15 +0000 (10:08 +0100)]
(bug #2856) Activate the duplicate patrons detection and check birthdate only if one is set

This patch activate the check of unique member, it was checked but not shown, and the member was added even if a duplicate was
detected.
It improve the duplicate detection, to check the birthdate only if it was specified in the form.
And fix an url of "Yes" link(if the borrower added IS the duplicate detected).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTest suite: C4::Members::GetMemberDetails
Allen Reinmeyer [Wed, 24 Dec 2008 22:09:49 +0000 (16:09 -0600)]
Test suite: C4::Members::GetMemberDetails

Tests for C4::Members::GetMemberDetails.  Validates both calls
using either borrowernumber or cardnumber.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTest suite: C4::Members::GetMember
Allen Reinmeyer [Wed, 24 Dec 2008 22:09:50 +0000 (16:09 -0600)]
Test suite: C4::Members::GetMember

Tests the functionality of C4::Members::GetMember.  Tests exist
for every way that GetMember could be called on to search for an
existing member.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2874 [3/3] flagsrequired => { circulate => "circulate_remaining_permissions" }
John Beppu [Mon, 22 Dec 2008 17:07:55 +0000 (11:07 -0600)]
bug 2874 [3/3] flagsrequired => { circulate => "circulate_remaining_permissions" }

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2874 [2/3]: added awareness of circulate/override_renewals subpermission
John Beppu [Mon, 22 Dec 2008 17:07:54 +0000 (11:07 -0600)]
bug 2874 [2/3]: added awareness of circulate/override_renewals subpermission

A few minor whitespace cleanups were thrown in as well.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2874 [1/3]: schema changes for overriding blocked renewals
John Beppu [Mon, 22 Dec 2008 17:07:53 +0000 (11:07 -0600)]
bug 2874 [1/3]: schema changes for overriding blocked renewals

Two new subpermissions of circulate were added:
- circulate_remaining_permissions
- override_renewals

Also, the following schema alterations were made
- permissions.code      : varchar(30) => varchar(64)
- user_permissions.code : varchar(30) => varchar(64)

  (The reason for this was because "circulate_remaining_permissions"
   was greater than 30 characters long and wouldn't fit.)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoImprovements to offline circ report (linking to title and patron records), and moving...
Owen Leonard [Thu, 2 Oct 2008 20:19:55 +0000 (15:19 -0500)]
Improvements to offline circ report (linking to title and patron records), and moving English strings from the script into the template to allow translation.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoA script generating keyword clouds from Zebra Indexes
Frederic Demians [Thu, 18 Dec 2008 17:38:25 +0000 (18:38 +0100)]
A script generating keyword clouds from Zebra Indexes

See embedded perldoc to see how it works.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agotest suite: C4::ImportBatch
Andrew Moore [Fri, 19 Dec 2008 22:12:25 +0000 (16:12 -0600)]
test suite: C4::ImportBatch

Here are some tests for functions in C4::ImportBatch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2868: fixing typo in NoZebraIndexes setting in sysprefs
Andrew Moore [Wed, 17 Dec 2008 21:13:49 +0000 (15:13 -0600)]
Bug 2868: fixing typo in NoZebraIndexes setting in sysprefs

There was a missing ' in the value of the NoZebraIndexes syspref. This
patch fixes it.

It's not necessary to fix the existing settings because
C4::Biblio::GetNoZebraIndexes is liberal enough to be able to parse even
the deformed value.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2799: DB rev 008 for library transfer limits
Galen Charlton [Wed, 17 Dec 2008 20:09:26 +0000 (14:09 -0600)]
bug 2799: DB rev 008 for library transfer limits

Also fix automatic merge that put the 008
update in the wrong place.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2799: followup 2
Galen Charlton [Wed, 17 Dec 2008 20:40:53 +0000 (14:40 -0600)]
bug 2799: followup 2

* fix TMPL_IF embedded in HTML tag
* convert a remaining <TMPL_VAR> to a comment

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2799: followup
Galen Charlton [Wed, 17 Dec 2008 20:09:27 +0000 (14:09 -0600)]
bug 2799: followup

Various fixes to meet coding standards, template
design, and basic functionality:

* (bug 2505) turned on warnings
* use correct loop names when populating
  $template
* put all TMPL_FOO in HTML comments
* assign UseBranchTransferLimits to circulation tab
* change template to use "library" instead of "branch"
* use standard YUI scripting for toolbar buttons
* add new script to admin menu

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2799: database update to "branch transfer limits"
Andrew Moore [Wed, 17 Dec 2008 20:09:25 +0000 (14:09 -0600)]
Bug 2799: database update to "branch transfer limits"

This patch cleans up kohastructure.sql a bit:
 * moved the table definition to above the ending comments
 * added a "drop table..." statement

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdded 'Branch Transfer Limits' Feature requested by Geauga County Library System.
Kyle Hall [Wed, 17 Dec 2008 20:09:24 +0000 (14:09 -0600)]
Added 'Branch Transfer Limits' Feature requested by Geauga County Library System.

Added syspref to updatedatabase.

Updated kohastructure.sql with the limits table.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agowhitespace cleanup - admin/systempreferences.pl
Galen Charlton [Wed, 17 Dec 2008 15:53:40 +0000 (09:53 -0600)]
whitespace cleanup - admin/systempreferences.pl

applied xt/perltidyrc to admin/systempreferences.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2860: allow setting of a syspref value to '0'
Galen Charlton [Wed, 17 Dec 2008 15:53:39 +0000 (09:53 -0600)]
bug 2860: allow setting of a syspref value to '0'

Fixed invalid test for the existence of CGI parameter; if it
is possible that the value of a parameter can be 0, it is
not sufficient to test like this:

if ($input->param('foo')) { ...

since the test will fail for *any* value that evaluates
to Perl false.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: turn on warnings in admin/systemprefernces.pl
Galen Charlton [Wed, 17 Dec 2008 15:53:38 +0000 (09:53 -0600)]
bug 2505: turn on warnings in admin/systemprefernces.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2867 [2/2]: db changes for single-screen syspref editing
Jesse Weaver [Mon, 10 Nov 2008 22:20:06 +0000 (16:20 -0600)]
bug 2867 [2/2]: db changes for single-screen syspref editing

Database revision 007.

[Includes a correction by Galen Charlton]

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2867 [1/2]: Move basic editing to main system preferences screen
Jesse Weaver [Wed, 17 Dec 2008 15:53:37 +0000 (09:53 -0600)]
bug 2867 [1/2]: Move basic editing to main system preferences screen

This allows the user to edit system preferences from the main system
preferences screen, rather than having to dig down into each
preference.

[This patch includes some modifications by Galen Charlton
 to fix XHTML validation errors and a couple errors that
 would crop up under warnings.]

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2824: improving error messages from test suite errors
Andrew Moore [Wed, 17 Dec 2008 14:39:17 +0000 (08:39 -0600)]
bug 2824: improving error messages from test suite errors

This patch adds two warning messages to places in the test suite that
may fail. I don't think it changes any functionality, but it sure makes
it easier to figure out what has gone wrong.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofixed "too many opened files" error during translation
Marc Chantreux [Fri, 12 Dec 2008 17:13:14 +0000 (18:13 +0100)]
fixed "too many opened files" error during translation

The problem was that the TmplTokenizer create global symbols to store file
handlers so the objects aren't destroyed by the garbage collector.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2864 [2/2]: move rss/* to misc/cronjobs/rss/*
Galen Charlton [Mon, 15 Dec 2008 15:58:47 +0000 (09:58 -0600)]
bug 2864 [2/2]: move rss/* to misc/cronjobs/rss/*

As rss.pl is not a CGI script, moved it to join the
other cronjobs.  Full documentation of the script
is in misc/cronjobs/rss/rss.pl, but to summarize:

[1] rss.pl is run on the command line to produce
    an RSS XML document.  The output should be
    placed in a directory accessible to the OPAC
    (or staff) web interface so that users can download
    the RSS feed.  An example of usage:

    misc/cronjobs/rss.pl lastAcquired.conf

    Normally rss.pl should be run periodically (e.g., daily)
    to keep the feed up-to-date.

[2] The configuration file (e.g., lastAcquired.conf) lists

    * name of the template file to use
    * path of output file
    * SQL query

    rss.pl runs the SQL query, then feeds the output of the
    query through the template to produce the output file.

[3] The template file (e.g., lastAcquired.tmpl) uses
    HTML::Template syntax like any of the HTML
    templates for the web interface.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2864 [1/2]: restore rss/rss.pl
Galen Charlton [Mon, 15 Dec 2008 15:58:46 +0000 (09:58 -0600)]
bug 2864 [1/2]: restore rss/rss.pl

Restored basic functionality of rss/rss.pl:

[1] Reverted "Add support for itemtypes. Still need to hardcode 'reservable'"
    Revert commit e7575b00edd6098a869425a69856916108e84914, which had
    replace the script with one of its templates.

[2] Minor fixes to update it, including using HTML::Template::Pro instead
    of HTML::Template, perltidy, and turning on warnings.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2859: add index on bibioitems.issn (DB rev 006)
Galen Charlton [Thu, 11 Dec 2008 17:59:55 +0000 (11:59 -0600)]
bug 2859: add index on bibioitems.issn (DB rev 006)

Because of a query on biblioitems.issn in
C4::Breeding::ImportBreeding(), libraries doing a lot
of bib imports were experiencing performance problems,
which adding an index will alleviate.

A possible longer term solution is to index all
reservoir/import batch records in a Zebra database.

[LL bug 232]

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agominor fixes to UNIMARC_sync_date_created_with_marc_biblio.pl
Galen Charlton [Thu, 11 Dec 2008 16:36:04 +0000 (10:36 -0600)]
minor fixes to UNIMARC_sync_date_created_with_marc_biblio.pl

- chmod 0755
- add newline in error message shown if attempting to
  run in MARC21 database

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoUpdate MARC to add 099$c and 099$d fields, with :
Henri-Damien LAURENT [Fri, 21 Nov 2008 21:37:59 +0000 (22:37 +0100)]
Update MARC to add 099$c and 099$d fields, with :

This allow the acquisition date sorting in UNIMARC.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdd default classification sources and filing rules to fr-FR
Frederic Demians [Mon, 17 Nov 2008 16:38:46 +0000 (17:38 +0100)]
Add default classification sources and filing rules to fr-FR

fr-FR install wasn't populating classication rules tables:

  * class_sort_rules
  * class_sources

This patch add default values for those tables
mandatory during installation process.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2841: add an hidden field with booksellerid
Nahuel Angelinetti [Wed, 3 Dec 2008 16:52:39 +0000 (17:52 +0100)]
bug 2841: add an hidden field with booksellerid

The bug throw because the booksellerid is not in the request, and the newordersuggestion.pl need this one to import
it in a new order.
So I just added an hidden input in the search form.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2827: better language check
Nahuel Angelinetti [Thu, 27 Nov 2008 13:43:21 +0000 (14:43 +0100)]
bug 2827: better language check

This patch use a regexp instead 'eq' to find if french is an opaclanguage.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2831) Add a variable, that create a js error, and make the script not working.
Nahuel ANGELINETTI [Thu, 11 Dec 2008 15:48:16 +0000 (16:48 +0100)]
(bug #2831) Add a variable, that create a js error, and make the script not working.

the variable added is numberpattern, and is get from the document.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2829: add a plugin for 010$a field and 210$c
Nahuel Angelinetti [Thu, 11 Dec 2008 15:39:27 +0000 (16:39 +0100)]
bug 2829: add a plugin for 010$a field and 210$c

This plugin use the 'ISBN' field(010$a) to search the "publishercode"(editor) and set it in as 210$c.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoadd calls to clear_syspref_cache()
Galen Charlton [Thu, 11 Dec 2008 15:48:37 +0000 (09:48 -0600)]
add calls to clear_syspref_cache()

temporary until mutator for sysprefs created

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding some fixing for NoZebra
Henri-Damien LAURENT [Wed, 19 Nov 2008 13:58:15 +0000 (14:58 +0100)]
Adding some fixing for NoZebra

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug in rebuild_zebra verbose logging - found another print I didn't want to see all...
Michael Hafen [Fri, 21 Nov 2008 16:34:44 +0000 (09:34 -0700)]
bug in rebuild_zebra verbose logging - found another print I didn't want to see all the time

Add the phrase 'if ( $verbose_logging )' to the two print statements
concerning the skipping of biblio or authority records.

I recently had to split biblio and authority index updating in my cron
script ( had some really big records so had to add the -x switch which
should only be used on biblios accourding to the help ).  So I noticed
that rebuild_zebra.pl printed messages that it was skipping biblios or
authorities.

This patch is to conditionalize those prints based on the verbose
logging switch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2689: Fix the checked box "manual history" and fix the "null" value of serial...
Nahuel Angelinetti [Fri, 17 Oct 2008 13:45:27 +0000 (15:45 +0200)]
bug 2689: Fix the checked box "manual history" and fix the "null" value of serial next issue publication date

first part of this patch is the manual history must be checked if in database its checked.
Then, when you do not enter a value "Next issue publication date", a null value was set in the database, now, if empty, the  "first issue date" is set as "next issue".
And finally, We check that "manual history" is checked to show the "Subscription history" part. Actually its never shown.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMissing ";" in javascript
Nahuel Angelinetti [Tue, 9 Dec 2008 10:23:14 +0000 (11:23 +0100)]
Missing ";" in javascript

This patch fix a missing ; in javascript, that open the renew popup. Instead it open it in the current window.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFix something that doesn't work in some perl versions.
Nahuel Angelinetti [Wed, 10 Dec 2008 09:24:36 +0000 (10:24 +0100)]
Fix something that doesn't work in some perl versions.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years ago(bug #2853) fix an error in returned string
Nahuel Angelinetti [Tue, 9 Dec 2008 14:15:43 +0000 (15:15 +0100)]
(bug #2853) fix an error in returned string

this patch delete the last char of the returned string, that is not conform to unimarc, only one blankchar must be at the end of
the leader string.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: enable warnings for opac/sco/sco-main.pl
Galen Charlton [Mon, 8 Dec 2008 14:45:39 +0000 (08:45 -0600)]
bug 2505: enable warnings for opac/sco/sco-main.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMinor cleanup, unused variable removed.
Joe Atzberger [Mon, 8 Dec 2008 17:42:47 +0000 (11:42 -0600)]
Minor cleanup, unused variable removed.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2857: fix UTF-8 conversion issues in web services
Galen Charlton [Wed, 10 Dec 2008 20:02:17 +0000 (14:02 -0600)]
bug 2857: fix UTF-8 conversion issues in web services

svc/bib and svc/new_bib had two problems related to UTF-8 character conversion:

[1] Couple instances of "Wide character" warnings
[2] When saving a new (MARC21) bib whose Leader/09 was not 'a', did not apply
default character conversion and set the Leader/09 to 'a'.

Fix includes two parts:

[1] Setting :utf8 on STDOUT
[2] Doing default MARC-8 to UTF-8 conversion if applicable

This patch also turns on warnings in all scripts under svc per bug 2505.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: turned on warnings for suggestion/acceptorreject.pl
Galen Charlton [Wed, 10 Dec 2008 23:30:03 +0000 (17:30 -0600)]
bug 2505: turned on warnings for suggestion/acceptorreject.pl

Also fixed case where HTML could have unclosed <form> elements,
leading to warnings in the error log when accepting pending
or rejected suggestions.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: turn on warnings in reviews/reviewswaiting.pl
Galen Charlton [Wed, 10 Dec 2008 23:36:56 +0000 (17:36 -0600)]
bug 2505: turn on warnings in reviews/reviewswaiting.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2505: turn on warnings in two scripts
Galen Charlton [Thu, 11 Dec 2008 14:26:10 +0000 (08:26 -0600)]
bug 2505: turn on warnings in two scripts

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2848 [2/2]: Letters for holds notifications (DB rev 005)
Jesse Weaver [Fri, 5 Dec 2008 21:11:14 +0000 (15:11 -0600)]
bug 2848 [2/2]: Letters for holds notifications (DB rev 005)

This adds the necessary SQL for holds notifications to work.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2848 [1/2]: Send notifications to patrons when hold is received
Jesse Weaver [Fri, 5 Dec 2008 21:11:13 +0000 (15:11 -0600)]
bug 2848 [1/2]: Send notifications to patrons when hold is received

This adds notification code to Reserves.pm to send notifications through email
or SMS to patrons when their hold is filled. It does not send the letter
directly, but instead uses EnqueueLetter. Also, it relies on
EnhancedPatronMessaging being on and the hold notification being turned on for
the specific patron through the 'Messaging' tab.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoDB rev 003 -> 004 (for Allen's 2754 patch)
Galen Charlton [Fri, 5 Dec 2008 13:43:55 +0000 (07:43 -0600)]
DB rev 003 -> 004 (for Allen's 2754 patch)

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug #2754 Show hold request priority in OPAC
Allen Reinmeyer [Fri, 5 Dec 2008 13:43:54 +0000 (07:43 -0600)]
Bug #2754 Show hold request priority in OPAC

Code changes to display the priority level on holds in OPAC. Turned on/off based on System Preference.  Database changes to systempreferences for both English and French

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2613: allow web self-check to accept patron ID
Galen Charlton [Fri, 5 Dec 2008 23:17:02 +0000 (17:17 -0600)]
bug 2613: allow web self-check to accept patron ID

Use 'patronid' instead of 'userid' as the query parameter
for passing the patron userid or barcode around; 'userid'
is claimed by C4::Auth and should be used only for
authentication pages.

Fixes the problem where entering a patron's card
number would cause a redirect to the OPAC login
page.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCleaning up Self-Checkout interface, adding some basic formatting based on other...
Owen Leonard [Wed, 26 Nov 2008 20:46:07 +0000 (14:46 -0600)]
Cleaning up Self-Checkout interface, adding some basic formatting based on other standard styles.

Patch also removes a hard-coded English string from the script and some unused functions from sco.js.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2825: improving readability of output messages in updatedatabase.pl
Andrew Moore [Wed, 26 Nov 2008 19:22:50 +0000 (13:22 -0600)]
bug 2825: improving readability of output messages in updatedatabase.pl

Two recent changes to updatedatabase.pl don't include newlines at the
end of their status messages. This patch adds them. This is only useful
if you're running this from the command line.

Note: there is an opportunity here to extract some of the simlar logic
from each of these blocks to a function.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2675 users public lists do not show on opac-detail.pl
Allen Reinmeyer [Wed, 3 Dec 2008 18:10:50 +0000 (12:10 -0600)]
Bug 2675 users public lists do not show on opac-detail.pl

Code change actually occurs in opac-addbybiblionumber.pl accessed via opac-detail.pl.
Bug fix allows for users to select the 'save to lists' link and add item
to their private, public and all open lists.  Previously, only private
and open lists were retrieved and listed.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTrivial cleanup
Joe Atzberger [Thu, 4 Dec 2008 04:33:03 +0000 (22:33 -0600)]
Trivial cleanup

2 FIXME's added.
Vestigial "8-character tabs" comment removed.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoremove superfluous retrieval of $ENV{'REMOTE_USER'}
Galen Charlton [Fri, 5 Dec 2008 17:09:32 +0000 (11:09 -0600)]
remove superfluous retrieval of $ENV{'REMOTE_USER'}

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoMinor logical cleanup.
Joe Atzberger [Mon, 1 Dec 2008 21:38:07 +0000 (15:38 -0600)]
Minor logical cleanup.

The conditional assignment previously was:
    $norequests = 0 && $norequests if ....

The execution of "0" served no purpose.
Also consolidated RequestOnOpac logic to the one line that cared about it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoComment out unused variable.
Joe Atzberger [Mon, 1 Dec 2008 21:38:06 +0000 (15:38 -0600)]
Comment out unused variable.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2758: don't confirm checkout if fine balance is 0
Galen Charlton [Tue, 2 Dec 2008 16:58:40 +0000 (10:58 -0600)]
bug 2758: don't confirm checkout if fine balance is 0

Fixes problem where if the IssuingInProcess preference is ON,
the operator is always required to confirm a checkout if
the patron has had any fine transactions at all, even if
the patron's balance is 0.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug #2757: change the opac login generation
Nahuel Angelinetti [Tue, 2 Dec 2008 10:24:30 +0000 (11:24 +0100)]
bug #2757: change the opac login generation

This patch change the method used to generate a new login for a user when his informations are modified(memberentry.pl).

Before the login are generated with the first letter of lastname and the 9 first letter of the surname, and do not verify the
login already exist).
Now the login will be lastname.surnameX, where X is an incremented digit if the login already exists.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCorrections and tweaks to recent lists fixes.
Owen Leonard [Mon, 24 Nov 2008 20:34:18 +0000 (14:34 -0600)]
Corrections and tweaks to recent lists fixes.

Corrected an instance where <!-- TMPL_IF --> was embedded inside an HTML tag (conflicts with translation script). Modified styling of confirm button to match existing interface convention.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTweaks to display of "views" tabs (MARC, ISBD, etc).
Owen Leonard [Wed, 26 Nov 2008 20:45:58 +0000 (14:45 -0600)]
Tweaks to display of "views" tabs (MARC, ISBD, etc).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix POD error introduced in last patch
Galen Charlton [Mon, 1 Dec 2008 17:12:55 +0000 (11:12 -0600)]
fix POD error introduced in last patch

Run prove xt/author/podcorrectness.t to verify
that POD is syntactically correct.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2817: Added support to pull Amazon information based on UPC, EAN, and 13-digit...
Danny Bouman [Mon, 24 Nov 2008 19:21:08 +0000 (14:21 -0500)]
bug 2817: Added support to pull Amazon information based on UPC, EAN, and 13-digit ISBN

By adding support for UPC,EAN and 13-digit ISBN we are able to pull much more content from Amazon, especially on most music and dvd content which
does not have an ISBN.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoPure whitespace edits. Trying to make members more readable.
Joe Atzberger [Mon, 24 Nov 2008 23:42:32 +0000 (17:42 -0600)]
Pure whitespace edits. Trying to make members more readable.

Use git diff -w to show only one line (a blank one) was edited (removed).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2818 - Allow patron editing to coexist with minPasswordLength and '****'.
Joe Atzberger [Mon, 24 Nov 2008 23:42:33 +0000 (17:42 -0600)]
Bug 2818 - Allow patron editing to coexist with minPasswordLength and '****'.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix permissions error introduced by previous patch
Galen Charlton [Mon, 1 Dec 2008 16:38:23 +0000 (10:38 -0600)]
fix permissions error introduced by previous patch

Modules in C4 should not be executable.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoNo more used since a while.
Nahuel Angelinetti [Thu, 27 Nov 2008 16:39:12 +0000 (17:39 +0100)]
No more used since a while.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoDelete all calls to thesaurus_popup.pl, and no more used plugins.
Nahuel Angelinetti [Mon, 1 Dec 2008 09:06:01 +0000 (10:06 +0100)]
Delete all calls to thesaurus_popup.pl, and no more used plugins.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoUpdating default UNIMARC framework
Henri-Damien LAURENT [Sat, 22 Nov 2008 06:21:21 +0000 (07:21 +0100)]
Updating default UNIMARC framework

The previous one was out of date and contained some errors.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2801 followup
Galen Charlton [Fri, 21 Nov 2008 00:49:23 +0000 (18:49 -0600)]
bug 2801 followup

Quell unitialized value warnings emited when
'use warnings' was added.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2801 followup... [SQUASHABLE]
Joe Atzberger [Tue, 18 Nov 2008 20:50:00 +0000 (14:50 -0600)]
Bug 2801 followup... [SQUASHABLE]

Apparently my git skills are lacking, since I intended this one-liner
to be amended into the patch I resent.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2801 and other cleanup.
Joe Atzberger [Tue, 18 Nov 2008 16:33:54 +0000 (10:33 -0600)]
Bug 2801 and other cleanup.

In addition to documented bug:
~ SearchString took an unused argument,
~ refactored repeated code into column_picks(),
~ fixed display during delete_confirm, including "name",
~ removed unused C4::Dates dependency.

Resend: typo fixed.

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

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2531 followup
Galen Charlton [Fri, 21 Nov 2008 00:12:42 +0000 (18:12 -0600)]
bug 2531 followup

* & -> &amp; in URL for XHTML validation
* (unrelated to prior patch) remove broken link to
  button-bg.png for confirm button

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug #2531 Cannot delete private shelf
Allen Reinmeyer [Mon, 17 Nov 2008 21:10:27 +0000 (15:10 -0600)]
Bug #2531 Cannot delete private shelf

Added logic to create a list of private lists to check on deletion.  Multiple pages have the ability to delete lists, both private and public.  Default option of retrieving a user's lists is retrieving the public ones.  Now private lists of the owner are retrieved as well and checked upon deletion request.  This allows any method of deletion to delete the correct list without examining the myriad of options currently used to indicate display of privateshelves and expect all pages to know whether a shelf number comes from a private or public list.  Since deletion occurs based on shelf number and the virtualshelves table has a primary key on shelf number, this will not cause unwanted deletion of shelves.

UPDATE:  added logic for conditional creation of shelves.  Also note change in staff side as deletions worked except for viewing a populated shelf, then deleting.  The confirm message showed, but the display was of public shelves, not private.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2787 : Fix Global Due date
Ryan Higgins [Wed, 19 Nov 2008 15:56:12 +0000 (09:56 -0600)]
bug 2787 : Fix Global Due date

This patch also lightly refactors the conditional structure wrapping the
call to AddItem, renaming a couple variables to remove double-negatives and
abstruse logic.  If SpecifyDueDate is ON, then the input box is used to pass
GlobalDueDate, allowing modification by circ staff, resulting in a default
'sticky' due date of the global due date in that mode.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix for bug 2447: yuipath variable not getting passed to help pages
Jesse Weaver [Wed, 19 Nov 2008 13:58:31 +0000 (14:58 +0100)]
fix for bug 2447: yuipath variable not getting passed to help pages

This updates help.pl to pass the yuipath variable correctly. It also adds some script references to help-top.inc to avoid other js errors.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoGet back boolean button on OPAC Advanced Search page
Frédéric DEMIANS [Wed, 19 Nov 2008 13:58:30 +0000 (14:58 +0100)]
Get back boolean button on OPAC Advanced Search page

With this patch, on OPAC Advanced Search page, when More Options is
selected, a boolean combo box appears in front of fields combo box.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2790 no UNIMARC search on avail items [1/2]
Henri-Damien LAURENT [Wed, 19 Nov 2008 13:58:25 +0000 (14:58 +0100)]
Bug 2790 no UNIMARC search on avail items [1/2]

Changing record.abs file to add the management of acquisition date,
modification date and lost

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2365: Serials prediction errors: Roll-overs occur with wrong issue in some cases.
Ryan Higgins [Wed, 19 Nov 2008 15:55:38 +0000 (09:55 -0600)]
Bug 2365: Serials prediction errors: Roll-overs occur with wrong issue in some cases.

See bugzilla for details.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2802 followup: remove another reference to "branches"
Galen Charlton [Tue, 18 Nov 2008 22:50:25 +0000 (16:50 -0600)]
bug 2802 followup: remove another reference to "branches"

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug fix : 2805
Henri-Damien LAURENT [Tue, 18 Nov 2008 15:18:15 +0000 (16:18 +0100)]
bug fix : 2805

deleting die on lack of user/password in config file.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoLanguage corrections: branch -> library. Changing "Libraries, branches, and groups...
Owen Leonard [Tue, 18 Nov 2008 16:01:27 +0000 (10:01 -0600)]
Language corrections: branch -> library. Changing "Libraries, branches, and groups" to "Libraries and groups." Fixes Bug 2802.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2654: require circulate permission to upload offline circ files
Galen Charlton [Tue, 18 Nov 2008 21:40:18 +0000 (15:40 -0600)]
bug 2654: require circulate permission to upload offline circ files

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2606: reduce size of offline circ patron database
Galen Charlton [Tue, 18 Nov 2008 21:40:17 +0000 (15:40 -0600)]
bug 2606: reduce size of offline circ patron database

Reduce the size of the SQLite database produced by
create_koc_db.pl by emiting only the columns
actually used by Kyle Hall's offline circ client.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoadd isbn to cataloging search results
Michael Hafen [Tue, 18 Nov 2008 22:17:16 +0000 (15:17 -0700)]
add isbn to cataloging search results

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoDo not show the language chooser
Nahuel Angelinetti [Mon, 17 Nov 2008 16:18:18 +0000 (17:18 +0100)]
Do not show the language chooser

This patch change the footer of authority search results popup to use the popup footer, so do not show the language chooser.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agomoved Test_CSV_Various.t to xt/author
Galen Charlton [Mon, 17 Nov 2008 16:18:38 +0000 (10:18 -0600)]
moved Test_CSV_Various.t to xt/author

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoResults of CSV module analysis, testing various CSV modules and configs.
Joe Atzberger [Fri, 7 Nov 2008 21:49:59 +0000 (15:49 -0600)]
Results of CSV module analysis, testing various CSV modules and configs.

The upshot is that despite appearances, whe should NEVER use Text::CSV::Unicode
or any CSV module in non-binary mode.  Text::CSV works in binary mode
by using Text::CSV_XS, so the behavior is the same for both (in binary).

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agouse DESTDIR now
Marc Chantreux [Mon, 10 Nov 2008 16:40:13 +0000 (17:40 +0100)]
use DESTDIR now

takes carre of changes in Makefile.PL about debian install

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFixes bug #2788 - CoinS support
Frederic Demians [Sun, 16 Nov 2008 08:50:54 +0000 (09:50 +0100)]
Fixes bug #2788 - CoinS support

1 In URL rft.tpages is replaced with rft.page. That's what
  Zotero expects.
2 TMPL_VAR NAME="size" is replaced with "pages" for 1.
3 TMPL_VAR NAME="publisher" is replaced with "publishercode"
  the correct value for rft.pub

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