Bug 6939

Summary:	DefaultClassificationSource not used in 942$2
Product:	Koha	Reporter:	Nicole C. Engard <nengard>
Component:	Cataloging	Assignee:	Colin Campbell <colin.campbell>
Status:	CLOSED FIXED	QA Contact:	Bugs List <koha-bugs>
Severity:	normal
Priority:	P5 - low	CC:	chris, koha.sekjal, paul.poulain
Version:	3.8
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Attachments:	Proposed Patch [SIGNED-OFF] Bug 6939 DefaultClassificationSource should be default in 942

Description Nicole C. Engard 2011-09-29 13:31:25 UTC

The DefaultClassificationSource preference sets the choice for the 952$2 but isn't used in the 942$2 - it should be used in both places.

Comment 1 Colin Campbell 2012-06-18 10:32:24 UTC Comment hidden (obsolete)

Created attachment 10347 [details] [review]
Proposed Patch

With this patch applied the value of the syspref should be reflected in the default value for 942 $2 in Add MARC Record (Marc21)

Comment 2 Nicole C. Engard 2012-06-21 15:55:37 UTC

Created attachment 10414 [details] [review]
[SIGNED-OFF] Bug 6939 DefaultClassificationSource should be default in 942

A logic bug in the original code adding DefaultClassificationSource
caused the default not to be used (the last used value was used instead)
The bug was fixed in additem by bug 2398 but addbiblio's use
of the code was not fixed.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>

Comment 3 Ian Walls 2012-06-25 17:57:01 UTC

Simple fix to logic, easy to read, marking Passed QA.

Comment 4 Paul Poulain 2012-06-28 07:30:04 UTC

(I haven't tested this patch, as I don't have classificationSource field / MARC21 in my test setup)