[MGNLDMS-126] DMS Module Makes Web Pages With "dms" At Start Break Created: 03/Jul/08 Updated: 03/Jul/14 Resolved: 05/Sep/08 |
|
| Status: | Closed |
| Project: | Document Management System (closed) |
| Component/s: | None |
| Affects Version/s: | 1.2.2 |
| Fix Version/s: | 1.2.6 |
| Type: | Bug | Priority: | Major |
| Reporter: | Sean McMains | Assignee: | Jan Haderka |
| 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: | |||||||||||||
| Description |
|
To reproduce:
Expected Behavior:
Actual Behavior:
|
| Comments |
| Comment by Magnolia International [ 03/Jul/08 ] |
|
If not related, this seems at least similar to |
| Comment by Sean McMains [ 08/Jul/08 ] |
|
Jeff, one of our programmers, has identified the issue as a missing slash in the URI 2 Repository mapping. Here's a patch that fixes the configuration and adds the bootstrap task. It appears to wholly fix the issue here. |
| Comment by Jeff Snider [ 11/Jul/08 ] |
|
We've run into other issues with our proposed fix. We're continuing to investigate the problem. Please disregard the attached patch. |
| Comment by Jan Haderka [ 03/Sep/08 ] |
|
Could you please describe what other issues you had with proposed patch? on first glance it looks fine. Thx. |
| Comment by Jeff Snider [ 04/Sep/08 ] |
|
What we found was that the URI 2 Repository mappings are actually two way, but only on some places. Changing it from "/dms" to "/dms/" worked fine when browsing through the dms in admin central, but it broke from the editor paragraphs that had internal links back to the dms. It created a link to "/dms//folder/file.doc". With the extra slash, it caused a 404. Since the problem only appears for root level pages and we create those pages for our users, we decided it was easier to just avoid it with process than programming. |
| Comment by Magnolia International [ 05/Sep/08 ] |
|
Jan, can you check this with regards to the fixes you've made for the linked issues? |
| Comment by Jan Haderka [ 05/Sep/08 ] |
|
Works fine with the other recent changes. Links constructed in the paragraph are fine as well and do not contain double slashes. |
| Comment by Magnolia International [ 12/Sep/08 ] |
|
Fixed version handler: now safely check and modify the existing URIPrefix property instead of re-bootstrapping the complete mapping |