Page MenuHomePhabricator

ProofreadPage extension should handle its namespaces in a more sensible manner
Closed, ResolvedPublic

Description

Author: beau

Description:
Current configuration of ProofreadPage extension involves:

  • manually creating required namespaces
  • updating pages in MediaWiki namespace to point at those namespaces

The extension should configure its namespaces automatically. It should use English names as canonical ones and localized ones visible to users. It should also try to use the same namespace numbers, but this should be configurable if there are collisiions.


Version: unspecified
Severity: normal
See Also:
T48196: Dump stats: wikisource stats are much too low

Details

Reference
bz37483

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 12:27 AM
bzimport added a project: ProofreadPage.
bzimport set Reference to bz37483.

Are now all the namespace aliases for Wikisources redundant?
[[oldwikisource:User:Nemo_bis/wgNamespaceAliases]]

No, they aren't because:
1 Author namespace is not managed by ProofreadPage
2 The extension doesn't add aliases if the namespace is created in the wiki configuration an not by the extension.

A thing to do is maybe to remove creation of page and index namespaces in $wgExtraNamespaces in order to let the extension create namespaces and, so, create aliases.

(In reply to comment #4)

No, they aren't because:
1 Author namespace is not managed by ProofreadPage

Yes, I meant for Page: and Index:

2 The extension doesn't add aliases if the namespace is created in the wiki
configuration an not by the extension.

A thing to do is maybe to remove creation of page and index namespaces in
$wgExtraNamespaces in order to let the extension create namespaces and, so,
create aliases.

So I guess this works only for namespaces with standard numbers? If yes, it's better to have redundant $wgExtraNamespaces and $wgNamespaceAliases than a config with partial info.

If the namespaces related to numbers sets in $wgProofreadPageNamespaceIds don't exist, the Proofread Page extension create these namespaces with the numbers set in $wgProofreadPageNamespaceIds with labels stored in i18n of the extension.

So, Proofead Page can create namespaces with not standard numbers if $wgProofreadPageNamespaceIds is correctly setup.

So,
$wgLanguageCode = 'fr';
$wgProofreadPageNamespaceIds = array(

'page' => 102,
'index' => 104

);
will setup the same namspaces with the same ids as
$wgLanguageCode = 'fr';
$wgProofreadPageNamespaceIds = array(

'page' => 102,
'index' => 104

);
$wgExtraNamespaces[102] = 'Page';
$wgExtraNamespaces[104] = 'Livre';
$wgNamespaceAliases[102] = 'Page';
$wgNamespaceAliases[104] = 'Index';

Thank for the explanation, I'll ensure it's copied to the extension page (if it was already, excuse me).

I see $wgProofreadPageNamespaceIds defined in the WMF config, so indeed a cleanup there should be feasible. Can you file a bug so that we don't forget?

(In reply to comment #7)

Thank for the explanation, I'll ensure it's copied to the extension page (if
it
was already, excuse me).

I see $wgProofreadPageNamespaceIds defined in the WMF config, so indeed a
cleanup there should be feasible. Can you file a bug so that we don't forget?

That's bug 44320 but it has no updates in last month.
Also, does Proofread Page automatically add its namespaces to $wgNamespacesToBeSearchedDefault and $wgContentNamespaces ? That's an easily overloooked but very important piece of configuration and I find some numbers in WikiStats suspicious even after http://lists.wikimedia.org/pipermail/analytics/2013-August/000850.html

Sorry, I was busy in June and beginning of July so I haven't worked on it and now I'm in holidays. I'll make the cleaning in September.

No, Proofread Page doesn't automatically add its namespaces to
$wgNamespacesToBeSearchedDefault and $wgContentNamespaces. About $wgNamespacesToBeSearchedDefault I'm not sure it's a good idea to add Page: and Index: namespaces to it by default as most of Wikisources doesn't add the Page: namespace to $wgNamespacesToBeSearchedDefault. OK for $wgContentNamespaces. Feel free to open a new bug about it.