]> git.koha-community.org Git - koha.git/log
koha.git
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 years agoremoving maxItemsInSearchResults syspref from script
Joshua Ferraro [Tue, 24 Jun 2008 12:30:43 +0000 (07:30 -0500)]
removing maxItemsInSearchResults syspref from script

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

16 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>
16 years agoUpdating release number
Joshua Ferraro [Mon, 23 Jun 2008 13:05:49 +0000 (08:05 -0500)]
Updating release number

16 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

16 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>
16 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>
16 years agotranslation fix
Joshua Ferraro [Mon, 23 Jun 2008 12:20:21 +0000 (07:20 -0500)]
translation fix

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

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

16 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.

16 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

16 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)

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

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

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

16 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>
16 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

16 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>
16 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

16 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>
16 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>
16 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>
16 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>
16 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>
16 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>
16 years agoINSTALL.opensuse - updated URL for Koha 3 RC1 .tar.gz
Ricardo Dias Marques [Fri, 20 Jun 2008 23:45:33 +0000 (00:45 +0100)]
INSTALL.opensuse - updated URL for Koha 3 RC1 .tar.gz

Updated wget line in INSTALL.opensuse for downloading
the Koha 3 RC1 ".tar.gz" file

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug: 2176 improvements to database upgrade path
Andrew Moore [Fri, 20 Jun 2008 22:43:13 +0000 (17:43 -0500)]
bug: 2176 improvements to database upgrade path

This patch calls the new optional database SQL scripts to that sensible data is isntalled.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2248 [2/2]: import item status display in search results
Galen Charlton [Fri, 20 Jun 2008 22:37:05 +0000 (17:37 -0500)]
bug 2248 [2/2]: import item status display in search results

The in transit status now displays as such in the
OPAC search results.  In the non-XSLT version, the status
is checked only for bibs having 20 or fewer items to avoid
extra hits on the database during result set presentation.
In the XSLT version, all items are checked.

Note that because an item's transfer status is not
stored in the MARC record, the transfer status
has no effect when limiting a search by item
availability.  For a future version, the transit status
should be added to the Zebra index.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2248 [1/2]: improve item status display in OPAC detail
Galen Charlton [Fri, 20 Jun 2008 22:37:04 +0000 (17:37 -0500)]
bug 2248 [1/2]: improve item status display in OPAC detail

The following statuses are now displayed correctly in the
bib details page in the OPAC:

* In transit
* Damaged
* Not for loan

Prior to this patch, items of these statuses displayed
as available.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agofix for 2117: opacbookbag should have the term 'cart' in the description
Joshua Ferraro [Fri, 20 Jun 2008 22:45:07 +0000 (17:45 -0500)]
fix for 2117: opacbookbag should have the term 'cart' in the description

16 years agofix for 2118: virtualshelves should say 'list' in the description
Joshua Ferraro [Fri, 20 Jun 2008 22:44:02 +0000 (17:44 -0500)]
fix for 2118: virtualshelves should say 'list' in the description

16 years agofix to 2142: maxItemsInSearchResults No longer used
Joshua Ferraro [Fri, 20 Jun 2008 22:42:42 +0000 (17:42 -0500)]
fix to 2142: maxItemsInSearchResults No longer used

16 years agofix to bug 1914: OpacNav system preference description contains typo
Joshua Ferraro [Fri, 20 Jun 2008 22:29:18 +0000 (17:29 -0500)]
fix to bug 1914: OpacNav system preference description contains typo

16 years agofix for 1948: LibraryName has wrong description
Joshua Ferraro [Fri, 20 Jun 2008 22:28:10 +0000 (17:28 -0500)]
fix for 1948: LibraryName has wrong description

16 years agofix to 2211: OPACUserCSS has wrong default value
Joshua Ferraro [Fri, 20 Jun 2008 22:26:12 +0000 (17:26 -0500)]
fix to 2211: OPACUserCSS has wrong default value

16 years agofix to 2179: Language Issue: Should be Holds not Reserves
Joshua Ferraro [Fri, 20 Jun 2008 22:09:49 +0000 (17:09 -0500)]
fix to 2179: Language Issue: Should be Holds not Reserves

16 years agofix for bug 1961: sprintf value needed in C4/Circulation.pm
Joshua Ferraro [Fri, 20 Jun 2008 22:04:27 +0000 (17:04 -0500)]
fix for bug 1961: sprintf value needed in C4/Circulation.pm

16 years agopatch from cnighs, fixing warn in Context.pm
Joshua Ferraro [Fri, 20 Jun 2008 22:00:58 +0000 (17:00 -0500)]
patch from cnighs, fixing warn in Context.pm

16 years agobugfix for bug introduced by anonymous session code
Chris Nighswonger [Fri, 20 Jun 2008 21:44:52 +0000 (16:44 -0500)]
bugfix for bug introduced by anonymous session code

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2201 - autoBarcode code in acq is old. Modified to only work with 'increment...
Ryan Higgins [Fri, 20 Jun 2008 21:36:01 +0000 (16:36 -0500)]
bug 2201 - autoBarcode code in acq is old. Modified to only work with 'increment', and added FIXME

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agofix for bug 2045: Network Printers Setting Does Nothing?
Joshua Ferraro [Fri, 20 Jun 2008 21:41:14 +0000 (16:41 -0500)]
fix for bug 2045: Network Printers Setting Does Nothing?

16 years agofix for 2199 for new installations, existing installations
Joshua Ferraro [Fri, 20 Jun 2008 21:35:29 +0000 (16:35 -0500)]
fix for 2199 for new installations, existing installations
can just change it manually in sysprefs to YesNo

16 years agoRemove EXPRs from includes file.
Joe Atzberger (siptest [Fri, 20 Jun 2008 21:21:14 +0000 (16:21 -0500)]
Remove EXPRs from includes file.

This prevents the following log entries:
    error EXPR:at pos 42: non-initialized variable OPACViewOthersSuggestions
EXPR must NOT be used unless the author can guarantee ALL names referenced
will be defined at the BEGINNING of TMPL execution.  That suggests that in
.inc files, only variables populated by get_template... are valid, since
the author cannot guarantee what other templates might include the file.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoCreated INSTALL.opensuse file
Ricardo Dias Marques [Fri, 20 Jun 2008 21:56:21 +0000 (22:56 +0100)]
Created INSTALL.opensuse file

Created installation instructions for installing Koha 3 in openSUSE 11.0

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoAnother API call fix to get the vendor name.
Ryan Higgins [Fri, 20 Jun 2008 20:56:40 +0000 (15:56 -0500)]
Another API call fix to get the vendor name.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoUse C4::Debug to conditionalize warn.
Joe Atzberger [Fri, 20 Jun 2008 20:18:15 +0000 (15:18 -0500)]
Use C4::Debug to conditionalize warn.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoUse C4::Debug on warn statements.
Joe Atzberger [Fri, 20 Jun 2008 20:18:14 +0000 (15:18 -0500)]
Use C4::Debug on warn statements.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoScript is not an Exporter. Convert to using C4::Debug.
Joe Atzberger [Fri, 20 Jun 2008 19:36:33 +0000 (14:36 -0500)]
Script is not an Exporter. Convert to using C4::Debug.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoThere is no doc-head-calendar.inc. I think it's called calendar.inc and
Joe Atzberger [Fri, 20 Jun 2008 19:36:32 +0000 (14:36 -0500)]
There is no doc-head-calendar.inc. I think it's called calendar.inc and

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoReplace EXPR.
Joe Atzberger [Fri, 20 Jun 2008 19:36:31 +0000 (14:36 -0500)]
Replace EXPR.

Prevents log error:
EXPR:at pos 14: non-initialized variable notforloantext

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoRemove EXPRs. Also added all jacket images to Overdue tab.
Joe Atzberger [Fri, 20 Jun 2008 19:36:29 +0000 (14:36 -0500)]
Remove EXPRs. Also added all jacket images to Overdue tab.

Standardized table layout with colspan=2 for Title header,
the first being the icon or jacket image.  Discovered a bug
when I fixed the "toggle/highlight" code to work (using loop
context var __odd__), namely that the highlight styling overrides
the red overdue styling.  Commented out that conditional with a
FIXME.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoFix call to wrong API fcn resulting in missing Vendor information on two acq pages.
Ryan Higgins [Fri, 20 Jun 2008 20:19:15 +0000 (15:19 -0500)]
Fix call to wrong API fcn resulting in missing Vendor information on two acq pages.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug: 2176 - database fix
Andrew Moore [Fri, 20 Jun 2008 19:58:42 +0000 (14:58 -0500)]
bug: 2176 - database fix

I had the create table statements in the wrong order, so they would not apply cleanly.
This is an update to database version 091, and does not take a new number.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoFix broken link and quick formatting of currency.
Ryan Higgins [Fri, 20 Jun 2008 19:49:21 +0000 (14:49 -0500)]
Fix broken link and quick formatting of currency.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBugfix bad template variable, causing no data to display on bookfund.pl
Ryan Higgins [Fri, 20 Jun 2008 19:40:51 +0000 (14:40 -0500)]
Bugfix bad template variable, causing no data to display on bookfund.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBugfix 1417 : library was not being saved on order edits.
Ryan Higgins [Fri, 20 Jun 2008 19:21:49 +0000 (14:21 -0500)]
Bugfix 1417 : library was not being saved on order edits.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agofix for 2257: OPAC news does not display if more than one language is checked in...
Joshua Ferraro [Fri, 20 Jun 2008 19:30:31 +0000 (14:30 -0500)]
fix for 2257: OPAC news does not display if more than one language is checked in opaclanguages pref

16 years agobug 1710 and 1739: availability for item-level holds
Galen Charlton [Fri, 20 Jun 2008 18:59:33 +0000 (13:59 -0500)]
bug 1710 and 1739: availability for item-level holds

Fixed rules for determining whether an item is
available for an item-level hold request.  Create a
new function in C4::Reserves, IsAvailableForItemRequest(),
that checks whether an item is potentially available for
an item-level hold request.

An item is considered available if:

* it is not lost AND,
* it is not marked not for loan AND,
* it is not withdrawn AND,
* it is not damaged (unless the AllowHoldsOnDamagedItems syspref is ON), AND
* it is not on loan (unless the AllowOnShelfHolds syspref is ON)

Preventing a hold request on withdrawn items is bug 1739, as is
the new preference on whether to allow holds on damaged items.

Removing the condition that an item cannot be requested if
it has already been requested by another patron is the topic of bug 1710.

Note that this patch does not change the behavior where if
independent branches is on and the canreservefromotherbranches
syspref is off, a staff operator is prevented from placing
an item-level hold request on an item from a different branch.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 1710 and 1739: two new sysprefs (DB rev 092)
Galen Charlton [Fri, 20 Jun 2008 18:59:32 +0000 (13:59 -0500)]
bug 1710 and 1739: two new sysprefs (DB rev 092)

As part of fixing hold request bugs 1710 and 1739,
two new system preferences are defined:

AllowOnShelfHolds: YesNo, default value OFF
  * if ON, allow item-level hold requests to be
    placed for available items that are not on loan

AllowHoldsOnDamagedItems: YesNo, default value ON
  * if OFF, item-level hold requests cannot
    be placed on items that are marked damaged.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoFix for 2187: http_accept_language feature not working
Joshua Ferraro [Fri, 20 Jun 2008 19:09:12 +0000 (14:09 -0500)]
Fix for 2187: http_accept_language feature not working

16 years agobug 2176: adding new system preference to French sysprefs script.
Andrew Moore [Fri, 20 Jun 2008 18:13:44 +0000 (13:13 -0500)]
bug 2176: adding new system preference to French sysprefs script.

I forot to add the new EnhancedMessagingPreferences system preference to the French SQL. I
hope I added it in the right place. The 'explanation' is untranslated.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176: adding SMS::Send to list of dependencies
Andrew Moore [Fri, 20 Jun 2008 18:02:04 +0000 (13:02 -0500)]
Bug 2176: adding SMS::Send to list of dependencies

SMS::Send is used by the new enhanced messaging stuff. It's a pure-perl module that
provides a consistent interface to SMS sending gateways. There are alredy
a handful of driver modules for gateways on CPAN, and adding one for a new gateway
is possible.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoAdding an automated test for POD correctness.
Andrew Moore [Fri, 20 Jun 2008 18:02:03 +0000 (13:02 -0500)]
Adding an automated test for POD correctness.

This test script checks all perl files in the C4 directory for POD correctness. It is
completely optional, and as of yet, is not actually run when you run the test suite.
It's just a handy way to check our POD documentation, which should be correct before
we release code.

This script can be invoked with 'prove -v xt/author/podcorrectness.t'

Additionally, this is the first test in the 'xt' directory. Current perl testing practices
have reserved the 'xt' directory for tests that should be run by the code authors before
release. See:
http://perl-qa.hexten.net/wiki/index.php/Oslo_QA_Hackathon_2008_:Achievements#Testing_Best_Practices
for more information.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176 (5/5): adding staff interface to patron messaging preferences
Andrew Moore [Fri, 20 Jun 2008 18:02:02 +0000 (13:02 -0500)]
Bug 2176 (5/5): adding staff interface to patron messaging preferences

I added another tab to the "patron" area of the staff interface to let staff
edit patron messaging preferences.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176 (4/5): adding RSS feed for patron overdue alerts
Andrew Moore [Fri, 20 Jun 2008 18:02:01 +0000 (13:02 -0500)]
Bug 2176 (4/5): adding RSS feed for patron overdue alerts

opac-mymessages.pl and opac-mymessages.tmpl generate an RSS feed of a patron's messages from the message_queue.

Some more methods in C4::Letters to let us pluck out the right entries in the queue.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176 (3/5): adding methods to manage message_queue, new advance_notices.pl, new...
Andrew Moore [Fri, 20 Jun 2008 18:02:00 +0000 (13:02 -0500)]
Bug 2176 (3/5): adding methods to manage message_queue, new advance_notices.pl, new C4::SMS module

I've added methods to to C4::Letters to manage the database table
message_queue. This will let us keep track of messages sent
via email, sms, and rss to patrons. That way, we can show the history,
deal with failures, and reconstruct an RSS feed when needed.

misc/cronjobs/overduenotics.pl has been added. It prepares advance notices
and item due notices and stages messages to be sent in the message_queue
table.

C4::Overdues::Getoverdues now takes two optional arguments to tell it how
old of overdues to fetch.

Also, a C4::Circualtion::getUpcomingDueIssues method was added that
advance_notices.pl uses.

misc/cronjobs/process_message_queue.pl has been added. It sends the email
or SMS messages out of the message queue.

The C4::SMS module didn't work at all, and it has been rebuilt to use
an external perl module from CPAN, SMS::Send.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176 (2/5): adding patron interface to update messaging preferences
Andrew Moore [Fri, 20 Jun 2008 18:01:59 +0000 (13:01 -0500)]
Bug 2176 (2/5): adding patron interface to update messaging preferences

This patch allows patrons to update their messaging preferences. This
includes methods in C4::Members to manage patron messaging preferences.

added cgi script to allow patron to edit their messaging preferences

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agorefactor C4::Auth::get_template_and_user for readability
Andrew Moore [Fri, 20 Jun 2008 18:01:58 +0000 (13:01 -0500)]
refactor C4::Auth::get_template_and_user for readability

I'd like to add another template parameter to geta_template_and user, but found it diffiult due to unreadability.

I stanadardized the whitespace in a portion of get_template_and_user,
alphabetized the parameters, finding a duplicate in one spot,
and then extracted the common template parameters that are set the same
regardless of template type.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoBug 2176 (1/5): database update: adding message_queue table and tables to hold patron...
Andrew Moore [Fri, 20 Jun 2008 15:58:31 +0000 (10:58 -0500)]
Bug 2176 (1/5): database update: adding message_queue table and tables to hold patron messaging preferences

Updates to kohastructure.sql and updatedatabase.pl:
- add new message_queue table
- add message_transport_types table
- add message_attributes table
- add message_transports table
- add borrower_message_preferences table
- add borrower_message_transport_preferences table
- adding EnhancedMessagingPreferenes to sysprefs SQL.

Added column to borrowers table to hold SMS Alert Number.
Added some more sample notices (letters) that will be sent for patron alerts
added some sample SQL to configure messaging.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agokohabug 2238 This patch allows patrons to add books to public open/free lists whether...
Chris Nighswonger [Fri, 20 Jun 2008 05:36:10 +0000 (00:36 -0500)]
kohabug 2238 This patch allows patrons to add books to public open/free lists whether they are logged in or not.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agokohabug 1873 Fixes shelves.pl so that the date is displayed in the 'date added' colum...
Chris Nighswonger [Fri, 20 Jun 2008 02:08:16 +0000 (21:08 -0500)]
kohabug 1873 Fixes shelves.pl so that the date is displayed in the 'date added' column of the virtual shelf contents

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agokohabug 1875 Public lists/virtualshelves are displayed and viewable whether a patron...
Chris Nighswonger [Wed, 11 Jun 2008 12:10:02 +0000 (07:10 -0500)]
kohabug 1875 Public lists/virtualshelves are displayed and viewable whether a patron is logged in or not.

NOTE: This patch introduces code which generates an anonymous session when a patron first browses to OPAC.
This anonymous session contains a minimal amount of information including the results of a query to discover
all public lists/shevles. When the user logs in, the anonymous session is cleared and a new session created
for that user.

kohabug 1875 - fix error when editing a patron record

C4::Auth::checkauth was not distinguishing between a
'userid' input from an OPAC or staff login form and
a 'userid' input from (e.g.,) the patron editor.
Consequently, adding or editing a patron record would
result in Koha trying to log in as the new patron.

To resolve this, added a hidden input to all login
forms, 'koha_login_context', to explicitly signal
when a login is occurring.  The value of this input
can be 'opac', 'intranet', or 'sco' - the value is
not used at present, but may be of use later.

C4::Auth - added debug flag to two warns

kohabug 1875 - create anonymous sessions only for OPAC

No need to create an anonymous session for the intranet.

set yuipath correct for login pages

When preparing the template parameters for a login form,
C4::Auth was overriding the value of yuipath set
by C4::Output::gettemplate(), thus causing 404 errors
if the 'yuipath' syspref was set to 'local'.

kohabug 1875 - avoid warns viewing lists anonymously

During an anonymous OPAC session, the $loggedinuser variable
is not set.  As the undefined value causes warns in
C4::VirtualShelves::Page::shelfpage, for the purpose of the
shelfpage call the loggedinuser is set to -1, which should
not correspond to any real borrower number.

This is admittedly a hack to avoid digging through all
of C4::VirtualShelves to deal with lists viewed anonymously.

kohabug 1875 Refactoring of &ModShelf to avoid overwriting list owner needlessly

kohabug 1875 Avoid warning if can't find owner of shelf

Since virtualshelves.owner is not a true FK of borrowersnumber.number,
set ownername to '' if can't find the patron

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2265: fix problem when cloning MARC field
Galen Charlton [Fri, 20 Jun 2008 14:35:42 +0000 (09:35 -0500)]
bug 2265: fix problem when cloning MARC field

Correctly set indicators when cloning a field
in the MARC editor.  Prior this fix, a cloned
field would not be saved.  Bug introduced in
patch for bug 2207.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agoDarrel's patch to fix sorting by size of order
Joshua Ferraro [Fri, 20 Jun 2008 14:14:17 +0000 (09:14 -0500)]
Darrel's patch to fix sorting by size of order

16 years agosorting authority list
paul [Fri, 20 Jun 2008 10:34:46 +0000 (12:34 +0200)]
sorting authority list

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agovarious fixes for acquisition histsearch
paul [Fri, 20 Jun 2008 10:34:45 +0000 (12:34 +0200)]
various fixes for acquisition histsearch

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agojust doing a <= / >= instead of a < / > in history search.
paul [Fri, 20 Jun 2008 10:34:43 +0000 (12:34 +0200)]
just doing a <= / >= instead of a < / > in history search.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - remove base64 functions from C4::Koha
Galen Charlton [Fri, 20 Jun 2008 13:11:23 +0000 (08:11 -0500)]
bug 2000 - remove base64 functions from C4::Koha

With the removal of admin/finesrules.pl and admin/issuingrules.pl,
the functions str_to_base64() and base64_to_str() in C4::Koha
are no longer used.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - remove admin/issuingrules.pl
Galen Charlton [Fri, 20 Jun 2008 13:11:22 +0000 (08:11 -0500)]
bug 2000 - remove admin/issuingrules.pl

Patch to remove issuingrules.pl in favor of
using smart-rules.pl to manage loan and fine
rules.  Several reasons for this:

* issuingrules.pl's matrix could grow rather large
  if the library has a large number of item types
  and patron categories
* successfully entering rules via issuingrules.pl
  requires placing commas within input fields
* a sparse circulation policy matrix takes the
  same amount of screen space as one that uses
  rules for a lot of specific patron category/item type
  combinations.
* having two administrative interfaces to the same
  policy settings could be confusing.
* UI design of smart-rules.pl better lends itself
  to adding more policy setting attributes to the
  rules matrix.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - removed finesrules.pl
Galen Charlton [Fri, 20 Jun 2008 13:11:21 +0000 (08:11 -0500)]
bug 2000 - removed finesrules.pl

Removed the separate admin/finesrules.pl script
to set the fines policy matrix: since admin/finesrules.pl
and admin/issuingrules.pl both touch issuingrules.pl, creating
a specific fine rule could silently override a default issuing
rule and prevent items from being checked out.

Circulation policy matrix settings for fines are now
handled in admin/smart-rules.pl

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - changes to TooMany()
Galen Charlton [Fri, 20 Jun 2008 13:11:20 +0000 (08:11 -0500)]
bug 2000 - changes to TooMany()

The C4::Circulation::TooMany() function, which determines
if a patron is at the maximum loan limit, has been
changed to work as follows:

1. Checks the issuing rule that would apply to the
   prospective loan to see if a loan limit (maxissueqty)
   has been set.
2. Counts the number of loans that the patron has
   that would fall under that loan rule.

IMPORTANT: that means that if a specific loan rule
exists for the branch, patron category, and item type
in question, *only* that rule's maxissueqty will be
checked here - it will not go on to check whether
a less specific rule has a lower loan limit.

3. If adding one more loan would bring that count
   over the limit, returns a "too many" error.
4. If the loan hasn't exceeded a specific limit, checks
   whether a branch/patron category circ rule has
   specified a loan limit, regardless of item type.
   If the patron has already reached *that* limit, returns
   the "too many" error.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - add C4::Circulation::GetBranchBorrowerCircRule
Galen Charlton [Fri, 20 Jun 2008 13:11:19 +0000 (08:11 -0500)]
bug 2000 - add C4::Circulation::GetBranchBorrowerCircRule

This routine retrieves the branch/patron category circulation
rules for a given branch and patron category.  The return
value is a hashref containing the following key:

maxissueqty - maximum number of loans across all item types

This will first check for a specific branch and
category match from branch_borrower_circ_rules.

If no rule is found, it will then check default_branch_circ_rules
(same branch, default category).  If no rule is found,
it will then check default_borrower_circ_rules (default
branch, same category), then failing that, default_circ_rules
(default branch, default category).

If no rule has been found in the database, it will default to
the built in rule:

maxissueqty - undef

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000: addition changes to rules editor
Galen Charlton [Fri, 20 Jun 2008 13:11:18 +0000 (08:11 -0500)]
bug 2000: addition changes to rules editor

* Added ability to specify total loans allowed at a library
  for the default patron category.  If set, the default
  limit is applied if no rule for the specific library
  and patron category is set.
* Added ability to specify default total loans allowed
  for the default library; this is applied if no rule
  for the specify library is set.
* Form now indicates if the number of current checkouts
  allowed is unlimited.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - add total loan limit to alt. issuing rules
Galen Charlton [Fri, 20 Jun 2008 13:11:17 +0000 (08:11 -0500)]
bug 2000 - add total loan limit to alt. issuing rules

The alternate issuing rules editor can now allow
defining the maximum number of loans that a borrower
of a given category can take out per branch, regardless
of item type.

The form for entering this limit now appears below
the form for setting loan rules per patron category
and item type.  The form only appears if a specific
branch is chosen, not if the default branch is used.

Also, some terminology changes:

* "Amount Loanable" => "Current Checkouts Allowed"
* "Amount" => "Fine Amount"
* "Grace Period" => "Fine Grace Period"
* "Charging Interval" => "Fine Charging Interval"
* "Loan time" => "Loan Period"

Documentation change: new screenshots for the alternate
loan rules form.

squashme terminology

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
16 years agobug 2000 - fix order of issuingrule lookup
Galen Charlton [Fri, 20 Jun 2008 13:11:16 +0000 (08:11 -0500)]
bug 2000 - fix order of issuingrule lookup

Extended help on the alternate circulation rules
form to list the order of issuingrules lookup as
follows:

same library, same patron type, same item type</li>
same library, same patron type, default item type</li>
same library, default patron type, same item type</li>
same library, default patron type, default item type</li>
default library, same patron type, same item type</li>
default library, same patron type, default item type</li>
default library, default patron type, same item type</li>
default library, default patron type, default item type</li>

This includes modifying two routines in C4::Circulation to
follow this order: GetLoanLength() and GetIssuingRule().

The reason for this change is to have Koha exhaust all issuingrules
possibilities for a branch before checking the rules for
the default branch - this is consistent with what an admin
might expect from looking at the issuingrules forms, which display
settings a branch at a time, and is more consistent with how
circulation rules should work for indepdendent branches.

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