Last modified: 2013-01-28 11:29:12 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 T27589, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 25589 - Hardcoded namespace checks
Hardcoded namespace checks
Status: RESOLVED WORKSFORME
Product: mwEmbed
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Normal major (vote)
: ---
Assigned To: Michael Dale
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2010-10-19 19:45 UTC by Derk-Jan Hartman
Modified: 2013-01-28 11:29 UTC (History)
1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Derk-Jan Hartman 2010-10-19 19:45:53 UTC 
Currently many parts of the tools have hardcoded namespace checks such as "indexOf( "File:") and Template:

This should be adapted to a general check routine that can recognize both normalized namespaces and localized namespace names. (Provided by wgFormattedNamespaces ).
Comment 1 Michael Dale 2010-10-19 20:05:52 UTC 
I agree. But this is a bit tricky with cross site requests as each site has their own namespace config that does not necessarily match canonical namespace of the other site. So site A could have 102 -> wikiProject in Spanish while site B could have 102 -> TimedText in a bunch of languages. 

So the representation needs to be per api-domain.. which is oky since the authoritative lookup could correspond with api request and most of the lookups could use the local wgFormattedNamespaces var.
Comment 2 Andre Klapper 2012-12-02 14:22:53 UTC 
Wondering if this is still an issue now that native support for mwEmbed via TMH is in place.

TheDJ, mdale: Is this still an issue?
Comment 3 Andre Klapper 2013-01-26 00:41:24 UTC 
TheDJ, mdale: Is this still an issue?
Comment 4 Michael Dale 2013-01-26 00:51:29 UTC 
Not sure. I think the issue can be resolved working, in TMH we use namespaces wherever we can. And the old tool ( mwEmbed ) has been deprecated.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links