[MAGNOLIA-3340] Make info.magnolia.link.CompleteUrlPathTransformer.prefixLink(String) fit for all variants of slash usage Created: 27/Oct/10 Updated: 19/Dec/16 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 4.3.7, 4.3.8 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major |
| Reporter: | Felix Rabe | Assignee: | Philipp Bärfuss |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | link | ||
| Remaining Estimate: | 10m | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | 10m | ||
| Attachments: |
|
| Template: |
|
| Patch included: |
Yes
|
| 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)
|
| Date of First Response: |
| Description |
|
Current code only checks the case where both base ends with "/" and linkStr starts with "/", and treats all other combinations (including when both are false) as safe to concatenate - which it is not. |
| Comments |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |