Avoid passing integers to mw.RegExp.escape
authorSanthosh Thottingal <santhosh.thottingal@gmail.com>
Tue, 31 May 2016 09:48:14 +0000 (15:18 +0530)
committerSanthosh Thottingal <santhosh.thottingal@gmail.com>
Tue, 31 May 2016 09:48:14 +0000 (15:18 +0530)
Passing the index value of the loop to mw.RegExp.escape
causes javascript error at str.replace. mw.RegExp.escape
expects string parameters. It is not required to escape
the loop index here.

Follow-up to c81ab8ae5f89056d23a4b0d36066120b9701d07e.

Bug: T135743
Change-Id: I818bc2c671193e79848ae183f509473200c6abf5

resources/src/moment-locale-overrides.js

index 9af0598..dd33b00 100644 (file)
@@ -27,7 +27,7 @@ moment.locale( moment.locale(), {
                if ( mw.config.get( 'wgTranslateNumerals' ) ) {
                        for ( i = 0; i < 10; i++ ) {
                                if ( table[ i ] !== undefined ) {
-                                       s = s.replace( new RegExp( mw.RegExp.escape( i ), 'g' ), table[ i ] );
+                                       s = s.replace( new RegExp( i, 'g' ), table[ i ] );
                                }
                        }
                }