On 5/5/16, 11:38 PM, "Harbs" <harbs.li...@gmail.com> wrote:
>FWIW, I also tried >list1 += < id=“1”/>; >instead of >list1 += list4 >but that did not work either. > >I re-read the spec and it does look like you are reading it right, but it >does not make sense to me and I don’t know how that jives with the >behavior in other cases. For example (from the spec): > >> Similarly, after evaluating the statements below, >> var e = <employees> >> <employee id="1"><name>Joe</name><age>20</age></employee> >> <employee id="2"><name>Sue</name><age>30</age></employee> >> </employees>; >> // append employees 3 and 4 to the end of the employee list >> e.employee += <employee id="3"><name>Fred</name></employee> + >> <employee id="4"><name>Carol</name></employee>; >> the variable “e” would contain the XML value: >> <employees> >> <employee id="1"><name>Joe</name><age>20</age></employee> >> <employee id="2"><name>Sue</name><age>30</age></employee> >> <employee id="3"><name>Fred</name></employee> >> <employee id="4"><name>Carol</name></employee> >> </employees>; > >If e.employee loses its targetObject I don’t see how 3 and 4 get appended >to the XML. Well, the spec is checking to see if the thing appended is an XMLList and in the above example, I think you are appending XML not XMLList so that might explain the different behavior. > >I also have no idea what this comment means: >> Note, however, that the "+=" operation does not necessarily preserve >>the identity of the XML object specified by the LeftHandSideExpression. I think that means that the XML object may be replaced instead of modified. > >I do believe that my case is pretty “fringe”. I don’t expect that people >are adding objects to XMLLists and relying on the fact that it will not >get added to the original XML. > >I’m ready to call it a day and say the XML is ready for the 0.7.0 >release. I fully expect there to be more bugs (especially revolving >around namespaces which is not tested very well), but my test cases are >all working well. Way cool to hear that. Add a blurb to the RELEASE_NOTES. -Alex
