Rename node.attributes to attribs to prevent name clash with DOM property with the same name.

Author: oozcitak

src/XMLDocumentCB.coffee

@@ -87,7 +87,7 @@ module.exports = class XMLDocumentCB
       when NodeType.Comment then @comment node.value
       when NodeType.Element 
         attributes = {}
-        for own attName, att of node.attributes
+        for own attName, att of node.attribs
           attributes[attName] = att.value
         @node node.name, attributes
       when NodeType.Dummy   then @dummy()
@@ -186,9 +186,9 @@ module.exports = class XMLDocumentCB
     else
       value = value.apply() if isFunction value
       if @options.keepNullAttributes and not value?
-        @currentNode.attributes[name] = new XMLAttribute @, name, ""
+        @currentNode.attribs[name] = new XMLAttribute @, name, ""
       else if value?
-        @currentNode.attributes[name] = new XMLAttribute @, name, value
+        @currentNode.attribs[name] = new XMLAttribute @, name, value
 
     return @
 
@@ -425,7 +425,7 @@ module.exports = class XMLDocumentCB
         chunk = @writer.indent(node, @writerOptions, @currentLevel) + '<' + node.name
 
         # attributes
-        for own name, att of node.attributes
+        for own name, att of node.attribs
           chunk += @writer.attribute att,  @writerOptions, @currentLevel
 
         chunk += (if node.children then '>' else '/>') + @writer.endline(node,  @writerOptions, @currentLevel)

src/XMLElement.coffee

@@ -21,7 +21,7 @@ module.exports = class XMLElement extends XMLNode
 
     @name = @stringify.name name
     @type = NodeType.Element
-    @attributes = {}
+    @attribs = {}
 
     @attribute attributes if attributes?
 
@@ -50,9 +50,9 @@ module.exports = class XMLElement extends XMLNode
       clonedSelf.documentObject = null
 
     # clone attributes
-    clonedSelf.attributes = {}
-    for own attName, att of @attributes
-      clonedSelf.attributes[attName] = att.clone()
+    clonedSelf.attribs = {}
+    for own attName, att of @attribs
+      clonedSelf.attribs[attName] = att.clone()
 
     # clone child nodes
     clonedSelf.children = []
@@ -77,9 +77,9 @@ module.exports = class XMLElement extends XMLNode
     else
       value = value.apply() if isFunction value
       if @options.keepNullAttributes and not value?
-        @attributes[name] = new XMLAttribute @, name, ""
+        @attribs[name] = new XMLAttribute @, name, ""
       else if value?
-        @attributes[name] = new XMLAttribute @, name, value
+        @attribs[name] = new XMLAttribute @, name, value
 
     return @
 
@@ -96,9 +96,9 @@ module.exports = class XMLElement extends XMLNode
 
     if Array.isArray name # expand if array
       for attName in name
-        delete @attributes[attName]
+        delete @attribs[attName]
     else
-      delete @attributes[name]
+      delete @attribs[name]
 
     return @
 

src/XMLNode.coffee

@@ -54,7 +54,7 @@ module.exports = class XMLNode
     Object.defineProperty @, 'nextSibling', get: () ->
         i = @parent.children.indexOf @
         @parent.children[i + 1] or null
-    #Object.defineProperty @, 'attributes', get: () -> @attributes # NamedNodeMap ???
+    #Object.defineProperty @, 'attributes', get: () -> @attribs # NamedNodeMap ???
     Object.defineProperty @, 'ownerDocument', get: () -> @document()
 
   # Creates a child element node

src/XMLStreamWriter.coffee

@@ -87,7 +87,7 @@ module.exports = class XMLStreamWriter extends XMLWriterBase
     @stream.write @indent(node, options, level) + '<' + node.name
 
     # attributes
-    for own name, att of node.attributes
+    for own name, att of node.attribs
       @attribute att, options, level
 
     childNodeCount = node.children.length

src/XMLWriterBase.coffee

@@ -186,7 +186,7 @@ module.exports = class XMLWriterBase
     r += @indent(node, options, level) + '<' + node.name
 
     # attributes
-    for own name, att of node.attributes
+    for own name, att of node.attribs
       r += @attribute att, options, level
 
     childNodeCount = node.children.length

test/basic/tostring.coffee

@@ -2,7 +2,7 @@ suite 'Test toString() function with built-in XML writer:', ->
   test 'Nodes', ->
     eq xml('root').doc().toString(), '<?xml version="1.0"?><root/>'
     eq xml('root').toString(), '<root/>'
-    eq xml('root').att('att', 'val').attributes['att'].toString(), ' att="val"'
+    eq xml('root').att('att', 'val').attribs['att'].toString(), ' att="val"'
     eq xml('root').dat('val').children[0].toString(), '<![CDATA[val]]>'
     eq xml('root').com('val').children[0].toString(), '<!-- val -->'
     eq xml('root').ins('target', 'val').children[0].toString(), '<?target val?>'