Handle self-closing xml elements in XmlFragment.

Fixes #41

Author: evert

CHANGELOG.md

@@ -1,15 +1,16 @@
 ChangeLog
 =========
 
-0.4.2 (2015-??-??)
------------------
+0.4.2 (2015-03-20)
+------------------
 
 * Removed `constants.php` again. They messed with PHPUnit and don't really
   provide a great benefit.
+* #41: Correctly handle self-closing xml elements.
 
 
 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.

lib/Element/XmlFragment.php

@@ -96,6 +96,9 @@ function xmlSerialize(Writer $writer) {
                                 break;
                         }
                     }
+                    if ($reader->isEmptyElement) {
+                        $writer->endElement();
+                    }
                     break;
                 case Reader::CDATA :
                 case Reader::TEXT :

tests/Sabre/XML/Element/XmlFragmentTest.php

@@ -93,6 +93,11 @@ function xmlProvider() {
                 '<foo:element xmlns:foo="http://example.org/ns" xmlns="http://sabredav.org/ns"><child>hello</child></foo:element>',
                 '<x1:element xmlns:x1="http://example.org/ns"><child>hello</child></x1:element>',
             ],
+            [
+                '<foo:element xmlns:foo="http://example.org/ns"><child/></foo:element>',
+                '<foo:element xmlns:foo="http://example.org/ns" xmlns="http://sabredav.org/ns"><child/></foo:element>',
+                '<x1:element xmlns:x1="http://example.org/ns"><child/></x1:element>',
+            ],
         ];
 
     }