c-harris / ODataMetadata

src/Edm/Validation/ValidationRules/IRecordExpression/RecordExpressionPropertiesMatchType.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
             !$context->checkIsBad($expression->getDeclaredType()))
         {
             $discoveredErrors = null;
-            ExpressionTypeChecker::TryAssertRecordAsType($expression, $expression->getDeclaredType(),  null, false, $discoveredErrors);
+            ExpressionTypeChecker::TryAssertRecordAsType($expression, $expression->getDeclaredType(), null, false, $discoveredErrors);
             foreach ($discoveredErrors as $error)
             {
                 $context->AddRawError($error);

Braces +2 added lines, -4 removed lines

@@ -23,12 +23,10 @@
         if (
             $expression->getDeclaredType() != null &&
             !$context->checkIsBad($expression) &&
-            !$context->checkIsBad($expression->getDeclaredType()))
-        {
+            !$context->checkIsBad($expression->getDeclaredType())) {
             $discoveredErrors = null;
             ExpressionTypeChecker::TryAssertRecordAsType($expression, $expression->getDeclaredType(),  null, false, $discoveredErrors);
-            foreach ($discoveredErrors as $error)
-            {
+            foreach ($discoveredErrors as $error) {
                 $context->AddRawError($error);
             }
         }

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetNavigationPropertyMappingMustPointToValidTargetForProperty.php

Braces +2 added lines, -4 removed lines

@@ -21,8 +21,7 @@ 
     public function __invoke(ValidationContext $context, ?IEdmElement $set)
     {
         assert($set instanceof IEntitySet);
-        foreach ($set->getNavigationTargets() as $mapping)
-        {
+        foreach ($set->getNavigationTargets() as $mapping) {
             if (
                 !(
                     $mapping->getTargetEntitySet()->getElementType()->IsOrInheritsFrom(
@@ -31,8 +30,7 @@ 
                     $mapping->getNavigationProperty()->ToEntityType()->IsOrInheritsFrom(
                         $mapping->getTargetEntitySet()->getElementType())
                 ) &&
-                !$context->checkIsBad($mapping->getTargetEntitySet()))
-            {
+                !$context->checkIsBad($mapping->getTargetEntitySet())) {
                 $context->AddError(
                     $set->Location(),
                     EdmErrorCode::EntitySetNavigationPropertyMappingMustPointToValidTargetForProperty(),

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetRecursiveNavigationPropertyMappingsMustPointBackToSourceEntitySet.php

Braces +2 added lines, -4 removed lines

@@ -22,13 +22,11 @@
     public function __invoke(ValidationContext $context, ?IEdmElement $set)
     {
         assert($set instanceof IEntitySet);
-        foreach ($set->getNavigationTargets() as $mapping)
-        {
+        foreach ($set->getNavigationTargets() as $mapping) {
             if (
                 $mapping->getNavigationProperty()->containsTarget() &&
                 $mapping->getNavigationProperty()->getDeclaringType()->IsOrInheritsFrom($mapping->getNavigationProperty()->ToEntityType()) &&
-                $mapping->getTargetEntitySet() !== $set)
-            {
+                $mapping->getTargetEntitySet() !== $set) {
                 $context->AddError(
                     $set->Location(),
                     EdmErrorCode::EntitySetRecursiveNavigationPropertyMappingsMustPointBackToSourceEntitySet(),

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetAssociationSetNameMustBeValid.php

Braces +2 added lines, -4 removed lines

@@ -19,10 +19,8 @@
     public function __invoke(ValidationContext $context, ?IEdmElement $set)
     {
         assert($set instanceof IEntitySet);
-        foreach ($set->getNavigationTargets() as $mapping)
-        {
-            if ($mapping->getNavigationProperty()->GetPrimary() === $mapping->getNavigationProperty())
-            {
+        foreach ($set->getNavigationTargets() as $mapping) {
+            if ($mapping->getNavigationProperty()->GetPrimary() === $mapping->getNavigationProperty()) {
                 Helpers::CheckForNameError(
                     $context,
                     $context->getModel()->GetAssociationSetName(

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetInaccessibleEntityType.php

Braces +1 added lines, -2 removed lines

@@ -19,8 +19,7 @@
     public function __invoke(ValidationContext $context, ?IEdmElement $entitySet)
     {
         assert($entitySet instanceof IEntitySet);
-        if (!$context->checkIsBad($entitySet->getElementType()))
-        {
+        if (!$context->checkIsBad($entitySet->getElementType())) {
             Helpers::CheckForUnreachableTypeError($context, $entitySet->getElementType(), $entitySet->Location());
         }
     }

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetNavigationMappingMustBeBidirectional.php

Braces +2 added lines, -4 removed lines

@@ -22,8 +22,7 @@ 
     public function __invoke(ValidationContext $context, ?IEdmElement $set)
     {
         assert($set instanceof IEntitySet);
-        foreach ($set->getNavigationTargets() as $mapping)
-        {
+        foreach ($set->getNavigationTargets() as $mapping) {
             $property = $mapping->getNavigationProperty();
 
             $opposingTarget = $mapping->getTargetEntitySet()->findNavigationTarget($property->getPartner());
@@ -34,8 +33,7 @@ 
                     $opposingTarget != null ||
                     $property->getPartner()->DeclaringEntityType()->findProperty($property->getPartner()->getName()) === $property->getPartner()
                 ) &&
-                $opposingTarget !== $set)
-            {
+                $opposingTarget !== $set) {
                 $context->AddError(
                     $set->Location(),
                     EdmErrorCode::EntitySetNavigationMappingMustBeBidirectional(),

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetCanOnlyBeContainedByASingleNavigationProperty.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $containmentFound = false;
         foreach ($set->getContainer()->EntitySets() as $otherSet)
         {
-            foreach ( $otherSet->getNavigationTargets() as $mapping)
+            foreach ($otherSet->getNavigationTargets() as $mapping)
             {
                 $property = $mapping->getNavigationProperty();
 

Braces +4 added lines, -8 removed lines

@@ -21,16 +21,12 @@
     {
         assert($set instanceof IEntitySet);
         $containmentFound = false;
-        foreach ($set->getContainer()->EntitySets() as $otherSet)
-        {
-            foreach ( $otherSet->getNavigationTargets() as $mapping)
-            {
+        foreach ($set->getContainer()->EntitySets() as $otherSet) {
+            foreach ( $otherSet->getNavigationTargets() as $mapping) {
                 $property = $mapping->getNavigationProperty();
 
-                if ($mapping->getTargetEntitySet() === $set && $property->containsTarget())
-                {
-                    if ($containmentFound)
-                    {
+                if ($mapping->getTargetEntitySet() === $set && $property->containsTarget()) {
+                    if ($containmentFound) {
                         $context->AddError(
                             $set->Location(),
                             EdmErrorCode::EntitySetCanOnlyBeContainedByASingleNavigationProperty(),

src/Edm/Validation/ValidationRules/IEntitySet/EntitySetTypeHasNoKeys.php

Braces +1 added lines, -2 removed lines

@@ -26,8 +26,7 @@
                 $entitySet->getElementType()->Key() === null ||
                 count($entitySet->getElementType()->Key()) !== 0
             ) &&
-            !$context->checkIsBad($entitySet->getElementType()))
-        {
+            !$context->checkIsBad($entitySet->getElementType())) {
 
             $context->AddError(
                 $entitySet->Location(),

src/Edm/Validation/ValidationRules/IEdmElement/ElementDirectValueAnnotationFullNameMustBeUnique.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
                     $annotation->Location(),
                     EdmErrorCode::DuplicateDirectValueAnnotationFullName(),
                     StringConst::EdmModel_Validator_Semantic_ElementDirectValueAnnotationFullNameMustBeUnique($annotation->getNamespaceUri(), $annotation->getName()));
-            }else{
+            } else {
                 $annotationNameSet[] = $annotation->getNamespaceUri() . ':' . $annotation->getName();
             }
         }

Braces +3 added lines, -5 removed lines

@@ -20,16 +20,14 @@
     public function __invoke(ValidationContext $context, ?IEdmElement $item)
     {
         $annotationNameSet = [];
-        foreach ($context->getModel()->getDirectValueAnnotationsManager()->getDirectValueAnnotations($item) as $annotation)
-        {
+        foreach ($context->getModel()->getDirectValueAnnotationsManager()->getDirectValueAnnotations($item) as $annotation) {
             assert($annotation instanceof IDirectValueAnnotation);
-            if (in_array($annotation->getNamespaceUri() . ':' . $annotation->getName(), $annotationNameSet))
-            {
+            if (in_array($annotation->getNamespaceUri() . ':' . $annotation->getName(), $annotationNameSet)) {
                 $context->AddError(
                     $annotation->Location(),
                     EdmErrorCode::DuplicateDirectValueAnnotationFullName(),
                     StringConst::EdmModel_Validator_Semantic_ElementDirectValueAnnotationFullNameMustBeUnique($annotation->getNamespaceUri(), $annotation->getName()));
-            }else{
+            } else {
                 $annotationNameSet[] = $annotation->getNamespaceUri() . ':' . $annotation->getName();
             }
         }