https://www.mediawiki.org/wiki/Special:Code/MediaWiki/112619

Revision: 112619
Author:   awjrichards
Date:     2012-02-28 17:49:37 +0000 (Tue, 28 Feb 2012)
Log Message:
-----------
Removing any invocation of MobileFrontend::updateMobileUrlPath() for now but 
leaving functionality in place for when we're ready to implement.

Modified Paths:
--------------
    trunk/extensions/MobileFrontend/MobileFrontend.body.php
    trunk/extensions/MobileFrontend/MobileFrontend.php

Modified: trunk/extensions/MobileFrontend/MobileFrontend.body.php
===================================================================
--- trunk/extensions/MobileFrontend/MobileFrontend.body.php     2012-02-28 
17:44:35 UTC (rev 112618)
+++ trunk/extensions/MobileFrontend/MobileFrontend.body.php     2012-02-28 
17:49:37 UTC (rev 112619)
@@ -1271,7 +1271,6 @@
        public function getMobileUrl( $url, $forceHttps = false ) {
                $parsedUrl = wfParseUrl( $url );
                $this->updateMobileUrlHost( $parsedUrl );
-               $this->updateMobileUrlPath( $parsedUrl );
                if ( $forceHttps ) {
                        $parsedUrl[ 'scheme' ] = 'https';
                }
@@ -1311,6 +1310,12 @@
 
        /**
         * Update path of given URL to conform to mobile URL template.
+        *
+        * NB: this is not actually being used anywhere at the moment. It will 
+        * take some magic to get MW to properly handle path modifications like
+        * this is intended to provide. This will hopefully be implemented 
someday
+        * in the not to distant future.
+        *
         * @param $parsedUrl array
         *              Result of parseUrl() or wfParseUrl()
         */

Modified: trunk/extensions/MobileFrontend/MobileFrontend.php
===================================================================
--- trunk/extensions/MobileFrontend/MobileFrontend.php  2012-02-28 17:44:35 UTC 
(rev 112618)
+++ trunk/extensions/MobileFrontend/MobileFrontend.php  2012-02-28 17:49:37 UTC 
(rev 112619)
@@ -83,8 +83,8 @@
  * This will be used to transcode regular URLs into mobile URLs for the 
  * mobile view.
  * 
- * It's possible to specify the 'mobileness' of the URL either in the 
- * host portion, or in the path portion.
+ * It's possible to specify the 'mobileness' of the URL in the host portion of
+ * the URL.
  *
  * You can either statically or dynamically create the host-portion of your
  * mobile URL. To statically create it, just set $wgMobileUrlTemplate to 
@@ -99,14 +99,6 @@
  * So, if you wanted a mobile URL scheme that turned "en.wikipedia.org" into
  * "en.m.wikipedia.org", your URL template would look like:
  *             %h0.m.%h1.%h2
- *
- * If you prefer to use template the path, be sure to exclude $wgScriptPath.
- * You can specify where the non-mobile path portion appears with the token %p.
- * For instance:
- *             $wgMobileUrlTemplate = "/mobile/%p";
- *
- * At the moment, this might behave strangely if you want to template both the
- * host and path portions of the URL.
  */
 $wgMobileUrlTemplate = '%h0.m.%h1.%h2';
 


_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to