For reasons unknown, 240$a is marked as a URL in the default MARC21 frameworks. It isn't, and treating it as one in the MARC view results in broken links. For example, links like: http://192.168.122.28:8080/cgi-bin/koha/catalogue/Menschwerdung%20Gottes
Still valid in current master.
Created attachment 34025 [details] [review] Bug 8909: MARC21 240$a (Uniform title) is not a URL Patch removes the 'isUrl' attribute from all 240$a fields in the various frameworks. The change removes the broken link from the staff detail MARC tab view. Changes are a little easier to spot with: git diff --color-words HEAD^ To test: - Load some or all of the frameworks - Verify that 240$a is no longer marked as being an URL
Created attachment 34689 [details] [review] [SIGNED OFF]Bug 8909: MARC21 240$a (Uniform title) is not a URL Patch removes the 'isUrl' attribute from all 240$a fields in the various frameworks. The change removes the broken link from the staff detail MARC tab view. Changes are a little easier to spot with: git diff --color-words HEAD^ To test: - Load some or all of the frameworks - Verify that 240$a is no longer marked as being an URL Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
I can't QA this for obvious reasons :)
Created attachment 34861 [details] [review] [PASSED QA] Bug 8909: MARC21 240$a (Uniform title) is not a URL Patch removes the 'isUrl' attribute from all 240$a fields in the various frameworks. The change removes the broken link from the staff detail MARC tab view. Changes are a little easier to spot with: git diff --color-words HEAD^ To test: - Load some or all of the frameworks - Verify that 240$a is no longer marked as being an URL Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Thanks Katrina!