MaxSem has uploaded a new change for review.

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


Change subject: Remove support for api.php?action=parse&mobileformat=wml
......................................................................

Remove support for api.php?action=parse&mobileformat=wml

It's messy and unused.
Instead, return HTML and set warning.

Change-Id: I4b87ae34e9ff7d11fa834dee1a2571d4f06d2454
---
M includes/api/ApiParseExtender.php
1 file changed, 3 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/20/70320/1

diff --git a/includes/api/ApiParseExtender.php 
b/includes/api/ApiParseExtender.php
index 1298b5b..bd4e9bb 100644
--- a/includes/api/ApiParseExtender.php
+++ b/includes/api/ApiParseExtender.php
@@ -72,18 +72,10 @@
                                $title = Title::newFromText( 
$data['parse']['title'] );
                                $html = MobileFormatter::wrapHTML( 
$data['parse']['text']['*'] );
                                if ( MobileContext::parseContentFormat( 
$params['mobileformat'] ) === 'WML' ) {
-                                       $context = new WmlContext();
-                                       $context->setCurrentUrl( 
$title->getCanonicalURL() );
-                                       $context->setRequestedSegment( isset( 
$params['section'] )
-                                                       ? $params['section'] + 
1 // Segment numbers start from 1
-                                                       : 0
-                                       );
-                                       $context->setUseFormat( 'wml' ); // 
Force WML links just in case
-                                       $context->setOnlyThisSegment( isset( 
$params['section'] ) );
-                                       $mf = new MobileFormatterWML( $html, 
$title, $context );
-                               } else {
-                                       $mf = new MobileFormatterHTML( $html, 
$title );
+                                       // @todo: make mobileformat accept only 
HTML on July 25, 2013
+                                       $module->setWarning( 'mobileformat=wml 
is not supported anymore' );
                                }
+                               $mf = new MobileFormatterHTML( $html, $title );
                                $mf->removeImages( $params['noimages'] );
                                $mf->setIsMainPage( $params['mainpage'] );
                                $mf->enableExpandableSections( 
!$params['mainpage'] );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4b87ae34e9ff7d11fa834dee1a2571d4f06d2454
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to