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).

4 files changed, 62 insertions, 16 deletions

diff --git a/modules/grils/gril.go b/modules/grils/gril.go
index 7b782b3..f78d31f 100644
--- a/modules/grils/gril.go
+++ b/modules/grils/gril.go
@@ -46,12 +46,12 @@ func ImagePath(id int, prioritizeThumbnail bool) string {
         // Image Priority
         // without prioritizeThumbnail: Anilist > ACD (big) > ACD (thumbnail)
         // with prioritizeThumbnail: Anilist > ACD (thumbnail) > ACD (big)
-        
+
         anilistPath := fmt.Sprintf("assets/img/gril/%d/%d.jpg", DataSourceAnilist, id)
         if _, err := os.Stat(anilistPath); err == nil {
                 return fmt.Sprintf(anilistPath)
         }
-        
+
         var big string
         exts := []string{"png", "jpg", "gif", "jpeg"}
         for _, ext := range exts {
diff --git a/modules/grils/grils.go b/modules/grils/grils.go
index fe947bc..791ea5e 100644
--- a/modules/grils/grils.go
+++ b/modules/grils/grils.go
@@ -6,16 +6,22 @@ import (
         "net/http"
         "regexp"
         "strconv"
+    "time"
         "strings"
-        "time"
 
         "fagott.pw/charakterin"
         "fagott.pw/grilist/frontend"
         "fagott.pw/grilist/grilist"
+    "fagott.pw/grilist/cache"
 
         "github.com/julienschmidt/httprouter"
 )
 
+type CachedGril struct {
+    Created time.Time
+    Gril *Gril
+}
+
 var (
         pgArrayReg = regexp.MustCompile(`(((?P<value>(([^",\\{}\s(NULL)])+|"([^"\\]|\\"|\\\\)*")))(,)?)`)
         pgValueIdx int
@@ -32,6 +38,7 @@ func findIdx() {
 
 type GrilsModule struct {
         g *grilist.Grilist
+    c *cache.Cache
 }
 
 func (m *GrilsModule) Name() string {
@@ -43,6 +50,8 @@ func (m *GrilsModule) Init(g *grilist.Grilist) {
         m.g = g
         m.g.Router.GET("/gril/:id", m.viewGril)
         m.g.Router.GET("/gril/:id/*rest", m.viewGril)
+    
+    m.c = cache.New()
 }
 
 func (m *GrilsModule) getGrils(whereClause string, params ...interface{}) ([]*Gril, error) {
@@ -143,6 +152,10 @@ func (m *GrilsModule) ProvideDashboardData(user *charakterin.User) []grilist.Das
 }
 
 func (m *GrilsModule) FromID(id int) (*Gril, error) {
+    if g, ok := m.c.Get(id); ok {
+        return g.(*Gril), nil
+    }
+    
         gril := &Gril{
                 ID: id,
         }
@@ -155,6 +168,7 @@ func (m *GrilsModule) FromID(id int) (*Gril, error) {
         if err != nil {
                 return nil, err
         }
+    m.c.Insert(id, gril)
         return gril, nil
 }
 
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"))
diff --git a/modules/user/user.go b/modules/user/user.go
index 06245e2..92a04db 100644
--- a/modules/user/user.go
+++ b/modules/user/user.go
@@ -27,13 +27,13 @@ func (m *Module) Name() string {
 
 func (m *Module) Init(g *grilist.Grilist) {
         m.g = g
-        
+
         gm, ok := g.Modules["Lists"]
         if !ok {
                 log.Fatal("tags: lists module not found")
         }
         m.lists = gm.(*lists.Module)
-        
+
         m.g.Router.GET("/user/:id", m.viewUser)
         m.g.Router.GET("/user/:id/*rest", m.viewUser)
 }
@@ -44,13 +44,13 @@ func (m *Module) ProvideDashboardData(user *charakterin.User) []grilist.Dashboar
 
 func (m *Module) viewUser(w http.ResponseWriter, r *http.Request, p httprouter.Params) {
         currentUser, _ := m.g.Charakterin.GetUserFromRequest(r)
-        
+
         userID, err := strconv.Atoi(p.ByName("id"))
         if err != nil {
                 http.Redirect(w, r, "/", 302)
                 return
         }
-        
+
         user := currentUser
         if user == nil || currentUser.ID != userID {
                 user, err = m.g.Charakterin.GetUserByID(userID)
@@ -59,13 +59,13 @@ func (m *Module) viewUser(w http.ResponseWriter, r *http.Request, p httprouter.P
                         return
                 }
         }
-        
+
         userLists := m.lists.GetUserLists(user, true)
-        
+
         data := m.g.Renderer.DefaultData()
         data["user"] = currentUser
         data["DisplayUser"] = user
         data["Lists"] = lists.ListsToCards(userLists)
-        
+
         m.g.Renderer.RenderPage("user", w, data)
 }