Last modified: 2014-11-17 09:21:07 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 T66786, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64786 - Review the ExtraLanguageLink extension for deployment
Review the ExtraLanguageLink extension for deployment
Status: NEW
Product: Wikimedia
Classification: Unclassified
Extension setup (Other open bugs)
unspecified
All All
: Normal enhancement with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 64902 64903
Blocks: 16962 31235 32189
  Show dependency treegraph
 
Reported: 2014-05-03 08:04 UTC by This, that and the other (TTO)
Modified: 2014-11-17 09:21 UTC (History)
6 users (show)

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


Attachments

Description This, that and the other (TTO) 2014-05-03 08:04:26 UTC
This is a really simple extension that allows semi-arbitrary links to be appended to the "Languages" section of a page's sidebar. It can be locked down so that (for example) extra language links can only be added to the Main Page (bug 16962), or so that extra language links may only point to Old Wikisource (bug 32189).

It's only about 200 lines of code all up, so it shouldn't take very long to review.

Here is "Greg's checklist" for this extension:

== TODO/Check list ==
Extension page on mediawiki.org: Yes
Bugzilla component: Requested in bug 64784
Extension in Gerrit: Repo creation requested
Design Review: No, but since it is so simple, I doubt it really needs one
Architecture/Performance Review: Not yet
Security Review: Not yet
Community support: Not needed in the case of bug 16962 (ELL offers a more robust implementation of an existing "feature"); seems unnecessary in the case of bug 32189 (simple uncontroversial enhancement request)
Comment 1 This, that and the other (TTO) 2014-05-03 10:54:25 UTC
CCing a few people.
Comment 2 Bartosz Dziewoński 2014-05-03 21:03:51 UTC
Did you mean to CC somebody else than me? Because I don't see what I could be supposed to do about this. :)
Comment 3 This, that and the other (TTO) 2014-05-04 03:51:05 UTC
(In reply to Bartosz Dziewoński from comment #2)
> Did you mean to CC somebody else than me? Because I don't see what I could
> be supposed to do about this. :)

I'm not sure what I meant to do... You're welcome to un-CC yourself if you like.
Comment 4 Greg Grossmeier 2014-05-05 15:33:55 UTC
Architecture/Performance Review: bug 64903
Security Review: bug 64902

CC'ing Dan for Product perspective. Dan, let me know what you think on this one.
Comment 5 This, that and the other (TTO) 2014-05-06 08:08:44 UTC
Code's up in the repository.

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


Navigation
Links