From ce01e262ac04ee4dcdbebd7fcea6f15f3ae44d3e Mon Sep 17 00:00:00 2001
From: Adam Wight <spam@ludd.net>
Date: Tue, 23 Apr 2019 14:14:47 -0700
Subject: [PATCH] Use POST to submit wikitext to mw.api.parse

It was impossible to parse wikitext longer than the maximum URL length of about
2000 characters, because our library only used GET requests.  This patch
switches to using POST for inline content, while still using GET when parsing
existing pages.

Bug: T216837
Change-Id: I971b7fc197517a2761f2100fc2aeda087ff52a78
---
 resources/src/mediawiki.api/parse.js                        | 4 ++--
 .../resources/mediawiki.api/mediawiki.api.parse.test.js     | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/resources/src/mediawiki.api/parse.js b/resources/src/mediawiki.api/parse.js
index 42b0771dbf..447b9365d4 100644
--- a/resources/src/mediawiki.api/parse.js
+++ b/resources/src/mediawiki.api/parse.js
@@ -26,13 +26,13 @@
 			if ( mw.Title && content instanceof mw.Title ) {
 				// Parse existing page
 				config.page = content.getPrefixedDb();
+				apiPromise = this.get( config );
 			} else {
 				// Parse wikitext from input
 				config.text = String( content );
+				apiPromise = this.post( config );
 			}
 
-			apiPromise = this.get( config );
-
 			return apiPromise
 				.then( function ( data ) {
 					return data.parse.text;
diff --git a/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js b/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js
index 0bcce12476..29cffafc56 100644
--- a/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js
+++ b/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js
@@ -7,7 +7,7 @@
 	} ) );
 
 	QUnit.test( '.parse( string )', function ( assert ) {
-		this.server.respondWith( /action=parse.*&text='''Hello(\+|%20)world'''/, [ 200,
+		this.server.respondWith( 'POST', /api.php/, [ 200,
 			{ 'Content-Type': 'application/json' },
 			'{ "parse": { "text": "<p><b>Hello world</b></p>" } }'
 		] );
@@ -18,7 +18,7 @@
 	} );
 
 	QUnit.test( '.parse( Object.toString )', function ( assert ) {
-		this.server.respondWith( /action=parse.*&text='''Hello(\+|%20)world'''/, [ 200,
+		this.server.respondWith( 'POST', /api.php/, [ 200,
 			{ 'Content-Type': 'application/json' },
 			'{ "parse": { "text": "<p><b>Hello world</b></p>" } }'
 		] );
@@ -33,7 +33,7 @@
 	} );
 
 	QUnit.test( '.parse( mw.Title )', function ( assert ) {
-		this.server.respondWith( /action=parse.*&page=Earth/, [ 200,
+		this.server.respondWith( 'GET', /action=parse.*&page=Earth/, [ 200,
 			{ 'Content-Type': 'application/json' },
 			'{ "parse": { "text": "<p><b>Earth</b> is a planet.</p>" } }'
 		] );
-- 
2.20.1