simplesamlphp / saml2

src/SAML2/Certificate/KeyLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
      * are not configured to be used with the usage given
      *
      * @param array $configuredKeys
-     * @param       $usage
+     * @param       null|string $usage
      */
     public function loadKeys(array $configuredKeys, $usage)
     {

src/SAML2/Utils.php

Doc Comments +1 added lines, -1 removed lines

@@ -248,7 +248,7 @@
      *
      * @param  DOMElement $node          The element we should fetch the attribute from.
      * @param  string     $attributeName The name of the attribute.
-     * @param  mixed      $default       The value that should be returned if the attribute doesn't exist.
+     * @param  false|null      $default       The value that should be returned if the attribute doesn't exist.
      * @return bool|mixed The value of the attribute, or $default if the attribute doesn't exist.
      * @throws Exception
      */

src/SAML2/Assertion.php

Doc Comments +7 added lines, -7 removed lines

@@ -657,7 +657,7 @@ 
     /**
      * Check whether the NameId is encrypted.
      *
-     * @return true if the NameId is encrypted, false if not.
+     * @return boolean if the NameId is encrypted, false if not.
      */
     public function isNameIdEncrypted()
     {
@@ -785,7 +785,7 @@ 
      * This function returns null if there are no restrictions on how early the
      * assertion can be used.
      *
-     * @return int|null The earliest timestamp this assertion is valid.
+     * @return integer The earliest timestamp this assertion is valid.
      */
     public function getNotBefore()
     {
@@ -812,7 +812,7 @@ 
      * This function returns null if there are no restrictions on how
      * late the assertion can be used.
      *
-     * @return int|null The latest timestamp this assertion is valid.
+     * @return integer The latest timestamp this assertion is valid.
      */
     public function getNotOnOrAfter()
     {
@@ -870,7 +870,7 @@ 
     /**
      * Retrieve the AuthnInstant of the assertion.
      *
-     * @return int|null The timestamp the user was authenticated, or NULL if the user isn't authenticated.
+     * @return integer The timestamp the user was authenticated, or NULL if the user isn't authenticated.
      */
     public function getAuthnInstant()
     {
@@ -1032,7 +1032,7 @@ 
      * See:
      * @url http://docs.oasis-open.org/security/saml/v2.0/saml-authn-context-2.0-os.pdf
      *
-     * @return \SAML2\XML\Chunk|null
+     * @return Chunk
      */
     public function getAuthnContextDecl()
     {
@@ -1062,7 +1062,7 @@ 
      *
      * The URI reference MAY directly resolve into an XML document containing the referenced declaration.
      *
-     * @return string
+     * @return Chunk
      */
     public function getAuthnContextDeclRef()
     {
@@ -1094,7 +1094,7 @@ 
     /**
      * Retrieve all attributes.
      *
-     * @return array All attributes, as an associative array.
+     * @return \DOMElement[] All attributes, as an associative array.
      */
     public function getAttributes()
     {

src/SAML2/AuthnRequest.php

Doc Comments +1 added lines, -1 removed lines

@@ -569,7 +569,7 @@
      * The returned NameId is in the format used by \SAML2\Utils::addNameId().
      *
      * @see \SAML2\Utils::addNameId()
-     * @return array|null The name identifier of the assertion.
+     * @return string The name identifier of the assertion.
      * @throws \Exception
      */
     public function getNameId()

src/SAML2/LogoutRequest.php

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
     /**
      * Check whether the NameId is encrypted.
      *
-     * @return true if the NameId is encrypted, false if not.
+     * @return boolean if the NameId is encrypted, false if not.
      */
     public function isNameIdEncrypted()
     {

src/SAML2/Response.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     /**
      * Retrieve the assertions in this response.
      *
-     * @return \SAML2\Assertion[]|\SAML2\EncryptedAssertion[]
+     * @return Utilities\ArrayCollection
      */
     public function getAssertions()
     {

src/SAML2/Response/Processor.php

Doc Comments +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
      * @param \SAML2\Configuration\Destination      $currentDestination
      * @param \SAML2\Response                       $response
      *
-     * @return \SAML2\Assertion[] Collection (\SAML2\Utilities\ArrayCollection) of \SAML2\Assertion objects
+     * @return \SAML2\Utilities\ArrayCollection Collection (\SAML2\Utilities\ArrayCollection) of \SAML2\Assertion objects
      */
     public function process(
         ServiceProvider $serviceProviderConfiguration,
@@ -134,7 +134,7 @@ 
     /**
      * @param \SAML2\Response $response
      *
-     * @return \SAML2\Assertion[]
+     * @return \SAML2\Utilities\ArrayCollection
      */
     private function processAssertions(Response $response)
     {

src/SAML2/SOAPClient.php

Doc Comments +4 added lines

@@ -212,6 +212,10 @@
      * @param $soapmessage Soap response needs to be type DOMDocument
      * @return $soapfaultstring string|null
      */
+
+    /**
+     * @param \DOMDocument $soapMessage
+     */
     private function getSOAPFault($soapMessage)
     {
         $soapFault = Utils::xpQuery($soapMessage->firstChild, '/soap-env:Envelope/soap-env:Body/soap-env:Fault');