Feeling Lucky
?
?
[#24390] - [3.9][JLayout images] Remove unnecessary isset(...)
- Fixed in Code Base
- 28 Mar 2019
- Medium
- Build: staging
- # 24390
- Diff
- ReLater:patch-7
User tests: Successful: Unsuccessful:
Testing Instructions
- Code review. We don't need the additional isset(...) check because !empty(...) is sufficient.
Expected result
- No changes with output of intro images in category blogs and featured blogs
- No changes with output of full images in article views
Actual result
- Unnecessary code.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Layout |
| Labels |
Added:
?
|
||
| Title |
|
||||||
SharkyKZ
- comment
- 28 Mar 2019
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24390.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 28 Mar 2019
Status "Ready To Commit".
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-03-28 13:12:49 |
| Closed_By | ⇒ | HLeithner |
HLeithner
- comment
- 28 Mar 2019
thx
I have tested this item✅ successfully on 7d9faeb
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24390.