[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.12-6-ge6d956e

Tue Jun 26 10:04:21 CEST 2018

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  e6d956e7d6d4e0197a737b814a3df514c0b8875c (commit)
       via  1e25709f9ce7ca10f2f14c9ae93ee5b5578c55b0 (commit)
       via  6538e4c3165837f10499aa99cc5ab2b00bc6f211 (commit)
       via  58a9df3808f85d4a53a002a2619cee07571966c2 (commit)
      from  628f77746e8d471ec288574c1437dc5e56b0b657 (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 e6d956e7d6d4e0197a737b814a3df514c0b8875c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Jun 12 13:12:25 2018 -0300

    Bug 20922: Remove use of Koha::Number::Price in updatedatabase.pl

    Koha::Number::Format->round use Number::Format->round with a precision=2
    We should use it directly instead of Koha::* modules. It will avoid the
    DB entry to fail because schema changes.

    From the koha-devel list:

    http://lists.koha-community.org/pipermail/koha-devel/2018-June/044608.html

    16.06.00.042

    Upgrade to 16.06.00.041 done (Bug 14629 - Add aggressive ISSN matching
    feature equivalent to the aggressive ISBN matcher)
    DBD::mysql::st execute failed: Unknown column 'me.p_sep_by_space' in
    'field list' [for Statement "SELECT `me`.`currency`, `me`.`symbol`,
    `me`.`isocode`, `me`.`timestamp`, `me`.`rate`, `me`.`active`,
    `me`.`archived`, `me`.`p_sep_by_space` FROM `currency` `me` WHERE (
    `active` = ? )" with ParamValues: 0=1] at
    /usr/local/share/perl/5.24.1/DBIx/Class/Storage/DBI.pm line 1836.
    DBIx::Class::Storage::DBI::_dbh_execute(): Unknown column
    'me.p_sep_by_space' in 'field list' at
    /inlibro/git/koha-csf-prod-inlibro/Koha/Objects.pm line 209

    Basically, the update code uses Koha::Number::Price, which in full
    modern object mode goes for its newly added *p_sep_by_space* _in the
    18.05 code_.  But the DB doesn't have it yet (it comes with 17.12.00.022).

    Signed-off-by: Blou <philippe.blouin at inlibro.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 93de5885098b9982a3ab9ad73ff73111d71ac3ef)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 878f645c5dd6433cb78402b0658cca75e1b4e561)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 1e25709f9ce7ca10f2f14c9ae93ee5b5578c55b0
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Jun 1 11:00:57 2018 +0200

    Bug 18821: (QA follow-up) Last tweaks for performance

    [1] passing unsafe has no use since it is a scalar, removed it to unconfuse
    [2] remove caching when pref is disabled
    [3] caching userid removes the need for calling Patron->find each time
    [4] subsequent changes in unit test
    [5] cosmetic renames to move from session to daily basis (changed dev angle)

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    First call going thru Koha::Patron takes about 0.0150 sec.
    Subsequent calls only use caching and take about 0.0006 sec.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 2bae585fcd4d5a5e31c0503cc8ea1cd3ea0f29ba)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit f0159f8f5712a526f2c1af44b4f9ecc0d82c15a6)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 6538e4c3165837f10499aa99cc5ab2b00bc6f211
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Sat May 26 06:40:24 2018 +0000

    Bug 18821: Convert to using cache with date checking

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 1e19bb32844dedc3d2d4d506e22f60bfac2b4057)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 30ea1a4b4b783635e817dd3c06a3d0c881961d59)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 58a9df3808f85d4a53a002a2619cee07571966c2
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Mon May 7 17:29:12 2018 +0000

    Bug 18821: TrackLastPatronActivity is a performance killer

    Test Plan:
    1) Apply this patch
    2) Start a new session ( a private browser window works well )
    3) Note the lastseen column in the borrowers table is updated
    4) Browse a few pages, not the lastseen column is not updated again
    5) Close the browser window and repeat steps 2-4
    6) prove t/db_dependent/Auth.t

    Signed-off-by: Charles Farmer <charles.farmer at inLibro.com>
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 0aa5eec2332f0e0ae7f6adefebc274b02f8830c5)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 68d84e0d5441e5eb1ca17b3eb1515424ed2dd04d)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 C4/Auth.pm                             |   37 +++++++++++++++++++++-----
 installer/data/mysql/updatedatabase.pl |   45 ++++++++++++++++----------------
 t/db_dependent/Auth.t                  |   44 ++++++++++++++++++++++++++++++-
 3 files changed, 96 insertions(+), 30 deletions(-)

hooks/post-receive
-- 
main Koha release repository