Passed
Pull Request — master (#7691)
by Matthew
09:57
created
lib/Doctrine/ORM/Tools/Console/Command/SchemaTool/CreateCommand.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,9 +28,9 @@  discard block
 block discarded – undo
28 28
     protected function configure()
29 29
     {
30 30
         $this->setName('orm:schema-tool:create')
31
-             ->setDescription('Processes the schema and either create it directly on EntityManager Storage Connection or generate the SQL output')
32
-             ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
33
-             ->setHelp(<<<'EOT'
31
+                ->setDescription('Processes the schema and either create it directly on EntityManager Storage Connection or generate the SQL output')
32
+                ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
33
+                ->setHelp(<<<'EOT'
34 34
 Processes the schema and either create it directly on EntityManager Storage Connection or generate the SQL output.
35 35
 
36 36
 <comment>Hint:</comment> If you have a database with tables that should not be managed
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 
40 40
     $config->setFilterSchemaAssetsExpression($regexp);
41 41
 EOT
42
-             );
42
+                );
43 43
     }
44 44
 
45 45
     /**
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/SchemaTool/UpdateCommand.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -35,11 +35,11 @@  discard block
 block discarded – undo
35 35
     protected function configure()
36 36
     {
37 37
         $this->setName($this->name)
38
-             ->setDescription('Executes (or dumps) the SQL needed to update the database schema to match the current mapping metadata')
39
-             ->addOption('complete', null, InputOption::VALUE_NONE, 'If defined, all assets of the database which are not relevant to the current metadata will be dropped.')
40
-             ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Dumps the generated SQL statements to the screen (does not execute them).')
41
-             ->addOption('force', 'f', InputOption::VALUE_NONE, 'Causes the generated SQL statements to be physically executed against your database.')
42
-             ->setHelp(<<<'EOT'
38
+                ->setDescription('Executes (or dumps) the SQL needed to update the database schema to match the current mapping metadata')
39
+                ->addOption('complete', null, InputOption::VALUE_NONE, 'If defined, all assets of the database which are not relevant to the current metadata will be dropped.')
40
+                ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Dumps the generated SQL statements to the screen (does not execute them).')
41
+                ->addOption('force', 'f', InputOption::VALUE_NONE, 'Causes the generated SQL statements to be physically executed against your database.')
42
+                ->setHelp(<<<'EOT'
43 43
 The <info>%command.name%</info> command generates the SQL needed to
44 44
 synchronize the database schema with the current mapping metadata of the
45 45
 default entity manager.
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
     $config->setFilterSchemaAssetsExpression($regexp);
71 71
 EOT
72
-             );
72
+                );
73 73
     }
74 74
 
75 75
     /**
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/ClearCache/MetadataCommand.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
     protected function configure()
31 31
     {
32 32
         $this->setName('orm:clear-cache:metadata')
33
-             ->setDescription('Clear all metadata cache of the various cache drivers')
34
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
-             ->setHelp(<<<'EOT'
33
+                ->setDescription('Clear all metadata cache of the various cache drivers')
34
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
+                ->setHelp(<<<'EOT'
36 36
 The <info>%command.name%</info> command is meant to clear the metadata cache of associated Entity Manager.
37 37
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
38 38
 instance completely.
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
50 50
 because of a limitation of its execution nature.
51 51
 EOT
52
-             );
52
+                );
53 53
     }
54 54
 
55 55
     /**
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $em          = $this->getHelper('em')->getEntityManager();
58 58
         $cacheDriver = $em->getConfiguration()->getMetadataCacheImpl();
59 59
 
60
-        if (! $cacheDriver) {
60
+        if ( ! $cacheDriver) {
61 61
             throw new InvalidArgumentException('No Metadata cache driver is configured on given EntityManager.');
62 62
         }
63 63
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
             $message = $result ? 'Successfully flushed cache entries.' : $message;
80 80
         }
81 81
 
82
-        if (! $result) {
82
+        if ( ! $result) {
83 83
             $ui->error($message);
84 84
 
85 85
             return 1;
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/ClearCache/QueryRegionCommand.php 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -27,11 +27,11 @@  discard block
 block discarded – undo
27 27
     protected function configure()
28 28
     {
29 29
         $this->setName('orm:clear-cache:region:query')
30
-             ->setDescription('Clear a second-level cache query region')
31
-             ->addArgument('region-name', InputArgument::OPTIONAL, 'The query region to clear.')
32
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all query regions will be deleted/invalidated.')
33
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
34
-             ->setHelp(<<<'EOT'
30
+                ->setDescription('Clear a second-level cache query region')
31
+                ->addArgument('region-name', InputArgument::OPTIONAL, 'The query region to clear.')
32
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all query regions will be deleted/invalidated.')
33
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
34
+                ->setHelp(<<<'EOT'
35 35
 The <info>%command.name%</info> command is meant to clear a second-level cache query region for an associated Entity Manager.
36 36
 It is possible to delete/invalidate all query region, a specific query region or flushes the cache provider.
37 37
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 Finally, be aware that if <info>--flush</info> option is passed,
56 56
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
57 57
 EOT
58
-             );
58
+                );
59 59
     }
60 60
 
61 61
     /**
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
             $name = Cache::DEFAULT_QUERY_REGION_NAME;
76 76
         }
77 77
 
78
-        if (! $cache instanceof Cache) {
78
+        if ( ! $cache instanceof Cache) {
79 79
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
80 80
         }
81 81
 
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
             $queryCache  = $cache->getQueryCache($name);
84 84
             $queryRegion = $queryCache->getRegion();
85 85
 
86
-            if (! $queryRegion instanceof DefaultRegion) {
86
+            if ( ! $queryRegion instanceof DefaultRegion) {
87 87
                 throw new InvalidArgumentException(sprintf(
88 88
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
89 89
                     is_object($queryRegion) ? get_class($queryRegion) : gettype($queryRegion)
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/ClearCache/QueryCommand.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
     protected function configure()
31 31
     {
32 32
         $this->setName('orm:clear-cache:query')
33
-             ->setDescription('Clear all query cache of the various cache drivers')
34
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
-             ->setHelp(<<<'EOT'
33
+                ->setDescription('Clear all query cache of the various cache drivers')
34
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
+                ->setHelp(<<<'EOT'
36 36
 The <info>%command.name%</info> command is meant to clear the query cache of associated Entity Manager.
37 37
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
38 38
 instance completely.
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
50 50
 because of a limitation of its execution nature.
51 51
 EOT
52
-             );
52
+                );
53 53
     }
54 54
 
55 55
     /**
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $em          = $this->getHelper('em')->getEntityManager();
58 58
         $cacheDriver = $em->getConfiguration()->getQueryCacheImpl();
59 59
 
60
-        if (! $cacheDriver) {
60
+        if ( ! $cacheDriver) {
61 61
             throw new InvalidArgumentException('No Query cache driver is configured on given EntityManager.');
62 62
         }
63 63
 
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
             $message = $result ? 'Successfully flushed cache entries.' : $message;
79 79
         }
80 80
 
81
-        if (! $result) {
81
+        if ( ! $result) {
82 82
             $ui->error($message);
83 83
 
84 84
             return 1;
Please login to merge, or discard this patch.
Doctrine/ORM/Tools/Console/Command/ClearCache/CollectionRegionCommand.php 2 patches
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -27,13 +27,13 @@  discard block
 block discarded – undo
27 27
     protected function configure()
28 28
     {
29 29
         $this->setName('orm:clear-cache:region:collection')
30
-             ->setDescription('Clear a second-level cache collection region')
31
-             ->addArgument('owner-class', InputArgument::OPTIONAL, 'The owner entity name.')
32
-             ->addArgument('association', InputArgument::OPTIONAL, 'The association collection name.')
33
-             ->addArgument('owner-id', InputArgument::OPTIONAL, 'The owner identifier.')
34
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
35
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
36
-             ->setHelp(<<<'EOT'
30
+                ->setDescription('Clear a second-level cache collection region')
31
+                ->addArgument('owner-class', InputArgument::OPTIONAL, 'The owner entity name.')
32
+                ->addArgument('association', InputArgument::OPTIONAL, 'The association collection name.')
33
+                ->addArgument('owner-id', InputArgument::OPTIONAL, 'The owner identifier.')
34
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
35
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
36
+                ->setHelp(<<<'EOT'
37 37
 The <info>%command.name%</info> command is meant to clear a second-level cache collection regions for an associated Entity Manager.
38 38
 It is possible to delete/invalidate all collection region, a specific collection region or flushes the cache provider.
39 39
 
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 Finally, be aware that if <info>--flush</info> option is passed,
58 58
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
59 59
 EOT
60
-             );
60
+                );
61 61
     }
62 62
 
63 63
     /**
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
         $ownerId    = $input->getArgument('owner-id');
76 76
         $cache      = $em->getCache();
77 77
 
78
-        if (! $cache instanceof Cache) {
78
+        if ( ! $cache instanceof Cache) {
79 79
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
80 80
         }
81 81
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         if ($input->getOption('flush')) {
87 87
             $collectionRegion = $cache->getCollectionCacheRegion($ownerClass, $assoc);
88 88
 
89
-            if (! $collectionRegion instanceof DefaultRegion) {
89
+            if ( ! $collectionRegion instanceof DefaultRegion) {
90 90
                 throw new InvalidArgumentException(sprintf(
91 91
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
92 92
                     is_object($collectionRegion) ? get_class($collectionRegion) : gettype($collectionRegion)
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/ClearCache/ResultCommand.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,9 +30,9 @@  discard block
 block discarded – undo
30 30
     protected function configure()
31 31
     {
32 32
         $this->setName('orm:clear-cache:result')
33
-             ->setDescription('Clear all result cache of the various cache drivers')
34
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
-             ->setHelp(<<<'EOT'
33
+                ->setDescription('Clear all result cache of the various cache drivers')
34
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
35
+                ->setHelp(<<<'EOT'
36 36
 The <info>%command.name%</info> command is meant to clear the result cache of associated Entity Manager.
37 37
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
38 38
 instance completely.
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
50 50
 because of a limitation of its execution nature.
51 51
 EOT
52
-             );
52
+                );
53 53
     }
54 54
 
55 55
     /**
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $em          = $this->getHelper('em')->getEntityManager();
58 58
         $cacheDriver = $em->getConfiguration()->getResultCacheImpl();
59 59
 
60
-        if (! $cacheDriver) {
60
+        if ( ! $cacheDriver) {
61 61
             throw new InvalidArgumentException('No Result cache driver is configured on given EntityManager.');
62 62
         }
63 63
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
             $message = $result ? 'Successfully flushed cache entries.' : $message;
80 80
         }
81 81
 
82
-        if (! $result) {
82
+        if ( ! $result) {
83 83
             $ui->error($message);
84 84
 
85 85
             return 1;
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/ClearCache/EntityRegionCommand.php 2 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -27,12 +27,12 @@  discard block
 block discarded – undo
27 27
     protected function configure()
28 28
     {
29 29
         $this->setName('orm:clear-cache:region:entity')
30
-             ->setDescription('Clear a second-level cache entity region')
31
-             ->addArgument('entity-class', InputArgument::OPTIONAL, 'The entity name.')
32
-             ->addArgument('entity-id', InputArgument::OPTIONAL, 'The entity identifier.')
33
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
34
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
35
-             ->setHelp(<<<'EOT'
30
+                ->setDescription('Clear a second-level cache entity region')
31
+                ->addArgument('entity-class', InputArgument::OPTIONAL, 'The entity name.')
32
+                ->addArgument('entity-id', InputArgument::OPTIONAL, 'The entity identifier.')
33
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
34
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
35
+                ->setHelp(<<<'EOT'
36 36
 The <info>%command.name%</info> command is meant to clear a second-level cache entity region for an associated Entity Manager.
37 37
 It is possible to delete/invalidate all entity region, a specific entity region or flushes the cache provider.
38 38
 
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 Finally, be aware that if <info>--flush</info> option is passed,
57 57
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
58 58
 EOT
59
-             );
59
+                );
60 60
     }
61 61
 
62 62
     /**
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -73,18 +73,18 @@
 block discarded – undo
73 73
         $entityId    = $input->getArgument('entity-id');
74 74
         $cache       = $em->getCache();
75 75
 
76
-        if (! $cache instanceof Cache) {
76
+        if ( ! $cache instanceof Cache) {
77 77
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
78 78
         }
79 79
 
80
-        if (! $entityClass && ! $input->getOption('all')) {
80
+        if ( ! $entityClass && ! $input->getOption('all')) {
81 81
             throw new InvalidArgumentException('Invalid argument "--entity-class"');
82 82
         }
83 83
 
84 84
         if ($input->getOption('flush')) {
85 85
             $entityRegion = $cache->getEntityCacheRegion($entityClass);
86 86
 
87
-            if (! $entityRegion instanceof DefaultRegion) {
87
+            if ( ! $entityRegion instanceof DefaultRegion) {
88 88
                 throw new InvalidArgumentException(sprintf(
89 89
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
90 90
                     is_object($entityRegion) ? get_class($entityRegion) : gettype($entityRegion)
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/GH6699Test.php 1 patch
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -20,11 +20,11 @@  discard block
 block discarded – undo
20 20
     public function testMixedParametersWithZeroNumber() : void
21 21
     {
22 22
         $query = $this->em->createQueryBuilder()
23
-                           ->select('u')
24
-                           ->from(CmsUser::class, 'u')
25
-                           ->andWhere('u.username = :username')
26
-                           ->andWhere('u.id = ?0')
27
-                           ->getQuery();
23
+                            ->select('u')
24
+                            ->from(CmsUser::class, 'u')
25
+                            ->andWhere('u.username = :username')
26
+                            ->andWhere('u.id = ?0')
27
+                            ->getQuery();
28 28
 
29 29
         $query->setParameter('username', 'bar');
30 30
         $query->setParameter(0, 0);
@@ -39,13 +39,13 @@  discard block
 block discarded – undo
39 39
     public function testMixedParametersWithZeroNumberOnQueryBuilder() : void
40 40
     {
41 41
         $query = $this->em->createQueryBuilder()
42
-                           ->select('u')
43
-                           ->from(CmsUser::class, 'u')
44
-                           ->andWhere('u.username = :username')
45
-                           ->andWhere('u.id = ?0')
46
-                           ->setParameter('username', 'bar')
47
-                           ->setParameter(0, 0)
48
-                           ->getQuery();
42
+                            ->select('u')
43
+                            ->from(CmsUser::class, 'u')
44
+                            ->andWhere('u.username = :username')
45
+                            ->andWhere('u.id = ?0')
46
+                            ->setParameter('username', 'bar')
47
+                            ->setParameter(0, 0)
48
+                            ->getQuery();
49 49
 
50 50
         $query->execute();
51 51
 
Please login to merge, or discard this patch.