Last modified: 2012-05-28 19:55:54 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 T37950, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 35950 - Replacing maintenance/Maintenance.php's STDOUT by print
Replacing maintenance/Maintenance.php's STDOUT by print
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Maintenance scripts (Other open bugs)
unspecified
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-13 18:43 UTC by christian
Modified: 2012-05-28 19:55 UTC (History)
1 user (show)

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


Attachments

Description christian 2012-04-13 18:43:13 UTC
The current use of STDOUT in the output functions of core/maintenance/Maintenance.php is problematic for PHPUnit testing, as we cannot meaningfully rewire it without causing much trouble.

For the cli sapi, Maintenance currently uses STDOUT, for other sapis, it uses print.

If we'd change STDOUT to prints regardless of the sapi, this would solve the problems for testing. And by what I have seen from the maintenance code, this should not cause problems. ... But I am just a
newbie :D

Are there any reasons to stick to STDOUT, or corner cases that would require STDOUT?
Comment 1 christian 2012-05-28 19:55:54 UTC
Patch to remove the STDOUTs that cause problems has been committed upstream.

Git commit: 0c33345146fc6cfb05b25614d923b875cc4cfd5b
Gerrit link: https://gerrit.wikimedia.org/r/#/c/4981/

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


Navigation
Links