Test Failed
Pull Request — master (#85)
by Alex
03:56
created
src/Serialisers/IronicSerialiser.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -621,10 +621,10 @@
 block discarded – undo
621 621
     {
622 622
         $queryParameterString = null;
623 623
         foreach ([ODataConstants::HTTPQUERY_STRING_FILTER,
624
-                     ODataConstants::HTTPQUERY_STRING_EXPAND,
625
-                     ODataConstants::HTTPQUERY_STRING_ORDERBY,
626
-                     ODataConstants::HTTPQUERY_STRING_INLINECOUNT,
627
-                     ODataConstants::HTTPQUERY_STRING_SELECT, ] as $queryOption) {
624
+                        ODataConstants::HTTPQUERY_STRING_EXPAND,
625
+                        ODataConstants::HTTPQUERY_STRING_ORDERBY,
626
+                        ODataConstants::HTTPQUERY_STRING_INLINECOUNT,
627
+                        ODataConstants::HTTPQUERY_STRING_SELECT, ] as $queryOption) {
628 628
             $value = $this->getService()->getHost()->getQueryStringItem($queryOption);
629 629
             if (!is_null($value)) {
630 630
                 if (!is_null($queryParameterString)) {
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
                 .' $propKind != ResourcePropertyKind::RESOURCE_REFERENCE'
170 170
             );
171 171
             $propTail = ResourcePropertyKind::RESOURCE_REFERENCE == $propKind ? 'entry' : 'feed';
172
-            $propType = 'application/atom+xml;type='.$propTail;
172
+            $propType = 'application/atom+xml;type=' . $propTail;
173 173
             $propName = $prop->getName();
174 174
             $nuLink->title = $propName;
175 175
             $nuLink->name = ODataConstants::ODATA_RELATED_NAMESPACE . $propName;
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
      *
213 213
      * @return ODataFeed
214 214
      */
215
-    public function writeTopLevelElements(QueryResult &$entryObjects)
215
+    public function writeTopLevelElements(QueryResult & $entryObjects)
216 216
     {
217 217
         assert(is_array($entryObjects->results), '!is_array($entryObjects->results)');
218 218
 
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
         $resourceSet = $this->getRequest()->getTargetResourceSetWrapper()->getResourceSet();
250 250
         $requestTop = $this->getRequest()->getTopOptionCount();
251 251
         $pageSize = $this->getService()->getConfiguration()->getEntitySetPageSize($resourceSet);
252
-        $requestTop = (null == $requestTop) ? $pageSize + 1 : $requestTop;
252
+        $requestTop = (null == $requestTop) ? $pageSize+1 : $requestTop;
253 253
 
254 254
         if (true === $entryObjects->hasMore && $requestTop > $pageSize) {
255 255
             $stackSegment = $setName;
@@ -332,7 +332,7 @@  discard block
 block discarded – undo
332 332
      *
333 333
      * @return ODataPropertyContent
334 334
      */
335
-    public function writeTopLevelComplexObject(QueryResult &$complexValue, $propertyName, ResourceType &$resourceType)
335
+    public function writeTopLevelComplexObject(QueryResult & $complexValue, $propertyName, ResourceType & $resourceType)
336 336
     {
337 337
         $result = $complexValue->results;
338 338
 
@@ -361,14 +361,14 @@  discard block
 block discarded – undo
361 361
      *                                    bag object
362 362
      * @return ODataPropertyContent
363 363
      */
364
-    public function writeTopLevelBagObject(QueryResult &$BagValue, $propertyName, ResourceType &$resourceType)
364
+    public function writeTopLevelBagObject(QueryResult & $BagValue, $propertyName, ResourceType & $resourceType)
365 365
     {
366 366
         $result = $BagValue->results;
367 367
 
368 368
         $propertyContent = new ODataPropertyContent();
369 369
         $odataProperty = new ODataProperty();
370 370
         $odataProperty->name = $propertyName;
371
-        $odataProperty->typeName = 'Collection('.$resourceType->getFullName().')';
371
+        $odataProperty->typeName = 'Collection(' . $resourceType->getFullName() . ')';
372 372
         $odataProperty->value = $this->writeBagValue($resourceType, $result);
373 373
 
374 374
         $propertyContent->properties[] = $odataProperty;
@@ -384,7 +384,7 @@  discard block
 block discarded – undo
384 384
      *                                            primitive property to be written
385 385
      * @return ODataPropertyContent
386 386
      */
387
-    public function writeTopLevelPrimitive(QueryResult &$primitiveValue, ResourceProperty &$resourceProperty = null)
387
+    public function writeTopLevelPrimitive(QueryResult & $primitiveValue, ResourceProperty & $resourceProperty = null)
388 388
     {
389 389
         assert(null != $resourceProperty, 'Resource property must not be null');
390 390
         $propertyContent = new ODataPropertyContent();
@@ -633,7 +633,7 @@  discard block
 block discarded – undo
633 633
         $segmentWrappers = $this->getStack()->getSegmentWrappers();
634 634
         $count = count($segmentWrappers);
635 635
 
636
-        return 0 == $count ? $this->getRequest()->getTargetResourceSetWrapper() : $segmentWrappers[$count - 1];
636
+        return 0 == $count ? $this->getRequest()->getTargetResourceSetWrapper() : $segmentWrappers[$count-1];
637 637
     }
638 638
 
639 639
     /**
@@ -658,7 +658,7 @@  discard block
 block discarded – undo
658 658
         $skipToken = $internalOrderByInfo->buildSkipTokenValue($lastObject);
659 659
         assert(!is_null($skipToken), '!is_null($skipToken)');
660 660
         $token = (1 < $numSegments) ? '$skiptoken=' : '$skip=';
661
-        $skipToken = '?'.$queryParameterString.$token.$skipToken;
661
+        $skipToken = '?' . $queryParameterString . $token . $skipToken;
662 662
 
663 663
         return $skipToken;
664 664
     }
@@ -692,7 +692,7 @@  discard block
 block discarded – undo
692 692
 
693 693
         $topCountValue = $this->getRequest()->getTopOptionCount();
694 694
         if (!is_null($topCountValue)) {
695
-            $remainingCount = $topCountValue - $this->getRequest()->getTopCount();
695
+            $remainingCount = $topCountValue-$this->getRequest()->getTopCount();
696 696
             if (0 < $remainingCount) {
697 697
                 if (!is_null($queryParameterString)) {
698 698
                     $queryParameterString .= '&';
@@ -727,7 +727,7 @@  discard block
 block discarded – undo
727 727
      *
728 728
      * @return string
729 729
      */
730
-    private function primitiveToString(IType &$type, $primitiveValue)
730
+    private function primitiveToString(IType & $type, $primitiveValue)
731 731
     {
732 732
         if ($type instanceof Boolean) {
733 733
             $stringValue = (true === $primitiveValue) ? 'true' : 'false';
@@ -821,7 +821,7 @@  discard block
 block discarded – undo
821 821
      * @param $result
822 822
      * @return ODataBagContent|null
823 823
      */
824
-    protected function writeBagValue(ResourceType &$resourceType, $result)
824
+    protected function writeBagValue(ResourceType & $resourceType, $result)
825 825
     {
826 826
         assert(null == $result || is_array($result), 'Bag parameter must be null or array');
827 827
         $typeKind = $resourceType->getResourceTypeKind();
@@ -854,7 +854,7 @@  discard block
 block discarded – undo
854 854
      * @param string|null   $propertyName
855 855
      * @return ODataPropertyContent
856 856
      */
857
-    protected function writeComplexValue(ResourceType &$resourceType, &$result, $propertyName = null)
857
+    protected function writeComplexValue(ResourceType & $resourceType, &$result, $propertyName = null)
858 858
     {
859 859
         assert(is_object($result), 'Supplied $customObject must be an object');
860 860
 
Please login to merge, or discard this patch.
src/Controllers/ODataController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -58,9 +58,9 @@
 block discarded – undo
58 58
                 $cerealRequest = serialize($request);
59 59
                 $cerealMeta = serialize($meta);
60 60
                 $cerealResponse = serialize($odataResponse);
61
-                Storage::put($xTest.'request', $cerealRequest);
62
-                Storage::put($xTest.'metadata', $cerealMeta);
63
-                Storage::put($xTest.'response', $cerealResponse);
61
+                Storage::put($xTest . 'request', $cerealRequest);
62
+                Storage::put($xTest . 'metadata', $cerealMeta);
63
+                Storage::put($xTest . 'response', $cerealResponse);
64 64
             }
65 65
         }
66 66
 
Please login to merge, or discard this patch.
src/Providers/MetadataControllerProvider.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@
 block discarded – undo
53 53
                 foreach ($lock as $barrel => $roll) {
54 54
                     assert(
55 55
                         !array_key_exists($barrel, $metamix[$key]),
56
-                        'Mapping already defined for model '.$key.' and CRUD verb '.$barrel
56
+                        'Mapping already defined for model ' . $key . ' and CRUD verb ' . $barrel
57 57
                     );
58 58
                     $metamix[$key][$barrel] = $roll;
59 59
                 }
Please login to merge, or discard this patch.
src/Query/LaravelQuery.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -310,12 +310,12 @@  discard block
 block discarded – undo
310 310
         $map = $raw->getMetadata();
311 311
 
312 312
         if (!array_key_exists($class, $map)) {
313
-            throw new \POData\Common\InvalidOperationException('Controller mapping missing for class '.$class.'.');
313
+            throw new \POData\Common\InvalidOperationException('Controller mapping missing for class ' . $class . '.');
314 314
         }
315 315
         $goal = $raw->getMapping($class, $verb);
316 316
         if (null == $goal) {
317 317
             throw new \POData\Common\InvalidOperationException(
318
-                'Controller mapping missing for '.$verb.' verb on class '.$class.'.'
318
+                'Controller mapping missing for ' . $verb . ' verb on class ' . $class . '.'
319 319
             );
320 320
         }
321 321
 
@@ -391,7 +391,7 @@  discard block
 block discarded – undo
391 391
                 throw new ODataException($e->getMessage(), 500);
392 392
             }
393 393
         }
394
-        throw new ODataException('Target model not successfully '.$lastWord, 422);
394
+        throw new ODataException('Target model not successfully ' . $lastWord, 422);
395 395
     }
396 396
 
397 397
     /**
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
434 434
         }
435 435
         $outData = $result->getData();
436 436
         if (is_object($outData)) {
437
-            $outData = (array)$outData;
437
+            $outData = (array) $outData;
438 438
         }
439 439
 
440 440
         if (!is_array($outData)) {
Please login to merge, or discard this patch.
src/Query/LaravelReadQuery.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -135,8 +135,8 @@  discard block
 block discarded – undo
135 135
         $nullFilter = true;
136 136
         $isvalid = null;
137 137
         if (isset($filterInfo)) {
138
-            $method = 'return '.$filterInfo->getExpressionAsString().';';
139
-            $clln = '$'.$resourceSet->getResourceType()->getName();
138
+            $method = 'return ' . $filterInfo->getExpressionAsString() . ';';
139
+            $clln = '$' . $resourceSet->getResourceType()->getName();
140 140
             $isvalid = create_function($clln, $method);
141 141
             $nullFilter = false;
142 142
         }
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
                     // so we can't bail out early
165 165
                     $rawCount += $results->count();
166 166
                     // now bolt on filtrate to accumulating result set if we haven't accumulated enough bitz
167
-                    if ($rawTop > $resultSet->count() + $skip) {
167
+                    if ($rawTop > $resultSet->count()+$skip) {
168 168
                         $resultSet = collect(array_merge($resultSet->all(), $results->all()));
169 169
                         $sliceAmount = min($skip, $resultSet->count());
170 170
                         $resultSet = $resultSet->slice($sliceAmount);
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
         if (QueryType::COUNT() == $queryType || QueryType::ENTITIES_WITH_COUNT() == $queryType) {
203 203
             $result->count = $resultCount;
204 204
         }
205
-        $hazMore = $bulkSetCount > $skip + count($resultSet);
205
+        $hazMore = $bulkSetCount > $skip+count($resultSet);
206 206
         $result->hasMore = $hazMore;
207 207
         return $result;
208 208
     }
@@ -365,7 +365,7 @@  discard block
 block discarded – undo
365 365
         }
366 366
         $propertyName = $targetProperty->getName();
367 367
         if (!method_exists($sourceEntityInstance, $propertyName)) {
368
-            $msg = 'Relation method, '.$propertyName.', does not exist on supplied entity.';
368
+            $msg = 'Relation method, ' . $propertyName . ', does not exist on supplied entity.';
369 369
             throw new InvalidArgumentException($msg);
370 370
         }
371 371
         // take key descriptor and turn it into where clause here, rather than in getResource call
Please login to merge, or discard this patch.
src/Query/LaravelExpressionProvider.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
         };
57 57
         $this->functionDescriptionParsers[ODataConstants::STRFUN_ENDSWITH] = function ($params) {
58 58
             return '(strcmp(substr('.$params[0].', strlen('.$params[0].') - strlen('.$params[1].')), '
59
-                   .$params[1].') === 0)';
59
+                    .$params[1].') === 0)';
60 60
         };
61 61
         $this->functionDescriptionParsers[ODataConstants::STRFUN_INDEXOF] = function ($params) {
62 62
             return 'strpos('.$params[0].', '.$params[1].')';
Please login to merge, or discard this patch.
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -52,81 +52,81 @@  discard block
 block discarded – undo
52 52
     public function __construct()
53 53
     {
54 54
         $this->functionDescriptionParsers[ODataConstants::STRFUN_COMPARE] = function ($params) {
55
-            return 'strcmp('.$params[0].', '.$params[1].')';
55
+            return 'strcmp(' . $params[0] . ', ' . $params[1] . ')';
56 56
         };
57 57
         $this->functionDescriptionParsers[ODataConstants::STRFUN_ENDSWITH] = function ($params) {
58
-            return '(strcmp(substr('.$params[0].', strlen('.$params[0].') - strlen('.$params[1].')), '
59
-                   .$params[1].') === 0)';
58
+            return '(strcmp(substr(' . $params[0] . ', strlen(' . $params[0] . ') - strlen(' . $params[1] . ')), '
59
+                   .$params[1] . ') === 0)';
60 60
         };
61 61
         $this->functionDescriptionParsers[ODataConstants::STRFUN_INDEXOF] = function ($params) {
62
-            return 'strpos('.$params[0].', '.$params[1].')';
62
+            return 'strpos(' . $params[0] . ', ' . $params[1] . ')';
63 63
         };
64 64
         $this->functionDescriptionParsers[ODataConstants::STRFUN_REPLACE] = function ($params) {
65
-            return 'str_replace('.$params[1].', '.$params[2].', '.$params[0].')';
65
+            return 'str_replace(' . $params[1] . ', ' . $params[2] . ', ' . $params[0] . ')';
66 66
         };
67 67
         $this->functionDescriptionParsers[ODataConstants::STRFUN_STARTSWITH] = function ($params) {
68
-            return '(strpos('.$params[0].', '.$params[1].') === 0)';
68
+            return '(strpos(' . $params[0] . ', ' . $params[1] . ') === 0)';
69 69
         };
70 70
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TOLOWER] = function ($params) {
71
-            return 'strtolower('.$params[0].')';
71
+            return 'strtolower(' . $params[0] . ')';
72 72
         };
73 73
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TOUPPER] = function ($params) {
74
-            return 'strtoupper('.$params[0].')';
74
+            return 'strtoupper(' . $params[0] . ')';
75 75
         };
76 76
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TRIM] = function ($params) {
77
-            return 'trim('.$params[0].')';
77
+            return 'trim(' . $params[0] . ')';
78 78
         };
79 79
         $this->functionDescriptionParsers[ODataConstants::STRFUN_SUBSTRING] = function ($params) {
80 80
             return count($params) == 3 ?
81
-                'substr('.$params[0].', '.$params[1].', '.$params[2].')' : 'substr('.$params[0].', '.$params[1].')';
81
+                'substr(' . $params[0] . ', ' . $params[1] . ', ' . $params[2] . ')' : 'substr(' . $params[0] . ', ' . $params[1] . ')';
82 82
         };
83 83
         $this->functionDescriptionParsers[ODataConstants::STRFUN_SUBSTRINGOF] = function ($params) {
84
-            return '(strpos('.$params[1].', '.$params[0].') !== false)';
84
+            return '(strpos(' . $params[1] . ', ' . $params[0] . ') !== false)';
85 85
         };
86 86
         $this->functionDescriptionParsers[ODataConstants::STRFUN_CONCAT] = function ($params) {
87
-            return $params[0].' . '.$params[1];
87
+            return $params[0] . ' . ' . $params[1];
88 88
         };
89 89
         $this->functionDescriptionParsers[ODataConstants::STRFUN_LENGTH] = function ($params) {
90
-            return 'strlen('.$params[0].')';
90
+            return 'strlen(' . $params[0] . ')';
91 91
         };
92 92
         $this->functionDescriptionParsers[ODataConstants::GUIDFUN_EQUAL] = function ($params) {
93
-            return self::TYPE_NAMESPACE.'Guid::guidEqual('.$params[0].', '.$params[1].')';
93
+            return self::TYPE_NAMESPACE . 'Guid::guidEqual(' . $params[0] . ', ' . $params[1] . ')';
94 94
         };
95 95
         $this->functionDescriptionParsers[ODataConstants::DATETIME_COMPARE] = function ($params) {
96
-            return self::TYPE_NAMESPACE.'DateTime::dateTimeCmp('.$params[0].', '.$params[1].')';
96
+            return self::TYPE_NAMESPACE . 'DateTime::dateTimeCmp(' . $params[0] . ', ' . $params[1] . ')';
97 97
         };
98 98
         $this->functionDescriptionParsers[ODataConstants::DATETIME_YEAR] = function ($params) {
99
-            return self::TYPE_NAMESPACE.'DateTime::year('.$params[0].')';
99
+            return self::TYPE_NAMESPACE . 'DateTime::year(' . $params[0] . ')';
100 100
         };
101 101
         $this->functionDescriptionParsers[ODataConstants::DATETIME_MONTH] = function ($params) {
102
-            return self::TYPE_NAMESPACE.'DateTime::month('.$params[0].')';
102
+            return self::TYPE_NAMESPACE . 'DateTime::month(' . $params[0] . ')';
103 103
         };
104 104
         $this->functionDescriptionParsers[ODataConstants::DATETIME_DAY] = function ($params) {
105
-            return self::TYPE_NAMESPACE.'DateTime::day('.$params[0].')';
105
+            return self::TYPE_NAMESPACE . 'DateTime::day(' . $params[0] . ')';
106 106
         };
107 107
         $this->functionDescriptionParsers[ODataConstants::DATETIME_HOUR] = function ($params) {
108
-            return self::TYPE_NAMESPACE.'DateTime::hour('.$params[0].')';
108
+            return self::TYPE_NAMESPACE . 'DateTime::hour(' . $params[0] . ')';
109 109
         };
110 110
         $this->functionDescriptionParsers[ODataConstants::DATETIME_MINUTE] = function ($params) {
111
-            return self::TYPE_NAMESPACE.'DateTime::minute('.$params[0].')';
111
+            return self::TYPE_NAMESPACE . 'DateTime::minute(' . $params[0] . ')';
112 112
         };
113 113
         $this->functionDescriptionParsers[ODataConstants::DATETIME_SECOND] = function ($params) {
114
-            return self::TYPE_NAMESPACE.'DateTime::second('.$params[0].')';
114
+            return self::TYPE_NAMESPACE . 'DateTime::second(' . $params[0] . ')';
115 115
         };
116 116
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_ROUND] = function ($params) {
117
-            return 'round('.$params[0].')';
117
+            return 'round(' . $params[0] . ')';
118 118
         };
119 119
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_CEILING] = function ($params) {
120
-            return 'ceil('.$params[0].')';
120
+            return 'ceil(' . $params[0] . ')';
121 121
         };
122 122
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_FLOOR] = function ($params) {
123
-            return 'floor('.$params[0].')';
123
+            return 'floor(' . $params[0] . ')';
124 124
         };
125 125
         $this->functionDescriptionParsers[ODataConstants::BINFUL_EQUAL] = function ($params) {
126
-            return self::TYPE_NAMESPACE.'Binary::binaryEqual('.$params[0].', '.$params[1].')';
126
+            return self::TYPE_NAMESPACE . 'Binary::binaryEqual(' . $params[0] . ', ' . $params[1] . ')';
127 127
         };
128 128
         $this->functionDescriptionParsers['is_null'] = function ($params) {
129
-            return 'is_null('.$params[0].')';
129
+            return 'is_null(' . $params[0] . ')';
130 130
         };
131 131
     }
132 132
     /**
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
      */
158 158
     public function setResourceType(ResourceType $resourceType)
159 159
     {
160
-        $this->iteratorName = '$'.$resourceType->getName();
160
+        $this->iteratorName = '$' . $resourceType->getName();
161 161
         $this->resourceType = $resourceType;
162 162
     }
163 163
     /**
@@ -282,11 +282,11 @@  discard block
 block discarded – undo
282 282
         $parent = $expression;
283 283
         $variable = null;
284 284
         do {
285
-            $variable = $parent->getResourceProperty()->getName().self::MEMBER_ACCESS.$variable;
285
+            $variable = $parent->getResourceProperty()->getName() . self::MEMBER_ACCESS . $variable;
286 286
             $parent = $parent->getParent();
287 287
         } while ($parent != null);
288 288
         $variable = rtrim($variable, self::MEMBER_ACCESS);
289
-        $variable = $this->getIteratorName().self::MEMBER_ACCESS.$variable;
289
+        $variable = $this->getIteratorName() . self::MEMBER_ACCESS . $variable;
290 290
         return $variable;
291 291
     }
292 292
     /**
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
318 318
      */
319 319
     private function prepareBinaryExpression($operator, $left, $right)
320 320
     {
321
-        return self::OPEN_BRACKET.$left.' '.$operator.' '.$right.self::CLOSE_BRACKET;
321
+        return self::OPEN_BRACKET . $left . ' ' . $operator . ' ' . $right . self::CLOSE_BRACKET;
322 322
     }
323 323
     /**
324 324
      * To format unary expression.
@@ -330,6 +330,6 @@  discard block
 block discarded – undo
330 330
      */
331 331
     private function prepareUnaryExpression($operator, $child)
332 332
     {
333
-        return $operator.self::OPEN_BRACKET.$child.self::CLOSE_BRACKET;
333
+        return $operator . self::OPEN_BRACKET . $child . self::CLOSE_BRACKET;
334 334
     }
335 335
 }
Please login to merge, or discard this patch.
src/Controllers/MetadataControllerTrait.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -35,27 +35,27 @@  discard block
 block discarded – undo
35 35
         assert(0 < count($this->mapping), 'Mapping array must not be empty');
36 36
 
37 37
         if (!array_key_exists($modelName, $this->mapping)) {
38
-            throw new \Exception('Metadata mapping for model '.$modelName.' not defined');
38
+            throw new \Exception('Metadata mapping for model ' . $modelName . ' not defined');
39 39
         }
40 40
 
41 41
         $this->checkCrudVerbDefined($crudVerb);
42 42
 
43 43
         $lookup = $this->mapping[$modelName];
44 44
         if (!is_array($lookup)) {
45
-            throw new \Exception('Metadata mapping for model '.$modelName.' not an array');
45
+            throw new \Exception('Metadata mapping for model ' . $modelName . ' not an array');
46 46
         }
47 47
 
48 48
         if (!array_key_exists($crudVerb, $lookup)) {
49
-            throw new \Exception('Metadata mapping for CRUD verb '.$crudVerb.' on model '.$modelName.' not defined');
49
+            throw new \Exception('Metadata mapping for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' not defined');
50 50
         }
51 51
         $result = $lookup[$crudVerb];
52 52
         if (!isset($result)) {
53
-            throw new \Exception('Metadata mapping for CRUD verb '.$crudVerb.' on model '.$modelName.' null');
53
+            throw new \Exception('Metadata mapping for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' null');
54 54
         }
55 55
 
56 56
         if (!method_exists($this, $result)) {
57 57
             throw new \Exception(
58
-                'Metadata target for CRUD verb '.$crudVerb.' on model '.$modelName.' does not exist'
58
+                'Metadata target for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' does not exist'
59 59
             );
60 60
         }
61 61
 
@@ -77,17 +77,17 @@  discard block
 block discarded – undo
77 77
         // check that mapping array is well formed and sane, rather than waiting to stab us with a spatula
78 78
         foreach ($this->mapping as $key => $map) {
79 79
             if (!is_array($map)) {
80
-                throw new \Exception('Metadata mapping for model '.$key.' not an array');
80
+                throw new \Exception('Metadata mapping for model ' . $key . ' not an array');
81 81
             }
82 82
             foreach ($map as $verb => $method) {
83 83
                 $this->checkCrudVerbDefined($verb);
84 84
                 if (!isset($method)) {
85
-                    throw new \Exception('Metadata mapping for CRUD verb '.$verb.' on model '.$key.' null');
85
+                    throw new \Exception('Metadata mapping for CRUD verb ' . $verb . ' on model ' . $key . ' null');
86 86
                 }
87 87
 
88 88
                 if (!method_exists($this, $method)) {
89 89
                     throw new \Exception(
90
-                        'Metadata target for CRUD verb '.$verb.' on model '.$key.' does not exist'
90
+                        'Metadata target for CRUD verb ' . $verb . ' on model ' . $key . ' does not exist'
91 91
                     );
92 92
                 }
93 93
                 $parmArray = $this->getParameterNames($method);
Please login to merge, or discard this patch.
src/Providers/MetadataProvider.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 
14 14
 class MetadataProvider extends MetadataBaseProvider
15 15
 {
16
-    protected $multConstraints = [ '0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
16
+    protected $multConstraints = ['0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
17 17
     protected static $metaNAMESPACE = 'Data';
18 18
 
19 19
     /**
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
         $knownKeys = array_keys($knownSide);
192 192
         $unknownKeys = array_keys($unknownSide);
193 193
         $dualKeys = array_intersect($knownKeys, $unknownKeys);
194
-        assert(count($hooks) == (count($unknownKeys) + count($knownKeys) - count($dualKeys)));
194
+        assert(count($hooks) == (count($unknownKeys)+count($knownKeys)-count($dualKeys)));
195 195
         // if either list is empty, bail out - there's nothing to do
196 196
         if (0 === count($knownSide) || 0 === count($unknownSide)) {
197 197
             return [];
Please login to merge, or discard this patch.
src/Models/MetadataTrait.php 2 patches
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -301,17 +301,17 @@
 block discarded – undo
301 301
                         $lastCode = $code[strlen($code) - 1];
302 302
                         assert('}' == $lastCode, 'Final character of function definition must be closing brace');
303 303
                         foreach ([
304
-                                     'hasMany',
305
-                                     'hasManyThrough',
306
-                                     'belongsToMany',
307
-                                     'hasOne',
308
-                                     'belongsTo',
309
-                                     'morphOne',
310
-                                     'morphTo',
311
-                                     'morphMany',
312
-                                     'morphToMany',
313
-                                     'morphedByMany'
314
-                                 ] as $relation) {
304
+                                        'hasMany',
305
+                                        'hasManyThrough',
306
+                                        'belongsToMany',
307
+                                        'hasOne',
308
+                                        'belongsTo',
309
+                                        'morphOne',
310
+                                        'morphTo',
311
+                                        'morphMany',
312
+                                        'morphToMany',
313
+                                        'morphedByMany'
314
+                                    ] as $relation) {
315 315
                             $search = '$this->' . $relation . '(';
316 316
                             if ($pos = stripos($code, $search)) {
317 317
                                 //Resolve the relation's model to a Relation object.
Please login to merge, or discard this patch.
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
                         $reflection = new \ReflectionMethod($model, $method);
285 285
 
286 286
                         $file = new \SplFileObject($reflection->getFileName());
287
-                        $file->seek($reflection->getStartLine() - 1);
287
+                        $file->seek($reflection->getStartLine()-1);
288 288
                         $code = '';
289 289
                         while ($file->key() < $reflection->getEndLine()) {
290 290
                             $code .= $file->current();
@@ -297,8 +297,8 @@  discard block
 block discarded – undo
297 297
                             'Function definition must have keyword \'function\''
298 298
                         );
299 299
                         $begin = strpos($code, 'function(');
300
-                        $code = substr($code, $begin, strrpos($code, '}') - $begin + 1);
301
-                        $lastCode = $code[strlen($code) - 1];
300
+                        $code = substr($code, $begin, strrpos($code, '}')-$begin+1);
301
+                        $lastCode = $code[strlen($code)-1];
302 302
                         assert('}' == $lastCode, 'Final character of function definition must be closing brace');
303 303
                         foreach ([
304 304
                                      'hasMany',
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
                         break;
464 464
                     }
465 465
                 }
466
-                assert(in_array($fkMethodName, $methodList), 'Selected method, '.$fkMethodName.', not in method list');
466
+                assert(in_array($fkMethodName, $methodList), 'Selected method, ' . $fkMethodName . ', not in method list');
467 467
                 $rkMethodName = 'getQualifiedRelatedPivotKeyName';
468 468
                 foreach ($rkList as $option) {
469 469
                     if (in_array($option, $methodList)) {
@@ -471,7 +471,7 @@  discard block
 block discarded – undo
471 471
                         break;
472 472
                     }
473 473
                 }
474
-                assert(in_array($rkMethodName, $methodList), 'Selected method, '.$rkMethodName.', not in method list');
474
+                assert(in_array($rkMethodName, $methodList), 'Selected method, ' . $rkMethodName . ', not in method list');
475 475
                 $line = ['fk' => $fkMethodName, 'rk' => $rkMethodName];
476 476
                 static::$methodPrimary[get_class($foo)] = $line;
477 477
             }
@@ -500,7 +500,7 @@  discard block
 block discarded – undo
500 500
                         break;
501 501
                     }
502 502
                 }
503
-                assert(in_array($fkMethodName, $methodList), 'Selected method, '.$fkMethodName.', not in method list');
503
+                assert(in_array($fkMethodName, $methodList), 'Selected method, ' . $fkMethodName . ', not in method list');
504 504
                 $rkMethodName = 'getQualifiedParentKeyName';
505 505
                 foreach ($rkList as $option) {
506 506
                     if (in_array($option, $methodList)) {
@@ -508,7 +508,7 @@  discard block
 block discarded – undo
508 508
                         break;
509 509
                     }
510 510
                 }
511
-                assert(in_array($rkMethodName, $methodList), 'Selected method, '.$rkMethodName.', not in method list');
511
+                assert(in_array($rkMethodName, $methodList), 'Selected method, ' . $rkMethodName . ', not in method list');
512 512
                 $line = ['fk' => $fkMethodName, 'rk' => $rkMethodName];
513 513
                 static::$methodAlternate[get_class($foo)] = $line;
514 514
             }
@@ -555,10 +555,10 @@  discard block
 block discarded – undo
555 555
 
556 556
             $keyRaw = $isBelong ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
557 557
             $keySegments = explode('.', $keyRaw);
558
-            $keyName = $keySegments[count($keySegments) - 1];
558
+            $keyName = $keySegments[count($keySegments)-1];
559 559
             $localRaw = $isBelong ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
560 560
             $localSegments = explode('.', $localRaw);
561
-            $localName = $localSegments[count($localSegments) - 1];
561
+            $localName = $localSegments[count($localSegments)-1];
562 562
             $first = $keyName;
563 563
             $last = $localName;
564 564
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ);
@@ -584,10 +584,10 @@  discard block
 block discarded – undo
584 584
 
585 585
             $keyName = $isBelong ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
586 586
             $keySegments = explode('.', $keyName);
587
-            $keyName = $keySegments[count($keySegments) - 1];
587
+            $keyName = $keySegments[count($keySegments)-1];
588 588
             $localRaw = $isBelong ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
589 589
             $localSegments = explode('.', $localRaw);
590
-            $localName = $localSegments[count($localSegments) - 1];
590
+            $localName = $localSegments[count($localSegments)-1];
591 591
             $first = $isBelong ? $localName : $keyName;
592 592
             $last = $isBelong ? $keyName : $localName;
593 593
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ);
@@ -611,10 +611,10 @@  discard block
 block discarded – undo
611 611
 
612 612
             $keyRaw = $isMany ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
613 613
             $keySegments = explode('.', $keyRaw);
614
-            $keyName = $keySegments[count($keySegments) - 1];
614
+            $keyName = $keySegments[count($keySegments)-1];
615 615
             $localRaw = $isMany ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
616 616
             $localSegments = explode('.', $localRaw);
617
-            $localName = $localSegments[count($localSegments) - 1];
617
+            $localName = $localSegments[count($localSegments)-1];
618 618
             $first = $isMany ? $keyName : $localName;
619 619
             $last = $isMany ? $localName : $keyName;
620 620
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ);
@@ -637,10 +637,10 @@  discard block
 block discarded – undo
637 637
 
638 638
             $keyRaw = $isMany ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
639 639
             $keySegments = explode('.', $keyRaw);
640
-            $keyName = $keySegments[count($keySegments) - 1];
640
+            $keyName = $keySegments[count($keySegments)-1];
641 641
             $localRaw = $isMany ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
642 642
             $localSegments = explode('.', $localRaw);
643
-            $localName = $localSegments[count($localSegments) - 1];
643
+            $localName = $localSegments[count($localSegments)-1];
644 644
 
645 645
             $first = $keyName;
646 646
             $last = (isset($localName) && '' != $localName) ? $localName : $foo->getRelated()->getKeyName();
Please login to merge, or discard this patch.