Browse Source

Fix invalid reference in feeds template (#1820)

* Fix invalid reference in feeds template

* Comment for GetActAvatar

* Add integration test
tags/v1.2.0-rc1
Ethan Koenig Lunny Xiao 8 years ago
parent
commit
ab3b136cce
3 changed files with 16 additions and 1 deletions
  1. +9
    -0
      integrations/issue_test.go
  2. +6
    -0
      models/action.go
  3. +1
    -1
      templates/user/dashboard/feeds.tmpl

+ 9
- 0
integrations/issue_test.go View File

@@ -11,6 +11,15 @@ import (
"github.com/stretchr/testify/assert"
)

func TestNoLoginViewIssues(t *testing.T) {
prepareTestEnv(t)

req, err := http.NewRequest("GET", "/user2/repo1/issues", nil)
assert.NoError(t, err)
resp := MakeRequest(req)
assert.EqualValues(t, http.StatusOK, resp.HeaderCode)
}

func TestNoLoginViewIssue(t *testing.T) {
prepareTestEnv(t)



+ 6
- 0
models/action.go View File

@@ -142,6 +142,12 @@ func (a *Action) ShortActUserName() string {
return base.EllipsisString(a.GetActUserName(), 20)
}

// GetActAvatar the action's user's avatar link
func (a *Action) GetActAvatar() string {
a.loadActUser()
return a.ActUser.AvatarLink()
}

// GetRepoUserName returns the name of the action repository owner.
func (a *Action) GetRepoUserName() string {
a.loadRepo()


+ 1
- 1
templates/user/dashboard/feeds.tmpl View File

@@ -1,7 +1,7 @@
{{range .Feeds}}
<div class="news">
<div class="ui left">
<img class="ui avatar image" src="{{.ActAvatar}}" alt="">
<img class="ui avatar image" src="{{.GetActAvatar}}" alt="">
</div>
<div class="ui grid">
<div class="ui fifteen wide column">


Loading…
Cancel
Save