Created attachment 6777 [details] [review] functionnal -> functional The attached patches fix various typos and mis-spellings, NB that the Component Documentation is probably not adequate. The patches need to be applied in order. 0001 functionnal -> functional 0002 wether -> whether 0003 developper -> developer 0004 'the the ' - > 'the '
Created attachment 6778 [details] [review] wether -> whether
Created attachment 6779 [details] [review] developper -> developer
Created attachment 6780 [details] [review] the the -> the
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.
Patch 1 is okay now. Looking forward to update of 2, 3 and 4.
(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?
(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?
Created attachment 6811 [details] [review] wether -> whether Changes only code, not .po files.
Created attachment 6812 [details] [review] the the -> the No more changing of the the .po files ;)
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.
Created attachment 6813 [details] [review] developper -> developer re-created.
(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.
Created attachment 6814 [details] [review] wether -> whether, developper -> developer
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.
A bunch of typo corrections. Mostly fixing 'the the' and missing/extra letters. Marking as Passed QA
patch pushed. As it's a comment-only patch, marking fixed immediatly, nothing can be really "tested"