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

[enhancement] New default media source #14307

Ruslan-Aleev opened this Issue Jan 24, 2019 · 2 comments


None yet
5 participants
Copy link

Ruslan-Aleev commented Jan 24, 2019

Feature request


In MODX, the FIlesystem source is set by default, but any user can see the system files in the file viewer, which is not correct (if user access is not set).
And also, if the sources for TV are incorrectly set, then all uploaded files will go to the root folder, cluttering up the root, and here you can, by mistake, delete the system files with the uploaded ones.

I suggest creating a second media source and linking it to the /assets/ folder and set it as default. This folder /assets/ already in MODX and it is intended for non-system files. For example, it can also upload static files to the new source.

p.s. It is also worth changing the picture "No Preview Available" to an icon, since The picture is of poor quality and does not look very good, see the picture below.



This comment has been minimized.

Copy link

Ibochkarev commented Feb 12, 2019

@digitalpenguin I ask you to Express your point of view on how to better organize and set up a new media source. What more opinions.


This comment has been minimized.

Copy link

digitalpenguin commented Feb 12, 2019

Considering that MODX is installed with a single admin/sudo user, I would suggest setting up an assets media source like @Ruslan-Aleev mentioned but make it the default media source rather than the filesystem root.
At the moment IIRC new user groups automatically have access to the file system root unless manually configured otherwise. Perhaps by default new user groups could have access to the Assets media source but not the filesystem root.
( I could be wrong about that, I haven't had time to double check).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment