Marcoil has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/133250

Change subject: Move data-mw-arginfo to dataAttribs
......................................................................

Move data-mw-arginfo to dataAttribs

Change-Id: I1f26222a933341b2746008a8f1f23282c478cb7d
---
M lib/dom.wrapTemplates.js
M lib/ext.core.TemplateHandler.js
2 files changed, 4 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/parsoid 
refs/changes/50/133250/1

diff --git a/lib/dom.wrapTemplates.js b/lib/dom.wrapTemplates.js
index 52bd25e..d92a255 100644
--- a/lib/dom.wrapTemplates.js
+++ b/lib/dom.wrapTemplates.js
@@ -455,10 +455,9 @@
                r = tplRanges[i];
 
                // Extract argInfo and clear it
-               var argInfo = r.startElem.getAttribute("data-mw-arginfo");
+               var argInfo = DU.getDataParsoid(r.startElem)['data-mw-arginfo'];
                if (argInfo) {
                        argInfo = JSON.parse(argInfo);
-                       r.startElem.removeAttribute("data-mw-arginfo");
                }
 
                env.log("trace/tplwrap/merge", function() {
diff --git a/lib/ext.core.TemplateHandler.js b/lib/ext.core.TemplateHandler.js
index e98481e..07fddbb 100644
--- a/lib/ext.core.TemplateHandler.js
+++ b/lib/ext.core.TemplateHandler.js
@@ -797,7 +797,7 @@
                                async.each(params, getParamHTML.bind(this), 
function (err) {
                                        // Use a data-attribute to prevent the 
sanitizer from stripping this
                                        // attribute before it reaches the DOM 
pass where it is needed.
-                                       chunk[0].attribs.push(new 
KV("data-mw-arginfo", JSON.stringify(argInfo)));
+                                       chunk[0].dataAttribs['data-mw-arginfo'] 
= JSON.stringify(argInfo);
                                        env.dp( 
'TemplateHandler._encapsulateTemplate', chunk );
                                        cb({tokens: chunk});
                                }.bind(this));
@@ -805,11 +805,11 @@
                                cb({tokens: [], async: true});
                                return;
                        } else {
-                               chunk[0].attribs.push(new KV("data-mw-arginfo", 
JSON.stringify(argInfo)));
+                               chunk[0].dataAttribs['data-mw-arginfo'] = 
JSON.stringify(argInfo);
                        }
                } else {
                        // Don't add the HTML template parameters, just use 
their wikitext
-                       chunk[0].attribs.push(new KV("data-mw-arginfo", 
JSON.stringify(argInfo)));
+                       chunk[0].dataAttribs['data-mw-arginfo'] = 
JSON.stringify(argInfo);
                }
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/133250
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f26222a933341b2746008a8f1f23282c478cb7d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/parsoid
Gerrit-Branch: dom_tpl_params_4
Gerrit-Owner: Marcoil <marc...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to