Feeling Lucky
?
[#12524] - Wrong method return type
- Fixed in Code Base
- 23 Oct 2016
- Medium
- Build: staging
- # 12524
- Diff
- demis-palma:user-menulogout
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
The method UsersControllerUser::menulogout() is declared to return "boolean", but actually it returns void. It has two exit points, and on both it does not return any value.
The "boolean" declaration has been clearly copied and pasted by another method.
Summary of Changes
Changed return type declaration to void.
Testing Instructions
Just merge on review.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Front End Components |
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-10-23 15:31:36 |
| Closed_By | ⇒ | zero-24 |
zero-24
- comment
- 23 Oct 2016
Thanks
I have tested this item✅ successfully on d3106a3
on code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12524.