use C4::Debug instead of checking DEBUG env var directly

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

fixed typo in template

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

XHTML fixes for various circ templates

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

change references to 'reserves' in billing report template

New billing report has nothing to do with hold requests per se,
so change template variables in order to not confuse customizers.

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

various date-related cleanups in circ

* use Date::Calc to perform date arithmetic
* use Date::Calc::Today to consistently format the current date
* format date per syspref in overdue report

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

disallow division by zero in hold ratio report

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

another "Reserves" => "Hold" in template

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

minor circ template cleanup

[1] did some "Reserve" => "Hold" to keep language consistent
[2] Moved hold ratios report and billing report on circ
    home page to group with related reports

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

All patches to Koha 3 beta to date. All work with branches.

 Billing.pl     -- new, gives bills that occur in a range, works
 pendingreserves.pl  -- fixed, works now, with branches also
 reserveratios.pl  -- indicates distressed reserves
 itemslost.pl  -- Fix to this to make it more useful and fix bugs
 Itmes.pm -- small change to work for itemslost, should not affect
             anything else
 and all tmpl files.

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

Adding PATCRD print type

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

fixed locating OPAC stylesheet in a "standard" install

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

fixed MODS biblio export on a "standard" install

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

bug 1986: fix card view in Z39.50 search results

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

fixed memory leak I introduced

Accidentally introducing a circular reference in a
MARC::Record object does not lead to goodness, particularly
if you export lots and lots of them.

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

work around issue in MARC::Charset

Because of a bug in MARC::Charset 0.98, if a string to convert from
MARC-8 to UTF-8 has (a) one or more diacritics that (b) are only in character positions
128 to 255 inclusive, the resulting converted string is not in
UTF-8, but the legacy 8-bit encoding (e.g., ISO-8859-1).  As a result,
when such a record is converted to XML using ->as_xml_record(), the resulting
XML can be truncated at the offending character.  An example of such a record
is one that has a price in Briish pounds in the 260$c but no other diacritics.

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

Adding a couple of missing YUI js files; Adding comment preview to comments system (needs review).

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

Improving comments style and markup, adding highlighting for comments made by logged-in user.

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

Correcting a small case-sensitive bug.

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

for DebugLevel=2; improved reporting of version numbers

Koha software and DB version numbers now reported;
also improved formatting of multi-line error messages.

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

improved formatting of fatal error messages sent to browser

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

fixed invocation of CGI::Carp::set_message

The message handler in C4::Context is now correctly
passed CGI::Carp.

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

bug 1890: fixing calls to C4::Search::SimpleSearch

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

Bug 1980: adding parameters to C4::Search::SimpleSearch to allow limiting the number of results returned.

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

Adding file size limit to picture-upload.pl

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

Fixes to make all label/patron card templates uniform

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

Adding sample label form for Patron Cards

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

Catalogue => Search in staff home page

Changed to match similar change in the menu
bar.

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

avoid croak on latest CVS version of MARC::Record

The semantics of MARC::Record->delete_field() changed
slightly in the latest CVS version of MARC::Record.
delete_field() will now croak if passed an undef
scalar, while the version of MARC::Record from CPAN
accepts that input.  Since some packages of MARC::Record
in the wild appear to be based on the CVS version, not
the CPAN version, added a check where needed to avoid
the croak.

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

More work on patron card generation feature

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

Adding member search to patron card generation feature

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

Work on adding patron search to patron card generation feature.

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

Tweaking the interface of OPAC shelves to more closely match the pattern set by the staff client. Simplifies and focuses on specific tasks.

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

Notifying the user when they have selected no items and clicked 'Add to list.'

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

Visual tweaks for login screen.

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

Correction to prevent js error when table has no data.

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

Change LateOrders to inclusive <=, s/supplier/vendor , s/parcel/shipment.

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

still more rebuild_zebra refactoring

Merged duplicate code for indexing bibs and
authorities into a single index_records() function.

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

IMPORTANT: added -z option to rebuild_zebra.pl

The -z option, when used in conjunction with -a and/or -b,
selects the records to reindex from the zebraqueue table.
Both record updates and record deletes are handled.

-z is cannot be used with -s or -r: the updated records
must always be freshly exported, and if zebraqueue
is to be processed, it's assumed that you don't want
to drop the Zebra index first.

This means that rebuild_zebra.pl -b -a -x can be
used as a cronjob to update the indexes periodically; it
is believed that this will offer much better indexing
performance on some setups as compared to zebraqueue_daemon.pl,
which uses Z39.50 extended services to send record updates
to Zebra.

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

rebuild_zebra: exit if both -a and -x specified

At moment using both -a (index authorities) and
-x (export records as MARC XML) is not allowed -
if the Zebra authority database is using the DOM
filter, zebraidx will not be able to process the
exported records correctly.

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

more rebuild_zebra.pl refactoring

1. Logic to fix up record IDs, UNIMARC 100 field,
   and record leader now in separate functions.
2. Removed (incorrect) logic to save corrected record
   in database.

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

refactor rebuild_zebra: new routine for invoking zebraidx

Created a routine for calling zebraidx, replacing
separate invocations for bibs and authorities.

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

rebuild_zebra.pl: removed disused $limit option

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

Markup corrections. Tweaking holds interface.

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

Adding function to empty cart when user logs out.

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

Markup correction.

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

Removing a couple of unused templates

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

Removing old patronimages js check, adding missing patron image variable to script.

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

Reformatting holds list to better match checkouts list

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

Fixing bug that caused all items in checked-out list to show up as overdue; Reformatting table and tweaking select all/none behavior.

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

Fixing bug where language menu footer wouldn't "stick."

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

Beginning work on Calendar compatibility fixes

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

Send borrower mods to branchemail instead of syspref.

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

Add print slip to hold confirmation dialog.

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

Report borrower's home library on reserves library mismatch.

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

just some notes for RMs :-)

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

updating installation docs

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

updating release notes

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

updating translations for release

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

Two utilities to help manage translations

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

Adding some new translations

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

Updating translations for release

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

Experimental XSLT-based display of results and details pages To enable, create two new YesNo sysprefs: XSLTDetailsDisplay and XSLTResultsDisplay, and set them to ON. Feature to be expanded in a future version, this experimental code is for proof of concept only and doesn't introduce any changes if sysprefs are not created.

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

do not let MARC::Batch open MARC files

The version of MARC::Batch->new() distributed with version
2.0.0 of MARC::Record, if given a file name, will
open it using the ':utf8' layer.  This results in an
incorrect character conversion when processing records
in the MARC-8 character encoding.

To avoid this, batch jobs that use MARC::Batch now
open the file themselves, then pass the file handle
to MARC::Batch->new().

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

remove some unneeded use statements

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

Fixing branch select onChange() function, hopefully fixes Bug 1957, Holiday Calendar broken in Internet Explorer

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

partial fix for bug 1417

The Notes field should now be displayed correctly
when editing an order - the query in GetOrder
was modified to have aqorder's notes column
appear last, which means that it is the one
selected for the data hash (instead of biblio.notes
or biblioitems.notes).

This is an ugly fix - GetOrder needs to be
refactored so that it doesn't depend on
selecting all columns from several different
tables.

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

bug: 1953. cleaning up Acquisition::NewOrder just a little bit.

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

bug: 1953. using placeholders in C4::Acquisition::NewOrder.

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

bug 1953: using placeholders in C4::Acquisition::GetParcel

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

bug 1953: using placeholders in C4::Acquisition::GetParcel

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

Some reworking of suggestions management interface in preparation for tackling hdl's proposal of wider scope. Includes change to Suggestions.pm to enable passing of borrowernumber to the template.

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

bug fix : 1804 : Now empty authorities and empty search return results.

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

Bug fixing : 1746 : password was not escaped.

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

Some Bugfixing for Acquisitions : ModOrderReceive would create 2 lines in baskets And titles were displayed twice in parcel.tmpl

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

bug fixing : when closing a basket, supplier was searched by name, not by id.

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

don't display the name of the library in singleBranchMode

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

fixing singlebranchmode display issues

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

adding OPACShelfbrowse to OPAC tab

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

fixing availability limit

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

adding title indexes for 780

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

don't show library facet if singleBranchMode is ON

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

Adding a couple benchmark scripts

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

obey opaclanguagesdisplay syspref

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

bug 1953: use bind variables in C4::Acquisition::GetPendingOrders

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

use bind variables in C4::Acquisition::GetPendingOrders

I improved the tests a bit for this module so that they at least skip
if there's not enough data in the database to test with.
I was unable to test the actual execution path through the change I actually made.

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

Markup correction.

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

Fix to enable OPACUserCSS preference (allows user to add CSS globally in the OPAC via an embedded style block)

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

More changes to properly implement opacuserlogin preference.

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

Changes to properly implement opacuserlogin preference.

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

Changes to allow highlighting of current tab on user pages.

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

Updates to tweak the way suggestions and suggestions links display when various suggestions-related prefs are on or off.

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

Commenting out unused call to get_batches.

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

Modifications to viewlog.pl to keep it in the context from which it was called.

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

Correcting warn concerning kohaversion.

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

Updating patronimages syspref to reflect current sysprefs.sql

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

use bind variables in GetBorrowersWhoHaveNotBorrowedSince query

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

more old_issues SQL cleanup

* no need to test for 'returndate is null' in queries
  on old_issues
* added more stylistic corrections

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

Revert "Updatedatabase UPDATE For more_subfields_xml to be added to deleteditems at the same time as it is to items."

This reverts commit eaac37aa6ea8c4f2b79adaa4186bc08a9ac9efd4.

Rev 052 in updatedatabase.pl already adds more_subfields_xml
to deleteditems - the reverted patch would attempt to add
more_subfields_xml to deleteditems twice for anybody
upgrading from < 048.

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

More problems with old_issues.

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

Assigning bug 1835 : change password would never log password change.

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