Feeling Lucky
?
?
[#36415] - [4] Delete lots of code
- Fixed in Code Base
- 21 Jan 2022
- Medium
- Build: 4.0-dev
- # 36415
- Diff
- PhilETaylor:deleteallthings
User tests: Successful: Unsuccessful:
Code review.
Delete all the things. Make them smaller. Prettier. Modern. More Professional. Less Code = Less Bugs
- 'return' is unnecessary as the last statement in a method
- Ternary expression replaced with short version
- Unnecessary 'return' statement
- 'continue' is unnecessary as the last statement in a loop
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Labels |
Added:
?
|
||
| Category | Libraries | ⇒ | Administration com_finder com_languages Front End com_content Libraries Plugins |
| Title |
|
||||||
RickR2H
- comment
- 16 Jan 2022
I have tested this item
Nice cleanup! thanks @PhilETaylor
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36415.
| Status | Pending | ⇒ | Ready to Commit |
RickR2H
- comment
- 16 Jan 2022
RTX
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36415.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-01-21 13:03:34 |
| Closed_By | ⇒ | bembelimen | |
| Labels |
Added:
?
|
||
bembelimen
- comment
- 21 Jan 2022
Thx
I have tested this item✅ successfully on 7d30e21
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36415.