[koha-commits] main Koha release repository branch master updated. v3.20.00-976-ga636f22

Git repo owner gitmaster at git.koha-community.org
Thu Oct 29 16:03:44 CET 2015


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  a636f2223cec1c190b7379529eba88c8c584ab70 (commit)
       via  f461de794b2538319f67b266e2c90228f46eb3ad (commit)
       via  e4cc11ee13b9f3b78a4c5f32920ecb073ceeb268 (commit)
       via  a97d3e9513e38a8b4d6f09f07ae2645cb9ae1a51 (commit)
       via  4307591f222f8ee337f854472c18ac364402fe6d (commit)
       via  b70e1cd0530bafb9f97973c65395ec3e31c2e2d2 (commit)
       via  421fae24f1cdcbe0ccc18de784454d0ca07ebe90 (commit)
      from  201935cb6244e652f6a5d3b239a88ad869af6e2f (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 a636f2223cec1c190b7379529eba88c8c584ab70
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Oct 28 08:41:19 2015 +0000

    Bug 14931: (qa followu-up) small improvements
    
    - Avoid 1 call to dt_from_string in some cases
    - Do not use $_
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit f461de794b2538319f67b266e2c90228f46eb3ad
Author: Marc Véron <veron at veron.ch>
Date:   Thu Oct 15 13:01:50 2015 +0200

    Bug 14931: (Follow-up) Validate date entry in contracts
    
     Go to Home > Acquisitions, search for an active vendor and view the vendor
    - Click on 'Contracts' at the left
      (admin/aqcontract.pl)
    - Create and edit contracts and verify that it works OK.
      (Note: there is no help for this page, see Bug 14929)
    
    Amended for comment #7  25.10.2015 / mv
    
    Signed-off-by: Jesse Weaver <pianohacker at gmail.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit e4cc11ee13b9f3b78a4c5f32920ecb073ceeb268
Author: Marc Véron <veron at veron.ch>
Date:   Wed Sep 30 18:50:42 2015 +0200

    Bug 14931: Remove C4::Dates from files admin/aq*.pl
    
    This patch removes C4::Dates from:
    
    - admin/aqbudgets.pl
    - admin/aqplan.pl
    - admin/aqcontract.pl
    
    To test:
    
    - Apply patch
    
    - Go to Home > Adminstration > Budgets > All funds
      (admin/aqbudgets.pl)
    - Verify that everything displays as before
    
    - Edit one of the funds and click on 'Planning - Plan by months'
      (admin/aqplan.pl)
    - Verify that planning works as before
    
    - Go to Home > Acquisitions, search for an active vendor and view the vendor
    - Click on 'Contracts' at the left
      (admin/aqcontract.pl)
    - Create and edit contracts and verify that it works OK.
      (Note: there is no help for this page, see Bug 14929)
    
    Signed-off-by: Hector Castro <hector.hecaxmmx at gmail.com>
    Works as advertised. Tested with dateformat syspref
    Dates in planning by months not needed to be on iso format, aqplan.pl
    uses instead Date::Calc
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit a97d3e9513e38a8b4d6f09f07ae2645cb9ae1a51
Author: Marc Véron <veron at veron.ch>
Date:   Tue Oct 27 20:06:11 2015 +0100

    Bug 14977: Followup to fix issue with NewsChannels.t
    
    The problem was that get_opac_new returns dates already syspref-formatted.
    
    To test:
    - Apply patch
    - Verify that t/db_dependent/NewsChannels.t does not fail.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit 4307591f222f8ee337f854472c18ac364402fe6d
Author: Marc Véron <veron at veron.ch>
Date:   Thu Oct 8 00:13:01 2015 +0200

    Bug 14977: Remove C4::Dates from files t/db_dependent/*.t
    
    This patch removes C4::Dates from following files:
    
    -  t/db_dependent/Budgets.t
    -  t/db_dependent/Holds.t
    -  t/db_dependent/Koha_template_plugin_KohaDates.t
    -  t/db_dependent/Log.t
    -  t/db_dependent/NewsChannels.t
    
    To test:
    - First run all tests above
    - Apply patch
    - Then run tests again. The results should be the same.
      (Note: I get 2 fails with NewsChannels.t, they are the same with
       and without patch).
    
    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    
    NewsChanell pass for me before patch, no after.
    Followup follows :)
    Rest of tests pass
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit b70e1cd0530bafb9f97973c65395ec3e31c2e2d2
Author: Marc Véron <veron at veron.ch>
Date:   Tue Oct 27 17:50:21 2015 +0100

    Bug 14903: (followup)Remove C4::Dates from circ/circulation.pl
    
    Follow up to fix QA issues from comment #15
    ...and amended for comment #18
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit 421fae24f1cdcbe0ccc18de784454d0ca07ebe90
Author: Marc Véron <veron at veron.ch>
Date:   Fri Oct 9 18:48:30 2015 +0200

    Bug 14903: Remove C4::Dates from circ/circulation.pl (and more)
    
    Patch to remove deprectated C4::Dates from:
    - circ/circulation.pl
    - C4/Circulation.pm
    - koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
    
    To test:
    - Apply patch
    - Go to the checkout site (Home > Circulation > Checkouts)
    - Verify that data displays properly (including for users with a card
      that expires in the  near future, see syspref 'NotifyBorrowerDeparture')
    - Search for regressions
    - prove -v t/db_dependent/Circulation.t
    
    (Amended following comment #9 / #11  25.10.2015 /mv)
    
    Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

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

Summary of changes:
 C4/Circulation.pm                                  |   23 ++++++--------
 admin/aqbudgets.pl                                 |    1 -
 admin/aqcontract.pl                                |   32 ++++++++++++--------
 admin/aqplan.pl                                    |    1 -
 circ/circulation.pl                                |   27 ++++++++---------
 .../prog/en/modules/admin/aqcontract.tt            |    9 +++---
 .../prog/en/modules/circ/circulation.tt            |   12 ++++----
 t/db_dependent/Budgets.t                           |    1 -
 t/db_dependent/Holds.t                             |    3 +-
 t/db_dependent/Koha_template_plugin_KohaDates.t    |    6 +---
 t/db_dependent/Log.t                               |    8 ++---
 t/db_dependent/NewsChannels.t                      |   11 ++++---
 12 files changed, 67 insertions(+), 67 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list