| @@ -311,7 +311,7 @@ | ||
| 311 | 311 | } | 
| 312 | 312 | |
| 313 | 313 | /** | 
| 314 | - * @param $context | |
| 314 | + * @param \Surfnet\StepupGateway\GatewayBundle\Saml\ResponseContext $context | |
| 315 | 315 | * @return SAML2_Response | 
| 316 | 316 | */ | 
| 317 | 317 | private function createResponseFailureResponse($context) | 
| @@ -53,6 +53,9 @@ discard block | ||
| 53 | 53 | } | 
| 54 | 54 | } | 
| 55 | 55 | |
| 56 | + /** | |
| 57 | + * @param string $provider | |
| 58 | + */ | |
| 56 | 59 | private function loadProviderConfiguration( | 
| 57 | 60 | $provider, | 
| 58 | 61 | array $configuration, | 
| @@ -223,6 +226,9 @@ discard block | ||
| 223 | 226 |          $container->setDefinition('gssp.provider.' . $provider . '.metadata.factory', $metadataFactory); | 
| 224 | 227 | } | 
| 225 | 228 | |
| 229 | + /** | |
| 230 | + * @param string $provider | |
| 231 | + */ | |
| 226 | 232 | private function createRouteConfig($provider, $routeName) | 
| 227 | 233 |      { | 
| 228 | 234 | return [ | 
| @@ -26,7 +26,7 @@ | ||
| 26 | 26 |  { | 
| 27 | 27 | /** | 
| 28 | 28 | * @param KeyHandle $keyHandle | 
| 29 | - * @return Registration|null | |
| 29 | + * @return Registration | |
| 30 | 30 | */ | 
| 31 | 31 | public function findByKeyHandle(KeyHandle $keyHandle) | 
| 32 | 32 |      { | 
| @@ -184,7 +184,7 @@ | ||
| 184 | 184 | |
| 185 | 185 | /** | 
| 186 | 186 | * @param KeyHandle $keyHandle | 
| 187 | - * @return null|Registration | |
| 187 | + * @return Registration | |
| 188 | 188 | */ | 
| 189 | 189 | public function findRegistrationByKeyHandle(KeyHandle $keyHandle) | 
| 190 | 190 |      { | 
| @@ -146,6 +146,7 @@ | ||
| 146 | 146 | /** | 
| 147 | 147 | * @param SAML2_Assertion $assertion | 
| 148 | 148 | * @param SAML2_Assertion $assertion | 
| 149 | + * @param string $authnContextClassRef | |
| 149 | 150 | */ | 
| 150 | 151 | private function addAuthenticationStatementTo(SAML2_Assertion $assertion, $authnContextClassRef) | 
| 151 | 152 |      { | 
| @@ -346,6 +346,9 @@ | ||
| 346 | 346 | $this->smsService->clearSmsVerificationState(); | 
| 347 | 347 | } | 
| 348 | 348 | |
| 349 | + /** | |
| 350 | + * @param string $identityNameId | |
| 351 | + */ | |
| 349 | 352 | private function determineInstitutionsByIdentityNameId($identityNameId) | 
| 350 | 353 |      { | 
| 351 | 354 | return $this->secondFactorRepository->getAllInstitutions($identityNameId); | 
| @@ -202,7 +202,7 @@ | ||
| 202 | 202 | } | 
| 203 | 203 | |
| 204 | 204 | /** | 
| 205 | - * @return null|IdentityProvider | |
| 205 | + * @return null|string | |
| 206 | 206 | */ | 
| 207 | 207 | public function getAuthenticatingIdpEntityId() | 
| 208 | 208 |      { | 
| @@ -79,7 +79,7 @@ discard block | ||
| 79 | 79 | } | 
| 80 | 80 | |
| 81 | 81 | /** | 
| 82 | - * @return mixed|null | |
| 82 | + * @return string | |
| 83 | 83 | */ | 
| 84 | 84 | public function getAuthMethod() | 
| 85 | 85 |      { | 
| @@ -87,7 +87,7 @@ discard block | ||
| 87 | 87 | } | 
| 88 | 88 | |
| 89 | 89 | /** | 
| 90 | - * @return mixed|null | |
| 90 | + * @return string | |
| 91 | 91 | */ | 
| 92 | 92 | public function getContext() | 
| 93 | 93 |      { | 
| @@ -110,7 +110,7 @@ discard block | ||
| 110 | 110 | |
| 111 | 111 | /** | 
| 112 | 112 | * @param string $key | 
| 113 | - * @param mixed $value Any scalar | |
| 113 | + * @param string $value Any scalar | |
| 114 | 114 | */ | 
| 115 | 115 | protected function set($key, $value) | 
| 116 | 116 |      { |