[koha-commits] main Koha release repository branch 17.11.x updated. v17.11.14-7-gd504715

Git repo owner gitmaster at git.koha-community.org
Wed Jan 30 10:06:23 CET 2019


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.11.x has been updated
       via  d504715ea7db808ed2adc5c75f7fd0488715487d (commit)
       via  35e696ec85a5fd66cc9872a426154ed70781c4d7 (commit)
       via  477347b93e755020a2f346306e039c5c92584344 (commit)
       via  f8324ec2f8ba8863a504286d4080729ce749b437 (commit)
       via  3ea288aa2d5ff46cace2c27cff2d3df105adcc3c (commit)
       via  f2922920e4fc2c2c22f2bb38d70c5b671a44a852 (commit)
       via  90fc1355d909874f47644117c998d833b9e5bf79 (commit)
      from  577d559b38b95c1ed4d7d302e8f83c636214b5ca (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 d504715ea7db808ed2adc5c75f7fd0488715487d
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Tue Jan 8 14:32:19 2019 +0100

    Bug 22082: ambiguous column in patron stats
    
    In Reports > Patrons statistics, the combination of activity and extended attributes generates an invalid SQL query with error :
    ERROR 1052 (23000): Column 'borrowernumber' in IN/ALL/ANY subquery is ambiguous
    
    This is because FROM contains tables borrowers and borrower_attributes both containing borrowernumber column.
    
    Test plan :
    1) Be sure to have a patron extended attribute
    2) Be sure you have some recent circulation activity
    3) Go to Reports > Patrons statistics
    4) Select 'active' in 'Patron activity' and 1 years of activity
    5) Check 'Row' in 'Patron category'
    6) Check 'Columns' in a extended attribute
    7) Submit
    8) Without patch you have no results, with patch you have exepected results
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 8d2324e6de6f639904d188cd432e14add4874779)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit bccbb54491d3722411cee786000959abade33aa7)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit cf3df9058af418d70b02085f8bf724989567cee1)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 35e696ec85a5fd66cc9872a426154ed70781c4d7
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Wed Nov 21 13:40:35 2018 +0000

    Bug 21491: Display 'Yes' as selected if no refundlostitemfee rules are set
    
    To test:
    1 - Delete all refund_lost_item_fee_rules form the db
        delete from refund_lost_item_fee_rules;
    2 - View the Standard circ rules - Default lost item fee refund shows as
    'No'
    3 - View circ rules for a single library, shows as 'Default (Yes)'
    4 - Return a lost item, it actually does refund (this is in the
            LostItemFeeRules code, default 1 if not set)
    5 - Apply patch
    6 - Rule shows as 'Yes'
    7 - View a specifc rule, still says Default (Yes)
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 7028cb8aea3f03f120c54269554953ec7475cdd8)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit b940797800ae4546c455aba7fa6570918a97b6af)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit 86cc395636c2f45e214c9fbae70d244b4bf6f338)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 477347b93e755020a2f346306e039c5c92584344
Author: helene hickey <hickeyhe at wgc.school.nz>
Date:   Tue Jan 15 03:47:58 2019 +0000

    Bug 22054: Updated the error message for attempting to renew an onsite checkout
    
    To test:
    
    1) Enable OnSiteCheckouts system preference
    2) Checkout an item checking the onsite checkout checkbos in more
    options
    3) Go to circulation > pending on-site checkouts
    4) Verify the checkout shows
    5) Go to circulation > renew
    6) Try to renew the on-site checkout using the barcode
    7) Verify display:
    
    onsite_checkout  Continue without renewing
    
    8) apply the patch and reload page with error message
    9) verify display:
    
    Item cannot be renewed because it's an onsite checkout
    
    Signed-off-by: David Nind <david at davidnind.com>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 4eab8308353948c81d9a05978584073354cb43e5)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 3dd7d9dfad0052761e4a3e7a93fce6e7d8055c93)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit c0b2ad7da5a7ea79dbb79ad1b57c96a01431bb3a)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit f8324ec2f8ba8863a504286d4080729ce749b437
Author: Ethan Amohia <ethan.amohia at gmail.com>
Date:   Tue Jan 15 21:45:01 2019 +0000

    Bug 22095: Replaced dead links with working links
    
    I have updated the broken link everywhere it occurs.
    
    Confirm that the new link works in the installer.
    
    Signed-off-by: David Nind <david at davidnind.com>
    
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit dc8e663c9eeb54ebd26ecf9c7d5ef7c953db283c)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 726c0e4bc4216c3c53780169635987bdc1071720)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit d82228876363bbfa461f84b70a9078c48d292bb8)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 3ea288aa2d5ff46cace2c27cff2d3df105adcc3c
Author: Olivia Lu <olivialokm at gmail.com>
Date:   Tue Jan 15 03:52:30 2019 +0000

    Bug 21866: Changed "report" to "plugins" for the warning "This report was written for an older version of Koha. Run at your own risk"
    
    To find this change:
    
    1. Enable use plugins in global system preferences
    2. Go to tools then click tool plugins
    3. Verify that this string now says plugins
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 5e2e101037bb2bfb8579fcee331388e56658db84)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 2c6b8230b8fa7dd0a7746327c126565fc8b47dc9)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit c6762426d47e1249c8707db0daab39bf5eca8439)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit f2922920e4fc2c2c22f2bb38d70c5b671a44a852
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Wed Dec 26 14:55:53 2018 -0300

    Bug 21912: (QA follow-up) Do not delete existing patrons
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 8e6c789255a60958ed12eb399b1acd21819fd40c)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 9b6b80488a6b9315b201a3f6d11d0341e58122b5)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit 91fbe63d18c7370a1105c21de890fc81c7583f3f)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 90fc1355d909874f47644117c998d833b9e5bf79
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Thu Nov 29 11:59:05 2018 -0300

    Bug 21912: Add tests for Koha::Objects->search
    
    This patch adds simple tests for the current behaviour. Specifically the
    return values in both scalar and list context.
    
    To test:
    - Apply this patch
    - Run:
      $ prove t/db_dependent/Koha/Objects.t
    => SUCCESS: Tests pass
    - Sign off :-D
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 7669697732e3abf3b2c5a32446115d277be7342e)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 464dcd7d9e2e7ba4e604ef0e7b5665dd8cd8b9b3)
    
    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit dc096dafe6453bbb7412da799f5acc0fff5da781)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

-----------------------------------------------------------------------

Summary of changes:
 .../optional/marc21_holdings_coded_values.sql      |    2 +-
 .../optional/marc21_holdings_coded_values.txt      |    2 +-
 .../en/optional/marc21_holdings_coded_values.sql   |    2 +-
 .../en/optional/marc21_holdings_coded_values.txt   |    2 +-
 .../optional/marc21_holdings_coded_values.sql      |    2 +-
 .../optional/marc21_holdings_coded_values.txt      |    2 +-
 .../facultatif/marc21_holdings_coded_values.sql    |    2 +-
 .../facultatif/marc21_holdings_coded_values.txt    |    2 +-
 .../optional/marc21_holdings_coded_values.sql      |    2 +-
 .../optional/marc21_holdings_coded_values.txt      |    2 +-
 .../2-Valgfritt/marc21_holdings_coded_values.sql   |    2 +-
 .../2-Valgfritt/marc21_holdings_coded_values.txt   |    2 +-
 .../optional/marc21_holdings_coded_values.sql      |    2 +-
 .../optional/marc21_holdings_coded_values.txt      |    2 +-
 .../prog/en/modules/admin/smart-rules.tt           |    4 +--
 .../intranet-tmpl/prog/en/modules/circ/renew.tt    |    3 ++
 .../prog/en/modules/plugins/plugins-home.tt        |    2 +-
 reports/borrowers_stats.pl                         |    4 +--
 t/db_dependent/Koha/Objects.t                      |   30 ++++++++++++++++++--
 19 files changed, 50 insertions(+), 21 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list