c-harris / ODataMetadata

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

Braces +6 added lines, -15 removed lines

@@ -15,12 +15,9 @@ 
         assert($expression instanceof IIfExpression);
         $errors = null;
 
-        if ($expression->getTestExpression() != null)
-        {
+        if ($expression->getTestExpression() != null) {
             $followup[] = $expression->getTestExpression();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $expression,
@@ -30,12 +27,9 @@ 
             );
         }
 
-        if ($expression->getTrueExpression() != null)
-        {
+        if ($expression->getTrueExpression() != null) {
             $followup[] = $expression->getTrueExpression();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $expression,
@@ -45,12 +39,9 @@ 
             );
         }
 
-        if ($expression->getFalseExpression() != null)
-        {
+        if ($expression->getFalseExpression() != null) {
             $followup[] = $expression->getFalseExpression();
-        }
-        else
-        {
+        } else {
             InterfaceValidator::CollectErrors(
                 InterfaceValidator::CreatePropertyMustNotBeNullError(
                     $expression, "FalseExpression"

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

Braces +2 added lines, -4 removed lines

@@ -42,10 +42,8 @@
         assert($expression instanceof IExpression);
 // Trying to reduce amount of noise in errors - if this expression is bad, then most likely it will have an unacceptable kind, no need to report it.
         $expressionKindError = null;
-        if (!InterfaceValidator::IsCheckableBad($expression))
-        {
-            switch ($expression->getExpressionKind())
-            {
+        if (!InterfaceValidator::IsCheckableBad($expression)) {
+            switch ($expression->getExpressionKind()) {
                 case ExpressionKind::IntegerConstant():
                     $expressionKindError = InterfaceValidator::CheckForInterfaceKindValueMismatchError($expression, $expression->getExpressionKind(), "ExpressionKind", IIntegerConstantExpression::class);
                     break;

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/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

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

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 (

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

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();
         }