r109801 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r109800‎ | r109801 | r109802 >
Date:02:53, 23 January 2012
Author:varnent
Status:ok
Tags:
Comment:
fix to r109715 - ty Nikerabbit & Reedy
Modified paths:
  • /trunk/extensions/AddThis/AddThis.body.php (modified) (history)
  • /trunk/extensions/AddThis/AddThis.php (modified) (history)

Diff [purge]

Index: trunk/extensions/AddThis/AddThis.body.php
@@ -64,8 +64,7 @@
6565 # Check if page is in content namespace and the setting to enable/disable article header tooblar either on the main page or at all
6666 if ( !MWNamespace::isContent( $article->getTitle()->getNamespace() )
6767 || !$wgAddThisHeader
68 - || ( ($article->getTitle()->getText() == wfMsg('mainpage') ) && !$wgAddThisMain )
69 -
 68+ || ( $article->getTitle()->equals( Title::newMainPage() ) && !$wgAddThisMain )
7069 ) {
7170 return true;
7271 }
Index: trunk/extensions/AddThis/AddThis.php
@@ -9,7 +9,7 @@
1010 * @license GPL
1111 *
1212 * Loosely based on the Google Translator extension by Joachim De Schrijver
13 - * Thank you to Johnduhart, Roan Kattouw, Unikum111, catrope, and folks at AddThis for feedback, bug reporting and cleaning up code
 13+ * Thank you to Johnduhart, Roan Kattouw, Unikum111, catrope, Nikerabbit, Reedy and folks at AddThis for feedback and cleaning up code
1414 * Thank you to Raymond, Kghbln, Michawiki, Toliño, McDutchie, Bjankuloski06, SPQRobin, Veeven, Gucci Mane Burrr, Kaajawa, Purodha, Kwj2772,
1515 * Unikum111, Y-M D, Xuacu, Naudefj, Gomoko, Anakmalaysia, פוילישער, Shizhao, and others mentioned in AddThis.i18n.php for translation work
1616 */

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r109715better backwards compatibility to MW1.17varnent02:06, 22 January 2012

Status & tagging log