r112648 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r112647‎ | r112648 | r112649 >
Date:21:43, 28 February 2012
Author:reedy
Status:ok
Tags:
Comment:
Modified paths:
  • /branches/REL1_19/extensions (modified) (history)
  • /branches/REL1_19/extensions/CentralAuth/specials/SpecialMergeAccount.php (modified) (history)
  • /branches/REL1_19/extensions/CheckUser/SpecialCheckUserLog.php (modified) (history)
  • /branches/REL1_19/extensions/DoubleWiki/DoubleWiki_body.php (modified) (history)
  • /branches/REL1_19/extensions/UploadWizard (modified) (history)
  • /branches/REL1_19/extensions/UploadWizard/UploadWizard.i18n.php (modified) (history)
  • /branches/REL1_19/extensions/Vector/Vector.php (modified) (history)

Diff [purge]

Index: branches/REL1_19/extensions/UploadWizard/UploadWizard.i18n.php
@@ -38,7 +38,7 @@
3939 'mwe-upwiz-help-desk' => 'Help Desk',
4040 'mwe-upwiz-help-desk-url' => 'Help Desk',
4141 'mwe-upwiz-add-file-n' => 'Add another file',
42 - 'mwe-upwiz-add-file-0-free' => 'Drop media file to donate here',
 42+ 'mwe-upwiz-add-file-0-free' => 'Select a media file to donate',
4343 'mwe-upwiz-transport-started' => 'Starting...',
4444 'mwe-upwiz-uploading' => 'Uploading...',
4545 'mwe-upwiz-transported' => 'Finished uploading...',
Property changes on: branches/REL1_19/extensions/UploadWizard
___________________________________________________________________
Modified: svn:mergeinfo
4646 Merged /trunk/extensions/UploadWizard:r112517,112562
Index: branches/REL1_19/extensions/CentralAuth/specials/SpecialMergeAccount.php
@@ -57,16 +57,22 @@
5858 switch( $this->mMergeAction ) {
5959 case "dryrun":
6060 $this->doDryRunMerge();
 61+ break;
6162 case "initial":
6263 $this->doInitialMerge();
 64+ break;
6365 case "cleanup":
6466 $this->doCleanupMerge();
 67+ break;
6568 case "attach":
6669 $this->doAttachMerge();
 70+ break;
6771 case "remove":
68 - $this->doUnattach();
 72+ $this->doUnattach(); // FIXME: Method is undefined
 73+ break;
6974 default:
70 - $this->invalidAction();
 75+ $this->invalidAction(); // FIXME: Method is undefined
 76+ break;
7177 }
7278 return;
7379 }
Index: branches/REL1_19/extensions/DoubleWiki/DoubleWiki_body.php
@@ -66,60 +66,59 @@
6767 $nt = Title::newFromText( $l );
6868 $iw = $nt->getInterwiki();
6969
70 - if ( $iw !== $match_request ) {
71 - continue;
72 - }
73 - $key = wfMemcKey( 'doublewiki', $wgLang->getCode(), $nt->getPrefixedDbKey() );
74 - $cachedText = $wgMemc->get( $key );
 70+ if ( $iw === $match_request ) {
7571
76 - if( $cachedText ) {
77 - $text = $cachedText;
78 - } else {
79 - $url = $nt->getCanonicalURL();
80 - $myURL = $out->getTitle()->getLocalURL();
81 - $languageName = $wgContLang->getLanguageName( $iw );
82 - $myLanguage = $wgLang->getLanguageName( $wgContLang->getCode() );
83 - $translation = Http::get( wfAppendQuery( $url, array( 'action' => 'render' ) ) );
 72+ $key = wfMemcKey( 'doublewiki', $wgLang->getCode(), $nt->getPrefixedDbKey() );
 73+ $cachedText = $wgMemc->get( $key );
8474
85 - if ( $translation !== null ) {
86 - break;
87 - }
88 - /**
89 - * first find all links that have no 'class' parameter.
90 - * these links are local so we add '?match=xx' to their url,
91 - * unless it already contains a '?'
92 - */
93 - $translation = preg_replace(
94 - "/<a href=\"http:\/\/([^\"\?]*)\"(([\s]+)(c(?!lass=)|[^c\>\s])([^\>\s]*))*\>/i",
95 - "<a href=\"http://\\1?match={$wgContLanguageCode}\"\\2>", $translation );
96 - // now add class='extiw' to these links
97 - $translation = preg_replace(
98 - "/<a href=\"http:\/\/([^\"]*)\"(([\s]+)(c(?!lass=)|[^c\>\s])([^\>\s]*))*\>/i",
99 - "<a href=\"http://\\1\" class=\"extiw\"\\3>", $translation );
100 - // use class='extiw' for images too
101 - $translation = preg_replace(
102 - "/<a href=\"http:\/\/([^\"]*)\"([^\>]*)class=\"image\"([^\>]*)\>/i",
103 - "<a href=\"http://\\1\"\\2class=\"extiw\"\\3>", $translation );
 75+ if( $cachedText ) {
 76+ $text = $cachedText;
 77+ } else {
 78+ $url = $nt->getCanonicalURL();
 79+ $myURL = $out->getTitle()->getLocalURL();
 80+ $languageName = $wgContLang->getLanguageName( $iw );
 81+ $myLanguage = $wgLang->getLanguageName( $wgContLang->getCode() );
 82+ $translation = Http::get( wfAppendQuery( $url, array( 'action' => 'render' ) ) );
10483
105 - // add prefixes to internal links, in order to prevent duplicates
106 - $translation = preg_replace( "/<a href=\"#(.*?)\"/i", "<a href=\"#l_\\1\"",
107 - $translation );
108 - $translation = preg_replace( "/<li id=\"(.*?)\"/i", "<li id=\"l_\\1\"",
109 - $translation );
110 - $text = preg_replace( "/<a href=\"#(.*?)\"/i", "<a href=\"#r_\\1\"", $text );
111 - $text = preg_replace( "/<li id=\"(.*?)\"/i", "<li id=\"r_\\1\"", $text );
 84+ if ( $translation !== null ) {
 85+ /**
 86+ * first find all links that have no 'class' parameter.
 87+ * these links are local so we add '?match=xx' to their url,
 88+ * unless it already contains a '?'
 89+ */
 90+ $translation = preg_replace(
 91+ "/<a href=\"http:\/\/([^\"\?]*)\"(([\s]+)(c(?!lass=)|[^c\>\s])([^\>\s]*))*\>/i",
 92+ "<a href=\"http://\\1?match={$wgContLanguageCode}\"\\2>", $translation );
 93+ // now add class='extiw' to these links
 94+ $translation = preg_replace(
 95+ "/<a href=\"http:\/\/([^\"]*)\"(([\s]+)(c(?!lass=)|[^c\>\s])([^\>\s]*))*\>/i",
 96+ "<a href=\"http://\\1\" class=\"extiw\"\\3>", $translation );
 97+ // use class='extiw' for images too
 98+ $translation = preg_replace(
 99+ "/<a href=\"http:\/\/([^\"]*)\"([^\>]*)class=\"image\"([^\>]*)\>/i",
 100+ "<a href=\"http://\\1\"\\2class=\"extiw\"\\3>", $translation );
112101
113 - // add ?match= to local links of the local wiki
114 - $text = preg_replace( "/<a href=\"\/([^\"\?]*)\"/i",
115 - "<a href=\"/\\1?match={$match_request}\"", $text );
 102+ // add prefixes to internal links, in order to prevent duplicates
 103+ $translation = preg_replace( "/<a href=\"#(.*?)\"/i", "<a href=\"#l_\\1\"",
 104+ $translation );
 105+ $translation = preg_replace( "/<li id=\"(.*?)\"/i", "<li id=\"l_\\1\"",
 106+ $translation );
 107+ $text = preg_replace( "/<a href=\"#(.*?)\"/i", "<a href=\"#r_\\1\"", $text );
 108+ $text = preg_replace( "/<li id=\"(.*?)\"/i", "<li id=\"r_\\1\"", $text );
116109
117 - // do the job
118 - $text = $this->matchColumns ( $text, $myLanguage, $myURL, $wgContLanguageCode,
119 - $translation, $languageName, $url, $match_request );
 110+ // add ?match= to local links of the local wiki
 111+ $text = preg_replace( "/<a href=\"\/([^\"\?]*)\"/i",
 112+ "<a href=\"/\\1?match={$match_request}\"", $text );
120113
121 - $wgMemc->set( $key, $text, $wgDoubleWikiCacheTime );
 114+ // do the job
 115+ $text = $this->matchColumns ( $text, $myLanguage, $myURL, $wgContLanguageCode,
 116+ $translation, $languageName, $url, $match_request );
 117+
 118+ $wgMemc->set( $key, $text, $wgDoubleWikiCacheTime );
 119+ }
 120+ }
 121+ break;
122122 }
123 - break;
124123 }
125124 return true;
126125 }
Index: branches/REL1_19/extensions/CheckUser/SpecialCheckUserLog.php
@@ -60,6 +60,10 @@
6161 }
6262 }
6363 }
 64+
 65+ $out->addHTML( $this->getSkin()->makeKnownLinkObj(
 66+ Title::makeTitle( NS_SPECIAL, 'CheckUser' ),
 67+ wfMsgHtml( 'checkuser-log-return' ) ) );
6468
6569 $searchTypes = array( 'initiator', 'target' );
6670 $select = "<select name=\"cuSearchType\" style='margin-top:.2em;'>\n";
Index: branches/REL1_19/extensions/Vector/Vector.php
@@ -125,6 +125,7 @@
126126 'jquery.suggestions',
127127 'jquery.autoEllipsis',
128128 'jquery.placeholder',
 129+ 'mediawiki.legacy.mwsuggest', // to ensure we disable it in proper order
129130 ),
130131 ),
131132 );
Property changes on: branches/REL1_19/extensions
___________________________________________________________________
Modified: svn:mergeinfo
132133 Merged /trunk/extensions:r112074,112153,112160,112295,112517,112562

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r112074Falling through to the floor is going to end in tears...reedy01:53, 22 February 2012
r112153* (bug 34598) Double search dropdowns sometimes appearing...brion21:55, 22 February 2012
r112295Revert r99937 to fix bug 34674 in trunkreedy03:40, 24 February 2012
r112517Revert r110506: undo change to phrasing on Special:UploadWizard's most import...brion21:15, 27 February 2012
r112562Restore "checkuser-log-return" ("Return to CheckUser main form") link to the ...saper01:57, 28 February 2012
r112628(Bug 34776) Unattached accounts add literal \n on Special:CentralAuth...platonides19:14, 28 February 2012

Status & tagging log