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

Wrong include path for wp-load.php #226

Merged

Conversation

rockschtar
Copy link
Contributor

Fixed include path for wp-load.php The wp-content directory can be various on WordPress installations and should not be hardcoded for determine a path. See https://codex.wordpress.org/Determining_Plugin_and_Content_Directories for more information.

@cristianraiber
Copy link
Contributor

cc @andyluak

@haynzz
Copy link

haynzz commented Mar 22, 2022

@rockschtar I filed an issue and only afterwards saw that you have a pull request that could hopefully fix my issue.
Can you double check my issue and see if your changes would address my problem?
I have also included a potentially more generic solution in Issue #232.
If it doesn't let me know if you would mind adapting your PR otherwise I would create a separate one.

@rockschtar
Copy link
Contributor Author

rockschtar commented Mar 22, 2022

@haynzz Yes, the PR fixes your problem. I had the same problem with a Composer based WordPress project. I don't know why the PR hasn't been accepted yet. It is definitely a bug.

@cristianraiber cristianraiber merged commit ca8a7a7 into nikolaystrikhar:master Mar 23, 2022
@cristianraiber
Copy link
Contributor

Will release this shortly.

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

Successfully merging this pull request may close these issues.

None yet

3 participants