[MediaWiki-commits] [Gerrit] Revert "Remove hardcoded $wgCanonicalServer from $wgResource... - change (operations/mediawiki-config)

2015-09-15 Thread Krinkle (Code Review)
Krinkle has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/238532

Change subject: Revert "Remove hardcoded $wgCanonicalServer from 
$wgResourceBasePath"
..

Revert "Remove hardcoded $wgCanonicalServer from $wgResourceBasePath"

Works as intended in that mw.loader.addSource and wgLoadScript
are now relative. But this change also caused page HTML (which
was previously okay) to now use en.wikipedia.org instead of
en.m.wikipedia.org. Probably an wfExpandUrl() somewhere.

This reverts commit f41d6b0f65c40e817c07368b838f0f42db63c9a9.

Change-Id: Ic86977ec683275bbe177d4fc74291c41a878baad
---
M wmf-config/CommonSettings.php
M wmf-config/mobile.php
2 files changed, 21 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config 
refs/changes/32/238532/1

diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php
index 6cc2637..c595998 100755
--- a/wmf-config/CommonSettings.php
+++ b/wmf-config/CommonSettings.php
@@ -201,20 +201,19 @@
 unset( $wgStylePath );
 unset( $wgStyleSheetPath );
 
-$wgInternalServer = $wgCanonicalServer;
-$wgArticlePath = '/wiki/$1';
-
-$wgScriptPath  = '/w';
-$wgScript = "{$wgScriptPath}/index.php";
-$wgRedirectScript = "{$wgScriptPath}/redirect.php";
-$wgLoadScript = "{$wgScriptPath}/load.php";
-$wgLocalStylePath = "{$wgScriptPath}/static/$wmgVersionNumber/skins";
-
-// Don't include a hostname in these urls (T106966, T112646)
-$wgResourceBasePath = "/static/{$wmgVersionNumber}";
+$wgLoadScript = "{$wgCanonicalServer}/w/load.php";
+$wgResourceBasePath = "{$wgCanonicalServer}/static/{$wmgVersionNumber}";
 $wgExtensionAssetsPath = "{$wgResourceBasePath}/extensions";
 $wgStylePath = "{$wgResourceBasePath}/skins";
 
+$wgArticlePath = "/wiki/$1";
+
+$wgScriptPath  = '/w';
+$wgLocalStylePath = "$wgScriptPath/static/$wmgVersionNumber/skins";
+$wgScript = $wgScriptPath . '/index.php';
+$wgRedirectScript = $wgScriptPath . '/redirect.php';
+$wgInternalServer = $wgCanonicalServer;
+
 // Deprecated
 $wgStyleSheetPath = $wgStylePath;
 
diff --git a/wmf-config/mobile.php b/wmf-config/mobile.php
index 6f89055..9158a2e 100644
--- a/wmf-config/mobile.php
+++ b/wmf-config/mobile.php
@@ -65,7 +65,17 @@
$wgMFNoMobilePages = $wmgMFNoMobilePages;
 
$wgHooks['EnterMobileMode'][] = function() {
-   global $wgCentralAuthCookieDomain;
+   global $wgCentralAuthCookieDomain, $wgExtensionAssetsPath,
+   $wgLoadScript, $wgResourceBasePath, $wgStylePath, 
$wgStyleSheetPath;
+
+   $mobileContext = MobileContext::singleton();
+   $wgLoadScript = $mobileContext->getMobileUrl( $wgLoadScript );
+   $wgResourceBasePath = $mobileContext->getMobileUrl( 
$wgResourceBasePath );
+   $wgExtensionAssetsPath = "{$wgResourceBasePath}/extensions";
+   $wgStylePath = "{$wgResourceBasePath}/skins";
+
+   // Deprecated
+   $wgStyleSheetPath = $wgStylePath;
 
// Hack for T49647
if ( $wgCentralAuthCookieDomain == 'commons.wikimedia.org' ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/238532
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic86977ec683275bbe177d4fc74291c41a878baad
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: Krinkle 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits


[MediaWiki-commits] [Gerrit] Revert "Remove hardcoded $wgCanonicalServer from $wgResource... - change (operations/mediawiki-config)

2015-09-15 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Revert "Remove hardcoded $wgCanonicalServer from 
$wgResourceBasePath"
..


Revert "Remove hardcoded $wgCanonicalServer from $wgResourceBasePath"

Works as intended in that mw.loader.addSource and wgLoadScript
are now relative. But this change also caused page HTML (which
was previously okay) to now use en.wikipedia.org instead of
en.m.wikipedia.org. Probably an wfExpandUrl() somewhere.

This reverts commit f41d6b0f65c40e817c07368b838f0f42db63c9a9.

Change-Id: Ic86977ec683275bbe177d4fc74291c41a878baad
---
M wmf-config/CommonSettings.php
M wmf-config/mobile.php
2 files changed, 21 insertions(+), 12 deletions(-)

Approvals:
  Krinkle: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php
index 6cc2637..c595998 100755
--- a/wmf-config/CommonSettings.php
+++ b/wmf-config/CommonSettings.php
@@ -201,20 +201,19 @@
 unset( $wgStylePath );
 unset( $wgStyleSheetPath );
 
-$wgInternalServer = $wgCanonicalServer;
-$wgArticlePath = '/wiki/$1';
-
-$wgScriptPath  = '/w';
-$wgScript = "{$wgScriptPath}/index.php";
-$wgRedirectScript = "{$wgScriptPath}/redirect.php";
-$wgLoadScript = "{$wgScriptPath}/load.php";
-$wgLocalStylePath = "{$wgScriptPath}/static/$wmgVersionNumber/skins";
-
-// Don't include a hostname in these urls (T106966, T112646)
-$wgResourceBasePath = "/static/{$wmgVersionNumber}";
+$wgLoadScript = "{$wgCanonicalServer}/w/load.php";
+$wgResourceBasePath = "{$wgCanonicalServer}/static/{$wmgVersionNumber}";
 $wgExtensionAssetsPath = "{$wgResourceBasePath}/extensions";
 $wgStylePath = "{$wgResourceBasePath}/skins";
 
+$wgArticlePath = "/wiki/$1";
+
+$wgScriptPath  = '/w';
+$wgLocalStylePath = "$wgScriptPath/static/$wmgVersionNumber/skins";
+$wgScript = $wgScriptPath . '/index.php';
+$wgRedirectScript = $wgScriptPath . '/redirect.php';
+$wgInternalServer = $wgCanonicalServer;
+
 // Deprecated
 $wgStyleSheetPath = $wgStylePath;
 
diff --git a/wmf-config/mobile.php b/wmf-config/mobile.php
index 6f89055..9158a2e 100644
--- a/wmf-config/mobile.php
+++ b/wmf-config/mobile.php
@@ -65,7 +65,17 @@
$wgMFNoMobilePages = $wmgMFNoMobilePages;
 
$wgHooks['EnterMobileMode'][] = function() {
-   global $wgCentralAuthCookieDomain;
+   global $wgCentralAuthCookieDomain, $wgExtensionAssetsPath,
+   $wgLoadScript, $wgResourceBasePath, $wgStylePath, 
$wgStyleSheetPath;
+
+   $mobileContext = MobileContext::singleton();
+   $wgLoadScript = $mobileContext->getMobileUrl( $wgLoadScript );
+   $wgResourceBasePath = $mobileContext->getMobileUrl( 
$wgResourceBasePath );
+   $wgExtensionAssetsPath = "{$wgResourceBasePath}/extensions";
+   $wgStylePath = "{$wgResourceBasePath}/skins";
+
+   // Deprecated
+   $wgStyleSheetPath = $wgStylePath;
 
// Hack for T49647
if ( $wgCentralAuthCookieDomain == 'commons.wikimedia.org' ) {

-- 
To view, visit https://gerrit.wikimedia.org/r/238532
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic86977ec683275bbe177d4fc74291c41a878baad
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: Krinkle 
Gerrit-Reviewer: Krinkle 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits