[koha-commits] main Koha release repository branch master updated. v16.11.00-45-g2fcceb7

[Git repo owner]

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  2fcceb7f402abdc7ca30d72d11e1815a1309b41b (commit)
       via  4b0f08e0e1bb785fc08a1f63e99ebcdbbc75a50e (commit)
       via  8ed2cd10bebf5df7cbc4834aada7e6b266e37507 (commit)
       via  b6d5748c001febc5acd67938d12ba25844d11fbc (commit)
       via  ed9cf81c3aa8fd82f9347556364e9d2e6cef8a82 (commit)
       via  30dd8f99204837e2e76015badd93025599cd552c (commit)
       via  e716299c32e201e2cf442a937a7fa7a6a47f6907 (commit)
      from  427ba8b0c7f7a976e3c90a4f811836532e0f3875 (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 2fcceb7f402abdc7ca30d72d11e1815a1309b41b
Author: Kyle M Hall <kyle at kylehall.info>
Date:   Fri Dec 9 12:56:49 2016 -0500

    Bug 15896 [QA Followup] - Supress warn when running unit tests
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 4b0f08e0e1bb785fc08a1f63e99ebcdbbc75a50e
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Nov 4 13:05:18 2016 +0100

    Bug 15896: [QA Follow-up] Fix parameter name in POD
    
    Also adjust typo feees.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 8ed2cd10bebf5df7cbc4834aada7e6b266e37507
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Nov 4 10:40:38 2016 +0000

    Bug 15896 - Unit Tests
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit b6d5748c001febc5acd67938d12ba25844d11fbc
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Nov 4 11:39:12 2016 +0100

    Bug 15896: [QA Follow-up] Add accountlines_id parameter in paycollect
    
    We can solve the minor problem reported on 15906 now by using the
    accountlines_id parameter of this report.
    
    Test plan:
    [1] Add two manual fines (say 20 and 30).
    [2] Pay the second one in full, and check that the first one is not paid
        first. So the 20 should remain and not the 30 with 20 outstanding.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit ed9cf81c3aa8fd82f9347556364e9d2e6cef8a82
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Feb 24 12:36:20 2016 +0000

    Bug 15896 - Use Koha::Account::pay internally for makepayment
    
    This is the second patch in a series to unify all payment functions into
    a single mathod
    
    Test Plan:
    1) Apply this patch
    2) prove t/db_dependent/Accounts.t
    3) Test fine payment via the "Pay" button
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    
    Followed test plan, works as expected.
    Signed-off-by: Marc Veron <veron at veron.ch>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 30dd8f99204837e2e76015badd93025599cd552c
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Nov 4 09:46:08 2016 +0100

    Bug 15906: [QA Follow-up] Correct a partial payment
    
    The amount in accountlines should be saved as a negative amount, but
    amount outstanding is correct.
    
    Note: This routine is actually used only once (in paycollect.pl). And is
    only cosmetically tested in Accounts.t.
    
    Test plan:
    [1] Create a manual invoice, Fine, say 20.
    [2] Click on the Pay button next to this fine, and pay 1.
    [3] Pay again, using 'Pay amount' button, and pay 2.
    [4] Pay again, using 'Pay' on the same line, the full remaining amount.
    [5] Verify that you see -1, -2 and -17 on the Accounts tab.
    [6] Run t/db_dependent/Accounts.t
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit e716299c32e201e2cf442a937a7fa7a6a47f6907
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Feb 24 15:51:07 2016 +0000

    Bug 15906 - Remove use of makepayment in paycollect.pl
    
    Test plan:
    1) Apply this patch
    2) Make a payment in full using the "Pay" button
    3) Note payment succeeds
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

Summary of changes:
 C4/Accounts.pm            |   92 ++-------------------------------------------
 Koha/Account.pm           |   81 ++++++++++++++++++++++++++++++++++-----
 members/paycollect.pl     |    8 +++-
 t/db_dependent/Accounts.t |   14 ++++++-
 4 files changed, 92 insertions(+), 103 deletions(-)


hooks/post-receive
-- 
main Koha release repository