Bug 7146 - Update timestamps when deleting a biblio
Summary: Update timestamps when deleting a biblio
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Marcel de Rooy
QA Contact: Ian Walls
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-03 12:35 UTC by Marcel de Rooy
Modified: 2013-02-28 15:23 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Patch (3.34 KB, patch)
2011-11-03 12:59 UTC, Marcel de Rooy
Details | Diff | Splinter Review
Signed off patch (3.35 KB, patch)
2011-11-04 06:58 UTC, Julian Maurice
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel de Rooy 2011-11-03 12:35:32 UTC
Currently, when you delete an item, the timestamp column in deleteditems is updated with current time. (This comes from an [unintentional] additional update statement in DelItem.) It makes deletion time visible.
In the past, the marcxml was updated too at that moment, resulting in an updated timestamp in biblioitems too. The timestamp in biblio was not touched.

If you delete a biblio however, the timestamps in deletedbiblio and deletedbiblioitems do not reflect time of deletion. They still show the time of last update before the record was deleted. This last update can be extracted from MARC field 005 too.

This behavior is not consistent nor logical. I would suggest to add a statement in DelBiblio to force updating the timestamp in deletedbiblio(items) too. It makes the time of deletion visible in the record too. The time of deletion of a biblio can be very useful for e.g. synchronizing purposes.
Comment 1 Marcel de Rooy 2011-11-03 12:59:45 UTC Comment hidden (obsolete)
Comment 2 Julian Maurice 2011-11-04 06:58:05 UTC
Created attachment 6181 [details] [review]
Signed off patch

It works as expected, signed off.
Comment 3 Ian Walls 2011-11-04 18:40:26 UTC
Adds simple change for consistency of behaviour.  Comments include reference back to this full bug report, and the existing behaviour is noted as well as the added.  Marking as Passed QA
Comment 4 Paul Poulain 2011-11-06 11:05:07 UTC
Patch pushed, please test