No Code Attached Yet
avatar seaoffate-net
seaoffate-net
23 Mar 2025

Steps to reproduce the issue

I created a random image module from site modules random image. I have the image folder pointing to images/myimages/walking. in that folder i have some .jpg, some .jpeg and some .png images. in the image type box I can have one type only so png displays those files. jpg will display the jpg files and jpeg will display the jpeg all no problem. if I enter jpeg, jpg i get "no images" displayed i have tried a few variations with and without spaces with and without commas with and without "" but I can only get one type to display at any one time.

Expected result

Actual result

System information (as much as possible)

Ubuntu linux, apache2 php 8.3 joomla 5, gantry 5 and Helium

Additional comments

The obvious easy work around is to only have one type of image ext so I will just have .jpeg files and will rename the jpg to jpeg.

avatar seaoffate-net seaoffate-net - open - 23 Mar 2025
avatar joomla-cms-bot joomla-cms-bot - change - 23 Mar 2025
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Mar 2025
avatar AdarshSantoria
AdarshSantoria - comment - 23 Mar 2025

Hi @seaoffate-net,
This feature wasn't available before, but I’ve submitted a fix in PR #45200 to support multiple image extensions.
You can enter values like jpg,jpeg,png now.

avatar QuyTon QuyTon - change - 24 Mar 2025
Status New Closed
Closed_Date 0000-00-00 00:00:00 2025-03-24 05:23:28
Closed_By QuyTon
avatar QuyTon QuyTon - close - 24 Mar 2025
avatar seaoffate-net
seaoffate-net - comment - 24 Mar 2025

Hi Guys
Thankyou,
I didn't know that it wasn't supposed to accept more than one ext type. It
was never a big thing really as their is the easy work around of having the
same ext type in the directory, but i guess it all makes things slightly
better for not much cost. So thanks again

Regards Nigel

On Mon, Mar 24, 2025 at 5:23 AM Quy Ton @.***> wrote:

Closed #45196 #45196 as
completed.


Reply to this email directly, view it on GitHub
#45196 (comment), or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BQYWBDBQI42XMP2ZJEWG4N32V6JGHAVCNFSM6AAAAABZS6LXAGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJWHE2DQNZQGYZTGMY
.
You are receiving this because you were mentioned.Message ID:
@.***>

avatar fgsw
fgsw - comment - 24 Mar 2025

@seaoffate-net Please test pull request #45200.

When a pull request has two successfully tests by two users, who are not the author of the pull request, it gets "Ready to Commit" status and label. Then maintainers will review and merge if ok.

And just because something is tested the maintainers may decide, that its a change or feature that they dont want to accept.

Add a Comment

Login with GitHub to post a comment