Bug 7356

Summary: Fix various typos and mis-spellings
Product: Koha Reporter: Marc Balmer <marc>
Component: Developer documentationAssignee: Marc Balmer <marc>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: koha.sekjal, m.de.rooy, paul.poulain
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Text to go in the release notes:
Version(s) released in:
Attachments: functionnal -> functional
wether -> whether
developper -> developer
the the -> the
wether -> whether
the the -> the
developper -> developer
wether -> whether, developper -> developer
Squashed patches, signed off

Description Marc Balmer 2011-12-14 09:11:10 UTC Comment hidden (obsolete)
Comment 1 Marc Balmer 2011-12-14 09:11:36 UTC Comment hidden (obsolete)
Comment 2 Marc Balmer 2011-12-14 09:12:20 UTC Comment hidden (obsolete)
Comment 3 Marc Balmer 2011-12-14 09:12:46 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2011-12-14 12:43:54 UTC
Thanks for your work!

First patch: Attachment is not viewable in your browser because its MIME type (application/octet-stream) is not one that your browser is able to display. 

Second & Third patch: Resend please without changing po files. See point 4.

Fourth patch: I would not recommend changing things in the PO files this way; new PO files will come from Pootle. Should be corrected at the base. Contact translation manager.
Comment 5 Marcel de Rooy 2011-12-15 07:30:30 UTC
Patch 1 is okay now. Looking forward to update of 2, 3 and 4.
Comment 6 Marc Balmer 2011-12-15 13:07:22 UTC
(In reply to comment #5)
> Patch 1 is okay now. Looking forward to update of 2, 3 and 4.

Patch 3 does not change any .po files, it should be ok, right?
Comment 7 Marcel de Rooy 2011-12-15 13:10:43 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > Patch 1 is okay now. Looking forward to update of 2, 3 and 4.
> Patch 3 does not change any .po files, it should be ok, right?

OK. Overlooked something?
Comment 8 Marc Balmer 2011-12-15 13:41:58 UTC Comment hidden (obsolete)
Comment 9 Marc Balmer 2011-12-15 13:42:42 UTC Comment hidden (obsolete)
Comment 10 Marcel de Rooy 2011-12-15 14:20:47 UTC
Applying: developper -> developer.
error: patch failed: authorities/authorities.pl:204
error: authorities/authorities.pl: patch does not apply
error: patch failed: cataloguing/addbiblio.pl:414
error: cataloguing/addbiblio.pl: patch does not apply
error: patch failed: cataloguing/plugin_launcher.pl:27
error: cataloguing/plugin_launcher.pl: patch does not apply
Patch failed at 0001.
Comment 11 Marc Balmer 2011-12-15 14:23:57 UTC Comment hidden (obsolete)
Comment 12 Marc Balmer 2011-12-15 14:31:52 UTC
(In reply to comment #10)
> Applying: developper -> developer.
> error: patch failed: authorities/authorities.pl:204
> error: authorities/authorities.pl: patch does not apply
> error: patch failed: cataloguing/addbiblio.pl:414
> error: cataloguing/addbiblio.pl: patch does not apply
> error: patch failed: cataloguing/plugin_launcher.pl:27
> error: cataloguing/plugin_launcher.pl: patch does not apply
> Patch failed at 0001.

ok. I have to merge to patches now, sorry.
Comment 13 Marc Balmer 2011-12-15 14:32:35 UTC Comment hidden (obsolete)
Comment 14 Owen Leonard 2011-12-29 18:46:59 UTC
Created attachment 6956 [details] [review]
Squashed patches, signed off

Several instances of typos corrected in third-party libraries, which I probably would have skipped. But there's no harm in correcting them.
Comment 15 Ian Walls 2012-01-11 02:20:43 UTC
A bunch of typo corrections.  Mostly fixing 'the the' and missing/extra letters.  Marking as Passed QA
Comment 16 Paul Poulain 2012-01-13 10:52:09 UTC
patch pushed. As it's a comment-only patch, marking fixed immediatly, nothing can be really "tested"