Correctly handling empty namespaces xmlns="".

Fixes #40

CHANGELOG.md

@@ -1,6 +1,13 @@
 ChangeLog
 =========
 
+0.4.1 (2015-03-19)
+-----------------
+
+* #40: An element with an empty namespace (xmlns="") is not allowed to have a
+  prefix. This is now fixed.
+
+
 0.4.0 (2015-03-18)
 ------------------
 

composer.json

@@ -35,6 +35,7 @@
     "bin" : [
     ],
     "require-dev": {
-        "squizlabs/php_codesniffer": "*"
+        "squizlabs/php_codesniffer": "*",
+        "phpunit/phpunit" : "*"
     }
 }

lib/Version.php

@@ -14,6 +14,6 @@ class Version {
     /**
      * Full version number
      */
-    const VERSION = '0.4.0';
+    const VERSION = '0.4.1';
 
 }

lib/Writer.php

@@ -166,10 +166,17 @@ function startElement($name) {
                 $result = $this->startElementNS($this->namespaceMap[$namespace], $localName, null);
             } else {
 
-                if (!isset($this->adhocNamespaces[$namespace])) {
-                    $this->adhocNamespaces[$namespace] = 'x' . (count($this->adhocNamespaces)+1);
+                // An empty namespace means it's the global namespace. This is
+                // allowed, but it mustn't get a prefix.
+                if ($namespace==="") {
+                    $result = $this->startElement($localName);
+                    $this->writeAttribute('xmlns', '');
+                } else {
+                    if (!isset($this->adhocNamespaces[$namespace])) {
+                        $this->adhocNamespaces[$namespace] = 'x' . (count($this->adhocNamespaces)+1);
+                    }
+                    $result = $this->startElementNS($this->adhocNamespaces[$namespace], $localName, $namespace);
                 }
-                $result = $this->startElementNS($this->adhocNamespaces[$namespace], $localName, $namespace);
             }
 
         } else {

tests/Sabre/XML/WriterTest.php

@@ -164,7 +164,7 @@ function testEmptyNamespace() {
         ], <<<HI
 <?xml version="1.0"?>
 <s:root xmlns:s="http://sabredav.org/ns">
- <x1:elem1 xmlns:x1="">bar</x1:elem1>
+ <elem1 xmlns="">bar</elem1>
 </s:root>
 
 HI