OpenConext / Stepup-RA

src/Surfnet/StepupRa/RaBundle/Repository/VettingProcedureRepository.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
 {
     /**
      * @param VettingProcedure $vettingProcedure
+     * @return void
      */
     public function store(VettingProcedure $vettingProcedure);
 

src/Surfnet/StepupRa/RaBundle/Security/Authentication/Token/SamlToken.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * Returns the user credentials.
      *
-     * @return mixed The user credentials
+     * @return string The user credentials
      */
     public function getCredentials()
     {

src/Surfnet/StepupRa/SamlStepupProviderBundle/DependencyInjection/SurfnetStepupRaSamlStepupProviderExtension.php

Doc Comments +6 added lines

@@ -50,6 +50,9 @@ 
         }
     }
 
+    /**
+     * @param string $provider
+     */
     private function loadProviderConfiguration(
         $provider,
         array $configuration,
@@ -198,6 +201,9 @@ 
         $container->setDefinition('gssp.provider.' . $provider . '.metadata.factory', $metadataFactory);
     }
 
+    /**
+     * @param string $provider
+     */
     private function createRouteConfig($provider, $routeName)
     {
         // In the future, we ought to wrap this in an object.

src/Surfnet/StepupRa/RaBundle/Security/Authentication/AuthenticatedSessionStateHandler.php

Doc Comments +3 added lines

@@ -65,6 +65,7 @@ 
 
     /**
      * @param string $uri
+     * @return void
      */
     public function setCurrentRequestUri($uri);
 
@@ -76,6 +77,7 @@ 
     /**
      * Migrates the current session to a new session id while maintaining all
      * session attributes.
+     * @return void
      */
     public function migrate();
 
@@ -84,6 +86,7 @@ 
      *
      * Clears all session attributes and flashes and regenerates the
      * session and deletes the old session from persistence
+     * @return void
      */
     public function invalidate();
 }

src/Surfnet/StepupRa/RaBundle/Security/Authentication/Provider/SamlProvider.php

Doc Comments +3 added lines

@@ -125,6 +125,9 @@
         return $authenticatedToken;
     }
 
+    /**
+     * @param string $attribute
+     */
     private function getSingleStringValue($attribute, AssertionAdapter $translatedAssertion)
     {
         $values = $translatedAssertion->getAttributeValue($attribute);

src/Surfnet/StepupRa/RaBundle/Security/Authentication/SamlAuthenticationStateHandler.php

Doc Comments +2 added lines

@@ -27,6 +27,7 @@ 
     
     /**
      * @param string $requestId
+     * @return void
      */
     public function setRequestId($requestId);
     
@@ -37,6 +38,7 @@ 
 
     /**
      * Removes the requestId from the session
+     * @return void
      */
     public function clearRequestId();
 }

src/Surfnet/StepupRa/RaBundle/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/StepupRa/RaBundle/Service/SecondFactorService.php

Doc Comments -1 removed lines

@@ -51,7 +51,6 @@
 
     /**
      * @param string $registrationCode
-     * @param string $actorInstitution
      * @param string $actorId
      * @return null|VerifiedSecondFactor
      */

src/Surfnet/StepupRa/RaBundle/Service/VettingService.php

Doc Comments +2 added lines, -2 removed lines

@@ -435,7 +435,7 @@ 
     }
 
     /**
-     * @param $procedureId
+     * @param string $procedureId
      * @return string
      * @throws UnknownVettingProcedureException
      */
@@ -445,7 +445,7 @@ 
     }
 
     /**
-     * @param $procedureId
+     * @param string $procedureId
      * @return string
      * @throws UnknownVettingProcedureException
      */