From bf51b2e4a035a988b9e9cef737966846c619dace Mon Sep 17 00:00:00 2001
From: jan <jan@ruken.pw>
Date: Mon, 10 Oct 2016 15:16:43 +0200
Subject: lists: immer einen neuen user holen, nicht aus dem Cache nehmen. Wenn
 wir ne DB query machen, koennen wir auch gleich das result nehmen.


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
 			continue
 		}
 
-		if clist, ok := m.c.Get(list.ID); ok {
-			// weiteres parsen abbrechen
-			lists = append(lists, clist.(*List))
-			continue
-		}
-
 		// Owner kriegn
 		owner, err := m.g.Charakterin.GetUserByID(ownerID)
 		if err != nil {
@@ -175,6 +169,11 @@ func (m *Module) GetUserLists(u *charakterin.User, withGrils bool) []*List {
 func (m *Module) FromID(id int, withGrils bool) (*List, error) {
 	if lst, ok := m.c.Get(id); ok {
 		l := lst.(*List)
+		// einen neuen User holen, wir wollen ja sicher gehen und so.
+		if owner, err := m.g.Charakterin.GetUserByID(l.Owner.ID); err == nil {
+			l.Owner = owner
+		}
+
 		// Potenzieller Optimierungsbedarf: neue query, wenn die Liste leer ist. updateGrilOrder nutzt derzeit aber genau dieses verhalten.
 		if withGrils && len(l.Grils) == 0 {
 			if err := m.getListGrils(l); err != nil {
-- 
cgit v0.10.1