Summary: | CataloguingLog pref description wrong | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | System Administration | Assignee: | Paul Poulain <paul.poulain> |
Status: | CLOSED INVALID | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, koha.sekjal, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: |
Description
Nicole C. Engard
2010-05-28 00:01:29 UTC
Can anyone confirm that this is an issue? Based on the following discussion, I recommend removing the warning from this preference: [10:51] <nengard> can someone confirm that bug 4835 is a bug - if not can you explain to me what that log tracks? [10:51] <munin> Bug http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4835 normal, P5, ---, paul.poulain, NEW, CataloguingLog pref description wrong [10:59] <gmcharlt> nengard: "it is not advisable to [10:59] <gmcharlt> turn this on. " is essentially FUD, in my opinion [11:00] <nengard> that's what it sounds like to me!!! [11:00] <nengard> what does it track exactly? just cataloging actions? or other actions as well? [11:00] <gmcharlt> changes to bibs and items [11:01] <nengard> then i see no reason why that would be a problem [11:01] <nengard> well i know we cna't change it for 3.2 but i'll put it on my 3.4 list [11:01] <gmcharlt> specifically bib and item creates, updates, and deletes [11:02] <gmcharlt> oddly, it does not cause authority record updates to be logged [11:03] <nengard> hmmm Circulation::AddIssue (Circulation.pm) has a call to Items::ModItem (Items.pm) which makes changes to the biblio record and hence calls logaction(). This bug can be fixed in two ways (IMHO): 1) Ommit logging a "change" action on circulation events by passing a boolean flag to ModItem to log or not to log. 2) Wait for the removal of circulation data from marcxml and marc fields. |