OpenConext / Stepup-Gateway

src/Surfnet/StepupGateway/SamlStepupProviderBundle/DependencyInjection/SurfnetStepupGatewaySamlStepupProviderExtension.php

Doc Comments +6 added lines

@@ -53,6 +53,9 @@ 
         }
     }
 
+    /**
+     * @param string $provider
+     */
     private function loadProviderConfiguration(
         $provider,
         array $configuration,
@@ -223,6 +226,9 @@ 
         $container->setDefinition('gssp.provider.' . $provider . '.metadata.factory', $metadataFactory);
     }
 
+    /**
+     * @param string $provider
+     */
     private function createRouteConfig($provider, $routeName)
     {
         return [

src/Surfnet/StepupGateway/U2fVerificationBundle/Repository/RegistrationRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 {
     /**
      * @param KeyHandle $keyHandle
-     * @return Registration|null
+     * @return Registration
      */
     public function findByKeyHandle(KeyHandle $keyHandle)
     {

src/Surfnet/StepupGateway/U2fVerificationBundle/Service/VerificationService.php

Doc Comments +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
 
     /**
      * @param KeyHandle $keyHandle
-     * @return null|Registration
+     * @return Registration
      */
     public function findRegistrationByKeyHandle(KeyHandle $keyHandle)
     {

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Adfs/StateHandler.php

Doc Comments +3 added lines, -3 removed lines

@@ -79,7 +79,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return string
      */
     public function getAuthMethod()
     {
@@ -87,7 +87,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return string
      */
     public function getContext()
     {
@@ -110,7 +110,7 @@ 
 
     /**
      * @param string $key
-     * @param mixed $value Any scalar
+     * @param string $value Any scalar
      */
     protected function set($key, $value)
     {

src/Surfnet/StepupGateway/GatewayBundle/Controller/GatewayController.php

Doc Comments +1 added lines, -1 removed lines

@@ -331,7 +331,7 @@
     }
 
     /**
-     * @param $context
+     * @param \Surfnet\StepupGateway\GatewayBundle\Saml\ResponseContext $context
      * @return SAMLResponse
      */
     private function createResponseFailureResponse($context)

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Saml/ResponseFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 
     /**
      * @param Assertion $assertion
-     * @param $authnContextClassRef
+     * @param string $authnContextClassRef
      */
     private function addAuthenticationStatementTo(Assertion $assertion, $authnContextClassRef)
     {

src/Surfnet/StepupGateway/GatewayBundle/Exception/AssertionFailedException.php

Doc Comments +5 added lines

@@ -27,6 +27,11 @@
     private $constraints;
 
     // @codingStandardsIgnoreStart Compliance with beberlei/assert's invalid argument exception
+
+    /**
+     * @param string $message
+     * @param integer $code
+     */
     public function __construct($message, $code, $propertyPath = null, $value, array $constraints = array())
     {
         parent::__construct($message, $code);

src/Surfnet/StepupGateway/GatewayBundle/Entity/ServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
      * provider. This methods checks if a given URL matches the allowed URLs.
      *
      * @param LoggerInterface $logger
-     * @param $acsLocationInAuthnRequest
+     * @param string|null $acsLocationInAuthnRequest
      * @return string
      */
     public function determineAcsLocation(LoggerInterface $logger, $acsLocationInAuthnRequest)
@@ -126,7 +126,7 @@ 
      *
      *
      * @param LoggerInterface $logger
-     * @param $acsLocationInAuthnRequest
+     * @param string|null $acsLocationInAuthnRequest
      * @return string
      */
     public function determineAcsLocationforAdfs(LoggerInterface $logger, $acsLocationInAuthnRequest)