From 2dea21a53263ec907ccc581aaf0d0e7784b5ac20 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=B4me=20Chilliet?= <come.chilliet@nextcloud.com>
Date: Thu, 2 Oct 2025 15:16:48 +0200
Subject: [PATCH] chore: Get rid of unused user_ldap AJAX endpoints
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Louis Chmn <louis@chmn.me>
---
 apps/user_ldap/ajax/deleteConfiguration.php | 24 ----------------
 apps/user_ldap/ajax/getConfiguration.php    | 23 ---------------
 apps/user_ldap/ajax/setConfiguration.php    | 32 ---------------------
 apps/user_ldap/appinfo/routes.php           |  6 ----
 4 files changed, 85 deletions(-)
 delete mode 100644 apps/user_ldap/ajax/deleteConfiguration.php
 delete mode 100644 apps/user_ldap/ajax/getConfiguration.php
 delete mode 100644 apps/user_ldap/ajax/setConfiguration.php

diff --git a/apps/user_ldap/ajax/deleteConfiguration.php b/apps/user_ldap/ajax/deleteConfiguration.php
deleted file mode 100644
index 68bce69f982..00000000000
--- a/apps/user_ldap/ajax/deleteConfiguration.php
+++ /dev/null
@@ -1,24 +0,0 @@
-<?php
-
-use OCA\User_LDAP\Helper;
-use OCP\Server;
-use OCP\Util;
-
-/**
- * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
-// Check user and app status
-\OC_JSON::checkAdminUser();
-\OC_JSON::checkAppEnabled('user_ldap');
-\OC_JSON::callCheck();
-
-$prefix = (string)$_POST['ldap_serverconfig_chooser'];
-$helper = Server::get(Helper::class);
-if ($helper->deleteServerConfiguration($prefix)) {
-	\OC_JSON::success();
-} else {
-	$l = Util::getL10N('user_ldap');
-	\OC_JSON::error(['message' => $l->t('Failed to delete the server configuration')]);
-}
diff --git a/apps/user_ldap/ajax/getConfiguration.php b/apps/user_ldap/ajax/getConfiguration.php
deleted file mode 100644
index 79a7372813c..00000000000
--- a/apps/user_ldap/ajax/getConfiguration.php
+++ /dev/null
@@ -1,23 +0,0 @@
-<?php
-
-use OCA\User_LDAP\LDAP;
-
-/**
- * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
-// Check user and app status
-\OC_JSON::checkAdminUser();
-\OC_JSON::checkAppEnabled('user_ldap');
-\OC_JSON::callCheck();
-
-$prefix = (string)$_POST['ldap_serverconfig_chooser'];
-$ldapWrapper = new LDAP();
-$connection = new \OCA\User_LDAP\Connection($ldapWrapper, $prefix);
-$configuration = $connection->getConfiguration();
-if (isset($configuration['ldap_agent_password']) && $configuration['ldap_agent_password'] !== '') {
-	// hide password
-	$configuration['ldap_agent_password'] = '**PASSWORD SET**';
-}
-\OC_JSON::success(['configuration' => $configuration]);
diff --git a/apps/user_ldap/ajax/setConfiguration.php b/apps/user_ldap/ajax/setConfiguration.php
deleted file mode 100644
index 815ef040257..00000000000
--- a/apps/user_ldap/ajax/setConfiguration.php
+++ /dev/null
@@ -1,32 +0,0 @@
-<?php
-
-use OCA\User_LDAP\LDAP;
-
-/**
- * SPDX-FileCopyrightText: 2018-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-only
- */
-// Check user and app status
-\OC_JSON::checkAdminUser();
-\OC_JSON::checkAppEnabled('user_ldap');
-\OC_JSON::callCheck();
-
-$prefix = (string)$_POST['ldap_serverconfig_chooser'];
-
-// Checkboxes are not submitted, when they are unchecked. Set them manually.
-// only legacy checkboxes (Advanced and Expert tab) need to be handled here,
-// the Wizard-like tabs handle it on their own
-$chkboxes = ['ldap_configuration_active', 'ldap_override_main_server',
-	'ldap_turn_off_cert_check'];
-foreach ($chkboxes as $boxid) {
-	if (!isset($_POST[$boxid])) {
-		$_POST[$boxid] = 0;
-	}
-}
-
-$ldapWrapper = new LDAP();
-$connection = new \OCA\User_LDAP\Connection($ldapWrapper, $prefix);
-$connection->setConfiguration($_POST);
-$connection->saveConfiguration();
-\OC_JSON::success();
diff --git a/apps/user_ldap/appinfo/routes.php b/apps/user_ldap/appinfo/routes.php
index 688d8014555..11c0d2ca6f5 100644
--- a/apps/user_ldap/appinfo/routes.php
+++ b/apps/user_ldap/appinfo/routes.php
@@ -9,14 +9,8 @@ declare(strict_types=1);
  */
 $this->create('user_ldap_ajax_clearMappings', 'apps/user_ldap/ajax/clearMappings.php')
 	->actionInclude('user_ldap/ajax/clearMappings.php');
-$this->create('user_ldap_ajax_deleteConfiguration', 'apps/user_ldap/ajax/deleteConfiguration.php')
-	->actionInclude('user_ldap/ajax/deleteConfiguration.php');
-$this->create('user_ldap_ajax_getConfiguration', 'apps/user_ldap/ajax/getConfiguration.php')
-	->actionInclude('user_ldap/ajax/getConfiguration.php');
 $this->create('user_ldap_ajax_getNewServerConfigPrefix', 'apps/user_ldap/ajax/getNewServerConfigPrefix.php')
 	->actionInclude('user_ldap/ajax/getNewServerConfigPrefix.php');
-$this->create('user_ldap_ajax_setConfiguration', 'apps/user_ldap/ajax/setConfiguration.php')
-	->actionInclude('user_ldap/ajax/setConfiguration.php');
 $this->create('user_ldap_ajax_testConfiguration', 'apps/user_ldap/ajax/testConfiguration.php')
 	->actionInclude('user_ldap/ajax/testConfiguration.php');
 $this->create('user_ldap_ajax_wizard', 'apps/user_ldap/ajax/wizard.php')