c-harris / ODataMetadata

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

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
     protected function VisitT($functionImport, array &$followup, array &$references): iterable
     {
         assert($functionImport instanceof IFunctionImport);
-        if ($functionImport->getEntitySet() != null)
-        {
+        if ($functionImport->getEntitySet() != null) {
             $followup[] = $functionImport->getEntitySet();
         }
 

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

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     {
         assert($function instanceof IFunction);
         // No need to return the return type as followup - it is handled as such in the IFunctionBase visitor.
-        return ($function->getParameters() === null) ? [ InterfaceValidator::CreatePropertyMustNotBeNullError($function, "ReturnType") ] : null;
+        return ($function->getParameters() === null) ? [InterfaceValidator::CreatePropertyMustNotBeNullError($function, "ReturnType")] : null;
     }
 
     public function forType(): string

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

Spacing +1 added lines, -1 removed lines

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

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

Braces +4 added lines, -10 removed lines

@@ -15,12 +15,9 @@ 
         assert($binding instanceof IPropertyValueBinding);
         $errors = null;
 
-        if ($binding->getValue() != null)
-        {
+        if ($binding->getValue() != null) {
             $followup[] = $binding->getValue();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $binding,
@@ -29,12 +26,9 @@ 
             );
         }
 
-        if ($binding->getBoundProperty() != null)
-        {
+        if ($binding->getBoundProperty() != null) {
             $references[] = $binding->getBoundProperty();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $binding,

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

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         }
         else
         {
-            return [ InterfaceValidator::CreatePropertyMustNotBeNullError($expression, "ReferencedFunction") ];
+            return [InterfaceValidator::CreatePropertyMustNotBeNullError($expression, "ReferencedFunction")];
         }
     }
 

Braces +2 added lines, -5 removed lines

@@ -15,16 +15,13 @@
     protected function VisitT($expression, array &$followup, array &$references): iterable
     {
         assert($expression instanceof IFunctionReferenceExpression);
-        if ($expression->getReferencedFunction() != null)
-        {
+        if ($expression->getReferencedFunction() != null) {
             assert($expression->getReferencedFunction() instanceof ISchemaElement ||
                 $expression instanceof IEntityContainerElement,
                 "Return as followup if the referenced object is not a schema function or a function import.");
             $references[] = $expression->getReferencedFunction();
             return null;
-        }
-        else
-        {
+        } else {
             return [ InterfaceValidator::CreatePropertyMustNotBeNullError($expression, "ReferencedFunction") ];
         }
     }

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

Spacing +14 added lines, -14 removed lines

@@ -39,7 +39,7 @@ 
         {
             case ValueKind::Binary():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -51,7 +51,7 @@ 
 
             case ValueKind::Boolean():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -63,7 +63,7 @@ 
 
             case ValueKind::Collection():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -75,7 +75,7 @@ 
 
             case ValueKind::DateTime():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -87,7 +87,7 @@ 
 
             case ValueKind::DateTimeOffset():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -99,7 +99,7 @@ 
 
             case ValueKind::Decimal():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -111,7 +111,7 @@ 
 
             case ValueKind::Enum():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -123,7 +123,7 @@ 
 
             case ValueKind::Floating():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -135,7 +135,7 @@ 
 
             case ValueKind::Guid():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -147,7 +147,7 @@ 
 
             case ValueKind::Integer():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -159,7 +159,7 @@ 
 
             case ValueKind::Null():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -171,7 +171,7 @@ 
 
             case ValueKind::String():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -183,7 +183,7 @@ 
 
             case ValueKind::Structured():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",
@@ -195,7 +195,7 @@ 
 
             case ValueKind::Time():
                 InterfaceValidator::CollectErrors(
-                    InterfaceValidator::CheckForInterfaceKindValueMismatchError (
+                    InterfaceValidator::CheckForInterfaceKindValueMismatchError(
                         $value,
                         $value->getValueKind(),
                         "ValueKind",

Braces +2 added lines, -4 removed lines

@@ -29,14 +29,12 @@
     {
         assert($value instanceof IValue);
         $errors = null;
-        if ($value->getType() != null)
-        {
+        if ($value->getType() != null) {
             // Value owns its type reference, so it goes as a followup.
             $followup[] = $value->getType();
         }
 
-        switch ($value->getValueKind())
-        {
+        switch ($value->getValueKind()) {
             case ValueKind::Binary():
                 InterfaceValidator::CollectErrors(
                     InterfaceValidator::CheckForInterfaceKindValueMismatchError (

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
                 );
                 break;
 
-            case ValueKind::Null():
+            case ValueKind::null():
                 InterfaceValidator::CollectErrors(
                     InterfaceValidator::CheckForInterfaceKindValueMismatchError (
                         $value,

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

Spacing +1 added lines, -1 removed lines

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

Braces +2 added lines, -5 removed lines

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

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

Braces +2 added lines, -4 removed lines

@@ -15,8 +15,7 @@ 
         assert($annotation instanceof IDirectValueAnnotation);
         $errors = null;
 
-        if ($annotation->getNamespaceUri() == null)
-        {
+        if ($annotation->getNamespaceUri() == null) {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $annotation,
@@ -26,8 +25,7 @@ 
             );
         }
 
-        if ($annotation->getValue() == null)
-        {
+        if ($annotation->getValue() == null) {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $annotation,

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

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
 
         InterfaceValidator::ProcessEnumerable($expression, $expression->getProperties(), "Properties", $followup, $errors);
 
-        if ($expression->getDeclaredType() != null)
-        {
+        if ($expression->getDeclaredType() != null) {
             // Record constructor owns its type reference, so it goes as a followup.
             $followup[] = $expression->getDeclaredType();
         }