Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

mediawiki.api: Don't apply file extension stripping to category names

Browse source 
`mw.Title#getNameText()` strips anything following a dot (.), assuming
it's a file name (which doesn't make sense for category names).

C.f. https://doc.wikimedia.org/mediawiki-core/master/js/#!/api/mw.Title-method-getMainText

This causes some trouble in UploadWizard's category-suggestion system.

This bug has been there since the very first version (cb0cf72eba)
which, ironically, was specifically written for UploadWizard.

Bug: 64764
Change-Id: I2925e1050215272cdf03c8c6b1de6ba79348f6a5
This commit is contained in:
rillke 2014-05-03 21:34:23 +02:00 committed by Timo Tijhof
parent 9db61c9ab5
commit 4ffb4cf0b0
3 changed files with 33 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
resources/src/mediawiki.api/mediawiki.api.category.js
View file

@ -73,7 +73,7 @@
var texts = []; var texts = [];
if ( data.query && data.query.allpages ) { if ( data.query && data.query.allpages ) {
$.each( data.query.allpages, function ( i, category ) { $.each( data.query.allpages, function ( i, category ) {
texts.push( new mw.Title( category.title ).getNameText() ); texts.push( new mw.Title( category.title ).getMainText() );
} ); } );
} }
return texts; return texts;

2
tests/qunit/QUnitTestResources.php
View file

@ -71,6 +71,7 @@ return array(
'tests/qunit/suites/resources/mediawiki/mediawiki.user.test.js', 'tests/qunit/suites/resources/mediawiki/mediawiki.user.test.js',
'tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js', 'tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js',
'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js', 'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js',
'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.category.test.js',
'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js', 'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.parse.test.js',
'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.watch.test.js', 'tests/qunit/suites/resources/mediawiki.api/mediawiki.api.watch.test.js',
'tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js', 'tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js',
@ -96,6 +97,7 @@ return array(
'jquery.tablesorter', 'jquery.tablesorter',
'jquery.textSelection', 'jquery.textSelection',
'mediawiki.api', 'mediawiki.api',
'mediawiki.api.category',
'mediawiki.api.parse', 'mediawiki.api.parse',
'mediawiki.api.watch', 'mediawiki.api.watch',
'mediawiki.jqueryMsg', 'mediawiki.jqueryMsg',

30
tests/qunit/suites/resources/mediawiki.api/mediawiki.api.category.test.js Normal file
View file

@ -0,0 +1,30 @@
( function ( mw ) {
QUnit.module( 'mediawiki.api.category', QUnit.newMwEnvironment( {
setup: function () {
this.server = this.sandbox.useFakeServer();
}
} ) );
QUnit.test( '.getCategoriesByPrefix()', function ( assert ) {
QUnit.expect( 1 );
var api = new mw.Api();
api.getCategoriesByPrefix( 'Foo' ).done( function ( matches ) {
assert.deepEqual(
matches,
[ 'Food', 'Fool Supermarine S.6', 'Fools' ]
);
} );
this.server.respond( function ( req ) {
req.respond( 200, { 'Content-Type': 'application/json' },
'{ "query": { "allpages": [ ' +
'{ "title": "Category:Food" },' +
'{ "title": "Category:Fool Supermarine S.6" },' +
'{ "title": "Category:Fools" }' +
'] } }'
);
} );
} );
}( mediaWiki ) );