Last modified: 2013-07-18 09:14: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 T53576, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51576 - Investigate whether MW_NO_OUTPUT_BUFFER can be removed from WebStart.php
Investigate whether MW_NO_OUTPUT_BUFFER can be removed from WebStart.php
Status: NEW
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.22.0
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-17 21:38 UTC by MZMcBride
Modified: 2013-07-18 09:14 UTC (History)
4 users (show)

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


Attachments

Description MZMcBride 2013-07-17 21:38:07 UTC
As reported by e0je in #mediawiki earlier today, includes/WebStart.php currently contains the following line (cf. <https://git.wikimedia.org/blob/mediawiki%2Fcore.git/cd951c7458abfd91c8564874d5ffe08dc7b1b817/includes%2FWebStart.php#L149>):

---
if ( !defined( 'MW_NO_OUTPUT_BUFFER' ) && ob_get_level() == 0 ) {
---

Searching through the current codebase, there were no other references to "MW_NO_OUTPUT_BUFFER".

Researching the history found the following:


Tim introduced "MW_NO_OUTPUT_BUFFER" on 2007-02-19:
* https://git.wikimedia.org/commit/mediawiki%2Fcore.git/28dc3ec8884af80faaa996e72be4f945e8466c95
* https://git.wikimedia.org/blobdiff/mediawiki%2Fcore.git/28dc3ec8884af80faaa996e72be4f945e8466c95/includes%2FWebStart.php
* https://www.mediawiki.org/wiki/Special:Code/MediaWiki/19996

This change added "MW_NO_OUTPUT_BUFFER" to includes/WebStart.php and config/index.php.


Brion moved most of config/index.php's logic to config/Installer.php on 2009-08-19:
* https://git.wikimedia.org/commitdiff/mediawiki%2Fcore.git/eec1e42eea8657172f5f226ef13412e12703cfa5
* https://www.mediawiki.org/wiki/Special:Code/MediaWiki/55330


Chad H. deleted config/Installer.php on 2010-12-10:
* https://git.wikimedia.org/commit/mediawiki%2Fcore.git/fc16010af31a353c6fc0ad8c0020920c8ff1cacf
* https://www.mediawiki.org/wiki/Special:Code/MediaWiki/78180


As far as I can tell, MW_NO_OUTPUT_BUFFER is no longer defined in the codebase and the check in includes/WebStart.php is no longer needed. But perhaps I'm mistaken.
Comment 1 Tim Starling 2013-07-17 22:59:55 UTC
Actually, the bug is that it's unused, not that it exists. I filed bug 51582 for this.

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


Navigation
Links