From 1d70e7b81c12dcbe5029560ed2567ecfa1968687 Mon Sep 17 00:00:00 2001
From: Gabriel Wicke <gwicke@wikimedia.org>
Date: Fri, 27 Apr 2012 10:44:35 +0200
Subject: [PATCH] Disable preformatted text from indents in template args

Change-Id: I84144d3fab6541ed264d9b092806c8bf9de6e8b2
---
 modules/parser/ext.core.LinkHandler.js |  3 ++-
 modules/parser/pegTokenizer.pegjs.txt  | 29 +++++++++++++-------------
 2 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/modules/parser/ext.core.LinkHandler.js b/modules/parser/ext.core.LinkHandler.js
index 1b9a72e447..399b407aa3 100644
--- a/modules/parser/ext.core.LinkHandler.js
+++ b/modules/parser/ext.core.LinkHandler.js
@@ -328,7 +328,8 @@ ExternalLinkHandler.prototype.rank = 1.15;
 ExternalLinkHandler.prototype._imageExtensions = {
 	'jpg': true,
 	'png': true,
-	'gif': true
+	'gif': true,
+	'svg': true
 };
 
 ExternalLinkHandler.prototype._isImageLink = function ( href ) {
diff --git a/modules/parser/pegTokenizer.pegjs.txt b/modules/parser/pegTokenizer.pegjs.txt
index 2c5a8f3b3d..ada2cbf01b 100644
--- a/modules/parser/pegTokenizer.pegjs.txt
+++ b/modules/parser/pegTokenizer.pegjs.txt
@@ -520,7 +520,7 @@ block_line
       ) { 
           return st.concat(r); 
       }
-  / pre_indent
+  / ! { return stops.counters.nopre } pre_indent
   / pre
 
 /*
@@ -846,14 +846,14 @@ tplarg_or_template
     )
 
 template
-  = "{{" (newline / space)* 
-    target:template_name
-    params:(( newline / space )* "|"
-                r:( (newline / space )* &"|" { return new KV( '', '') } // empty argument
+  = "{{" nl_comment_space*
+    target:template_param_value
+    params:(nl_comment_space* "|"
+                r:( nl_comment_space* &"|" { return new KV( '', '') } // empty argument
                     / p:template_param { return p } 
                   ) { return r } 
             )* 
-    ( newline / space )* 
+    nl_comment_space* 
     "}}" {
       // Insert target as first positional attribute, so that it can be
       // generically expanded. The TemplateHandler then needs to shift it out
@@ -871,15 +871,15 @@ template
 
 tplarg 
   = "{{{" 
-    name:template_name?
-    params:( ( space / newline )* 
-              '|' ( space / newline )*  
+    name:template_param_value?
+    params:( nl_comment_space* 
+              '|' nl_comment_space*  
                r:( 
                     &'}}' { return new KV( '', '') }
                     / p:template_param { return p } 
                ) { return r }
            )* 
-    ( space / newline )* 
+    nl_comment_space* 
     "}}}" {
       name = flatten( name );
       params.unshift( { k: '', v: name } );
@@ -918,8 +918,6 @@ template_param
   / & [|}] { return new KV([], []); }
 
 
-template_name = template_param_value
-
 // FIXME: handle template args and templates in key! (or even parser functions?)
 template_param_name
   = & { return stops.push( 'equal', true ) }
@@ -934,14 +932,15 @@ template_param_name
   //= h:( !"}}" x:([^=|\n]) { return x } )* { return h.join(''); }
 
 template_param_value
-  = & { return stops.push( 'equal', false ) }
+  = & { stops.inc( 'nopre' ); return stops.push( 'equal', false ) }
     tpt:template_param_text
     {
+        stops.dec( 'nopre' );
         stops.pop( 'equal' );
         //console.warn( 'template param value matched: ' + pp( tpt ) );
         return tpt;
     }
-  / & { return stops.pop( 'equal' ) }
+  / & { stops.dec( 'nopre' ); return stops.pop( 'equal' ) }
 
 template_param_text
   = & { /*console.warn( 'tpt: ' + 
@@ -1881,6 +1880,8 @@ sol
         return nl.concat(cn, niToken);
     }
 
+nl_comment_space = newline / comment / space
+
 /**
  * noinclude / includeonly / onlyinclude productions. These are normally
  * handled by the generic_tag production, except where generic tags are not