Passed
Pull Request — master (#7448)
by Ilya
14:31
created
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1400Test.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -54,8 +54,8 @@
 block discarded – undo
54 54
         $user1 = $this->em->getReference(DDC1400User::class, $user1->id);
55 55
 
56 56
         $this->em->createQuery('SELECT a, s FROM ' . DDC1400Article::class . ' a JOIN a.userStates s WITH s.user = :activeUser')
57
-                  ->setParameter('activeUser', $user1)
58
-                  ->getResult();
57
+                    ->setParameter('activeUser', $user1)
58
+                    ->getResult();
59 59
 
60 60
         $queryCount = $this->getCurrentQueryCount();
61 61
 
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
 
55 55
         $user1 = $this->em->getReference(DDC1400User::class, $user1->id);
56 56
 
57
-        $this->em->createQuery('SELECT a, s FROM ' . DDC1400Article::class . ' a JOIN a.userStates s WITH s.user = :activeUser')
57
+        $this->em->createQuery('SELECT a, s FROM '.DDC1400Article::class.' a JOIN a.userStates s WITH s.user = :activeUser')
58 58
                   ->setParameter('activeUser', $user1)
59 59
                   ->getResult();
60 60
 
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Tools/Console/Command/SchemaTool/DropCommand.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -28,11 +28,11 @@  discard block
 block discarded – undo
28 28
     protected function configure()
29 29
     {
30 30
         $this->setName('orm:schema-tool:drop')
31
-             ->setDescription('Drop the complete database schema of EntityManager Storage Connection or generate the corresponding SQL output')
32
-             ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
33
-             ->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for the deletion of the database, but force the operation to run.")
34
-             ->addOption('full-database', null, InputOption::VALUE_NONE, 'Instead of using the Class Metadata to detect the database table schema, drop ALL assets that the database contains.')
35
-             ->setHelp(<<<'EOT'
31
+                ->setDescription('Drop the complete database schema of EntityManager Storage Connection or generate the corresponding SQL output')
32
+                ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
33
+                ->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for the deletion of the database, but force the operation to run.")
34
+                ->addOption('full-database', null, InputOption::VALUE_NONE, 'Instead of using the Class Metadata to detect the database table schema, drop ALL assets that the database contains.')
35
+                ->setHelp(<<<'EOT'
36 36
 Processes the schema and either drop the database schema of EntityManager Storage Connection or generate the SQL output.
37 37
 Beware that the complete database is dropped by this command, even tables that are not relevant to your metadata model.
38 38
 
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 
43 43
     $config->setFilterSchemaAssetsExpression($regexp);
44 44
 EOT
45
-             );
45
+                );
46 46
     }
47 47
 
48 48
     /**
Please login to merge, or discard this patch.
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.