Last modified: 2012-08-04 21:11:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T39098, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 37098 - Patchwork message
Patchwork message
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
ArticleFeedbackv5 (Other open bugs)
master
All All
: Normal major (vote)
: ---
Assigned To: Matthias Mullie
: i18n
Depends on:
Blocks: 39043
  Show dependency treegraph
 
Reported: 2012-05-24 18:32 UTC by Siebrand Mazeland
Modified: 2012-08-04 21:11 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Siebrand Mazeland 2012-05-24 18:32:28 UTC
<https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/ArticleFeedbackv5.git;a=blobdiff;f=ArticleFeedbackv5.i18n.php;h=026adbc7a40efd58ab5a93c174cc44bc1920eb02;hp=cab16efbcddefd0c824aa8ca4226d88efe98d6cf;hb=0981daf3acdcd48233511efc742331b70708cd0e;hpb=fff431cfe3f6a006e107bd67abccb7a64406d3db> introduced two messages that are patchwork:

+       'articlefeedbackv5-cta5-confirmation-followup' => 'Your feedback has been $1.',
+       'articlefeedbackv5-cta5-confirmation-followup-linktext' => 'posted here',

This should simply be "Your feedback has been [[$1 posted here]]" or something similar.

Found this our reviewing Gerrit diff in https://gerrit.wikimedia.org/r/#/c/8729/ for patch set 1 to patch set 3.
Comment 1 Siebrand Mazeland 2012-05-24 18:35:07 UTC
Top tip: Add some default CCs for ArticleFeedbackv5, because there aren't any.
Comment 2 Fabrice Florin 2012-05-24 22:48:25 UTC
Thanks, Siebrand!

Can you please clarify what you are proposing in layman's language? 

(I am not a programmer, so I don't understand what you mean by 'patchwork' of 'default CCs').

Much appreciated.
Comment 3 Siebrand Mazeland 2012-05-25 05:44:26 UTC
Default Ccs: send email to one or more email addresses once a bug has been reported for a certain component. This is bugzilla functionality,

Patchwork messages: Languages have varying word orders, and complex grammatical and syntactic rules. Messages put together from lots of pieces of text, possibly with some indirection, are very hard, if not impossible, to translate. Better make messages complete sentences each, with a full stop at the end. Several sentences can usually much more easily be combined into a text block, if needed. More at http://www.mediawiki.org/wiki/I18n#Avoid_patchwork_messages
Comment 4 Fabrice Florin 2012-05-25 21:34:14 UTC
Thank you Siebrand.

This is very helpful, and I bookmarked your excellent page with tips for localization:
http://www.mediawiki.org/wiki/I18n

I am asking Matthias Mullie, our new AFT5 developer, to address this patchwork issue, so that he can fix this in coming weeks. He will be in Berlin, so maybe you guys can address this in person, if you are going too.

Also, you and I should schedule a call when you get back from Berlin, to plan our next steps for internationalization of AFT5, which we hope to develop in June-July, with your help.

I thought I already got email Ccs for all posts about Article Feedback v5 tickets on Bugzilla -- but maybe I should make sure I enable it by default, as you suggest -- for both Matthias and myself, on both AFT and AFT5. Do you know where we can set this up?

Also, please make sure to add this keyword to any important bugs related to AFT5, so we can include it in this phase of development: aftv5-1.5 (I just added it to this post). Like this, we will pay closer attention to them, as we have a number of tickets that have been pushed back to the next phase and are not getting the same attention.

Cheers!

Fabrice
Comment 5 Matthias Mullie 2012-05-29 19:50:59 UTC
Patch is up at https://gerrit.wikimedia.org/r/#/c/8993/
Comment 6 Siebrand Mazeland 2012-06-05 20:57:39 UTC
Was merged and reverted in Gerrit change #10298.
Comment 7 Matthias Mullie 2012-06-06 13:58:01 UTC
(In reply to comment #6)
> Was merged and reverted in Gerrit change #10298.

Yes; Roan accidentally merged it but I had him pull it out since it depends on a core change (https://gerrit.wikimedia.org/r/#/c/8991/) awaiting approval.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links