Summary: | Thesaurus in authorities should not lock fields | ||
---|---|---|---|
Product: | Koha | Reporter: | Jared Camins-Esakov <jcamins> |
Component: | MARC Authority data support | Assignee: | Jared Camins-Esakov <jcamins> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, matted-34813, paul.poulain |
Version: | 3.8 | ||
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: | ||
Bug Depends on: | |||
Bug Blocks: | 8211 | ||
Attachments: |
Bug 8744: Thesaurus in authorities should not lock fields
SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-shouldd not lock fields SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-should-not-lock-fields |
Description
Jared Camins-Esakov
2012-09-09 12:52:16 UTC
Created attachment 12082 [details] [review] Bug 8744: Thesaurus in authorities should not lock fields 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). Test plan: 1) Make sure a 5xx field in one of your authority frameworks is thesaurus-controlled. 2) Note that you cannot edit the field directly. 3) Apply patch. 4) Note that you can now edit the field directly. Created attachment 12087 [details] [review] SIGNED-OFF-Bug-8744-Thesaurus-in-authorities-shouldd not lock fields Before this patch, I wasn't able to edit the Geographical name in authority 551 field. Now I can. It is necessary to be able to create 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. 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 Patch pushed to master Pushed to 3.8.x, will be in 3.8.6 This now works. |