Fix issue link in issues page

The link assumes that the issue belongs to the signed in
user, which is not necessarily the case
I changed it to use repository owner name instead

diff --git a/routers/user/home.go b/routers/user/home.go
index 0a805903ec91c346bc98b4469631830af12493ee..b19f3ff4aca7793996ce703b95e0cd6ad898af56 100644 (file)
--- a/routers/user/home.go
+++ b/routers/user/home.go
@@ -235,6 +235,12 @@ func Issues(ctx *middleware.Context) {
                        return
                }
 
+               err = issues[i].Repo.GetOwner()
+               if err != nil {
+                       ctx.Handle(500, "user.Issues(GetOwner)", err)
+                       return
+               }
+
                issues[i].Poster, err = models.GetUserById(issues[i].PosterId)
                if err != nil {
                        ctx.Handle(500, "user.Issues(GetUserById)", err)

diff --git a/templates/issue/user.tmpl b/templates/issue/user.tmpl
index f480c63a8f6bd1a4c5696c3f90a59111f7ac3615..d1c2bd9941845f1bd4bd39bf3ae31eb5b59ee528 100644 (file)
--- a/templates/issue/user.tmpl
+++ b/templates/issue/user.tmpl
@@ -36,7 +36,7 @@
                 {{range .Issues}}{{if .}}
                 <div class="list-group-item issue-item" id="issue-{{.Id}}">
                     <span class="number pull-right">#{{.Index}}</span>
-                    <h5 class="title"><a href="/{{$.SignedUser.Name}}/{{.Repo.Name}}/issues/{{.Index}}">{{.Name}}</a></h5>
+                    <h5 class="title"><a href="/{{.Repo.Owner.Name}}/{{.Repo.Name}}/issues/{{.Index}}">{{.Name}}</a></h5>
                     <p class="info">
                         <span class="author"><img class="avatar" src="{{.Poster.AvatarLink}}" alt="" width="20"/>
                         <a href="/user/{{.Poster.Name}}">{{.Poster.Name}}</a></span>