jenkins-bot has submitted this change and it was merged.

Change subject: Add a space if the key ends on a space, even if the value is 
empty
......................................................................


Add a space if the key ends on a space, even if the value is empty

RT testing showed that this variant is more common, even if the value is
actually empty. Follow the data.

Change-Id: Ic694de7967ef8ae31880229f2e0d2d021694ec23
---
M js/lib/mediawiki.WikitextSerializer.js
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Subramanya Sastry: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/js/lib/mediawiki.WikitextSerializer.js 
b/js/lib/mediawiki.WikitextSerializer.js
index 4511da9..28b8120 100644
--- a/js/lib/mediawiki.WikitextSerializer.js
+++ b/js/lib/mediawiki.WikitextSerializer.js
@@ -2725,7 +2725,7 @@
                                                                        if 
(vStr && !/^\s/.test(v)) {
                                                                                
vStr = ' ' + vStr;
                                                                        }
-                                                               } else if 
(k.match(/\s$/) && vStr && !v.match(/^\s/)) {
+                                                               } else if 
(/\s$/.test(k) && !/^\s/.test(vStr)) {
                                                                        // Only 
prefix value with space if key
                                                                        // ends 
with space too
                                                                        vStr = 
' ' + vStr;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic694de7967ef8ae31880229f2e0d2d021694ec23
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: GWicke <[email protected]>
Gerrit-Reviewer: Subramanya Sastry <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to