@@ -24,26 +24,26 @@ |
||
| 24 | 24 | |
| 25 | 25 | interface IUserLDAP { |
| 26 | 26 | |
| 27 | - //Functions used by LDAPProvider |
|
| 27 | + //Functions used by LDAPProvider |
|
| 28 | 28 | |
| 29 | - /** |
|
| 30 | - * Return access for LDAP interaction. |
|
| 31 | - * @param string $uid |
|
| 32 | - * @return Access instance of Access for LDAP interaction |
|
| 33 | - */ |
|
| 34 | - public function getLDAPAccess($uid); |
|
| 29 | + /** |
|
| 30 | + * Return access for LDAP interaction. |
|
| 31 | + * @param string $uid |
|
| 32 | + * @return Access instance of Access for LDAP interaction |
|
| 33 | + */ |
|
| 34 | + public function getLDAPAccess($uid); |
|
| 35 | 35 | |
| 36 | - /** |
|
| 37 | - * Return a new LDAP connection for the specified user. |
|
| 38 | - * @param string $uid |
|
| 39 | - * @return resource of the LDAP connection |
|
| 40 | - */ |
|
| 41 | - public function getNewLDAPConnection($uid); |
|
| 36 | + /** |
|
| 37 | + * Return a new LDAP connection for the specified user. |
|
| 38 | + * @param string $uid |
|
| 39 | + * @return resource of the LDAP connection |
|
| 40 | + */ |
|
| 41 | + public function getNewLDAPConnection($uid); |
|
| 42 | 42 | |
| 43 | - /** |
|
| 44 | - * Return the username for the given LDAP DN, if available. |
|
| 45 | - * @param string $dn |
|
| 46 | - * @return string|false with the username |
|
| 47 | - */ |
|
| 48 | - public function dn2UserName($dn); |
|
| 43 | + /** |
|
| 44 | + * Return the username for the given LDAP DN, if available. |
|
| 45 | + * @param string $dn |
|
| 46 | + * @return string|false with the username |
|
| 47 | + */ |
|
| 48 | + public function dn2UserName($dn); |
|
| 49 | 49 | } |
@@ -27,13 +27,13 @@ |
||
| 27 | 27 | |
| 28 | 28 | |
| 29 | 29 | abstract class BackendUtility { |
| 30 | - protected $access; |
|
| 30 | + protected $access; |
|
| 31 | 31 | |
| 32 | - /** |
|
| 33 | - * constructor, make sure the subclasses call this one! |
|
| 34 | - * @param Access $access an instance of Access for LDAP interaction |
|
| 35 | - */ |
|
| 36 | - public function __construct(Access $access) { |
|
| 37 | - $this->access = $access; |
|
| 38 | - } |
|
| 32 | + /** |
|
| 33 | + * constructor, make sure the subclasses call this one! |
|
| 34 | + * @param Access $access an instance of Access for LDAP interaction |
|
| 35 | + */ |
|
| 36 | + public function __construct(Access $access) { |
|
| 37 | + $this->access = $access; |
|
| 38 | + } |
|
| 39 | 39 | } |
@@ -28,52 +28,52 @@ |
||
| 28 | 28 | namespace OCA\User_LDAP; |
| 29 | 29 | |
| 30 | 30 | class WizardResult { |
| 31 | - protected $changes = array(); |
|
| 32 | - protected $options = array(); |
|
| 33 | - protected $markedChange = false; |
|
| 31 | + protected $changes = array(); |
|
| 32 | + protected $options = array(); |
|
| 33 | + protected $markedChange = false; |
|
| 34 | 34 | |
| 35 | - /** |
|
| 36 | - * @param string $key |
|
| 37 | - * @param mixed $value |
|
| 38 | - */ |
|
| 39 | - public function addChange($key, $value) { |
|
| 40 | - $this->changes[$key] = $value; |
|
| 41 | - } |
|
| 35 | + /** |
|
| 36 | + * @param string $key |
|
| 37 | + * @param mixed $value |
|
| 38 | + */ |
|
| 39 | + public function addChange($key, $value) { |
|
| 40 | + $this->changes[$key] = $value; |
|
| 41 | + } |
|
| 42 | 42 | |
| 43 | - /** |
|
| 44 | - * |
|
| 45 | - */ |
|
| 46 | - public function markChange() { |
|
| 47 | - $this->markedChange = true; |
|
| 48 | - } |
|
| 43 | + /** |
|
| 44 | + * |
|
| 45 | + */ |
|
| 46 | + public function markChange() { |
|
| 47 | + $this->markedChange = true; |
|
| 48 | + } |
|
| 49 | 49 | |
| 50 | - /** |
|
| 51 | - * @param string $key |
|
| 52 | - * @param array|string $values |
|
| 53 | - */ |
|
| 54 | - public function addOptions($key, $values) { |
|
| 55 | - if(!is_array($values)) { |
|
| 56 | - $values = array($values); |
|
| 57 | - } |
|
| 58 | - $this->options[$key] = $values; |
|
| 59 | - } |
|
| 50 | + /** |
|
| 51 | + * @param string $key |
|
| 52 | + * @param array|string $values |
|
| 53 | + */ |
|
| 54 | + public function addOptions($key, $values) { |
|
| 55 | + if(!is_array($values)) { |
|
| 56 | + $values = array($values); |
|
| 57 | + } |
|
| 58 | + $this->options[$key] = $values; |
|
| 59 | + } |
|
| 60 | 60 | |
| 61 | - /** |
|
| 62 | - * @return bool |
|
| 63 | - */ |
|
| 64 | - public function hasChanges() { |
|
| 65 | - return (count($this->changes) > 0 || $this->markedChange); |
|
| 66 | - } |
|
| 61 | + /** |
|
| 62 | + * @return bool |
|
| 63 | + */ |
|
| 64 | + public function hasChanges() { |
|
| 65 | + return (count($this->changes) > 0 || $this->markedChange); |
|
| 66 | + } |
|
| 67 | 67 | |
| 68 | - /** |
|
| 69 | - * @return array |
|
| 70 | - */ |
|
| 71 | - public function getResultArray() { |
|
| 72 | - $result = array(); |
|
| 73 | - $result['changes'] = $this->changes; |
|
| 74 | - if(count($this->options) > 0) { |
|
| 75 | - $result['options'] = $this->options; |
|
| 76 | - } |
|
| 77 | - return $result; |
|
| 78 | - } |
|
| 68 | + /** |
|
| 69 | + * @return array |
|
| 70 | + */ |
|
| 71 | + public function getResultArray() { |
|
| 72 | + $result = array(); |
|
| 73 | + $result['changes'] = $this->changes; |
|
| 74 | + if(count($this->options) > 0) { |
|
| 75 | + $result['options'] = $this->options; |
|
| 76 | + } |
|
| 77 | + return $result; |
|
| 78 | + } |
|
| 79 | 79 | } |
@@ -32,70 +32,70 @@ |
||
| 32 | 32 | |
| 33 | 33 | class UUIDFixInsert implements IRepairStep { |
| 34 | 34 | |
| 35 | - /** @var IConfig */ |
|
| 36 | - protected $config; |
|
| 35 | + /** @var IConfig */ |
|
| 36 | + protected $config; |
|
| 37 | 37 | |
| 38 | - /** @var UserMapping */ |
|
| 39 | - protected $userMapper; |
|
| 38 | + /** @var UserMapping */ |
|
| 39 | + protected $userMapper; |
|
| 40 | 40 | |
| 41 | - /** @var GroupMapping */ |
|
| 42 | - protected $groupMapper; |
|
| 41 | + /** @var GroupMapping */ |
|
| 42 | + protected $groupMapper; |
|
| 43 | 43 | |
| 44 | - /** @var IJobList */ |
|
| 45 | - protected $jobList; |
|
| 44 | + /** @var IJobList */ |
|
| 45 | + protected $jobList; |
|
| 46 | 46 | |
| 47 | - public function __construct(IConfig $config, UserMapping $userMapper, GroupMapping $groupMapper, IJobList $jobList) { |
|
| 48 | - $this->config = $config; |
|
| 49 | - $this->userMapper = $userMapper; |
|
| 50 | - $this->groupMapper = $groupMapper; |
|
| 51 | - $this->jobList = $jobList; |
|
| 52 | - } |
|
| 47 | + public function __construct(IConfig $config, UserMapping $userMapper, GroupMapping $groupMapper, IJobList $jobList) { |
|
| 48 | + $this->config = $config; |
|
| 49 | + $this->userMapper = $userMapper; |
|
| 50 | + $this->groupMapper = $groupMapper; |
|
| 51 | + $this->jobList = $jobList; |
|
| 52 | + } |
|
| 53 | 53 | |
| 54 | - /** |
|
| 55 | - * Returns the step's name |
|
| 56 | - * |
|
| 57 | - * @return string |
|
| 58 | - * @since 9.1.0 |
|
| 59 | - */ |
|
| 60 | - public function getName() { |
|
| 61 | - return 'Insert UUIDFix background job for user and group in batches'; |
|
| 62 | - } |
|
| 54 | + /** |
|
| 55 | + * Returns the step's name |
|
| 56 | + * |
|
| 57 | + * @return string |
|
| 58 | + * @since 9.1.0 |
|
| 59 | + */ |
|
| 60 | + public function getName() { |
|
| 61 | + return 'Insert UUIDFix background job for user and group in batches'; |
|
| 62 | + } |
|
| 63 | 63 | |
| 64 | - /** |
|
| 65 | - * Run repair step. |
|
| 66 | - * Must throw exception on error. |
|
| 67 | - * |
|
| 68 | - * @param IOutput $output |
|
| 69 | - * @throws \Exception in case of failure |
|
| 70 | - * @since 9.1.0 |
|
| 71 | - */ |
|
| 72 | - public function run(IOutput $output) { |
|
| 73 | - $installedVersion = $this->config->getAppValue('user_ldap', 'installed_version', '1.2.1'); |
|
| 74 | - if(version_compare($installedVersion, '1.2.1') !== -1) { |
|
| 75 | - return; |
|
| 76 | - } |
|
| 64 | + /** |
|
| 65 | + * Run repair step. |
|
| 66 | + * Must throw exception on error. |
|
| 67 | + * |
|
| 68 | + * @param IOutput $output |
|
| 69 | + * @throws \Exception in case of failure |
|
| 70 | + * @since 9.1.0 |
|
| 71 | + */ |
|
| 72 | + public function run(IOutput $output) { |
|
| 73 | + $installedVersion = $this->config->getAppValue('user_ldap', 'installed_version', '1.2.1'); |
|
| 74 | + if(version_compare($installedVersion, '1.2.1') !== -1) { |
|
| 75 | + return; |
|
| 76 | + } |
|
| 77 | 77 | |
| 78 | - foreach ([$this->userMapper, $this->groupMapper] as $mapper) { |
|
| 79 | - $offset = 0; |
|
| 80 | - $batchSize = 50; |
|
| 81 | - $jobClass = $mapper instanceof UserMapping ? UUIDFixUser::class : UUIDFixGroup::class; |
|
| 82 | - do { |
|
| 83 | - $retry = false; |
|
| 84 | - $records = $mapper->getList($offset, $batchSize); |
|
| 85 | - if(count($records) === 0){ |
|
| 86 | - continue; |
|
| 87 | - } |
|
| 88 | - try { |
|
| 89 | - $this->jobList->add($jobClass, ['records' => $records]); |
|
| 90 | - $offset += $batchSize; |
|
| 91 | - } catch (\InvalidArgumentException $e) { |
|
| 92 | - if(strpos($e->getMessage(), 'Background job arguments can\'t exceed 4000') !== false) { |
|
| 93 | - $batchSize = intval(floor(count($records) * 0.8)); |
|
| 94 | - $retry = true; |
|
| 95 | - } |
|
| 96 | - } |
|
| 97 | - } while (count($records) === $batchSize || $retry); |
|
| 98 | - } |
|
| 78 | + foreach ([$this->userMapper, $this->groupMapper] as $mapper) { |
|
| 79 | + $offset = 0; |
|
| 80 | + $batchSize = 50; |
|
| 81 | + $jobClass = $mapper instanceof UserMapping ? UUIDFixUser::class : UUIDFixGroup::class; |
|
| 82 | + do { |
|
| 83 | + $retry = false; |
|
| 84 | + $records = $mapper->getList($offset, $batchSize); |
|
| 85 | + if(count($records) === 0){ |
|
| 86 | + continue; |
|
| 87 | + } |
|
| 88 | + try { |
|
| 89 | + $this->jobList->add($jobClass, ['records' => $records]); |
|
| 90 | + $offset += $batchSize; |
|
| 91 | + } catch (\InvalidArgumentException $e) { |
|
| 92 | + if(strpos($e->getMessage(), 'Background job arguments can\'t exceed 4000') !== false) { |
|
| 93 | + $batchSize = intval(floor(count($records) * 0.8)); |
|
| 94 | + $retry = true; |
|
| 95 | + } |
|
| 96 | + } |
|
| 97 | + } while (count($records) === $batchSize || $retry); |
|
| 98 | + } |
|
| 99 | 99 | |
| 100 | - } |
|
| 100 | + } |
|
| 101 | 101 | } |
@@ -30,8 +30,8 @@ |
||
| 30 | 30 | use OCP\IConfig; |
| 31 | 31 | |
| 32 | 32 | class UUIDFixUser extends UUIDFix { |
| 33 | - public function __construct(UserMapping $mapper, LDAP $ldap, IConfig $config, Helper $helper) { |
|
| 34 | - $this->mapper = $mapper; |
|
| 35 | - $this->proxy = new Group_Proxy($helper->getServerConfigurationPrefixes(true), $ldap, $config); |
|
| 36 | - } |
|
| 33 | + public function __construct(UserMapping $mapper, LDAP $ldap, IConfig $config, Helper $helper) { |
|
| 34 | + $this->mapper = $mapper; |
|
| 35 | + $this->proxy = new Group_Proxy($helper->getServerConfigurationPrefixes(true), $ldap, $config); |
|
| 36 | + } |
|
| 37 | 37 | } |
@@ -30,31 +30,31 @@ |
||
| 30 | 30 | use OCA\User_LDAP\User_Proxy; |
| 31 | 31 | |
| 32 | 32 | abstract class UUIDFix extends QueuedJob { |
| 33 | - /** @var AbstractMapping */ |
|
| 34 | - protected $mapper; |
|
| 33 | + /** @var AbstractMapping */ |
|
| 34 | + protected $mapper; |
|
| 35 | 35 | |
| 36 | - /** @var Proxy */ |
|
| 37 | - protected $proxy; |
|
| 36 | + /** @var Proxy */ |
|
| 37 | + protected $proxy; |
|
| 38 | 38 | |
| 39 | - public function run($argument) { |
|
| 40 | - $isUser = $this->proxy instanceof User_Proxy; |
|
| 41 | - foreach($argument['records'] as $record) { |
|
| 42 | - $access = $this->proxy->getLDAPAccess($record['name']); |
|
| 43 | - $uuid = $access->getUUID($record['dn'], $isUser); |
|
| 44 | - if($uuid === false) { |
|
| 45 | - // record not found, no prob, continue with the next |
|
| 46 | - continue; |
|
| 47 | - } |
|
| 48 | - if($uuid !== $record['uuid']) { |
|
| 49 | - $this->mapper->setUUIDbyDN($uuid, $record['dn']); |
|
| 50 | - } |
|
| 51 | - } |
|
| 52 | - } |
|
| 39 | + public function run($argument) { |
|
| 40 | + $isUser = $this->proxy instanceof User_Proxy; |
|
| 41 | + foreach($argument['records'] as $record) { |
|
| 42 | + $access = $this->proxy->getLDAPAccess($record['name']); |
|
| 43 | + $uuid = $access->getUUID($record['dn'], $isUser); |
|
| 44 | + if($uuid === false) { |
|
| 45 | + // record not found, no prob, continue with the next |
|
| 46 | + continue; |
|
| 47 | + } |
|
| 48 | + if($uuid !== $record['uuid']) { |
|
| 49 | + $this->mapper->setUUIDbyDN($uuid, $record['dn']); |
|
| 50 | + } |
|
| 51 | + } |
|
| 52 | + } |
|
| 53 | 53 | |
| 54 | - /** |
|
| 55 | - * @param Proxy $proxy |
|
| 56 | - */ |
|
| 57 | - public function overrideProxy(Proxy $proxy) { |
|
| 58 | - $this->proxy = $proxy; |
|
| 59 | - } |
|
| 54 | + /** |
|
| 55 | + * @param Proxy $proxy |
|
| 56 | + */ |
|
| 57 | + public function overrideProxy(Proxy $proxy) { |
|
| 58 | + $this->proxy = $proxy; |
|
| 59 | + } |
|
| 60 | 60 | } |
@@ -33,39 +33,39 @@ |
||
| 33 | 33 | use Symfony\Component\Console\Output\OutputInterface; |
| 34 | 34 | |
| 35 | 35 | class DeleteConfig extends Command { |
| 36 | - /** @var \OCA\User_LDAP\Helper */ |
|
| 37 | - protected $helper; |
|
| 36 | + /** @var \OCA\User_LDAP\Helper */ |
|
| 37 | + protected $helper; |
|
| 38 | 38 | |
| 39 | - /** |
|
| 40 | - * @param Helper $helper |
|
| 41 | - */ |
|
| 42 | - public function __construct(Helper $helper) { |
|
| 43 | - $this->helper = $helper; |
|
| 44 | - parent::__construct(); |
|
| 45 | - } |
|
| 39 | + /** |
|
| 40 | + * @param Helper $helper |
|
| 41 | + */ |
|
| 42 | + public function __construct(Helper $helper) { |
|
| 43 | + $this->helper = $helper; |
|
| 44 | + parent::__construct(); |
|
| 45 | + } |
|
| 46 | 46 | |
| 47 | - protected function configure() { |
|
| 48 | - $this |
|
| 49 | - ->setName('ldap:delete-config') |
|
| 50 | - ->setDescription('deletes an existing LDAP configuration') |
|
| 51 | - ->addArgument( |
|
| 52 | - 'configID', |
|
| 53 | - InputArgument::REQUIRED, |
|
| 54 | - 'the configuration ID' |
|
| 55 | - ) |
|
| 56 | - ; |
|
| 57 | - } |
|
| 47 | + protected function configure() { |
|
| 48 | + $this |
|
| 49 | + ->setName('ldap:delete-config') |
|
| 50 | + ->setDescription('deletes an existing LDAP configuration') |
|
| 51 | + ->addArgument( |
|
| 52 | + 'configID', |
|
| 53 | + InputArgument::REQUIRED, |
|
| 54 | + 'the configuration ID' |
|
| 55 | + ) |
|
| 56 | + ; |
|
| 57 | + } |
|
| 58 | 58 | |
| 59 | 59 | |
| 60 | - protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 61 | - $configPrefix = $input->getArgument('configID'); |
|
| 60 | + protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 61 | + $configPrefix = $input->getArgument('configID'); |
|
| 62 | 62 | |
| 63 | - $success = $this->helper->deleteServerConfiguration($configPrefix); |
|
| 63 | + $success = $this->helper->deleteServerConfiguration($configPrefix); |
|
| 64 | 64 | |
| 65 | - if($success) { |
|
| 66 | - $output->writeln("Deleted configuration with configID '{$configPrefix}'"); |
|
| 67 | - } else { |
|
| 68 | - $output->writeln("Cannot delete configuration with configID '{$configPrefix}'"); |
|
| 69 | - } |
|
| 70 | - } |
|
| 65 | + if($success) { |
|
| 66 | + $output->writeln("Deleted configuration with configID '{$configPrefix}'"); |
|
| 67 | + } else { |
|
| 68 | + $output->writeln("Cannot delete configuration with configID '{$configPrefix}'"); |
|
| 69 | + } |
|
| 70 | + } |
|
| 71 | 71 | } |
@@ -34,59 +34,59 @@ |
||
| 34 | 34 | |
| 35 | 35 | class TestConfig extends Command { |
| 36 | 36 | |
| 37 | - protected function configure() { |
|
| 38 | - $this |
|
| 39 | - ->setName('ldap:test-config') |
|
| 40 | - ->setDescription('tests an LDAP configuration') |
|
| 41 | - ->addArgument( |
|
| 42 | - 'configID', |
|
| 43 | - InputArgument::REQUIRED, |
|
| 44 | - 'the configuration ID' |
|
| 45 | - ) |
|
| 46 | - ; |
|
| 47 | - } |
|
| 37 | + protected function configure() { |
|
| 38 | + $this |
|
| 39 | + ->setName('ldap:test-config') |
|
| 40 | + ->setDescription('tests an LDAP configuration') |
|
| 41 | + ->addArgument( |
|
| 42 | + 'configID', |
|
| 43 | + InputArgument::REQUIRED, |
|
| 44 | + 'the configuration ID' |
|
| 45 | + ) |
|
| 46 | + ; |
|
| 47 | + } |
|
| 48 | 48 | |
| 49 | - protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 50 | - $helper = new Helper(\OC::$server->getConfig()); |
|
| 51 | - $availableConfigs = $helper->getServerConfigurationPrefixes(); |
|
| 52 | - $configID = $input->getArgument('configID'); |
|
| 53 | - if(!in_array($configID, $availableConfigs)) { |
|
| 54 | - $output->writeln("Invalid configID"); |
|
| 55 | - return; |
|
| 56 | - } |
|
| 49 | + protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 50 | + $helper = new Helper(\OC::$server->getConfig()); |
|
| 51 | + $availableConfigs = $helper->getServerConfigurationPrefixes(); |
|
| 52 | + $configID = $input->getArgument('configID'); |
|
| 53 | + if(!in_array($configID, $availableConfigs)) { |
|
| 54 | + $output->writeln("Invalid configID"); |
|
| 55 | + return; |
|
| 56 | + } |
|
| 57 | 57 | |
| 58 | - $result = $this->testConfig($configID); |
|
| 59 | - if($result === 0) { |
|
| 60 | - $output->writeln('The configuration is valid and the connection could be established!'); |
|
| 61 | - } else if($result === 1) { |
|
| 62 | - $output->writeln('The configuration is invalid. Please have a look at the logs for further details.'); |
|
| 63 | - } else if($result === 2) { |
|
| 64 | - $output->writeln('The configuration is valid, but the Bind failed. Please check the server settings and credentials.'); |
|
| 65 | - } else { |
|
| 66 | - $output->writeln('Your LDAP server was kidnapped by aliens.'); |
|
| 67 | - } |
|
| 68 | - } |
|
| 58 | + $result = $this->testConfig($configID); |
|
| 59 | + if($result === 0) { |
|
| 60 | + $output->writeln('The configuration is valid and the connection could be established!'); |
|
| 61 | + } else if($result === 1) { |
|
| 62 | + $output->writeln('The configuration is invalid. Please have a look at the logs for further details.'); |
|
| 63 | + } else if($result === 2) { |
|
| 64 | + $output->writeln('The configuration is valid, but the Bind failed. Please check the server settings and credentials.'); |
|
| 65 | + } else { |
|
| 66 | + $output->writeln('Your LDAP server was kidnapped by aliens.'); |
|
| 67 | + } |
|
| 68 | + } |
|
| 69 | 69 | |
| 70 | - /** |
|
| 71 | - * tests the specified connection |
|
| 72 | - * @param string $configID |
|
| 73 | - * @return int |
|
| 74 | - */ |
|
| 75 | - protected function testConfig($configID) { |
|
| 76 | - $lw = new \OCA\User_LDAP\LDAP(); |
|
| 77 | - $connection = new Connection($lw, $configID); |
|
| 70 | + /** |
|
| 71 | + * tests the specified connection |
|
| 72 | + * @param string $configID |
|
| 73 | + * @return int |
|
| 74 | + */ |
|
| 75 | + protected function testConfig($configID) { |
|
| 76 | + $lw = new \OCA\User_LDAP\LDAP(); |
|
| 77 | + $connection = new Connection($lw, $configID); |
|
| 78 | 78 | |
| 79 | - //ensure validation is run before we attempt the bind |
|
| 80 | - $connection->getConfiguration(); |
|
| 79 | + //ensure validation is run before we attempt the bind |
|
| 80 | + $connection->getConfiguration(); |
|
| 81 | 81 | |
| 82 | - if(!$connection->setConfiguration(array( |
|
| 83 | - 'ldap_configuration_active' => 1, |
|
| 84 | - ))) { |
|
| 85 | - return 1; |
|
| 86 | - } |
|
| 87 | - if($connection->bind()) { |
|
| 88 | - return 0; |
|
| 89 | - } |
|
| 90 | - return 2; |
|
| 91 | - } |
|
| 82 | + if(!$connection->setConfiguration(array( |
|
| 83 | + 'ldap_configuration_active' => 1, |
|
| 84 | + ))) { |
|
| 85 | + return 1; |
|
| 86 | + } |
|
| 87 | + if($connection->bind()) { |
|
| 88 | + return 0; |
|
| 89 | + } |
|
| 90 | + return 2; |
|
| 91 | + } |
|
| 92 | 92 | } |
@@ -34,53 +34,53 @@ |
||
| 34 | 34 | |
| 35 | 35 | class SetConfig extends Command { |
| 36 | 36 | |
| 37 | - protected function configure() { |
|
| 38 | - $this |
|
| 39 | - ->setName('ldap:set-config') |
|
| 40 | - ->setDescription('modifies an LDAP configuration') |
|
| 41 | - ->addArgument( |
|
| 42 | - 'configID', |
|
| 43 | - InputArgument::REQUIRED, |
|
| 44 | - 'the configuration ID' |
|
| 45 | - ) |
|
| 46 | - ->addArgument( |
|
| 47 | - 'configKey', |
|
| 48 | - InputArgument::REQUIRED, |
|
| 49 | - 'the configuration key' |
|
| 50 | - ) |
|
| 51 | - ->addArgument( |
|
| 52 | - 'configValue', |
|
| 53 | - InputArgument::REQUIRED, |
|
| 54 | - 'the new configuration value' |
|
| 55 | - ) |
|
| 56 | - ; |
|
| 57 | - } |
|
| 37 | + protected function configure() { |
|
| 38 | + $this |
|
| 39 | + ->setName('ldap:set-config') |
|
| 40 | + ->setDescription('modifies an LDAP configuration') |
|
| 41 | + ->addArgument( |
|
| 42 | + 'configID', |
|
| 43 | + InputArgument::REQUIRED, |
|
| 44 | + 'the configuration ID' |
|
| 45 | + ) |
|
| 46 | + ->addArgument( |
|
| 47 | + 'configKey', |
|
| 48 | + InputArgument::REQUIRED, |
|
| 49 | + 'the configuration key' |
|
| 50 | + ) |
|
| 51 | + ->addArgument( |
|
| 52 | + 'configValue', |
|
| 53 | + InputArgument::REQUIRED, |
|
| 54 | + 'the new configuration value' |
|
| 55 | + ) |
|
| 56 | + ; |
|
| 57 | + } |
|
| 58 | 58 | |
| 59 | - protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 60 | - $helper = new Helper(\OC::$server->getConfig()); |
|
| 61 | - $availableConfigs = $helper->getServerConfigurationPrefixes(); |
|
| 62 | - $configID = $input->getArgument('configID'); |
|
| 63 | - if(!in_array($configID, $availableConfigs)) { |
|
| 64 | - $output->writeln("Invalid configID"); |
|
| 65 | - return; |
|
| 66 | - } |
|
| 59 | + protected function execute(InputInterface $input, OutputInterface $output) { |
|
| 60 | + $helper = new Helper(\OC::$server->getConfig()); |
|
| 61 | + $availableConfigs = $helper->getServerConfigurationPrefixes(); |
|
| 62 | + $configID = $input->getArgument('configID'); |
|
| 63 | + if(!in_array($configID, $availableConfigs)) { |
|
| 64 | + $output->writeln("Invalid configID"); |
|
| 65 | + return; |
|
| 66 | + } |
|
| 67 | 67 | |
| 68 | - $this->setValue( |
|
| 69 | - $configID, |
|
| 70 | - $input->getArgument('configKey'), |
|
| 71 | - $input->getArgument('configValue') |
|
| 72 | - ); |
|
| 73 | - } |
|
| 68 | + $this->setValue( |
|
| 69 | + $configID, |
|
| 70 | + $input->getArgument('configKey'), |
|
| 71 | + $input->getArgument('configValue') |
|
| 72 | + ); |
|
| 73 | + } |
|
| 74 | 74 | |
| 75 | - /** |
|
| 76 | - * save the configuration value as provided |
|
| 77 | - * @param string $configID |
|
| 78 | - * @param string $configKey |
|
| 79 | - * @param string $configValue |
|
| 80 | - */ |
|
| 81 | - protected function setValue($configID, $key, $value) { |
|
| 82 | - $configHolder = new Configuration($configID); |
|
| 83 | - $configHolder->$key = $value; |
|
| 84 | - $configHolder->saveConfiguration(); |
|
| 85 | - } |
|
| 75 | + /** |
|
| 76 | + * save the configuration value as provided |
|
| 77 | + * @param string $configID |
|
| 78 | + * @param string $configKey |
|
| 79 | + * @param string $configValue |
|
| 80 | + */ |
|
| 81 | + protected function setValue($configID, $key, $value) { |
|
| 82 | + $configHolder = new Configuration($configID); |
|
| 83 | + $configHolder->$key = $value; |
|
| 84 | + $configHolder->saveConfiguration(); |
|
| 85 | + } |
|
| 86 | 86 | } |