[koha-commits] main Koha release repository branch 16.05.x updated. v16.05.11-14-g9de99ca
Git repo owner
gitmaster at git.koha-community.org
Mon Apr 3 18:22:24 CEST 2017
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, 16.05.x has been updated
via 9de99ca2d5494177bf4e0c805cc46cb3124fdd5c (commit)
via 79551256839370ec7c711fe7ed91d08055d8d312 (commit)
via 1d2313ec3401c2cbbf81e70f4baf86ddfc0d4280 (commit)
via 99d35fdc71ed1690dc9c46aa8992088064e0250b (commit)
via 3440d402e86247baa27bd051b379cda6539c528a (commit)
via 32e7f2be6731bf1e39920e2aa39da26ada6831ea (commit)
via bb51b7447fcd7592e683648f4854c15f26a57eb5 (commit)
via 198f89bf1d2f072e1a0aaea032237d470272adf3 (commit)
via 2560b7fa72ab3801257f4cd3d65506eb3405952f (commit)
via b696300c3a2ddbd670775fe22a0cb95b61751abf (commit)
via dde4036df4fba9b8d3652a78c320c42f9e71c660 (commit)
via 9ae18c69dbbb34d7bacac27b1066f85af8a3a30c (commit)
via faef4f5d866edbb30ed6cf62823cc969e8f9f8c6 (commit)
via 94387428d5de8f2867653831a401a62f34a75040 (commit)
from d451096fa4e9aff62838f8faf3101b3e67217227 (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 9de99ca2d5494177bf4e0c805cc46cb3124fdd5c
Author: Mason James <mtj at kohaaloha.com>
Date: Tue Apr 4 04:20:43 2017 +1200
Increment version for 16.05.12 release
commit 79551256839370ec7c711fe7ed91d08055d8d312
Author: Mason James <mtj at kohaaloha.com>
Date: Tue Apr 4 04:15:43 2017 +1200
Translation updates for Koha 16.05.12
commit 1d2313ec3401c2cbbf81e70f4baf86ddfc0d4280
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Thu Feb 9 12:44:38 2017 +0100
Bug 15854: Use a READ and WRITE LOCK on message_queue
To make sure we will not never get a race conditions for these kinds of
notices, we need to add a LOCK on the message_queue table.
This does not smell the best way to do that, but I faced deadlock issues
when I tried to use "UPDATE FOR"
https://dev.mysql.com/doc/refman/5.7/en/innodb-locking-reads.html
https://dev.mysql.com/doc/refman/5.7/en/lock-tables.html
https://dev.mysql.com/doc/refman/5.7/en/commit.html
To test this patch, or another solution, you need to apply manually this
change:
my $message = C4::Message->find_last_message($borrower, $type, $mtt);
unless ( $message ) {
+ sleep(1);
C4::Message->enqueue($letter, $borrower, $mtt);
} else {
And repeat the test plan from first patch.
Do not forget to truncate the message_queue table.
Followed test plans, works as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit be156d9ad9e5bcfadab34d44f90e04fd61e256ad)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 99d35fdc71ed1690dc9c46aa8992088064e0250b
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Thu Feb 9 12:13:07 2017 +0100
Bug 15854: Simplify the code to limit race conditions
There is an obvious race condition when CHECKIN and RENEWAL are
generated from circulation.pl calling svc/renew or svc/checkin in AJAX.
The 2 first queries will try to get the id of the last message
(find_last_message) and if it does not exist, they will insert it.
Theorically that could be lead to have several "digest" messages for a
given patron.
I did not recreate more than 2 messages, from the third one at least one
of the two firsts existed in the DB already.
This patch just simplifies the code to make the SELECT and INSERT or
UPDATE closer and limit the race condition possibilities.
Test plan:
0. Set RenewalSendNotice and circ rules to have a lot of renewals available
1. Use batch checkouts (or one by one) to check out several items to a
patron
2. Empty message_queue (at least of this patron)
3. Renew them all at once ("select all" link, "renew or check in"
button)
4. Check the message_queue
Without this patch you have lot of chances to faced a race condition and
get at least 2 messages for the same patron. This is not expected, we
expect 1 digest with all the messages.
With this patch apply you have lot of chances not to face it, but it's
not 100% safe as we do not use a mechanism to lock the table at the DBMS
level.
Tested both patches together, works as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit 607b14516a955c9989e4764c69527edbc1f36ba0)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 3440d402e86247baa27bd051b379cda6539c528a
Author: Colin Campbell <colin.campbell at ptfs-europe.com>
Date: Wed Nov 9 14:37:33 2016 +0000
Bug 17605: [STABLE] Add currency to orders generated by quotes
Sets the vendors currency in the edi generated order
The currency used is agreed between vendor and library
and the value in the quote is optional
Edifact potentially allows the currency to be overwitten
by specifying another currency in the CUX segment but
we know currently of no supplier doing this
Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
commit 32e7f2be6731bf1e39920e2aa39da26ada6831ea
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Sun Jan 29 15:40:14 2017 +0100
Bug 18010: Remove potential exposure from gettemplate
A similar bad template check from C4::Auth::get_template_and_user
should be applied in C4::Templates::gettemplate.
Before this patch it would be possible to expose files like:
my $template = C4::Templates::gettemplate(
'/etc/passwd', 'intranet', CGI::new, 1
);
print $template->output;
Note that the is_plugin flag in the above call is the culprit. This patch
provides a quick security fix without touching get_template_and_user, and
can be backported to stable branches.
I will provide an enhanced and centralized check on report 17989, also
removing the is_plugin flag.
Note: We allow .pref here too for use in admin/preferences.pl.
Test plan:
[1] Run t/db_dependent/Auth.t (triggering get_template_and_user and
gettemplate).
[2] Run t/db_dependent/Templates.t again (see first test plan).
The tests should no longer fail.
[3] Open a page on opac or intranet.
[4] Open a systempreferences tab.
[5] Add a book to the cart and send it ([opac-]sendbasket uses gettemplate).
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit fbbd4b40f0d71b79194fc186e66f985488be26c6)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit bb51b7447fcd7592e683648f4854c15f26a57eb5
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Sun Jan 29 15:52:22 2017 +0100
Bug 18010: Unit test for gettemplate
A trivial test, similar to the ones in Auth.t.
Without the check in gettemplate (added in the second patch), the passwd
file will be exposed and the test fails.
Test plan:
Run t/db_dependent/Templates.t without second patch. The two tests in the
last subtest should fail.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit 5508897d5a2b73c30d50fed26a6bfed62bf49398)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Conflicts:
t/db_dependent/Templates.t
commit 198f89bf1d2f072e1a0aaea032237d470272adf3
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Wed Feb 1 12:42:43 2017 +0100
Bug 18028: Remove outdated install_misc directory
This directory is no longer maintained and contain outdated information
on how to install Koha.
On the Internet there are tutos using them and that can create confusion
to new user (yes it could be confusing as well to execute a file that
does no longer exist).
NOTE: last tweak was mid 2016, but before that 2015. So I agree with
the deletion.
Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit a6f50cbcb92c2575eb31e906ea54bf0386654906)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 2560b7fa72ab3801257f4cd3d65506eb3405952f
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Tue Feb 7 09:10:42 2017 +0100
Bug 18069: Remove residue of rebuild_zebra -x
Bug 17731 removed the -x option of rebuild_zebra but koha-rebuild-zebra
still cals the script with this option.
"Warning: You passed -x which is already the default and is now deprecated"
Test plan:
sudo koha-rebuild-zebra -f
should no longer raise the warning
Signed-off-by: Mason James <mtj at kohaaloha.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit fe898bc641b6d76d7b869654dce4ec21e219daf8)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit b696300c3a2ddbd670775fe22a0cb95b61751abf
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Tue Feb 14 15:22:40 2017 +0000
Bug 18094: Only search in searchable patron attributes if searching in standard fields
Test plan:
- Add a new patron attrbute and mark it searchable
- Populate a new patron with 'potato' in that field
- Add/edit another patron to have email potato at invalidemail.com'
- Perform a patron search with query 'potato' (in standard fields)
=> Both patrons are returned
- Perform a patron search with filters 'Email' and query 'potato'
=> Only 1 patron is returned and you are redirected to the patron detail page.
Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit 5823d4ca6a71404d502eeb7f1bf5dd152fd98e22)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit dde4036df4fba9b8d3652a78c320c42f9e71c660
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Tue Feb 14 16:19:25 2017 +0100
Bug 18094: Add tests to highlight the problem
Signed-off-by: Marc Véron <veron at veron.ch>
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit f86500351af0a1a4d1593eb2aa689b8ec52bc401)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Conflicts:
t/db_dependent/Utils/Datatables_Members.t
commit 9ae18c69dbbb34d7bacac27b1066f85af8a3a30c
Author: Marc Véron <veron at veron.ch>
Date: Sat Mar 11 19:17:37 2017 +0100
Bug 18246: Patron card creator: Unit not always displays properly in layouts
To reproduce:
1) Create a new layout
2) Select an unit in "Units selection"
3) Save
4) Edit layout again
5) Select other unit, e.g. 'SI Centimeter'
6) Save
7) Repeat
8) Edit again
Error: Latest unit seems to stick and can no longer be changed, because
in units drop down, multiple untis are marked as selected.
To test:
1) Apply patch
2) Repeat steps above, verify that value for units always display as expected.
Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit 37da40b323d8bc012724510fe4839dcb908c2b3b)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit faef4f5d866edbb30ed6cf62823cc969e8f9f8c6
Author: Marc Véron <veron at veron.ch>
Date: Sat Mar 11 17:18:10 2017 +0100
Bug 8603: Patron card creator - 'Barcode Type' doesn't stick in layouts
To reproduce:
1. Make a new layout
2. Tick "Print card number as barcode"
3. Change 'Barcode Type' to anything other than Code39
4. Save
5. Edit it again
6. Its back to Code39
To test:
1. Apply patch
2. Repeat steps above
3. Verify that 'Barcode type' is selected as expected
Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
https://bugs.koha-community.org/show_bug.cgi?id=18063
https://bugs.koha-community.org/show_bug.cgi?id=8063
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit 20ad341eefc555e0cf2b0f2862347e8a48b28935)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 94387428d5de8f2867653831a401a62f34a75040
Author: David Gustafsson <david.gustafsson at ub.gu.se>
Date: Mon Mar 20 15:13:13 2017 +0100
Bug 18305 - jquery.fixFloat.js breaks advanced MARC editor
Remove use of ECMASCRIPT 2015 default parameter syntax in
jquery.fixFloat.js to be compatible
with a wider range of browser versions.
Test plan:
1. Usa a not so modern browser, for example Chrome/Chromium < 49, any version of IE
besides Edge, Opera, or Firefox < 15.0.
2. Edit a bibliographic record in staff client and try switching to
advanced MARC editor (check console, no errors should appear).
Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Brendan A Gallagher <brendan at bywatersolutions.com>
(cherry picked from commit a649958315c98500ebe0ea3f91c05a8067d54e58)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
-----------------------------------------------------------------------
Summary of changes:
C4/Circulation.pm | 64 +++----
C4/Templates.pm | 1 +
C4/Utils/DataTables/Members.pm | 2 +-
Koha.pm | 2 +-
Koha/EDI.pm | 2 +
debian/scripts/koha-rebuild-zebra | 8 +-
install_misc/UpgradeBackup.pm | 94 ----------
install_misc/debian.packages | 148 ---------------
install_misc/environment_Makefile.PL | 80 --------
install_misc/ubuntu-packages.sh | 194 --------------------
install_misc/ubuntu.10.04.packages | 163 ----------------
install_misc/ubuntu.12.04.packages | 168 -----------------
install_misc/ubuntu.packages | 168 -----------------
installer/data/mysql/updatedatabase.pl | 6 +
.../lib/jquery/plugins/jquery.fixFloat.js | 3 +-
misc/translator/po/cs-CZ-marc-MARC21.po | 22 +--
misc/translator/po/cs-CZ-staff-prog.po | 53 +++---
misc/translator/po/de-DE-opac-bootstrap.po | 8 +-
misc/translator/po/de-DE-staff-help.po | 8 +-
misc/translator/po/de-DE-staff-prog.po | 13 +-
misc/translator/po/el-GR-marc-MARC21.po | 2 +-
misc/translator/po/el-GR-marc-NORMARC.po | 2 +-
misc/translator/po/el-GR-marc-UNIMARC.po | 2 +-
misc/translator/po/el-GR-opac-bootstrap.po | 2 +-
misc/translator/po/el-GR-pref.po | 19 +-
misc/translator/po/el-GR-staff-help.po | 36 ++--
misc/translator/po/el-GR-staff-prog.po | 41 ++---
misc/translator/po/fi-FI-marc-MARC21.po | 8 +-
misc/translator/po/fr-FR-marc-MARC21.po | 10 +-
misc/translator/po/fr-FR-opac-bootstrap.po | 10 +-
misc/translator/po/fr-FR-pref.po | 60 +++++-
misc/translator/po/fr-FR-staff-prog.po | 33 ++--
misc/translator/po/hi-marc-MARC21.po | 8 +-
misc/translator/po/hi-pref.po | 15 +-
misc/translator/po/hi-staff-prog.po | 49 ++---
misc/translator/po/it-IT-marc-MARC21.po | 10 +-
misc/translator/po/it-IT-pref.po | 18 +-
misc/translator/po/it-IT-staff-help.po | 12 +-
misc/translator/po/ko-Kore-KP-pref.po | 6 +-
patroncards/edit-layout.pl | 6 +-
t/db_dependent/Templates.t | 14 +-
t/db_dependent/Utils/Datatables_Members.t | 31 +++-
42 files changed, 354 insertions(+), 1247 deletions(-)
delete mode 100644 install_misc/UpgradeBackup.pm
delete mode 100644 install_misc/debian.packages
delete mode 100644 install_misc/environment_Makefile.PL
delete mode 100755 install_misc/ubuntu-packages.sh
delete mode 100644 install_misc/ubuntu.10.04.packages
delete mode 100644 install_misc/ubuntu.12.04.packages
delete mode 100644 install_misc/ubuntu.packages
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list