Jdlrobson has uploaded a new change for review.
https://gerrit.wikimedia.org/r/288094
Change subject: Allow brackets in createImgThumbnail
......................................................................
Allow brackets in createImgThumbnail
Bug: T129177
Change-Id: I74748c7efe67954272ce0a539455b0b00001a26a
---
M resources/ext.popups.renderer.article.js
M tests/qunit/ext.popups.renderer.article.test.js
2 files changed, 7 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Popups
refs/changes/94/288094/1
diff --git a/resources/ext.popups.renderer.article.js
b/resources/ext.popups.renderer.article.js
index 4182c47..2c08275 100644
--- a/resources/ext.popups.renderer.article.js
+++ b/resources/ext.popups.renderer.article.js
@@ -402,7 +402,7 @@
article.createImgThumbnail = function ( className, url ) {
return $( '<div>' )
.addClass( className )
- .css( 'background-image', 'url(' + url + ')' );
+ .css( 'background-image', 'url("' + url.replace( /"/gi,
'\\"' ) + '")' );
};
/**
diff --git a/tests/qunit/ext.popups.renderer.article.test.js
b/tests/qunit/ext.popups.renderer.article.test.js
index 30dae3a..b4247e4 100644
--- a/tests/qunit/ext.popups.renderer.article.test.js
+++ b/tests/qunit/ext.popups.renderer.article.test.js
@@ -2,6 +2,12 @@
QUnit.module( 'ext.popups.renderer.renderers.article',
QUnit.newMwEnvironment() );
+ QUnit.test( 'render.article.createImgThumbnail', 1, function ( assert )
{
+ var $container =
mw.popups.render.renderers.article.createImgThumbnail( 'foo', '/w/test
"123"(bar).gif' );
+ console.log( $container );
+ assert.equal( $container.css( 'background-image' ),
'url("/w/test \\"123\\"(bar).gif")' );
+ } );
+
QUnit.test( 'render.article.getProcessedElements', function ( assert ) {
QUnit.expect( 14 );
--
To view, visit https://gerrit.wikimedia.org/r/288094
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I74748c7efe67954272ce0a539455b0b00001a26a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Popups
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
