Upgraded from jQuery 1.2.3 to 1.3.2

http://docs.jquery.com/Release:jQuery_1.3
http://www.learningjquery.com/2009/03/3-quick-steps-for-a-painless-upgrade-to-jquery-13

Signed-off-by: John Beppu <john.beppu@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>

Fix for Bug 2713: Opac detail content overlaps when right column extends too far

Resubmitted. My proposed fix eliminates the OpacNav column on this page. It puts the "search for this title in..." links into a drop-down menu, and it moves the "similar items" display into a tab along with Holdings, Reviews, etc.

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

Bug 3384: Fixes invalid xhtml in advsearch.tmpl

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

Bug 3390 Enabling ZIP compression in pdf generation to reduce file size

Enabling compression gives a ~26% *reduction* over non-compressed Type 3 font embedding.
ie. 4.5 K/pg (compressed graphic) vs. 17.5 K/pg (uncompressed Type 3 font) vs 111 K/pg
(uncompressed graphic).

It also appears that most other applications that export in pdf use compression by
default. (OO Writer, etc.) So this approach appears justified. One could always add code
to allow the user to select embedding mode and compression.

Also correcting mode parameter value.

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

Bug 3390 Barcodes not displayed in Acro Reader >v7.0

Label pdfs containing barcodes generated by Koha do not display barcodes
properly in Adobe Reader >v7.0.

This does not appear to be a bug in Koha code nor in PDF::Reuse::Barcode code,
but rather in the backward compatibility of Adobe Reader.

This patch changes the barcode embedding method from a Type3
font to graphical embedding which appears to work around the issue.

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

Whoops Ricardo was already in the list

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

Updating history doc

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

(bug #3395) fix unimarc plugin 225$a

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

Bug 2889: Cleaning up toggle variables in readingrec.pl and tmpl.

Removed toggle variable from readingrec.pl and readingrec.tmpl.  Used __odd__ variable instead.

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

Bug 3392: Fixes highlighting in Patrons with no Checkouts report.

Fixed class definition and moved the definition from the <td> tags to the <tr> tags.

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

Correction to enable sending of update emails.

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

Moving the text of an error message from the script to the template where it can be translated. Partial fix for bug 1153.

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

Fix for Bug 2599 - Search limits not working for NoZebra

Adds the change suggested by Amer Denni to fix Bug 2599
in comment #7 at:

http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2599

For UNIMARC installations, this patch only works after
updating the "NoZebraIndexes" system preference to
UNIMARC indexes (by default, NoZebraIndexes is being
wrongly set to MARC21 values in UNIMARC installations).

That has also been reported as a Bug, by me (see Bug 2178):

Bug 2178 - "NoZebraIndexes" system preference is being set
to MARC21 values in UNIMARC installations
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2178

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

Bug#: 3358 File 3 of 3 patch for reserve notes (show)

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

Bug#: 3358 File 2 of 3 patch for reserve notes (show)

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

Bug#: 3358 File 1 of 3 patch for reserve notes (show)

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

bug 3355 followup - fix XHTML errors

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

(bug #3355) add pagination in bib search for subscriptions

This add the support of pagination, delete a javascript call, and use now a get method(why post was used?).

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

(bug #3284) fix superlibrarian check

This fix the superlibrarian check in the previous fix.

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

(bug #3284) fix borrower deletion in independantbranches mode

This patch fix the checks deleting a borrower in independantbranches mode.
Now, we check the user and the librarian are from the same branch, else we deny the deletion(and disable the link to delete).

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

(bug #3363) do not reload parent if you add a biblio in shelf

this patch make the script not reloading the parent if the user add a biblio in a already existing shelf(useless).

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

Bug 3229: Fixes sort-by feature for jquery tablesorter in lists.

This patch adds some variables for the sortfield that are passed through the URL so that the sort-by feature works when the jquery tablesorter is used for a list of titles in opac-shelves.tmpl.

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

Bug 3347: Follow-up patch. Added logic for non-loggedin user.

Adds a login message, instead of a "New List" link, to the public list table, if a non-logged in user views the table.

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

Bug 3347: Inconsistencies with public and private list tables in opac-shelves.tmpl.

This patch fixes inconsistencies with the public and private list tables in opac-shelves.tmpl.  It adds a "new list" link to the public list view, and it adds a "sort by" column.

This patch also removes the toggle variables from the tables in opac-shelves.tmpl and Page.pm, and uses the template __odd__ variable for table highlighting if javascript is turned off.

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

Bug 3346: Fixes invalid XHTML in opac-shelves.tmpl

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

Removing hard-coded $ symbols from the template, addressing Bug 2547

Pending a system for displaying the correct currency symbol based on the system's default currency all dollar-specific symbols should be removed in favor of no symbol.

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

Tweaking CSS on z39.50 popup to improve readability (Bug 3269)

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

Modifying the text of the error message for clarity and accuracy.

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

Correcting validation errors.

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

Bug 3360: Fix for unclosed <ol> element in export.tmpl

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

Bug 3362: Fixes invalid xhtml in stage-marc-import.tmpl

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

Misplaced </tbody> breaks table sorting. Fixes Bug 3365.

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

Reformatting interface for Notice Triggers (Bug 3366)

Breaking up the table of options into three separate ones for first, second, and third notices, each in its own tab.

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

Putting text form field first ensures field will correctly be given focus.

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

(bug #3364) add itemnotes in itemslost report

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

(bug #3368) index 464$t in unimarc

this patch add 464$t part of Title index

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

(bug #3369) add 995$* unimarc fields in "item" index

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

Spelling correction per convention: cataloguing -> cataloging (Bug 3277)

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

Bug 2889: Cleaned up toggle variable in opac-readingrecord.tmpl. Disregard previous patch.

Cleaned up toggle variable in opac-readingrecord.pl and tmpl.  Used __odd__ variable instead.

Missed </TMPL_UNLESS> in previous patch.

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

Bug 2889: Removed toggle variable from z3950_search.pl.

Removed unused toggle variable from z3959_search.pl.  Added __odd__ variable to table in z3950_search.tmpl for non-javascript users.

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

Another fix for Bug 3119 (Add date due to circ history)

Moving links (non-table data) out of table to correct persistent table border display issue.

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

Modified fines.pl to not throw warning if --out not specified

It appears, in Ryan's patch, that he wants to throw a warning to the log if
the directory specified in --out is not present.  (Further messages will
be given when the open-or-die occurs a few lines later.)  However, it was
throwing the warning if --out was not specified at all, which is
undesirable.  This patch modifies that bit to check for the presence of
whatever directory is going to be used, either --out, ENV{TMPDIR}, or /tmp.
As before, if the write to the directory fails for any reason--including
its' non-existence--that is handled later, but this message will help
inform the troubleshooter.

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

bug 3359 changed "registration branch" to "library" on moremember.tmpl for the sake of consistency.

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

Bug 3301 - Speed up rebuild_zebra script

With this patch, rebuild_zebra can re-index a whole Koha DB
quickly:

  rebuild_zebra -r -b -nosanitize

Biblio (authority) records are dump directly in a file
from marcxml field without beeing transformed into
MARC::Record object and corrected.

DOCUMENTATION:

rebuild_zebra.pl new paramater:

-nosanitize  export biblio/authority records directly from DB marcxml
             field without sanitizing records. It speed up
             dump process but could fail if DB contains badly
             encoded records. Works now only with -x and -b

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

Returns reworking to handle empty GetItemIssue

Code cannot rely on issueinformation being populated.

Note there is room for better efficiency to have AddReturn also provide the
itemnumber (where existing) so that GetItemnumberFromBarcode is not called
at both levels.  Unfortunately there is discrepancy between this idea (for
efficiency) and the stated purpose of the $iteminformation object returned,
since $iteminformation is specifically the info from the issues table and
MUST be empty when the item was not in fact issued.

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

checkoverdues should not require $dbh

Passing $dbh around is an ancient style that doesn't know
to use C4::Context.  C4::Context->dbh is efficient, especially
for modules that already use Context, including almost all C4.

I also internalized $today into the SQL using NOW() in the query
and removed sth->finish.  Even though I dislike the return style
that gives the count, then the array, I left it becuase I don't
have time to fix/test all the callers.  However, I did convert
it so it doesn't require a $count variable and its own loop.

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

Return amount for CREDITS, POD rework.

The main change here is to get an {amount} field for CREDITS,
not just CHARGES.  I also use a fallback to default value (5)
in case noissuescharge has been deleted or set to 0.

POD is reworked to keep the salient chunks
about patronflags in the correct section and format them for
easier reference.  Deprecated display layer code.

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

Cleanup Members.pm - minor

MoveMemberToDeleted is problematic.  It has a bad name, since it
only copies, not moves.  Also the POD for it referenced a totally
different function name and was largely useless.  Sanity checks inserted.

A couple sth->finish removed.

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

Remove unused variable and add FIXME.

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

Adding installation documentation for Ubuntu

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

fixed table row highlighting CSS class

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

Enable warnings in addbooks.pl Fix code generating undef and mis-matched op warnings Remove toggle variable

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

Functional SIP run and shutdown scripts. Update example SIPconfig.xml

Note the shutdown script is quick and dirty, and probably
platform (debian) specific.

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

Bug # 2368 Change holds to pull default date range

Delivered range was 10 years ago to yesterday; changed default
to be two days ago to today.  Also removed notes field from report
output; 5xx fields can be quite large and including them in the
display resulted in a very lengthy printout for staff.

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

Bug 3350 - fail on queued emails w/ no address

If a patron has no email address, we need to avoid stockpiling all
their messages indefinitely.  Otherwise they get mailbombed when
their email IS added.

Note that overdues should not be affected, since the overdues job
checks whether the patron email exists before sending the message
(falling back to the admin).  The other messaging features are the
targets affected by this patch.

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

Bug 3351 - Limit Serials Search for Biblio to an item type

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

Adding links to guided reports wizard, dictionary, create from sql to reports home page.

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

Bug Fixing : 3334

MANIFEST.SKIP with grouping orig back and old
had a problem with hold-transfer-slip.pl

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

Correct POD and misleading variable names from copy/paste laziness

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

(bug #3339) fix rollover showed value

This patch just print the value of "whenmorethan1", that was missed.

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

Bug 3338: Fix for missing closing brackets in authorities/detail.tmpl

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

bug 2893: extend conditions handled by AllowNotForLoanOverride

Prior to this patch, turning AllowNotForLoanOverride on
allowed the circ operator to permit the loan of an
item who's individual not-for-loan flag was set, but did
not allow the loan of an item whose item type's not-for-loan
flag was set.  This patch extends the definition of the
system preference so that both cases are covered.

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

use Memoize to cache results of GetAuthorisedValues

Thanks to Frédéric Demians for the original impetus
to improve this function and for doing profiling.

Note that Memoize is a core module since at least 5.8.0.

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

Add MARC21 Fast-add Framework.

Development is in progress for brief records
in Koha; until that's done, this framework can
be used to add a minimal record.
Created by debra.denault@liblime.com

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

Revision to my earlier patch for displaying the user name on checked-out items

My earlier patch (using the new syspref OPACShowCheckoutName mistakenly
displayed the Koha borrower number instead of the patron barcode number
in the OPAC display; this revises the script/template to use barcode.

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

Expand interactive SIP tests.

Also tweak interactive item dump to suppress huge fields (marc, marcxml).

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

Bug 3317: Author displaying inconsistently or not at all. Revision to previous patch.

Revision to previous patch for bug 3317.  Removes the EXPR command from opac-basket.tmpl, which was polluting the apache logs.

NTS: Don't use EXPR.

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

(bug #2829) allow isbn13 in sql requests

this patch add the support of the plugin to search for isbn13.

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

(bug #2829) mistake un isbn field

I compare the bad part of the isbn in my first version, now fixed.

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

Bug 3317: Author displaying inconsistently or not at all in Cart.

This patch fixes the author and added author fields for the 'More Details' view of opac-basket.tmpl.

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

Bug 3325: Fixes XHTML errors in opac-basket.tmpl.

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

Bug 3228: Fix for Type: label in XSLT displays.

Empty nodes must be generated in the XML for MARC21slim2OPACResults - typeOf008 and MARC21slim2OPACResults - materialTypeCode when the fields in the Marc record do not exist.  This patch fixes the "Type: " label.  More testing needs to be done to see if any other labels appear in the Opac if the data is not defined in the MARC record.

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

Add a Debian Lenny package

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

Bug 2889: Removed toggle variable from member.pl and .tmpl

Removed toggle variable from member.pl and member.tmpl.  Replaced with template variable __odd__.

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

Add Getopt::Long to fines.pl, and option to specify log output directory.

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

(bug #3329) Dependency missing in inventory.pl

The called function AddReturn is in C4::Circulation which is not loaded, this patch do it.

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

Use strftime to format sessionlog times

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

Fix an unstranslatable string in JavaScript

For an explanation, see:

http://wiki.koha.org/doku.php?id=en:development:codingguidelines#translatable_text

This bug blocks entirely serials module. It isn't possible
to create/edit a subscription. Brocken JavaScript prevents
Search for a vendor and Search for Biblio dialog box
to pop-up.

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

Update references to batchRebuildBiblioTables

Also fixed escaping of literal "$" characters in usage.

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

Fix bug 3330 - Report notes modification

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

Cleanup stopwords.pl and .tmpl

Remove "toggle" code and useless javascript.
Rework "SimpleSearch" to use just one argument.
Remove unnecessary dependencies.
Remove incorrectly pasted comments.
Remove queries that didn't do anything.
Enable warnings.  Use one $dbh.  Remove $sth->finish calls.

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

Cleanup - moremember.pl and .tmpl

Removed last 3 EXPR statements.  Removed toggle code.
Fixed jquery for warning on image deletion.
Fixed permission check for image upload to be granular.
Lots of formatting/whitespace cleanup.
Removed useless form and unused template vars.

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

Adding a notes column, fixing a typo in TMPL_VAR statement that caused notes not to display.

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

Bug 2889: Removed toggle variable from auth_tag_structure.pl and auth_subfields_structure.pl

Removed toggle variable from auth_tag_structure.pl and .tmpl.  Used template __odd__ in auth_tag_structure.tmpl.

Removed toggle variable from auth_subfields_structure.pl and .tmpl. Use __odd__. Removed a type in the highlight class definition. Fixed a minor xhtml error.

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

followup - restore correct conditional

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

fixed broken link in patron label search

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

Missed Chris Nighswonger from the developer list, fixed

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

Bug 2889: Removed unused toggle variables from booksellers.pl. Added table highlighting to booksellers.tmpl.

Removed unused toggle and line(even) variables from booksellers.pl.  Added highlighting to the two display tables in booksellers.tmpl.

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

Cleanup: biblio_framework and marctagstructure

Removed unused javascript.  Enabled warnings.  Use common $dbh.
Removed sth->finish calls.  Removed toggle code.  Added row highlight where
none had been implemented.  Repaired validation error w/ mislocated input tag.

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

Cleanup classsources.pl and .tmpl

Relatively minor cleaning.
Remove unused javascript.  Add row highlighting to both tables.  Remove conditionals where the
behavior is the same for both possibilities.

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

Bug 2889: Removed toggle from marc_subfields_structure.pl and .tmpl

Removed toggle variable from marc_subfields_structure.pl and marc_subfields_structure.tmpl.  Used template __odd__ variable instead.

Corrected a typo in the highlight class definition in .tmpl.

Corrected a readability issue with the description when deleting a subfield in .tmpl.

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

Bug 2889: Removed toggle variable from stopwords.pl and .tmpl.

Removed toggle variable from stopwords.pl and stopwords.tmpl. Used template __odd_ variable instead.

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

Cleanup - admin scripts

Fixed useless redirect statements that weren't even printed.
Removed unused javascript and script variables;
Removed sth->finish.  Use get instead of post on "cancel" button (no data to post).
Reduce duplicative variables (e.g. scriptname and script_name).

Many other problems are still in the scripts, including use of META-REFRESH instead of
redirect, and the whole idea of redirecting back to the same page for no valid reason.
We should be able to formulate the right response on THIS pass, not ask the browser to start over.

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

Bug 2889: Removed unused toggle variable from pcard-member-search.pl. Added table highlighting to .tmpl.

Removed unused toggle variable form pcard-member-search.pl.

Added highlighting to pcard-members-search.tmpl and fixed some XHMTL errors.

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

Bug 2889: Removed toggle variable from aqbudget.tmpl. Used template variable __odd__ instead.

Removed toggle variable from aqbudget.pl and aqbudget.tmpl. Used template variable __odd__ instead.

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

Add +(" ")+ to make few strings translatable (with Paul & Galen fix)

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

Adding 10,000th commit to the history

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

Bug 2889: Added template __odd__ variable to letter.tmpl

Used __odd__ variable instead of toggle in letter.tmpl.  No toggle variable was defined in letter.pl.

The table in letter.tmpl uses jquery for highlighting so the above may not be necessary, unless javascript is turned off.

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

Bug 2889: Removed toggle variable from categorie.pl and categorie.tmpl.

Removed toggle variable from categorie.pl and categorie.tmpl.  Used template __odd__variable in categorie.tmpl.

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

Bug 2889: Removed toggle variable from aqbookfund.pl and aqbookfund.tmpl.

Removed toggle variable from aqbookfund.pl and aqbookfund.tmpl. Used template __odd__ variable in aqbookfund.tmpl.

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