Last modified: 2010-11-02 23:44:32 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 T27700, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 25700 - Build new LandingCheck Extension (replaces GeoLite)
Build new LandingCheck Extension (replaces GeoLite)
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Extensions requests (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Ryan Kaldari
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-10-29 00:17 UTC by Ryan Kaldari
Modified: 2010-11-02 23:44 UTC (History)
3 users (show)

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


Attachments

Description Ryan Kaldari 2010-10-29 00:17:48 UTC
Now that we have geolookup on pageview, we don't actually need GeoLite anymore. All we really need is an extension that checks to see if a page exists or not. Here's how it should work ideally:

The target for a banner would be set by Javascript to http://www.wikimediafoundation.org/wiki/Special:LandingCheck?target=<pagename>&language=<languagecode>&country=<countrycode> (where the languagecode and countrycode are pulled from global vars). So for example:
http://www.wikimediafoundation.org/wiki/Special:LandingCheck?target=Appeal2&language=fr&country=DE

The new LandingCheck extension would then check to make sure that http://www.wikimediafoundation.org/wiki/Appeal2/fr/DE exists.

If it does, it redirects the user there. If it doesn't, it falls back to http://www.wikimediafoundation.org/wiki/Appeal2/fr

We could also use an optional fallback parameter to override the default fallback. It would also need to be able to pass along other query string params for tracking, etc.

Psuedocode:
if pagename includes "Special:" {
   test with SpecialPage::exists()
} else {
   test with Title::exists()
}
Comment 1 Ryan Kaldari 2010-11-02 23:44:32 UTC
Initial version is now checked in.

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


Navigation
Links