Bug 5783 - Authorities edit logging
Summary: Authorities edit logging
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: master
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Galen Charlton
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-21 14:04 UTC by Tomás Cohen Arazi
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:


Attachments
Initial patch (5.96 KB, patch)
2011-02-21 14:45 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Supplementary patch (2.12 KB, patch)
2011-03-17 17:03 UTC, Colin Campbell
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Tomás Cohen Arazi 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 2011-02-21 14:45:16 UTC
Created attachment 3190 [details] [review]
Initial patch
Comment 2 Tomás Cohen Arazi 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