[koha-commits] main Koha release repository branch master updated. v3.12.00-beta1-1309-gf17fab3
Git repo owner
gitmaster at git.koha-community.org
Fri Oct 11 06:13:07 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, master has been updated
via f17fab3ed9b4cef1bd8932376fbdc8733d899fab (commit)
via 4ce18e2a86b0af7548caaeaf26f7571d2244e35e (commit)
via 4458e86865d09196591646e29a319958c8680578 (commit)
from 728670f467484f8ec84aa4f9edebf1446f13b912 (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 f17fab3ed9b4cef1bd8932376fbdc8733d899fab
Author: Galen Charlton <gmc at esilibrary.com>
Date: Fri Oct 11 04:20:18 2013 +0000
Bug 10689: add regression test
This adds a test to verify that the public subscription
note is returned by a subscription search.
To test:
[1] Verify that t/db_dependent/Bookseller.t passes.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
commit 4ce18e2a86b0af7548caaeaf26f7571d2244e35e
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date: Wed Sep 18 15:11:46 2013 +0200
Bug 10689: make subscription public note appear when making a new order from a subscription
Test by creating a new order to a basket using "From a subscription" link
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Comment: Works as described. No koha-qa errors
When creating a new basket from a susbcription, public note shows
on Notes column.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
Tiny enhancement, passes tests and QA script.
Works as described.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
commit 4458e86865d09196591646e29a319958c8680578
Author: Fridolyn SOMERS <fridolyn.somers at biblibre.com>
Date: Tue Aug 6 12:13:34 2013 +0200
Bug 10689: make public note appear in subscriptions search
In a serials module, when searching subscriptions, the results table as
a "Notes" column.
In TT code, you see that it tries to display public note
"subscription.notes" and internal note "subscription.internalnotes".
The internal note is displayed well but not the public note.
You can see the 2 notes in serial details in summary tab.
The problem commes from the SQL query. A join is perform on subscription
and biblio, both containing a "notes" column.
This patch solves the problem by using a alias in query for both columns
(biblio.notes is acutally not used in template but could be).
Test plan :
- Edit a subscription
- Add public and internal notes. For example : "too busy" and "on holiday"
- Perform a subscription search that returns this subscription
=> "Notes" column contains both notes. For example : "too busy (on holiday)"
- Test with only public note
- Test with only internal note
Works as described.
Signed-off-by:Mathieu Saby <mathieu.saby at uhb.fr>
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
All tests and QA script pass.
Works as described, fixes a bug as the templates show that
the intention was to display both notes in the column.
Signed-off-by: Galen Charlton <gmc at esilibrary.com>
-----------------------------------------------------------------------
Summary of changes:
C4/Serials.pm | 9 ++++++++-
.../prog/en/modules/acqui/newordersubscription.tt | 2 +-
.../prog/en/modules/serials/serials-search.tt | 2 +-
t/db_dependent/Bookseller.t | 17 +++++++++++++----
4 files changed, 23 insertions(+), 7 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list