Ticket #2069: 2069.patch

FCKeditor.body.php

@@ -123 +123 @@
                                 
                 return true;
         }
+        
+        public function onSanitizerAfterFixTagAttributes($text, $element, &$attribs)
+        {
+                $text = preg_match_all("/Fckmw\d+fckmw/", $text, $matches);
+                
+                if (!empty($matches[0][0])) {
+                        global $leaveRawTemplates;
+                        if (!isset($leaveRawTemplates)) {
+                                $leaveRawTemplates = array();
+                        }
+                        $leaveRawTemplates = array_merge($leaveRawTemplates, $matches[0]);
+                        $attribs = array_merge($attribs, $matches[0]);
+                }
 
+                return true;
+        }
+
         public function registerHooks() {
                 global $wgHooks, $wgExtensionFunctions;
 
@@ -138 +154 @@
                 $wgHooks['LanguageGetMagic'][]                  = array($this, "onLanguageGetMagic");
                 $wgHooks['ParserBeforeInternalParse'][]         = array($this, "onParserBeforeInternalParse");
                 $wgHooks['EditPageBeforeConflictDiff'][]                = array($this, 'onEditPageBeforeConflictDiff');
+                $wgHooks['SanitizerAfterFixTagAttributes'][]    = array($this, 'onSanitizerAfterFixTagAttributes');
 
                 if ($this->debug) {
                         /*

FCKeditorParser.body.php

@@ -463 +463 @@
                 }
 
                 if (!empty($this->fck_mw_strtr_span)) {
+                        global $leaveRawTemplates;
+                        if (!empty($leaveRawTemplates)) {
+                                foreach ($leaveRawTemplates as $l) {
+                                        $this->fck_mw_strtr_span[$l] = substr($this->fck_mw_strtr_span[$l], 30, -7);
+                                }
+                        }
                         $parserOutput->setText(strtr($parserOutput->getText(), $this->fck_mw_strtr_span));
                 }
 

mw12/FCKeditorParser_OldPP.body.php

@@ -463 +463 @@
                 }
 
                 if (!empty($this->fck_mw_strtr_span)) {
+                        global $leaveRawTemplates;
+                        if (!empty($leaveRawTemplates)) {
+                                foreach ($leaveRawTemplates as $l) {
+                                        $this->fck_mw_strtr_span[$l] = substr($this->fck_mw_strtr_span[$l], 30, -7);
+                                }
+                        }
                         $parserOutput->setText(strtr($parserOutput->getText(), $this->fck_mw_strtr_span));
                 }
                 if (!empty($this->fck_matches)) {

plugins/mediawiki/fckplugin.js

@@ -672 +672 @@
                                 else
                                         sAttValue = htmlNode.getAttribute( sAttName, 2 ) ;      // We must use getAttribute to get it exactly as it is defined.
 
-                                attStr += ' ' + sAttName + '="' + String(sAttValue).replace( '"', '"' ) + '"' ;
+                                // leave templates
+                                if ( sAttName.StartsWith( '{{' ) && sAttName.EndsWith( '}}' ) ) {
+                                        attStr += ' ' + sAttName ;
+                                }
+                                else {
+                                        attStr += ' ' + sAttName + '="' + String(sAttValue).replace( '"', '"' ) + '"' ;
+                                }
                         }
                 }
                 return attStr ;