r95088 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r95087‎ | r95088 | r95089 >
Date:15:00, 20 August 2011
Author:siebrand
Status:ok (Comments)
Tags:
Comment:
Follow-up r84159. This was supposed to be an addition, not an override.
Modified paths:
  • /trunk/extensions/WikimediaMessages/WikimediaGrammarForms.php (modified) (history)

Diff [purge]

Index: trunk/extensions/WikimediaMessages/WikimediaGrammarForms.php
@@ -40,7 +40,14 @@
4141 'ВікіКрыніца' => 'ВікіКрыніцы',
4242 'ВікіНавіны' => 'ВікіНавін',
4343 'ВікіСлоўнік' => 'ВікіСлоўніка',
 44+ 'Віківіды' => 'Віківідаў',
 45+ 'Вікікнігі' => 'Вікікніг',
 46+ 'Вікікрыніца' => 'Вікікрыніцы',
 47+ 'Вікімэдыя' => 'Вікімэдыі'
 48+ 'Вікінавіны' => 'Вікінавін',
4449 'Вікіпэдыя' => 'Вікіпэдыі',
 50+ 'Вікіслоўнік' => 'Вікіслоўніка',
 51+ 'Вікісховішча'=> 'Вікісховішча',
4552 'Фундацыя «Вікімэдыя»' => 'Фундацыі «Вікімэдыя»'
4653 ),
4754 # akusative
@@ -48,7 +55,12 @@
4956 'ВікіКрыніца' => 'ВікіКрыніцу',
5057 'ВікіНавіны' => 'ВікіНавіны',
5158 'ВікіСлоўнік' => 'ВікіСлоўнік',
 59+ 'Вікікрыніца' => 'Вікікрыніцу',
 60+ 'Вікімэдыя' => 'Вікімэдыю'
 61+ 'Вікінавіны' => 'Вікінавіны',
5262 'Вікіпэдыя' => 'Вікіпэдыю',
 63+ 'Вікіслоўнік' => 'Вікіслоўнік',
 64+ 'Вікісховішча'=> 'Вікісховішча',
5365 'Фундацыя «Вікімэдыя»' => 'Фундацыю «Вікімэдыя»'
5466 ),
5567 # prepositional
@@ -58,36 +70,16 @@
5971 'ВікіКрыніца' => 'ВікіКрыніцы',
6072 'ВікіНавіны' => 'ВікіНавінах',
6173 'ВікіСлоўнік' => 'ВікіСлоўніку',
62 - 'Вікіпэдыя' => 'Вікіпэдыі',
63 - 'Фундацыя «Вікімэдыя»' => 'Фундацыі «Вікімэдыя»'
64 - ),
65 - 'родны' => array(
66 - 'Віківіды' => 'Віківідаў',
67 - 'Вікікнігі' => 'Вікікніг',
68 - 'Вікікрыніца' => 'Вікікрыніцы',
69 - 'Вікінавіны' => 'Вікінавін',
70 - 'Вікіслоўнік' => 'Вікіслоўніка',
71 - 'Вікісховішча'=> 'Вікісховішча',
72 - 'Вікімэдыя' => 'Вікімэдыі'
73 - ),
74 -
75 - 'вінавальны' => array(
76 - 'Вікікрыніца' => 'Вікікрыніцу',
77 - 'Вікінавіны' => 'Вікінавіны',
78 - 'Вікіслоўнік' => 'Вікіслоўнік',
79 - 'Вікісховішча'=> 'Вікісховішча',
80 - 'Вікімэдыя' => 'Вікімэдыю'
81 - ),
82 -
83 - 'месны' => array(
8474 'Віківіды' => 'Віківідах',
8575 'Вікікнігі' => 'Вікікнігах',
8676 'Вікікрыніца' => 'Вікікрыніцы',
 77+ 'Вікімэдыя' => 'Вікімэдыі'
8778 'Вікінавіны' => 'Вікінавінах',
 79+ 'Вікіпэдыя' => 'Вікіпэдыі',
8880 'Вікіслоўнік' => 'Вікіслоўніку',
8981 'Вікісховішча'=> 'Вікісховішчы',
90 - 'Вікімэдыя' => 'Вікімэдыі'
91 - )
 82+ 'Фундацыя «Вікімэдыя»' => 'Фундацыі «Вікімэдыя»'
 83+ ),
9284 ); # be-tarask
9385
9486 $wgGrammarForms['bs'] = array(

Follow-up revisions

RevisionCommit summaryAuthorDate
r95089MFT r95088.siebrand15:01, 20 August 2011
r95203Submitting PHP that does not compile makes Brion cry. Do not do this.neilk14:27, 22 August 2011
r95209*facepalm* Sprinkle some commas over r95089 (MFT of r95088)....siebrand15:16, 22 August 2011
r108433MFT r95088 r95203nikerabbit17:45, 9 January 2012

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r84159Update Wikimedia's $wgGrammarForms for be-tarask....siebrand09:15, 17 March 2011

Comments

#Comment by NeilK (talk | contribs)   14:31, 22 August 2011

This didn't compile. PLEASE don't do this.

I think this happened because, the be-tarask section avoided commas at the end of the series. If you appended a new item, you had to make sure you added the terminal comma. I fixed it to be more like the rest of the file, terminal commas for every line. (no reason to avoid that in PHP).

Status & tagging log