wip/koha-chris_n.git
11 years agoMore changes to update-child process. Many scripts missing necessary data for checkin...
Owen Leonard [Tue, 8 Jul 2008 14:28:55 +0000 (09:28 -0500)]
More changes to update-child process. Many scripts missing necessary data for checking how to process an update-child request. Also cleaning up update-child.tmpl for formatting. Minor text change (translation problem?)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
11 years agoScripts missing check for Child category type would not correctly show 'update child...
Owen Leonard [Tue, 8 Jul 2008 13:25:52 +0000 (08:25 -0500)]
Scripts missing check for Child category type would not correctly show 'update child patron' link in toolbar.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
11 years agoPartial fix for bug 2319: OPAC Shelf Browse results in a 404 error
Joshua Ferraro [Tue, 8 Jul 2008 22:52:52 +0000 (17:52 -0500)]
Partial fix for bug 2319: OPAC Shelf Browse results in a 404 error

This patch fixes instances where thereis no Next or Previous item on the shelf
that were previous resulting in 404 errors

11 years agofix for 2322: Failure to reach amazon.com to retrieve enhanced content causes fatal...
Joshua Ferraro [Tue, 8 Jul 2008 21:31:04 +0000 (16:31 -0500)]
fix for 2322: Failure to reach amazon.com to retrieve enhanced content causes fatal error in Koha

11 years agobugfixing (itemtype summary) : it summary is defined, the icon & author were not...
Paul POULAIN [Fri, 20 Jun 2008 15:36:58 +0000 (17:36 +0200)]
bugfixing (itemtype summary) : it summary is defined, the icon & author were not displayed

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
11 years agoPrior to this patch, XSL and scan indexes conflicted.
Joshua Ferraro [Tue, 8 Jul 2008 20:59:09 +0000 (15:59 -0500)]
Prior to this patch, XSL and scan indexes conflicted.
This patch resolves the issue by passing $scan to the
searchResults function to toggle XSL for scan results

11 years agoFix for bug 2312: Advanced search - search limit appearing twice of limit not the...
Joshua Ferraro [Tue, 8 Jul 2008 20:57:58 +0000 (15:57 -0500)]
Fix for bug 2312: Advanced search - search limit appearing twice of limit not the item type

11 years agobugfixing scan index
Paul POULAIN [Fri, 20 Jun 2008 15:36:56 +0000 (17:36 +0200)]
bugfixing scan index

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoFix for 2308: On-line help editing doesn't work
Joshua Ferraro [Tue, 8 Jul 2008 20:22:28 +0000 (15:22 -0500)]
Fix for 2308: On-line help editing doesn't work

12 years agoPatches bug #2308
Frederic Demians [Fri, 4 Jul 2008 14:10:59 +0000 (16:10 +0200)]
Patches bug #2308

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofix to deal with 0088 & 0089 chars during indexing
Paul POULAIN [Tue, 8 Jul 2008 19:43:16 +0000 (21:43 +0200)]
fix to deal with 0088 & 0089 chars during indexing

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofrench updated
Paul POULAIN [Tue, 8 Jul 2008 19:43:15 +0000 (21:43 +0200)]
french updated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoUpdates to translations sent via email prior to release
Joshua Ferraro [Tue, 8 Jul 2008 19:37:40 +0000 (14:37 -0500)]
Updates to translations sent via email prior to release

12 years agoUpdates for translations on translate.koha.org prior to release
Joshua Ferraro [Tue, 8 Jul 2008 18:44:06 +0000 (13:44 -0500)]
Updates for translations on translate.koha.org prior to release

12 years agobug 1909: add delete field/subfield button to MARC editor
Galen Charlton [Tue, 8 Jul 2008 17:12:01 +0000 (12:12 -0500)]
bug 1909: add delete field/subfield button to MARC editor

A button to delete (non-mandatory) subfields and fields
is added to the bib and authority MARC editors.  This button,
which displays as a hypen or minus sign after the plus sign
to clone a field, acts as follows:

[1] When deleting a whole MARC field, if at least
    one other repeat of that field exists, deletes
    the field from the editor page.  If the field
    to be deleted is the last instance of that tag,
    the contents of the field are cleared, not removed.
    This allows one to delete all 650 tags, then
    add a new one without having to reload the
    record in the editor.

[2] When deleting a subfield, if at least one other
    repeat of that subfield and its tag occurs
    *anywhere else in the record, not necessarily in the same tag*,
    deletes the subfield.
    Otherwise, if the subfield is the last occurrence
    of that tag/subfield combination, clears the input
    form instead.

Documentation note: new screenshots for MARC editor, plus
description of the '-' button.

Credit to MJ Ray for introducing the '-' button
and the UnCloneField JavaScript function.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoTry to edit help, don't test, for some reason -w wasn't working
Joshua Ferraro [Tue, 8 Jul 2008 14:43:30 +0000 (10:43 -0400)]
Try to edit help, don't test, for some reason -w wasn't working

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2203 [2/2]: use textareas for MARC21 authority 6XX
Galen Charlton [Tue, 8 Jul 2008 14:33:36 +0000 (09:33 -0500)]
bug 2203 [2/2]: use textareas for MARC21 authority 6XX

Use textareas for the 6XX fields when editing MARC21
authority records - these fields, not the 5XX, contain
the lengthy notes.

Note that because of the previous patch, both text inputs
and textareas permit entry of up to 9999 characters.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2203 [1/2]: increase input maxlength in MARC editor
Galen Charlton [Tue, 8 Jul 2008 14:33:35 +0000 (09:33 -0500)]
bug 2203 [1/2]: increase input maxlength in MARC editor

The maximum length allowed for input into a subfield in
the MARC editor has been increased to 9999 charaacters
(from 255), permitting data entry of a field that meets
the maximum MARC field limit of 9999 octets.

Also set the maximum length for the leader form input
to 24 characters and the length for the MARC21 008
to 40 characters.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2315: no crash if subfield code is a metacharacter
Galen Charlton [Mon, 7 Jul 2008 22:17:05 +0000 (17:17 -0500)]
bug 2315: no crash if subfield code is a metacharacter

When generating the display form of a heading that
happens to (invalidly) have a regular expression
metacharacter as a subfield label, do not crash.

An example of such a heading field is:

  <datafield tag="650" ind1=" " ind2="0">
    <subfield code="a">Dalziel, Andrew (Fictitious character</subfield>
    <subfield code=")">xFiction.</subfield>
  </datafield>

The error message associated with the crash is:

  Unmatched ) in regex; marked by <-- HERE in m/) <-- HERE / at
  /home/koha-pro/kohaclone/C4/Heading/MARC21.pm line 220.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofix for 2286: Remove link to Network Printers
Joshua Ferraro [Tue, 8 Jul 2008 05:37:45 +0000 (00:37 -0500)]
fix for 2286: Remove link to Network Printers

12 years agoFix for 2160: My Lists Tab on OPAC Profile
Joshua Ferraro [Tue, 8 Jul 2008 05:18:40 +0000 (00:18 -0500)]
Fix for 2160: My Lists Tab on OPAC Profile

12 years agoImproving the list of itemtypes to match the bridge iconset
Joshua Ferraro [Tue, 8 Jul 2008 04:32:51 +0000 (23:32 -0500)]
Improving the list of itemtypes to match the bridge iconset

12 years agoUpdating for translation (capitalization normalization)
Joshua Ferraro [Tue, 8 Jul 2008 04:32:27 +0000 (23:32 -0500)]
Updating for translation (capitalization normalization)

12 years agoBug 1881: Managing Suggestions - default SUGGEST authorised values
Joshua Ferraro [Tue, 8 Jul 2008 04:31:51 +0000 (23:31 -0500)]
Bug 1881: Managing Suggestions - default SUGGEST authorised values

12 years agobug 2295 [followup 2/2]: initialize data in startup method
Galen Charlton [Mon, 7 Jul 2008 17:54:52 +0000 (12:54 -0500)]
bug 2295 [followup 2/2]: initialize data in startup method

It was observed that the %thash and @formats variables
were not being properly initalized during a make single-test run.

To ensure initialization, created startup and shutdown
methods to initialize those values as part of the
test object.

I have not yet investigated why the original way of
setting up %thash and @formats did not work.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2295 [followup 1/2]: added name to one of tests in icondirectories.t
Galen Charlton [Mon, 7 Jul 2008 17:54:48 +0000 (12:54 -0500)]
bug 2295 [followup 1/2]: added name to one of tests in icondirectories.t

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2295 [4/4]: moving C4::Labels tests into t/lib/KohaTest
Andrew Moore [Mon, 7 Jul 2008 17:54:49 +0000 (12:54 -0500)]
bug 2295 [4/4]: moving C4::Labels tests into t/lib/KohaTest

t/Labels.t was dependent on a working test database, so I'm moving those
tests into t/lib/KohaTest

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2295 [3/4]: moving C4::Items tests into t/lib/KohaTest
Andrew Moore [Mon, 7 Jul 2008 17:54:50 +0000 (12:54 -0500)]
bug 2295 [3/4]: moving C4::Items tests into t/lib/KohaTest

The t/Items.t tests were actually dependent on the database, so I'm moving them
into t/lib/KohaTest.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2295 [2/4]: moving icondirectories.t from t to xt
Andrew Moore [Mon, 7 Jul 2008 17:54:47 +0000 (12:54 -0500)]
bug 2295 [2/4]: moving icondirectories.t from t to xt

the icondirectories.t test depends on the database, so it needed to be
moved. But, it's really more of an author test since it's testinging the
correct structure of our distribution, so I'm moving it to xt/author.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2295 [1/4]: moving C4::Dates tests into database dependent tests
Andrew Moore [Mon, 7 Jul 2008 17:54:51 +0000 (12:54 -0500)]
bug 2295 [1/4]: moving C4::Dates tests into database dependent tests

the t/Dates.t tests were actually databasase dependent. This patch replaces
t/Dates.t with t/lib/KohaTest/Dates/Usage.pm that relies on a database.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBUGFIX : unimarc leader is 24, not 25. Fixing the javascript test
Paul POULAIN [Wed, 2 Jul 2008 17:48:01 +0000 (19:48 +0200)]
BUGFIX : unimarc leader is 24, not 25. Fixing the javascript test

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBug#: 1989
DeAndre Carroll [Thu, 3 Jul 2008 18:01:12 +0000 (13:01 -0500)]
Bug#: 1989

Changed date popups in scheduler.tmpl to a date picker element
Made corresponding changes to scheduler.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2275: making SMS::Send module optional
Andrew Moore [Thu, 3 Jul 2008 19:08:46 +0000 (14:08 -0500)]
bug 2275: making SMS::Send module optional

I wrapped the use of the SMS::Send module in an eval to make failures graceful if it
is not present.
I also fixed an error with the number of tests in the SMS::Send tests.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agotest suite: way to run single test file
Galen Charlton [Thu, 3 Jul 2008 18:34:03 +0000 (13:34 -0500)]
test suite: way to run single test file

Added support for a new target in the test Makefile
to run a single test module.  If you do (for example)

make test-single TEST_FILES=lib/KohaTest/Biblio.pm

only the tests in that module will be run.  Unlike
the full test suite as run via 'make test', 'make test-single'
does not clear the test database before running the
tests.

Please note that "TEST_FILES=path/to/test/class.pm" is
required when using 'make test-single'.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBarcodes - OO replacements, extensible module, tests.
Joe Atzberger [Thu, 3 Jul 2008 19:18:01 +0000 (14:18 -0500)]
Barcodes - OO replacements, extensible module, tests.

This code is intended to replace current value_builder code in 3.2, but
it does not affect it directly (yet) and is safe to include in 3.0.
This structure will be used to handle more complicated formats, like those
with checkdigits.  Please note that "incremental" format is still STRONGLY
recommended because it will always perform the best, and most flexibly.
The desire to include other information (like branchcode) should compel
the proper use of the barcode generator to print the info ON the barcode,
not IN the barcode.

One of the nicer features of this structure is that you are able to
create a new barcode (of the same type) based on any previous Barcodes object.
That means you can create an array of 51 consecutive barcodes like:
my $x = C4::Barcodes->new('annual'); # for example
my @set = ($x);
for (1..50) {
push @set, $x=$x->new;
}
Importantly, this can happen without referencing the database after the
first constructor.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2171: remove cancel transfer
Galen Charlton [Thu, 3 Jul 2008 21:27:13 +0000 (16:27 -0500)]
bug 2171: remove cancel transfer

The "transfers to receive" report included
unused code to delete a transfer.  Since the
"transfers to receive" report is still useful
to give a library a report of items that it should
expect to receive, I am retaining the report.  However,
since the cancel transfer functionality would not
work even if it were uncommented in the template, that
portion is removed.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2171: removed "Transfers to do" report
Galen Charlton [Thu, 3 Jul 2008 21:27:12 +0000 (16:27 -0500)]
bug 2171: removed "Transfers to do" report

This hold request and transfer report no longer
works now that hold requests are processed via
checking items in.

This patch removes the report; please consult
the koha-devel message re "RFC - remove circ/transferstodo.pl prior to general release"
for more details.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2305: install tags/* to intranet CGI directory
Galen Charlton [Thu, 3 Jul 2008 20:07:46 +0000 (15:07 -0500)]
bug 2305: install tags/* to intranet CGI directory

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2297: improve ModBiblio() to avoid duplicate item fields
Galen Charlton [Thu, 3 Jul 2008 19:40:59 +0000 (14:40 -0500)]
bug 2297: improve ModBiblio() to avoid duplicate item fields

Prior to this patch, ModBiblio() would append
item tags from the previous version of the bib record
to the incoming bib record before saving the results,
even if the incoming bib record already has embedded
item tags.

For example, if a bib is retrieved using GetMarcBiblio() then
saved using ModBiblio(), the caller was obliged
to delete any item tags first to avoid duplication.

ModBiblio() now deletes item tags supplied in the
incoming MARC record.  This eliminates the possibility
of duplication, and removes any implication that
ModBiblio() can or should be used to modify item
records - ModItem() should be used for that.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agocommenting warn
Paul POULAIN [Wed, 2 Jul 2008 17:48:00 +0000 (19:48 +0200)]
commenting warn

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agotest suite add_biblios: don't use zebraqueue_daemon
Galen Charlton [Tue, 1 Jul 2008 17:36:29 +0000 (12:36 -0500)]
test suite add_biblios: don't use zebraqueue_daemon

The add_biblios() routine now reindexes all bibs
in batch instead of waiting for zebraqueue_daemon - this
is moderately faster.  A separate set of test
cases for zebraqueue_deamon will be witten.

No documentation changes.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoSimpleSearch test cases - make more robust
Galen Charlton [Tue, 1 Jul 2008 17:36:28 +0000 (12:36 -0500)]
SimpleSearch test cases - make more robust

Test cases in KohaTest/Search/SimpleSearch.pm assumed
that they were the first to call add_biblios(); as
this is not necessarily true when the entire test
suite is run, test now counts how many 'Finn Test'
bibs already exist.

No documentation changes.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2293: better validation in overdue rules editor
Galen Charlton [Tue, 1 Jul 2008 16:53:23 +0000 (11:53 -0500)]
bug 2293: better validation in overdue rules editor

If a rule should not be saved because a delay is
given but no letter or debar action is specified
for that delay, an error message is now reported.

Also:

* use "dialog alert" CSS class for confirmation
  that form was saved
* if form input is saved, retrieve the settings from the
  database again when displaying the form - this will
  help turn up any bugs where the script is not
  saving the overdue rules correctly.
* fix display of patron category in error messages

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agooverdue rules - added JavaScript for branch selector
Galen Charlton [Tue, 1 Jul 2008 16:53:22 +0000 (11:53 -0500)]
overdue rules - added JavaScript for branch selector

This makes the overdue notice rule editor consistent
with the branch selector for the circulation rules
editor.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2292: save overdule rules to correct branch
Galen Charlton [Tue, 1 Jul 2008 16:53:21 +0000 (11:53 -0500)]
bug 2292: save overdule rules to correct branch

Save overdue rules to the selected branch, not always
to the default branch.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2291: sort branches and letter types
Galen Charlton [Tue, 1 Jul 2008 16:53:20 +0000 (11:53 -0500)]
bug 2291: sort branches and letter types

The drop-down lists of branches and letter templates
in the overdue notice/status triggers tool are now
sorted in ascending alphabetical order.

No documentation change.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agospelling correction in comment in javascript file
Andrew Moore [Mon, 30 Jun 2008 14:14:42 +0000 (09:14 -0500)]
spelling correction in comment in javascript file

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoOAI package handle correctly unicode content
Philippe Jaillon [Mon, 30 Jun 2008 14:07:36 +0000 (16:07 +0200)]
OAI package handle correctly unicode content

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2176: spelling correction
Andrew Moore [Mon, 30 Jun 2008 13:22:17 +0000 (08:22 -0500)]
bug 2176: spelling correction

correcting spelling in opac messaging tab.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoOPACUserCSS has wrong default value (fix for 2211, for french)
Paul POULAIN [Mon, 30 Jun 2008 13:11:28 +0000 (15:11 +0200)]
OPACUserCSS has wrong default value (fix for 2211, for french)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofrench updated
Paul POULAIN [Mon, 30 Jun 2008 13:11:27 +0000 (15:11 +0200)]
french updated

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2278 - fix results pagination in auth_finder
Galen Charlton [Thu, 26 Jun 2008 21:16:16 +0000 (16:16 -0500)]
bug 2278 - fix results pagination in auth_finder

Fix same as for 2205 - the orderby parameter is
currently required for authority searches.

Also set default results per page to 20 instead of
19.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoDocuments translator dependency on 'gettext' Linux tools and add it to debian.packages
Frederic Demians [Thu, 26 Jun 2008 05:03:28 +0000 (07:03 +0200)]
Documents translator dependency on 'gettext' Linux tools and add it to debian.packages

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoFix alternating "highlight" to use __odd__ and combine consequetive
Joe Atzberger [Thu, 26 Jun 2008 01:00:33 +0000 (20:00 -0500)]
Fix alternating "highlight" to use __odd__ and combine consequetive

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBugfix Correcting opac-shelves template to allow patron to set list category to publi...
Chris Nighswonger [Wed, 25 Jun 2008 23:34:51 +0000 (18:34 -0500)]
Bugfix Correcting opac-shelves template to allow patron to set list category to public or private

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBugfix correcting 'New List' behavior on opac-search.pl
Chris Nighswonger [Wed, 25 Jun 2008 23:34:50 +0000 (18:34 -0500)]
Bugfix correcting 'New List' behavior on opac-search.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoCorrecting mistyped var names and adding error trapping to C4::VirtualShelves::Page
Chris Nighswonger [Wed, 25 Jun 2008 23:34:49 +0000 (18:34 -0500)]
Correcting mistyped var names and adding error trapping to C4::VirtualShelves::Page

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoScript to correct faulty statistics table entries.
Joe Atzberger [Wed, 25 Jun 2008 23:21:28 +0000 (18:21 -0500)]
Script to correct faulty statistics table entries.

This is the first iteration of this script, likely to be expanded
if other mappings are determined.  Currently it populates the
itemtype for all statistics that have itemnumbers and then pulls
borrownumbers from old_issues (or issues).  This has the benign
side effect of ignoring whether or not itemtype was a different
value at the time of the transaction.

It then tries to populate null borrowernumbers using a fairly
conservative approach, only where exactly one match exists, else warns.

The process of updating statistics is needlessly complex based on the
lack of a reliable primary key.  ALL such tables should have a numerical
primary key.  The impression that the timestamp and any other fields
are enough to combine for a "unique enough" key is wrong.  For example,
running SIP tests will add multiple transactions that might share
borrowernumber, itemnumber and timestamp.  Key uniqueness should be
logically inherent, not de facto.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoadding 'gettext' as this is requred by the translation utility
Joshua Ferraro [Thu, 26 Jun 2008 10:33:47 +0000 (05:33 -0500)]
adding 'gettext' as this is requred by the translation utility

12 years agobug 2176: fix typo in column name (DB rev 096)
Galen Charlton [Wed, 25 Jun 2008 20:24:41 +0000 (15:24 -0500)]
bug 2176: fix typo in column name (DB rev 096)

In borrower_message_preferences, wants_digets => wants_digest

Note that the typo is present only in databases that had
borrower_message_preferences created via updatedatabase.pl.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug: 2269 - adding a perltidyrc file
Andrew Moore [Mon, 23 Jun 2008 17:55:31 +0000 (12:55 -0500)]
bug: 2269 - adding a perltidyrc file

This patch adds xt/perltidyrc, a suggested perltidy configuration file.

It's in the xt directory because I'd like to eventually tie it in with Test::Perl::Critic
to run automated tests on tidyness and other Perl::Critic policies. That stuff belongs in
the xt directory since it's testing for developers use only.

I have only turned on a few directives based on the suggestions by MJ, historical discussion on
the mailing lists, and what I was told when I started contributing to koha. There are many other
directives in this file that are commented out. They are supposed to represent the defaults and
can be adjusted in the future if desired.

Use of this file is encouraged, but not required. It would be really nice if you could at least run
it on new files that you contribute. I would recommend not running it on files that are already
part of koha without writing some tests for those files and discussing your intentions to reformat
entire files in bulk.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug: 2272 - remove warning from C4::Koha::getitemtypeimagedir
Andrew Moore [Mon, 23 Jun 2008 17:35:58 +0000 (12:35 -0500)]
bug: 2272 - remove warning from C4::Koha::getitemtypeimagedir

I changed getitemtypeimagedir to set a default on its argument so that it would not complain if not passed 'opac'.
I improved the documentation on the method.
I edited the t/icondirecotries.t test script to explicitly pass an argument to both getitemtypeimagedir calls.
 - and I adjusted one line of whitespace to make similar things look similar
I added a test module for C4::Koha
I added a test module for C4::Koha::getitemtypeimagedir.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofix Date::ICal typo
Galen Charlton [Wed, 25 Jun 2008 16:41:38 +0000 (11:41 -0500)]
fix Date::ICal typo

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoInteractive tests for SIP abstraction representation and Members.
Joe Atzberger (siptest [Tue, 24 Jun 2008 22:35:13 +0000 (17:35 -0500)]
Interactive tests for SIP abstraction representation and Members.

The test for members is useful to compare against the SIP (ILS)
representation.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoBugfix Circulation, Expand ILS::Patron mapping.
Joe Atzberger (siptest [Tue, 24 Jun 2008 22:35:12 +0000 (17:35 -0500)]
Bugfix Circulation, Expand ILS::Patron mapping.

ILS::Patron is where most of the intelligence for SIP's representation
is lodged.  Currently there is difficulty with C4::Members functions.
GetMemberDetails is required for the needed flags, but it returns
empty structure on bad barcodes, where it should be undef.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2254 [3/3]: add authority type check
Galen Charlton [Wed, 25 Jun 2008 16:30:03 +0000 (11:30 -0500)]
bug 2254 [3/3]: add authority type check

Added a test to the MARC framework checks to
verify that all authority types used in a framework
are defined.

Documentation changes: possible new screenshots
and text to describe the new test.

NOTE: This patch adds strings to the checkmarc template, and thus
      violates the string freeze for 3.0.  I'm submitting anyway
      because 2254 is a blocker, but if no exception is made,
      it is safe to not apply this patch for 3.0.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2254 [2/3]: fix authority types in MARC21 frameworks (DB rev 095)
Galen Charlton [Wed, 25 Jun 2008 16:30:02 +0000 (11:30 -0500)]
bug 2254 [2/3]: fix authority types in MARC21 frameworks (DB rev 095)

Corrected two errors found in the authority types used
in the MARC21 frameworks:

* one subfield that used "Meeting Name" instead of "MEETI_NAME"
* various subfields that used "CORP0_NAME" instead of "CORPO_NAME"

The biggest visible consequence of this change is that corporate
name headings (110, 610, 710, 810, etc.) will now link correctly
to their authority records.

In addition, brought the fr-FR MARC21 framework scripts up-to-date
with respect to the changes made to the English versions.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2254 [1/3]: fixed GetAuthType(); avoid crash
Galen Charlton [Wed, 25 Jun 2008 16:30:01 +0000 (11:30 -0500)]
bug 2254 [1/3]: fixed GetAuthType(); avoid crash

Improved C4::AuthoritiesMarc::GetAuthType() so that
it returns either a hashref (if the authority type exists)
or undef (if it does not exist).  The same
accessor should not be used to either return a single
value or all values of a settings list.  Note that
all existing clients of GetAuthType are expecting
either a single hashref or undef; none of them
expected the arrayref that could be returned by
the previous version of the accessor.

When BiblioAddsAuthorities is ON, addbiblio.pl
now checks the return value of GetAuthType and
no longer crashes as follows if the MARC framework
specifies an invalid authority type for a given subfield:

Can't coerce array into hash at .../cataloging/addbiblio.pl line 738.

No documentation changes.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2206: always have $9 for headings fields
Galen Charlton [Tue, 24 Jun 2008 22:12:23 +0000 (17:12 -0500)]
bug 2206: always have $9 for headings fields

If a field can be linked to an authority record (i.e.,
an authtypecode is defined for subfield $a), always
include the subfield $9 in a (readonly) input element even
if it is marked hidden per the MARC framework.

Prior to this patch, a $9 marked hidden per the framework
would not be included in the form, causing the authority record
number link to not be added when choosing a heading from the authority
finder.  In other words, with BiblioAddsAuthorities OFF,
any authorized headings in bibs added via the MARC editor
would not have the subfield $9, and thus would appear
to not be used by any bibs.

Note that subfield $9 is set to be readonly, as changing the
authority number link does not currently change the
heading stored in the bib record.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoclean up old-style calls to GetMemberDetails
Galen Charlton [Tue, 24 Jun 2008 15:21:12 +0000 (10:21 -0500)]
clean up old-style calls to GetMemberDetails

GetMemberDetails() returns only one hashref now,
not two.  In all cases where the caller was
expecting two output values, the $flags return
was ignored anyway.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoPOD fixes - C4/Members.pm
Galen Charlton [Tue, 24 Jun 2008 15:04:41 +0000 (10:04 -0500)]
POD fixes - C4/Members.pm

Corrected and improved the description of the
return values of GetMemberDetails.

Also fixed errors reported by the podcorrectness
test.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2252: item-level hold not waiting until priority = 0
Galen Charlton [Mon, 23 Jun 2008 20:33:32 +0000 (15:33 -0500)]
bug 2252: item-level hold not waiting until priority = 0

When returning an item that has an item-level hold on it,
CheckReserves() will not consider it waiting unless the
priority has been set to 0 (i.e., the item has specifically
been marked as filling the request.)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2273: not-for-loan item should not fill hold request
Galen Charlton [Mon, 23 Jun 2008 16:15:01 +0000 (11:15 -0500)]
bug 2273: not-for-loan item should not fill hold request

If an item is not for loan (i.e., items.notforloan is not null),
do not use it to fill a title-level hold request when
item is checked in.  Prior to this patch, a notforloan item
could fill a request - CheckReserves() was checking only
the item's item type's not for loan flag.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoNote optional modules, for later reference.
MJ Ray [Mon, 23 Jun 2008 15:33:50 +0000 (16:33 +0100)]
Note optional modules, for later reference.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoremoving maxItemsInSearchResults syspref from script
Joshua Ferraro [Tue, 24 Jun 2008 12:30:43 +0000 (07:30 -0500)]
removing maxItemsInSearchResults syspref from script

12 years agoslef's patch for 'Algorithm::CheckDigits' => 0.50,
Joshua Ferraro [Mon, 23 Jun 2008 15:35:55 +0000 (10:35 -0500)]
slef's patch for 'Algorithm::CheckDigits'          => 0.50,

12 years agonew automated test for template translatability
Galen Charlton [Mon, 23 Jun 2008 15:01:33 +0000 (10:01 -0500)]
new automated test for template translatability

This test verifies that the English OPAC and staff templates
can be processed by the string extractor (tmpl_process3.pl)
without error.  If a template contains a parsing error
(at least as far as tmpl_process3.pl is concerned), it may
not be correctly converted when a language translation
is applied.

To run this test, do

prove -v xt/author/translatable-templates.t

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoUpdating release number
Joshua Ferraro [Mon, 23 Jun 2008 13:05:49 +0000 (08:05 -0500)]
Updating release number

12 years agoUpdating translations prior to RC1 release
Joshua Ferraro [Mon, 23 Jun 2008 13:04:10 +0000 (08:04 -0500)]
Updating translations prior to RC1 release

12 years agoDB Bump 094 - bug 2268 -- allow mixed case subfield labels in MARC21 by changing...
Ryan Higgins [Sun, 22 Jun 2008 21:35:48 +0000 (16:35 -0500)]
DB Bump 094 - bug 2268 -- allow mixed case subfield labels in MARC21 by changing db column collation.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoCorrecting Release notes to read RC1 rather than Beta
Chris Nighswonger [Sat, 21 Jun 2008 11:02:01 +0000 (06:02 -0500)]
Correcting Release notes to read RC1 rather than Beta

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agotranslation fix
Joshua Ferraro [Mon, 23 Jun 2008 12:20:21 +0000 (07:20 -0500)]
translation fix

12 years agoFixed errors for translation
Joshua Ferraro [Mon, 23 Jun 2008 12:11:40 +0000 (07:11 -0500)]
Fixed errors for translation

12 years agoFixed errors for translation
Joshua Ferraro [Mon, 23 Jun 2008 12:11:06 +0000 (07:11 -0500)]
Fixed errors for translation

12 years agoUpdating all translations except for:
Joshua Ferraro [Sat, 21 Jun 2008 04:23:07 +0000 (23:23 -0500)]
Updating all translations except for:

el-GR-i-staff-t-prog-v-3000000.po
es-ES-i-staff-t-prog-v-3000000.po
hy-Armn-i-staff-prog-v-3000000.po
ru-RU-i-staff-t-prog-v-3000000.po
tr-TR-i-staff-t-prog-v-3000000.po
uk-UA-i-staff-t-prog-v-3000000.po
zh-Hans-CN-i-staff-t-prog-v-3000000.po

These will require extensive modifs to apply correctly due to
syntax problems.

12 years agoAdding release notes for 3.0 stable RC1
Joshua Ferraro [Sat, 21 Jun 2008 02:34:26 +0000 (21:34 -0500)]
Adding release notes for 3.0 stable RC1

12 years agoUpdates to translation files (pre-update.pl)
Joshua Ferraro [Sat, 21 Jun 2008 02:18:41 +0000 (21:18 -0500)]
Updates to translation files (pre-update.pl)

12 years agoworkaround for translation script
Joshua Ferraro [Sat, 21 Jun 2008 01:12:55 +0000 (20:12 -0500)]
workaround for translation script

12 years agoworkaround for translation script
Joshua Ferraro [Sat, 21 Jun 2008 01:02:12 +0000 (20:02 -0500)]
workaround for translation script

12 years agoworkaround for translation update script
Joshua Ferraro [Sat, 21 Jun 2008 00:59:54 +0000 (19:59 -0500)]
workaround for translation update script

12 years agoAdding Public note to default Normal view in opac-detail page.
Ryan Higgins [Fri, 20 Jun 2008 23:39:04 +0000 (18:39 -0500)]
Adding Public note to default Normal view in opac-detail page.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofix for 1913: Remove opaclargeimage system preference
Joshua Ferraro [Fri, 20 Jun 2008 23:38:10 +0000 (18:38 -0500)]
fix for 1913: Remove opaclargeimage system preference

12 years agobug 1553 - fix breadcrumbs on add by suggestion page.
Ryan Higgins [Fri, 20 Jun 2008 23:19:06 +0000 (18:19 -0500)]
bug 1553 - fix breadcrumbs on add by suggestion page.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agofix for bug 2255: Sys Prefs need to be moved from Local Use
Joshua Ferraro [Fri, 20 Jun 2008 23:18:39 +0000 (18:18 -0500)]
fix for bug 2255: Sys Prefs need to be moved from Local Use

12 years agobug 1839 - add default shipment date to today.
Ryan Higgins [Fri, 20 Jun 2008 23:04:00 +0000 (18:04 -0500)]
bug 1839 - add default shipment date to today.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoRemove input for shipping cost as per bug 1965.
Ryan Higgins [Fri, 20 Jun 2008 23:03:44 +0000 (18:03 -0500)]
Remove input for shipping cost as per bug 1965.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoTidying up the Lists button menu
Chris Nighswonger [Fri, 20 Jun 2008 22:57:27 +0000 (17:57 -0500)]
Tidying up the Lists button menu

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agobug 2002: show item's in transit status in OPAC holds list
Galen Charlton [Fri, 20 Jun 2008 22:53:20 +0000 (17:53 -0500)]
bug 2002: show item's in transit status in OPAC holds list

If item is in transit, show that status in the OPAC
user's hold list rather than "waiting to be pulled".

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoDATABASE UPDATE: change copynumber column to allow free text - [ bug 2223 ] - WARNING...
Ryan Higgins [Fri, 20 Jun 2008 22:48:38 +0000 (17:48 -0500)]
DATABASE UPDATE: change copynumber column to allow free text - [ bug 2223 ] - WARNING, will take some time for large db.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
12 years agoCorrecting "Your Lists" to display properly when no lists exist.
Chris Nighswonger [Fri, 20 Jun 2008 22:48:15 +0000 (17:48 -0500)]
Correcting "Your Lists" to display properly when no lists exist.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>