Bug 14119

Summary: Missing de-DE DISCHARGE message
Product: Koha Reporter: Bernardo Gonzalez Kriegel <bgkriegel>
Component: PatronsAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, katrin.fischer, kyle.m.hall, mtompset, tomascohen, veron
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 14119: Missing de-DE DISCHARGE message
Bug 14119: Missing de-DE DISCHARGE message
Bug 14119 - Missing de-DE DISCHARGE message (German wording)
[SIGNED_OFF] Bug 14119 - Missing de-DE DISCHARGE message (German wording)
Bug 14119: Missing de-DE DISCHARGE message
Bug 14119 - Missing de-DE DISCHARGE message (German wording)
[PASSED QA] Bug 14119: Missing de-DE DISCHARGE message
[PASSED QA] Bug 14119 - Missing de-DE DISCHARGE message (German wording)

Description Bernardo Gonzalez Kriegel 2015-05-02 02:36:57 UTC
Bug 8007 DB changes forgot de-DE DISCHARGE message,
prove xt/sample_notices.t fails
Comment 1 Bernardo Gonzalez Kriegel 2015-05-02 02:43:31 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2015-05-02 02:47:42 UTC
If a German patch is provided, please reset assignee :)
I will sign.
Comment 3 Mark Tompsett 2015-05-02 03:00:23 UTC
This works, but I rather a German counter patch. :)
Comment 4 Mark Tompsett 2015-05-02 03:04:02 UTC Comment hidden (obsolete)
Comment 5 Marc Véron 2015-05-02 07:31:36 UTC Comment hidden (obsolete)
Comment 6 Bernardo Gonzalez Kriegel 2015-05-02 13:07:09 UTC Comment hidden (obsolete)
Comment 7 Bernardo Gonzalez Kriegel 2015-05-02 13:09:25 UTC
First patch obsoleted
Comment 8 Mark Tompsett 2015-05-02 15:45:15 UTC
Comment on attachment 38761 [details] [review]
Bug 14119: Missing de-DE DISCHARGE message

It looks like the second patch requires the first. Unchecking 'obsolete' for a clean apply.
Comment 9 Mark Tompsett 2015-05-02 15:54:43 UTC Comment hidden (obsolete)
Comment 10 Mark Tompsett 2015-05-02 15:55:14 UTC Comment hidden (obsolete)
Comment 11 Katrin Fischer 2015-05-03 17:02:22 UTC
Created attachment 38784 [details] [review]
[PASSED QA] Bug 14119: Missing de-DE DISCHARGE message

This patch adds DISCHARGE sample message to de-DE sample_notices.pl

To test:
1) prove xt/sample_notices.t, it fails
2) apply the patch
3) prove again, this time no errors

As original Bug 8007 sample message this is not translated,
so a German counter patch is welcome.

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

As noted, a German counter patch is preferred. :)

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 12 Katrin Fischer 2015-05-03 17:02:39 UTC
Created attachment 38785 [details] [review]
[PASSED QA] Bug 14119 - Missing de-DE DISCHARGE message (German wording)

This patch adds DISCHARGE sample message to de-DE sample_notices.pl
(in GErman wording)

To test:
1) prove xt/sample_notices.t, it fails
2) apply the patch
3) prove again, this time no errors

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Right wording :)
Fixed version

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 13 Tomás Cohen Arazi 2015-05-06 13:45:17 UTC
Patches pushed to master.

Good job Bernardo and Marc!