[koha-commits] main Koha release repository branch master updated. v3.16.00-353-g4e27878
Git repo owner
gitmaster at git.koha-community.org
Sun Aug 3 22:01:45 CEST 2014
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, master has been updated
via 4e278782f156a9d7452e6e9ddcefee9542b964fc (commit)
via ab0d9ae5163d22a9f0e971a80d8470275332fe67 (commit)
via c7ca45d0463099ac5cee8978d0f05960e3779c0f (commit)
via 5466a5f4ce9a3b71ce1cdf4927bc78d81e3262d7 (commit)
via 2db8ba555d40ee2bb8edb0342e86e03d5d2a7d04 (commit)
from 54c4c73e479f374dad7b5db6dc23272960aa2a01 (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 4e278782f156a9d7452e6e9ddcefee9542b964fc
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Tue Jul 29 15:00:56 2014 -0300
Bug 12673: Remove CGI::scrolling_list from C4::Items.pm
This patch removes only instance in this file
To test:
1. Apply the patch
2. (Esay way) Put on staff URL
/cgi-bin/koha/services/itemrecorddisplay.pl?biblionumber=N&itemnumber=M
3. Code replaced corresponds to pulldowns, verify information
is correct (check with another item edit view)
PrepareItemrecordDisplay() in Items.pm is filled with chunks of
HTML code that must be put on TT file. For now scrolling_list
is the only removed.
Signed-off-by: Owen Leonard <oleonard at myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Tested different item edit displays and the URL given above,
no regressions found. Also passes all tests and QA script.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit ab0d9ae5163d22a9f0e971a80d8470275332fe67
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Wed Jul 30 11:31:39 2014 -0300
Bug 12675: Add unit test to GetFrameworksLoop
to test:
1. Apply the patch
2. Run the test, prove t/db_dependent/Koha.t
3. Check that it pass
Signed-off-by: Owen Leonard <oleonard at myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit c7ca45d0463099ac5cee8978d0f05960e3779c0f
Author: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Date: Tue Jul 29 15:49:44 2014 -0300
Bug 12675: Remove CGI::scrolling_list from labeledMARCdetail.pl
This patch removes only instance in this file.
Also updates getframework POD on C4/Koha.pm,
adds new GetFrameworksLoop() func on same file from
suggested code, but with ordered result.
To test:
1. Apply the patch
2. Enable viewLabeledMARC syspref
3. On staff, search for a record, goto detail view
4. Clic on Labeled MARC
5. Framework pulldown was replaced, check changing
framework.
A bug was fixed, because selecting any fw and then Default
tries to load values from 'Default' fw code, which does not
exists.
Signed-off-by: Owen Leonard <oleonard at myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Works as described, passes tests and QA script.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 5466a5f4ce9a3b71ce1cdf4927bc78d81e3262d7
Author: Owen Leonard <oleonard at myacpl.org>
Date: Mon Jul 28 11:35:47 2014 -0400
Bug 12677: Use Bootstrap "label" style for inline OPAC renewal messages
This patch revises the styling of the renewal success message to use a
default Bootstrap style more suited to inline messages. I think this is
preferable to using the alert style because the alert is designed to be
a block-level element.
http://getbootstrap.com/2.3.2/components.html#labels-badges
Because Koha's CSS already uses a "label" class for something else I
duplicate the Bootstrap "label" class in opac.css as "blabel." The
Bootstrap color definitions are included.
To test, follow the test plan already provided:
1) Apply this patch
2) From the OPAC, renew some items
3) Note the "Renewed!" message in the "Renew" column of the checkouts
table
Patch works as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Works as described, no problems found.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
commit 2db8ba555d40ee2bb8edb0342e86e03d5d2a7d04
Author: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Date: Tue Jul 29 12:04:52 2014 +0300
Bug 12668 - Stray dollar ($) -sign in opac-reserve.pl
A dollar sign is hard-coded in opac-reserve.pl and becomes apparent when trying to place a reservation
when one has "too_much_oweing" or too much fines.
Removing the dollar sign so we just get
<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja 9.50.">
instead of
<"Käyttömaksujen katto ylitetty. Et voi tehdä varauksia. Sinulla on maksamattomia maksuja $9.50.">
Patch removes hard coded $ sign.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
The dollar sign is gone and the message still displays correctly.
Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
-----------------------------------------------------------------------
Summary of changes:
C4/Items.pm | 15 ++---
C4/Koha.pm | 71 +++++++++++++++++---
catalogue/labeledMARCdetail.pl | 27 +-------
.../prog/en/modules/catalogue/labeledMARCdetail.tt | 13 +++-
.../prog/en/modules/services/itemrecorddisplay.tt | 12 ++++
koha-tmpl/opac-tmpl/bootstrap/css/opac.css | 2 +-
.../opac-tmpl/bootstrap/en/modules/opac-user.tt | 2 +-
koha-tmpl/opac-tmpl/bootstrap/less/opac.less | 25 +++++++
opac/opac-reserve.pl | 2 +-
t/db_dependent/Koha.t | 47 ++++++++++++-
10 files changed, 170 insertions(+), 46 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list