GWicke has uploaded a new change for review.
https://gerrit.wikimedia.org/r/73006
Change subject: Handle VE sending template parameter keys with leading/trailing
spaces
......................................................................
Handle VE sending template parameter keys with leading/trailing spaces
Those would lead to a crash in the serializer before.
Change-Id: I3239807d6a4eb569545b4317aad32208036740a6
---
M js/lib/mediawiki.WikitextSerializer.js
1 file changed, 8 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid
refs/changes/06/73006/1
diff --git a/js/lib/mediawiki.WikitextSerializer.js
b/js/lib/mediawiki.WikitextSerializer.js
index d8140e1..0c0bc8c 100644
--- a/js/lib/mediawiki.WikitextSerializer.js
+++ b/js/lib/mediawiki.WikitextSerializer.js
@@ -2953,9 +2953,14 @@
// then push out remaining (new) parameters
keys.forEach(function(k) {
// Don't allow whitespace in keys
- k = k.trim();
- if (origKeys.indexOf(k) === -1) {
- pushArg(k);
+
+ var strippedK = k.trim();
+ if (origKeys.indexOf(strippedK) === -1)
{
+ if (strippedK !== k) {
+ // copy over
+ tpl.params[strippedK] =
tpl.params[k];
+ }
+ pushArg(strippedK);
}
});
--
To view, visit https://gerrit.wikimedia.org/r/73006
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3239807d6a4eb569545b4317aad32208036740a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: GWicke <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
