[MGNLETK-23] Selectors not considered in mapping i18nURI to content Created: 20/Apr/10 Updated: 03/Jul/14 Resolved: 22/Jul/10 |
|
| Status: | Closed |
| Project: | Extended Templating Kit (closed) |
| Component/s: | None |
| Affects Version/s: | 1.2.3 |
| Fix Version/s: | 1.3.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Matteo Pelucco | Assignee: | Jan Haderka |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | etk, i18n, selector, selectors | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Patch included: |
Yes
|
| Acceptance criteria: |
Empty
|
| Date of First Response: |
| Description |
|
With i18n enabled (it is also present in default i18n support), an URL with selectors inside, still valid in Magnolia 4.1 and 4.2, all links of a page accessed with a selector in URL are "null". e.g: URL like "/magnoliaAuthor/test.html" are OK You can try even on a demo-project website. Object involved: Patch provided. |
| Comments |
| Comment by Jan Haderka [ 21/Apr/10 ] |
|
Hey Matteo, And also the patch hunk doesn't match that code. The method signature is different and removed code doesn't exist. |
| Comment by Jan Haderka [ 21/Apr/10 ] |
|
Actually the killer of selectors in 4.3 was MultiSiteFilter and this issue was fixed in trunk by changes for |
| Comment by Jan Haderka [ 21/Apr/10 ] |
|
Applied modified patch to 1.2 branch ... there were other methods where the same workaround had to be applied to make it all work properly. |
| Comment by Jan Haderka [ 21/Apr/10 ] |
|
Remaining part in current trunk is the authoring support - currently the urls generated by the language switch remove selectors. |
| Comment by Matteo Pelucco [ 21/Apr/10 ] |
|
Hi Jan, just see your comment. Matteo |