cache hinzugefuegt. wird derzeit vom lists und grils modul verwendet. versuch die loadingzeit zu verringern (anscheinend wird im hintergrund auf jeder route der index requested, was ziemlich doof ist).

1 files changed, 38 insertions, 6 deletions

diff --git a/modules/lists/lists.go b/modules/lists/lists.go
index 836f6af..910162a 100644
--- a/modules/lists/lists.go
+++ b/modules/lists/lists.go
@@ -7,6 +7,7 @@ import (
         "fagott.pw/charakterin"
         "fagott.pw/grilist/frontend"
         "fagott.pw/grilist/grilist"
+        "fagott.pw/grilist/cache"
         "fagott.pw/grilist/modules/grils"
         "fmt"
         "github.com/julienschmidt/httprouter"
@@ -22,6 +23,7 @@ import (
 // Module und so.
 type Module struct {
         g     *grilist.Grilist
+    c *cache.Cache
         grils *grils.GrilsModule
 }
 
@@ -76,6 +78,8 @@ func (m *Module) Init(g *grilist.Grilist) {
         m.g.Router.POST("/new/list", m.createList)
 
         m.g.Router.GET("/api/lists/user", m.APIgetUserLists)
+    
+    m.c = cache.New()
 }
 
 func (m *Module) getListGrils(list *List) error {
@@ -85,6 +89,7 @@ func (m *Module) getListGrils(list *List) error {
         }
 
         defer rows.Close()
+    list.Grils = list.Grils[:0]
         for rows.Next() {
                 var grilID int
                 lg := &ListGril{}
@@ -123,6 +128,12 @@ func (m *Module) getLists(whereClause string, params ...interface{}) ([]*List, e
                         log.Println("error scanning row in getLists:", err)
                         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)
@@ -131,7 +142,8 @@ func (m *Module) getLists(whereClause string, params ...interface{}) ([]*List, e
                         continue
                 }
                 list.Owner = owner
-
+        
+        m.c.Insert(list.ID, list)
                 lists = append(lists, list)
         }
 
@@ -157,6 +169,17 @@ func (m *Module) GetUserLists(u *charakterin.User, withGrils bool) []*List {
 
 // FromID sucht nach der Liste mit der gegebenen ID und gibt sie, falls sie existiert, zurück.
 func (m *Module) FromID(id int, withGrils bool) (*List, error) {
+    if lst, ok := m.c.Get(id); ok {
+        l := lst.(*List)
+        // 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 {
+                                return nil, err
+                        }
+        }
+        return l, nil
+    }
+
         lists, err := m.getLists(`id = $1`, id)
         if err != nil {
                 return nil, err
@@ -210,11 +233,12 @@ func (m *Module) ProvideDashboardData(user *charakterin.User) []grilist.Dashboar
                 log.Println(err)
                 return categories
         }
-        //for _, list := range lists {
-        //if err := m.getListGrils(list); err != nil {
-        //      log.Println(err)
-        //}
-        //}
+    
+        for _, list := range lists {
+        if err := m.getListGrils(list); err != nil {
+            log.Println(err)
+        }
+        }
 
         categories = append(categories, grilist.DashboardCategory{
                 Title: "Neueste Listen",
@@ -242,12 +266,14 @@ func (m *Module) viewList(w http.ResponseWriter, r *http.Request, p httprouter.P
 
         id, err := strconv.Atoi(sid)
         if err != nil {
+        log.Println("redir")
                 http.Redirect(w, r, "/", 302)
                 return
         }
 
         list, err := m.FromID(id, true)
         if err != nil {
+        log.Println("redir")
                 http.Redirect(w, r, "/", 302)
                 return
         }
@@ -406,6 +432,12 @@ func (m *Module) updateGrilOrder(w http.ResponseWriter, r *http.Request, p httpr
                 http.Error(w, "could not update gril order", 500)
                 return
         }
+    
+    // wenn die liste im cache ist, die Grils clearen, damit beim naechsten aufruf die Gril-Liste neu geholt wird.
+    if l, ok := m.c.Get(listID); ok {
+        ls := l.(*List)
+        ls.Grils = ls.Grils[:0]
+    }
 
         w.WriteHeader(200)
         w.Write([]byte("ok"))