[koha-commits] main Koha release repository branch 3.14.x updated. v3.14.02-8-gbff4fac
Git repo owner
gitmaster at git.koha-community.org
Mon Jan 27 13:19:42 CET 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, 3.14.x has been updated
via bff4fac7b78fe8573527c8733f8364af351d23a9 (commit)
via ae55277499f06ae0d3e2973349d1f34cf7f60ec8 (commit)
via 252051df7378d1d23e726a8757efb4e6af22d1fa (commit)
from f1bb46172bdc90db00a08c2c78e169952fd22feb (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 bff4fac7b78fe8573527c8733f8364af351d23a9
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date: Thu Dec 19 11:05:54 2013 +0100
Bug 11279: (follow-up) remove unnecessary check on number of quotes
It is not necessary to process the case where the number of quotes
is just one, as int(rand(1)) will always produce 0, which is a valid
offset.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 8ee6e4290e2d1ca4fea53363b7ca3275cb0e6f58)
commit ae55277499f06ae0d3e2973349d1f34cf7f60ec8
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Thu Dec 12 00:54:08 2013 +0100
Bug 11279: Improve how a new quote of the day is selected
If there is a gap in the id sequence for the quotes table, it
is possible that no new quote will be selected. This will happen
particularly when a lot of the older quotes with low ids have been
deleted.
This patch improves the selection of a new quote.
To test:
- Load sample quotes
- Delete the first half of the quotes.
Note: With 34 quotes, delete the quotes with ids from 1-17
- Activate the QuoteOfTheDay system preference
- Check if a quote is displayed in OPAC
- Reload the page a few times, no quote should be displayed
Note: make sure you don't have a quote with the current
date in your quotes table before running those tests
- Run 'perl t/db_dependent/Koha.t'
Note: requires sample quotes!
- Apply patch
- Reload the OPAC start page
- Verify a quote was now picked
- Run 'perl t/db/dependent/Koha.t' again - all tests should still pass
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
Patch modified to use "LIMIT 1 OFFSET ?" rather than "LIMIT ?, 1"; the
latter construction does not work in PostgreSQL.
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit 5a3db23362eb076a2f6d5a87f87b29f4a436a14f)
Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
commit 252051df7378d1d23e726a8757efb4e6af22d1fa
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Thu Dec 12 20:50:52 2013 +0100
Bug 11279: regression test for picking random quote
This patch adds new tests based on the db_dependent tests in
t/db_dependent/Koha.t for GetDailyQuote().
Tests run on an empty database and don't require Test::Deep.
Tests will fail before applying the patch for 11297, but will
pass after applying the patch.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
(cherry picked from commit f874eec0793f83c3c0f1461fecb878ca57ed701b)
Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
-----------------------------------------------------------------------
Summary of changes:
C4/Koha.pm | 23 +++++------
t/db_dependent/Koha/GetDailyQuote.t | 77 +++++++++++++++++++++++++++++++++++
2 files changed, 87 insertions(+), 13 deletions(-)
create mode 100644 t/db_dependent/Koha/GetDailyQuote.t
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list