Page MenuHomePhabricator

Remove deprecated $wgExtraGenderNamespaces
Closed, InvalidPublic

Description

Author: ginger

Description:
Per Bug 59113 we looked around the code for things that have been deprecated
and found $wgExtraGenderNamespaces at includes/DefaultSettings.php:3400


Version: 1.23.0
Severity: enhancement
Whiteboard: aklapper-moreinfo

Details

Reference
bz60456

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 3:04 AM
bzimport set Reference to bz60456.
bzimport added a subscriber: Unknown Object (MLST).

What makes you think this is deprecated?

I'd love to know *why* you think this is deprecated (references for statements are welcome).

[[mw:Manual:$wgExtraGenderNamespaces]] does not say that this is deprecated (should it use [[mw:Template:Deprecated]]?), and a quick grep shows:

./core/includes/DefaultSettings.php:$wgExtraGenderNamespaces = array();
./core/languages/Language.php: global $wgExtraGenderNamespaces;
./core/languages/Language.php: $ns = $wgExtraGenderNamespaces + self::$dataCache->getItem( $this->mCode, 'namespaceGenderAliases' );
./core/languages/Language.php: global $wgExtraGenderNamespaces, $wgExtraNamespaces;
./core/languages/Language.php: if ( count( $wgExtraGenderNamespaces ) > 0 ) {
./core/languages/Language.php: // $wgExtraGenderNamespaces overrides everything
./core/languages/Language.php: global $wgExtraGenderNamespaces;
./core/languages/Language.php: $genders = $wgExtraGenderNamespaces + (array)self::$dataCache->getItem( $this->mCode, 'namespaceGenderAliases' );

Currently I am tempted to close this report as INVALID. :)

ginger wrote:

Not actually deprecated. I misinterpreted the comments due to submitting a series of seven bugs at once. Thanks for letting me know!