Algo-Web / ODataMetadata

src/MetadataV3/edm/EntityContainer/FunctionImportAnonymousType.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
             for ($j = $i + 1; $j < $numParms; $j++) {
                 assert(
                     $this->parameter[$j] instanceof TFunctionImportParameterType,
-                    get_class($this->parameter[$j]). ' ' . $j
+                    get_class($this->parameter[$j]) . ' ' . $j
                 );
                 $inName = $this->parameter[$j]->getName();
                 if ($outName == $inName) {

src/MetadataV3/edm/TSchemaType.php

Spacing +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
                     return false;
                 }
                 if (!in_array($navProp->getFromRole(), $roles)) {
-                    $msg = 'Navigation Property Role ' .$navProp->getToRole()
+                    $msg = 'Navigation Property Role ' . $navProp->getToRole()
                             . ' lacks a matching Property in the assocation';
                     return false;
                 }

src/MetadataV3/edm/Groups/GExpressionTrait.php

Spacing +2 added lines, -2 removed lines

@@ -687,12 +687,12 @@
                 $counter += isset($this->$name) ? 1 : 0;
             }
             if (-1 < $this->gExpressionMinimum && $counter < $this->gExpressionMinimum) {
-                $msg = $counter . ' fields not null.  Need minimum of '.$this->gExpressionMinimum. ': '
+                $msg = $counter . ' fields not null.  Need minimum of ' . $this->gExpressionMinimum . ': '
                         . get_class($this);
                 return false;
             }
             if (-1 < $this->gExpressionMaximum && $counter > $this->gExpressionMaximum) {
-                $msg = $counter . ' fields not null.  Need maximum of '.$this->gExpressionMaximum. ': '
+                $msg = $counter . ' fields not null.  Need maximum of ' . $this->gExpressionMaximum . ': '
                         . get_class($this);
                 return false;
             }

tests/TestTypeTest.php

Spacing +1 added lines, -1 removed lines

@@ -194,7 +194,7 @@
 
         $msg = null;
         $expected = 'OH NOES!';
-        $arr = [ $bar ];
+        $arr = [$bar];
 
         $foo = new testType();
         $this->assertFalse($foo->isChildArrayOK($arr, $msg), $msg);

tests/v3/edm/EntityContainer/AssociationSetAnonymousTypeTest.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 
     public function testTooManyEnds()
     {
-        $expected = 'Supplied array not a valid array: '.
+        $expected = 'Supplied array not a valid array: ' .
                     'AlgoWeb\ODataMetadata\MetadataV3\edm\EntityContainer\AssociationSetAnonymousType';
         $actual = null;
 

tests/MetadataManagerNavigationTest.php

Spacing +2 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
         $this->assertTrue($metadataManager->getEdmx()->isOK($msg), $msg);
 
         $expectedRelation = 'Data.Category_Products_Product_Category';
-        list($principalNav, ) = $metadataManager->addNavigationPropertyToEntityType(
+        list($principalNav,) = $metadataManager->addNavigationPropertyToEntityType(
             $categoryType, '*', 'Products', $productType, '1', 'Category', ['CategoryID'], ['CategoryID']
         );
         $this->assertEquals($expectedRelation, $principalNav->getRelationship());
@@ -147,7 +147,7 @@ 
         $associationType = null;
         foreach ($dom->getElementsByTagName('Association') as $eType) {
             foreach ($eType->attributes as $aType) {
-                if ($aType->name == 'Name' && 'Data.'.$aType->value == $ProductRelationship) {
+                if ($aType->name == 'Name' && 'Data.' . $aType->value == $ProductRelationship) {
                     $associationType = $eType;
                 }
             }

tests/TSchemaTypeTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
         for ($i = 0; $i < 100; $i++) {
             $actualMax = max($type->getRand(), $actualMax);
         }
-        $this->assertLessThanOrEqual($expectedMax, $actualMax, $actualMax . ' must be less than '.$expectedMax);
+        $this->assertLessThanOrEqual($expectedMax, $actualMax, $actualMax . ' must be less than ' . $expectedMax);
     }
 
     public function testGetRandMinimum()
@@ -42,6 +42,6 @@ 
         for ($i = 0; $i < 100; $i++) {
             $actualMin = min($type->getRand(), $actualMin);
         }
-        $this->assertGreaterThanOrEqual($expectedMin, $actualMin, $actualMin . ' must be less than '.$expectedMin);
+        $this->assertGreaterThanOrEqual($expectedMin, $actualMin, $actualMin . ' must be less than ' . $expectedMin);
     }
 }

tests/MetadataManagerTest.php

Spacing +21 added lines, -21 removed lines

@@ -172,7 +172,7 @@ 
         $this->assertTrue($metadataManager->getEdmx()->isOK($msg), $msg);
 
         $expectedRelation = 'Data.Category_Products_Product_Category';
-        list($principalNav, ) = $metadataManager->addNavigationPropertyToEntityType(
+        list($principalNav,) = $metadataManager->addNavigationPropertyToEntityType(
             $categoryType, '*', 'Products', $productType, '1', 'Category', ['CategoryID'], ['CategoryID']
         );
         $this->assertEquals($expectedRelation, $principalNav->getRelationship());
@@ -218,7 +218,7 @@ 
         $this->assertTrue($assoc instanceof TAssociationType);
         $this->assertTrue($assoc->isOK($msg), $msg);
 
-        $this->assertEquals('Data.'.$assoc->getName(), $principal->getRelationship());
+        $this->assertEquals('Data.' . $assoc->getName(), $principal->getRelationship());
         $ends = $assoc->getEnd();
 
         $this->assertEquals(2, count($ends));
@@ -252,7 +252,7 @@ 
         $this->assertTrue($assoc instanceof TAssociationType);
         $this->assertTrue($assoc->isOK($msg), $msg);
 
-        $this->assertEquals('Data.'.$assoc->getName(), $principal->getRelationship());
+        $this->assertEquals('Data.' . $assoc->getName(), $principal->getRelationship());
         $ends = $assoc->getEnd();
 
         $this->assertEquals(2, count($ends));
@@ -286,7 +286,7 @@ 
         $this->assertTrue($assoc instanceof TAssociationType);
         $this->assertTrue($assoc->isOK($msg), $msg);
 
-        $this->assertEquals('Data.'.$assoc->getName(), $principal->getRelationship());
+        $this->assertEquals('Data.' . $assoc->getName(), $principal->getRelationship());
         $ends = $assoc->getEnd();
 
         $this->assertEquals(2, count($ends));
@@ -320,7 +320,7 @@ 
         $this->assertTrue($assoc instanceof TAssociationType);
         $this->assertTrue($assoc->isOK($msg), $msg);
 
-        $this->assertEquals('Data.'.$assoc->getName(), $principal->getRelationship());
+        $this->assertEquals('Data.' . $assoc->getName(), $principal->getRelationship());
         $ends = $assoc->getEnd();
 
         $this->assertEquals(2, count($ends));
@@ -354,7 +354,7 @@ 
         $this->assertTrue($assoc instanceof TAssociationType);
         $this->assertTrue($assoc->isOK($msg), $msg);
 
-        $this->assertEquals('Data.'.$assoc->getName(), $principal->getRelationship());
+        $this->assertEquals('Data.' . $assoc->getName(), $principal->getRelationship());
         $ends = $assoc->getEnd();
 
         $this->assertEquals(2, count($ends));
@@ -598,7 +598,7 @@ 
     {
         list(, $metadataManager, $categoryType, $customerType) = $this->setUpMetadataForNavTests();
 
-        $expected =  'Invalid multiplicity combination - 1 1';
+        $expected = 'Invalid multiplicity combination - 1 1';
         $actual = null;
 
         try {
@@ -640,7 +640,7 @@ 
 
     public function testAddComplexType()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
 
         $name = 'Name';
         $accessType = 'Public';
@@ -652,7 +652,7 @@ 
         $result = $metadataManager->addComplexType($name, $accessType, $summary, $longDescription);
 
         $newCount = count($metadataManager->getEdmx()->getDataServiceType()->getSchema()[0]->getComplexType());
-        $this->assertEquals($oldCount+1, $newCount);
+        $this->assertEquals($oldCount + 1, $newCount);
         $this->assertNotNull($result);
         $this->assertTrue($result instanceof TComplexTypeType, get_class($result));
         $this->assertNotNull($result->getDocumentation());
@@ -660,7 +660,7 @@ 
 
     public function testAddComplexTypeWithOnlySummary()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
 
         $name = 'Name';
         $accessType = 'Public';
@@ -672,7 +672,7 @@ 
         $result = $metadataManager->addComplexType($name, $accessType, $summary, $longDescription);
 
         $newCount = count($metadataManager->getEdmx()->getDataServiceType()->getSchema()[0]->getComplexType());
-        $this->assertEquals($oldCount+1, $newCount);
+        $this->assertEquals($oldCount + 1, $newCount);
         $this->assertNotNull($result);
         $this->assertTrue($result instanceof TComplexTypeType, get_class($result));
         $this->assertNull($result->getDocumentation());
@@ -680,7 +680,7 @@ 
 
     public function testAddComplexTypeWithOnlyDescription()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
 
         $name = 'Name';
         $accessType = 'Public';
@@ -692,7 +692,7 @@ 
         $result = $metadataManager->addComplexType($name, $accessType, $summary, $longDescription);
 
         $newCount = count($metadataManager->getEdmx()->getDataServiceType()->getSchema()[0]->getComplexType());
-        $this->assertEquals($oldCount+1, $newCount);
+        $this->assertEquals($oldCount + 1, $newCount);
         $this->assertNotNull($result);
         $this->assertTrue($result instanceof TComplexTypeType, get_class($result));
         $this->assertNull($result->getDocumentation());
@@ -703,7 +703,7 @@ 
         $expected = 'Default value cannot be object or array';
         $actual = null;
 
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
         $complex = m::mock(TComplexTypeType::class);
         $name = 'name';
         $type = 'type';
@@ -722,7 +722,7 @@ 
         $expected = 'Default value cannot be object or array';
         $actual = null;
 
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
         $complex = m::mock(TComplexTypeType::class);
         $name = 'name';
         $type = 'type';
@@ -738,7 +738,7 @@ 
 
     public function testAddPropertyToComplexTypeDefaultValueBoolean()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
         $complex = m::mock(TComplexTypeType::class);
         $complex->shouldReceive('addToProperty')
             ->with(m::type(TComplexTypePropertyType::class))->andReturnNull()->once();
@@ -764,7 +764,7 @@ 
 
     public function testAddPropertyToComplexTypeDefaultValueBooleanOnlySummary()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
         $complex = m::mock(TComplexTypeType::class);
         $complex->shouldReceive('addToProperty')
             ->with(m::type(TComplexTypePropertyType::class))->andReturnNull()->once();
@@ -791,7 +791,7 @@ 
 
     public function testAddPropertyToComplexTypeDefaultValueBooleanOnlyDescription()
     {
-        list(, $metadataManager, , ) = $this->setUpMetadataForNavTests();
+        list(, $metadataManager,,) = $this->setUpMetadataForNavTests();
         $complex = m::mock(TComplexTypeType::class);
         $complex->shouldReceive('addToProperty')
             ->with(m::type(TComplexTypePropertyType::class))->andReturnNull()->once();
@@ -910,7 +910,7 @@ 
         $longDescription = new TTextType();
 
         $metadataManager = new MetadataManager();
-        list($result, ) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
+        list($result,) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
         $this->assertNotNull($result->getDocumentation());
     }
 
@@ -922,7 +922,7 @@ 
         $longDescription = null;
 
         $metadataManager = new MetadataManager();
-        list($result, ) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
+        list($result,) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
         $this->assertNull($result->getDocumentation());
     }
 
@@ -934,7 +934,7 @@ 
         $longDescription = new TTextType();
 
         $metadataManager = new MetadataManager();
-        list($result, ) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
+        list($result,) = $metadataManager->addEntityType($name, null, false, $accessType, $summary, $longDescription);
         $this->assertNull($result->getDocumentation());
     }
 

src/MetadataV3/edm/TEntityTypeType.php

Spacing +4 added lines, -4 removed lines

@@ -471,8 +471,8 @@ 
         foreach ($this->getProperty() as $prop) {
             $propName = $prop->getName();
             if (in_array($propName, $pArray)) {
-                $msg = 'Prop: Property Names, and Navigation Property Must Be Unique for '. $propName . ' on '
-                       . $this->getName() . ' '. __CLASS__;
+                $msg = 'Prop: Property Names, and Navigation Property Must Be Unique for ' . $propName . ' on '
+                       . $this->getName() . ' ' . __CLASS__;
                 return false;
             }
             $pArray[] = $propName;
@@ -480,8 +480,8 @@ 
         foreach ($this->getNavigationProperty() as $prop) {
             $propName = $prop->getName();
             if (in_array($propName, $pArray)) {
-                $msg = 'Nav: Property Names, and Navigation Property Must Be Unique for '. $propName . ' on '
-                       . $this->getName() . ' '. __CLASS__;
+                $msg = 'Nav: Property Names, and Navigation Property Must Be Unique for ' . $propName . ' on '
+                       . $this->getName() . ' ' . __CLASS__;
                 return false;
             }
             $pArray[] = $prop->getName();