fix(cs): run cs:fix

Signed-off-by: Julien Veyssier <[email protected]>
nextcloud · Aug 14, 2024 · 5792a2f · 5792a2f
1 parent ee6b9b9
commit 5792a2f
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/lib/Controller/LoginController.php b/lib/Controller/LoginController.php
@@ -208,7 +208,7 @@ private function buildProtocolErrorResponse(?bool $throttle = null): TemplateRes
 	 * @param string|null $redirectUrl
 	 * @return DataDisplayResponse|RedirectResponse|TemplateResponse
 	 */
-	public function login(int $providerId, string $redirectUrl = null) {
+	public function login(int $providerId, ?string $redirectUrl = null) {
 		if ($this->userSession->isLoggedIn()) {
 			return new RedirectResponse($redirectUrl);
 		}

diff --git a/lib/Controller/SettingsController.php b/lib/Controller/SettingsController.php
@@ -140,7 +140,7 @@ public function createProvider(string $identifier, string $clientId, string $cli
 		return new JSONResponse(array_merge($provider->jsonSerialize(), ['settings' => $providerSettings]));
 	}
 
-	public function updateProvider(int $providerId, string $identifier, string $clientId, string $discoveryEndpoint, string $clientSecret = null,
+	public function updateProvider(int $providerId, string $identifier, string $clientId, string $discoveryEndpoint, ?string $clientSecret = null,
 		array $settings = [], string $scope = 'openid email profile', ?string $endSessionEndpoint = null): JSONResponse {
 		$provider = $this->providerMapper->getProvider($providerId);
 

diff --git a/lib/Db/ProviderMapper.php b/lib/Db/ProviderMapper.php
@@ -105,9 +105,9 @@ public function getProviders() {
 	 * @throws MultipleObjectsReturnedException
 	 * @throws Exception
 	 */
-	public function createOrUpdateProvider(string $identifier, string $clientid = null,
-		string $clientsecret = null, string $discoveryuri = null, string $scope = 'openid email profile',
-		string $endsessionendpointuri = null) {
+	public function createOrUpdateProvider(string $identifier, ?string $clientid = null,
+		?string $clientsecret = null, ?string $discoveryuri = null, string $scope = 'openid email profile',
+		?string $endsessionendpointuri = null) {
 		try {
 			$provider = $this->findProviderByIdentifier($identifier);
 		} catch (DoesNotExistException $eNotExist) {

diff --git a/tests/stubs/oc_core_command_base.php b/tests/stubs/oc_core_command_base.php
@@ -15,7 +15,7 @@ class Base {
 
 		protected string $defaultOutputFormat = self::OUTPUT_FORMAT_PLAIN;
 
-		public function __construct(string $name = null) {
+		public function __construct(?string $name = null) {
 		}
 
 		protected function configure() {
@@ -36,7 +36,7 @@ public function setDescription(string $description) {
 		/**
 		 * @return $this
 		 */
-		public function addOption(string $name, $shortcut = null, int $mode = null, string $description = '', $default = null) {
+		public function addOption(string $name, $shortcut = null, ?int $mode = null, string $description = '', $default = null) {
 		}
 
 		public function getApplication(): ?Application {
@@ -51,7 +51,7 @@ public function getHelper(string $name) {
 		/**
 		 * @return $this
 		 */
-		public function addArgument(string $name, int $mode = null, string $description = '', $default = null) {
+		public function addArgument(string $name, ?int $mode = null, string $description = '', $default = null) {
 		}
 
 		protected function writeArrayInOutputFormat(InputInterface $input, OutputInterface $output, array $items, string $prefix = '  - '): void {