Last modified: 2014-08-10 21:10:55 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 T71087, 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 69087 - Special:Block: Use Config instead of globals
Special:Block: Use Config instead of globals
Status: NEW
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.24rc
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: config-obj
  Show dependency tree / graph
 
Reported: 2014-08-04 00:43 UTC by Kunal Mehta (Legoktm)
Modified: 2014-08-10 21:10 UTC (History)
1 user (show)

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

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

Description Kunal Mehta (Legoktm) 2014-08-04 00:43:39 UTC 
There are a lot of static funcitons in SpecialBlock so that the API can also re-use the same code which makes this difficult. Might need some more refactoring...?
Comment 1 Umherirrender 2014-08-04 09:57:23 UTC 
Static function SpecialBlock::processForm has a context source as param, which can provide a config object, but all the other functions needs another way to do.
Comment 2 Gerrit Notification Bot 2014-08-09 23:09:16 UTC 
Change 153266 had a related patch set uploaded by Florianschmidtwelzow:
WIP: Special:Block: Use config object instead of globals

https://gerrit.wikimedia.org/r/153266
Comment 3 Gerrit Notification Bot 2014-08-10 19:06:07 UTC 
Change 153266 abandoned by Florianschmidtwelzow:
WIP: Special:Block: Use config object instead of globals

Reason:
maybe first think about and then do something :/

https://gerrit.wikimedia.org/r/153266
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