cademichaels Posted January 21, 2014 Share Posted January 21, 2014 So here is the code: About 99% of the time it returns the XML listed under "Good Return" but almost randomly it is returning the code under "Bad Return". I echoed out all of the $ToID's and there is no weirdness (Only long integers) foreach ($xml->result->Accounts->row as $a) { $ToID = (String) $a->FL[0]; $FromData = (String) $a->FL[1]; $ToXML .= '<row no="' . $UpdateCount . '">'; $ToXML .= '<FL val="Id">' . $ToID . '</FL>'; $ToXML .= '<FL val="' . $ToField . '">' . $this->FixID($FromData) . '</FL>'; $ToXML .= "</row>"; $UpdateCount+=1; $TotalUpdateCount+=1; echo "$ToID <br>"; //For debugging } ---- Bad Return (Its missing a < after ID) <row no="162"> <FL val="Id">1058030000000072643/FL><FL val="MemberCode">Firms.338</FL> </row> ---- End bad return ---- Good Return <row no="163"> <FL val="Id">1058030000000072642</FL> <FL val="MemberCode">Firms.337</FL> </row> ---- End good return Quote Link to comment https://forums.phpfreaks.com/topic/285557-code-outputting-odd-xml/ Share on other sites More sharing options...
Solution Ch0cu3r Posted January 21, 2014 Solution Share Posted January 21, 2014 (edited) There is nothing wrong here with the XML syntax $ToXML .= '<row no="' . $UpdateCount . '">'; $ToXML .= '<FL val="Id">' . $ToID . '</FL>'; $ToXML .= '<FL val="' . $ToField . '">' . $this->FixID($FromData) . '</FL>'; $ToXML .= "</row>"; Where is $ToXML passed to next? The next process could be screwing with the XML structure. Edited January 21, 2014 by Ch0cu3r Quote Link to comment https://forums.phpfreaks.com/topic/285557-code-outputting-odd-xml/#findComment-1466060 Share on other sites More sharing options...
cademichaels Posted January 21, 2014 Author Share Posted January 21, 2014 Thanks for your help. I found the problem! (I wasn't clearing the string later in the code) Quote Link to comment https://forums.phpfreaks.com/topic/285557-code-outputting-odd-xml/#findComment-1466064 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.