291927 - XML.prototype.replace does not work as expected

Status: RESOLVED FIXED

(Rhino Graveyard :: E4X, defect)

Description

[Michael Lipp]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.7) Gecko/20050414 Firefox/1.0.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.7) Gecko/20050414 Firefox/1.0.3

The replace method does not work as expected, see below.

Reproducible: Always

Steps to Reproduce:
Run this:

var root = <root>text</root>;

print (root.toXMLString());
print (root.child(0).toXMLString());
root.replace (0, "new text");
print (root.toXMLString());

Actual Results:  
<root>text</root>
text
<root/>


Expected Results:  
<root>text</root>
text
<root>new text</root>

Comment 1

[Igor Bukanov]

Reassigning to [e-mail] pending resolution of bug 288433

Comment 2

[David Caldwell]

This bug is fixed in the upcoming non-XMLBeans implementation of Rhino E4X; see bug 355677.

Comment 3

[David Caldwell]

Attachment: proposed testcase

Comment 4

[David Caldwell]

The results under Rhino without XMLBeans now match the submitter's expected results.  See bug 355677.

Comment 5

[Bob Clary [:bc] (inactive)]

Comment on attachment 254033 [details]
proposed testcase

ditto the issues in bug 288027. Since you are working on tests which are identifiable with definite sections in the spec, I think they should be named accordingly... e4x/XML/13.4.4.32-01.js. I prefer using 01,02, etc just for sorting sake in case we ever have more than 9 subtests.

David, again, if the changes to the license block, etc. are ok with you, I'll go ahead and check in the modified version. Thanks for taking the time to do this. I _really_ do appreciate it.

Comment 6

[David Caldwell]

Attachment: Fixed license and filename

Comment 7

[Bob Clary [:bc] (inactive)]

Comment on attachment 256636 [details]
Fixed license and filename

+ with Michael added as contributor. Sorry I have missed reviewing this test. I am checking it in now.

Comment 8

[Bob Clary [:bc] (inactive)]

/cvsroot/mozilla/js/tests/e4x/XML/13.4.4.32-01.js,v  <--  13.4.4.32-01.js
initial revision: 1.1

Comment 9

[Norris Boyd]

Adding target milestone of 1.6R6 based on the date this bug was resolved FIXED.