Algo-Web / POData-Laravel

src/Serialisers/IronicSerialiser.php

Indentation +4 added lines, -4 removed lines

@@ -621,10 +621,10 @@
     {
         $queryParameterString = null;
         foreach ([ODataConstants::HTTPQUERY_STRING_FILTER,
-                     ODataConstants::HTTPQUERY_STRING_EXPAND,
-                     ODataConstants::HTTPQUERY_STRING_ORDERBY,
-                     ODataConstants::HTTPQUERY_STRING_INLINECOUNT,
-                     ODataConstants::HTTPQUERY_STRING_SELECT, ] as $queryOption) {
+                        ODataConstants::HTTPQUERY_STRING_EXPAND,
+                        ODataConstants::HTTPQUERY_STRING_ORDERBY,
+                        ODataConstants::HTTPQUERY_STRING_INLINECOUNT,
+                        ODataConstants::HTTPQUERY_STRING_SELECT, ] as $queryOption) {
             $value = $this->getService()->getHost()->getQueryStringItem($queryOption);
             if (!is_null($value)) {
                 if (!is_null($queryParameterString)) {

Spacing +15 added lines, -15 removed lines

@@ -174,7 +174,7 @@ 
             }
             // despite all set up, checking, etc, if we haven't picked a concrete resource type,
             // wheels have fallen off, so blow up
-            assert(!$resourceType->isAbstract(), 'Concrete resource type not selected for payload '.$payloadClass);
+            assert(!$resourceType->isAbstract(), 'Concrete resource type not selected for payload ' . $payloadClass);
         }
 
         // make sure we're barking up right tree
@@ -182,7 +182,7 @@ 
         $targClass = $resourceType->getInstanceType()->getName();
         assert(
             $entryObject->results instanceof $targClass,
-            'Object being serialised not instance of expected class, '.$targClass. ', is actually '.$payloadClass
+            'Object being serialised not instance of expected class, ' . $targClass . ', is actually ' . $payloadClass
         );
 
         $rawProp = $resourceType->getAllProperties();
@@ -229,7 +229,7 @@ 
                 .' $propKind != ResourcePropertyKind::RESOURCE_REFERENCE'
             );
             $propTail = ResourcePropertyKind::RESOURCE_REFERENCE == $propKind ? 'entry' : 'feed';
-            $propType = 'application/atom+xml;type='.$propTail;
+            $propType = 'application/atom+xml;type=' . $propTail;
             $propName = $prop->getName();
             $nuLink->title = $propName;
             $nuLink->name = ODataConstants::ODATA_RELATED_NAMESPACE . $propName;
@@ -280,7 +280,7 @@ 
      *
      * @return ODataFeed
      */
-    public function writeTopLevelElements(QueryResult &$entryObjects)
+    public function writeTopLevelElements(QueryResult & $entryObjects)
     {
         $res = $entryObjects->results;
         assert(is_array($res) || $res instanceof Collection, '!is_array($entryObjects->results)');
@@ -321,7 +321,7 @@ 
         $resourceSet = $this->getRequest()->getTargetResourceSetWrapper()->getResourceSet();
         $requestTop = $this->getRequest()->getTopOptionCount();
         $pageSize = $this->getService()->getConfiguration()->getEntitySetPageSize($resourceSet);
-        $requestTop = (null === $requestTop) ? $pageSize + 1 : $requestTop;
+        $requestTop = (null === $requestTop) ? $pageSize+1 : $requestTop;
 
         if (true === $entryObjects->hasMore && $requestTop > $pageSize) {
             $stackSegment = $setName;
@@ -404,7 +404,7 @@ 
      *
      * @return ODataPropertyContent
      */
-    public function writeTopLevelComplexObject(QueryResult &$complexValue, $propertyName, ResourceType &$resourceType)
+    public function writeTopLevelComplexObject(QueryResult & $complexValue, $propertyName, ResourceType & $resourceType)
     {
         $result = $complexValue->results;
 
@@ -433,14 +433,14 @@ 
      *                                    bag object
      * @return ODataPropertyContent
      */
-    public function writeTopLevelBagObject(QueryResult &$BagValue, $propertyName, ResourceType &$resourceType)
+    public function writeTopLevelBagObject(QueryResult & $BagValue, $propertyName, ResourceType & $resourceType)
     {
         $result = $BagValue->results;
 
         $propertyContent = new ODataPropertyContent();
         $odataProperty = new ODataProperty();
         $odataProperty->name = $propertyName;
-        $odataProperty->typeName = 'Collection('.$resourceType->getFullName().')';
+        $odataProperty->typeName = 'Collection(' . $resourceType->getFullName() . ')';
         $odataProperty->value = $this->writeBagValue($resourceType, $result);
 
         $propertyContent->properties[$propertyName] = $odataProperty;
@@ -456,7 +456,7 @@ 
      *                                            primitive property to be written
      * @return ODataPropertyContent
      */
-    public function writeTopLevelPrimitive(QueryResult &$primitiveValue, ResourceProperty &$resourceProperty = null)
+    public function writeTopLevelPrimitive(QueryResult & $primitiveValue, ResourceProperty & $resourceProperty = null)
     {
         assert(null != $resourceProperty, 'Resource property must not be null');
         $propertyContent = new ODataPropertyContent();
@@ -715,7 +715,7 @@ 
         $segmentWrappers = $this->getStack()->getSegmentWrappers();
         $count = count($segmentWrappers);
 
-        return 0 == $count ? $this->getRequest()->getTargetResourceSetWrapper() : $segmentWrappers[$count - 1];
+        return 0 == $count ? $this->getRequest()->getTargetResourceSetWrapper() : $segmentWrappers[$count-1];
     }
 
     /**
@@ -740,7 +740,7 @@ 
         $skipToken = $internalOrderByInfo->buildSkipTokenValue($lastObject);
         assert(!is_null($skipToken), '!is_null($skipToken)');
         $token = (1 < $numSegments) ? '$skiptoken=' : '$skip=';
-        $skipToken = '?'.$queryParameterString.$token.$skipToken;
+        $skipToken = '?' . $queryParameterString . $token . $skipToken;
 
         return $skipToken;
     }
@@ -774,7 +774,7 @@ 
 
         $topCountValue = $this->getRequest()->getTopOptionCount();
         if (!is_null($topCountValue)) {
-            $remainingCount = $topCountValue - $this->getRequest()->getTopCount();
+            $remainingCount = $topCountValue-$this->getRequest()->getTopCount();
             if (0 < $remainingCount) {
                 if (!is_null($queryParameterString)) {
                     $queryParameterString .= '&';
@@ -809,7 +809,7 @@ 
      *
      * @return string
      */
-    private function primitiveToString(IType &$type, $primitiveValue)
+    private function primitiveToString(IType & $type, $primitiveValue)
     {
         if ($type instanceof Boolean) {
             $stringValue = (true === $primitiveValue) ? 'true' : 'false';
@@ -909,7 +909,7 @@ 
      * @param $result
      * @return ODataBagContent|null
      */
-    protected function writeBagValue(ResourceType &$resourceType, $result)
+    protected function writeBagValue(ResourceType & $resourceType, $result)
     {
         assert(null == $result || is_array($result), 'Bag parameter must be null or array');
         $typeKind = $resourceType->getResourceTypeKind();
@@ -942,7 +942,7 @@ 
      * @param string|null   $propertyName
      * @return ODataPropertyContent
      */
-    protected function writeComplexValue(ResourceType &$resourceType, &$result, $propertyName = null)
+    protected function writeComplexValue(ResourceType & $resourceType, &$result, $propertyName = null)
     {
         assert(is_object($result), 'Supplied $customObject must be an object');
 

src/Query/LaravelExpressionProvider.php

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_ENDSWITH] = function ($params) {
             return '(strcmp(substr('.$params[0].', strlen('.$params[0].') - strlen('.$params[1].')), '
-                   .$params[1].') === 0)';
+                    .$params[1].') === 0)';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_INDEXOF] = function ($params) {
             return 'strpos('.$params[0].', '.$params[1].')';

Spacing +31 added lines, -31 removed lines

@@ -52,81 +52,81 @@ 
     public function __construct()
     {
         $this->functionDescriptionParsers[ODataConstants::STRFUN_COMPARE] = function ($params) {
-            return 'strcmp('.$params[0].', '.$params[1].')';
+            return 'strcmp(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_ENDSWITH] = function ($params) {
-            return '(strcmp(substr('.$params[0].', strlen('.$params[0].') - strlen('.$params[1].')), '
-                   .$params[1].') === 0)';
+            return '(strcmp(substr(' . $params[0] . ', strlen(' . $params[0] . ') - strlen(' . $params[1] . ')), '
+                   .$params[1] . ') === 0)';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_INDEXOF] = function ($params) {
-            return 'strpos('.$params[0].', '.$params[1].')';
+            return 'strpos(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_REPLACE] = function ($params) {
-            return 'str_replace('.$params[1].', '.$params[2].', '.$params[0].')';
+            return 'str_replace(' . $params[1] . ', ' . $params[2] . ', ' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_STARTSWITH] = function ($params) {
-            return '(strpos('.$params[0].', '.$params[1].') === 0)';
+            return '(strpos(' . $params[0] . ', ' . $params[1] . ') === 0)';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TOLOWER] = function ($params) {
-            return 'strtolower('.$params[0].')';
+            return 'strtolower(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TOUPPER] = function ($params) {
-            return 'strtoupper('.$params[0].')';
+            return 'strtoupper(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_TRIM] = function ($params) {
-            return 'trim('.$params[0].')';
+            return 'trim(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_SUBSTRING] = function ($params) {
             return count($params) == 3 ?
-                'substr('.$params[0].', '.$params[1].', '.$params[2].')' : 'substr('.$params[0].', '.$params[1].')';
+                'substr(' . $params[0] . ', ' . $params[1] . ', ' . $params[2] . ')' : 'substr(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_SUBSTRINGOF] = function ($params) {
-            return '(strpos('.$params[1].', '.$params[0].') !== false)';
+            return '(strpos(' . $params[1] . ', ' . $params[0] . ') !== false)';
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_CONCAT] = function ($params) {
-            return $params[0].' . '.$params[1];
+            return $params[0] . ' . ' . $params[1];
         };
         $this->functionDescriptionParsers[ODataConstants::STRFUN_LENGTH] = function ($params) {
-            return 'strlen('.$params[0].')';
+            return 'strlen(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::GUIDFUN_EQUAL] = function ($params) {
-            return self::TYPE_NAMESPACE.'Guid::guidEqual('.$params[0].', '.$params[1].')';
+            return self::TYPE_NAMESPACE . 'Guid::guidEqual(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_COMPARE] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::dateTimeCmp('.$params[0].', '.$params[1].')';
+            return self::TYPE_NAMESPACE . 'DateTime::dateTimeCmp(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_YEAR] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::year('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::year(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_MONTH] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::month('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::month(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_DAY] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::day('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::day(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_HOUR] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::hour('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::hour(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_MINUTE] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::minute('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::minute(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::DATETIME_SECOND] = function ($params) {
-            return self::TYPE_NAMESPACE.'DateTime::second('.$params[0].')';
+            return self::TYPE_NAMESPACE . 'DateTime::second(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_ROUND] = function ($params) {
-            return 'round('.$params[0].')';
+            return 'round(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_CEILING] = function ($params) {
-            return 'ceil('.$params[0].')';
+            return 'ceil(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::MATHFUN_FLOOR] = function ($params) {
-            return 'floor('.$params[0].')';
+            return 'floor(' . $params[0] . ')';
         };
         $this->functionDescriptionParsers[ODataConstants::BINFUL_EQUAL] = function ($params) {
-            return self::TYPE_NAMESPACE.'Binary::binaryEqual('.$params[0].', '.$params[1].')';
+            return self::TYPE_NAMESPACE . 'Binary::binaryEqual(' . $params[0] . ', ' . $params[1] . ')';
         };
         $this->functionDescriptionParsers['is_null'] = function ($params) {
-            return 'is_null('.$params[0].')';
+            return 'is_null(' . $params[0] . ')';
         };
     }
     /**
@@ -157,7 +157,7 @@ 
      */
     public function setResourceType(ResourceType $resourceType)
     {
-        $this->iteratorName = '$'.$resourceType->getName();
+        $this->iteratorName = '$' . $resourceType->getName();
         $this->resourceType = $resourceType;
     }
     /**
@@ -282,11 +282,11 @@ 
         $parent = $expression;
         $variable = null;
         do {
-            $variable = $parent->getResourceProperty()->getName().self::MEMBER_ACCESS.$variable;
+            $variable = $parent->getResourceProperty()->getName() . self::MEMBER_ACCESS . $variable;
             $parent = $parent->getParent();
         } while ($parent != null);
         $variable = rtrim($variable, self::MEMBER_ACCESS);
-        $variable = $this->getIteratorName().self::MEMBER_ACCESS.$variable;
+        $variable = $this->getIteratorName() . self::MEMBER_ACCESS . $variable;
         return $variable;
     }
     /**
@@ -318,7 +318,7 @@ 
      */
     private function prepareBinaryExpression($operator, $left, $right)
     {
-        return self::OPEN_BRACKET.$left.' '.$operator.' '.$right.self::CLOSE_BRACKET;
+        return self::OPEN_BRACKET . $left . ' ' . $operator . ' ' . $right . self::CLOSE_BRACKET;
     }
     /**
      * To format unary expression.
@@ -330,6 +330,6 @@ 
      */
     private function prepareUnaryExpression($operator, $child)
     {
-        return $operator.self::OPEN_BRACKET.$child.self::CLOSE_BRACKET;
+        return $operator . self::OPEN_BRACKET . $child . self::CLOSE_BRACKET;
     }
 }

src/Models/MetadataTrait.php

Indentation +11 added lines, -11 removed lines

@@ -301,17 +301,17 @@
                         $lastCode = $code[strlen($code) - 1];
                         assert('}' == $lastCode, 'Final character of function definition must be closing brace');
                         foreach ([
-                                     'hasMany',
-                                     'hasManyThrough',
-                                     'belongsToMany',
-                                     'hasOne',
-                                     'belongsTo',
-                                     'morphOne',
-                                     'morphTo',
-                                     'morphMany',
-                                     'morphToMany',
-                                     'morphedByMany'
-                                 ] as $relation) {
+                                        'hasMany',
+                                        'hasManyThrough',
+                                        'belongsToMany',
+                                        'hasOne',
+                                        'belongsTo',
+                                        'morphOne',
+                                        'morphTo',
+                                        'morphMany',
+                                        'morphToMany',
+                                        'morphedByMany'
+                                    ] as $relation) {
                             $search = '$this->' . $relation . '(';
                             if ($pos = stripos($code, $search)) {
                                 //Resolve the relation's model to a Relation object.

Spacing +17 added lines, -17 removed lines

@@ -307,7 +307,7 @@ 
                         $reflection = new \ReflectionMethod($model, $method);
 
                         $file = new \SplFileObject($reflection->getFileName());
-                        $file->seek($reflection->getStartLine() - 1);
+                        $file->seek($reflection->getStartLine()-1);
                         $code = '';
                         while ($file->key() < $reflection->getEndLine()) {
                             $code .= $file->current();
@@ -320,8 +320,8 @@ 
                             'Function definition must have keyword \'function\''
                         );
                         $begin = strpos($code, 'function(');
-                        $code = substr($code, $begin, strrpos($code, '}') - $begin + 1);
-                        $lastCode = $code[strlen($code) - 1];
+                        $code = substr($code, $begin, strrpos($code, '}')-$begin+1);
+                        $lastCode = $code[strlen($code)-1];
                         assert('}' == $lastCode, 'Final character of function definition must be closing brace');
                         foreach ([
                                      'hasMany',
@@ -489,7 +489,7 @@ 
                         break;
                     }
                 }
-                assert(in_array($fkMethodName, $methodList), 'Selected method, '.$fkMethodName.', not in method list');
+                assert(in_array($fkMethodName, $methodList), 'Selected method, ' . $fkMethodName . ', not in method list');
                 $rkMethodName = 'getQualifiedRelatedPivotKeyName';
                 foreach ($rkList as $option) {
                     if (in_array($option, $methodList)) {
@@ -497,7 +497,7 @@ 
                         break;
                     }
                 }
-                assert(in_array($rkMethodName, $methodList), 'Selected method, '.$rkMethodName.', not in method list');
+                assert(in_array($rkMethodName, $methodList), 'Selected method, ' . $rkMethodName . ', not in method list');
                 $line = ['fk' => $fkMethodName, 'rk' => $rkMethodName];
                 static::$methodPrimary[get_class($foo)] = $line;
             }
@@ -520,13 +520,13 @@ 
             } else {
                 $methodList = get_class_methods(get_class($foo));
                 $fkCombo = array_values(array_intersect($fkList, $methodList));
-                assert(1 <= count($fkCombo), 'Expected at least 1 element in foreign-key list, got '.count($fkCombo));
+                assert(1 <= count($fkCombo), 'Expected at least 1 element in foreign-key list, got ' . count($fkCombo));
                 $fkMethodName = $fkCombo[0];
-                assert(in_array($fkMethodName, $methodList), 'Selected method, '.$fkMethodName.', not in method list');
+                assert(in_array($fkMethodName, $methodList), 'Selected method, ' . $fkMethodName . ', not in method list');
                 $rkCombo = array_values(array_intersect($rkList, $methodList));
-                assert(1 <= count($rkCombo), 'Expected at least 1 element in related-key list, got '.count($rkCombo));
+                assert(1 <= count($rkCombo), 'Expected at least 1 element in related-key list, got ' . count($rkCombo));
                 $rkMethodName = $rkCombo[0];
-                assert(in_array($rkMethodName, $methodList), 'Selected method, '.$rkMethodName.', not in method list');
+                assert(in_array($rkMethodName, $methodList), 'Selected method, ' . $rkMethodName . ', not in method list');
                 $line = ['fk' => $fkMethodName, 'rk' => $rkMethodName];
                 static::$methodAlternate[get_class($foo)] = $line;
             }
@@ -578,10 +578,10 @@ 
 
             $keyRaw = $isBelong ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
             $keySegments = explode('.', $keyRaw);
-            $keyName = $keySegments[count($keySegments) - 1];
+            $keyName = $keySegments[count($keySegments)-1];
             $localRaw = $isBelong ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
             $localSegments = explode('.', $localRaw);
-            $localName = $localSegments[count($localSegments) - 1];
+            $localName = $localSegments[count($localSegments)-1];
             $first = $keyName;
             $last = $localName;
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ);
@@ -607,10 +607,10 @@ 
 
             $keyName = $isBelong ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
             $keySegments = explode('.', $keyName);
-            $keyName = $keySegments[count($keySegments) - 1];
+            $keyName = $keySegments[count($keySegments)-1];
             $localRaw = $isBelong ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
             $localSegments = explode('.', $localRaw);
-            $localName = $localSegments[count($localSegments) - 1];
+            $localName = $localSegments[count($localSegments)-1];
             $first = $isBelong ? $localName : $keyName;
             $last = $isBelong ? $keyName : $localName;
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ);
@@ -634,10 +634,10 @@ 
 
             $keyRaw = $isMany ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
             $keySegments = explode('.', $keyRaw);
-            $keyName = $keySegments[count($keySegments) - 1];
+            $keyName = $keySegments[count($keySegments)-1];
             $localRaw = $isMany ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
             $localSegments = explode('.', $localRaw);
-            $localName = $localSegments[count($localSegments) - 1];
+            $localName = $localSegments[count($localSegments)-1];
             $first = $isMany ? $keyName : $localName;
             $last = $isMany ? $localName : $keyName;
             $this->addRelationsHook($hooks, $first, $property, $last, $mult, $targ, 'unknown');
@@ -660,10 +660,10 @@ 
 
             $keyRaw = $isMany ? $foo->$fkMethodName() : $foo->$fkMethodAlternate();
             $keySegments = explode('.', $keyRaw);
-            $keyName = $keySegments[count($keySegments) - 1];
+            $keyName = $keySegments[count($keySegments)-1];
             $localRaw = $isMany ? $foo->$rkMethodName() : $foo->$rkMethodAlternate();
             $localSegments = explode('.', $localRaw);
-            $localName = $localSegments[count($localSegments) - 1];
+            $localName = $localSegments[count($localSegments)-1];
 
             $first = $keyName;
             $last = (isset($localName) && '' != $localName) ? $localName : $foo->getRelated()->getKeyName();

src/Query/LaravelReadQuery.php

Spacing +5 added lines, -5 removed lines

@@ -140,8 +140,8 @@ 
         $nullFilter = true;
         $isvalid = null;
         if (isset($filterInfo)) {
-            $method = 'return '.$filterInfo->getExpressionAsString().';';
-            $clln = '$'.$resourceSet->getResourceType()->getName();
+            $method = 'return ' . $filterInfo->getExpressionAsString() . ';';
+            $clln = '$' . $resourceSet->getResourceType()->getName();
             $isvalid = create_function($clln, $method);
             $nullFilter = false;
         }
@@ -169,7 +169,7 @@ 
                     // so we can't bail out early
                     $rawCount += $results->count();
                     // now bolt on filtrate to accumulating result set if we haven't accumulated enough bitz
-                    if ($rawTop > $resultSet->count() + $skip) {
+                    if ($rawTop > $resultSet->count()+$skip) {
                         $resultSet = collect(array_merge($resultSet->all(), $results->all()));
                         $sliceAmount = min($skip, $resultSet->count());
                         $resultSet = $resultSet->slice($sliceAmount);
@@ -207,7 +207,7 @@ 
         if (QueryType::COUNT() == $queryType || QueryType::ENTITIES_WITH_COUNT() == $queryType) {
             $result->count = $resultCount;
         }
-        $hazMore = $bulkSetCount > $skip + count($resultSet);
+        $hazMore = $bulkSetCount > $skip+count($resultSet);
         $result->hasMore = $hazMore;
         return $result;
     }
@@ -367,7 +367,7 @@ 
     ) {
         $propertyName = $targetProperty->getName();
         if (!method_exists($sourceEntityInstance, $propertyName)) {
-            $msg = 'Relation method, '.$propertyName.', does not exist on supplied entity.';
+            $msg = 'Relation method, ' . $propertyName . ', does not exist on supplied entity.';
             throw new InvalidArgumentException($msg);
         }
         // take key descriptor and turn it into where clause here, rather than in getResource call

src/Providers/MetadataControllerProvider.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
                     }
                     assert(
                         !$alreadyKey,
-                        'Mapping already defined for model '.$key.' and CRUD verb '.$barrel
+                        'Mapping already defined for model ' . $key . ' and CRUD verb ' . $barrel
                     );
                     $metamix[$key][$barrel] = $roll;
                 }

src/Controllers/MetadataControllerTrait.php

Spacing +8 added lines, -8 removed lines

@@ -40,7 +40,7 @@ 
         assert(0 < count($this->mapping), 'Mapping array must not be empty');
 
         if (!array_key_exists($modelName, $this->mapping)) {
-            throw new \Exception('Metadata mapping for model '.$modelName.' not defined');
+            throw new \Exception('Metadata mapping for model ' . $modelName . ' not defined');
         }
 
         $this->checkCrudVerbDefined($crudVerb);
@@ -48,7 +48,7 @@ 
 
         $lookup = $this->mapping[$modelName];
         if (!is_array($lookup)) {
-            throw new \Exception('Metadata mapping for model '.$modelName.' not an array');
+            throw new \Exception('Metadata mapping for model ' . $modelName . ' not an array');
         }
 
         if (!array_key_exists($crudVerb, $lookup)) {
@@ -56,16 +56,16 @@ 
                 // optional crud verbs don't have to be defined - so we can return null
                 return null;
             }
-            throw new \Exception('Metadata mapping for CRUD verb '.$crudVerb.' on model '.$modelName.' not defined');
+            throw new \Exception('Metadata mapping for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' not defined');
         }
         $result = $lookup[$crudVerb];
         if (!isset($result)) {
-            throw new \Exception('Metadata mapping for CRUD verb '.$crudVerb.' on model '.$modelName.' null');
+            throw new \Exception('Metadata mapping for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' null');
         }
 
         if (!method_exists($this, $result)) {
             throw new \Exception(
-                'Metadata target for CRUD verb '.$crudVerb.' on model '.$modelName.' does not exist'
+                'Metadata target for CRUD verb ' . $crudVerb . ' on model ' . $modelName . ' does not exist'
             );
         }
 
@@ -87,17 +87,17 @@ 
         // check that mapping array is well formed and sane, rather than waiting to stab us with a spatula
         foreach ($this->mapping as $key => $map) {
             if (!is_array($map)) {
-                throw new \Exception('Metadata mapping for model '.$key.' not an array');
+                throw new \Exception('Metadata mapping for model ' . $key . ' not an array');
             }
             foreach ($map as $verb => $method) {
                 $this->checkCrudVerbDefined($verb);
                 if (!isset($method)) {
-                    throw new \Exception('Metadata mapping for CRUD verb '.$verb.' on model '.$key.' null');
+                    throw new \Exception('Metadata mapping for CRUD verb ' . $verb . ' on model ' . $key . ' null');
                 }
 
                 if (!method_exists($this, $method)) {
                     throw new \Exception(
-                        'Metadata target for CRUD verb '.$verb.' on model '.$key.' does not exist'
+                        'Metadata target for CRUD verb ' . $verb . ' on model ' . $key . ' does not exist'
                     );
                 }
                 $parmArray = $this->getParameterNames($method);

src/Query/LaravelQuery.php

Spacing +7 added lines, -7 removed lines

@@ -356,12 +356,12 @@ 
         $map = $raw->getMetadata();
 
         if (!array_key_exists($class, $map)) {
-            throw new \POData\Common\InvalidOperationException('Controller mapping missing for class '.$class.'.');
+            throw new \POData\Common\InvalidOperationException('Controller mapping missing for class ' . $class . '.');
         }
         $goal = $raw->getMapping($class, $verb);
         if (null == $goal) {
             throw new \POData\Common\InvalidOperationException(
-                'Controller mapping missing for '.$verb.' verb on class '.$class.'.'
+                'Controller mapping missing for ' . $verb . ' verb on class ' . $class . '.'
             );
         }
 
@@ -435,7 +435,7 @@ 
                 throw new ODataException($e->getMessage(), 500);
             }
         }
-        throw new ODataException('Target model not successfully '.$lastWord, 422);
+        throw new ODataException('Target model not successfully ' . $lastWord, 422);
     }
 
     /**
@@ -478,7 +478,7 @@ 
         }
         $outData = $result->getData();
         if (is_object($outData)) {
-            $outData = (array)$outData;
+            $outData = (array) $outData;
         }
 
         if (!is_array($outData)) {
@@ -666,7 +666,7 @@ 
             $otherPropName = $this->getMetadataProvider()
                 ->resolveReverseProperty($sourceEntityInstance, $targetEntityInstance, $navPropName);
             if (null === $otherPropName) {
-                $msg = 'Bad navigation property, '.$navPropName.', on source model '.get_class($sourceEntityInstance);
+                $msg = 'Bad navigation property, ' . $navPropName . ', on source model ' . get_class($sourceEntityInstance);
                 throw new \InvalidArgumentException($msg);
             }
             $this->unhookSingleModel(
@@ -700,7 +700,7 @@ 
         }
         $targType = $relation->getRelated();
         if (!$targetEntityInstance instanceof $targType) {
-            $msg = 'Target instance must be of type compatible with relation declared in method '.$navPropName;
+            $msg = 'Target instance must be of type compatible with relation declared in method ' . $navPropName;
             throw new \InvalidArgumentException($msg);
         }
         return $relation;
@@ -741,7 +741,7 @@ 
                 $var = new $varType();
                 if ($spec['isRequest']) {
                     $var->setMethod($isCreate ? 'POST' : 'PUT');
-                    $bulkData = [ 'data' => $data];
+                    $bulkData = ['data' => $data];
                     if (null !== $keyDescriptors) {
                         $keys = [];
                         foreach ($keyDescriptors as $desc) {

src/Providers/MetadataProvider.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 
 class MetadataProvider extends MetadataBaseProvider
 {
-    protected $multConstraints = [ '0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
+    protected $multConstraints = ['0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
     protected static $metaNAMESPACE = 'Data';
     protected static $relationCache;
     protected static $isBooted = false;
@@ -216,7 +216,7 @@ 
         $knownKeys = array_keys($knownSide);
         $unknownKeys = array_keys($unknownSide);
         $dualKeys = array_intersect($knownKeys, $unknownKeys);
-        assert(count($hooks) == (count($unknownKeys) + count($knownKeys) - count($dualKeys)));
+        assert(count($hooks) == (count($unknownKeys)+count($knownKeys)-count($dualKeys)));
         // if either list is empty, bail out - there's nothing to do
         if (0 === count($knownSide) || 0 === count($unknownSide)) {
             return [];

src/Models/MetadataRelationHolder.php

Indentation +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
 
         $maxRel = $isKnown ? PHP_INT_MAX : 2 * $numRel;
         $msg = 'Key '.$keyName. ' on class '.$className . ' should have no more than '
-               .$maxRel.' lines, has '.count($result);
+                .$maxRel.' lines, has '.count($result);
         assert($maxRel >= count($result), $msg);
         return $result;
     }

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 
 class MetadataRelationHolder
 {
-    protected $multConstraints = [ '0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
+    protected $multConstraints = ['0..1' => ['1'], '1' => ['0..1', '*'], '*' => ['1', '*']];
     protected $relations = [];
 
     public function __construct()
@@ -26,7 +26,7 @@ 
         }
         $className = get_class($model);
         if (array_key_exists($className, $this->relations)) {
-            $msg = $className.' already added';
+            $msg = $className . ' already added';
             throw new \InvalidArgumentException($msg);
         }
         $this->relations[$className] = $model->getRelationships();
@@ -113,8 +113,8 @@ 
         }
 
         $maxRel = $isKnown ? PHP_INT_MAX : 2 * $numRel;
-        $msg = 'Key '.$keyName. ' on class '.$className . ' should have no more than '
-               .$maxRel.' lines, has '.count($result);
+        $msg = 'Key ' . $keyName . ' on class ' . $className . ' should have no more than '
+               .$maxRel . ' lines, has ' . count($result);
         assert($maxRel >= count($result), $msg);
         return $result;
     }