-
-
Notifications
You must be signed in to change notification settings - Fork 485
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
Disallow mkdir in favor of wp_mkdir_p #1895
Comments
@jrfnl, I was looking for good first issues to contribute to this repository and I found this one. Is the idea of this issue to add I noticed that Thanks |
@rodrigoprimo Maybe you should ask the OP instead of me ? It may also be that this was fixed already without anyone noticing that this issue could be closed. |
@rodrigoprimo if it's already in Can you confirm that you get an error for code like:
If yes, this issue can be closed |
You will get a warning
So I'll close this. |
Is your feature request related to a problem?
Disallow mkdir in favor of wp_mkdir_p
To allow for extendability of the mkdir via wp_mkdir_p, to ensure the directories are automatically created with correct permissions (& also ensures that devs dont even try to "0777" work around)
The text was updated successfully, but these errors were encountered: