Feeling Lucky
?
?
[#17472] - [4.0] Fix for #17449
- Fixed in Code Base
- 10 Aug 2017
- Medium
- Build: 4.0-dev
- Easy Test
- # 17472
- Diff
- kavadas:4.0-dev
- Pending hound Hound is busy reviewing changes... Details
User tests: Successful: Unsuccessful:
joomla-cms-bot
- | Category | ⇒ | Modules Front End |
| Status | New | ⇒ | Pending |
| Easy | No | ⇒ | Yes |
brianteeman
- comment
- 10 Aug 2017
While this is correct I wonder why we didn't have the problem before
kavadas
- comment
- 10 Aug 2017
@brianteeman Because in 3.x the main module file ( mod_random_image.php ) had a check for empty images array. In 4.0 this check is moved inside the layout file:
laoneo
- comment
- 10 Aug 2017
Which is how it should be.
laoneo
- comment
- 10 Aug 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17472.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 10 Aug 2017
RTC after two successful tests.
brianteeman
- comment
- 10 Aug 2017
Ok thanks
infograf768
- | Title |
|
||||||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-10 21:09:35 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
wilsonge
- comment
- 10 Aug 2017
Thanks!
I have tested this item✅ successfully on 9d77b88
"No Images" is shown.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17472.