Followup r80554: coding style, add var
authorRoan Kattouw <catrope@users.mediawiki.org>
Wed, 19 Jan 2011 17:49:05 +0000 (17:49 +0000)
committerRoan Kattouw <catrope@users.mediawiki.org>
Wed, 19 Jan 2011 17:49:05 +0000 (17:49 +0000)
resources/mediawiki.action/mediawiki.action.edit.js

index ebfc942..032dc64 100644 (file)
@@ -3,15 +3,14 @@
  */
 
 //make sure edit summary does not exceed byte limit
-$( '#wpSummary' ).attr( 'maxLength', 250 ).keypress( function (e) {
-               
+$( '#wpSummary' ).attr( 'maxLength', 250 ).keypress( function( e ) {
        // first check to see if this is actually a character key
        // being pressed.
        // Based on key-event info from http://unixpapa.com/js/key.html
        // JQuery should also normalize e.which to be consistent cross-browser,
        // however the same check is still needed regardless of jQuery.
 
-       if (e.which === 0 || e.charCode === 0 || e.ctrlKey || e.altKey || e.metaKey) {
+       if ( e.which === 0 || e.charCode === 0 || e.ctrlKey || e.altKey || e.metaKey ) {
                return true; //a special key (backspace, etc) so don't interfere.
        }
 
@@ -21,9 +20,9 @@ $( '#wpSummary' ).attr( 'maxLength', 250 ).keypress( function (e) {
        // and the actual character takes 4 bytes in UTF-8 (2*2=4). Might not work perfectly in edge cases
        // such as illegal sequences, but that should never happen.
 
-       len = this.value.replace(/[\u0080-\u07FF\uD800-\uDFFF]/g, '**').replace(/[\u0800-\uD7FF\uE000-\uFFFF]/g, '***').length;
+       var len = this.value.replace( /[\u0080-\u07FF\uD800-\uDFFF]/g, '**' ).replace( /[\u0800-\uD7FF\uE000-\uFFFF]/g, '***' ).length;
        //247 as this doesn't count character about to be inserted.
-       if (len > 247) {
+       if ( len > 247 ) {
                e.preventDefault();
        }
 });