[koha-commits] main Koha release repository branch 16.11.x updated. v16.11.00-37-g75c3cda
Git repo owner
gitmaster at git.koha-community.org
Sun Dec 18 22:26:13 CET 2016
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.11.x has been updated
via 75c3cda017e4998974fad285e21bf63b08e4fce6 (commit)
via 2bfd9a95f09c29d1ae116213f83b416a3922cf8c (commit)
via 327c2ba63612c56599d590a0a2dd8539d255b145 (commit)
via 80955d365af98627110783bc16b12964d117273d (commit)
via 2820755f153c19107d8c1e67371941b42ab901d2 (commit)
via 45bd59064f117f984b4bc041c12136a7d6a880c5 (commit)
via 1dcde292a41927266113ff53392ea159d3f7dcc4 (commit)
via c30a6b1131adc88ed2e23227618ccd046f84fde6 (commit)
via 5b7119c87d5187f1d45c3ef82597264b2574225d (commit)
via 9af15e2cb487280e3c4b5859a5edb77a0403c09e (commit)
via 59d2080d204f3a6b980be35744564500d2a05bdb (commit)
from 10755f1c74b442289c1229caf4cfdda6a3bb32ad (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 75c3cda017e4998974fad285e21bf63b08e4fce6
Author: Kyle M Hall <kyle at kylehall.info>
Date: Mon Dec 12 06:59:30 2016 -0500
Bug 17761 - Renewing or returning item via the checkouts table causes lost and damaged statuses to disappear
Due to the fact that the lost and damaged status are contained within the due_due span,
when the item is renewed or returned these statuses disappear.
Some librarians find this disconcerting and it seems that it would be better to keep the visible.
Test Plan:
1) Find a lost item checked out to a patron
2) Renew that lost item via the checkouts table
3) Notice the lost status disappears from that row
4) Apply this patch
5) Refresh the page ( you may want to clear your browser cache as well )
6) Renew the item again
7) The lost status should remain visible!
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Signed-off-by: Edie Discher <edischer at cityoffargo.com>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 97eb773c0a4054f0538c8966e207734f6f56bdd5)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 2bfd9a95f09c29d1ae116213f83b416a3922cf8c
Author: Chris Kirby <chris.kirby at ilsleypubliclibrary.org>
Date: Mon Nov 28 00:15:02 2016 -0500
Bug 17652: opac-account.pl does not include login branchcode
opac-account.pl does not include the login branchcode, whereas the other pages
available to users (opac-user, opac-memberentry, opac-passwd, opac-readingrecord,
opac-suggestions, opac-messaging, opac-shelves, etc.) on login DO provide
the login branchcode. Having the login branchcode available on opac-account.pl
would facilitate customization of the OPAC for individual branches.
To test:
1) Login to the OPAC and open /cgi-bin/koha/opac-account.pl
2) Inspect the body tag and observe the class attribute does not include
the login branchcode.
3) Apply the patch.
4) Repeat step 2.
5) Observe that the login branchcode is present.
Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 1bfcebfb7fa0837428fb577a6fc059886fe37afb)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 327c2ba63612c56599d590a0a2dd8539d255b145
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Fri Nov 25 13:22:42 2016 +0100
Bug 17681: Exiisting typos might thow some feees when recieved
Yes, we are fixing these four typos here.
Test plan:
[1] Read the changes.
[2] Run t/Auth_with_shibboleth.t
[3] Run git grep recieved| grep -v -e 'recievedlist' | grep -v -e 'serials-recieve.tt'
Note: serials-recieve.tt is just history. Bonus points for the one who makes
us get rid of that column recievedlist.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
EDIT:
Rebased. Change in Accounts has been corrected already.
Removed the po file.
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit b5c1a965b165093fb66a5db356da84b4a62bf9a6)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 80955d365af98627110783bc16b12964d117273d
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date: Fri Dec 2 17:16:10 2016 -0300
Bug 17547: (MARC21|NORMARC) have Chronological term field $9 indexed
This patch makes Zebra index the 648$9 link for chronological terms on
bibliographic records. This way an authority search on chronological terms
will show the right number in 'Used in X records' message.
To test:
- Have a record with a 648 field, linked to an authority record (i.e. with an authid on 648$9).
- Search for the record, notice it is indexed.
- Perform an authority search for the chronological term
=> FAIL: the term is linked to our record, but koha shows '0' count.
- Apply the patch
- Run:
$ cd kohaclone
$ xsltproc etc/zebra/xsl/koha-indexdefs-to-zebra.xsl \
etc/zebradb/marc_defs/marc21/biblios/biblio-koha-indexdefs.xml \
> etc/zebradb/marc_defs/marc21/biblios/biblio-zebra-indexdefs.xsl
$ git diff
=> SUCCESS: Notice the shipped etc/zebradb/marc_defs/marc21/biblios/biblio-zebra-indexdefs.xsl
is up-to-date
- Run:
$ sudo cp etc/zebradb/marc_defs/marc21/biblios/biblio-zebra-indexdefs.xsl \
/etc/koha/zebradb/marc_defs/marc21/biblios/biblio-zebra-indexdefs.xsl
$ sudo koha-restart-zebra kohadev
$ sudo koha-rebuild-zebra -f -b -v kohadev
- Search for the record, notice it is indexed.
- Perform an authority search for the chronological term
=> SUCCESS: the term is linked to our record, usage count is 1
- Sign off :-D
I assume NORMARC is similar on this regard. Feel free to fail it if the NORMARC part of the
patch is wrong.
Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Hugo Agud <hagud at orex.es>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit cb3f899826966b5da1242292091aa71ab4831618)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 2820755f153c19107d8c1e67371941b42ab901d2
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Mon Dec 12 13:48:40 2016 +0100
Bug 17759: Remove invalid guarantorid's in Members.t
See also bug 17733. As long as we have no FK, these invalid guarantorid's
can make tests fail.
Adding a sql statement to clear them in the beginning of the test.
Test plan:
[1] Since it seems that AUTO_INC+8, +9 and +10 may fail one or two tests,
we should manipulate one or two borrowers:
Run this SQL query first:
SELECT `AUTO_INCREMENT` FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_SCHEMA = '[your database]' and table_name='borrowers';
Now run: UPDATE borrowers SET guarantorid=[former value + 8] WHERE borrowernumber=[some existing borrowernumber]
[2] Without this patch, run Members.t. Should fail two tests.
[3] Apply the patch. Fetch the new AUTO_INCREMENT value and update
guarantorid accordingly again for the manipulated borrower.
[4] Run the test. It should not fail.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 82d75e124e416aef13f11d43837438799bd850dd)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 45bd59064f117f984b4bc041c12136a7d6a880c5
Author: Nick Clemens <nick at bywatersolutions.com>
Date: Thu Dec 8 19:41:33 2016 +0000
Bug 17749 - Missing l in '.pl' in link on waitingreserves.tt
Fixes a small typo introduced by Bug 17010
waitingreserves.p#[% tab %]
should be
waitingreserves.pl#[% tab %]
Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit a1592ec10be56569b64774a3dde1e85c53ba36d9)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 1dcde292a41927266113ff53392ea159d3f7dcc4
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Tue Dec 6 17:09:21 2016 +0100
Bug 17733: Fix Members.t
In some dirty DB (Jenkins), the borrowers.guarantorid field is not set
to a valid patron id.
That make some tests fail if we create a patron with that patron id.
In my DB, auto increment for borrowers is set to 52 before the tests
On the 4th run of the tests, some tests fail.
Before I got a patron with a guarantorid=112 (I let you do the math).
Test plan:
The easiest is to trust me
Todo: Add a FK!
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Opened a new report for special case: Bug 17759.
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 62074e39f0bbce1665f2b329beecc61fc55c994e)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit c30a6b1131adc88ed2e23227618ccd046f84fde6
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Thu Dec 15 08:43:05 2016 +0100
Bug 17666: [QA Follow-up] Mark File::Slurp as required
Not optional since we are 'using' it in updatedatabase.pl.
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 6b67f635b14b6ce1337785f65ef1dfaa4a36258e)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 5b7119c87d5187f1d45c3ef82597264b2574225d
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Mon Dec 12 21:44:59 2016 +0000
Bug 17666: Revert "Bug 16685 [QA Followup] - Use Path::Tiny::slurp instead of File::Slurp"
This reverts commit 7ac1bc35372990520a7c67870b2c702c41bbebcc.
On a kohadevbox:
Error flock (1) on
'/home/vagrant/kohaclone/installer/data/mysql/atomicupdate/bug_xxxx.perl':
No locks available at installer/data/mysql/updatedatabase.pl line 13637.
This is because Path::Tiny->slurp[_utf8] uses flocks which does not work
with NFS
Test plan:
Execute a .perl atomic update file on a kohadevbox
=> Without this patch you should get the flock error
=> With this patch the update should work fine
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit c9e95aa22e56536654305ab32ab1b15a27e89557)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 9af15e2cb487280e3c4b5859a5edb77a0403c09e
Author: Owen Leonard <oleonard at myacpl.org>
Date: Mon Nov 21 08:26:13 2016 -0500
Bug 11274 Sent Notices Tab Not Working Correctly
This patch implements the notices toggle event using a different method.
Using on() lets us bind the event to a parent element, allowing the
event to attach to elements which may not exist on the page yet.
Test plan the same as original patch:
1) have a patron with more then 20 notices sent
2) go to patron profile -> notices
-> without patch, the showing/hidding of notices content does work
correctly only on first loaded page
-> with patch, it should work correctly everywhere - try to use
paginator, searching, ordering....
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 6163337cd8caab98db993e4d51ac73abbdd83b86)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
commit 59d2080d204f3a6b980be35744564500d2a05bdb
Author: Julian Maurice <julian.maurice at biblibre.com>
Date: Thu Dec 8 10:13:35 2016 +0100
Bug 17744: Remove the oai_dc:dcCollection wrapper in OAI responses
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
(cherry picked from commit 3e9dc5d51c5a14fcefe8611c532fe970e930b72a)
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
-----------------------------------------------------------------------
Summary of changes:
C4/Installer/PerlDependencies.pm | 9 ++------
Koha/OAI/Server/Repository.pm | 2 +-
Koha/Object.pm | 2 +-
acqui/finishreceive.pl | 2 +-
.../marc21/biblios/biblio-koha-indexdefs.xml | 3 +++
.../marc21/biblios/biblio-zebra-indexdefs.xsl | 9 ++++++++
.../normarc/biblios/biblio-koha-indexdefs.xml | 3 +++
.../normarc/biblios/biblio-zebra-indexdefs.xsl | 24 ++++++++++++++++++++
installer/data/mysql/updatedatabase.pl | 4 ++--
.../prog/en/modules/circ/waitingreserves.tt | 2 +-
.../prog/en/modules/members/notices.tt | 20 ++++++++--------
koha-tmpl/intranet-tmpl/prog/js/checkouts.js | 3 ++-
.../opac-tmpl/bootstrap/en/modules/opac-account.tt | 2 +-
misc/cronjobs/cleanup_database.pl | 2 +-
misc/release_notes/release_notes_200RC4.txt | 4 ++--
t/Auth_with_shibboleth.t | 4 ++--
t/db_dependent/Members.t | 6 +++++
17 files changed, 71 insertions(+), 30 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list