diff options
author | jan <jan@ruken.pw> | 2016-10-10 13:16:43 (UTC) |
---|---|---|
committer | jan <jan@ruken.pw> | 2016-10-10 13:16:43 (UTC) |
commit | bf51b2e4a035a988b9e9cef737966846c619dace (patch) | |
tree | 5022edbeb7a70bddb9655f58704fb1457b8f568a | |
parent | 9d4ebd033d5f0d6ad6d1d6f52fd16e0bdc17a49e (diff) |
lists: immer einen neuen user holen, nicht aus dem Cache nehmen.
Wenn wir ne DB query machen, koennen wir auch gleich das result nehmen.
-rw-r--r-- | modules/lists/lists.go | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/modules/lists/lists.go b/modules/lists/lists.go index 6129502..d1913be 100644 --- a/modules/lists/lists.go +++ b/modules/lists/lists.go | |||
@@ -133,12 +133,6 @@ func (m *Module) getLists(whereClause string, params ...interface{}) ([]*List, e | |||
133 | continue | 133 | continue |
134 | } | 134 | } |
135 | 135 | ||
136 | if clist, ok := m.c.Get(list.ID); ok { | ||
137 | // weiteres parsen abbrechen | ||
138 | lists = append(lists, clist.(*List)) | ||
139 | continue | ||
140 | } | ||
141 | |||
142 | // Owner kriegn | 136 | // Owner kriegn |
143 | owner, err := m.g.Charakterin.GetUserByID(ownerID) | 137 | owner, err := m.g.Charakterin.GetUserByID(ownerID) |
144 | if err != nil { | 138 | if err != nil { |
@@ -175,6 +169,11 @@ func (m *Module) GetUserLists(u *charakterin.User, withGrils bool) []*List { | |||
175 | func (m *Module) FromID(id int, withGrils bool) (*List, error) { | 169 | func (m *Module) FromID(id int, withGrils bool) (*List, error) { |
176 | if lst, ok := m.c.Get(id); ok { | 170 | if lst, ok := m.c.Get(id); ok { |
177 | l := lst.(*List) | 171 | l := lst.(*List) |
172 | // einen neuen User holen, wir wollen ja sicher gehen und so. | ||
173 | if owner, err := m.g.Charakterin.GetUserByID(l.Owner.ID); err == nil { | ||
174 | l.Owner = owner | ||
175 | } | ||
176 | |||
178 | // Potenzieller Optimierungsbedarf: neue query, wenn die Liste leer ist. updateGrilOrder nutzt derzeit aber genau dieses verhalten. | 177 | // Potenzieller Optimierungsbedarf: neue query, wenn die Liste leer ist. updateGrilOrder nutzt derzeit aber genau dieses verhalten. |
179 | if withGrils && len(l.Grils) == 0 { | 178 | if withGrils && len(l.Grils) == 0 { |
180 | if err := m.getListGrils(l); err != nil { | 179 | if err := m.getListGrils(l); err != nil { |