[koha-commits] main Koha release repository branch master updated. v16.11.00-541-gfd8c2d4

[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  fd8c2d4e35f0ed4d7661eea2f1238fe2d094b969 (commit)
       via  17e84750d9d90f3d2df466d1c8d8a63bbd5ad09d (commit)
       via  cb58b11f6769f69e109c1f42a78b3756d429dc80 (commit)
       via  6572ee07dbcd9fde4169c6b9e3f3f8a8e0eaf516 (commit)
      from  9b92a494e6d91297f460eb8ed073a6d3dec5b2d9 (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 fd8c2d4e35f0ed4d7661eea2f1238fe2d094b969
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Feb 7 14:03:54 2017 +0100

    Bug 18070: Correct error from podchecker on line 1376
    
    Resolves:
     *** ERROR: Spurious =cut command at line 1376 in file C4/AuthoritiesMarc.pm
    
    Test plan:
    [1] Run podchecker on C4/AuthoritiesMarc.pm
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 17e84750d9d90f3d2df466d1c8d8a63bbd5ad09d
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Feb 14 16:41:26 2017 +0100

    Bug 18070: [Follow-up] What should DelAuthority return?
    
    Script tools/batch_delete_records.pl only checks the return value of
    DelAuthority. The return value depends on DBI rows, which is not always
    reliable. It may return -1 when it 'does not know'.
    
    Testing $@ in tools/batch_delete_records.pl should actually be enough.
    The return value was discarded in C4/ImportBatch.pm. Removing an unused
    variable.
    
    Test plan:
    [1] Pick an authority record with a few linked biblios.
        Delete this authority record via tools/batch_delete_records.pl.
        Check if the linked biblio records are cleaned up.
    [2] Bonus: Make a typo in the SQL statement of DelAuthority. Check
        if batch_delete_records shows you the error message.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit cb58b11f6769f69e109c1f42a78b3756d429dc80
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Jan 31 13:01:58 2017 +0100

    Bug 18070: Extend sub merge to remove fields for deleted authorities
    
    In order to accomplish this, we need to add some additional checks in
    the merge routine. The actual change to remove the field, is quite
    small.
    
    Furthermore, we need to add a merge call in DelAuthority and adjust
    the merge cron job accordingly.
    
    The change is well supported by additional tests, including a simulation
    of postponed removal via cron, if dontmerge is enabled.
    
    Note: Deleting an authority with linked biblios is tested on the next
    patch.
    
    Test plan:
    [1] Run t/db_dependent/Authorities/Merge.t
    [2] Delete an authority without linked biblios from the Authorities
        module. If the indexer is not fast enough, wait a bit and refresh to
        verify that the authority is gone on authorities-home.pl.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 6572ee07dbcd9fde4169c6b9e3f3f8a8e0eaf516
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Jan 31 11:46:21 2017 +0100

    Bug 18070: Deleting an authority should update the linked biblio records
    
    Adding a test where we delete an authority and prove that the linked
    biblio still contains a reference to it.
    
    Note: Currently, you can only delete a used authority from Tools,
    batch record deletion. If you do, the biblio records will still
    contain references to the deleted authority.
    
    Test plan:
    [1] Run t/db_dependent/Authorities/Merge.t
        Last test should fail: not ok 1 - Field 609 should be gone too
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

Summary of changes:
 C4/AuthoritiesMarc.pm                   |   54 +++++++++++++++++++------------
 C4/ImportBatch.pm                       |    2 +-
 misc/migration_tools/merge_authority.pl |    9 ++++--
 t/db_dependent/Authorities/Merge.t      |   48 ++++++++++++++++++++++++++-
 tools/batch_delete_records.pl           |    4 +--
 5 files changed, 89 insertions(+), 28 deletions(-)


hooks/post-receive
-- 
main Koha release repository