Qeti / POData

src/POData/UriProcessor/QueryProcessor/ExpressionParser/Expressions/PropertyAccessExpression.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
     /**
      * Get the resource type of the property hold by this expression
      * 
-     * @return ResourceType
+     * @return \POData\Providers\Metadata\ResourceType
      */
     public function getResourceType()
     {

src/POData/UriProcessor/QueryProcessor/ExpressionParser/Expressions/UnaryExpression.php

Doc Comments +2 added lines, -2 removed lines

@@ -16,9 +16,9 @@
     /**
      * Construct a new instance of UnaryExpression
      * 
-     * @param unknown_type $child    Child expression
+     * @param AbstractExpression $child    Child expression
      * @param unknown_type $nodeType Expression node type
-     * @param unknown_type $type     Expression type
+     * @param \POData\Providers\Metadata\Type\IType $type     Expression type
      */
     public function __construct($child, $nodeType, $type)
     {

src/POData/UriProcessor/QueryProcessor/FunctionDescription.php

Doc Comments +12 added lines, -12 removed lines

@@ -255,7 +255,7 @@ 
     /** 
      * Get function description for string comparison
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function stringComparisonFunctions()
     {
@@ -270,7 +270,7 @@ 
     /**
      * Get function description for datetime comparison
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function dateTimeComparisonFunctions()
     {
@@ -285,7 +285,7 @@ 
     /**
      * Get function description for guid equality check
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function guidEqualityFunctions()
     {
@@ -300,7 +300,7 @@ 
     /**
      * Get function description for binary equality check
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function binaryEqualityFunctions()
     {
@@ -315,7 +315,7 @@ 
     /**
      * Get function descriptions for arithmetic operations
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function arithmeticOperationFunctions()
     {      
@@ -350,7 +350,7 @@ 
     /**      
      * Get function descriptions for arithmetic add operations
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[] indexed by function name
+     * @return FunctionDescription[] indexed by function name
      */
     public static function addOperationFunctions()
     {
@@ -360,7 +360,7 @@ 
     /**
      * Get function descriptions for arithmetic subtract operations
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[] indexed by function name
+     * @return FunctionDescription[] indexed by function name
      */
     public static function subtractOperationFunctions()
     {
@@ -370,7 +370,7 @@ 
     /**      
      * Get function descriptions for logical operations
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function logicalOperationFunctions()
     {
@@ -385,7 +385,7 @@ 
     /**
      * Get function descriptions for relational operations
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function relationalOperationFunctions()
     {
@@ -415,7 +415,7 @@ 
     /**
      * Get function descriptions for unary not operation
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function notOperationFunctions()
     {
@@ -442,7 +442,7 @@ 
     /**      
      * Get function description for unary negate operator
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[]
+     * @return FunctionDescription[]
      */
     public static function negateOperationFunctions()
     {
@@ -657,7 +657,7 @@ 
      * 
      * @throws ODataException
      * 
-     * @return \POData\UriProcessor\QueryProcessor\FunctionDescription[] Array of matching functions
+     * @return FunctionDescription[] Array of matching functions
      */
     public static function verifyFunctionExists($expressionToken)
     {

Unused Use Statements -3 removed lines

@@ -6,7 +6,6 @@ 
 use POData\Common\Messages;
 use POData\Common\ODataConstants;
 use POData\Providers\Metadata\Type\Null1;
-use POData\Providers\Metadata\Type\INavigationType;
 use POData\Providers\Metadata\Type\Int64;
 use POData\Providers\Metadata\Type\Int16;
 use POData\Providers\Metadata\Type\Guid;
@@ -17,10 +16,8 @@ 
 use POData\Providers\Metadata\Type\Int32;
 use POData\Providers\Metadata\Type\String;
 use POData\Providers\Metadata\Type\Boolean;
-use POData\Providers\Metadata\Type\Void;
 use POData\Providers\Metadata\Type\Binary;
 use POData\Providers\Metadata\Type\IType;
-use POData\UriProcessor\QueryProcessor\ExpressionParser\Expressions\ExpressionType;
 use POData\UriProcessor\QueryProcessor\ExpressionParser\Expressions\AbstractExpression;
 use POData\UriProcessor\QueryProcessor\ExpressionParser\ExpressionToken;
 

Indentation +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
     {
         $str = $this->returnType->getFullTypeName() . ' ' . $this->name . '(';
 
-	    foreach ($this->argumentTypes as $argumentType) {
+        foreach ($this->argumentTypes as $argumentType) {
             $str .= $argumentType->getFullTypeName() . ', ';
         }
 
@@ -247,7 +247,7 @@ 
                         array(new Double())
                     )
                 )
-          );
+            );
 
         return $functions;
     }

Spacing +7 added lines, -7 removed lines

@@ -69,13 +69,13 @@ 
      */
     public function getPrototypeAsString()
     {
-        $str = $this->returnType->getFullTypeName() . ' ' . $this->name . '(';
+        $str = $this->returnType->getFullTypeName().' '.$this->name.'(';
 
 	    foreach ($this->argumentTypes as $argumentType) {
-            $str .= $argumentType->getFullTypeName() . ', ';
+            $str .= $argumentType->getFullTypeName().', ';
         }
 
-        return rtrim($str, ', ') . ')';
+        return rtrim($str, ', ').')';
     }
 
     /**      
@@ -469,7 +469,7 @@ 
     {
         $string = null;
         foreach ($argExpressions as $argExpression) {
-            $string .= $argExpression->getType()->getFullTypeName() . ', ';
+            $string .= $argExpression->getType()->getFullTypeName().', ';
         }
 
         $string = rtrim($string, ', ');
@@ -630,7 +630,7 @@ 
     public static function validateUnaryOpArguments($expressionToken, $argExpression)
     {
         //Unary not
-        if (strcmp($expressionToken->Text, ODataConstants::KEYWORD_NOT) == 0 ) {
+        if (strcmp($expressionToken->Text, ODataConstants::KEYWORD_NOT) == 0) {
             $function = self::findFunctionWithPromotion(
                 self::notOperationFunctions(), 
                 array($argExpression)
@@ -671,7 +671,7 @@ 
             
         }
 
-        $filterFunctions =  self::filterFunctionDescriptions();
+        $filterFunctions = self::filterFunctionDescriptions();
         return $filterFunctions[$expressionToken->Text];
     }
 
@@ -695,7 +695,7 @@ 
         if ($function == null) {
             $protoTypes = null;
             foreach ($functions as $function) {
-                $protoTypes .=  $function->getPrototypeAsString() . '; ';
+                $protoTypes .= $function->getPrototypeAsString().'; ';
             }
 
             throw ODataException::createSyntaxError(

src/POData/UriProcessor/QueryProcessor/OrderByParser/OrderByInfo.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace POData\UriProcessor\QueryProcessor\OrderByParser;
 use POData\Common\Messages;
-use POData\Common\InvalidOperationException;
 
 /**
  * Class OrderByInfo

src/POData/UriProcessor/QueryProcessor/OrderByParser/OrderByLeafNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
      * 
      * @see library/POData/QueryProcessorOrderByParser.OrderByBaseNode::getResourceType()
      * 
-     * @return void
+     * @return \POData\Providers\Metadata\ResourceType
      */
     public function getResourceType()
     {

Spacing +12 added lines, -12 removed lines

@@ -111,25 +111,25 @@
         
         foreach ($ancestors as $i => $anscestor) {
             if ($i == 0) {
-                $parameterNames = array (
-                    '$' . $anscestor . 'A', '$' . $anscestor . 'B'
+                $parameterNames = array(
+                    '$'.$anscestor.'A', '$'.$anscestor.'B'
                 );
                 $accessor1 = $parameterNames[0];
                 $accessor2 = $parameterNames[1];
-                $flag1 = '$flag1 = ' . 'is_null(' . $accessor1. ') || ';
-                $flag2 = '$flag2 = ' . 'is_null(' . $accessor2. ') || '; 
+                $flag1 = '$flag1 = '.'is_null('.$accessor1.') || ';
+                $flag2 = '$flag2 = '.'is_null('.$accessor2.') || '; 
             } else {
-                $accessor1 .= '->' . $anscestor;
-                $accessor2 .= '->' . $anscestor;
-                $flag1 .= 'is_null(' .$accessor1 . ')' . ' || ';
-                $flag2 .= 'is_null(' .$accessor2 . ')' . ' || ';
+                $accessor1 .= '->'.$anscestor;
+                $accessor2 .= '->'.$anscestor;
+                $flag1 .= 'is_null('.$accessor1.')'.' || ';
+                $flag2 .= 'is_null('.$accessor2.')'.' || ';
             }
         }
 
-        $accessor1 .= '->' . $this->propertyName;
-        $accessor2 .= '->' . $this->propertyName;
-        $flag1 .= 'is_null(' . $accessor1 . ')';
-        $flag2 .= 'is_null(' . $accessor2 . ')';
+        $accessor1 .= '->'.$this->propertyName;
+        $accessor2 .= '->'.$this->propertyName;
+        $flag1 .= 'is_null('.$accessor1.')';
+        $flag2 .= 'is_null('.$accessor2.')';
 
         $code = "$flag1; 
              $flag2; 

src/POData/UriProcessor/QueryProcessor/OrderByParser/OrderByNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      * 
      * @see library/POData/QueryProcessorOrderByParser.OrderByBaseNode::getResourceType()
      * 
-     * @return ResourceType
+     * @return \POData\Providers\Metadata\ResourceType
      */
     public function getResourceType()
     {

src/POData/UriProcessor/QueryProcessor/QueryProcessor.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use POData\UriProcessor\QueryProcessor\SkipTokenParser\SkipTokenParser;
 use POData\UriProcessor\QueryProcessor\OrderByParser\OrderByParser;
 use POData\UriProcessor\QueryProcessor\ExpressionParser\ExpressionParser2;
-use POData\UriProcessor\QueryProcessor\ExpressionParser\FilterInfo;
 use POData\UriProcessor\QueryProcessor\ExpandProjectionParser\ExpandProjectionParser;
 use POData\Common\Messages;
 use POData\Common\ODataException;

Indentation +40 added lines, -40 removed lines

@@ -71,21 +71,21 @@ 
 
         $isSingleResult = $request->isSingleResult();
 
-	    //$top, $skip, $order, $inlinecount & $count are only applicable if:
-	    //The query targets a resource collection
+        //$top, $skip, $order, $inlinecount & $count are only applicable if:
+        //The query targets a resource collection
         $this->_setQueryApplicable = ($request->getTargetKind() == TargetKind::RESOURCE() && !$isSingleResult);
-	    //Or it's a $count resource (although $inlinecount isn't applicable in this case..but there's a check somewhere else for this
-	    $this->_setQueryApplicable |= $request->queryType == QueryType::COUNT();
+        //Or it's a $count resource (although $inlinecount isn't applicable in this case..but there's a check somewhere else for this
+        $this->_setQueryApplicable |= $request->queryType == QueryType::COUNT();
 
-	    //Paging is allowed if
-	    //The request targets a resource collection
-	    //and the request isn't for a $count segment
-	    $this->_pagingApplicable = $this->request->getTargetKind() == TargetKind::RESOURCE() && !$isSingleResult && ($request->queryType != QueryType::COUNT());
+        //Paging is allowed if
+        //The request targets a resource collection
+        //and the request isn't for a $count segment
+        $this->_pagingApplicable = $this->request->getTargetKind() == TargetKind::RESOURCE() && !$isSingleResult && ($request->queryType != QueryType::COUNT());
 
-	    $targetResourceType = $this->request->getTargetResourceType();
+        $targetResourceType = $this->request->getTargetResourceType();
         $targetResourceSetWrapper = $this->request->getTargetResourceSetWrapper();
 
-	    $this->_expandSelectApplicable = !is_null($targetResourceType)
+        $this->_expandSelectApplicable = !is_null($targetResourceType)
             && !is_null($targetResourceSetWrapper)
             && $targetResourceType->getResourceTypeKind() == ResourceTypeKind::ENTITY
             && !$this->request->isLinkUri();
@@ -270,7 +270,7 @@ 
             || $kind == TargetKind::COMPLEX_OBJECT()
             || $this->request->queryType == QueryType::COUNT() )
         ) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorQueryFilterOptionNotApplicable()
             );
         }
@@ -296,27 +296,27 @@ 
     {
         $inlineCount = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_INLINECOUNT );
 
-	    //If it's not specified, we're done
-	    if(is_null($inlineCount)) return;
+        //If it's not specified, we're done
+        if(is_null($inlineCount)) return;
 
-	    //If the service doesn't allow count requests..then throw an exception
+        //If the service doesn't allow count requests..then throw an exception
         if (!$this->service->getConfiguration()->getAcceptCountRequests()) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::configurationCountNotAccepted()
             );
         }
 
         $inlineCount = trim($inlineCount);
 
-	    //if it's set to none, we don't do inline counts
+        //if it's set to none, we don't do inline counts
         if ($inlineCount === ODataConstants::URI_ROWCOUNT_OFFOPTION) {
             return;
         }
 
-	    //You can't specify $count & $inlinecount together
-	    //TODO: ensure there's a test for this case see #55
+        //You can't specify $count & $inlinecount together
+        //TODO: ensure there's a test for this case see #55
         if ($this->request->queryType == QueryType::COUNT() ) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorInlineCountWithValueCount()
             );
         }
@@ -325,13 +325,13 @@ 
 
 
         if ($inlineCount === ODataConstants::URI_ROWCOUNT_ALLOPTION) {
-	        $this->request->queryType = QueryType::ENTITIES_WITH_COUNT();
+            $this->request->queryType = QueryType::ENTITIES_WITH_COUNT();
 
             $this->request->raiseMinVersionRequirement( 2, 0 );
             $this->request->raiseResponseVersion( 2, 0 );
 
         } else {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorInvalidInlineCountOptionError()
             );
         }
@@ -361,13 +361,13 @@ 
         }
 
         if (!$this->_pagingApplicable) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorSkipTokenNotAllowed()
             );
         }
 
         if (!$this->_isSSPagingRequired()) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorSkipTokenCannotBeAppliedForNonPagedResourceSet($this->request->getTargetResourceSetWrapper())
             );
         }
@@ -412,7 +412,7 @@ 
 
         if (!is_null($select)) {
             if (!$this->service->getConfiguration()->getAcceptProjectionRequests()) {
-				throw ODataException::createBadRequestError( Messages::configurationProjectionsNotAccepted() );
+                throw ODataException::createBadRequestError( Messages::configurationProjectionsNotAccepted() );
             }
 
             $this->_checkExpandOrSelectApplicable( ODataConstants::HTTPQUERY_STRING_SELECT );
@@ -423,19 +423,19 @@ 
         // 'RootProjectionNode' is required while generating next page Link
         if ($this->_expandSelectApplicable || $this->request->isLinkUri() ) {
 
-			$rootProjectionNode = ExpandProjectionParser::parseExpandAndSelectClause(
-				 $this->request->getTargetResourceSetWrapper(),
-				 $this->request->getTargetResourceType(),
-				 $this->request->getInternalOrderByInfo(),
-				 $this->request->getSkipCount(),
-				 $this->request->getTopCount(),
-				 $expand,
-				 $select,
-				 $this->service->getProvidersWrapper()
-			);
-			if ($rootProjectionNode->isSelectionSpecified()) {
-			    $this->request->raiseMinVersionRequirement(2, 0 );
-			}
+            $rootProjectionNode = ExpandProjectionParser::parseExpandAndSelectClause(
+                    $this->request->getTargetResourceSetWrapper(),
+                    $this->request->getTargetResourceType(),
+                    $this->request->getInternalOrderByInfo(),
+                    $this->request->getSkipCount(),
+                    $this->request->getTopCount(),
+                    $expand,
+                    $select,
+                    $this->service->getProvidersWrapper()
+            );
+            if ($rootProjectionNode->isSelectionSpecified()) {
+                $this->request->raiseMinVersionRequirement(2, 0 );
+            }
 
             if ($rootProjectionNode->hasPagedExpandedResult()) {
                 $this->request->raiseResponseVersion( 2, 0 );
@@ -533,7 +533,7 @@ 
             || !is_null($serviceHost->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_SKIPTOKEN))
             || !is_null($serviceHost->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_TOP))
         ) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorNoQueryOptionsApplicable()
             );
         }
@@ -551,7 +551,7 @@ 
     private function _checkSetQueryApplicable()
     {
         if (!$this->_setQueryApplicable) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorQuerySetOptionsNotApplicable()
             );
         }
@@ -572,7 +572,7 @@ 
     private function _checkExpandOrSelectApplicable($queryItem)
     {
         if (!$this->_expandSelectApplicable) {
-			throw ODataException::createBadRequestError(
+            throw ODataException::createBadRequestError(
                 Messages::queryProcessorSelectOrExpandOptionNotApplicable($queryItem)
             );
         }

Spacing +28 added lines, -28 removed lines

@@ -65,7 +65,7 @@ 
      * @param RequestDescription $request Description of the request submitted by client.
      * @param IService        $service        Reference to the service implementation.
      */
-    private function __construct(RequestDescription $request, IService $service ) {
+    private function __construct(RequestDescription $request, IService $service) {
         $this->request = $request;
         $this->service = $service;
 
@@ -102,7 +102,7 @@ 
      * 
      * @throws ODataException
      */
-    public static function process(RequestDescription $request, IService $service ) {
+    public static function process(RequestDescription $request, IService $service) {
         $queryProcessor = new QueryProcessor($request, $service);
         if ($request->getTargetSource() == TargetSource::NONE) {
             //A service directory, metadata or batch request
@@ -148,7 +148,7 @@ 
     private function _processSkipAndTop()
     {
         $value = null;
-        if ($this->_readSkipOrTopOption( ODataConstants::HTTPQUERY_STRING_SKIP, $value ) ) {
+        if ($this->_readSkipOrTopOption(ODataConstants::HTTPQUERY_STRING_SKIP, $value)) {
             $this->request->setSkipCount($value);
         }
 
@@ -160,7 +160,7 @@ 
                 ->getResourceSetPageSize(); 
         }
 
-        if ($this->_readSkipOrTopOption(ODataConstants::HTTPQUERY_STRING_TOP, $value) ) {
+        if ($this->_readSkipOrTopOption(ODataConstants::HTTPQUERY_STRING_TOP, $value)) {
             $this->request->setTopOptionCount($value);
             if ($isPagingRequired && $pageSize < $value) {
                 //If $top is greater than or equal to page size, 
@@ -196,7 +196,7 @@ 
      */
     private function _processOrderBy()
     {
-        $orderBy = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_ORDERBY );
+        $orderBy = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_ORDERBY);
 
         if (!is_null($orderBy)) {
             $this->_checkSetQueryApplicable();
@@ -217,12 +217,12 @@ 
          *     RequestDescription::getTopCount will give non-null value.
          *      
          */
-        if (!is_null($this->request->getSkipCount())|| !is_null($this->request->getTopCount())) {
-            $orderBy = !is_null($orderBy) ? $orderBy . ', ' : null;
+        if (!is_null($this->request->getSkipCount()) || !is_null($this->request->getTopCount())) {
+            $orderBy = !is_null($orderBy) ? $orderBy.', ' : null;
             $keys = array_keys($targetResourceType->getKeyProperties());
             //assert(!empty($keys))
             foreach ($keys as $key) {
-                $orderBy = $orderBy . $key . ', ';
+                $orderBy = $orderBy.$key.', ';
             }
 
             $orderBy = rtrim($orderBy, ', ');
@@ -260,7 +260,7 @@ 
      */ 
     private function _processFilter()
     {
-        $filter = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_FILTER );
+        $filter = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_FILTER);
         if (is_null($filter)) {
             return;
         }
@@ -268,7 +268,7 @@ 
         $kind = $this->request->getTargetKind();
         if (!($kind == TargetKind::RESOURCE()
             || $kind == TargetKind::COMPLEX_OBJECT()
-            || $this->request->queryType == QueryType::COUNT() )
+            || $this->request->queryType == QueryType::COUNT())
         ) {
 			throw ODataException::createBadRequestError(
                 Messages::queryProcessorQueryFilterOptionNotApplicable()
@@ -277,7 +277,7 @@ 
         $resourceType = $this->request->getTargetResourceType();
         $expressionProvider = $this->service->getProvidersWrapper()->getExpressionProvider();
         $filterInfo = ExpressionParser2::parseExpression2($filter, $resourceType, $expressionProvider);
-        $this->request->setFilterInfo( $filterInfo );
+        $this->request->setFilterInfo($filterInfo);
 
     }
 
@@ -294,10 +294,10 @@ 
      */
     private function _processCount()
     {
-        $inlineCount = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_INLINECOUNT );
+        $inlineCount = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_INLINECOUNT);
 
 	    //If it's not specified, we're done
-	    if(is_null($inlineCount)) return;
+	    if (is_null($inlineCount)) return;
 
 	    //If the service doesn't allow count requests..then throw an exception
         if (!$this->service->getConfiguration()->getAcceptCountRequests()) {
@@ -315,7 +315,7 @@ 
 
 	    //You can't specify $count & $inlinecount together
 	    //TODO: ensure there's a test for this case see #55
-        if ($this->request->queryType == QueryType::COUNT() ) {
+        if ($this->request->queryType == QueryType::COUNT()) {
 			throw ODataException::createBadRequestError(
                 Messages::queryProcessorInlineCountWithValueCount()
             );
@@ -327,8 +327,8 @@ 
         if ($inlineCount === ODataConstants::URI_ROWCOUNT_ALLOPTION) {
 	        $this->request->queryType = QueryType::ENTITIES_WITH_COUNT();
 
-            $this->request->raiseMinVersionRequirement( 2, 0 );
-            $this->request->raiseResponseVersion( 2, 0 );
+            $this->request->raiseMinVersionRequirement(2, 0);
+            $this->request->raiseResponseVersion(2, 0);
 
         } else {
 			throw ODataException::createBadRequestError(
@@ -355,7 +355,7 @@ 
      */
     private function _processSkipToken()
     {
-        $skipToken = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_SKIPTOKEN );
+        $skipToken = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_SKIPTOKEN);
         if (is_null($skipToken)) {
             return;
         }
@@ -383,8 +383,8 @@ 
             $skipToken
         );
         $this->request->setInternalSkipTokenInfo($internalSkipTokenInfo);
-        $this->request->raiseMinVersionRequirement( 2, 0 );
-        $this->request->raiseResponseVersion( 2, 0 );
+        $this->request->raiseMinVersionRequirement(2, 0);
+        $this->request->raiseResponseVersion(2, 0);
 
 
     }
@@ -402,26 +402,26 @@ 
      */
     private function _processExpandAndSelect()
     {
-        $expand = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_EXPAND );
+        $expand = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_EXPAND);
 
         if (!is_null($expand)) {
-            $this->_checkExpandOrSelectApplicable(ODataConstants::HTTPQUERY_STRING_EXPAND );
+            $this->_checkExpandOrSelectApplicable(ODataConstants::HTTPQUERY_STRING_EXPAND);
         }
 
-        $select = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_SELECT );
+        $select = $this->service->getHost()->getQueryStringItem(ODataConstants::HTTPQUERY_STRING_SELECT);
 
         if (!is_null($select)) {
             if (!$this->service->getConfiguration()->getAcceptProjectionRequests()) {
-				throw ODataException::createBadRequestError( Messages::configurationProjectionsNotAccepted() );
+				throw ODataException::createBadRequestError(Messages::configurationProjectionsNotAccepted());
             }
 
-            $this->_checkExpandOrSelectApplicable( ODataConstants::HTTPQUERY_STRING_SELECT );
+            $this->_checkExpandOrSelectApplicable(ODataConstants::HTTPQUERY_STRING_SELECT);
         }
 
         // We will generate RootProjectionNode in case of $link request also, but
         // expand and select in this case must be null (we are ensuring this above)
         // 'RootProjectionNode' is required while generating next page Link
-        if ($this->_expandSelectApplicable || $this->request->isLinkUri() ) {
+        if ($this->_expandSelectApplicable || $this->request->isLinkUri()) {
 
 			$rootProjectionNode = ExpandProjectionParser::parseExpandAndSelectClause(
 				 $this->request->getTargetResourceSetWrapper(),
@@ -434,13 +434,13 @@ 
 				 $this->service->getProvidersWrapper()
 			);
 			if ($rootProjectionNode->isSelectionSpecified()) {
-			    $this->request->raiseMinVersionRequirement(2, 0 );
+			    $this->request->raiseMinVersionRequirement(2, 0);
 			}
 
             if ($rootProjectionNode->hasPagedExpandedResult()) {
-                $this->request->raiseResponseVersion( 2, 0 );
+                $this->request->raiseResponseVersion(2, 0);
             }
-            $this->request->setRootProjectionNode($rootProjectionNode );
+            $this->request->setRootProjectionNode($rootProjectionNode);
 
         }
     } 

Braces +3 added lines, -1 removed lines

@@ -297,7 +297,9 @@
         $inlineCount = $this->service->getHost()->getQueryStringItem( ODataConstants::HTTPQUERY_STRING_INLINECOUNT );
 
 	    //If it's not specified, we're done
-	    if(is_null($inlineCount)) return;
+	    if(is_null($inlineCount)) {
+	        return;
+	    }
 
 	    //If the service doesn't allow count requests..then throw an exception
         if (!$this->service->getConfiguration()->getAcceptCountRequests()) {

src/POData/UriProcessor/QueryProcessor/SkipTokenParser/InternalSkipTokenInfo.php

Unused Use Statements -3 removed lines

@@ -2,10 +2,7 @@
 
 namespace POData\UriProcessor\QueryProcessor\SkipTokenParser;
 
-use POData\Providers\Metadata\Type\Guid;
 use POData\Providers\Metadata\Type\Null1;
-use POData\Providers\Metadata\Type\DateTime;
-use POData\Providers\Metadata\Type\String;
 use POData\Providers\Metadata\ResourceType;
 use POData\UriProcessor\QueryProcessor\OrderByParser\InternalOrderByInfo;
 use POData\Common\Messages;

Spacing +3 added lines, -3 removed lines

@@ -72,8 +72,8 @@ 
      *                                                                orderby path segments.
      * @param ResourceType                  &$resourceType            Reference to the type of the resource pointed by the request uri.
      */
-    public function __construct(InternalOrderByInfo &$internalOrderByInfo, 
-        $orderByValuesInSkipToken, ResourceType &$resourceType
+    public function __construct(InternalOrderByInfo & $internalOrderByInfo, 
+        $orderByValuesInSkipToken, ResourceType & $resourceType
     ) {
         $this->_internalOrderByInfo = $internalOrderByInfo;        
         $this->_orderByValuesInSkipToken = $orderByValuesInSkipToken;
@@ -282,7 +282,7 @@ 
                     } else if ($isLastSegment) {
                         $type = $subPathSegment->getInstanceType();
                         $value = $type->convertToOData($currentObject);
-                        $nextPageLink .= $value . ', ';
+                        $nextPageLink .= $value.', ';
                     }                    
                 } catch (\ReflectionException $reflectionException) {
                     throw ODataException::createInternalServerError(

src/POData/UriProcessor/QueryProcessor/SkipTokenParser/SkipTokenInfo.php

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace POData\UriProcessor\QueryProcessor\SkipTokenParser;
 
-use POData\Common\Messages;
-use POData\Providers\Metadata\ResourceType;
 use POData\UriProcessor\QueryProcessor\OrderByParser\OrderByInfo;
 
 /**

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      * @param array(int,(array(string,IType))) $orderByValuesInSkipToken Collection of values in the skiptoken corrosponds
      *                                                                   to the orderby path segments.
      */
-    public function __construct(OrderByInfo &$orderByInfo, $orderByValuesInSkipToken)
+    public function __construct(OrderByInfo & $orderByInfo, $orderByValuesInSkipToken)
     {
         $this->_orderByInfo = $orderByInfo;
         $this->_orderByValuesInSkipToken = $orderByValuesInSkipToken;