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

302 redirect on default language #309

Closed
herrvigg opened this issue Jun 23, 2018 · 1 comment
Closed

302 redirect on default language #309

herrvigg opened this issue Jun 23, 2018 · 1 comment
Labels
legacy issue Legacy issue imported from original repo

Comments

@herrvigg
Copy link
Collaborator

Issue by ghost
Wednesday Nov 11, 2015 at 18:09 GMT
Originally opened as qTranslate-Team/qtranslate-x#309


Hi!

I believe that I have found a bug.
The "qtranxf_init_language" method is performing a 302 redirect when I select the option: "Hide URL language information for default language.".

However, it would not be correct to add 301 as a parameter in wp_redirect?
Example: wp_redirect($target, 301) - line 81;

I say this for the sake of SEO.

http://example.com/default_lang/page.php ---> (302) http://example.com/page.php

Correct:

http://example.com/default_lang/page.php ---> (301) http://example.com/page.php

Can you could understand?

@herrvigg herrvigg added the legacy issue Legacy issue imported from original repo label Jun 23, 2018
@herrvigg
Copy link
Collaborator Author

Comment by johnclause
Thursday Nov 12, 2015 at 03:42 GMT


Is this about one of these:

https://qtranslatexteam.wordpress.com/2015/02/26/browser-redirection-based-on-language/

qTranslate-Team/qtranslate-x#306

The latter is a problem if you use caching plugin, the first one is "by design". Which one is your problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
legacy issue Legacy issue imported from original repo
Projects
None yet
Development

No branches or pull requests

1 participant