r98492 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r98491‎ | r98492 | r98493 >
Date:00:05, 30 September 2011
Author:johnduhart
Status:ok
Tags:
Comment:
Followup r98490, since the module names changed the variable names need changing as well
Modified paths:
  • /trunk/extensions/AbuseFilter/modules/ext.abuseFilter.edit.js (modified) (history)
  • /trunk/extensions/AbuseFilter/modules/ext.abuseFilter.examine.js (modified) (history)
  • /trunk/extensions/AbuseFilter/modules/ext.abuseFilter.tools.js (modified) (history)

Diff [purge]

Index: trunk/extensions/AbuseFilter/modules/ext.abuseFilter.edit.js
@@ -53,7 +53,7 @@
5454 * @param {Object} data Data returned from the AJAX request
5555 */
5656 this.processSyntaxResult = function( data ) {
57 - data = data.checkfiltersyntax;
 57+ data = data.abusefilterchecksyntax;
5858 $.removeSpinner( 'abusefilter-syntaxcheck' );
5959 $( '#mw-abusefilter-syntaxcheck' ).removeAttr("disabled");
6060
Index: trunk/extensions/AbuseFilter/modules/ext.abuseFilter.examine.js
@@ -68,7 +68,7 @@
6969 }
7070 } else {
7171 var exClass;
72 - if ( data.checkfiltermatch.result ) {
 72+ if ( data.abusefiltercheckmatch.result ) {
7373 exClass = 'mw-abusefilter-examine-match';
7474 msg = 'abusefilter-examine-match';
7575 } else {
Index: trunk/extensions/AbuseFilter/modules/ext.abuseFilter.tools.js
@@ -31,7 +31,7 @@
3232 $.removeSpinner( 'abusefilter-expr' );
3333
3434 $( '#mw-abusefilter-expr-result' )
35 - .html( mw.html.escape( data.evalfilterexpression.result ) );
 35+ .html( mw.html.escape( data.abusefilterevalexpression.result ) );
3636 };
3737
3838 /**
@@ -73,7 +73,7 @@
7474 break;
7575 }
7676 } else {
77 - msg = mw.message( 'abusefilter-reautoconfirm-done', data.unblockautopromote.user ).toString();
 77+ msg = mw.message( 'abusefilter-reautoconfirm-done', data.abusefilterunblockautopromote.user ).toString();
7878 }
7979
8080 mw.util.jsMessage( msg );

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r98490Followup r95572, rename API modules to reduce chance of collisions per CRjohnduhart23:30, 29 September 2011

Status & tagging log