From: Roan Kattouw <catrope@users.mediawiki.org>
Date: Tue, 14 Dec 2010 16:56:56 +0000 (+0000)
Subject: Use $.getJSON()'s middle parameter rather than DIY URL building in AJAX watch JS
X-Git-Tag: 1.31.0-rc.0~33301
X-Git-Url: http://git.cyclocoop.org/%24image?a=commitdiff_plain;h=21977e3bfa67f42e1b05464a6c9efef1df7b3369;p=lhc%2Fweb%2Fwiklou.git

Use $.getJSON()'s middle parameter rather than DIY URL building in AJAX watch JS
---

diff --git a/resources/mediawiki.action/mediawiki.action.watch.ajax.js b/resources/mediawiki.action/mediawiki.action.watch.ajax.js
index 0ef5b85c18..72ff5e3d30 100644
--- a/resources/mediawiki.action/mediawiki.action.watch.ajax.js
+++ b/resources/mediawiki.action/mediawiki.action.watch.ajax.js
@@ -82,10 +82,18 @@ $( document ).ready( function() {
 		}
 
 		wgAjaxWatch.setLinkText( $link, $link.data( 'action' ) + 'ing' );
+		
+		var reqData = {
+			'action': 'watch',
+			'format': 'json',
+			'title': $link.data( 'target' )
+		};
+		if ( $link.data( 'action' ) == 'unwatch' ) {
+			reqData['unwatch'] = '';
+		}
 		$.getJSON( mw.config.get( 'wgScriptPath' )
-				+ '/api' + mw.config.get( 'wgScriptExtension' ) + '?action=watch&format=json&title='
-				+ encodeURIComponent( $link.data( 'target' ) )
-				+ ( $link.data( 'action' ) == 'unwatch' ? '&unwatch' : '' ),
+				+ '/api' + mw.config.get( 'wgScriptExtension' ),
+			reqData,
 			function( data, textStatus, xhr ) {
 				wgAjaxWatch.processResult( data, $link );
 			}