Completed
Pull Request — master (#89)
by Christopher
18:09 queued 12:31
created
src/MetadataV3/edm/EntityContainer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     public function setName($name)
83 83
     {
84 84
         if (!$this->isTSimpleIdentifierValid($name)) {
85
-            $msg = "Name(" . $name . ") must be a valid TSimpleIdentifier";
85
+            $msg = "Name(".$name.") must be a valid TSimpleIdentifier";
86 86
             throw new \InvalidArgumentException($msg);
87 87
         }
88 88
         $this->name = $name;
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
     public function isOK(&$msg = null)
398 398
     {
399 399
         if (!$this->isTSimpleIdentifierValid($this->name)) {
400
-            $msg = "Name(" . $this->name . ") must be a valid TSimpleIdentifier " . __FILE__ . ":" . __LINE__;
400
+            $msg = "Name(".$this->name.") must be a valid TSimpleIdentifier ".__FILE__.":".__LINE__;
401 401
             return false;
402 402
         }
403 403
 
@@ -450,7 +450,7 @@  discard block
 block discarded – undo
450 450
         }
451 451
         foreach ($this->associationSet as $assocationSet) {
452 452
             if (!in_array($assocationSet->getEnd()[0]->getEntitySet(), $entityNames)) {
453
-                $msg = "The entitysets for assocations must have a valid entity set. " . $assocationSet->getName() . " Does not";
453
+                $msg = "The entitysets for assocations must have a valid entity set. ".$assocationSet->getName()." Does not";
454 454
                 return false;
455 455
             }
456 456
         }
Please login to merge, or discard this patch.
src/MetadataV3/edm/TSchemaType.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                 die($this->getNamespace());
146 146
                 return false;
147 147
             }*/
148
-            $eSetType = str_replace($this->getNamespace() . ".", "", $eSetType);
148
+            $eSetType = str_replace($this->getNamespace().".", "", $eSetType);
149 149
             if (!in_array($eSetType, $entityTypeNames)) {
150 150
                 $msg = "entitySet Types should have a matching type name in entity Types";
151 151
                 return false;
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
 
155 155
         // Check Associations to assocationSets
156 156
         if (count($assocationSets) != count($associationNames)) {
157
-            $msg = "we have " . count($assocationSets) . "assocation sets and " . count($associationNames) . " Assocations, they should be the same";
157
+            $msg = "we have ".count($assocationSets)."assocation sets and ".count($associationNames)." Assocations, they should be the same";
158 158
         }
159 159
         if (count($associationNames) * 2 < count($navigationProperties)) {
160 160
             $msg = "we have two many navigation propertys. should have no more then double the number of assocations.";
@@ -162,20 +162,20 @@  discard block
 block discarded – undo
162 162
 
163 163
         foreach ($associationNames as $assocationName => $assocationEnds) {
164 164
             if (!array_key_exists($assocationName, $assocationSets)) {
165
-                $msg = "assocation " . $assocationName . " exists without matching assocationSet";
165
+                $msg = "assocation ".$assocationName." exists without matching assocationSet";
166 166
                 return false;
167 167
             }
168 168
 
169 169
             if (!array_key_exists($assocationName, $navigationProperties)) {
170
-                $msg = "assocation " . $assocationName . " exists without matching Natvigation Property";
170
+                $msg = "assocation ".$assocationName." exists without matching Natvigation Property";
171 171
                 return false;
172 172
             }
173 173
             if (!in_array($assocationSets[$assocationName][0]->getRole(), [$assocationEnds[0]->getRole(), $assocationEnds[1]->getRole()])) {
174
-                $msg = "assocation Set role " . $assocationSets[$assocationName][0]->getRole() . "lacks a matching property in the attached assocation";
174
+                $msg = "assocation Set role ".$assocationSets[$assocationName][0]->getRole()."lacks a matching property in the attached assocation";
175 175
                 return false;
176 176
             }
177 177
             if (!in_array($assocationSets[$assocationName][1]->getRole(), [$assocationEnds[0]->getRole(), $assocationEnds[1]->getRole()])) {
178
-                $msg = "assocation Set role " . $assocationSets[$assocationName][0]->getRole() . "lacks a matching property in the attached assocation";
178
+                $msg = "assocation Set role ".$assocationSets[$assocationName][0]->getRole()."lacks a matching property in the attached assocation";
179 179
                 return false;
180 180
             }
181 181
         }
Please login to merge, or discard this patch.