balint777 / POData

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

Indentation +2 added lines, -2 removed lines

@@ -103,8 +103,8 @@
                 Messages::orderByLeafNodeArgumentShouldBeNonEmptyArray()
             );
         }
-		$flag1 = "";
-		$flag2 = "";
+        $flag1 = "";
+        $flag2 = "";
         $parameterNames = null;
         $accessor1 = null;
         $accessor2 = null;

src/POData/OperationContext/Web/IncomingRequest.php

Indentation +7 added lines, -7 removed lines

@@ -62,8 +62,8 @@ 
         $this->_queryOptionsCount = null;
         $this->_headers = null;
         $this->getHeaders();
-		//Not sure about this
-		$this->getQueryParameters();
+        //Not sure about this
+        $this->getQueryParameters();
     }
 
     /**
@@ -138,10 +138,10 @@ 
 
             $this->_rawUrl .= "://" . $_SERVER[HttpProcessUtility::headerToServerKey(ODataConstants::HTTPREQUEST_HEADER_HOST)];
             $this->_rawUrl .= $_SERVER[ODataConstants::HTTPREQUEST_URI];
-			//It looks like it is not working, for example if there is an $inlinecount then it will double the parameter
-			//if (!empty($_SERVER[ODataConstants::HTTPREQUEST_QUERY_STRING])) {
-			//	$this->_rawUrl .= "?" . $_SERVER[ODataConstants::HTTPREQUEST_QUERY_STRING];
-			//}
+            //It looks like it is not working, for example if there is an $inlinecount then it will double the parameter
+            //if (!empty($_SERVER[ODataConstants::HTTPREQUEST_QUERY_STRING])) {
+            //	$this->_rawUrl .= "?" . $_SERVER[ODataConstants::HTTPREQUEST_QUERY_STRING];
+            //}
         }
 
         return $this->_rawUrl;
@@ -194,7 +194,7 @@ 
         if (is_null($this->_queryOptions)) {
             $queryString = $this->getQueryString();
             $this->_queryOptions = array();
-			//example $filter=CustomerID gt \'ALFKI\'
+            //example $filter=CustomerID gt \'ALFKI\'
             foreach (explode('&', $queryString) as $queryOptionAsString) {
                 $queryOptionAsString = trim($queryOptionAsString);
                 if (!empty($queryOptionAsString)) {

src/POData/OperationContext/ServiceHost.php

Indentation +2 added lines, -2 removed lines

@@ -124,12 +124,12 @@
     {
         if (is_null($this->_absoluteRequestUri)) {
             $this->_absoluteRequestUriAsString = $this->_operationContext->incomingRequest()->getRawUrl();
-			$originalRequest = $this->_operationContext->incomingRequest();
+            $originalRequest = $this->_operationContext->incomingRequest();
             // Validate the uri first
             try {
                 $this->_fullAbsoluteRequestUriAsString = $this->_absoluteRequestUriAsString;
                 $this->_fullAbsoluteRequestUri = new Url($this->_fullAbsoluteRequestUriAsString);
-				$this->_fullAbsoluteRequestUri->setQueryOptions($originalRequest->getQueryParameters());
+                $this->_fullAbsoluteRequestUri->setQueryOptions($originalRequest->getQueryParameters());
             } catch (UrlFormatException $exception) {
                 throw ODataException::createBadRequestError($exception->getMessage());
             }

src/POData/Common/Url.php

Indentation +8 added lines, -8 removed lines

@@ -68,19 +68,19 @@
         }
     }
 
-	 /**
-     * Gets query options
-     *
-     * @return array
-     */
+        /**
+         * Gets query options
+         *
+         * @return array
+         */
     public function getQueryOptions()
     {
         return $this->_queryOptions;
     }
 
-	 /**
-     * Sets query options
-     */
+        /**
+         * Sets query options
+         */
     public function setQueryOptions($_queryOptions)
     {
         $this->_queryOptions = $_queryOptions;

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

Braces +3 added lines, -1 removed lines

@@ -91,7 +91,9 @@
 				$resourceType  = $resourceType->getResourceType();
 			}
 			$resourceProperty = $resourceType->resolveProperty($property->getName());
-			if(is_null($resourceProperty)) continue;
+			if(is_null($resourceProperty)) {
+			    continue;
+			}
 
 			$resourcePropertyType = $resourceProperty->getResourceType();
 			$resourceKind = $resourcePropertyType->getResourceTypeKind();

Indentation +50 added lines, -50 removed lines

@@ -65,7 +65,7 @@ 
      */
     private $_dummyObject;
 
-	private $_rootOrderByNode;
+    private $_rootOrderByNode;
     /**
      * Creates new instance of OrderByParser
      *
@@ -79,55 +79,55 @@ 
     }
 
     private static function _mock($resourceType): object {
-		$instanceType = $resourceType->getInstanceType();
-		$reflection = new \ReflectionClass($instanceType->name);
-		if($reflection->isAbstract()) {
-			return new stdClass();
-		}
-		$mock = $reflection->newInstanceWithoutConstructor();
-
-		foreach ($reflection->getProperties() as $property) {
-			$property->setAccessible(true);
-
-			$type = $property->getType();
-			if($resourceType instanceof ResourceProperty) {
-				$resourceType  = $resourceType->getResourceType();
-			}
-			$resourceProperty = $resourceType->resolveProperty($property->getName());
-			if(is_null($resourceProperty)) continue;
-
-			$resourcePropertyType = $resourceProperty->getResourceType();
-			$resourceKind = $resourcePropertyType->getResourceTypeKind();
-			if ($type && !$type->allowsNull()) {
-				if ($resourceKind === ResourceTypeKind::PRIMITIVE) {
-					switch ($type->getName()) {
-						case 'int':
-							$property->setValue($mock, 0);
-							break;
-						case 'string':
-							$property->setValue($mock, '');
-							break;
-						case 'bool':
-							$property->setValue($mock, false);
-							break;
-						case 'array':
-							// If the property is of type array, set it as an empty array
-							$property->setValue($mock, []);
-							break;
-						default:
-							break;
-					}
-				} else {
-					continue;
-				}
-			} else {
-				// If the property allows null, set it to null
-				$property->setValue($mock, null);
-			}
-		}
-
-		return $mock;
-	}
+        $instanceType = $resourceType->getInstanceType();
+        $reflection = new \ReflectionClass($instanceType->name);
+        if($reflection->isAbstract()) {
+            return new stdClass();
+        }
+        $mock = $reflection->newInstanceWithoutConstructor();
+
+        foreach ($reflection->getProperties() as $property) {
+            $property->setAccessible(true);
+
+            $type = $property->getType();
+            if($resourceType instanceof ResourceProperty) {
+                $resourceType  = $resourceType->getResourceType();
+            }
+            $resourceProperty = $resourceType->resolveProperty($property->getName());
+            if(is_null($resourceProperty)) continue;
+
+            $resourcePropertyType = $resourceProperty->getResourceType();
+            $resourceKind = $resourcePropertyType->getResourceTypeKind();
+            if ($type && !$type->allowsNull()) {
+                if ($resourceKind === ResourceTypeKind::PRIMITIVE) {
+                    switch ($type->getName()) {
+                        case 'int':
+                            $property->setValue($mock, 0);
+                            break;
+                        case 'string':
+                            $property->setValue($mock, '');
+                            break;
+                        case 'bool':
+                            $property->setValue($mock, false);
+                            break;
+                        case 'array':
+                            // If the property is of type array, set it as an empty array
+                            $property->setValue($mock, []);
+                            break;
+                        default:
+                            break;
+                    }
+                } else {
+                    continue;
+                }
+            } else {
+                // If the property allows null, set it to null
+                $property->setValue($mock, null);
+            }
+        }
+
+        return $mock;
+    }
 
     /**
      * This function perform the following tasks with the help of internal helper

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
     private static function _mock($resourceType): object {
 		$instanceType = $resourceType->getInstanceType();
 		$reflection = new \ReflectionClass($instanceType->name);
-		if($reflection->isAbstract()) {
+		if ($reflection->isAbstract()) {
 			return new stdClass();
 		}
 		$mock = $reflection->newInstanceWithoutConstructor();
@@ -90,11 +90,11 @@ 
 			$property->setAccessible(true);
 
 			$type = $property->getType();
-			if($resourceType instanceof ResourceProperty) {
-				$resourceType  = $resourceType->getResourceType();
+			if ($resourceType instanceof ResourceProperty) {
+				$resourceType = $resourceType->getResourceType();
 			}
 			$resourceProperty = $resourceType->resolveProperty($property->getName());
-			if(is_null($resourceProperty)) continue;
+			if (is_null($resourceProperty)) continue;
 
 			$resourcePropertyType = $resourceProperty->getResourceType();
 			$resourceKind = $resourcePropertyType->getResourceTypeKind();