style(php): run composer cs:fix

Signed-off-by: Baptiste Fotia <[email protected]>
arawa · Dec 20, 2023 · f70a3d6 · f70a3d6
1 parent 393c337
commit f70a3d6
Show file tree

Hide file tree

Showing 26 changed files with 507 additions and 555 deletions.
diff --git a/lib/Controller/FileCSVController.php b/lib/Controller/FileCSVController.php
diff --git a/lib/Exceptions/Notifications/AbstractNotificationException.php b/lib/Exceptions/Notifications/AbstractNotificationException.php
@@ -2,23 +2,20 @@
 
 namespace OCA\Workspace\Exceptions\Notifications;
 
-abstract class AbstractNotificationException extends \Exception
-{
-
-    public function __construct(
-        private string $title,
-        string $message,
-        /**
-        * @var integer from OCP\AppFramework\Http
-        */
-        int $code
-    )
-    {
-        parent::__construct($message, $code);
-    }
+abstract class AbstractNotificationException extends \Exception {
+
+	public function __construct(
+		private string $title,
+		string $message,
+		/**
+		 * @var integer from OCP\AppFramework\Http
+		 */
+		int $code
+	) {
+		parent::__construct($message, $code);
+	}
 
-    public function getTitle(): string
-    {
-        return $this->title;
-    }
+	public function getTitle(): string {
+		return $this->title;
+	}
 }
diff --git a/lib/Exceptions/Notifications/BadMimeType.php b/lib/Exceptions/Notifications/BadMimeType.php
@@ -4,10 +4,8 @@
 
 use OCP\AppFramework\Http;
 
-class BadMimeType extends AbstractNotificationException
-{
-    public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST)
-    {
-        parent::__construct($title, $message, $code);
-    }
+class BadMimeType extends AbstractNotificationException {
+	public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST) {
+		parent::__construct($title, $message, $code);
+	}
 }
diff --git a/lib/Exceptions/Notifications/InvalidCsvFormatException.php b/lib/Exceptions/Notifications/InvalidCsvFormatException.php
@@ -4,10 +4,8 @@
 
 use OCP\AppFramework\Http;
 
-class InvalidCsvFormatException extends AbstractNotificationException
-{
-    public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST)
-    {
-        parent::__construct($title, $message, $code);
-    }
+class InvalidCsvFormatException extends AbstractNotificationException {
+	public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST) {
+		parent::__construct($title, $message, $code);
+	}
 }
diff --git a/lib/Exceptions/Notifications/InvalidSeparatorCsvException.php b/lib/Exceptions/Notifications/InvalidSeparatorCsvException.php
@@ -4,10 +4,8 @@
 
 use OCP\AppFramework\Http;
 
-class InvalidSeparatorCsvException extends AbstractNotificationException
-{
-    public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST)
-    {
-        parent::__construct($title, $message, $code);
-    }
+class InvalidSeparatorCsvException extends AbstractNotificationException {
+	public function __construct(private string $title, string $message, int $code = Http::STATUS_BAD_REQUEST) {
+		parent::__construct($title, $message, $code);
+	}
 }
diff --git a/lib/Exceptions/Notifications/UserDoesntExistException.php b/lib/Exceptions/Notifications/UserDoesntExistException.php
@@ -4,14 +4,12 @@
 
 use OCP\AppFramework\Http;
 
-class UserDoesntExistException extends AbstractNotificationException
-{
-    public function __construct(
-        private string $title,
-        string $message,
-        int $code = Http::STATUS_BAD_REQUEST
-    )
-    {
-        parent::__construct($title, $message, $code);
-    }
+class UserDoesntExistException extends AbstractNotificationException {
+	public function __construct(
+		private string $title,
+		string $message,
+		int $code = Http::STATUS_BAD_REQUEST
+	) {
+		parent::__construct($title, $message, $code);
+	}
 }
diff --git a/lib/Files/Csv.php b/lib/Files/Csv.php
@@ -28,7 +28,7 @@
 use Exception;
 
 /**
- * @deprecated 
+ * @deprecated
  * @uses OCA\Workspace\Files\Csv\ImportUsers\{Header,HeaderExtractor,HeaderValidator,Parser}
  * @uses OCA\Workspace\Files\Csv\{CheckMimeType,SeparatorDetector}
  */

diff --git a/lib/Files/Csv/CheckMimeType.php b/lib/Files/Csv/CheckMimeType.php
@@ -4,19 +4,17 @@
 
 use OCP\Files\Node;
 
-class CheckMimeType
-{
-    private const MIME_TYPE = 'text/csv';
+class CheckMimeType {
+	private const MIME_TYPE = 'text/csv';
 
-    public function checkOnNode(Node $file): bool {
-        return $file->getMimetype() !== self::MIME_TYPE;
+	public function checkOnNode(Node $file): bool {
+		return $file->getMimetype() !== self::MIME_TYPE;
 	}
 
-    /**
-     * @param array $file as $_FILE
-     */
-    public function checkOnArray(array $file): bool
-    {
-        return $file['type'] !== self::MIME_TYPE;
-    }
+	/**
+	 * @param array $file as $_FILE
+	 */
+	public function checkOnArray(array $file): bool {
+		return $file['type'] !== self::MIME_TYPE;
+	}
 }
diff --git a/lib/Files/Csv/CsvHeaderExtractorInterface.php b/lib/Files/Csv/CsvHeaderExtractorInterface.php
@@ -2,8 +2,7 @@
 
 namespace OCA\Workspace\Files\Csv;
 
-interface CsvHeaderExtractorInterface
-{
-    public static function getIndex(array $haystack, array $needles): int|bool;
-    public static function getHeaderName(array $needles, array $haystack): string;
+interface CsvHeaderExtractorInterface {
+	public static function getIndex(array $haystack, array $needles): int|bool;
+	public static function getHeaderName(array $needles, array $haystack): string;
 }
diff --git a/lib/Files/Csv/CsvParserInterface.php b/lib/Files/Csv/CsvParserInterface.php
@@ -4,7 +4,6 @@
 
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
 
-interface CsvParserInterface
-{
-    public function parser(ManagerConnectionFileInterface $file): array;
+interface CsvParserInterface {
+	public function parser(ManagerConnectionFileInterface $file): array;
 }
diff --git a/lib/Files/Csv/CsvReader.php b/lib/Files/Csv/CsvReader.php
@@ -2,35 +2,32 @@
 
 namespace OCA\Workspace\Files\Csv;
 
-use OCA\Workspace\Files\Csv\Separator;
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
 
 /**
  * Use CscReader to read without consuming too much memory.
  */
-class CsvReader
-{
-    public readonly array $headers;
+class CsvReader {
+	public readonly array $headers;
 
-    public function __construct(private ManagerConnectionFileInterface $file)
-    {
-        $handle = $file->open();
-        $this->headers = fgetcsv($handle, 1000, Separator::COMMA);
-        $file->close();
-    }
+	public function __construct(private ManagerConnectionFileInterface $file) {
+		$handle = $file->open();
+		$this->headers = fgetcsv($handle, 1000, Separator::COMMA);
+		$file->close();
+	}
 
-    public function read(): \Generator {
-        $handle = $this->file->open();
-        fgetcsv($handle, 1000, Separator::COMMA);
+	public function read(): \Generator {
+		$handle = $this->file->open();
+		fgetcsv($handle, 1000, Separator::COMMA);
 
-        try {
-            while(($data = fgetcsv($handle, 1000, Separator::COMMA)) !== false) {
-                yield array_combine($this->headers, $data);
-            }
-        } catch (\Exception $reason){
-            throw new \Exception($reason->getMessage());
-        } finally {
-            $this->file->close();
-        }
-    }
+		try {
+			while(($data = fgetcsv($handle, 1000, Separator::COMMA)) !== false) {
+				yield array_combine($this->headers, $data);
+			}
+		} catch (\Exception $reason) {
+			throw new \Exception($reason->getMessage());
+		} finally {
+			$this->file->close();
+		}
+	}
 }
diff --git a/lib/Files/Csv/CsvValidatorInterface.php b/lib/Files/Csv/CsvValidatorInterface.php
@@ -4,7 +4,6 @@
 
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
 
-interface CsvValidatorInterface
-{
-    public function validate(ManagerConnectionFileInterface $file): bool;
+interface CsvValidatorInterface {
+	public function validate(ManagerConnectionFileInterface $file): bool;
 }
diff --git a/lib/Files/Csv/ImportUsers/Header.php b/lib/Files/Csv/ImportUsers/Header.php
@@ -2,8 +2,7 @@
 
 namespace OCA\Workspace\Files\Csv\ImportUsers;
 
-class Header
-{
-    public const DISPLAY_NAME = ["username", "displayname", "name"];
+class Header {
+	public const DISPLAY_NAME = ["username", "displayname", "name"];
 	public const ROLE = ["role", "status", "userrole"];
 }
diff --git a/lib/Files/Csv/ImportUsers/HeaderExtractor.php b/lib/Files/Csv/ImportUsers/HeaderExtractor.php
@@ -4,9 +4,8 @@
 
 use OCA\Workspace\Files\Csv\CsvHeaderExtractorInterface;
 
-class HeaderExtractor implements CsvHeaderExtractorInterface
-{
-    public static function getIndex(array $haystack, array $needles): int|bool {
+class HeaderExtractor implements CsvHeaderExtractorInterface {
+	public static function getIndex(array $haystack, array $needles): int|bool {
 		$index = null;
 		foreach($haystack as $key => $value) {
 			$index = array_search($value, $needles);
@@ -17,22 +16,21 @@ public static function getIndex(array $haystack, array $needles): int|bool {
 		return false;
 	}
 
-    /**
-     * @throws \Exception when nothing header is found.
-     */
-    public static function getHeaderName(array $needles, array $haystacks): string
-    {
-        $found = array_filter(
-            $needles,
-            fn($needle) => in_array($needle, $haystacks)
-        );
+	/**
+	 * @throws \Exception when nothing header is found.
+	 */
+	public static function getHeaderName(array $needles, array $haystacks): string {
+		$found = array_filter(
+			$needles,
+			fn ($needle) => in_array($needle, $haystacks)
+		);
 
-        if (empty($found)) {
-            throw new \Exception("The $needles needles is not present in $haystacks haystacks");
-        }
+		if (empty($found)) {
+			throw new \Exception("The $needles needles is not present in $haystacks haystacks");
+		}
 
-        $key = (string)array_values($found)[0];
+		$key = (string)array_values($found)[0];
 
-        return $key;
-    }
+		return $key;
+	}
 }
diff --git a/lib/Files/Csv/ImportUsers/HeaderValidator.php b/lib/Files/Csv/ImportUsers/HeaderValidator.php
@@ -5,8 +5,7 @@
 use OCA\Workspace\Files\Csv\CsvValidatorInterface;
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
 
-class HeaderValidator implements CsvValidatorInterface
-{   
+class HeaderValidator implements CsvValidatorInterface {
 	public function validate(ManagerConnectionFileInterface $file): bool {
 		$res = false;
 		if (($handle = $file->open()) !== false) {

diff --git a/lib/Files/Csv/ImportUsers/Parser.php b/lib/Files/Csv/ImportUsers/Parser.php
@@ -2,30 +2,27 @@
 
 namespace OCA\Workspace\Files\Csv\ImportUsers;
 
-use OCA\Workspace\Files\Csv\CsvReader;
 use OCA\Workspace\Files\Csv\CsvParserInterface;
-use OCA\Workspace\Files\Csv\ImportUsers\Header;
+use OCA\Workspace\Files\Csv\CsvReader;
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
-use OCA\Workspace\Files\Csv\ImportUsers\HeaderExtractor;
 use OCA\Workspace\Users\Formatter\UserImportedFormatter;
 
-class Parser implements CsvParserInterface
-{
+class Parser implements CsvParserInterface {
 
-    /**
-     * @return UserImportedFormatter[]
-     */
-    public function parser(ManagerConnectionFileInterface $file): array {
-        $users = [];
+	/**
+	 * @return UserImportedFormatter[]
+	 */
+	public function parser(ManagerConnectionFileInterface $file): array {
+		$users = [];
 
-        $csvReader = new CsvReader($file);
+		$csvReader = new CsvReader($file);
 
-        $uid = HeaderExtractor::getHeaderName($csvReader->headers, Header::DISPLAY_NAME);
-        $role = HeaderExtractor::getHeaderName($csvReader->headers, Header::ROLE);
+		$uid = HeaderExtractor::getHeaderName($csvReader->headers, Header::DISPLAY_NAME);
+		$role = HeaderExtractor::getHeaderName($csvReader->headers, Header::ROLE);
 
-        foreach ($csvReader->read() as $data ) {
-            $users[] = new UserImportedFormatter($data[$uid], $data[$role]);
-        }
+		foreach ($csvReader->read() as $data) {
+			$users[] = new UserImportedFormatter($data[$uid], $data[$role]);
+		}
 
 		return $users;
 	}

diff --git a/lib/Files/Csv/Separator.php b/lib/Files/Csv/Separator.php
@@ -2,7 +2,6 @@
 
 namespace OCA\Workspace\Files\Csv;
 
-class Separator
-{
-    public const COMMA = ',';
+class Separator {
+	public const COMMA = ',';
 }
diff --git a/lib/Files/Csv/SeparatorDetector.php b/lib/Files/Csv/SeparatorDetector.php
@@ -2,21 +2,18 @@
 
 namespace OCA\Workspace\Files\Csv;
 
-use OCA\Workspace\Files\Csv\Separator;
 use OCA\Workspace\Files\ManagerConnectionFileInterface;
 
-class SeparatorDetector
-{
-    private const SIZE = 1000;
-
-    public static function isComma(ManagerConnectionFileInterface $file): bool
-    {
-        $handle = $file->open();
-
-        $firstLine = fread($handle, self::SIZE);
+class SeparatorDetector {
+	private const SIZE = 1000;
+
+	public static function isComma(ManagerConnectionFileInterface $file): bool {
+		$handle = $file->open();
+
+		$firstLine = fread($handle, self::SIZE);
 
-        $file->close();
+		$file->close();
 
-        return strpos($firstLine, Separator::COMMA) !== false;
-    }
+		return strpos($firstLine, Separator::COMMA) !== false;
+	}
 }