mirror of
https://github.com/nextcloud/server.git
synced 2026-04-21 06:08:46 -04:00
Add a command to set a system config value
This commit is contained in:
Joas Schilling
parent
5370bba0b6
commit
a83eac3762
3 changed files with 199 additions and 0 deletions
83
core/command/config/system/setconfig.php
Normal file
83
core/command/config/system/setconfig.php
Normal file
|
|
@ -0,0 +1,83 @@
|
|||
<?php
|
||||
/**
|
||||
* @author Joas Schilling <nickvergessen@owncloud.com>
|
||||
*
|
||||
* @copyright Copyright (c) 2015, ownCloud, Inc.
|
||||
* @license AGPL-3.0
|
||||
*
|
||||
* This code is free software: you can redistribute it and/or modify
|
||||
* it under the terms of the GNU Affero General Public License, version 3,
|
||||
* as published by the Free Software Foundation.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU Affero General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU Affero General Public License, version 3,
|
||||
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
||||
*
|
||||
*/
|
||||
|
||||
namespace OC\Core\Command\Config\System;
|
||||
|
||||
use OC\Core\Command\Base;
|
||||
use OC\SystemConfig;
|
||||
use OCP\IAppConfig;
|
||||
use Symfony\Component\Console\Input\InputArgument;
|
||||
use Symfony\Component\Console\Input\InputInterface;
|
||||
use Symfony\Component\Console\Input\InputOption;
|
||||
use Symfony\Component\Console\Output\OutputInterface;
|
||||
|
||||
class SetConfig extends Base {
|
||||
/** * @var SystemConfig */
|
||||
protected $systemConfig;
|
||||
|
||||
/**
|
||||
* @param SystemConfig $systemConfig
|
||||
*/
|
||||
public function __construct(SystemConfig $systemConfig) {
|
||||
parent::__construct();
|
||||
$this->systemConfig = $systemConfig;
|
||||
}
|
||||
|
||||
protected function configure() {
|
||||
parent::configure();
|
||||
|
||||
$this
|
||||
->setName('config:system:set')
|
||||
->setDescription('Set a system config value')
|
||||
->addArgument(
|
||||
'name',
|
||||
InputArgument::REQUIRED,
|
||||
'Name of the config to set'
|
||||
)
|
||||
->addOption(
|
||||
'value',
|
||||
null,
|
||||
InputOption::VALUE_REQUIRED,
|
||||
'The new value of the config'
|
||||
)
|
||||
->addOption(
|
||||
'update-only',
|
||||
null,
|
||||
InputOption::VALUE_NONE,
|
||||
'Only updates the value, if it is not set before, it is not being added'
|
||||
)
|
||||
;
|
||||
}
|
||||
|
||||
protected function execute(InputInterface $input, OutputInterface $output) {
|
||||
$configName = $input->getArgument('name');
|
||||
|
||||
if (!in_array($configName, $this->systemConfig->getKeys()) && $input->hasParameterOption('--update-only')) {
|
||||
$output->writeln('<comment>Value has not been updated, as it has not been set before.</comment>');
|
||||
return 1;
|
||||
}
|
||||
$configValue = $input->getOption('value');
|
||||
|
||||
$this->systemConfig->setValue($configName, $configValue);
|
||||
$output->writeln('<info>System config value ' . $configName . ' set to ' . $configValue . '</info>');
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
|
|
@ -41,6 +41,7 @@ if (\OC::$server->getConfig()->getSystemValue('installed', false)) {
|
|||
$application->add(new OC\Core\Command\Background\Ajax(\OC::$server->getConfig()));
|
||||
|
||||
$application->add(new OC\Core\Command\Config\ListConfigs(\OC::$server->getSystemConfig(), \OC::$server->getAppConfig()));
|
||||
$application->add(new OC\Core\Command\Config\System\SetConfig(\OC::$server->getSystemConfig()));
|
||||
|
||||
$application->add(new OC\Core\Command\Db\GenerateChangeScript());
|
||||
$application->add(new OC\Core\Command\Db\ConvertType(\OC::$server->getConfig(), new \OC\DB\ConnectionFactory()));
|
||||
|
|
|
|||
115
tests/core/command/config/system/setconfigtest.php
Normal file
115
tests/core/command/config/system/setconfigtest.php
Normal file
|
|
@ -0,0 +1,115 @@
|
|||
<?php
|
||||
/**
|
||||
* @author Joas Schilling <nickvergessen@owncloud.com>
|
||||
*
|
||||
* @copyright Copyright (c) 2015, ownCloud, Inc.
|
||||
* @license AGPL-3.0
|
||||
*
|
||||
* This code is free software: you can redistribute it and/or modify
|
||||
* it under the terms of the GNU Affero General Public License, version 3,
|
||||
* as published by the Free Software Foundation.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU Affero General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU Affero General Public License, version 3,
|
||||
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
||||
*
|
||||
*/
|
||||
|
||||
namespace Tests\Core\Command\Config\System;
|
||||
|
||||
|
||||
use OC\Core\Command\Config\System\SetConfig;
|
||||
use Test\TestCase;
|
||||
|
||||
class SetConfigTest extends TestCase {
|
||||
/** @var \PHPUnit_Framework_MockObject_MockObject */
|
||||
protected $systemConfig;
|
||||
|
||||
/** @var \PHPUnit_Framework_MockObject_MockObject */
|
||||
protected $consoleInput;
|
||||
/** @var \PHPUnit_Framework_MockObject_MockObject */
|
||||
protected $consoleOutput;
|
||||
|
||||
/** @var \Symfony\Component\Console\Command\Command */
|
||||
protected $command;
|
||||
|
||||
protected function setUp() {
|
||||
parent::setUp();
|
||||
|
||||
$systemConfig = $this->systemConfig = $this->getMockBuilder('OC\SystemConfig')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
$this->consoleInput = $this->getMock('Symfony\Component\Console\Input\InputInterface');
|
||||
$this->consoleOutput = $this->getMock('Symfony\Component\Console\Output\OutputInterface');
|
||||
|
||||
/** @var \OC\SystemConfig $systemConfig */
|
||||
$this->command = new SetConfig($systemConfig);
|
||||
}
|
||||
|
||||
|
||||
public function setData() {
|
||||
return [
|
||||
[
|
||||
'name',
|
||||
'newvalue',
|
||||
true,
|
||||
true,
|
||||
true,
|
||||
'info',
|
||||
],
|
||||
[
|
||||
'name',
|
||||
'newvalue',
|
||||
false,
|
||||
true,
|
||||
false,
|
||||
'comment',
|
||||
],
|
||||
];
|
||||
}
|
||||
|
||||
/**
|
||||
* @dataProvider setData
|
||||
*
|
||||
* @param string $configName
|
||||
* @param mixed $newValue
|
||||
* @param bool $configExists
|
||||
* @param bool $updateOnly
|
||||
* @param bool $updated
|
||||
* @param string $expectedMessage
|
||||
*/
|
||||
public function testList($configName, $newValue, $configExists, $updateOnly, $updated, $expectedMessage) {
|
||||
$this->systemConfig->expects($this->once())
|
||||
->method('getKeys')
|
||||
->willReturn($configExists ? [$configName] : []);
|
||||
|
||||
if ($updated) {
|
||||
$this->systemConfig->expects($this->once())
|
||||
->method('setValue')
|
||||
->with($configName, $newValue);
|
||||
}
|
||||
|
||||
$this->consoleInput->expects($this->once())
|
||||
->method('getArgument')
|
||||
->with('name')
|
||||
->willReturn($configName);
|
||||
$this->consoleInput->expects($this->any())
|
||||
->method('getOption')
|
||||
->with('value')
|
||||
->willReturn($newValue);
|
||||
$this->consoleInput->expects($this->any())
|
||||
->method('hasParameterOption')
|
||||
->with('--update-only')
|
||||
->willReturn($updateOnly);
|
||||
|
||||
$this->consoleOutput->expects($this->any())
|
||||
->method('writeln')
|
||||
->with($this->stringContains($expectedMessage));
|
||||
|
||||
\Test_Helper::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
|
||||
}
|
||||
}
|
||||
Loading…
Reference in a new issue