Bug 8744 - Thesaurus in authorities should not lock fields
Summary: Thesaurus in authorities should not lock fields
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: MARC Authority data support (show other bugs)
Version: 3.8
Hardware: All All
: P5 - low normal (vote)
Assignee: Jared Camins-Esakov
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 8211
  Show dependency treegraph
 
Reported: 2012-09-09 12:52 UTC by Jared Camins-Esakov
Modified: 2013-12-05 20:01 UTC (History)
3 users (show)

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


Attachments
Bug 8744: Thesaurus in authorities should not lock fields (1.56 KB, patch)
2012-09-09 12:55 UTC, Jared Camins-Esakov
Details | Diff | Splinter Review
SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-shouldd not lock fields (1.58 KB, patch)
2012-09-09 19:48 UTC, wajasu
Details | Diff | Splinter Review
SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-should-not-lock-fields (1.58 KB, patch)
2012-09-09 19:52 UTC, wajasu
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jared Camins-Esakov 2012-09-09 12:52:16 UTC
Although fields in the authority editor should support thesaurus control, they should never be locked, as authority records commonly refer to other records that may not be in use locally (and therefore might not be in the authority file).
Comment 1 Jared Camins-Esakov 2012-09-09 12:55:00 UTC Comment hidden (obsolete)
Comment 2 wajasu 2012-09-09 19:48:40 UTC Comment hidden (obsolete)
Comment 3 wajasu 2012-09-09 19:52:35 UTC
Created attachment 12088 [details] [review]
SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-should-not-lock-fields

Now one can create heirarchies of geographical names inthe authority 551 $a field for example.  This was needed to support bug 8211 for expanded search functionaility.
Comment 4 Paul Poulain 2012-10-03 13:38:24 UTC
QA comment:
 * tiny patch, template-only, no side-effect risk
 * passes koha-qa.pl
 * another use case = authority A is NT for authority B, that is BT for authority A. Which one we create first ? ;-)

passed QA
Comment 5 Paul Poulain 2012-10-03 13:52:35 UTC
Patch pushed to master
Comment 6 Chris Cormack 2012-10-03 19:28:21 UTC
Pushed to 3.8.x, will be in 3.8.6
Comment 7 Jared Camins-Esakov 2012-12-03 13:15:48 UTC
This now works.