[koha-commits] main Koha release repository branch 3.20.x updated. v3.20.03-55-g3d74a82

Tue Sep 22 19:50:38 CEST 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, 3.20.x has been updated
       via  3d74a820b23b7b912dea969f88a865bc714be50c (commit)
       via  cf88d263353ef5305d8e34097310eeeac520dba1 (commit)
       via  970af1b6abe0cc61374fc372c98365c6e04e6e94 (commit)
       via  715a7bf4f5ff7c05e310a5e98f7cdc177afd3e8f (commit)
       via  085272aec0af72e3178f2c6e6ae53f6ea1d48bf1 (commit)
       via  ad0375f740ff4b035a5a43c75f30d578d0d7d251 (commit)
       via  74e6691cf84e51b20832b45cf7520aec85788771 (commit)
       via  0a0f8b09d947aabef21c6cb79e796786ce3be7d5 (commit)
       via  3caa3703c272be52b6cf161a086271ca9be3eb01 (commit)
       via  04319600c32fc3145cfd2457258e97174e4f2cd6 (commit)
       via  b609ddc5dbb02b631f776e9bc664d88e7c1beeae (commit)
      from  fc2b3400eff5965adf5ffd523213de868ad43ac8 (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 3d74a820b23b7b912dea969f88a865bc714be50c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Sep 2 14:28:52 2015 +0100

    Bug 7634: Prevent permanent_location to be set to NULL if not defined

    The permanent_location is correctly filled when an item is added from
    the cataloguing module (routine set_item_default_location from
    cataloguing/additem.pl).
    But when records are imported, this filled is not managed. It's only on
    editing (_do_column_fixes_for_mod called from ModItem).

    This patch set the permanent_location item fields to the location value for all
    items created, even the imported ones.

    Test plan:
    0/ Do not apply this patch
    1/ Import a record with items using the "Stage MARC for import" tool
    2/ Check the values for the permanent_location in the items table.
    They are set to NULL
    3/ Apply this patch
    4/ Repeat 2 and confirm that now the permanent_location values are set
    to the location values.

    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit a2ca59efb877945f7b2ebc92e7643d4cdaf9d4e1)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit cf88d263353ef5305d8e34097310eeeac520dba1
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Sep 2 14:46:02 2015 +0100

    Bug 7634: Add tests

    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit ae5ee62d598ccb0cfe4fb6cc4a3b2ab6408febd8)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 970af1b6abe0cc61374fc372c98365c6e04e6e94
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Sep 2 14:43:15 2015 +0100

    Bug 7634: Make the tests pass

    This patch just fixed a master bug, if your database already has some
    items.homebranch set to CPL

    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit 704aed77ea03bf8a0a0e4ece457e91fb7937742d)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 715a7bf4f5ff7c05e310a5e98f7cdc177afd3e8f
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Jul 14 11:38:41 2015 +0100

    Bug 6756: Add info messages

    NOTE: Before patch "./misc/cronjobs/batch_anonymise.pl --help" had no
          message, and neither did the anonymizing tool in the staff client.
          After the patch, both had informative messages.

    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit 1acddd0f1b41e9e5f75b75d9d101c8b0879c00c5)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 085272aec0af72e3178f2c6e6ae53f6ea1d48bf1
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Jul 14 11:30:51 2015 +0100

    Bug 6756: Fix bad behaviors if AnonymousPatron is not defined

    There are at least 2 wrong behaviors if the AnonymousPatron pref is not
    defined (0 or empty string).
    1/ If you use the clean borrower tools, you will get a successful
    message when the nothing happened (the history has not been anonymised).
    2/ At the OPAC, if a patron ask for delete his reading history, he will
    get an error message "The deletion of your reading history failed,
    because there is a problem with the configuration of this feature.
    Please help to fix the system by informing your libr    ary of this
    error". IMO this should not happen, the history should be anonymised.

    With this patch, the old_issues.borrowernumber field will be set to NULL
    if the AnonymousPatron pref if not defined.

    Test plan:
    1/ Fill the pref with "" or 0
    2/ At the OPAC, go on the privacy tab and click on the "Immedia deletion" button.
    You should get a green and friendly message. Confirm that the history
    has been anonymised.
    3/ Use the "Batch patron anonymization" tools (tools/cleanborrowers.pl)
    to anonymize the checkout history.
    Confirm that a) it works and b) you get a message.

    Try again with AnonymousPatron set to a valid patron. You should not see
    any changes with the current behaviors.

    NOTE: This patch tweaks C4/Circulation.pm and provides tests.
          applying just this, and running prove success. Reverting just
          C4/Circulation.pm fails, as expected.
          Tested OPAC stuff with both patches applied.

    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit 224b1c7976567511f1d6715784274169e006edcf)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit ad0375f740ff4b035a5a43c75f30d578d0d7d251
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Sep 14 10:28:01 2015 +0100

    Bug 14813: Fix encoding issues on editing help pages

    It has been introduced by bug 11944.

    Test plan:
    1/ Install and set the fr-FR language (or ar-Arab).
    2/ Go on the help page and edit it.
    3/ The textarea should not contain encoding issues with this patch.

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit d671b8661cf02d8b039eb5230ab732fd1fde2301)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 74e6691cf84e51b20832b45cf7520aec85788771
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Jul 10 15:18:48 2015 +0100

    Bug 8723: Update hold_fill_targets, tmp_holdsqueue and linktracker tables

    Same as previous patch for 3 other tables.

    Test plan:
    Same as before but the hold should exist to the 3 tables before the
    move.

    Signed-off-by: Nick Clemens <nick at quecheelibrary.org>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit c7a6745bc0e0241e5f39a12e2ff4e17678288b14)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 0a0f8b09d947aabef21c6cb79e796786ce3be7d5
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Jul 10 15:04:55 2015 +0100

    Bug 8723: Update the reserves table for item-level holds

    If an item is moved from a biblio to another, the holds should be
    updated too.

    See discussion on the bug report for more information.

    Test plan:
    1/ Place a item-level hold on biblio1
    2/ Move the item to biblio2
    3/ Confirm that the hold still exists and point to the biblio2

    This patch should not change the existing behavior for bib-level holds.

    Signed-off-by: Nick Clemens <nick at quecheelibrary.org>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit 9f8f33e351e1ae5810441a0c9ff7226aafb63c8b)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 3caa3703c272be52b6cf161a086271ca9be3eb01
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Fri Jul 10 14:15:42 2015 +0100

    Bug 8723: Cover the existing behavior

    This patch cover the MoveItemFromBiblio subroutine

    Signed-off-by: Nick Clemens <nick at quecheelibrary.org>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit cb4ce631c316a41ac47a17bfdaa4e5591e0e37df)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit 04319600c32fc3145cfd2457258e97174e4f2cd6
Author: Marc Véron <veron at veron.ch>
Date:   Sat Sep 5 16:41:58 2015 +0200

    Bug 14782: Add possibility to search patrons using the username (userid)

    This patch makes it possible to search for users using the username (userid / login name).

    To test:

    - Apply patch
    - Do searches from Home > Patrons
    - Search after a full username or parts of a username with Search fields = Standard and Search fields = Userid
    - Perform the searches from the top bar (expand with [+]) and from the "Filters" part at the left
    - Make sure that other searches behave as before

    Signed-off-by: Joonas Kylmälä <j.kylmala at gmail.com>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit a71cc14a52589f45bc30dad81b8c7fc0ac35d9fc)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

commit b609ddc5dbb02b631f776e9bc664d88e7c1beeae
Author: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date:   Sun Sep 13 18:06:46 2015 +0200

    Bug 14782: Add tests

    Run:
    perl t/db_dependent/Utils/Datatables_Members.t

    Signed-off-by: Tomas Cohen Arazi <tomascohen at unc.edu.ar>
    (cherry picked from commit f704fbebd86e0034cf285b2dd29c0d85cb2086a2)
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>

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

Summary of changes:
 C4/Circulation.pm                                  |    5 +-
 C4/Items.pm                                        |   27 +++-
 C4/Utils/DataTables/Members.pm                     |    3 +-
 edithelp.pl                                        |   15 +--
 .../prog/en/includes/patron-search.inc             |    5 +
 .../prog/en/modules/members/member.tt              |    5 +
 .../prog/en/modules/tools/cleanborrowers.tt        |    4 +
 misc/cronjobs/batch_anonymise.pl                   |    1 +
 svc/members/search                                 |    2 +-
 t/db_dependent/Circulation/AnonymiseIssueHistory.t |  138 ++++++++++++++++++++
 t/db_dependent/Items.t                             |   18 ++-
 t/db_dependent/Items/MoveItemFromBiblio.t          |   84 ++++++++++++
 t/db_dependent/Utils/Datatables_Members.t          |   25 +++-
 13 files changed, 310 insertions(+), 22 deletions(-)
 create mode 100644 t/db_dependent/Circulation/AnonymiseIssueHistory.t
 create mode 100644 t/db_dependent/Items/MoveItemFromBiblio.t

hooks/post-receive
-- 
main Koha release repository