c-harris / ODataMetadata

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIDecimalTypeReference.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         assert($typeRef instanceof IDecimalTypeReference);
         $primitive = $typeRef->getDefinition();
         assert($primitive instanceof IPrimitiveType);
-        return $typeRef->getDefinition() != null && !$primitive->getPrimitiveKind()->isDecimal() ? [ InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef) ] : null;
+        return $typeRef->getDefinition() != null && !$primitive->getPrimitiveKind()->isDecimal() ? [InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef)] : null;
     }
 
     public function forType(): string

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIFunctionBase.php

Braces +1 added lines, -2 removed lines

@@ -20,8 +20,7 @@
         // Return type is optional for function imports and is required for MDFs. Both cases are derived interfaces (IEdmFunctionImport and IEdmFunction).
         // So, from the point of view of this interface, we consider return type as optional and it is expected that IEdmFunction visitor will have
         // an additional null check for the return type.
-        if ($function->getReturnType() != null)
-        {
+        if ($function->getReturnType() != null) {
             // Function owns its return type reference, so it goes as a followup.
             $followup[] = $function->getReturnType();
         }

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIEntitySet.php

Braces +7 added lines, -17 removed lines

@@ -16,12 +16,9 @@ 
         assert($item instanceof IEntitySet);
         $errors = null;
 
-        if ($item->getElementType() != null)
-        {
+        if ($item->getElementType() != null) {
             $references[] = $item->getElementType();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $item,
@@ -41,14 +38,10 @@ 
         /**
          * @var INavigationTargetMapping $navTargetMapping
          */
-        foreach ($navTargetMappings as $navTargetMapping)
-        {
-            if ($navTargetMapping->getNavigationProperty() != null)
-            {
+        foreach ($navTargetMappings as $navTargetMapping) {
+            if ($navTargetMapping->getNavigationProperty() != null) {
                 $references[] = $navTargetMapping->getNavigationProperty();
-            }
-            else
-            {
+            } else {
                 InterfaceValidator::CollectErrors(
                     InterfaceValidator::CreatePropertyMustNotBeNullError(
                         $navTargetMapping,
@@ -57,12 +50,9 @@ 
                     $errors);
             }
 
-            if ($navTargetMapping->getTargetEntitySet() != null)
-            {
+            if ($navTargetMapping->getTargetEntitySet() != null) {
                 $references[] = $navTargetMapping->getTargetEntitySet();
-            }
-            else
-            {
+            } else {
                 InterfaceValidator::CollectErrors(
                     InterfaceValidator::CreatePropertyMustNotBeNullError(
                         $navTargetMapping,

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIPrimitiveTypeReference.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     protected function VisitT($typeRef, array &$followup, array &$references): iterable
     {
         assert($typeRef instanceof IPrimitiveTypeReference);
-        return $typeRef->getDefinition() != null && !$typeRef->getDefinition()->getTypeKind()->isPrimitive() ? [ InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef) ] : null;
+        return $typeRef->getDefinition() != null && !$typeRef->getDefinition()->getTypeKind()->isPrimitive() ? [InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef)] : null;
     }
 
     public function forType(): string

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIValueTermReferenceExpression.php

Braces +4 added lines, -10 removed lines

@@ -15,12 +15,9 @@ 
         assert($expression instanceof IValueTermReferenceExpression);
         $errors = null;
 
-                if ($expression->getBase() != null)
-                {
+                if ($expression->getBase() != null) {
                     $followup[] = $expression->getBase();
-                }
-                else
-                {
+                } else {
                     InterfaceValidator::CollectErrors(
                         InterfaceValidator::CreatePropertyMustNotBeNullError(
                             $expression,
@@ -30,12 +27,9 @@ 
                     );
                 }
 
-                if ($expression->getTerm() != null)
-                {
+                if ($expression->getTerm() != null) {
                     $references[] = $expression->getTerm();
-                }
-                else
-                {
+                } else {
                     InterfaceValidator::CollectErrors(
                         InterfaceValidator::CreatePropertyMustNotBeNullError(
                             $expression,

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIFunctionParameter.php

Braces +5 added lines, -12 removed lines

@@ -17,13 +17,10 @@ 
 
         $errors = null;
 
-                if ($parameter->getType() != null)
-                {
+                if ($parameter->getType() != null) {
                     // Parameter owns its type reference, so it goes as a followup.
                     $followup[] = $parameter->getType();
-                }
-                else
-                {
+                } else {
                     InterfaceValidator::CollectErrors(
                         InterfaceValidator::CreatePropertyMustNotBeNullError(
                             $parameter, "Type"
@@ -32,12 +29,9 @@ 
                     );
                 }
 
-                if ($parameter->getDeclaringFunction() != null)
-                {
+                if ($parameter->getDeclaringFunction() != null) {
                     $references[] = $parameter->getDeclaringFunction();
-                }
-                else
-                {
+                } else {
                     InterfaceValidator::CollectErrors(
                         InterfaceValidator::CreatePropertyMustNotBeNullError(
                             $parameter,
@@ -48,8 +42,7 @@ 
                 }
 
                 if ($parameter->getMode()->getValue() < FunctionParameterMode::None()->getValue() ||
-                    $parameter->getMode()->getValue() > FunctionParameterMode::InOut()->getValue())
-                {
+                    $parameter->getMode()->getValue() > FunctionParameterMode::InOut()->getValue()) {
                     InterfaceValidator::CollectErrors(
                         InterfaceValidator::CreateEnumPropertyOutOfRangeError(
                             $parameter,

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIEntityType.php

Braces +1 added lines, -2 removed lines

@@ -14,8 +14,7 @@
     {
         assert($type instanceof IEntityType);
         $errors = null;
-                if ($type->getDeclaredKey() != null)
-                {
+                if ($type->getDeclaredKey() != null) {
                     InterfaceValidator::ProcessEnumerable($type, $type->getDeclaredKey(), "DeclaredKey", $references, $errors);
                 }
 

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIEntityReferenceType.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
         }
         else
         {
-            return [ InterfaceValidator::CreatePropertyMustNotBeNullError($type, "EntityType") ];
+            return [InterfaceValidator::CreatePropertyMustNotBeNullError($type, "EntityType")];
         }
     }
 

Braces +2 added lines, -5 removed lines

@@ -13,13 +13,10 @@
     protected function VisitT($type, array &$followup, array &$references): iterable
     {
         assert($type instanceof IEntityReferenceType);
-        if ($type->getEntityType() != null)
-        {
+        if ($type->getEntityType() != null) {
             $references[] = $type->getEntityType();
             return null;
-        }
-        else
-        {
+        } else {
             return [ InterfaceValidator::CreatePropertyMustNotBeNullError($type, "EntityType") ];
         }
     }

src/Edm/Validation/Internal/InterfaceValidator/VisitorOfIStructuredTypeReference.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     protected function VisitT($typeRef, array &$followup, array &$references): iterable
     {
         assert($typeRef instanceof IStructuredTypeReference);
-        return $typeRef->getDefinition() != null && !$typeRef->getDefinition()->getTypeKind()->IsStructured() ? [ InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef) ] : null;
+        return $typeRef->getDefinition() != null && !$typeRef->getDefinition()->getTypeKind()->IsStructured() ? [InterfaceValidator::CreateTypeRefInterfaceTypeKindValueMismatchError($typeRef)] : null;
     }
 
     public function forType(): string