View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005775 | OXID eShop (all versions) | 4.06. Language and translations | public | 2014-05-26 09:57 | 2015-07-08 09:44 |
Reporter | mboesing | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | 4.8.5 / 5.1.5 | ||||
Target Version | 4.8.9 / 5.1.9 | Fixed in Version | 4.9.5 / 5.2.5 | ||
Summary | 0005775: DefaultLanguage collabs language selection in frontend | ||||
Description | oxLang::processUrl terminates language selection logic. DefaultLanguage will get deleted from created URL so non-SEO URL will created without the "lang=" parameter. If your browser language differs from the default shop language, you can only visit the link in your default language. Thats not smart. | ||||
Steps To Reproduce | 1. Set your browser to "english" and your default shop language to "german". 2. Delete all cookies and set your browser to "disable cookies" (however) 3. Visit your shop (yup, its english now) 4. Change language to the default shop language (german) 5. Browse through the shop | ||||
Additional Information | - SEO-Links work fine since they relate to languages - Non-SEO Links will be missing the "lang=" parameter for default language since it gets removed by "oxLang::processUrl" Since my browser has cookies disabled, it wont store the language cookie tho and the Non-SEO Link Page will get displayed in my browsers language. To fix this, you should always add this language parameter if the default language differs from the customers browser. | ||||
Tags | Languages, Session | ||||
Theme | Azure | ||||
Browser | All | ||||
PHP Version | Not defined | ||||
Database Version | Not defined | ||||
|
I've created a Pull Request on github since this bug is open for about 4 months and nothing happened. https://github.com/OXID-eSales/oxideshop_ce/pull/163 |
|
Fixed with pull request: https://github.com/OXID-eSales/oxideshop_ce/pull/163 |