commenting warn

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

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>

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>

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>

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>

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>

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>

spelling correction in comment in javascript file

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

OAI package handle correctly unicode content

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

bug 2176: spelling correction

correcting spelling in opac messaging tab.

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

OPACUserCSS has wrong default value (fix for 2211, for french)

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

french updated

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

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>

Documents translator dependency on 'gettext' Linux tools and add it to debian.packages

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

Fix alternating "highlight" to use __odd__ and combine consequetive

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

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>

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>

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>

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>

adding 'gettext' as this is requred by the translation utility

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>

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>

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>

fix Date::ICal typo

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

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>

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>

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>

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>

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>

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>

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>

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>

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>

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>

Note optional modules, for later reference.

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

removing maxItemsInSearchResults syspref from script

slef's patch for 'Algorithm::CheckDigits'          => 0.50,

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>

Updating release number

Updating translations prior to RC1 release

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>

Correcting Release notes to read RC1 rather than Beta

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

translation fix

Fixed errors for translation

Fixed errors for translation

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.

Adding release notes for 3.0 stable RC1

Updates to translation files (pre-update.pl)

workaround for translation script

workaround for translation script

workaround for translation update script

Adding Public note to default Normal view in opac-detail page.

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

fix for 1913: Remove opaclargeimage system preference

bug 1553 - fix breadcrumbs on add by suggestion page.

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

fix for bug 2255: Sys Prefs need to be moved from Local Use

bug 1839 - add default shipment date to today.

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

Remove input for shipping cost as per bug 1965.

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

Tidying up the Lists button menu

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

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>

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>

Correcting "Your Lists" to display properly when no lists exist.

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

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>

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>

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>

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>

fix for 2117: opacbookbag should have the term 'cart' in the description

fix for 2118: virtualshelves should say 'list' in the description

fix to 2142: maxItemsInSearchResults No longer used

fix to bug 1914: OpacNav system preference description contains typo

fix for 1948: LibraryName has wrong description

fix to 2211: OPACUserCSS has wrong default value

fix to 2179: Language Issue: Should be Holds not Reserves

fix for bug 1961: sprintf value needed in C4/Circulation.pm

patch from cnighs, fixing warn in Context.pm

bugfix for bug introduced by anonymous session code

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

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>

fix for bug 2045: Network Printers Setting Does Nothing?

fix for 2199 for new installations, existing installations
can just change it manually in sysprefs to YesNo

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>

Created INSTALL.opensuse file

Created installation instructions for installing Koha 3 in openSUSE 11.0

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

Another API call fix to get the vendor name.

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

Use C4::Debug to conditionalize warn.

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

Use C4::Debug on warn statements.

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

Script is not an Exporter. Convert to using C4::Debug.

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

There is no doc-head-calendar.inc. I think it's called calendar.inc and

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

Replace EXPR.

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

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

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>

Fix call to wrong API fcn resulting in missing Vendor information on two acq pages.

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

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>

Fix broken link and quick formatting of currency.

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

Bugfix bad template variable, causing no data to display on bookfund.pl

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

Bugfix 1417 : library was not being saved on order edits.

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

fix for 2257: OPAC news does not display if more than one language is checked in opaclanguages pref

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>

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>

Fix for 2187: http_accept_language feature not working

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>

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>

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>

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>