wikimedia/mediawiki-extensions-Popups
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/Popups synced 2024-11-27 17:00:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Move the article renderer in to the renderers property"

Browse source
This commit is contained in:
jenkins-bot 2015-10-01 08:50:39 +00:00 committed by Gerrit Code Review
parent 495014a917 1565a1d3c9
commit 486e34fb38
3 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
resources/ext.popups.renderer.article.js
View file

@ -564,6 +564,6 @@
}
};
mw.popups.render.article = article;
mw.popups.render.renderers.article = article;
} )( jQuery, mediaWiki );

8
resources/ext.popups.renderer.js
View file

@ -54,6 +54,12 @@
*/
mw.popups.render.currentRequest = undefined;
/**
* Object to store all renderers
* @property {Object} renderers
*/
mw.popups.render.renderers = {};
/**
* Close all other popups and render the new one from the cache
* or by finding and calling the correct renderer
@ -104,7 +110,7 @@
.done( function () {
// TODO: check for link type and call correct renderer
// There is only one popup type right now so it isn't necessary
var cachePopup = mw.popups.render.article.init( link );
var cachePopup = mw.popups.render.renderers.article.init( link );
mw.popups.render.openTimer = mw.popups.render.wait( mw.popups.render.POPUP_DELAY - mw.popups.render.API_DELAY );

4
tests/qunit/ext.popups.renderer.article.test.js
View file

@ -1,13 +1,13 @@
( function ( $, mw ) {
QUnit.module( 'ext.popups.renderer.article', QUnit.newMwEnvironment() );
QUnit.module( 'ext.popups.renderer.renderers.article', QUnit.newMwEnvironment() );
QUnit.test( 'render.article.getProcessedElements', function ( assert ) {
QUnit.expect( 13 );
function test( extract, title, expected, msg ) {
var $div = $( '<div>' ).append(
mw.popups.render.article.getProcessedElements( extract, title )
mw.popups.render.renderers.article.getProcessedElements( extract, title )
);
assert.equal( $div.html(), expected, msg );
}