diff --git a/apps/user_ldap/lib/Controller/ConfigAPIController.php b/apps/user_ldap/lib/Controller/ConfigAPIController.php index d98e6d41b52b1..d78ee12f42bb5 100644 --- a/apps/user_ldap/lib/Controller/ConfigAPIController.php +++ b/apps/user_ldap/lib/Controller/ConfigAPIController.php @@ -6,9 +6,6 @@ */ namespace OCA\User_LDAP\Controller; -use OC\CapabilitiesManager; -use OC\Core\Controller\OCSController; -use OC\Security\IdentityProof\Manager; use OCA\User_LDAP\Configuration; use OCA\User_LDAP\ConnectionFactory; use OCA\User_LDAP\Helper; @@ -19,34 +16,19 @@ use OCP\AppFramework\OCS\OCSBadRequestException; use OCP\AppFramework\OCS\OCSException; use OCP\AppFramework\OCS\OCSNotFoundException; +use OCP\AppFramework\OCSController; use OCP\IRequest; -use OCP\IUserManager; -use OCP\IUserSession; -use OCP\ServerVersion; use Psr\Log\LoggerInterface; class ConfigAPIController extends OCSController { public function __construct( string $appName, IRequest $request, - CapabilitiesManager $capabilitiesManager, - IUserSession $userSession, - IUserManager $userManager, - Manager $keyManager, - ServerVersion $serverVersion, private Helper $ldapHelper, private LoggerInterface $logger, private ConnectionFactory $connectionFactory, ) { - parent::__construct( - $appName, - $request, - $capabilitiesManager, - $userSession, - $userManager, - $keyManager, - $serverVersion, - ); + parent::__construct($appName, $request); } /** diff --git a/core/Controller/OCSController.php b/core/Controller/OCSController.php index fb0280479c48c..e1e220a6439dc 100644 --- a/core/Controller/OCSController.php +++ b/core/Controller/OCSController.php @@ -20,7 +20,11 @@ use OCP\ServerVersion; use OCP\Util; -class OCSController extends \OCP\AppFramework\OCSController { +/** + * If you are here because you can't extend the OCSController class, you got the wrong one. + * Please use {@see \OCP\AppFramework\OCSController}! + */ +final class OCSController extends \OCP\AppFramework\OCSController { public function __construct( string $appName, IRequest $request, diff --git a/core/Controller/WhatsNewController.php b/core/Controller/WhatsNewController.php index af8c3d4853be2..7fa22b0faec1e 100644 --- a/core/Controller/WhatsNewController.php +++ b/core/Controller/WhatsNewController.php @@ -6,8 +6,6 @@ */ namespace OC\Core\Controller; -use OC\CapabilitiesManager; -use OC\Security\IdentityProof\Manager; use OC\Updater\ChangesCheck; use OCP\AppFramework\Db\DoesNotExistException; use OCP\AppFramework\Http; @@ -17,27 +15,21 @@ use OCP\Defaults; use OCP\IConfig; use OCP\IRequest; -use OCP\IUserManager; use OCP\IUserSession; use OCP\L10N\IFactory; use OCP\PreConditionNotMetException; -use OCP\ServerVersion; -class WhatsNewController extends OCSController { +class WhatsNewController extends \OCP\AppFramework\OCSController { public function __construct( string $appName, IRequest $request, - CapabilitiesManager $capabilitiesManager, private IUserSession $userSession, - IUserManager $userManager, - Manager $keyManager, - ServerVersion $serverVersion, private IConfig $config, private ChangesCheck $whatsNewService, private IFactory $langFactory, private Defaults $defaults, ) { - parent::__construct($appName, $request, $capabilitiesManager, $userSession, $userManager, $keyManager, $serverVersion); + parent::__construct($appName, $request); } /** diff --git a/core/openapi-administration.json b/core/openapi-administration.json index 39e38822e899b..f482f4992c131 100644 --- a/core/openapi-administration.json +++ b/core/openapi-administration.json @@ -644,6 +644,10 @@ } }, "tags": [ + { + "name": "ocs", + "description": "If you are here because you can't extend the OCSController class, you got the wrong one. Please use {@see \\OCP\\AppFramework\\OCSController}!" + }, { "name": "avatar", "description": "Class AvatarController" diff --git a/core/openapi-ex_app.json b/core/openapi-ex_app.json index 4df685320c208..2e346ca231647 100644 --- a/core/openapi-ex_app.json +++ b/core/openapi-ex_app.json @@ -1367,6 +1367,10 @@ } }, "tags": [ + { + "name": "ocs", + "description": "If you are here because you can't extend the OCSController class, you got the wrong one. Please use {@see \\OCP\\AppFramework\\OCSController}!" + }, { "name": "avatar", "description": "Class AvatarController" diff --git a/core/openapi-full.json b/core/openapi-full.json index 3da076db32943..1cd5800081617 100644 --- a/core/openapi-full.json +++ b/core/openapi-full.json @@ -11788,6 +11788,10 @@ } }, "tags": [ + { + "name": "ocs", + "description": "If you are here because you can't extend the OCSController class, you got the wrong one. Please use {@see \\OCP\\AppFramework\\OCSController}!" + }, { "name": "avatar", "description": "Class AvatarController" diff --git a/core/openapi.json b/core/openapi.json index 6d667a04e708e..67b6e3fc6ce9b 100644 --- a/core/openapi.json +++ b/core/openapi.json @@ -10152,6 +10152,10 @@ } }, "tags": [ + { + "name": "ocs", + "description": "If you are here because you can't extend the OCSController class, you got the wrong one. Please use {@see \\OCP\\AppFramework\\OCSController}!" + }, { "name": "avatar", "description": "Class AvatarController" diff --git a/openapi.json b/openapi.json index 1a777cb726310..1577e283930c4 100644 --- a/openapi.json +++ b/openapi.json @@ -17,6 +17,10 @@ } ], "tags": [ + { + "name": "core/ocs", + "description": "If you are here because you can't extend the OCSController class, you got the wrong one. Please use {@see \\OCP\\AppFramework\\OCSController}!" + }, { "name": "core/avatar", "description": "Class AvatarController"