Last modified: 2014-01-04 19:43:40 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T28514, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 26514 - let action=parse&prop=languageshtml return the right html
let action=parse&prop=languageshtml return the right html
Status: RESOLVED INVALID
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.18.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-12-30 19:54 UTC by Umherirrender
Modified: 2014-01-04 19:43 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Umherirrender 2010-12-30 19:54:30 UTC
The prop languageshtml returns the html for the otherlanguages section. But this section is in the most skin a html list and not a pipe seperator list.

It is possible to have a html list in that skins where it is a html list? (MonoBookTemplate (includes Skins MonoBook, Modern, Chick, Simple, MySkin) and VectorTemplate (includes Vector skin))

Thanks.
Comment 1 Sam Reed (reedy) 2010-12-30 20:02:24 UTC
CC'd Priyanka

Added in r77896

+	private function categoriesHtml( $categories ) {
+		global $wgOut, $wgUser;
+		$wgOut->addCategoryLinks( $categories );
+		$sk = $wgUser->getSkin();
+		return $sk->getCategories();
+	}


Surely that does what you're asking for...?

Unless there's some extra skin parsing somewhere along the way
Comment 2 Umherirrender 2010-12-30 20:36:20 UTC
It is the following (languagesHtml, not categoriesHtml):

+	private function languagesHtml( $languages ) {
+		global $wgOut, $wgUser;
+		$wgOut->setLanguageLinks( $languages );
+		$sk = $wgUser->getSkin();
+		return $sk->otherLanguages();
+	}

The function is not right for MonoBookTemplate and VectorTemplate, because there it is done at another place, not by overriding otherLanguages.
Comment 3 Umherirrender 2014-01-04 19:43:40 UTC
languageshtml was deprecated with Gerrit change #96364, so no need to fix this anymore.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links