Bug 7146

Summary:	Update timestamps when deleting a biblio
Product:	Koha	Reporter:	Marcel de Rooy <m.de.rooy>
Component:	Cataloging	Assignee:	Marcel de Rooy <m.de.rooy>
Status:	CLOSED FIXED	QA Contact:	Ian Walls <koha.sekjal>
Severity:	normal
Priority:	PATCH-Sent (DO NOT USE)	CC:	julian.maurice, koha.sekjal, paul.poulain
Version:	3.6
Hardware:	All
OS:	All
See Also:	http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9732
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Attachments:	Patch Signed off patch

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)

Created attachment 6163 [details] [review]
Patch

Test this patch by comparing the timestamps in deletedbiblio, deletedbiblioitems and deleteditems when deleting all items and the biblio itself (with and without this patch).
With this patch, they should all refer to the actual deletion time.

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