Last modified: 2012-04-12 13:53:46 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 T27512, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 25512 - List of subcategories on a category page show the "category" ns prefix for all subcategories (didn't use to)
List of subcategories on a category page show the "category" ns prefix for al...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.16.x
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://beeldengeluidwiki.nl/index.php...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-10-13 10:14 UTC by BjornW
Modified: 2012-04-12 13:53 UTC (History)
2 users (show)

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


Attachments
A screenshot of an category index page in which all subcatgory titles are prefixed with the Dutch word for category (categorie) (157.59 KB, image/png)
2010-10-13 10:14 UTC, BjornW
Details
patch that removes the category prefix from the subcategory listing. (541 bytes, patch)
2010-10-21 18:40 UTC, Bawolff (Brian Wolff)
Details

Description BjornW 2010-10-13 10:14:21 UTC
Created attachment 7732 [details]
A screenshot of an category index page in which all subcatgory titles are prefixed with the Dutch word for category (categorie)

After updating our Wiki (Beeldengeluidwiki.nl) to the latest stable version (1.16.0) the category index pages such as (http://beeldengeluidwiki.nl/index.php/Categorie:Genres) have the title prefixed with the type (category) in the index. After consulting #mediawiki I got the advice to set the variable $wgCategoryPrefixedDefaultSortkey (http://www.mediawiki.org/wiki/Manual:$wgCategoryPrefixedDefaultSortkey) to false and running refreshLinks.php. 

After doing so I would expect the prefixes to have disappeared from the category index pages, instead I still see the prefixes as you can see in the attached screenshot.

I assume this is a bug in the expected behaviour of $wgCategoryPrefixedDefaultSortkey or am doing something wrong?
Comment 1 Bawolff (Brian Wolff) 2010-10-15 02:23:18 UTC
Thats the expected behaviour. $wgCategoryPrefixedDefaultSortkey just changes the sort order, not what is displayed. So if Talk:Foo is in a category, setting $wgCategoryPrefixedDefaultSortkey to false will cause it to be filed under F instead of T (assuming no alternate sortkey has been specified). However it will still be displayed as Talk:Foo no matter how $wgCategoryPrefixedDefaultSortkey is set.

However, in regards to the core issue you are having:
Showing the category prefix for each item on the subcategory list appears to be something thats new in 1.16. (specifically it seems to have been introduced in r53780 , potentially accidentally) I'm not sure if this is a behaviour we want or not.
Comment 2 Bawolff (Brian Wolff) 2010-10-15 02:34:31 UTC
Ignore what I said about r53780. I'm not sure what caused the change in behaviour, but that revision couldn't of been it since the old behaviour of not showing category was present after that revision
Comment 3 BjornW 2010-10-15 08:12:21 UTC
Mhhm how can I help in determining what the cause was of the current behaviour? I would like to have a way to remove it which is 'compatible' with the rest of the Mediawiki developments so it does not regress in a newer version.
Comment 4 Bawolff (Brian Wolff) 2010-10-21 18:40:57 UTC
Created attachment 7747 [details]
patch that removes the category prefix from the subcategory listing.

ok, it was caused by r53780. I got confused as the bug wasn't present on wikipedia, and wikipedia is running post-r53780 code. However, that is because wikipedia is using categorytree which overrides the category page display. If you disable categorytree, the bug is present - http://en.wikipedia.org/wiki/Category:Canada?notree

I'm attaching this as a patch instead of committing it since I currently know nothing about the language converter, and have no idea what r53780 was doing. Thus I need to take a moment to read up on lang converter before committing to make sure I'm not doing something stupid/wrong.
Comment 5 BjornW 2010-10-22 10:04:47 UTC
@Bawolff: Could you explain to me how I can safely patch 1.16 (aka the last stable version)? AFAIK r53780 is part of it. It seems to me your patch expects some other changes as well which are already part of trunk?
Comment 6 Bawolff (Brian Wolff) 2010-10-22 12:03:22 UTC
(In reply to comment #5)
> @Bawolff: Could you explain to me how I can safely patch 1.16 (aka the last
> stable version)? AFAIK r53780 is part of it. It seems to me your patch expects
> some other changes as well which are already part of trunk?

Replace the null on line 158 of CategoryPage.php (second argument to $this->getSkin()->link) with $title->getText();
Comment 7 BjornW 2010-10-22 12:19:31 UTC
Bawolff, works like a charm! Thanks for your help! I hope this gets fixed in the trunk as well :)
Comment 8 Bawolff (Brian Wolff) 2010-12-08 16:02:32 UTC
Crap, totally forgot about this.

fixed in r78071

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


Navigation
Links