r45741 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r45740‎ | r45741 | r45742 >
Date:19:01, 14 January 2009
Author:raymond
Status:resolved (Comments)
Tags:
Comment:
* Wrap success/failure/norev messages into a div with own classes
* Add class "error" for failure/norev
* Remove "span class=error" from some localizations
** Makes default behaviour more consistent over all languages
Modified paths:
  • /trunk/phase3/includes/EditPage.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDa.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDe.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesDsb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesGsw.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesHsb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesKsh.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesLb.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesNds.php (modified) (history)
  • /trunk/phase3/languages/messages/MessagesStq.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/EditPage.php
@@ -177,22 +177,22 @@
178178 }
179179 if ( $result ) {
180180 # Inform the user of our success and set an automatic edit summary
181 - $this->editFormPageTop .= $wgOut->parse( wfMsgNoTrans( 'undo-success' ) );
 181+ $this->editFormPageTop .= $wgOut->wrapWikiMsg( '<div class="mw-undo-success">$1</div>', 'undo-success' );
182182 $firstrev = $oldrev->getNext();
183183 # If we just undid one rev, use an autosummary
184184 if ( $firstrev->mId == $undo ) {
185 - $this->summary = wfMsgForContent('undo-summary', $undo, $undorev->getUserText());
 185+ $this->summary = wfMsgForContent( 'undo-summary', $undo, $undorev->getUserText() );
186186 }
187187 $this->formtype = 'diff';
188188 } else {
189189 # Warn the user that something went wrong
190 - $this->editFormPageTop .= $wgOut->parse( wfMsgNoTrans( 'undo-failure' ) );
 190+ $this->editFormPageTop .= $wgOut->wrapWikiMsg( '<div class="error mw-undo-failure">$1</div>', 'undo-failure' );
191191 }
192192 } else {
193193 // Failed basic sanity checks.
194194 // Older revisions may have been removed since the link
195195 // was created, or we may simply have got bogus input.
196 - $this->editFormPageTop .= $wgOut->parse( wfMsgNoTrans( 'undo-norev' ) );
 196+ $this->editFormPageTop .= $wgOut->wrapWikiMsg( '<div class="error mw-undo-norev">$1</div>', 'undo-norev' );
197197 }
198198 } else if ( $section != '' ) {
199199 if ( $section == 'new' ) {
Index: trunk/phase3/languages/messages/MessagesDsb.php
@@ -770,7 +770,7 @@
771771
772772 # "Undo" feature
773773 'undo-success' => 'Wobźěłanje móžo se wótpóraś. Pšosym pśeglěduj dołojcne pśirownowanje aby se wěsty był, až to wót wěrnosći coš, a pón składuj změny, aby se wobźěłanje doskóńcnje wótpórało.',
774 -'undo-failure' => '<span class="error">Změna njejo se mógała wótpóraś, dokulaž jo něchten pótrjefjony wótrězk mjaztym změnił.</span>',
 774+'undo-failure' => 'Změna njejo se mógała wótpóraś, dokulaž jo něchten pótrjefjony wótrězk mjaztym změnił.',
775775 'undo-norev' => 'Změna njeda se wótwrośiś, dokulaž njeeksistěčujo abo jo se wulašowała.',
776776 'undo-summary' => 'Wersija $1 wót [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskusija]]) jo se anulěrowała',
777777
Index: trunk/phase3/languages/messages/MessagesKsh.php
@@ -942,8 +942,8 @@
943943
944944 # "Undo" feature
945945 'undo-success' => 'De Änderung könnte mer zeröck nämme. Beloor Der de Ungerscheid un dann donn di Sigg avspeichere, wann De dengks, et es en Oodenung esu.',
946 -'undo-failure' => '<span class="error">Dat kunnt mer nit zeröck nämme, dä Afschnedd wood enzwesche ald widder beärbeidt.</span>',
947 -'undo-norev' => '<span class="error">Do ka\'mer nix zeröck nämme. Di Version jidd_et nit, odder se es verstoche odder fottjeschmesse woode.</span>',
 946+'undo-failure' => 'Dat kunnt mer nit zeröck nämme, dä Afschnedd wood enzwesche ald widder beärbeidt.',
 947+'undo-norev' => 'Do ka\'mer nix zeröck nämme. Di Version jidd_et nit, odder se es verstoche odder fottjeschmesse woode.',
948948 'undo-summary' => 'De Änderung $1 fum [[Special:Contributions/$2|$2]] ([[User talk:$2|Klaaf]]) zeröck jenomme.',
949949
950950 # Account creation failure
Index: trunk/phase3/languages/messages/MessagesNds.php
@@ -841,7 +841,7 @@
842842
843843 # "Undo" feature
844844 'undo-success' => 'De Ännern kann trüchdreiht warrn. Vergliek ünnen de Versionen, dat ok allens richtig is, un spieker de Sied denn af.',
845 -'undo-failure' => '<span class="error">Kunn de Siet nich op de vörige Version trüchdreihn. De Afsnitt is twischendör al wedder ännert worrn.</span>',
 845+'undo-failure' => 'Kunn de Siet nich op de vörige Version trüchdreihn. De Afsnitt is twischendör al wedder ännert worrn.',
846846 'undo-norev' => 'De Ännern kunn nich trüchdreiht warrn, de gifft dat nich oder is wegsmeten worrn.',
847847 'undo-summary' => 'Ännern $1 vun [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskuschoon]]) trüchdreiht.',
848848
Index: trunk/phase3/languages/messages/MessagesGsw.php
@@ -763,7 +763,7 @@
764764
765765 # "Undo" feature
766766 'undo-success' => 'Zum die Änderig ruckgängig z mache, kontrollier bitte d Bearbeitig in dr Verglichsaasicht un druck derno uf „Syte spichere“.',
767 -'undo-failure' => '<span class="error">D Änderig het nid chenne ruckgängig gmacht wäre, wel dää Abschnitt mittlerwyli gänderet woren isch.</span>',
 767+'undo-failure' => 'D Änderig het nid chenne ruckgängig gmacht wäre, wel dää Abschnitt mittlerwyli gänderet woren isch.',
768768 'undo-norev' => 'D Bearbeitig het nid chenne ruckgängig gmacht wäre, wel si nid vorhande oder glescht isch.',
769769 'undo-summary' => 'D Änderig $1 vu [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskussion]]) isch ruckgängig gmacht wore.',
770770
Index: trunk/phase3/languages/messages/MessagesHsb.php
@@ -750,7 +750,7 @@
751751
752752 # "Undo" feature
753753 'undo-success' => 'Wersija je so wuspěšnje wotstroniła. Prošu přepruwuj deleka w přirunanskim napohledźe, hač twoja změna bu přewzata a klikń potom na „Składować”, zo by změnu składował.',
754 -'undo-failure' => '<span class="error">Wobdźěłanje njehodźeše so wotstronić, dokelž wotpowědny wotrězk bu mjeztym změnjeny.</span>',
 754+'undo-failure' => 'Wobdźěłanje njehodźeše so wotstronić, dokelž wotpowědny wotrězk bu mjeztym změnjeny.',
755755 'undo-norev' => 'Změna njeda so cofnyć, dokelž njeeksistuje abo bu wušmórnjena.',
756756 'undo-summary' => 'Změna $1 [[Special:Contributions/$2|$2]] ([[User talk:$2|diskusija]]) bu cofnjena.',
757757
Index: trunk/phase3/languages/messages/MessagesLb.php
@@ -764,7 +764,7 @@
765765
766766 # "Undo" feature
767767 'undo-success' => "D'Ännerung gëtt réckgängeg gemaach. Iwwerpréift w.e.g. de Verglach ënnedrënner fir nozekuckeen ob et esou richteg ass, duerno späichert w.e.g d'Ännerungen of fir dës Aktioun ofzeschléissen.",
768 -'undo-failure' => '<span class="error">D\'Ännerung konnt net réckgängeg gemaach ginn, wëll de betraffenen Abschnitt an der Tëschenzäit geännert gouf.</span>',
 768+'undo-failure' => 'D\'Ännerung konnt net réckgängeg gemaach ginn, wëll de betraffenen Abschnitt an der Tëschenzäit geännert gouf.',
769769 'undo-norev' => "D'Ännerung kann net zréckgesat ginn, well et se net gëtt oder well se scho geläscht ass.",
770770 'undo-summary' => 'Ännerung $1 vu(n) [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskussioun]] | [[Special:Contributions/$2|{{MediaWiki:Contribslink}}]]) annulléieren.',
771771
Index: trunk/phase3/languages/messages/MessagesStq.php
@@ -672,7 +672,7 @@
673673
674674 # "Undo" feature
675675 'undo-success' => 'Ju Annerenge kuud mäd Ärfoulch tourääch annerd wäide. Jädden ju Beoarbaidenge in ju Ferglieksansicht kontrollierje un dan ap „Siede spiekerje“ klikke, uum ju tou spiekerjen.',
676 -'undo-failure' => '<span class="error">Ju Annerenge kuud nit tourääch annerd wäide, deer ju betroffene Oudeelenge intwisken ferannerd wuude.</span>',
 676+'undo-failure' => 'Ju Annerenge kuud nit tourääch annerd wäide, deer ju betroffene Oudeelenge intwisken ferannerd wuude.',
677677 'undo-norev' => 'Ju Beoarbaidenge kuud nit räägels troald wäide, deer ju nit foarhounden is of läsked wuude.',
678678 'undo-summary' => 'Annerenge $1 fon [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskussion]]) wuude tourääch annerd.',
679679
Index: trunk/phase3/languages/messages/MessagesDa.php
@@ -781,7 +781,7 @@
782782
783783 # "Undo" feature
784784 'undo-success' => 'Ændringen er nu annulleret. Kontroller venligst bearbejdningen i sammenligningen og klik så på „Gem side“, for at gemme den.',
785 -'undo-failure' => '<span class="error">Ændringen kunne ikke annulleres, da det pågældende afsnit i mellemtiden er ændret.</span>',
 785+'undo-failure' => 'Ændringen kunne ikke annulleres, da det pågældende afsnit i mellemtiden er ændret.',
786786 'undo-norev' => 'Ændringen kunne ikke annuleres fordi den ikke eksisterer eller er blevet slettet.',
787787 'undo-summary' => 'Fjerner version $1 af [[Special:Contributions/$2|$2]] ([[User talk:$2|diskussion]])',
788788
Index: trunk/phase3/languages/messages/MessagesDe.php
@@ -959,7 +959,7 @@
960960
961961 # "Undo" feature
962962 'undo-success' => 'Um die Änderung rückgängig zu machen, bitte die Bearbeitung in der Vergleichsansicht kontrollieren und dann auf „Seite speichern“ klicken.',
963 -'undo-failure' => '<span class="error">Die Änderung konnte nicht rückgängig gemacht werden, da der betroffene Abschnitt zwischenzeitlich verändert wurde.</span>',
 963+'undo-failure' => 'Die Änderung konnte nicht rückgängig gemacht werden, da der betroffene Abschnitt zwischenzeitlich verändert wurde.',
964964 'undo-norev' => 'Die Bearbeitung konnte nicht rückgängig gemacht werden, da sie nicht vorhanden ist oder gelöscht wurde.',
965965 'undo-summary' => 'Änderung $1 von [[Special:Contributions/$2|$2]] ([[User talk:$2|Diskussion]]) wurde rückgängig gemacht.',
966966

Follow-up revisions

RevisionCommit summaryAuthorDate
r45747Follow up r45741 per IAlex comment:...raymond20:32, 14 January 2009

Comments

#Comment by IAlex (talk | contribs)   20:18, 14 January 2009
+	$this->editFormPageTop .= $wgOut->wrapWikiMsg( ... )

the $this->editFormPageTop .= part is useless here since OutputPage::wrapWikiMsg() directly adds the html to OutputPage::$mBodytext rather than returning it.

#Comment by Raymond (talk | contribs)   20:34, 14 January 2009

Fixed in r45747

Status & tagging log