code style

.forgejo/workflows/ci.yaml

@@ -18,7 +18,7 @@ jobs:
       - name: Setup PHP
         uses: https://github.com/shivammathur/setup-php@v2
         with:
-          php-version: '8.0'
+          php-version: '8.1'
           extensions: apcu
       - name: Composer
         run: composer install

.php-cs-fixer.dist.php

@@ -9,13 +9,16 @@ $finder = PhpCsFixer\Finder::create()
 	->exclude('vendor')
 	->in(__DIR__)
 ;
-return PhpCsFixer\Config::create()
+return (new PhpCsFixer\Config())
 	->setRules([
 		'@PSR2' => true,
+		'curly_braces_position' => [
+			'classes_opening_brace' => 'same_line',
+			'functions_opening_brace' => 'same_line',
+		],
 		'array_syntax' => ['syntax' => 'short'],
 		'braces' => ['position_after_functions_and_oop_constructs' => 'same'],
-		'binary_operator_spaces' => ['align_double_arrow' => true, 'align_equals' => false],
+		'binary_operator_spaces' => ['operators' => ['=>' => 'align_single_space']],
 	])
 	->setIndent("\t")
-	->setFinder($finder)
-	;
+	->setFinder($finder);

src/Exception/InvalidTicket.php

@@ -9,5 +9,4 @@ declare(strict_types=1);
 namespace Icewind\SMB\Exception;
 
 class InvalidTicket extends Exception {
-
-}
+}

src/Exception/RevisionMismatchException.php

@@ -12,4 +12,4 @@ class RevisionMismatchException extends Exception {
 	public function __construct(string $message = 'Protocol version mismatch', int $code = 0, ?Throwable $previous = null) {
 		parent::__construct($message, $code, $previous);
 	}
-}
+}

src/KerberosTicket.php

@@ -26,7 +26,7 @@ class KerberosTicket {
 		return $this->cacheName;
 	}
 
-	public function getName(): string{
+	public function getName(): string {
 		return $this->krb5->getName();
 	}
 
@@ -82,4 +82,4 @@ class KerberosTicket {
 		}
 		return $ticket;
 	}
-}
+}

src/Native/NativeState.php

@@ -430,4 +430,4 @@ class NativeState {
 			}
 		}
 	}
-}
+}

src/Wrapped/Share.php

@@ -349,7 +349,7 @@ class Share extends AbstractShare {
 		$connection->write('get ' . $source . ' ' . $this->system->getFD(5));
 		$connection->write('exit');
 		$fh = $connection->getFileOutputStream();
-		$fh = CallbackWrapper::wrap($fh, function() use ($connection) {
+		$fh = CallbackWrapper::wrap($fh, function () use ($connection) {
 			$connection->write('');
 		});
 		if (!is_resource($fh)) {
@@ -379,7 +379,7 @@ class Share extends AbstractShare {
 
 		// use a close callback to ensure the upload is finished before continuing
 		// this also serves as a way to keep the connection in scope
-		$stream = CallbackWrapper::wrap($fh, function() use ($connection) {
+		$stream = CallbackWrapper::wrap($fh, function () use ($connection) {
 			$connection->write('');
 		}, null, function () use ($connection) {
 			$connection->close(false); // dont terminate, give the upload some time

tests/AbstractShareTest.php

@@ -48,8 +48,6 @@ abstract class AbstractShareTest extends TestCase {
 	abstract public function getServerClass(): string;
 
 	public function setUp(): void {
-//		ob_end_flush();
-//		var_dump($this->getName());
 		$this->config = json_decode(file_get_contents(__DIR__ . '/config.json'));
 		$options = new Options();
 		$options->setMinProtocol(IOptions::PROTOCOL_SMB2);