Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

Move better-wp-config.php to wp-includes? #16

Open
markkap opened this issue Apr 15, 2019 · 1 comment
Open

Move better-wp-config.php to wp-includes? #16

markkap opened this issue Apr 15, 2019 · 1 comment

Comments

@markkap
Copy link

markkap commented Apr 15, 2019

Seems like a good convention is that the only files in the root directory are files that are intended to be accessed from the outside. As better-wp-config.php is not intended to be accessed from the outside, wp-includes might be a better place for it.

@markkap
Copy link
Author

markkap commented Apr 15, 2019

hmmm, I see it might be a problem for WordPress installs, but for calmPress and classicPress that is probably the best way to go. So this brings back the need to be able to "tell" the code where the configuration is located. My suggestion is to move the line
wp_config()->load_config();
to wp_config.php and pass the directory in which the configuration is located as a parameter

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant