Legoktm has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/384917 )
Change subject: Don't trigger PHP errors for unused Shell\Command ...................................................................... Don't trigger PHP errors for unused Shell\Command Change-Id: Id29da4f21a44ccb18d8a2ae11348d69ca3233aa5 (cherry picked from commit 4d9a95f97005705ade05c71d4c1a64fd3844272c) --- M includes/shell/Command.php M tests/phpunit/includes/shell/CommandTest.php 2 files changed, 5 insertions(+), 14 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/17/384917/1 diff --git a/includes/shell/Command.php b/includes/shell/Command.php index 8b83585..9afaf9c 100644 --- a/includes/shell/Command.php +++ b/includes/shell/Command.php @@ -83,12 +83,14 @@ */ public function __destruct() { if ( !$this->everExecuted ) { + $context = [ 'command' => $this->command ]; $message = __CLASS__ . " was instantiated, but execute() was never called."; if ( $this->method ) { - $message .= " Calling method: {$this->method}."; + $message .= ' Calling method: {method}.'; + $context['method'] = $this->method; } - $message .= " Command: {$this->command}"; - trigger_error( $message, E_USER_NOTICE ); + $message .= ' Command: {command}'; + $this->logger->warning( $message, $context ); } } diff --git a/tests/phpunit/includes/shell/CommandTest.php b/tests/phpunit/includes/shell/CommandTest.php index 33a7f44..dc49599 100644 --- a/tests/phpunit/includes/shell/CommandTest.php +++ b/tests/phpunit/includes/shell/CommandTest.php @@ -6,17 +6,6 @@ * @group Shell */ class CommandTest extends PHPUnit_Framework_TestCase { - /** - * @expectedException PHPUnit_Framework_Error_Notice - */ - public function testDestruct() { - if ( defined( 'HHVM_VERSION' ) ) { - $this->markTestSkipped( 'destructors are unreliable in HHVM' ); - } - $command = new Command(); - $command->params( 'true' ); - } - private function requirePosix() { if ( wfIsWindows() ) { $this->markTestSkipped( 'This test requires a POSIX environment.' ); -- To view, visit https://gerrit.wikimedia.org/r/384917 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id29da4f21a44ccb18d8a2ae11348d69ca3233aa5 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: REL1_30 Gerrit-Owner: Legoktm <lego...@member.fsf.org> Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits