Merge pull request #8 from anthonygore/Fix-xml-transport-decorator

christiaan · christiaan · commit 5ce0ab3e087e · 2016-01-05T14:34:28.000+01:00
Fixing bug which causes empty element when ampersand is used
diff --git a/Transport/XmlDataTransportDecorator.php b/Transport/XmlDataTransportDecorator.php
@@ -67,7 +67,8 @@ private function encodeRecords(array $records)
                         $value = $value->format('Y-m-d H:i:s');
                     }
                 }
-                $keyValue = $row->addChild('FL', $value);
+                $keyValue = $row->addChild('FL');
+                $keyValue[0] = $value;
                 $keyValue->addAttribute('val', $key);
             }
         }
@@ -201,4 +202,4 @@ private function parseResponsePostRecordsMultiple($xml)
 
         return $records;
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,8 @@ private function encodeRecords(array $records)`
`67`	`67`	`$value = $value->format('Y-m-d H:i:s');`
`68`	`68`	`}`
`69`	`69`	`}`
`70`		`- $keyValue = $row->addChild('FL', $value);`
	`70`	`+ $keyValue = $row->addChild('FL');`
	`71`	`+ $keyValue[0] = $value;`
`71`	`72`	`$keyValue->addAttribute('val', $key);`
`72`	`73`	`}`
`73`	`74`	`}`
`@@ -201,4 +202,4 @@ private function parseResponsePostRecordsMultiple($xml)`
`201`	`202`
`202`	`203`	`return $records;`
`203`	`204`	`}`
`204`		`-}`
	`205`	`+}`