Revision: 2184
Author: olavmrk
Date: Tue Feb 16 04:22:03 2010
Log: SAMLBuilder: Move ArtifactResolutionService to the correct position.

According to the schema, the ArtifactResolutionService element(s)
should be located before the SingleLogoutService element(s).

Patch by Danny Bollaert <[email protected]>.
http://code.google.com/p/simplesamlphp/source/detail?r=2184

Modified:
 /trunk/lib/SimpleSAML/Metadata/SAMLBuilder.php

=======================================
--- /trunk/lib/SimpleSAML/Metadata/SAMLBuilder.php      Fri Feb 12 00:35:08 2010
+++ /trunk/lib/SimpleSAML/Metadata/SAMLBuilder.php      Tue Feb 16 04:22:03 2010
@@ -420,6 +420,10 @@

                $this->addCertificate($e, $metadata);

+               if ($metadata->hasValue('ArtifactResolutionService')){
+ $this->addEndpoints($e, 'ArtifactResolutionService', $metadata->getEndpoints('ArtifactResolutionService'));
+               }
+
$this->addEndpoints($e, 'SingleLogoutService', $metadata->getEndpoints('SingleLogoutService'));

                if ($metadata->hasValue('NameIDFormat')) {
@@ -427,10 +431,6 @@
$t->appendChild($this->document->createTextNode($metadata->getString('NameIDFormat')));
                        $e->appendChild($t);
                }
-
-               if ($metadata->hasValue('ArtifactResolutionService')){
- $this->addEndpoints($e, 'ArtifactResolutionService', $metadata->getEndpoints('ArtifactResolutionService'));
-               }

$this->addEndpoints($e, 'SingleSignOnService', $metadata->getEndpoints('SingleSignOnService'));

--
You received this message because you are subscribed to the Google Groups 
"simpleSAMLphp commits" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/simplesamlphp-commits?hl=en.

Reply via email to