Bug 5660

Summary: Webinstaller should check if KohaOpacLanguage exists in staff
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: Installation and upgrade (web-based installer)Assignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, paul.poulain
Version: Main   
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: 7026    
Attachments: Patch
Revised patch
Rebased patch
Rebased patch
Rebased patch
Signed off patch
proposed patch (passed QA)

Description Marcel de Rooy 2011-01-26 15:03:50 UTC
While upgrading, I stumbled into this error: 
[error] [client ...] built-in find_file: can't find file /usr/share/koha/testclone/koha-tmpl/intranet-tmpl/prog/nl-NL/modules/installer/step3.tmpl with path = [ '/usr/share/koha/testclone/koha-tmpl/intranet-tmpl/prog/nl-NL/includes' ], referer: http://.../cgi-bin/koha/installer/install.pl?step=3

What is the case? I installed language nl-NL only on the opac side. But the installer simply assumes it to be on the intranet side too.

Will send a small patch for this bug.
Comment 1 Marcel de Rooy 2011-01-26 15:25:25 UTC Comment hidden (obsolete)
Comment 2 MJ Ray (software.coop) 2011-02-06 11:14:03 UTC
This bug is mentioned in:
Bug 5660: Webinstaller should check if KohaOpacLanguage exists on staff side http://lists.koha-community.org/pipermail/koha-patches/2011-January/013705.html
Comment 3 Marcel de Rooy 2011-04-07 12:28:59 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2011-04-11 08:29:00 UTC Comment hidden (obsolete)
Comment 5 Marcel de Rooy 2011-04-11 08:37:21 UTC Comment hidden (obsolete)
Comment 6 Chris Cormack 2011-04-11 21:55:29 UTC
I would like to get this into the 3.4.0 release, can someone sign off please
Comment 7 Marcel de Rooy 2011-04-27 10:00:54 UTC Comment hidden (obsolete)
Comment 8 Chris Cormack 2011-05-17 00:46:37 UTC Comment hidden (obsolete)
Comment 9 Paul Poulain 2011-08-04 12:55:15 UTC
Created attachment 4840 [details] [review]
proposed patch (passed QA)

QA comment

Nothing specific to say: small (& usefull) patch, no DB change, no internal change
Comment 10 Chris Cormack 2011-08-04 23:10:49 UTC
Pushed
Comment 11 Marcel de Rooy 2011-10-03 07:25:34 UTC
Closing it myself now; waited two months..