Feeling Lucky
?
[#7761] - Fix some strange code
- Closed
- 26 Aug 2015
- Medium
- Build: staging
- Easy Test
- # 7761
- Diff
- zero-24:fix_some_strange_code
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This PR fix some nonsense code.
The old way
$false = false;
[some checks]
return $false;The new way
[some checks]
return false;Note
I'm execluding administrator\components\com_joomlaupdate\restore.php as it is 3rd party code.
How to test
-
Access the following Backend edit views:
- com_languages --> language
- com_menus --> menu
- com_modules --> modul
- com_plguins --> plugin
- com_templates --> style
or just review that code that is maybe the easyest version ;)
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Components |
| Easy | No | ⇒ | Yes |
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 25 Aug 2015
Thanks. RTC'ing. Just fixed some merge conflicts.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7761.
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
Bakual
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-26 13:08:13 |
| Closed_By | ⇒ | Bakual |
| Labels |
Removed:
?
|
||
@test looks good to me
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7761.