This is an automated email from the ASF dual-hosted git repository.

gregdove pushed a commit to branch emulation_improvements
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git

commit d2640200e744db33f21c89e5b1f2f5b6e294aacb
Author: greg-dove <[email protected]>
AuthorDate: Sun May 5 18:46:51 2024 +1200

    reverting this change until I make unit tests to verify
---
 frameworks/projects/XML/src/main/royale/XML.as | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/frameworks/projects/XML/src/main/royale/XML.as 
b/frameworks/projects/XML/src/main/royale/XML.as
index e90882e1d5..0472d5603c 100644
--- a/frameworks/projects/XML/src/main/royale/XML.as
+++ b/frameworks/projects/XML/src/main/royale/XML.as
@@ -1177,15 +1177,18 @@ package
                        if(propNum.toString() == propertyName)
                        {
                                //@todo fixme, this is wrong, it should return 
a list of that child index content, not the self reference
-                               /*if(propertyName != "0")
-                                       return null;*/
-                               if(propNum >= 0 && propNum < _children.length)
+                               //before making this change, verify with unit 
tests...
+                               /*if(propNum >= 0 && propNum < _children.length)
                                {
                                        list.append(_children[propNum]);
                                        list.targetObject = this;
-                               }
-                               /*list.append(this);
-                               list.targetObject = this;*/
+                               }*/
+                               //temp original restored
+                               if(propertyName != "0")
+                                       return null;
+                               list.append(this);
+                               list.targetObject = this;
+                               //end temp original restored
                                return list;
                        }
                        //support MultiQName for multiple use namespace 
directives:

Reply via email to