]> git.koha-community.org Git - koha.git/log
koha.git
15 years agoUnravelling links in masthead. Regularize logic and improve readability.
Joe Atzberger [Fri, 19 Sep 2008 21:24:22 +0000 (16:24 -0500)]
Unravelling links in masthead. Regularize logic and improve readability.

Link to Tag Cloud displayed on Adv. Search page even when TagsEnabled was OFF.
This was due to unnecessary duplicative code for Adv. Search vs. other pages.
Note: with this revision, the link to "Advanced Search" will display on all pages,
even on the Advanced Search page itself.  This is in fact desirable, rather than
the set of links collapsing and changing position depending on page being viewed.
If the old premise were to be followed (that Adv. Search link should not display on its
own page), then the same interface sensibility should apply to ALL links in the
masthead: i.e., Tag Cloud link should not display on its own page, same for Browse
by Subject, Most Popular, etc.  I would argue even this is undesirable.  At most,
give the links in moresearches some IDs and let each page unlink/restyle the
"current" link with jquery.  IMHO, masthead itself should not be engage in such trifling
interface polishes.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix for bug 2543: report output by tabulation a bit too literal
Jesse Weaver [Wed, 10 Sep 2008 00:31:31 +0000 (19:31 -0500)]
fix for bug 2543: report output by tabulation a bit too literal

This repurposes the Reports module for general reports, and moves its functionality to a different module, Reports::Guided. It also updates all of the reports that can export to correctly treat the 'tabulation' value of the 'delimiter' syspref.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup 5 to renewal limit override
Galen Charlton [Wed, 22 Oct 2008 23:11:31 +0000 (18:11 -0500)]
followup 5 to renewal limit override

With removal of content_hidden CSS class from
the circ templates, undoing change to staff-global.css

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup 4 to renewal limit override
Galen Charlton [Wed, 22 Oct 2008 23:11:30 +0000 (18:11 -0500)]
followup 4 to renewal limit override

* use style="display: none" instead of content_hidden
* fix JavaScript so that actions that check/uncheck
  items for a renew or return do not affect
  the override renewal limit checkbox

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup 3 to to renewal limit override
Galen Charlton [Wed, 22 Oct 2008 23:11:29 +0000 (18:11 -0500)]
followup 3 to to renewal limit override

style="display: none" suffices to have a span
be hidden initially; content_hidden is not necessary.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAllow renewal limits to be overridden
Michael Hafen [Wed, 22 Oct 2008 23:11:26 +0000 (18:11 -0500)]
Allow renewal limits to be overridden

Originally by Jesse Weaver <jesse.weaver@liblime.com>

This patch creates a new system preference, AllowRenewalLimitOverride,
that, if YES, allows the renewal limit to be manually overridden. It
updates C4::Circulation and reserve/renewscript.pl to obey this.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoSteamlining javascript for cloning additem block and tweaking style.
Owen Leonard [Thu, 16 Oct 2008 15:45:29 +0000 (10:45 -0500)]
Steamlining javascript for cloning additem block and tweaking style.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2692: remove escape="url" from opac-facets.inc
Galen Charlton [Wed, 22 Oct 2008 20:22:46 +0000 (15:22 -0500)]
bug 2692: remove escape="url" from opac-facets.inc

Based on Owen Leonard's patch, but redone to exclude
a dependency on an earlier unapplied patch.

For future enhancement, limit_cgi and the like should
probably not be constructed directly in C4::Search.
Instead, it should pass back a representation of the search
that the template can use to construct the appropriate URLs.
Done right, that would also make it easier to implement
things like search history.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTwo template bug-fixes: removing debugging output from opac-detail.tmpl, and turning...
Owen Leonard [Tue, 14 Oct 2008 18:50:47 +0000 (13:50 -0500)]
Two template bug-fixes: removing debugging output from opac-detail.tmpl, and turning off html-escaping in addbooks.tmpl to prevent incorrect HTML output. Need to figure out how to allow HTML tags in titles /and/ do search term highlighting.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup - use &amp; in place of &
Galen Charlton [Thu, 16 Oct 2008 20:45:41 +0000 (15:45 -0500)]
followup - use &amp; in place of &

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug in 'Add/Change link to item details...' I missed Previous checkouts area
Michael Hafen [Tue, 7 Oct 2008 17:30:27 +0000 (11:30 -0600)]
bug in 'Add/Change link to item details...' I missed Previous checkouts area

This turns the barcode into a link to moredetail.pl in the Previous
checkouts area of the circulation page, so it matches the Todays
checkouts area.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoSome changes to improve display on Internet Explorer.
Owen Leonard [Wed, 15 Oct 2008 14:05:11 +0000 (09:05 -0500)]
Some changes to improve display on Internet Explorer.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agobug 2677: restore RandomizeHoldsQueueWeight and StaticHoldsQueueWeight
Galen Charlton [Wed, 15 Oct 2008 13:52:34 +0000 (08:52 -0500)]
bug 2677: restore RandomizeHoldsQueueWeight and StaticHoldsQueueWeight

Restoring the functionality of these sysprefs as part of the
rework of build_holds_queue.pl:

* StaticHoldsQueueWeight is an ordered list of library codes.
  The order of the libraries in this list determines the order
  of libraries in which available items will be chosen to be
  targeted to holds.  Note that if this list does not include
  a library, items from that library will never be targeted to
  holds and staff will never be asked to retrieve items for
  title-level holds from those libraries.

* RandomizeHoldsQueueWeight is a Boolean parameter, indicating
  whether to randomize the libraries in the StaticHoldsQueueWeight
  list.

If neither RandomizeHoldsQueueWeight nor StaticHoldsQueueWeight is
set, then the libraries will be used to target items to holds in
alphabetical order by library code.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoRemove ambiguity from SQL statement for holds queue.
Ryan Higgins [Wed, 8 Oct 2008 05:31:24 +0000 (00:31 -0500)]
Remove ambiguity from SQL statement for holds queue.

holdingbranch table not referenced in view_holdsqueue.pl, causing irresolvable ambiguity in
sql query when limited by library (holdingbranch).  This was causing DBI errors to be thrown
in test database, and library-limited holdsqueue display was thus empty.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoResult of a cherry-pick, tidying up a comment
Chris Cormack [Fri, 4 Sep 2009 01:28:05 +0000 (13:28 +1200)]
Result of a cherry-pick, tidying up a comment

15 years agofollowup - make barcode_to_borrowernumber() work as named
Galen Charlton [Tue, 14 Oct 2008 18:10:28 +0000 (13:10 -0500)]
followup - make barcode_to_borrowernumber() work as named

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoCorrections to manage marc import interface, including fix for inaccurate progress...
Owen Leonard [Thu, 2 Oct 2008 16:23:49 +0000 (11:23 -0500)]
Corrections to manage marc import interface, including fix for inaccurate progress bar.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFixing up mess after conflict
Chris Cormack [Fri, 4 Sep 2009 01:18:58 +0000 (13:18 +1200)]
Fixing up mess after conflict

15 years agobug: query squashes issues.timestamp which circulation.pl needs to sort todaysissues
Michael Hafen [Wed, 10 Sep 2008 19:12:56 +0000 (13:12 -0600)]
bug: query squashes issues.timestamp which circulation.pl needs to sort todaysissues

Little change to the query so we get the right timestamp to sort on.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoAdding progress bar and some other visual enhancements to file upload interactions.
Owen Leonard [Tue, 30 Sep 2008 00:00:20 +0000 (19:00 -0500)]
Adding progress bar and some other visual enhancements to file upload interactions.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoMore fixes for Bug 2600, enabling the appearance of HTML tags in item titles. Resubmi...
Owen Leonard [Wed, 17 Sep 2008 15:03:34 +0000 (10:03 -0500)]
More fixes for Bug 2600, enabling the appearance of HTML tags in item titles. Resubmission.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2615: remove unneeded 'require Exporter'
Chris Cormack [Fri, 4 Sep 2009 00:52:34 +0000 (12:52 +1200)]
bug 2615: remove unneeded 'require Exporter'

Most Perl scripts (as opposed to modules) do
not need to require Exporter.

No user-visible or documentation changes.

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

catalogue/detail.pl
cataloguing/value_builder/barcode.pl
cataloguing/value_builder/dateaccessioned.pl
cataloguing/value_builder/marc21_field_003.pl
cataloguing/value_builder/marc21_field_005.pl
cataloguing/value_builder/marc21_field_040c.pl
cataloguing/value_builder/marc21_field_040d.pl
opac/opac-authorities-home.pl
opac/opac-detail.pl
opac/opac-main.pl
opac/opac-showreviews.pl
opac/opac-suggestions.pl
opac/sco/sco-main.pl
tools/export.pl

15 years agoBug 2365 : Inner counter not properly set for serials subscriptions not starting...
Ryan Higgins [Tue, 16 Sep 2008 11:05:33 +0000 (06:05 -0500)]
Bug 2365 : Inner counter not properly set for serials subscriptions not starting on first issue.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoAltering system log search form to conform with other similar forms; escaping output...
Owen Leonard [Sat, 13 Sep 2008 15:50:11 +0000 (10:50 -0500)]
Altering system log search form to conform with other similar forms; escaping output of "info" variable.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2596: remove URL escaping added by previous patch
Galen Charlton [Wed, 17 Sep 2008 02:58:13 +0000 (21:58 -0500)]
bug 2596: remove URL escaping added by previous patch

If URL in an href comes from a MARC record, it should not
be URL-escaped - otherwise, link is interpreted as relative
to MARCdetail.pl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFix for Bug 2596, "restore ability to hyperlink URL fields in MARC detail display"
Owen Leonard [Wed, 10 Sep 2008 13:11:07 +0000 (08:11 -0500)]
Fix for Bug 2596, "restore ability to hyperlink URL fields in MARC detail display"

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoCan't escape title variable as HTML because of search term highlighting.
Owen Leonard [Tue, 9 Sep 2008 19:54:04 +0000 (14:54 -0500)]
Can't escape title variable as HTML because of search term highlighting.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoTrying again: Standardizing styling of error messages; other markup corrections.
Owen Leonard [Tue, 9 Sep 2008 19:53:49 +0000 (14:53 -0500)]
Trying again: Standardizing styling of error messages; other markup corrections.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoBugs 2541 and 2587 - AddIssue must return date object as intended.
Joe Atzberger (siptest [Thu, 11 Sep 2008 03:30:04 +0000 (22:30 -0500)]
Bugs 2541 and 2587 - AddIssue must return date object as intended.

SIP actually relied on the AddIssue return that was not reliable.
AddRenew also updated to return C4::Dates object for datedue.

Please note, any running SIPServer will have to be restarted
*immediately* after applying this patch, because although Koha
C4 behaves as normal, the SIP server runs as a Net::Server with
components cached.  Changes will not be applied until SIPServer
restarts, and so checkout actions may fail until then.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoRemove double conditional, fix one FIXME.
Joe Atzberger [Mon, 8 Sep 2008 17:49:40 +0000 (12:49 -0500)]
Remove double conditional, fix one FIXME.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoA few more en-US spelling corrections; Markup corrections; Standardizing on "email...
Owen Leonard [Tue, 9 Sep 2008 15:26:40 +0000 (10:26 -0500)]
A few more en-US spelling corrections; Markup corrections; Standardizing on "email" instead of "e-mail"; Copying isbn-formatting from opac-detail.pl to detail.pl;

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agobug fix 2590 :
Henri-Damien LAURENT [Mon, 8 Sep 2008 15:45:07 +0000 (17:45 +0200)]
bug fix 2590 :

seems that when itemtypeimagelocation returns undef, the hash is messed up.
put it at the end of the hash and everything is fine.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agobugfix: need to return something if an itemtype doesn't have an image
Michael Hafen [Thu, 28 Aug 2008 20:58:21 +0000 (14:58 -0600)]
bugfix: need to return something if an itemtype doesn't have an image

I noticed that on the search pages if an itemtype doesn't have an image it's information doesn't make it out the the template.  I don't understand why, but this fixes it.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2582 [2/2]: fix NULL issues.issuedate
Galen Charlton [Thu, 4 Sep 2008 17:23:47 +0000 (12:23 -0500)]
bug 2582 [2/2]: fix NULL issues.issuedate

Corrects loans whose issuedate was set to NULL
during a renewal as follows:

* First, tries to extract the loan date from
  the statistics table.
* For any remaining rows,  sets the issue date
  to the date of the last renewal.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agobug 2582 [1/2]: don't set issuedate during renewal
Galen Charlton [Thu, 4 Sep 2008 17:23:46 +0000 (12:23 -0500)]
bug 2582 [1/2]: don't set issuedate during renewal

Changed so that issues.issuedate is never modified
during a renewal, since that column records the original
date of the loan.  Changed the name and interpretation
of the $issuedate parameter of AddRenewal() to
$lastreneweddate, allowing (e.g.) offline circulation
to set the date of the renewal without changing the
issue date.

As a result of the original bug, issues.issuedate can be
set to NULL for loans that were renewed via the OPAC,
self checkout, or the staff interface when explicitly
renewing a loan.  Loans that were renewed by checking
the item out to the same patron will have the issue date
changed to the date of the last renewal.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFixing markup errors, tweaking show/hide javascript.
Owen Leonard [Fri, 29 Aug 2008 14:06:03 +0000 (09:06 -0500)]
Fixing markup errors, tweaking show/hide javascript.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2569: added the zipcode to display along with the patron address
Danny Bouman [Thu, 28 Aug 2008 19:22:39 +0000 (15:22 -0400)]
bug 2569: added the zipcode to display along with the patron address

Added the zipcode to display along with the patron address when you are viewing a Patron from the intranet.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agofixing permission
Galen Charlton [Tue, 2 Sep 2008 21:34:41 +0000 (16:34 -0500)]
fixing permission

15 years agoAdd/Change link to item details (catalogue/moredetail.pl) for circulation and patron...
Michael Hafen [Fri, 22 Aug 2008 23:24:43 +0000 (17:24 -0600)]
Add/Change link to item details (catalogue/moredetail.pl) for circulation and patron details

This adds the link to item details to circulation.pl
and changes the link on member details to include the itemnumber and #item... in the link.

Forgot the returns.pl.  Will do that in the next patch.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFix for Bug 2557, "Guarantor information - error messaee when clicking borrower#...
Owen Leonard [Tue, 26 Aug 2008 19:06:02 +0000 (14:06 -0500)]
Fix for Bug 2557, "Guarantor information - error messaee when clicking borrower# link." Also correcting a couple of minor markup errors.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFixing tablesorter, display of renew links
Owen Leonard [Mon, 25 Aug 2008 16:14:36 +0000 (11:14 -0500)]
Fixing tablesorter, display of renew links

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoSpelling corrections: organisation->organization, authorise->authorize, catalogue...
Owen Leonard [Mon, 25 Aug 2008 18:38:56 +0000 (13:38 -0500)]
Spelling corrections: organisation->organization, authorise->authorize, catalogue->catalog

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoIf itemnumber is sent to catalogue/moredetail.pl use it
Michael Hafen [Fri, 29 Aug 2008 19:36:51 +0000 (13:36 -0600)]
If itemnumber is sent to catalogue/moredetail.pl use it

If itemnumber is part of the url for catalogue/moredetail.pl
put it to use.
This happens in members/boraccount.pl if a fine is attached to a
barcode.

This patch makes moredetail.pl show only the item specified by
itemnumber.

Implements some changes to the patch suggested on
patches@koha.org
give visual cue on UI and a link out.
More friendly pruning of the other items.

Thanks to Galen Charlton and Joe Atzberger

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoRevert "If itemnumber is sent to catalogue/moredetail.pl use it"
Galen Charlton [Tue, 2 Sep 2008 20:36:02 +0000 (15:36 -0500)]
Revert "If itemnumber is sent to catalogue/moredetail.pl use it"

Applied wrong version of patch.  Oops.

This reverts commit 9eb1465b2874a8a65195065bed083d26614ca287.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFines fixes: apparent problems with fines prevent processing.
Joe Atzberger [Tue, 26 Aug 2008 03:57:01 +0000 (22:57 -0500)]
Fines fixes: apparent problems with fines prevent processing.

CalcFine returned values that mismatched expectations in fines.pl.

fines.pl refactored: added debugging, prevent needless recreation of
Calendar objects by storing them in hash by branch.
Still outstanding problems with fines, including the output of a field
that has no other references in Koha (so is always undef) and the
incorrect description of FinesMode.

Calendar exported "new" erroneously.  I also cleaned up the queries to
avoid needlessly compiling additional statement handles.

Please test and consider application to 3.0 maintenance.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoIf itemnumber is sent to catalogue/moredetail.pl use it
Michael Hafen [Thu, 21 Aug 2008 21:55:27 +0000 (15:55 -0600)]
If itemnumber is sent to catalogue/moredetail.pl use it

If itemnumber is part of the url for catalogue/moredetail.pl put it to use.
This happens in members/boraccount.pl if a fine is attached to a barcode.

This patch makes moredetail.pl show only the item specified by itemnumber.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoMinor SIP cleanup, conditionalize warnings.
Joe Atzberger [Wed, 27 Aug 2008 14:44:28 +0000 (09:44 -0500)]
Minor SIP cleanup, conditionalize warnings.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agofix number two for bug 1204: Lists of issues and overdues on opac-user.pl
Jesse Weaver [Mon, 25 Aug 2008 16:51:25 +0000 (11:51 -0500)]
fix number two for bug 1204: Lists of issues and overdues on opac-user.pl

This makes opac-user.pl sort the output from GetPendingIssues itself, by descend
ing due date.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFix for Bug 2424: Altering layout of advanced search screen to make it compatible...
Owen Leonard [Wed, 30 Jul 2008 13:24:20 +0000 (08:24 -0500)]
Fix for Bug 2424: Altering layout of advanced search screen to make it compatible with Internet Explorer 6. Documentarians: this will require an updated screenshot.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agobug 2545: fix XSLTDetailsDisplay
Galen Charlton [Fri, 22 Aug 2008 00:59:45 +0000 (19:59 -0500)]
bug 2545: fix XSLTDetailsDisplay

Corrected so that when XSLTDetailsDisplay is ON but
XSLTResultsDisplay is OFF, the portion of the OPAC bib
details page that's derived from the MARC bib record
is no longer missing.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoFix itemtype image problem in moremember.pl
Jesse Weaver [Fri, 22 Aug 2008 15:54:31 +0000 (10:54 -0500)]
Fix itemtype image problem in moremember.pl

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2509: fix file permissions
Galen Charlton [Thu, 21 Aug 2008 21:39:45 +0000 (16:39 -0500)]
bug 2509: fix file permissions

Following suggestion by Vincent Danjean for Debian
packaging, 0755 -> 0644 for non-executable
files.

Also removed shebang from a few modules in C4.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2540: allow download of report
Galen Charlton [Thu, 21 Aug 2008 15:48:31 +0000 (10:48 -0500)]
bug 2540: allow download of report

$error return C4::Reports::execute_query is
always defined.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agofix for bug 1771: Template errors with remote itemtype image
Jesse Weaver [Wed, 20 Aug 2008 18:21:50 +0000 (13:21 -0500)]
fix for bug 1771: Template errors with remote itemtype image

This adds a new function, getitemtypeimagelocation, that returns the image
unmodified for absolute urls and returns the proper intranet or opac path
otherwise. It also updates all of the relevant files to use that function.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFix for minor error in shelfbrowser queries
Jesse Weaver [Tue, 19 Aug 2008 22:11:21 +0000 (17:11 -0500)]
Fix for minor error in shelfbrowser queries

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofix for bug 2477: Improve Speed of the Shelf Browser
Jesse Weaver [Tue, 19 Aug 2008 19:01:23 +0000 (14:01 -0500)]
fix for bug 2477: Improve Speed of the Shelf Browser

This patch updates the queries for the shelf browser in opac-detail.pl for a
slight performance gain. It also adds a param for dateformat to C4::Auth to
fix a warning.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2527: avoid targeting of items on hold shelf
Galen Charlton [Fri, 15 Aug 2008 21:38:22 +0000 (16:38 -0500)]
bug 2527: avoid targeting of items on hold shelf

SQL fix to properly ensure that if an item is on
the hold shelf, it will not be used for request
targeting, which applies only to items that are
not already on the hold shelf.

Prior to this fix, when checking out an item on the
hold shelf that fills patron A's hold request, it
was possible for that request to not be marked as
filled if another patron had an item-level request
on the item; the second patron's request was
incorrectly targeted by that item.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2524: improving the submitted test reports
Andrew Moore [Fri, 15 Aug 2008 18:17:14 +0000 (13:17 -0500)]
Bug 2524: improving the submitted test reports

switching from 'prove' to 'perl' to run the test scripts when we're preparing
them to be submitted to smolder. Prove adds an additional summary at the end
of the test run which confuses the TAP parser in smolder. It thinks that an
additional test has been run, and it fails it.

Then, I added a '-' before the command in the Makefile. THat's so that even
if 'perl' returns a failing code (because some tess failed), the rest of the
rules are run and a report gets submitted. Othewise, only passing reports
get sent.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoBug 2524: Adding script to submit test reports to smolder server
Andrew Moore [Fri, 15 Aug 2008 13:41:28 +0000 (08:41 -0500)]
Bug 2524: Adding script to submit test reports to smolder server

This patch adds a target in the t/Makefile and a script in t/smolder_smoke_signal
to submit the results of test runs to a smolder server. It will allow users to submit
their own test results to a central location by running "make submit" from the t/
directory.

Smolder is a web-based application that aggregates and presents the output of
tests. We have an instance running at:
http://arwen.metavore.com:8000/app/public_projects/smoke_reports/1

More information about using smolder for Koha is available at:
http://wiki.koha.org/doku.php?id=en:development:smoke_reports

I intend to have the test suite run each night automatically and the results
sent to the smolder server. This will help us keep track of the current state
of the the code and possibly catch bugs introduced by changes. I encourge others
to submit their own test results if they want, especially if they are on an unusual
platform or are with a non-standard configuration.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2522 [3/3]: populate hold_fill_targets
Galen Charlton [Fri, 15 Aug 2008 01:59:51 +0000 (20:59 -0500)]
bug 2522 [3/3]: populate hold_fill_targets

This batch job now automatically populates the
holds request targeting table hold_fill_targets.

This patch is essentially a revamp of the job,
and includes fixes for the following bugs:

2281 (holds queue report including unavailable items)
2331 (holds queue report not working with item-level
      holds)
2332 (holds queue script should attempt to fill
      many requests as possible).

Several functions in this batch job are candidates
for being moved to C4::Reserves:

GetBibsWithPendingHoldRequests()
GetPendingHoldRequestsForBib()
GetItemsAvailableToFillHoldRequestsForBib()
MapItemsToHoldRequests()

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2522 [2/3]: C4::Reserves support for request targeting
Galen Charlton [Fri, 15 Aug 2008 01:59:50 +0000 (20:59 -0500)]
bug 2522 [2/3]: C4::Reserves support for request targeting

_Findgroupreserve, which identifies which hold request an
item should fill, is modified to check to see if
that item is targeted to fill a specific hold request.

It first checks for a targeted match with an item-level hold request,
then a targeted match with a title-level request.  If no
such targeted match exists, it then checks for the top entries
in the holds queue.

The hold targeting map (i.e., the hold_fill_targets table) is
populated by the build_holds_queue.pl batch job.  If that
job is not used, the behavior of _Findgroupreserve
is not changed.

This patch also

* adjusts ModReserveMinusPriority so that it calls
  _FixPriority().
* adjusts circ/returns.pl so that it
  correctly detects transfers.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2522 [1/3]: DB rev 3.1/001 - hold request targetting
Galen Charlton [Fri, 15 Aug 2008 01:59:49 +0000 (20:59 -0500)]
bug 2522 [1/3]: DB rev 3.1/001 - hold request targetting

For bug 2522, add a new table called hold_fill_targets to
implement hold request targeting.  This table
has the following columns:

  borrowernumber     - identify request
  biblionumber       - ditto
  itemnumber         - item targeted to fill request
  source_branchcode  - current location of item
  item_level_request - if 1, request is item-level

For bug 2331, adds a column called item_level_request
to tmp_holdsqueue.

Note: both of the item_level_request columns can
go away once reserves is modified so that
it is clear whether a request is item-level or
bib-level.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoFix for bug 2155: duplicate checkin box on returns screen
Jesse Weaver [Thu, 14 Aug 2008 18:08:47 +0000 (13:08 -0500)]
Fix for bug 2155: duplicate checkin box on returns screen

This changes some css and markup to make the return box at the top not show when one is on the checkin screen. It also moves the markup for the autocompleting search box to its own file.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agobug 2341: OPAC title-level holds permitted when items are on-order
Galen Charlton [Thu, 14 Aug 2008 16:48:41 +0000 (11:48 -0500)]
bug 2341: OPAC title-level holds permitted when items are on-order

Changed so that the presence of an on-order item (items.notforloan =- -1)
permits a title-level hold request to be made.  Prior to this patch,
if all of the items linked to the bib were on-order, no title-level
request was permitted.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agofollowup to Mason's labels patch
Galen Charlton [Wed, 31 Dec 2008 20:46:09 +0000 (14:46 -0600)]
followup to Mason's labels patch

* fix XHTML errors
* fix problems with paging through results list

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoThe current searching in labels is a bit minimal, and current only does keyword searc...
Mason James [Wed, 12 Nov 2008 06:55:39 +0000 (19:55 +1300)]
The current searching in labels is a bit minimal, and current only does keyword searching.

there is some old 2.2 code for more complete labels-searching, thats been commented out

ive logged a 'bug' for this - 2777

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=2777

this fix...
  - enables the index-searching functionality
  - adds date-ranges
  - uses the numSearchResults syspref, not a hardcoded '20'
  - rewrote old 2.2 page-num code, (didnt work for 3.x)
  - some indent and whitepages tidys.
  - unused 2.2 search-code removed.

Mason

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agokohabug 2507 Fixing labels search to indicate keyword rather than title search
Chris Nighswonger [Wed, 13 Aug 2008 14:38:25 +0000 (09:38 -0500)]
kohabug 2507 Fixing labels search to indicate keyword rather than title search

Documentation may require an updated screen shot

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agomasthead - realign terminology for Tag Cloud
Chris Cormack [Thu, 3 Sep 2009 20:34:04 +0000 (08:34 +1200)]
masthead - realign terminology for Tag Cloud

Tag Cloud should link to user-generated tags, in a cloud display.
What used to be called "Tag Cloud" (ISBD subject links for UNIMARC libraries)
should be called a "Subject ...".  AFAICT it doesn't currently use a "cloudy"
display, so some other term might be more appropriate.

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

koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc

15 years agoFrench opac updates
Chris Cormack [Mon, 31 Aug 2009 22:22:49 +0000 (10:22 +1200)]
French opac updates

15 years agoItalian updates
Chris Cormack [Sat, 29 Aug 2009 09:38:41 +0000 (21:38 +1200)]
Italian updates
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoItalian Translation updates
Chris Cormack [Wed, 26 Aug 2009 10:24:52 +0000 (22:24 +1200)]
Italian Translation updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoMalay po files
Chris Cormack [Mon, 24 Aug 2009 08:58:51 +0000 (20:58 +1200)]
Malay po files

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoDutch language updates
Chris Cormack [Sun, 23 Aug 2009 10:45:54 +0000 (22:45 +1200)]
Dutch language updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoItalian and Turkish updates
Chris Cormack [Thu, 20 Aug 2009 10:35:47 +0000 (22:35 +1200)]
Italian and Turkish updates

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoTurkish updates for 3.0.x
Chris Cormack [Sat, 15 Aug 2009 18:58:02 +0000 (06:58 +1200)]
Turkish updates for 3.0.x

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
15 years agoTurkish and German updates
Chris Cormack [Sat, 15 Aug 2009 06:09:03 +0000 (18:09 +1200)]
Turkish and German updates

15 years agoFixing a bug in Turkish
Chris Cormack [Sun, 9 Aug 2009 09:42:26 +0000 (21:42 +1200)]
Fixing a bug in Turkish

15 years agoDutch, German and Turkish language updates
Chris Cormack [Sat, 8 Aug 2009 08:25:50 +0000 (20:25 +1200)]
Dutch, German and Turkish language updates

15 years agoGerman and French updates
Chris Cormack [Wed, 5 Aug 2009 02:58:22 +0000 (14:58 +1200)]
German and French updates

15 years agoGerman, French, NZ english, and Tagalog updates
Chris Cormack [Fri, 31 Jul 2009 19:29:25 +0000 (07:29 +1200)]
German, French, NZ english, and Tagalog updates

15 years agoGerman, French, Italian, Dutch and Tetum updates
Chris Cormack [Tue, 28 Jul 2009 08:17:39 +0000 (20:17 +1200)]
German, French, Italian, Dutch and Tetum updates

15 years agoTagalog added for translation
Chris Cormack [Thu, 9 Jul 2009 03:54:41 +0000 (15:54 +1200)]
Tagalog added for translation

15 years agoGerman updates
Chris Cormack [Thu, 9 Jul 2009 03:53:08 +0000 (15:53 +1200)]
German updates

15 years agoLittle fix to Maori
Chris Cormack [Mon, 6 Jul 2009 01:33:00 +0000 (13:33 +1200)]
Little fix to Maori

15 years agoMaori Opac Translation finished
Chris Cormack [Mon, 6 Jul 2009 01:18:20 +0000 (13:18 +1200)]
Maori Opac Translation finished

15 years agoBengali, french canadian and tetun updates
Chris Cormack [Sat, 4 Jul 2009 10:44:37 +0000 (22:44 +1200)]
Bengali, french canadian and tetun updates

15 years agoPreparing 3.0.3 v3.00.03
Henri-Damien LAURENT [Fri, 26 Jun 2009 13:57:18 +0000 (15:57 +0200)]
Preparing 3.0.3

15 years agoBug Fixing : 3334
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>
15 years agoBug Fix 3320 : condition on enabling field input was wrong
Henri-Damien LAURENT [Fri, 26 Jun 2009 08:05:30 +0000 (10:05 +0200)]
Bug Fix 3320 : condition on enabling field input was wrong

15 years agoFrench and greek updates
Chris Cormack [Mon, 22 Jun 2009 23:23:30 +0000 (11:23 +1200)]
French and greek updates

15 years agoTetun, German and Turkish updates
Chris Cormack [Thu, 18 Jun 2009 13:57:56 +0000 (01:57 +1200)]
Tetun, German and Turkish updates

15 years agoTurkish updates
Chris Cormack [Tue, 16 Jun 2009 02:53:17 +0000 (14:53 +1200)]
Turkish updates

15 years agoGerman and Turkish updates
Chris Cormack [Sun, 14 Jun 2009 19:30:48 +0000 (07:30 +1200)]
German and Turkish updates

15 years agobug Fixing opac/opac-reserve.pl was broken
Henri-Damien LAURENT [Fri, 12 Jun 2009 19:51:51 +0000 (21:51 +0200)]
bug Fixing opac/opac-reserve.pl was broken

15 years agoTurkish updates
Chris Cormack [Thu, 11 Jun 2009 11:01:05 +0000 (23:01 +1200)]
Turkish updates

15 years agoHindi and Turkish translations
Chris Cormack [Tue, 9 Jun 2009 19:45:27 +0000 (07:45 +1200)]
Hindi and Turkish translations

15 years agobug 3299: fix setting renewal due date explicitly
Galen Charlton [Sat, 6 Jun 2009 19:26:18 +0000 (14:26 -0500)]
bug 3299: fix setting renewal due date explicitly

When renewing a loan from the patron details page, setting
the renewal due date now works again.  Broken by patch
for bug 2770.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
15 years agoTurkish and Greek updates
Chris Cormack [Sun, 7 Jun 2009 11:08:06 +0000 (23:08 +1200)]
Turkish and Greek updates

15 years agoHindi, German and Turkish updates
Chris Cormack [Fri, 5 Jun 2009 19:03:24 +0000 (07:03 +1200)]
Hindi, German and Turkish updates

15 years agominor update on release notes v3.00.02-final
Henri-Damien LAURENT [Thu, 4 Jun 2009 12:09:39 +0000 (14:09 +0200)]
minor update on release notes