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, 17.05.x has been updated
       via  d4897d2d6abfe6943ff8bdaae53cfb8b5a162876 (commit)
       via  8fdbaee0e4678a3e5d3f9a64ee6bb5d6ba9118ea (commit)
       via  c2d7c71c91bf5b3579abdca5937ddb0c611aaca8 (commit)
       via  4177b81d49d27720f7a2b67d1b3a3524d8d78db5 (commit)
      from  1f34c499c8bedc260eb07930d59caa97f03f1736 (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 d4897d2d6abfe6943ff8bdaae53cfb8b5a162876
Author: Jonathan Druart <>
Date:   Mon Jan 22 17:04:28 2018 -0300

    Bug 19975: Fix search by tags at the OPAC
    This bug has certainly be caused by
      commit 091d6c513bcbee224ff06477e79be48cea7fe825
        Bug 17843: Replace C4::Koha::getitemtypeinfo with Koha::ItemTypes
    The reason is quite simple, in Koha::ItemType->translated_description
    (called in l.229) there is an eval to know if we have
    access to the translated description of the item type, to avoid
    retrieving it again.
    The evaluation of $@ later in is supposed to test the 2
    eval made few lines before (a "normal" search, without tags), but
    $@ contains the error message from the *last* eval command.
    So we are raising an error that have been correctly handled in
    Test plan:
    At the OPAC, click Tag cloud, then click any of the tags
    => Without the patch you get
    Koha::ItemType::get_column generated this error: 
DBIx::Class::Row::get_column(): No such column 'translated_description' on 
Koha::Schema::Result::Itemtype at /usr/share/koha/lib/Koha/ line 307
    => With the patch applied the page is correctly displayed
    Signed-off-by: Claire Gravely <>
    Signed-off-by: Marcel de Rooy <>
    Signed-off-by: Jonathan Druart <>
    Signed-off-by: Nick Clemens <>
    (cherry picked from commit fc172d3252b8587006f7d6b4d747ce2b339ea803)
    Signed-off-by: Fridolin Somers <>

commit 8fdbaee0e4678a3e5d3f9a64ee6bb5d6ba9118ea
Author: Jonathan Druart <>
Date:   Mon Jan 22 17:18:59 2018 -0300

    Bug 19985: Make TestBuilder.t pass even if default_circ_rules is not empty
    This table cannot contain more than 1 row, so we need to remove its data
    before trying to generate a new entry.
    Test plan:
    Set a default circ rule
     prove t/db_dependent/TestBuilder.t
    should return green
    Without this patch you get:
        #   Failed test 'TestBuilder should be able to create an object for 
every source'
        #   at t/db_dependent/TestBuilder.t line 78.
        #          got: '1'
        #     expected: '0'
        # The following sources have not been generated correctly: 
        # Looks like you failed 1 test of 1.
    Signed-off-by: Mark Tompsett <>
    Signed-off-by: Jonathan Druart <>
    Signed-off-by: Nick Clemens <>
    (cherry picked from commit 55d8ea59e7964d05ffb68e7bfcf14d6556ed8739)
    Signed-off-by: Fridolin Somers <>

commit c2d7c71c91bf5b3579abdca5937ddb0c611aaca8
Author: Kyle M Hall <>
Date:   Mon May 8 16:47:06 2017 +0000

    Bug 18477: Populate article_requests table values for AR_PENDING notices
    The default AR_PENDING letter does not print populate 
<<article_requests.*>> fields.
    Test Plan:
    1) Enable article requests ( syspref: ArticleRequests => Enable, Circ and 
fine rules ALL/ALL (or a given category/itemtype): Article requests => Yes )
    2) Search for a title, click in to catalog detail, click 'Request Article' 
    3) Select Patron
    4) Fill out request information
    5) Click 'Place request'
    6) Go to notices in patron record, inspect 'Article Request Received' 
notice. You should see the request information from step 4, but it doesn't 
    7) Apply this patch
    8) Repeat steps 2-6, not the fields now display!
    Signed-off-by: Eric Phetteplace <>
    Signed-off-by: Mark Tompsett <>
    Signed-off-by: Jonathan Druart <>
    Signed-off-by: Nick Clemens <>
    (cherry picked from commit e1a61134eb921760ebe3ac992a151591bb3977bf)
    Signed-off-by: Fridolin Somers <>

commit 4177b81d49d27720f7a2b67d1b3a3524d8d78db5
Author: Jonathan Druart <>
Date:   Tue Jan 2 14:36:43 2018 -0300

    Bug 18477: Add tests
    Signed-off-by: Mark Tompsett <>
    Signed-off-by: Jonathan Druart <>
    Signed-off-by: Nick Clemens <>
    (cherry picked from commit 2a0423778e73467beeb57379db49b49b811894aa)
    Signed-off-by: Fridolin Somers <>


Summary of changes:
 Koha/           |    1 +
 opac/              |    2 +-
 t/db_dependent/ArticleRequests.t |   25 +++++++++++++++++++++++--
 t/db_dependent/TestBuilder.t     |    1 +
 4 files changed, 26 insertions(+), 3 deletions(-)

main Koha release repository
koha-commits mailing list

Reply via email to