Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#31071] - [4] Ternary Expression can be safely replaced with condition removing codesmell

? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
11 Oct 2020

Code review

Ternary Expression can be safely replaced with condition removing code smell

avatar PhilETaylor PhilETaylor - open - 11 Oct 2020
avatar PhilETaylor PhilETaylor - change - 11 Oct 2020
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Oct 2020
Category ⇒ Administration com_associations com_finder com_menus Layout Libraries Front End Plugins
avatar ChristineWk ChristineWk - test_item - 12 Oct 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 12 Oct 2020

I have tested this item ✅ successfully on 44ac23f

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31071.

avatar Quy Quy - test_item - 12 Oct 2020 - Tested successfully
avatar Quy
Quy - comment - 12 Oct 2020

I have tested this item ✅ successfully on 44ac23f

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31071.

avatar Quy Quy - change - 12 Oct 2020
Status Pending ⇒ Ready to Commit
avatar Quy
Quy - comment - 12 Oct 2020

RTC

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31071.

avatar PhilETaylor PhilETaylor - change - 12 Oct 2020
Labels Added: ? ?
avatar PhilETaylor
PhilETaylor - comment - 12 Oct 2020

Applied suggestions

avatar HLeithner HLeithner - close - 12 Oct 2020
avatar HLeithner HLeithner - merge - 12 Oct 2020
avatar HLeithner HLeithner - change - 12 Oct 2020
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2020-10-12 17:43:08
Closed_By ⇒ HLeithner
avatar HLeithner
HLeithner - comment - 12 Oct 2020

Thanks

Add a Comment

Login with GitHub to post a comment