Merge pull request #30 from data-licious/master

Some fixes

Author: tmilos

src/AerialShip/LightSaml/Model/Assertion/AuthnStatement.php

@@ -91,7 +91,7 @@ protected function prepareForXml() {
      * @return \DOMElement
      */
     function getXml(\DOMNode $parent, SerializationContext $context) {
-        $result = $context->getDocument()->createElement('AuthnStatement');
+        $result = $context->getDocument()->createElementNS(Protocol::NS_ASSERTION, 'saml:AuthnStatement');
         $parent->appendChild($result);
 
         $result->setAttribute('AuthnInstant', Helper::time2string($this->getAuthnInstant()));
@@ -134,4 +134,4 @@ function loadFromXml(\DOMElement $xml) {
     }
 
 
-}
+}

src/AerialShip/LightSaml/Model/Assertion/Subject.php

@@ -68,7 +68,7 @@ public function getSubjectConfirmations()
      */
     public function getXml(\DOMNode $parent, SerializationContext $context)
     {
-        $result = $context->getDocument()->createElement('Subject');
+        $result = $context->getDocument()->createElementNS(Protocol::NS_ASSERTION, 'saml:Subject');
         $parent->appendChild($result);
 
         if ($this->getNameID()) {
@@ -132,4 +132,4 @@ protected function loadXmlCallback($object)
         }
     }
 
-}
+}

src/AerialShip/LightSaml/Model/Assertion/SubjectConfirmation.php

@@ -92,7 +92,7 @@ protected function prepareForXml() {
     function getXml(\DOMNode $parent, SerializationContext $context) {
         $this->prepareForXml();
 
-        $result = $context->getDocument()->createElement('SubjectConfirmation');
+        $result = $context->getDocument()->createElementNS(Protocol::NS_ASSERTION, 'saml:SubjectConfirmation');
         $parent->appendChild($result);
 
         $result->setAttribute('Method', $this->getMethod());
@@ -150,4 +150,4 @@ function ($obj) {
         );
     }
 
-}
+}

src/AerialShip/LightSaml/Model/Assertion/SubjectConfirmationData.php

@@ -122,14 +122,14 @@ public function getRecipient() {
      * @return \DOMElement
      */
     function getXml(\DOMNode $parent, SerializationContext $context) {
-        $result = $context->getDocument()->createElement('SubjectConfirmationData');
+        $result = $context->getDocument()->createElementNS(Protocol::NS_ASSERTION, 'saml:SubjectConfirmationData');
         $parent->appendChild($result);
 
         if ($this->getNotBefore()) {
             $result->setAttribute('NotBefore', Helper::time2string($this->getNotBefore()));
         }
         if ($this->getNotOnOrAfter()) {
-            $result->setAttribute('NotOnOrAfter', Helper::time2string($this->getNotBefore()));
+            $result->setAttribute('NotOnOrAfter', Helper::time2string($this->getNotOnOrAfter()));
         }
 
         foreach (array('Recipient', 'InResponseTo', 'Address') as $name) {
@@ -160,4 +160,4 @@ function loadFromXml(\DOMElement $xml) {
     }
 
 
-}
+}