Completed
Pull Request — develop (#114)
by A.
04:56 queued 02:24
created
DependencyInjection/SurfnetStepupSelfServiceSamlStepupProviderExtension.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -50,6 +50,9 @@  discard block
 block discarded – undo
50 50
         }
51 51
     }
52 52
 
53
+    /**
54
+     * @param string $provider
55
+     */
53 56
     private function loadProviderConfiguration(
54 57
         $provider,
55 58
         array $configuration,
@@ -198,6 +201,9 @@  discard block
 block discarded – undo
198 201
         $container->setDefinition('gssp.provider.' . $provider . '.metadata.factory', $metadataFactory);
199 202
     }
200 203
 
204
+    /**
205
+     * @param string $provider
206
+     */
201 207
     private function createRouteConfig($provider, $routeName)
202 208
     {
203 209
         // In the future, we ought to wrap this in an object.
Please login to merge, or discard this patch.
SelfServiceBundle/Security/Authentication/Token/SamlToken.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
     /**
38 38
      * Returns the user credentials.
39 39
      *
40
-     * @return mixed The user credentials
40
+     * @return string The user credentials
41 41
      */
42 42
     public function getCredentials()
43 43
     {
Please login to merge, or discard this patch.
Surfnet/StepupSelfService/SelfServiceBundle/Service/SecondFactorService.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -128,6 +128,11 @@
 block discarded – undo
128 128
                $vettedSecondFactors->getTotalItems() > 0;
129 129
     }
130 130
 
131
+    /**
132
+     * @param string $identityId
133
+     * @param string $state
134
+     * @param string $secondFactorId
135
+     */
131 136
     public function identityHasSecondFactorOfStateWithId($identityId, $state, $secondFactorId)
132 137
     {
133 138
         switch ($state) {
Please login to merge, or discard this patch.
SelfServiceBundle/Service/U2fSecondFactor/ProofOfPossessionResult.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -58,6 +58,9 @@
 block discarded – undo
58 58
         return new self;
59 59
     }
60 60
 
61
+    /**
62
+     * @param string $secondFactorId
63
+     */
61 64
     public static function secondFactorCreated($secondFactorId, RegistrationVerificationResult $u2fResult)
62 65
     {
63 66
         $result = new self;
Please login to merge, or discard this patch.
Security/Authentication/AuthenticatedSessionStateHandler.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -65,6 +65,7 @@  discard block
 block discarded – undo
65 65
 
66 66
     /**
67 67
      * @param string $uri
68
+     * @return void
68 69
      */
69 70
     public function setCurrentRequestUri($uri);
70 71
 
@@ -76,6 +77,7 @@  discard block
 block discarded – undo
76 77
     /**
77 78
      * Migrates the current session to a new session id while maintaining all
78 79
      * session attributes.
80
+     * @return void
79 81
      */
80 82
     public function migrate();
81 83
 
@@ -84,6 +86,7 @@  discard block
 block discarded – undo
84 86
      *
85 87
      * Clears all session attributes and flashes and regenerates the
86 88
      * session and deletes the old session from persistence
89
+     * @return void
87 90
      */
88 91
     public function invalidate();
89 92
 }
Please login to merge, or discard this patch.
Security/Authentication/SamlAuthenticationStateHandler.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -27,6 +27,7 @@  discard block
 block discarded – undo
27 27
 
28 28
     /**
29 29
      * @param string $requestId
30
+     * @return void
30 31
      */
31 32
     public function setRequestId($requestId);
32 33
 
@@ -37,6 +38,7 @@  discard block
 block discarded – undo
37 38
 
38 39
     /**
39 40
      * Removes the requestId from the session
41
+     * @return void
40 42
      */
41 43
     public function clearRequestId();
42 44
 }
Please login to merge, or discard this patch.
StepupSelfService/SelfServiceBundle/Exception/AssertionFailedException.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -27,6 +27,11 @@
 block discarded – undo
27 27
     private $constraints;
28 28
 
29 29
     // @codingStandardsIgnoreStart Compliance with beberlei/assert's invalid argument exception
30
+
31
+    /**
32
+     * @param string $message
33
+     * @param integer $code
34
+     */
30 35
     public function __construct($message, $code, $propertyPath = null, $value, array $constraints = array())
31 36
     {
32 37
         parent::__construct($message, $code);
Please login to merge, or discard this patch.