[koha-commits] main Koha release repository branch master updated. v3.16.00-563-g6c34c8b

Mon Sep 8 17:39:16 CEST 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  6c34c8b650c1894a7696fc26de72d25ce94f2b5d (commit)
       via  6405f8b53ff0fb1be38018b5ddba50fa9630bb37 (commit)
       via  b338a40e807dbbb064073d8cc1779a1e7ba8a021 (commit)
       via  f6b770686f9d7c4f3983a821a86597bd1e88e24e (commit)
       via  6069ed645ff0778fd2ec1003a02e55cb2cdbcf04 (commit)
       via  6af4da3695c4770e528cfe750da949e3e412658a (commit)
       via  b0e5e2c7494e43662867d4049c1c8ecb1afecc80 (commit)
      from  e86371a0aa3c024d3a60af71ab055c9415efb844 (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 6c34c8b650c1894a7696fc26de72d25ce94f2b5d
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Mon Sep 8 12:28:41 2014 -0300

    Bug 7817: (followup) add test on MARC bibliographic framework test

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6405f8b53ff0fb1be38018b5ddba50fa9630bb37
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Mon Sep 8 12:04:40 2014 -0300

    Bug 7817: DBRev 3.17.00.022

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit b338a40e807dbbb064073d8cc1779a1e7ba8a021
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Mon Aug 25 13:24:03 2014 +0200

    Bug 7817: QA Follow-up: Print a warning when permanent location is mapped

    Item field permanent location should not be mapped to the MARC structure.
    If a library did so, this dbrev will print a warning.
    If refers to report 12818 for help in moving the data to more_subfields_xml.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Tested by putting permanent_location in and out of the MARC structure.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit f6b770686f9d7c4f3983a821a86597bd1e88e24e
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Mon Aug 18 13:25:48 2014 +0200

    Bug 7817: Follow-up for original patch

    This patch removes the commented line for permanent_location.
    It adds a more general comment.
    Adjusts the exists check on permanent_location.
    Adds a reference to bug 12817 that will deal with paidfor similarly.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6069ed645ff0778fd2ec1003a02e55cb2cdbcf04
Author: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Date:   Wed Jun 18 18:31:13 2014 +0300

    Bug 7817 - Items Permanent location (shelving location) is set to NULL when item is edited

    A quick fix to prevent more damages.
    No perceived side-effects so far.

    Signed-off-by: David Cook <dcook at prosentient.com.au>

    This is actually a perfectly good fix for this issue. I've changed
    the explanatory comment to explain why.

    Another option would be to remove the 'exists' check in the sub
    _do_column_fixes_for_mod(), but this is just as functional.

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Detailed comment on Bugzilla.
    Adding a small follow-up.

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit 6af4da3695c4770e528cfe750da949e3e412658a
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Aug 28 13:47:18 2014 -0400

    Bug 12467 [QA Followup] - Unit Tests

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

commit b0e5e2c7494e43662867d4049c1c8ecb1afecc80
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Mon Jun 23 12:21:16 2014 -0400

    Bug 12467 - Lost items marked as not on loan even if they are!

    The cronjob longoverdue.pl does not require that an item marked as lost
    be returned automatically, but there is a line in ModItem that
    automatically marks the item's onloan as false if itemlost is set!

    Test Plan:
    1) Mark an item as lost with longoverdue.pl, without --mark-returned
    2) Inspect the db, note that items.onloan is now 0
    3) Apply this patch
    4) Mark repeat step 1
    5) Inspect the db, noe that items.onloan is still 1
    6) Test marking an item as lost from staff interface,
       ensure there are no regressions.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Patch works according to test plan and fixes a data loss bug.

    Some notes:
    - This patch would be nicer with a regression test.
    - Also checked that returning the item removes lost status and onloan still.
    - Tried to test with --mark-returned, but couldn't get it to
      return my item neither with nor without the patch. (see comment on
      bug report)

    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

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

Summary of changes:
 C4/Circulation.pm                                  |    2 ++
 C4/Items.pm                                        |   10 +++---
 admin/checkmarc.pl                                 |   15 +++++++++
 installer/data/mysql/updatedatabase.pl             |   14 ++++++++
 .../prog/en/modules/admin/checkmarc.tt             |   34 ++++++++++++++++++++
 kohaversion.pl                                     |    2 +-
 t/db_dependent/Circulation.t                       |   12 +++++--
 7 files changed, 81 insertions(+), 8 deletions(-)

hooks/post-receive
-- 
main Koha release repository