From a640a455f35ccec26a80e9f9de5693dbb9e71c85 Mon Sep 17 00:00:00 2001 From: Tristan Darricau Date: Fri, 9 May 2014 00:07:56 +0200 Subject: [ticket/12074] Managing extensions doesn't produce any log entry PHPBB3-12074 --- phpBB/phpbb/console/command/extension/command.php | 6 +++++- phpBB/phpbb/console/command/extension/disable.php | 2 ++ phpBB/phpbb/console/command/extension/enable.php | 2 ++ phpBB/phpbb/console/command/extension/purge.php | 2 ++ 4 files changed, 11 insertions(+), 1 deletion(-) (limited to 'phpBB/phpbb/console/command/extension') diff --git a/phpBB/phpbb/console/command/extension/command.php b/phpBB/phpbb/console/command/extension/command.php index edde7ce2e2..d133c2224f 100644 --- a/phpBB/phpbb/console/command/extension/command.php +++ b/phpBB/phpbb/console/command/extension/command.php @@ -13,9 +13,13 @@ abstract class command extends \phpbb\console\command\command /** @var \phpbb\extension\manager */ protected $manager; - function __construct(\phpbb\extension\manager $manager) + /** @var \phpbb\log\log */ + protected $log; + + function __construct(\phpbb\extension\manager $manager, \phpbb\log\log $log) { $this->manager = $manager; + $this->log = $log; parent::__construct(); } diff --git a/phpBB/phpbb/console/command/extension/disable.php b/phpBB/phpbb/console/command/extension/disable.php index e4de70ca34..fb0c23762f 100644 --- a/phpBB/phpbb/console/command/extension/disable.php +++ b/phpBB/phpbb/console/command/extension/disable.php @@ -35,11 +35,13 @@ class disable extends command if ($this->manager->enabled($name)) { + $this->log->add('critical', ANONYMOUS, '', 'LOG_EXT_DISABLE_ERROR', time(), array($name)); $output->writeln("Could not disable extension $name"); return 1; } else { + $this->log->add('admin', ANONYMOUS, '', 'LOG_EXT_DISABLE', time(), array($name)); $output->writeln("Successfully disabled extension $name"); return 0; } diff --git a/phpBB/phpbb/console/command/extension/enable.php b/phpBB/phpbb/console/command/extension/enable.php index ee7dae76aa..e20a5ba81d 100644 --- a/phpBB/phpbb/console/command/extension/enable.php +++ b/phpBB/phpbb/console/command/extension/enable.php @@ -35,11 +35,13 @@ class enable extends command if ($this->manager->enabled($name)) { + $this->log->add('admin', ANONYMOUS, '', 'LOG_EXTENSION_ENABLE', time(), array($name)); $output->writeln("Successfully enabled extension $name"); return 0; } else { + $this->log->add('critical', ANONYMOUS, '', 'LOG_EXT_ENABLE_ERROR', time(), array($name)); $output->writeln("Could not enable extension $name"); return 1; } diff --git a/phpBB/phpbb/console/command/extension/purge.php b/phpBB/phpbb/console/command/extension/purge.php index c2e1d2928c..234d32f302 100644 --- a/phpBB/phpbb/console/command/extension/purge.php +++ b/phpBB/phpbb/console/command/extension/purge.php @@ -35,11 +35,13 @@ class purge extends command if ($this->manager->enabled($name)) { + $this->log->add('critical', ANONYMOUS, '', 'LOG_EXT_PURGE_ERROR', time(), array($name)); $output->writeln("Could not purge extension $name"); return 1; } else { + $this->log->add('admin', ANONYMOUS, '', 'LOG_EXT_PURGE', time(), array($name)); $output->writeln("Successfully purge extension $name"); return 0; } -- cgit v1.2.1