Bug 20746

Summary: Improve behaviour of onboarding tool for Italian by standardizing file structure of it-IT installer
Product: Koha Reporter: Giuseppe Ciaccio <giuseppe.ciaccio>
Component: Installation and upgrade (web-based installer)Assignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P3 CC: giuseppe.ciaccio, gmcharlt, jonathan.druart, katrin.fischer, martin.renvoize, mtompset
Version: master   
Hardware: PC   
OS: Linux   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20746: Move it-IT files to optional
k
Bug 20746: Move it-IT files to optional
Bug 20746: Move it-IT files to optional

Description Giuseppe Ciaccio 2018-05-09 16:04:40 UTC
The install/config wizard follows a number of consecutive steps.  At some point there is a step "create a patron category", where one can define the "admin" patron category, later referred to when subsequently creating the administrator account.

However, when choosing italian language for the install wizard, the "create a patron category" step is skipped.  Therefore, in the step of creating an administrator account, one has to choose "staff" as the category.

This by no means hampers the installation/configuration (apparently), but looks suspicious and deserves some investigation IMHO.
Comment 1 Jonathan Druart 2018-05-09 16:31:02 UTC
This step is skipped because patron categories are marked as "mandatory", they are in the following files: installer/data/mysql/it-IT/necessari/patron_categories.sql

If you uncheck the checkbox related to this file, you will see this step.
Comment 2 Giuseppe Ciaccio 2018-05-09 17:11:46 UTC
Fine, but then I cannot see the reason why a "mandatory" step could be optional. Anyway the real point is that I would not expect the default behaviour of an install wizard to depend on the chosen language.  IMHO this must be fixed (and the fix seems trivial, by the way).
Comment 3 Jonathan Druart 2018-05-09 17:36:12 UTC
The main problem I guess is that italian installer files do not follow the same pattern as other installer files.
Comment 4 Jonathan Druart 2018-05-09 17:36:46 UTC
Created attachment 75212 [details] [review]
Bug 20746: Move it-IT files to optional
Comment 5 Katrin Fischer 2018-05-10 15:54:58 UTC
Created attachment 75228 [details] [review]
k
Comment 6 Katrin Fischer 2018-05-10 15:55:25 UTC
Created attachment 75229 [details] [review]
Bug 20746: Move it-IT files to optional

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Mark Tompsett 2018-05-10 17:39:15 UTC
Created attachment 75236 [details] [review]
Bug 20746: Move it-IT files to optional

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Katrin Fischer 2018-05-10 17:40:45 UTC
Counting Mark for SO and mine for QA.
Comment 9 Jonathan Druart 2018-05-11 14:13:29 UTC
Pushed to master for 18.05, thanks to everybody involved!