Completed
Branch feature/pre-split (d27030)
by Anton
04:29
created
Excluded/old-orm/Entities/Schemas/RecordSchema.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
      * value using class name and Doctrine inflector.
131 131
      *
132 132
      * @see Record::$table
133
-     * @return mixed
133
+     * @return string
134 134
      */
135 135
     public function getTable()
136 136
     {
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
      * Get database where record data should be stored in. Database alias must be resolved.
150 150
      *
151 151
      * @see Record::$database
152
-     * @return mixed
152
+     * @return string
153 153
      */
154 154
     public function getDatabase()
155 155
     {
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
      * @param AbstractColumn $column
464 464
      * @param string         $definition
465 465
      * @param mixed          $default Default value declared by record schema.
466
-     * @return mixed
466
+     * @return AbstractColumn
467 467
      * @throws DefinitionException
468 468
      * @throws \Spiral\Database\Exceptions\SchemaException
469 469
      */
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
      */
181 181
     public function getTableID()
182 182
     {
183
-        return $this->getDatabase() . '.' . $this->getTable();
183
+        return $this->getDatabase().'.'.$this->getTable();
184 184
     }
185 185
 
186 186
     /**
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
424 424
             if (!empty($filter = $this->builder->getMutators($column->abstractType()))) {
425 425
                 //Mutator associated with type directly
426 426
                 $resolved += $filter;
427
-            } elseif (!empty($filter = $this->builder->getMutators('php:' . $column->phpType()))) {
427
+            } elseif (!empty($filter = $this->builder->getMutators('php:'.$column->phpType()))) {
428 428
                 //Mutator associated with php type
429 429
                 $resolved += $filter;
430 430
             }
Please login to merge, or discard this patch.
Excluded/ORM/Entities/RecordIterator.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@
 block discarded – undo
75 75
      *
76 76
      * @param RecordInterface|string|int $lookup
77 77
      *
78
-     * @return true
78
+     * @return boolean
79 79
      */
80 80
     public function has($lookup)
81 81
     {
Please login to merge, or discard this patch.
Excluded/ORM/Entities/RecordMapper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -881,7 +881,7 @@
 block discarded – undo
881 881
      * Parse single result row to generate data tree. Must pass parsing to evert nested loader.
882 882
      *
883 883
      * @param array $row
884
-     * @return bool
884
+     * @return boolean|null
885 885
      */
886 886
     private function parseRow(array $row)
887 887
     {
Please login to merge, or discard this patch.
Excluded/ORM/Entities/RecordSelector.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -457,7 +457,7 @@  discard block
 block discarded – undo
457 457
      *
458 458
      * @see findOne()
459 459
      * @param mixed $id Primary key value.
460
-     * @return RecordEntity|null
460
+     * @return null|RecordInterface
461 461
      * @throws SelectorException
462 462
      */
463 463
     public function findByPK($id)
@@ -482,7 +482,7 @@  discard block
 block discarded – undo
482 482
      * @see findByPK()
483 483
      * @param array $where    Selection WHERE statement.
484 484
      * @param bool  $setLimit Use limit 1.
485
-     * @return RecordEntity|null
485
+     * @return null|RecordInterface
486 486
      */
487 487
     public function findOne(array $where = [], $setLimit = true)
488 488
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -160,8 +160,8 @@  discard block
 block discarded – undo
160 160
         $offset = count($this->dataColumns);
161 161
 
162 162
         foreach ($columns as $column) {
163
-            $columnAlias = 'c' . (++$this->countColumns);
164
-            $this->dataColumns[] = $table . '.' . $column . ' AS ' . $columnAlias;
163
+            $columnAlias = 'c'.(++$this->countColumns);
164
+            $this->dataColumns[] = $table.'.'.$column.' AS '.$columnAlias;
165 165
         }
166 166
 
167 167
         return $offset;
@@ -533,7 +533,7 @@  discard block
 block discarded – undo
533 533
     public function count($column = self::DEFAULT_COUNTING_FIELD)
534 534
     {
535 535
         if ($column == self::DEFAULT_COUNTING_FIELD && !empty($this->loader->getPrimaryKey())) {
536
-            $column = 'DISTINCT(' . $this->loader->getPrimaryKey() . ')';
536
+            $column = 'DISTINCT('.$this->loader->getPrimaryKey().')';
537 537
         }
538 538
 
539 539
         return parent::count($column);
Please login to merge, or discard this patch.
Excluded/ORM/ORM.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -311,8 +311,8 @@  discard block
 block discarded – undo
311 311
      * own relations, loaders and schemas by altering ORM config.
312 312
      *
313 313
      * @param mixed         $type
314
-     * @param SchemaBuilder $builder
315
-     * @param RecordSchema  $record
314
+     * @param Entities\SchemaBuilder $builder
315
+     * @param Entities\Schemas\RecordSchema  $record
316 316
      * @param string        $name
317 317
      * @param array         $definition
318 318
      * @return Schemas\RelationInterface
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
      * @param string      $type      Loader type (usually related to relation type).
354 354
      * @param string      $container Data segment associated with loader.
355 355
      * @param array       $definition
356
-     * @param LoaderInterface|null $parent
356
+     * @param null|Loader $parent
357 357
      * @return Loader
358 358
      */
359 359
     public function loader($type, $container, array $definition, Loader $parent = null)
Please login to merge, or discard this patch.
Excluded/tests/ODM/StandaloneTest.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -80,6 +80,9 @@
 block discarded – undo
80 80
         ], $model->serializeData());
81 81
     }
82 82
 
83
+    /**
84
+     * @param string $directory
85
+     */
83 86
     protected function createLocator($directory)
84 87
     {
85 88
         $tokenizer = new Tokenizer(new FileManager(), new TokenizerConfig(), $this->createMemory());
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
     public function testIndexation()
27 27
     {
28 28
         $odm = $this->createODM();
29
-        $builder = $odm->schemaBuilder($this->createLocator(__DIR__ . '/Fixtures'));
29
+        $builder = $odm->schemaBuilder($this->createLocator(__DIR__.'/Fixtures'));
30 30
 
31 31
         $this->assertTrue($builder->hasDocument(Data::class));
32 32
         $this->assertTrue($builder->hasDocument(Element::class));
@@ -35,14 +35,14 @@  discard block
 block discarded – undo
35 35
     public function testUpdateSchema()
36 36
     {
37 37
         $odm = $this->createODM();
38
-        $builder = $odm->schemaBuilder($this->createLocator(__DIR__ . '/Fixtures'));
38
+        $builder = $odm->schemaBuilder($this->createLocator(__DIR__.'/Fixtures'));
39 39
         $odm->updateSchema($builder);
40 40
     }
41 41
 
42 42
     public function testEntities()
43 43
     {
44 44
         $odm = $this->createODM();
45
-        $odm->updateSchema($odm->schemaBuilder($this->createLocator(__DIR__ . '/Fixtures')));
45
+        $odm->updateSchema($odm->schemaBuilder($this->createLocator(__DIR__.'/Fixtures')));
46 46
 
47 47
         $data = $odm->document(Data::class);
48 48
         $element = $data->elements->create();
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     public function testSerialization()
53 53
     {
54 54
         $odm = $this->createODM();
55
-        $odm->updateSchema($odm->schemaBuilder($this->createLocator(__DIR__ . '/Fixtures')));
55
+        $odm->updateSchema($odm->schemaBuilder($this->createLocator(__DIR__.'/Fixtures')));
56 56
 
57 57
         $model = $odm->document(Data::class, [
58 58
                 'name'     => 'value',
Please login to merge, or discard this patch.
Excluded/Database/Drivers/SQLite/Schemas/ColumnSchema.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
             $enumValues[] = $this->table->driver()->getPDO()->quote($value);
117 117
         }
118 118
 
119
-        return "$statement CHECK ({$this->getName(true)} IN (" . implode(', ', $enumValues) . '))';
119
+        return "$statement CHECK ({$this->getName(true)} IN (".implode(', ', $enumValues).'))';
120 120
     }
121 121
 
122 122
     /**
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
             }
169 169
         }
170 170
 
171
-        $options = array_map(function ($value) {
171
+        $options = array_map(function($value) {
172 172
             return intval($value);
173 173
         }, explode(',', $options));
174 174
 
Please login to merge, or discard this patch.
Excluded/Database/Drivers/SQLite/Schemas/TableSchema.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@
 block discarded – undo
163 163
     {
164 164
         //Temporary table is required to copy data over
165 165
         $temporary = clone $this;
166
-        $temporary->setName('spiral_temp_' . $this->getName() . '_' . uniqid());
166
+        $temporary->setName('spiral_temp_'.$this->getName().'_'.uniqid());
167 167
 
168 168
         //We don't need any index in temporary table
169 169
         foreach ($temporary->getIndexes() as $index) {
Please login to merge, or discard this patch.
Excluded/Database/Drivers/MySQL/Schemas/ColumnSchema.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -189,14 +189,14 @@
 block discarded – undo
189 189
         if ($this->abstractType() == 'enum') {
190 190
 
191 191
             //Fetching enum values
192
-            $this->enumValues = array_map(function ($value) {
192
+            $this->enumValues = array_map(function($value) {
193 193
                 return trim($value, $value[0]);
194 194
             }, explode(',', $options));
195 195
 
196 196
             return;
197 197
         }
198 198
 
199
-        $options = array_map(function ($value) {
199
+        $options = array_map(function($value) {
200 200
             return intval($value);
201 201
         }, explode(',', $options));
202 202
 
Please login to merge, or discard this patch.