[MGNLDEMO-66] Tour page only displays english Created: 19/Jun/15 Updated: 12/Mar/19 Resolved: 22/Jun/15 |
|
| Status: | Closed |
| Project: | Magnolia Demo Projects |
| Component/s: | None |
| Affects Version/s: | 0.5 |
| Fix Version/s: | 0.5 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Christopher Zimmermann | Assignee: | Christopher Zimmermann |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||||||||||||||||||
| Description |
|
The tour page does not display in german, not just the tour but the navigation and footer, and all the links on the page do not contain a locale part ("/en/" or "/de/"). Here's what happens when tracing a request to the page: /de/tours/magnolia-travels/Vietnam--Tradition-and-Today.html It then REMOVES the locale from the url and stores this "clean" url to the currentUri on the aggregationState. 2. Then RegexpVirtualURIMapping (VirtualUriFilter) runs and maps the currentUri (which at this point has no locale) so /tours/magnolia-travels/Vietnam--Tradition-and-Today.html => /travel/tour?tour=magnolia-travels/Vietnam--Tradition-and-Today 3. The mapping is configured as a forward, but still this causes the I18nContentSupportFilter to run again. This time on the currentURI: /travel/tour Finding no locale in the URI, the locale is set to the fallback locale. Ideas for fixes:
Ideas that dont work:
|
| Comments |
| Comment by Christopher Zimmermann [ 22/Jun/15 ] |
|
Fix commited: Creating linked ticket on MAIN to evaluate whether this bypass should always be in place. |