Koha sample notices are loaded to 'letter' table from a text SQL file during Koha installation by web installer. The reference file is the one provided for English (en) installation : <koha_root>/installer/data/mysql/en/mandatory/sample_notices.sql Alternatives files are provided for other languages. Those files are difficult to keep synchronized with reference file. This could be an functional issue since some Koha operation depend on notice existence, for example Print Slip in Circulation. For example, for French installation, those notices are missing: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP
Created attachment 33216 [details] [review] Bug 13199 Add missing notices for fr-FR installation prove xt/sample_notices.sql identifies five missing notices for fr-FR. This patch add those missing notices: ISSUEQSLIP ISSUESLIP OPAC_REG_VERIFY RESERVESLIP TRANSFERSLIP
Created attachment 33217 [details] [review] Bug 13199 Test incoherence between 'en' sample notices and translated notices This test identifies incoherences between translated sample notices and the 'en' reference file. Koha sample notices are loaded to 'letter' table from a text SQL file during Koha installation by web installer. The reference file is the one provided for English (en) installation : <koha_root>/installer/data/mysql/en/mandatory/sample_notices.sql Alternatives files are provided for other languages. Those files are difficult to keep synchronized with reference file. This could be an functional issue since some Koha operation depend on notice existence, for example Print Slip in Circulation.
The test identifies other missing notices: - it-IT/necessari/notices.sql: OPAC_REG_VERIFY, RENEWAL - pl-PL/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - ru-RU/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - uk-UA/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP
Katrin: I've added you to this patch CC! You're in best position to confirm this bug may be disturbing/blocking for non-English installations, since you're the only one to succeed having a de-DE on a par with 'en' sample notices.
Hi Frederic, hopefully those will be the last inconsistencies to be cleaned up. I'd like to have the test, but it will need to pass. - Could you also provide a patch for ru-RU and uk-UA? Copying the English samples is fine - still a good help to get people started making their own translated notices.
> Could you also provide a patch > for ru-RU and uk-UA? Copying the English samples is fine - still a good help > to get people started making their own translated notices. Ok. Will do this evening. Do you have someone doing it-I & pl-PL?
Ah sorry, I missed those in the test output. If you could do them all that would be great.
For OPAC_REG_VERIFY, RENEWAL in it-IT/necessari/notices.sql the English strings are OK
Created attachment 33336 [details] [review] Bug 13199 Add missing notices for several installations prove xt/sample_notices.sql identifies missing notices. This patch add those missing notices: - fr-FR ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - it-IT/necessari/notices.sql: OPAC_REG_VERIFY, RENEWAL - pl-PL/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - ru-RU/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - uk-UA/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP TO TEST: (1) Without this patch, but with the patch adding xt/sample_notices.sql: prove xt/sample_notices.sql list all the above missing notices. (2) Apply this patch. And do prove again: nothing is reported.
Created attachment 33352 [details] [review] [SIGNED-OFF] Bug 13199: Test incoherence between 'en' sample notices and translated notices This test identifies incoherences between translated sample notices and the 'en' reference file. Koha sample notices are loaded to 'letter' table from a text SQL file during Koha installation by web installer. The reference file is the one provided for English (en) installation : <koha_root>/installer/data/mysql/en/mandatory/sample_notices.sql Alternatives files are provided for other languages. Those files are difficult to keep synchronized with reference file. This could be an functional issue since some Koha operation depend on notice existence, for example Print Slip in Circulation. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Test works and identify missing notices A few koha-qa errors fixed in followup
Created attachment 33353 [details] [review] [SIGNED-OFF] Bug 13199: Add missing notices for several installations prove xt/sample_notices.sql identifies missing notices. This patch add those missing notices: - fr-FR ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - it-IT/necessari/notices.sql: OPAC_REG_VERIFY, RENEWAL - pl-PL/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - ru-RU/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - uk-UA/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP TO TEST: (1) Without this patch, but with the patch adding xt/sample_notices.sql: prove xt/sample_notices.sql list all the above missing notices. (2) Apply this patch. And do prove again: nothing is reported. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Work as described, test stop failing No koha-qa errors
Created attachment 33354 [details] [review] Bug 13199: follow up to fix Licence and some koha-qa errors
Created attachment 33359 [details] [review] [PASSED QA] Bug 13199: Test incoherence between 'en' sample notices and translated notices This test identifies incoherences between translated sample notices and the 'en' reference file. Koha sample notices are loaded to 'letter' table from a text SQL file during Koha installation by web installer. The reference file is the one provided for English (en) installation : <koha_root>/installer/data/mysql/en/mandatory/sample_notices.sql Alternatives files are provided for other languages. Those files are difficult to keep synchronized with reference file. This could be an functional issue since some Koha operation depend on notice existence, for example Print Slip in Circulation. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Test works and identify missing notices A few koha-qa errors fixed in followup Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 33360 [details] [review] [PASSED QA] Bug 13199: Add missing notices for several installations prove xt/sample_notices.sql identifies missing notices. This patch add those missing notices: - fr-FR ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - it-IT/necessari/notices.sql: OPAC_REG_VERIFY, RENEWAL - pl-PL/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - ru-RU/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP - uk-UA/mandatory/sample_notices.sql: ISSUEQSLIP, ISSUESLIP, OPAC_REG_VERIFY, RESERVESLIP, TRANSFERSLIP TO TEST: (1) Without this patch, but with the patch adding xt/sample_notices.sql: prove xt/sample_notices.sql list all the above missing notices. (2) Apply this patch. And do prove again: nothing is reported. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Work as described, test stop failing No koha-qa errors Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 33361 [details] [review] [PASSED QA] Bug 13199: follow up to fix Licence and some koha-qa errors Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Thx Frederic and Bernardo!
Patches pushed to master. Thanks Frederic and Bernardo!
*** Bug 13808 has been marked as a duplicate of this bug. ***