Merge pull request #2 from phpcr/cs

update codestyle

Author: dbu

.gitattributes

@@ -0,0 +1,6 @@
+.gitattributes              export-ignore
+/.github/                   export-ignore
+.gitignore                  export-ignore
+/.php-cs-fixer.dist.php     export-ignore
+/phpunit.xml.dist           export-ignore
+/tests/                     export-ignore

.github/workflows/ci.yaml

@@ -20,6 +20,10 @@ jobs:
             fail-fast: false
             matrix:
                 include:
+                    - php-version: '7.4'
+                      symfony-version: '^4.4'
+                      monolog-version: '^1.0'
+
                     - php-version: '7.4'
                       symfony-version: '^5.4'
                       monolog-version: '^2.0'

.github/workflows/static.yml

@@ -0,0 +1,20 @@
+name: Static analysis
+
+on:
+  push:
+    branches:
+      - '[0-9]+.x'
+      - '[0-9]+.[0-9]+'
+      - '[0-9]+.[0-9]+.x'
+  pull_request:
+
+jobs:
+  php-cs-fixer:
+    name: PHP-CS-Fixer
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: PHP-CS-Fixer
+        uses: docker://oskarstark/php-cs-fixer-ga
+        with:
+          args: --dry-run

.gitignore

@@ -1,4 +1,5 @@
-tests/Resources/App/var
-composer.lock
-vendor
-.phpunit.result.cache
+/.php-cs-fixer.cache
+/composer.lock
+/tests/Resources/App/var
+/vendor
+/.phpunit.result.cache

.php-cs-fixer.dist.php

@@ -0,0 +1,12 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->in('src/')
+    ->in('tests/');
+$config = new PhpCsFixer\Config();
+
+return $config->setFinder($finder)
+    ->setRules([
+         '@Symfony' => true,
+         'array_syntax' => ['syntax' => 'short'],
+    ]);

.php_cs

@@ -23,9 +23,9 @@ class MigrateCommand extends Command
 {
     private $factory;
     private $container;
-    private $actions = array(
+    private $actions = [
         'up', 'down', 'top', 'bottom',
-    );
+    ];
 
     public function __construct(
         MigratorFactory $factory,

src/Command/MigrateCommand.php

@@ -55,19 +55,19 @@ public function execute(InputInterface $input, OutputInterface $output)
         $currentVersion = $this->versionStorage->getCurrentVersion();
 
         $table = new Table($output);
-        $table->setHeaders(array(
+        $table->setHeaders([
             '', 'Version', 'Date', 'Migrated', 'Path',
-        ));
+        ]);
 
         foreach ($versionCollection->getAllVersions() as $versionName => $versionClass) {
             $reflection = new \ReflectionClass($versionClass);
-            $table->addRow(array(
+            $table->addRow([
                 $versionName == $currentVersion ? '*' : '',
                 $versionName,
                 $this->getDate($versionName),
-                isset($executedVersions[$versionName]) ? '<info>' . $executedVersions[$versionName]['executed']->format('Y-m-d H:i:s')  . '</info>' : 'n/a',
+                isset($executedVersions[$versionName]) ? '<info>'.$executedVersions[$versionName]['executed']->format('Y-m-d H:i:s').'</info>' : 'n/a',
                 substr($reflection->getFileName(), strlen(getcwd()) + 1),
-            ));
+            ]);
         }
 
         $table->render();

src/Command/StatusCommand.php

@@ -24,16 +24,12 @@ class Configuration implements ConfigurationInterface
     public function getConfigTreeBuilder()
     {
         $treeBuilder = new TreeBuilder('phpcr_migrations');
-        if (method_exists($treeBuilder, 'getRootNode')) {
-            $root = $treeBuilder->getRootNode();
-        } else {
-            // BC layer for symfony/config 4.1 and older
-            $root = $treeBuilder->root('phpcr_migrations');
-        }
 
-        $root
+        $treeBuilder->getRootNode()
             ->children()
-                ->scalarNode('version_node_name')->defaultValue('jcr:versions')->end()
+                ->scalarNode('version_node_name')
+                    ->defaultValue('jcr:versions')
+                ->end()
                 ->arrayNode('paths')
                     ->prototype('scalar')->end()
                 ->end()

src/DependencyInjection/Configuration.php

@@ -22,7 +22,7 @@ public function load(array $configs, ContainerBuilder $container)
     {
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
-        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
 
         $container->setParameter('phpcr_migrations.version_node_name', $config['version_node_name']);
 
@@ -31,7 +31,7 @@ public function load(array $configs, ContainerBuilder $container)
         foreach ($container->getParameter('kernel.bundles') as $bundleFqn) {
             $reflection = new \ReflectionClass($bundleFqn);
             $path = dirname($reflection->getFileName());
-            $migrationsPath = $path . '/Resources/phpcr-migrations';
+            $migrationsPath = $path.'/Resources/phpcr-migrations';
 
             if (file_exists($migrationsPath)) {
                 $paths[] = $migrationsPath;