[koha-commits] main Koha release repository branch 3.12.x updated. v3.12.05-10-g3b9e882
Git repo owner
gitmaster at git.koha-community.org
Tue Oct 8 15:56:29 CEST 2013
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".
The branch, 3.12.x has been updated
via 3b9e88232d1296c1956c193820cf03d7e58edc63 (commit)
via 09b01f21276c309b4741abf31aafae9b2341c16f (commit)
via 9dba1f477703bb5d9bb152de9670fa6ee18c6999 (commit)
via 2448e200e2b520ce5bfe01f3256f1de0659d71c7 (commit)
via 14d2ac659a55e9a3fd601994546c538a1fb11f6f (commit)
via 492af6ce72dcefd09b6c80e225a5bc8da4b832eb (commit)
via 40e0fccdd9c3d09a95543671f90a3a163e225ce4 (commit)
from df14898902105a5404d640fae2478ab63185f387 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 3b9e88232d1296c1956c193820cf03d7e58edc63
Author: Owen Leonard <oleonard at myacpl.org>
Date: Fri May 31 12:05:49 2013 -0400
Bug 10349 - Don't show empty Descriptions/Title notes tabs in OPAC and staff
The descriptions/title notes tab appears on the detail page in both
staff client and OPAC even if there are no notes. This is probably a
relic of the pre-T:T days when it wasn't possible to use || in an IF.
This patch adds a check for the various variables which might trigger
the display of the tab.
To test, apply the patch and view records in the OPAC and staff client
which do and do not have title notes attached (whether that be in the
MARC record or in the biblio.notes column). In the OPAC Syndetics
content should also be tested if possible. The descriptions/title notes
tab should only appear if there is content.
Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
That's better - thx Owen!
Works nicely and passes all tests.
Tested in staff and OPAC, also in combination with NotesBlacklist
hiding all notes.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit f5c7badb31dff9afcbb69e4f58155560e98a922f)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
Works nicely.
commit 09b01f21276c309b4741abf31aafae9b2341c16f
Author: Janusz Kaczmarek <januszop at gmail.com>
Date: Mon Mar 18 22:16:22 2013 +0200
Bug 10324: fix display of editorial series in normal view (UNIMARC / no XSLT)
The patch corrects the issue -- the content of the field 225 shall be
displayes under Series now.
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Comment: Work as described, no koha-qa errors
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Comparing the XSLT with the normal view the patch seems to
work correctly. Passes all tests and QA script.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 384566c1295a81dce0d9de001fcde36f2a996dfa)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 9dba1f477703bb5d9bb152de9670fa6ee18c6999
Author: Galen Charlton <gmc at esilibrary.com>
Date: Sun Sep 8 01:20:05 2013 +0000
Bug 10272: (follow-up) add regression test
This patch adds a regression test for this bug, effectively
implementing the manual test plan in the previous patch.
To test:
[1] Verify that prove -v t/db_dependent/Reserves.t passes.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit e831a7eb3fe21561264cc756f40c539333e8ebbc)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 2448e200e2b520ce5bfe01f3256f1de0659d71c7
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date: Fri May 17 07:08:24 2013 -0500
Bug 10272: make CheckReserves respect ReservesControlBranch
CheckReserves was using the CircControl system preference to determine what
patrons an item can fill a hold for. It should be using ReservesControlBranch
instead.
Test Plan:
1) Set ReservesControlBranch to "item's home library".
2) Create an item at Library A, place holds for it for patrons at
Library B, Library C, and Library A in that order,
for pickup at the patrons home library.
3) Make sure the holds policy for Library A is set to
Hold Policy = "From home library" and
Return Policy = "Item returns home".
Make sure the holds policies for the other libraries are set to
Hold Policy = "From any library".
4) Check the item in at Library C, the hold for the patron at Library B
should pop up, even though it's in violation of the circulation rules.
Don't click the confirm button!
5) Apply this patch, and reload the page,
now the hold listed should be for the last hold,
the hold for the patron at Library A, which is correct.
This patch adds the subroutine C4::Reserves::GetReservesControlBranch as
an equivilent to C4::Circulation::_GetCircControlBranch.
Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Fixed POD so that arguments and explanation match (C<$item>).
Also tested opac-reserves.pl for regressions.
Passes all tests, QA script, and Reserves.t.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 78eba2f974ca9e8ff51f64d2356b004e7bfcecff)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 14d2ac659a55e9a3fd601994546c538a1fb11f6f
Author: Owen Leonard <oleonard at myacpl.org>
Date: Thu Aug 1 10:57:02 2013 -0400
Bug 10672 - Add subtitle to display of checkouts, overdues, and holds on the patron summary
In the OPAC, the patron summary page (opac-user.pl) should display
subtitle along with title in the lists of checkouts, overdues, and
holds. This patch adds it.
To test, log in to the OPAC as a patron who has checkouts, overdues, and
holds which include titles with subtitles. Titles should appear
correctly with and without subtitles.
Signed-off-by: Srdjan <srdjan at catalyst.net.nz>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit ec070fa8375d9001041fc089ff193b17f87ae4c1)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 492af6ce72dcefd09b6c80e225a5bc8da4b832eb
Author: Owen Leonard <oleonard at myacpl.org>
Date: Tue Aug 13 14:50:06 2013 -0400
Bug 10721 - Improve DataTables configuration on transfers to receive report
I noticed this template used a hidden span to allow correct sorting of
titles--a relic of the previous table sorting plugin. I have removed
that and added an improved DataTables configuration.
To test apply the patch and view the transfers to receive report for a
library which has multiple transfers to receive. Sorting should work
correctly on all columns, including correct date sorting regardless of
dateformat system preference. Title sorting should correctly exclude
articles.
Signed-off-by: David Cook <dcook at prosentient.com.au>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Passes all tests and works nicely.
Tested with different date formats.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 8c3d3eb01a91c073aa9a5f2cb016bba01f0529ab)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 40e0fccdd9c3d09a95543671f90a3a163e225ce4
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date: Fri Aug 16 15:10:56 2013 +0200
Bug 10742: improve converting simple search to Z39.50 search
In intranet, when search does not return results, there is a button
to perform the same search on Z39.50 servers. This works well when
coming from advanced search. But when using simple search from header,
in the Z39.50 search box "kw,wrdl" is added to the operand in title.
This patch simply adds kw as default value when the cgi does not have
idx parameter.
Test plan :
- In intranet, perform a search from header that does not return any
result. For example "afalseword"
- Click on "Z39.50 Search"
=> you get a popup with the searched word in title. For example:
Title: afalseword
- Go to advanced search page
- Enter the same word into first input and submit
- Click on "Z39.50 Search"
=> you get a popup with the searched word in title.
Signed-off-by: David Cook <dcook at prosentient.com.au>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Small change, fixes an annoying problem.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 0ab71d32641d092d11228f0e2282dd8805198720)
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
-----------------------------------------------------------------------
Summary of changes:
C4/Biblio.pm | 6 +-
C4/Reserves.pm | 30 ++++-
catalogue/search.pl | 2 +-
circ/transferstoreceive.pl | 2 +-
.../prog/en/modules/catalogue/detail.tt | 8 +-
.../prog/en/modules/circ/transferstoreceive.tt | 10 +-
koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 8 +-
koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt | 6 +-
opac/opac-reserve.pl | 2 +-
opac/opac-user.pl | 4 +-
t/db_dependent/Reserves.t | 125 +++++++++++++++++++-
11 files changed, 184 insertions(+), 19 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list