Feeling Lucky
?
[#8258] - Missing return argument
- Closed
- 3 Nov 2015
- Medium
- Build: staging
- # 8258
- Diff
- demis-palma:patch-5
- Failure The Travis CI build failed Details
User tests: Successful: Unsuccessful:
The method is declared to return JDocumentHTML, but contains a void return statement.
Currently the return value is ignored everywhere, therefore this patch won't have any side effects and can be merged safely.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-03 17:25:45 |
| Closed_By | ⇒ | demis-palma |
The automatic tests are not consistent with the documentation. They expect a null return when the data is an empty array.