[#19043] - Exception from captcha checkAnswer
- Closed
- 1 Apr 2020
- Medium
- Build: staging
- # 19043
- Diff
- saharin88:patch-6
- Success hound No violations found. Woof! Details
- Success continuous-integration/drone/pr the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
User tests: Successful: Unsuccessful:
Captcha plugins can not return error information. Therefore, it would be logical to throw an exception.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
@franz-wohlkoenig what is there to test, if we just catch the exception?
if you do not catch an exception, it will return a 404 page
thanks for Info @saharin88, so nothing to test.
@saharin88 there should be a way to test this changes,
otherwise there a very small chance that this will be accepted
please explain what user should do to see difference, before and after the patch, eg. how to get 404 page etc.
Once again, now there is no way to pass the error text when checking the CAPTCHA, if using the standard validation rule.
And if you throw an exception, without this edit, it will return 404 pages.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-04-01 19:46:31 |
| Closed_By | ⇒ | wilsonge |
@saharin88 can you please write Test Instructions?