Garry Collum [Fri, 17 Jul 2009 23:54:10 +0000 (19:54 -0400)]
Bug 2553: Fixes location drop-down alphabetization in Serials Statistics wizard form.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Wed, 15 Jul 2009 21:30:31 +0000 (17:30 -0400)]
Bug 3418: Fixes the alpha order of the system preference tags.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Thu, 16 Jul 2009 16:45:29 +0000 (12:45 -0400)]
Bug 3408: Tag cloud says login when logged in - fix.
Changed the logic in the html form to have the login prompt only appear if a user is not logged in. Separated the hidemylist functionality from the "show tags from other users" option. Created a link for "hide my tags" or "show my tags" depending on context.
The user can now choose how many tags of other users to show, and can also choose whether to show their own tags or not.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Matthias Meusburger [Fri, 17 Jul 2009 15:23:50 +0000 (17:23 +0200)]
Bug 2593 : way to duplicate items
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Mon, 20 Jul 2009 20:37:24 +0000 (15:37 -0500)]
Cannot use the same TMPL_VAR to select ALL options.
Only one select can be selected in this case. If preselection of search type is
desired, I recommend a clean implementation w/ jquery instead of adding a conditional
for each select. Please note, that is the only good way to get multiple preselections,
since the same .inc file will be used in each case. So a single variable like
<TMPL_IF NAME="index_bc_selected">
cannot be used, since it would set for ALL (3) advanced search dropdowns.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Mon, 20 Jul 2009 22:47:42 +0000 (17:47 -0500)]
Remove bogus "selected" logic from OPAC adv. search.
The only reason the TMPL_VAR works where it does is because it is part
of the TMPL_LOOP. It doesn't mean anything outside the loops, since
*everything* would be selected with the same variable, indiscriminantly.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Fri, 17 Jul 2009 13:51:04 +0000 (08:51 -0500)]
Correct broken spacing in docs
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Fri, 17 Jul 2009 14:56:31 +0000 (09:56 -0500)]
Add crontab instructrions.
Too many people have been confused about crontab dependent functions
"not working" automatically on their installations. This is probably
because there was no explicit step telling them to schedule the jobs.
The maintainers of the other INSTALL.* docs would be well advised
to insert similar lines in their docs, after confirming proper operation,
since cron/crontab are very system dependent.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Frédéric Demians [Wed, 15 Jul 2009 06:35:08 +0000 (08:35 +0200)]
Bug 3226 - Extended characters inconsistantly displayed
When displayed on result page, some UNIMARC biblio
records are marc8 to utf8 converted for whatever
reason! It appears that iso2709 records in Zebra can't
always properly by used to construct MARC::Record.
This patch ask Zebra to return xml record rather than
iso2709 and use it to create MARC::Record (like
C4::GetMarcBiblio). i
Fix facet display.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Wed, 15 Jul 2009 09:01:27 +0000 (11:01 +0200)]
(bug #3398) fix unimarc 210c plugin
this add some args to SearchAuthority();
Fix the pagination, and fix javascript script.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Wed, 15 Jul 2009 03:35:16 +0000 (23:35 -0400)]
Revert "Bug 3226 - Extended characters inconsistantly displayed"
This reverts commit
1ddb5df863c3166a20d55c8f852af7c5fd67e204.
Nahuel ANGELINETTI [Wed, 8 Jul 2009 11:46:48 +0000 (13:46 +0200)]
(bug #3405) fix on loan list in search result
The hash keys were made using a base and $item->{date_due}, but this key doesn't exists, so the possibility to have duplicate keys exists, and sometimes the others items were not shown. This patch fix this bug.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Frédéric Demians [Sat, 4 Jul 2009 04:57:37 +0000 (06:57 +0200)]
Bug 3226 - Extended characters inconsistantly displayed
When displayed on result page, some UNIMARC biblio
records are marc8 to utf8 converted for whatever
reason! It appears that iso2709 records in Zebra can't
always properly by used to construct MARC::Record.
This patch ask Zebra to return xml record rather than
iso2709 and use it to create MARC::Record (like
C4::GetMarcBiblio).
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nicole Engard [Thu, 9 Jul 2009 12:21:04 +0000 (07:21 -0500)]
Fix BUG #3406 - move tag cloud above 'my tags' for ease of reading
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Sat, 27 Jun 2009 17:48:18 +0000 (13:48 -0400)]
Bug 3317: Adds author and added-author to email sent from basket (Cart).
Adds the author and added author fields to the email that is sent from the shopping cart. This makes the data sent via email consistent with the data that is viewed in the basket.
There is still a problem with author authority records other than 100 tags appearing correctly in the shopping cart.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Ryan Higgins [Thu, 2 Jul 2009 01:37:42 +0000 (20:37 -0500)]
Bug 2770 follwup : force AddRenewal obey circControl system preference [revised]
Also prevents infinite-length renewals by returning undef
if passed an invalid date instead of renewing with no due date.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Tue, 30 Jun 2009 19:56:28 +0000 (14:56 -0500)]
Changes to lists to improve behavior for users with no javascript
This patch moves the creation of some js-based function links to the script so that non-functioning links will not display when javascript is turned off. Other links which require TMPL processing are hidden with CSS and shown with JS.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Wed, 15 Jul 2009 02:00:36 +0000 (22:00 -0400)]
fix crash introduced in previous patch
In the previous patch,
- my @subf = $field->subfields;
- (defined @subf) or @subf = ();
is not equivalent to
+ my @subf = $field->subfields || ();
as, the results of $field->subields is interpeted in scalar
context, not list context, resutling in @subf containing
the number of subfields, not the subfield data itself, which
engenders the error
Can't use string ("1") as an ARRAY ref while "strict refs" in use
later on. Changing the operator to 'or' instead of '||'
fixes this.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Wed, 8 Jul 2009 21:52:18 +0000 (16:52 -0500)]
Branch auto-selection in additem
Adding a new item should pre-select the library you are logged in at, where possible.
Use GetBranchesLoop. This solves indy branches filtering, and sort order problems.
Set default off of ->{selected}, to establish pre-selected branch.
Fix flags check in onlymine sub, so superlibrarian + other flags still gets everything.
Removed defined(@array) check, since perl -wc warns it is deprecated.
Note new FIXME for botched date processing.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Tue, 30 Jun 2009 14:14:50 +0000 (09:14 -0500)]
LDAP overhaul
Allow replicate and update to be zero.
Break out logic into separate subs.
Do only one bind attempt depending on setting, instead of
necessarily failing first before trying auth_by_bind.
POD added for active directory and to document permutations of
behavior given different conditions. Fixed mistaken debug lines
that called "print STDERR printf ...", i.e. printed the line to output
and "1" to the error log. Added principal_name feature for generating
bind user from Koha userid.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Thu, 2 Jul 2009 03:47:35 +0000 (22:47 -0500)]
Bug 2539. Remove unintegrated "spsuggest" code from search scripts.
The intent of kohaspsuggest is very confused. It wanted to simultaneously
log every search query, and get spelling suggestions from a totally different
database_host:db_user:db_password:database. The implementation seems
never have really made it into 3.0, however, since the only trace of the
tables it wants to query are in misc/migration_tools/22_to_30/phrase_log.sql
The job misc/spellcheck_suggest/make_spellcheck_suggest.pl does not even
target the same remote DB as the syspref tells the search code to do. Instead, it hits the
local DB and, not finding the tables that have no way of existing in a 3.0
install, it CREATES its own tables spontaneously, using MyISAM storage, no less.
This script should be drastically overhauled, or more likely just removed.
Other fixes:
Commented out unconditional warn!
Also reduced bizarre whitespace amounts in catalogue/search.pl and
fixed warning as recommended by perl -wc:
Scalar value @newresults[0] better written as $newresults[0]
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Wed, 15 Jul 2009 01:14:18 +0000 (21:14 -0400)]
new icon included in OPAC details page rework
Picked up icon from earlier version of Owen's patch
for bug 2713.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Thu, 2 Jul 2009 03:12:29 +0000 (22:12 -0500)]
Cleanup opac-search.pl
Use GetBranchesLoop. Use elsif. Use $format w/ a fallback "" value.
More to come.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Thu, 25 Jun 2009 12:08:22 +0000 (14:08 +0200)]
(bug #3267) show items.uri in opac and intranet
This add a column in normal view, with a link to items.uri URL if it exists.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Joe Atzberger [Thu, 9 Jul 2009 18:30:36 +0000 (13:30 -0500)]
Search breadcrumb spacing
Also integrated both halves of the "searchdesc" conditional so they use the
same "No Result found!" message and <p> tags.
[LL 570]
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
John Beppu [Thu, 9 Jul 2009 01:09:00 +0000 (20:09 -0500)]
Minor Fixes for jQuery 1.3.2 Upgrade
I looked through all the relevant Javascript on this site
to find spots that would break in the upgrade from jQuery 1.2
to jQuery 1.3, and I fixed them.
Signed-off-by: John Beppu <john.beppu@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Thu, 9 Jul 2009 01:09:01 +0000 (20:09 -0500)]
More changes to implement jQuery upgrade.
- Upgrading jquery checkboxes plugin
- Correcting references to checkboxes plugin
- Upgrading jquery hotkeys plugin
- Upgrading syntax for calls to hotkeys plugin
- Removing jquery dimensions plugin (functionality rolled into jQuery)
Signed-off-by: John Beppu <john.beppu@liblime.com>
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
John Beppu [Wed, 8 Jul 2009 18:12:15 +0000 (13:12 -0500)]
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>
Owen Leonard [Mon, 29 Jun 2009 20:00:09 +0000 (15:00 -0500)]
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>
Garry Collum [Thu, 2 Jul 2009 15:41:11 +0000 (11:41 -0400)]
Bug 3384: Fixes invalid xhtml in advsearch.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Chris Nighswonger [Sat, 4 Jul 2009 04:14:50 +0000 (00:14 -0400)]
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>
Chris Nighswonger [Fri, 3 Jul 2009 16:59:26 +0000 (12:59 -0400)]
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>
Chris Cormack [Mon, 13 Jul 2009 04:43:57 +0000 (16:43 +1200)]
Whoops Ricardo was already in the list
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Chris Cormack [Mon, 13 Jul 2009 04:41:17 +0000 (16:41 +1200)]
Updating history doc
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Mon, 6 Jul 2009 12:58:48 +0000 (14:58 +0200)]
(bug #3395) fix unimarc plugin 225$a
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Sat, 4 Jul 2009 20:48:17 +0000 (16:48 -0400)]
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>
Garry Collum [Sun, 5 Jul 2009 00:18:15 +0000 (20:18 -0400)]
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>
Owen Leonard [Mon, 6 Jul 2009 16:01:20 +0000 (11:01 -0500)]
Correction to enable sending of update emails.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Mon, 6 Jul 2009 14:06:51 +0000 (09:06 -0500)]
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>
Ricardo Dias Marques [Fri, 3 Jul 2009 16:03:43 +0000 (16:03 +0000)]
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>
Darrell Ulm [Mon, 6 Jul 2009 20:57:22 +0000 (16:57 -0400)]
Bug#: 3358 File 3 of 3 patch for reserve notes (show)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Darrell Ulm [Mon, 6 Jul 2009 20:57:21 +0000 (16:57 -0400)]
Bug#: 3358 File 2 of 3 patch for reserve notes (show)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Darrell Ulm [Mon, 6 Jul 2009 20:57:20 +0000 (16:57 -0400)]
Bug#: 3358 File 1 of 3 patch for reserve notes (show)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Mon, 13 Jul 2009 03:06:35 +0000 (23:06 -0400)]
bug 3355 followup - fix XHTML errors
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Wed, 24 Jun 2009 09:00:48 +0000 (11:00 +0200)]
(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>
Nahuel ANGELINETTI [Fri, 26 Jun 2009 09:25:01 +0000 (11:25 +0200)]
(bug #3284) fix superlibrarian check
This fix the superlibrarian check in the previous fix.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Thu, 25 Jun 2009 10:57:35 +0000 (12:57 +0200)]
(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>
Nahuel ANGELINETTI [Thu, 25 Jun 2009 10:44:11 +0000 (12:44 +0200)]
(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>
Garry Collum [Wed, 24 Jun 2009 01:42:05 +0000 (21:42 -0400)]
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>
Garry Collum [Sat, 20 Jun 2009 21:00:53 +0000 (17:00 -0400)]
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>
Garry Collum [Sat, 20 Jun 2009 02:42:46 +0000 (22:42 -0400)]
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>
Garry Collum [Sat, 20 Jun 2009 00:58:15 +0000 (20:58 -0400)]
Bug 3346: Fixes invalid XHTML in opac-shelves.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Mon, 29 Jun 2009 16:17:49 +0000 (11:17 -0500)]
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>
Owen Leonard [Mon, 29 Jun 2009 18:23:22 +0000 (13:23 -0500)]
Tweaking CSS on z39.50 popup to improve readability (Bug 3269)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Mon, 29 Jun 2009 15:08:30 +0000 (10:08 -0500)]
Modifying the text of the error message for clarity and accuracy.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Thu, 25 Jun 2009 15:44:46 +0000 (10:44 -0500)]
Correcting validation errors.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Thu, 25 Jun 2009 00:23:39 +0000 (20:23 -0400)]
Bug 3360: Fix for unclosed <ol> element in export.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Thu, 25 Jun 2009 02:23:46 +0000 (22:23 -0400)]
Bug 3362: Fixes invalid xhtml in stage-marc-import.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Mon, 29 Jun 2009 13:43:04 +0000 (08:43 -0500)]
Misplaced </tbody> breaks table sorting. Fixes Bug 3365.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Thu, 25 Jun 2009 16:45:02 +0000 (11:45 -0500)]
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>
Owen Leonard [Thu, 25 Jun 2009 14:02:26 +0000 (09:02 -0500)]
Putting text form field first ensures field will correctly be given focus.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Thu, 25 Jun 2009 13:19:37 +0000 (15:19 +0200)]
(bug #3364) add itemnotes in itemslost report
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Fri, 26 Jun 2009 07:13:50 +0000 (09:13 +0200)]
(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>
Nahuel ANGELINETTI [Fri, 26 Jun 2009 11:24:14 +0000 (13:24 +0200)]
(bug #3369) add 995$* unimarc fields in "item" index
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Owen Leonard [Sat, 27 Jun 2009 15:43:17 +0000 (10:43 -0500)]
Spelling correction per convention: cataloguing -> cataloging (Bug 3277)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Fri, 26 Jun 2009 00:27:56 +0000 (20:27 -0400)]
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>
Garry Collum [Fri, 26 Jun 2009 00:35:14 +0000 (20:35 -0400)]
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>
Owen Leonard [Thu, 25 Jun 2009 13:28:42 +0000 (08:28 -0500)]
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>
J. David Bavousett [Thu, 25 Jun 2009 14:43:39 +0000 (10:43 -0400)]
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>
Liz Rea [Thu, 25 Jun 2009 14:46:31 +0000 (09:46 -0500)]
bug 3359 changed "registration branch" to "library" on moremember.tmpl for the sake of consistency.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Frédéric Demians [Sun, 14 Jun 2009 05:17:40 +0000 (07:17 +0200)]
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>
Joe Atzberger [Thu, 18 Jun 2009 16:31:33 +0000 (11:31 -0500)]
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>
Joe Atzberger [Sat, 20 Jun 2009 00:32:42 +0000 (19:32 -0500)]
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>
Joe Atzberger [Sat, 20 Jun 2009 00:32:44 +0000 (19:32 -0500)]
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>
Joe Atzberger [Sat, 20 Jun 2009 00:32:45 +0000 (19:32 -0500)]
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>
Joe Atzberger [Sat, 20 Jun 2009 00:32:43 +0000 (19:32 -0500)]
Remove unused variable and add FIXME.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Chris Nighswonger [Tue, 23 Jun 2009 17:18:04 +0000 (13:18 -0400)]
Adding installation documentation for Ubuntu
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Wed, 24 Jun 2009 14:09:27 +0000 (09:09 -0500)]
fixed table row highlighting CSS class
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Colin Campbell [Tue, 23 Jun 2009 08:14:31 +0000 (09:14 +0100)]
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>
Joe Atzberger (siptest [Sat, 20 Jun 2009 00:32:41 +0000 (19:32 -0500)]
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>
Jane Wagner [Fri, 19 Jun 2009 13:10:20 +0000 (09:10 -0400)]
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>
Joe Atzberger [Mon, 22 Jun 2009 19:40:56 +0000 (14:40 -0500)]
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>
Frédéric Demians [Tue, 23 Jun 2009 12:01:14 +0000 (14:01 +0200)]
Bug 3351 - Limit Serials Search for Biblio to an item type
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Liz Rea [Tue, 23 Jun 2009 18:11:24 +0000 (13:11 -0500)]
Adding links to guided reports wizard, dictionary, create from sql to reports home page.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Henri-Damien LAURENT [Fri, 19 Jun 2009 11:23:33 +0000 (13:23 +0200)]
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>
Joe Atzberger [Thu, 18 Jun 2009 16:31:32 +0000 (11:31 -0500)]
Correct POD and misleading variable names from copy/paste laziness
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Nahuel ANGELINETTI [Thu, 18 Jun 2009 13:43:29 +0000 (15:43 +0200)]
(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>
Garry Collum [Thu, 18 Jun 2009 09:18:26 +0000 (05:18 -0400)]
Bug 3338: Fix for missing closing brackets in authorities/detail.tmpl
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Galen Charlton [Wed, 17 Jun 2009 13:00:57 +0000 (08:00 -0500)]
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>
Galen Charlton [Wed, 17 Jun 2009 21:27:34 +0000 (16:27 -0500)]
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>
Ryan Higgins [Mon, 15 Jun 2009 22:20:29 +0000 (17:20 -0500)]
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>
Jane Wagner [Tue, 16 Jun 2009 16:28:06 +0000 (12:28 -0400)]
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>
Joe Atzberger (siptest [Tue, 16 Jun 2009 17:11:35 +0000 (12:11 -0500)]
Expand interactive SIP tests.
Also tweak interactive item dump to suppress huge fields (marc, marcxml).
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Wed, 17 Jun 2009 00:37:51 +0000 (20:37 -0400)]
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>
Nahuel ANGELINETTI [Wed, 17 Jun 2009 13:46:45 +0000 (15:46 +0200)]
(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>
Nahuel ANGELINETTI [Wed, 17 Jun 2009 12:39:02 +0000 (14:39 +0200)]
(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>
Garry Collum [Sun, 14 Jun 2009 21:43:37 +0000 (17:43 -0400)]
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>
Garry Collum [Sat, 13 Jun 2009 02:44:27 +0000 (22:44 -0400)]
Bug 3325: Fixes XHTML errors in opac-basket.tmpl.
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Garry Collum [Sat, 13 Jun 2009 14:51:45 +0000 (10:51 -0400)]
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>
Frédéric Demians [Sun, 14 Jun 2009 05:17:42 +0000 (07:17 +0200)]
Add a Debian Lenny package
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>