Merge pull request #11362 from mark0n/rm_ldap_conf_cli

Add a CLI command that deletes an LDAP config

apps/user_ldap/appinfo/register_command.php

@@ -10,3 +10,4 @@ $application->add(new OCA\user_ldap\Command\ShowConfig());
 $application->add(new OCA\user_ldap\Command\SetConfig());
 $application->add(new OCA\user_ldap\Command\TestConfig());
 $application->add(new OCA\user_ldap\Command\CreateEmptyConfig());
+$application->add(new OCA\user_ldap\Command\DeleteConfig());

apps/user_ldap/command/deleteconfig.php

@@ -0,0 +1,44 @@
+<?php
+/**
+ * Copyright (c) 2014 Martin Konrad <info@martin-konrad.net>
+ * This file is licensed under the Affero General Public License version 3 or
+ * later.
+ * See the COPYING-README file.
+ */
+
+namespace OCA\user_ldap\Command;
+
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
+use \OCA\user_ldap\lib\Helper;
+
+class DeleteConfig extends Command {
+
+	protected function configure() {
+		$this
+			->setName('ldap:delete-config')
+			->setDescription('deletes an existing LDAP configuration')
+			->addArgument(
+					'configID',
+					InputArgument::REQUIRED,
+					'the configuration ID'
+				     )
+		;
+	}
+
+
+	protected function execute(InputInterface $input, OutputInterface $output) {
+		$configPrefix = $input->getArgument('configID');;
+
+		$success = Helper::deleteServerConfiguration($configPrefix);
+
+		if($success) {
+			$output->writeln("Deleted configuration with configID '{$configPrefix}'");
+		} else {
+			$output->writeln("Cannot delete configuration with configID '{$configPrefix}'");
+		}
+	}
+}

apps/user_ldap/lib/helper.php

@@ -111,9 +111,6 @@ class Helper {
 	 * @return bool true on success, false otherwise
 	 */
 	static public function deleteServerConfiguration($prefix) {
-		//just to be on the safe side
-		\OCP\User::checkAdminUser();
-
 		if(!in_array($prefix, self::getServerConfigurationPrefixes())) {
 			return false;
 		}