Skip to content

Commit 41ab93e

Browse files
authored
Merge pull request #9 from dotkernel/develop
take authorization result message into account when doing a PRG redir…
2 parents 2bfe330 + 10b4531 commit 41ab93e

2 files changed

Lines changed: 23 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,21 @@
1+
## 0.2.4 - 2017-07-06
2+
3+
### Changed
4+
* on authentication error, take the authentication result message into consideration too
5+
6+
### Added
7+
* Nothing
8+
9+
### Deprecated
10+
* Nothing
11+
12+
### Removed
13+
* Nothing
14+
15+
### Fixed
16+
* Nothing
17+
18+
119
## 0.2.3 - 2017-03-21
220

321
### Changed

src/Action/LoginAction.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,8 +155,11 @@ public function process(ServerRequestInterface $request, DelegateInterface $dele
155155
return $this->prgRedirect($error);
156156
}
157157
} else {
158-
$message = $this->options->getMessagesOptions()
159-
->getMessage(Utils::$authResultCodeToMessageMap[$result->getCode()]);
158+
$message = $result->getMessage();
159+
if (empty($message)) {
160+
$message = $this->options->getMessagesOptions()
161+
->getMessage(Utils::$authResultCodeToMessageMap[$result->getCode()]);
162+
}
160163
$params += [
161164
'error' => $message
162165
];

0 commit comments

Comments
 (0)