Feeling Lucky
[#809] - Code review
- Fixed in Code Base
- 25 Jun 2016
- Medium
- Build: master
- # 809
- Diff
- joomla:code-review
- Success continuous-integration/travis-ci/push The Travis CI build passed Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
So there are several unused use statements which my first class IDE was unable to spot :(
I was testing the SensioLabs thingy which gave me the following result: https://insight.sensiolabs.com/projects/1a93532b-bc3d-4b14-bc94-1189581b42a7/analyses/4
I think there are several things worth looking at e.g. Boolean should be compared strictly (70) - they say that this will improve performance..
Good thing is that it will only take 9.7 weeks to fix all "errors" - so they say
mbabker
- | Status | New | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-25 13:47:59 |
| Closed_By | ⇒ | mbabker |