diff --git a/modules/markup/orgmode/orgmode.go b/modules/markup/orgmode/orgmode.go
index abc641fbe2..ac1cedff6d 100644
--- a/modules/markup/orgmode/orgmode.go
+++ b/modules/markup/orgmode/orgmode.go
@@ -133,18 +133,18 @@ type Writer struct {
Ctx *markup.RenderContext
}
-const mailto = "mailto:"
-
-func (r *Writer) resolveLink(l org.RegularLink) string {
- link := html.EscapeString(l.URL)
- if l.Protocol == "file" {
- link = link[len("file:"):]
- }
- if len(link) > 0 && !markup.IsLinkStr(link) &&
- link[0] != '#' && !strings.HasPrefix(link, mailto) {
+func (r *Writer) resolveLink(kind, link string) string {
+ link = strings.TrimPrefix(link, "file:")
+ if !strings.HasPrefix(link, "#") && // not a URL fragment
+ !markup.IsLinkStr(link) && // not an absolute URL
+ !strings.HasPrefix(link, "mailto:") {
+ if kind == "regular" {
+ // orgmode reports the link kind as "regular" for "[[ImageLink.svg][The Image Desc]]"
+ // so we need to try to guess the link kind again here
+ kind = org.RegularLink{URL: link}.Kind()
+ }
base := r.Ctx.Links.Base
- switch l.Kind() {
- case "image", "video":
+ if kind == "image" || kind == "video" {
base = r.Ctx.Links.ResolveMediaLink(r.Ctx.IsWiki)
}
link = util.URLJoin(base, link)
@@ -154,29 +154,29 @@ func (r *Writer) resolveLink(l org.RegularLink) string {
// WriteRegularLink renders images, links or videos
func (r *Writer) WriteRegularLink(l org.RegularLink) {
- link := r.resolveLink(l)
+ link := r.resolveLink(l.Kind(), l.URL)
// Inspired by https://github.com/niklasfasching/go-org/blob/6eb20dbda93cb88c3503f7508dc78cbbc639378f/org/html_writer.go#L406-L427
switch l.Kind() {
case "image":
if l.Description == nil {
- fmt.Fprintf(r, ``, link, link)
+ _, _ = fmt.Fprintf(r, `
`, link, link)
} else {
- imageSrc := r.resolveLink(l.Description[0].(org.RegularLink))
- fmt.Fprintf(r, `
`, link, imageSrc, imageSrc)
+ imageSrc := r.resolveLink(l.Kind(), org.String(l.Description...))
+ _, _ = fmt.Fprintf(r, `
`, link, imageSrc, imageSrc)
}
case "video":
if l.Description == nil {
- fmt.Fprintf(r, ``, link, link)
+ _, _ = fmt.Fprintf(r, ``, link, link)
} else {
- videoSrc := r.resolveLink(l.Description[0].(org.RegularLink))
- fmt.Fprintf(r, ``, link, videoSrc, videoSrc)
+ videoSrc := r.resolveLink(l.Kind(), org.String(l.Description...))
+ _, _ = fmt.Fprintf(r, ``, link, videoSrc, videoSrc)
}
default:
description := link
if l.Description != nil {
description = r.WriteNodesAsString(l.Description...)
}
- fmt.Fprintf(r, `%s`, link, description)
+ _, _ = fmt.Fprintf(r, `%s`, link, description)
}
}
diff --git a/modules/markup/orgmode/orgmode_test.go b/modules/markup/orgmode/orgmode_test.go
index abf5ca8fcf..95f53c9cc9 100644
--- a/modules/markup/orgmode/orgmode_test.go
+++ b/modules/markup/orgmode/orgmode_test.go
@@ -10,26 +10,21 @@ import (
"code.gitea.io/gitea/modules/git"
"code.gitea.io/gitea/modules/markup"
"code.gitea.io/gitea/modules/setting"
- "code.gitea.io/gitea/modules/util"
"github.com/stretchr/testify/assert"
)
-const (
- AppURL = "http://localhost:3000/"
- Repo = "gogits/gogs"
- AppSubURL = AppURL + Repo + "/"
-)
+const AppURL = "http://localhost:3000/"
func TestRender_StandardLinks(t *testing.T) {
setting.AppURL = AppURL
- setting.AppSubURL = AppSubURL
test := func(input, expected string) {
buffer, err := RenderString(&markup.RenderContext{
Ctx: git.DefaultContext,
Links: markup.Links{
- Base: setting.AppSubURL,
+ Base: "/relative-path",
+ BranchPath: "branch/main",
},
}, input)
assert.NoError(t, err)
@@ -38,32 +33,30 @@ func TestRender_StandardLinks(t *testing.T) {
test("[[https://google.com/]]",
`