From 10cb502f163e3a73754d338147cd86a038f8970f Mon Sep 17 00:00:00 2001 From: Philipp Date: Fri, 23 Aug 2024 21:44:03 +0200 Subject: [PATCH] Fix deprecation warnings & remove old php_cs.dist --- .php-cs-fixer.dist.php | 14 ++++---- .php_cs.dist | 82 ------------------------------------------ 2 files changed, 8 insertions(+), 88 deletions(-) delete mode 100644 .php_cs.dist diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php index 9ed05cbc66..e80dfb85b5 100644 --- a/.php-cs-fixer.dist.php +++ b/.php-cs-fixer.dist.php @@ -7,6 +7,8 @@ declare(strict_types=1); +require_once __DIR__ . '/bin/dev/php-cs-fixer/vendor/autoload.php'; + $finder = PhpCsFixer\Finder::create() ->in(__DIR__) ->notPath('addon') @@ -41,10 +43,10 @@ return $config ], ], 'blank_line_after_namespace' => true, - 'braces' => [ - 'position_after_anonymous_constructs' => 'same', - 'position_after_control_structures' => 'same', - 'position_after_functions_and_oop_constructs' => 'next', + 'braces_position' => [ + 'anonymous_classes_opening_brace' => 'same_line', + 'control_structures_opening_brace' => 'same_line', + 'functions_opening_brace' => 'next_line_unless_newline_at_signature_end', ], 'elseif' => true, 'encoding' => true, @@ -60,7 +62,7 @@ return $config 'lowercase_keywords' => true, 'no_closing_tag' => true, 'no_spaces_after_function_name' => true, - 'no_spaces_inside_parenthesis' => true, + 'spaces_inside_parenthesis' => false, 'no_trailing_whitespace' => true, 'no_trailing_whitespace_in_comment' => true, 'no_unused_imports' => true, @@ -73,7 +75,7 @@ return $config 'visibility_required' => [ 'elements' => ['property', 'method'] ], - 'new_with_braces' => true, + 'new_with_parentheses' => true, ]) ->setFinder($finder) ->setIndent("\t"); diff --git a/.php_cs.dist b/.php_cs.dist deleted file mode 100644 index a090306638..0000000000 --- a/.php_cs.dist +++ /dev/null @@ -1,82 +0,0 @@ -in(__DIR__) - ->notPath('addon') - ->notPath('bin/dev') - ->notPath('config') - ->notPath('doc') - ->notPath('images') - ->notPath('mods') - ->notPath('spec') - ->notPath('vendor') - ->notPath('view/asset') - ->notPath('lang') - ->notPath('view/smarty3/compiled'); - -$config = new PhpCsFixer\Config(); -return $config - ->setRules([ - '@PSR1' => true, - '@PSR2' => true, - '@PSR12' => true, - 'align_multiline_comment' => true, - 'array_indentation' => true, - 'array_syntax' => [ - 'syntax' => 'short', - ], - 'binary_operator_spaces' => [ - 'default' => 'single_space', - 'operators' => [ - '=>' => 'align_single_space_minimal', - '=' => 'align_single_space_minimal', - '??' => 'align_single_space_minimal', - ], - ], - 'blank_line_after_namespace' => true, - 'braces' => [ - 'position_after_anonymous_constructs' => 'same', - 'position_after_control_structures' => 'same', - 'position_after_functions_and_oop_constructs' => 'next', - ], - 'elseif' => true, - 'encoding' => true, - 'full_opening_tag' => true, - 'function_declaration' => [ - 'closure_function_spacing' => 'one', - ], - 'indentation_type' => true, - 'line_ending' => true, - 'list_syntax' => [ - 'syntax' => 'long', - ], - 'lowercase_keywords' => true, - 'method_argument_space' => [], - 'no_closing_tag' => true, - 'no_spaces_after_function_name' => true, - 'no_spaces_inside_parenthesis' => true, - 'no_trailing_whitespace' => true, - 'no_trailing_whitespace_in_comment' => true, - 'no_unused_imports' => true, - 'single_blank_line_at_eof' => true, - 'single_class_element_per_statement' => true, - 'single_import_per_statement' => true, - 'single_line_after_imports' => true, - 'switch_case_space' => true, - 'ternary_operator_spaces' => false, - 'visibility_required' => [ - 'elements' => ['property', 'method'] - ], - 'new_with_braces' => true, - ]) - ->setFinder($finder) - ->setIndent("\t");