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

[4.0] Non default path's causes problems using the media manager #34358

Closed
pjdevries opened this issue Jun 2, 2021 · 5 comments
Closed

[4.0] Non default path's causes problems using the media manager #34358

pjdevries opened this issue Jun 2, 2021 · 5 comments

Comments

@pjdevries
Copy link
Contributor

Steps to reproduce the issue

Change both path's of the Media Manager configration, i.e. 'Path to Files Folder' and 'Path to Images Folder'.

Open Media Manager.

Expected result

A list of files in folder 'Path to Files Folder', if there are any, or an empty folder otherwise.

Actual result

The result depends on proper configuration of the FileSystem - Local plugin. If no directory is chosen, which is the default, the plugin returns [{"directory": "images"}] and the Media Manager shows nothing.

System information (as much as possible)

A fresh Joomla! 4.0 installation, or at least with unaltered FileSystem - Local plugin configuration.

Additional comments

My suggestion is to return Media Manager's 'Path to Files Folder' setting from FileSystem - Local as a default and not [{"directory": "images"}].

@richard67
Copy link
Member

@pjdevries Will that be solved, too, with your PR #34314 ? If so, this issue can be closed.

@pjdevries
Copy link
Contributor Author

@richard67 Unfortunately this will not be solved with PR #34314. It's a minor fix though, which I don't mind doing if that's o.k.

@richard67
Copy link
Member

... which I don't mind doing if that's o.k.

@pjdevries You mean you wanna fix it with another PR? That would be very o.k.

@pjdevries
Copy link
Contributor Author

@richard67 Yeah, that's what I meant. I'll do it right away.

@alikon
Copy link
Contributor

alikon commented Jun 2, 2021

please test #34364

@alikon alikon closed this as completed Jun 2, 2021
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

4 participants