aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjan <jan@ruken.pw>2016-10-10 13:16:43 (UTC)
committerjan <jan@ruken.pw>2016-10-10 13:16:43 (UTC)
commitbf51b2e4a035a988b9e9cef737966846c619dace (patch)
tree5022edbeb7a70bddb9655f58704fb1457b8f568a
parent9d4ebd033d5f0d6ad6d1d6f52fd16e0bdc17a49e (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.go11
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 {
175func (m *Module) FromID(id int, withGrils bool) (*List, error) { 169func (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 {