dotCMS / core

Headless/Hybrid Content Management System for Enterprises
http://dotcms.com
Other
845 stars 465 forks source link

Internal menu links to not link relatively, but link to the full domain, and prepend HTTP instead of HTTPs #27092

Open swicken-dotcms opened 8 months ago

swicken-dotcms commented 8 months ago

Parent Issue

No response

Problem Statement

When you create an internal menu link, it generates the full href including domain, rather than relative, and pretends HTTP instead of HTTPS

Steps to Reproduce

  1. From a demo starter, create a menu link under destinations that links to blogs, check show on menu.
  2. Go to the velocity playground portlet and use this code:
    #set($list = $navtool.getNav("/"))
    #foreach($n in $list)
    #set($children = $n.children)
    #if($children && $children.size() > 0)
        #foreach($ch in $children)
            $ch
            #set($gchildren = $ch.children)
            #if($gchildren && $gchildren.size() > 0)
                #foreach($gch in $gchildren)
                    $gch
                #end
            #end
        #end
    #else
        $n
    #end
    #end
  3. You will see printed out the menu link goes to http://demo.dotcms.com/blog/index

Acceptance Criteria

  1. Internal links should be relative links e.g /blog/index

dotCMS Version

23.10.24

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

https://dotcms.zendesk.com/agent/tickets/114583

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

github-actions[bot] commented 5 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.