Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

com_media Change Image modal paths #30662

Open
heleneross opened this issue Sep 16, 2020 · 3 comments
Open

com_media Change Image modal paths #30662

heleneross opened this issue Sep 16, 2020 · 3 comments

Comments

@heleneross
Copy link

heleneross commented Sep 16, 2020

Steps to reproduce the issue

Create a folder in the root of your site and add some images.
In the filesystem plugin add this in addition to the images directory.
Open an article for editing and select an intro image from this new folder. Save the article.
Click on the select button to change the intro image.
The Change Image modal is confused - it shows a non-existent folder under images called the same as the new folder you created and a an error message:
Error
File or Folder not found.

Expected result

The modal should show the folders as they exist, and the modal should open by showing the current image.

Actual result

Error message and incorrect path and folder.

System information (as much as possible)

Joomla! 4.0.0-beta3-dev
Apache/2.4.34 (Unix) LibreSSL/2.5.5 PHP/7.3.8

Additional comments

the images should make this issue clear
screen shot 2020-09-16 at 14 09 36screen shot 2020-09-16 at 14 09 36

@heleneross
Copy link
Author

screen shot 2020-09-16 at 14 13 43


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

@richard67
Copy link
Member

The filesystem plugin is one thing, the media field and media manager are other things. As far as I know, the media manager and media (or images?) field (which is used to get the images modal) are restricted to the "images" folder and folders below it.

Adding folders to the filesystem plugin may be good for other purpose, but it might be intended that these folders can't be used for images.

Someone who has more knowledge than me about how it is intended to work should confirm or correct me.

But it seems that if it is intended, then the issue is that we need better handling of that. The folder should not be available in this case for media/images, and there should not be shown a not existing sub-folder with that folder's name.

@infograf768
Copy link
Member

See also
#29721

@Hackwar Hackwar added the bug label Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants