[koha-commits] main Koha release repository branch 3.12.x updated. v3.12.00-beta1-201-gf1af1db

[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, 3.12.x has been updated
       via  f1af1db21d61972f0e6a7b0c5ebd6630569a827d (commit)
      from  88f2dd765f3c992d32083f5d9dd4a94c20d0219f (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 f1af1db21d61972f0e6a7b0c5ebd6630569a827d
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Wed Apr 3 13:05:52 2013 -0400

    Bug 8770 - amount does not equal amountoutstanding on a fine with no payments or credits
    
    If a lost item fee is owed and partially paid off when an item is
    returned and a refund is processed, Koha tries to pay off existing
    fees before adding any leftover balance as a credit on the account.
    
    However, those fee payments aren't actually processed, due to a bug
    where the accountnumber was quoted as a string literal, and thus the
    for the fee payment would fail. This did not result in an DB error,
    as the query was still valid SQL. Checking the return value of the
    query would have revealed that the accountline had not been updated.
    
    History:
    This bug was introduced on April 23, 2007 with the commit
    'reintroducing fixaccountforlostandreturned as requested by rosalie'.
    Commit id 111d590e9cdaf801256822e23f170bfc2365338f
    
    On July 30, 2009 the error was flagged with a FIXME and remained
    in that state until now.
    Commit id 51e8fc2cb6a0cf79f19206c822a4361aeca6f63f
    
    Test plan:
    1) Create a test patron
    2) Check out an item to that patron and give it a due date in the past
    3) Run fines.pl to generate the fine for the item
    4) Mark the item long overdue on the item tab (not in edit items)
    5) Pay for the lost item
    6) Check the item back in
    7) Note the incorrect fines. Only the lost fee balance  due is refunded,
       not the entire lost fee, but no other fees are paid off.
    8) Apply the patch
    9) Repeat steps 1-6, then not the fines are paid correctly
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Totel due is correct after applying the patch.
    All tests and QA script pass.
    
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 C4/Circulation.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
main Koha release repository