[koha-commits] main Koha release repository branch master updated. v3.14.00-627-g69216d2

Git repo owner gitmaster at git.koha-community.org
Sat Mar 22 14:33:21 CET 2014


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  69216d2217519cd6d8ad97710236251abdd89ae9 (commit)
      from  42ba4d42ff80bb77a69363c37c4a2252a104d41d (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 69216d2217519cd6d8ad97710236251abdd89ae9
Author: Jacek Ablewicz <abl at biblos.pk.edu.pl>
Date:   Mon Feb 24 17:37:43 2014 +0100

    Bug 11352: fix unexpected data loss issues with batch patron deletion/anonymization
    
    Batch patron deletion/anonymization tool has some issues:
    
    1) If 'dateformat' setting in I18N/L10N system preferences is set
    to anything other then 'iso' (eg. 'metric'), bulk deletion when using
    'expiration date is before' criterion is not working properly. Date
    entered in this field will be efectivelly ignored (or possibly
    sometimes wrongly interpreted as different date, in other format)
    on the final patron deletion stage. This may result in deleting
    (or moving to trash) more borrower records then intended.
    2) Bulk/batch patron deletion should skip borrowers with
    nonzero account balance (ones with oustanding fines or credits)
    3) This tool shouldn't offer to choose as deletion criterion
    those patron categories which have category_type set to 'S'
    (= staff patron categories)
    
    This patch fixes above mentioned problems. It also adds an option
    to "test run" patron batch deletion, and makes this option
    the default choice in "warning" stage.
    
    Test plan:
    - prepare test database with some patron records (at least 2,
    the more the better) set up in such a way that they will be vulnerable
    to issues 1 & 2
    - confirm issues 1,2
    - restore test database
    - apply patch
    - ensure issues 1 & 2 are no longer present - first by using new "test
    run" option: for #1, record counts in "warning" stage and "final" stage
    should be now the same; for #2, observe that patron records with nonzero
    balance are now excluded from deletion
    - redo the tests, this time choosing "delete permanently" and "move
    to trash" instead of "test run"
    - test #3 by changing "Category type" to "S" in some
    test patron categories - after that, those categories should no
    longer be choosable as deletion criteria.
    
    Signed-off-by: Magnus Enger <digitalutvikling at gmail.com>
    Tested with dateformat = dd/mm/yyyy. I tested with two expired patrons,
    one with fines and one without. Before the patch a lot of unexpected
    patrons were deleted along with the expected ones. After
    applying the patch only the expired patron was deleted, not the
    one with fines. The test run and the "real" run reported correct numbers.
    
    The patch also makes sure no patron categories with category_type = S
    are suggested for batch deletion.
    
    Note: The ergonomics of the "Batch delete/anonymize" tool is hardly
    optimal, but this patch fixes a real, data-loosing bug, so let's
    deal with the ergonomics later.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 .../prog/en/modules/tools/cleanborrowers.tt        |   27 ++++++++++------
 tools/cleanborrowers.pl                            |   33 +++++++++++---------
 2 files changed, 36 insertions(+), 24 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list