Bug 9752

Summary: Set marcflavor = normarc when NORMARC is chosen in the web installer
Product: Koha Reporter: Magnus Enger <magnus>
Component: Installation and upgrade (web-based installer)Assignee: Magnus Enger <magnus>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: chris, gmcharlt
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9752 - Set marcflavour = NORMARC when NORMARC is chosen
Bug 9752: Set marcflavour to NORMARC when NORMARC is chosen
[PASSED QA] Bug 9752: Set marcflavour to NORMARC when NORMARC is chosen

Description Magnus Enger 2013-03-06 10:28:40 UTC
I thought this was already being done, but alas. If you do a web install in Norwegian and choose NORMARC as the MARC dialect, the marcflavor syspref gets set to MARC21, not NORMARC. 

I hope the solution can be as simple as including something like this in the SQL file for the default NORMARC framework:

UPDATE systempreferences SET value = 'normarc' WHERE variable = 'marcflavor';

But I will have to look at the order in which things are loaded by the webinstaller first.
Comment 1 Magnus Enger 2013-03-06 14:25:26 UTC Comment hidden (obsolete)
Comment 2 Magnus Enger 2013-03-06 14:26:29 UTC
So the solution was not in the SQL file as I initially thought, but in Installer.pm.
Comment 3 Galen Charlton 2013-03-12 22:30:41 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-03-30 19:58:47 UTC
Created attachment 17052 [details] [review]
[PASSED QA] Bug 9752: Set marcflavour to NORMARC when NORMARC is chosen

Currently, if you install the Norwegian translations and run
through the web installer in Norwegian, choosing NORMARC as you
marcflavour, the marcflavour syspref is set to MARC21.

To test:
- Apply the patch
- Install nb-NO
- Run through the web installer, choosing nb-NO as the language
- Choose NORMARC as the MARC dialect
- When the web installer is done, check the value of the
  marcflavour syspref. It should be NORMARC, not MARC21.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Did a regression test installing UNIMARC too.
All tests and QA script pass.
Comment 5 Jared Camins-Esakov 2013-03-31 02:51:56 UTC
This patch has been pushed to master.
Comment 6 Chris Cormack 2013-03-31 06:49:56 UTC
Pushed to 3.10.x will be in 3.10.5
Comment 7 Magnus Enger 2013-04-03 19:27:15 UTC
Works as expected (tested with packages built off current master).