lightSAML / lightSAML

src/LightSaml/Model/Protocol/StatusCode.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
     }
 
     /**
-     * @param StatusCode|null $statusCode
+     * @param StatusCode $statusCode
      *
      * @return StatusCode
      */

src/LightSaml/Model/Protocol/SamlMessage.php

Doc Comments +1 added lines, -1 removed lines

@@ -224,7 +224,7 @@
     }
 
     /**
-     * @return \LightSaml\Model\Assertion\NameID|null
+     * @return Issuer|null
      */
     public function getIssuer()
     {

src/LightSaml/Bridge/Pimple/Container/Factory/ProviderContainerProvider.php

Spacing +3 added lines, -3 removed lines

@@ -23,15 +23,15 @@
      */
     public function register(Container $pimple)
     {
-        $pimple[ProviderContainer::ATTRIBUTE_VALUE_PROVIDER] = function () {
+        $pimple[ProviderContainer::ATTRIBUTE_VALUE_PROVIDER] = function() {
             throw new LightSamlBuildException('Attribute value provider not set');
         };
 
-        $pimple[ProviderContainer::SESSION_INFO_PROVIDER] = function () {
+        $pimple[ProviderContainer::SESSION_INFO_PROVIDER] = function() {
             throw new LightSamlBuildException('Session info provider not set');
         };
 
-        $pimple[ProviderContainer::NAME_ID_PROVIDER] = function () {
+        $pimple[ProviderContainer::NAME_ID_PROVIDER] = function() {
             throw new LightSamlBuildException('Name ID provider not set');
         };
     }

src/LightSaml/Validator/Model/Xsd/XsdValidator.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         libxml_clear_errors();
         $doc = new \DOMDocument();
 
-        set_error_handler(function ($errno, $errstr, $errfile, $errline, array $errcontext) use (&$result) {
+        set_error_handler(function($errno, $errstr, $errfile, $errline, array $errcontext) use (&$result) {
             $error = new XsdError(XsdError::FATAL, $errno, $errstr, 0, 0);
             $result[] = $error;
         });

src/LightSaml/State/Request/RequestState.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
     /**
      * @param string $id
-     * @param mixed  $nonce
+     * @param string  $nonce
      */
     public function __construct($id = null, $nonce = null)
     {

src/LightSaml/State/Sso/SsoState.php

Doc Comments +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
      * @deprecated Since 1.2, to be removed in 2.0. Use getParameters() instead
      *
      * @param string $name
-     * @param mixed  $value
+     * @param integer  $value
      *
      * @return SsoState
      */
@@ -144,8 +144,8 @@ 
     }
 
     /**
-     * @param $idpEntityId
-     * @param $spEntityId
+     * @param string|null $idpEntityId
+     * @param string|null $spEntityId
      * @param $nameId
      * @param $nameIdFormat
      * @param $sessionIndex

tests/LightSaml/Tests/State/Sso/SsoStateTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -242,8 +242,8 @@
     }
 
     /**
-     * @param array $arrIdp
-     * @param array $arrSp
+     * @param string[] $arrIdp
+     * @param string[] $arrSp
      *
      * @return SsoState
      */

Spacing +2 added lines, -2 removed lines

@@ -209,7 +209,7 @@ 
     {
         $state = new SsoState();
         $state->setLocalSessionId('local-id');
-        $state->getParameters()->add(['a' => 'aaa', 'b' => 2, 'c' => [1,2,3]]);
+        $state->getParameters()->add(['a' => 'aaa', 'b' => 2, 'c' => [1, 2, 3]]);
         $state->addSsoSession($session1 = new SsoSessionState());
         $session1->setIdpEntityId($idp1 = 'idp1');
         $state->addSsoSession($session2 = new SsoSessionState());
@@ -232,7 +232,7 @@ 
         $state->addSsoSession($session2 = new SsoSessionState());
         $session2->setIdpEntityId('idp-2');
 
-        $state->modify(function (SsoSessionState $session) use ($session1) {
+        $state->modify(function(SsoSessionState $session) use ($session1) {
             return $session->getIdpEntityId() != $session1->getIdpEntityId();
         });
 

tests/LightSaml/Tests/Store/TrustOptions/CompositeTrustOptionsStoreTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|TrustOptionsStoreInterface
+     * @return TrustOptionsStoreInterface
      */
     private function getTrustOptionsStoreMock()
     {

tests/LightSaml/Tests/Model/XmlDSig/SignatureStringReaderTest.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 
     public function test_validate_returns_false_when_no_signature_set()
     {
-        $publicKey = KeyHelper::createPublicKey(X509Certificate::fromFile(__DIR__ . '/../../../../../resources/sample/Certificate/saml.crt'));
+        $publicKey = KeyHelper::createPublicKey(X509Certificate::fromFile(__DIR__.'/../../../../../resources/sample/Certificate/saml.crt'));
         $reader = new SignatureStringReader();
         $result = $reader->validate($publicKey);
         $this->assertFalse($result);
@@ -37,8 +37,8 @@ 
 
     public function test_validate_correct_signature()
     {
-        $publicKey = KeyHelper::createPublicKey(X509Certificate::fromFile(__DIR__ . '/../../../../../resources/sample/Certificate/saml.crt'));
-        $privateKey = KeyHelper::createPrivateKey(__DIR__ . '/../../../../../resources/sample/Certificate/saml.pem', '', true);
+        $publicKey = KeyHelper::createPublicKey(X509Certificate::fromFile(__DIR__.'/../../../../../resources/sample/Certificate/saml.crt'));
+        $privateKey = KeyHelper::createPrivateKey(__DIR__.'/../../../../../resources/sample/Certificate/saml.pem', '', true);
         $data = 'Some message data';
         $signature = base64_encode($privateKey->signData($data));