[MGNLETK-59] site mapping: don't ignore the uri prefix when trying to match the path Created: 30/Jun/11 Updated: 23/Aug/11 Resolved: 23/Aug/11 |
|
| Status: | Closed |
| Project: | Extended Templating Kit (closed) |
| Component/s: | multisite |
| Affects Version/s: | 1.4.2 |
| Fix Version/s: | 1.4.3 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Philipp Bärfuss | Assignee: | Philipp Bärfuss |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||
| Issue Links: |
|
||||||||||||||||
| Template: |
|
||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Visible to: |
Raphael Joss
|
||||||||||||||||
| Description |
|
Otherwise all mappings which have an empty path (or /) will map. |
| Comments |
| Comment by Philipp Bärfuss [ 30/Jun/11 ] |
|
Attached a drafted fix which we implemented while debugging. |
| Comment by Stefan Baur [ 30/Jun/11 ] |
|
thank you very much for the patch. But as we are using git, meight it be possible to give us the two .class (or even .java??) files? Thanks for the very appreciated help! |
| Comment by Stefan Baur [ 01/Jul/11 ] |
|
hi again. i now have 1) checked out etk I get the error message: "(...) seems to be outdated (...)" can you give me the java or class file instead? |
| Comment by Stefan Baur [ 01/Jul/11 ] |
|
I have to increase the importance here (I hate that...) but this solution should get into the baloise release today to meet the go live deadline of the baloise website. |
| Comment by Christian Ringele [ 01/Jul/11 ] |
|
Patched java classes for SBauer |
| Comment by Christian Ringele [ 01/Jul/11 ] |
|
For Stefan Bauer |
| Comment by Stefan Baur [ 01/Jul/11 ] |
|
very nice, the first test seems to work nicely! |
| Comment by Stefan Baur [ 01/Jul/11 ] |
|
don't I need to add a dms mapping to the site def at all? |
| Comment by Philipp Bärfuss [ 17/Aug/11 ] |
|
the failing tests for the ETKSiteManager. the tests for the MultisiteFilter (url cleanup) is still missing. |
| Comment by Philipp Bärfuss [ 17/Aug/11 ] |
|
Now that |
| Comment by Philipp Bärfuss [ 23/Aug/11 ] |
|
Will finally be solved by a new algorithm ( |
| Comment by Philipp Bärfuss [ 23/Aug/11 ] |
|
fixed by implementing |
| Comment by Jan Haderka [ 23/Aug/11 ] |
|
Pls next time mention all the tickets fixed by the commit in the commit message. |