Bug 20785

Summary: Advanced Editor does not honor MarcFieldDocURL
Product: Koha Reporter: Caroline Cyr La Rose <caroline.cyr-la-rose>
Component: CatalogingAssignee: paxed <pasi.kallinen>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, m.de.rooy, martin.renvoize, nick, pasi.kallinen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20785: Advanced Editor does not honor MarcFieldDocURL
Bug 20785: Advanced Editor does not honor MarcFieldDocURL
Bug 20785: Advanced Editor does not honor MarcFieldDocURL

Description Caroline Cyr La Rose 2018-05-18 20:39:49 UTC
If you change the value of MarcFieldDocURL, and you catalog using the advanced editor, the help still points to Library of Congress.

To reproduce :
1) Leave MarcFieldDocURL at its default value (empty)
2) Enable EnableAdvancedCatalogingEditor
2) Go to the cataloguing module
3) Click on 'New record' and choose a framework
3.5) (If you are not in the basic editor, click on 'Settings' and choose 'Switch to basic editor')
4) Click on the question mark (?) next to any MARC field
5) Note that the default goes to www.loc.gov
6) Click on 'Settings'
7) Choose 'Switch to advanced editor' (click OK on the confirmation)
8) Put your cursor next to a MARC field
9) At the bottom of the page, click on the question mark ([?])
10) Note that the default goes to www.loc.gov
11) Go to Administration > Global System Preferences
12) Search for EnableAdvancedCatalogingEditor and enter http://www.marc21.ca/MaJ/BIB/B{FIELD}.pdf in the box
13) Save the system preferences
14) Redo steps 2 to 4
15) Note that the help goes to www.marc21.ca 
16) Redo steps 6 to 9
17) Note that the help still goes to www.loc.gov
Comment 1 paxed 2018-08-24 08:15:15 UTC
Created attachment 78110 [details] [review]
Bug 20785: Advanced Editor does not honor MarcFieldDocURL

The advanced MARC editor, Rancor, doesn't obey MarcFieldDocURL.
Make it do so.

Test plan:

1) Enable EnableAdvancedCatalogingEditor
2) Set MarcFieldDocURL to eg. http://{MARC}.example.com/{LANG}/{FIELD}
3) Go to Cataloging -> Advanced editor
4) Go to any field and press Ctrl-H
5) Marc field documentation on www.loc.gov or ifla.org should open
6) Apply patch
7) Repeat 3 and 4. The Marc field docs should open on the URL you set
Comment 2 Michal Denar 2018-09-27 08:18:57 UTC
Created attachment 79469 [details] [review]
Bug 20785: Advanced Editor does not honor MarcFieldDocURL

The advanced MARC editor, Rancor, doesn't obey MarcFieldDocURL.
Make it do so.

Test plan:

1) Enable EnableAdvancedCatalogingEditor
2) Set MarcFieldDocURL to eg. http://{MARC}.example.com/{LANG}/{FIELD}
3) Go to Cataloging -> Advanced editor
4) Go to any field and press Ctrl-H
5) Marc field documentation on www.loc.gov or ifla.org should open
6) Apply patch
7) Repeat 3 and 4. The Marc field docs should open on the URL you set

Signed-off-by: Pasi Kallinen <pasi.kallinen@joensuu.fi>
Signed-off-by: Michal Denar <black23@gmail.com>
Comment 3 Katrin Fischer 2018-09-27 11:30:33 UTC
Created attachment 79479 [details] [review]
Bug 20785: Advanced Editor does not honor MarcFieldDocURL

The advanced MARC editor, Rancor, doesn't obey MarcFieldDocURL.
Make it do so.

Test plan:

1) Enable EnableAdvancedCatalogingEditor
2) Set MarcFieldDocURL to eg. http://{MARC}.example.com/{LANG}/{FIELD}
3) Go to Cataloging -> Advanced editor
4) Go to any field and press Ctrl-H
5) Marc field documentation on www.loc.gov or ifla.org should open
6) Apply patch
7) Repeat 3 and 4. The Marc field docs should open on the URL you set

Signed-off-by: Pasi Kallinen <pasi.kallinen@joensuu.fi>
Signed-off-by: Michal Denar <black23@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Nick Clemens (kidclamp) 2018-09-28 19:56:25 UTC
Awesome work all!

Pushed to master for 18.11
Comment 5 Martin Renvoize 2018-10-03 09:41:25 UTC
Pushed to 18.05.x for 18.05.05
Comment 6 Fridolin Somers 2018-10-10 10:12:57 UTC
Pushed to 17.11.x for 17.11.11