[#45200] - [6.0] Enable support for multiple image extensions in Random Image module
- Pending
- Medium
- Build: 5.3-dev
- # 45200
- Diff
- AdarshSantoria:RandomImage
User tests: Successful: 0 Unsuccessful: 0
Pull Request for Issue #45196
Summary of Changes
This pull request enhances the mod_random_image module by allowing the use of multiple image file extensions in the Image Type parameter. Previously, the module only supported a single extension like jpg. With this change, users can input comma-separated values (e.g., jpg,jpeg,png) to support multiple file types simultaneously.
Testing Instructions
- Prepare a folder with images of various formats such as
.jpg,.jpeg,.png, and.webp. - Create or edit a Random Image module and set the image folder to the one prepared above.
- In the Image Type field, enter values like
jpg,jpeg,png, or even uppercase likePNG. - Save the module and check the frontend display.
- Images with all the listed extensions should now appear randomly.
Actual result BEFORE applying this Pull Request
Only images with the exact extension specified (e.g., only jpg) were displayed.
Entering multiple extensions resulted in "No images" being shown.
Expected result AFTER applying this Pull Request
The module correctly loads and displays images of any extension listed in the comma-separated input (e.g., jpg,jpeg,png).
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Front End |
| Labels |
Added:
PR-5.3-dev
|
||
I like the idea behind it. As Joomla! 5.3 is in feature freeze mode, I move it to 5.4, so it can be tested there.
| Title |
|
||||||
| Labels |
Added:
Feature
PR-5.4-dev
|
||
I like the idea behind it. As Joomla! 5.3 is in feature freeze mode, I move it to 5.4, so it can be tested there.
shouldnt that be 6.0? When @HLeithner did the big rebase the other week all unmerged features were moved to 6
| Title |
|
||||||
| Category | Modules Front End | ⇒ | Administration com_templates com_users JavaScript Repository NPM Change External Library Composer Change Installation Libraries Modules Front End Plugins Unit Tests |
Rebased on 6.0
| Labels |
Added:
Unit/System Tests
Composer Dependency Changed
NPM Resource Changed
PR-6.0-dev
Removed: PR-5.3-dev |
||
Hi @seaoffate-net, Can you please test it.
Hi @seaoffate-net, Can you please test it.