lists: immer einen neuen user holen, nicht aus dem Cache nehmen.

Wenn wir ne DB query machen, koennen wir auch gleich das result nehmen.

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
                         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 {