Bug 5783 - Authorities edit logging
Summary: Authorities edit logging
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement
Assignee: Galen Charlton
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-21 14:04 UTC by Tomás Cohen Arazi (tcohen)
Modified: 2012-10-25 23:03 UTC (History)
5 users (show)

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

Attachments
Initial patch (5.96 KB, patch)
2011-02-21 14:45 UTC, Tomás Cohen Arazi (tcohen) 		Details | Diff | Splinter Review
Supplementary patch (2.12 KB, patch)
2011-03-17 17:03 UTC, Colin Campbell 		Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tomás Cohen Arazi (tcohen) 2011-02-21 14:04:37 UTC 
Add the ability to log actions on authority records. The same as with biblios and items.

This involves:

* Create a syspref for controlling  this: AUTHORITIES
* Add a conditional (on the value of the syspref) call to Log::logaction in

AddAuthority
ModAuthority
DelAuthority

'object' should be the authid of the authority.

* Fix viewlog.pl and viewlog.tmpl to give access to this data.
Comment 1 Tomás Cohen Arazi (tcohen) 2011-02-21 14:45:16 UTC 
Created attachment 3190 [details] [review]
Initial patch
Comment 2 Tomás Cohen Arazi (tcohen) 2011-02-21 16:06:09 UTC 
Successfully tested on dev install
Comment 3 Galen Charlton 2011-03-03 21:49:51 UTC 
Submitted signed-off patch + two follow-ups.
Comment 4 MJ Ray (software.coop) 2011-03-05 10:34:10 UTC 
This bug is mentioned in:
[Bug 5783] Authorities edit logging http://lists.koha-community.org/pipermail/koha-patches/2011-February/013906.html
Comment 5 Chris Cormack 2011-03-07 02:16:43 UTC 
Pushed, please test and mark resolved
Comment 6 Colin Campbell 2011-03-17 17:01:21 UTC 
Oops looks like AuthoritiesLog has been addded to the en sysprefs from both patches. Third patch attached to reduce the two lines to one
Comment 7 Colin Campbell 2011-03-17 17:03:23 UTC 
Created attachment 3346 [details] [review]
Supplementary patch

Resolve duplication of entry in en
Comment 8 Colin Campbell 2011-03-18 13:05:36 UTC 
See bug 5893 for removal of duplicate entry