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

site_url() Should be home_url() in Some Cases #293

Closed
jaswrks opened this issue Aug 10, 2014 · 1 comment
Closed

site_url() Should be home_url() in Some Cases #293

jaswrks opened this issue Aug 10, 2014 · 1 comment

Comments

@jaswrks
Copy link
Contributor

jaswrks commented Aug 10, 2014

Throughout s2Member's source code there are references to site_url() when really this should be home_url() for the best compatibility with sites that run WP from a sub-directory. Much progress on this issue was made awhile back, but there are still a few references that use site_url(). We should take a closer look at the codebase overall and correct this once and for all.

Here is one example where it should really be home_url() to improve compatibility.

@jaswrks jaswrks changed the title site_url() should be home_url() in Some Cases site_url() Should be home_url() in Some Cases Aug 10, 2014
@jaswrks jaswrks added this to the Future Release milestone Aug 14, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 22, 2014
@jaswrks jaswrks self-assigned this Aug 22, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 22, 2014
jaswrks pushed a commit that referenced this issue Aug 22, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue Aug 22, 2014
jaswrks pushed a commit that referenced this issue Aug 22, 2014
jaswrks pushed a commit that referenced this issue Aug 22, 2014
This was referenced Aug 22, 2014
@jaswrks jaswrks closed this as completed Aug 22, 2014
@jaswrks
Copy link
Contributor Author

jaswrks commented Aug 22, 2014

Next release changelog:

  • Several instances of site_url() (a WordPress core function) have been converted to home_url() instead. This provides better compatibility with WordPress installations running from a sub-directory. Please see this GitHub issue if you'd like further details.

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

1 participant