pedro/go-blocks
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor template cache: rename types and add logging

Browse Source
This commit is contained in:
Pedro Pérez 2024-11-09 15:52:34 +01:00
parent 5171ce7967
commit 73557af314
2 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
config/config.go
View File

@ -12,6 +12,7 @@ import (
type App struct { type App struct {
DataSource string DataSource string
UseCache bool UseCache bool
// TODO: Extract Security field and use as ExtendedApp (or think a strategy for better library management)
Security Security Security Security
AppInfo AppInfo AppInfo AppInfo
} }

33
gorender/render.go
View File

@ -12,7 +12,7 @@ import (
"github.com/justinas/nosurf" "github.com/justinas/nosurf"
) )
type TemplateCache map[string]*template.Template type templateCache map[string]*template.Template
type Render struct { type Render struct {
EnableCache bool EnableCache bool
@ -23,21 +23,23 @@ type Render struct {
// páginas de la aplicación. Estas son las que van a ser llamadas para // páginas de la aplicación. Estas son las que van a ser llamadas para
// mostrar en pantalla. // mostrar en pantalla.
PageTemplatesPath string PageTemplatesPath string
TemplateCache TemplateCache
Functions template.FuncMap Functions template.FuncMap
templateCache templateCache
} }
type OptionFunc func(*Render) type OptionFunc func(*Render)
type Data map[string]interface{}
type FeedbackData map[string]string
type TemplateData struct { type TemplateData struct {
Data map[string]interface{} Data Data
// SessionData contiene los datos de la sesión del usuario. // SessionData contiene los datos de la sesión del usuario.
SessionData interface{} SessionData interface{}
// FeedbackData tiene como función mostrar los mensajes habituales de // FeedbackData tiene como función mostrar los mensajes habituales de
// información, advertencia, éxito y error. No va implícitamente relacionado // información, advertencia, éxito y error. No va implícitamente relacionado
// con los errores de validación de formularios pero pueden ser usados para // con los errores de validación de formularios pero pueden ser usados para
// ello. // ello.
FeedbackData map[string]string FeedbackData FeedbackData
// FormData es una estructura que contiene los errores de validación de los // FormData es una estructura que contiene los errores de validación de los
// formularios además de los valores que se han introducido en los campos. // formularios además de los valores que se han introducido en los campos.
FormData FormData FormData FormData
@ -58,7 +60,7 @@ func WithRenderOptions(opts *Render) OptionFunc {
if opts.EnableCache { if opts.EnableCache {
re.EnableCache = opts.EnableCache re.EnableCache = opts.EnableCache
re.TemplateCache, _ = re.createTemplateCache() re.templateCache, _ = re.createTemplateCache()
} }
} }
} }
@ -74,8 +76,8 @@ func New(opts ...OptionFunc) *Render {
EnableCache: false, EnableCache: false,
TemplatesPath: "templates", TemplatesPath: "templates",
PageTemplatesPath: "templates/pages", PageTemplatesPath: "templates/pages",
TemplateCache: TemplateCache{},
Functions: functions, Functions: functions,
templateCache: templateCache{},
} }
return config.apply(opts...) return config.apply(opts...)
@ -95,12 +97,17 @@ func addDefaultData(td *TemplateData, r *http.Request) *TemplateData {
} }
func (re *Render) Template(w http.ResponseWriter, r *http.Request, tmpl string, td *TemplateData) error { func (re *Render) Template(w http.ResponseWriter, r *http.Request, tmpl string, td *TemplateData) error {
var tc TemplateCache var tc templateCache
var err error var err error
if td == nil {
td = &TemplateData{}
}
if re.EnableCache { if re.EnableCache {
tc = re.TemplateCache tc = re.templateCache
} else { } else {
slog.Info("creating template cache")
tc, err = re.createTemplateCache() tc, err = re.createTemplateCache()
if err != nil { if err != nil {
slog.Error("error creating template cache:", "error", err) slog.Error("error creating template cache:", "error", err)
@ -137,7 +144,7 @@ func findHTMLFiles(root string) ([]string, error) {
return err return err
} }
if !d.IsDir() && filepath.Ext(path) == ".html" { if !d.IsDir() && filepath.Ext(path) == ".gohtml" {
files = append(files, path) files = append(files, path)
} }
@ -151,8 +158,8 @@ func findHTMLFiles(root string) ([]string, error) {
return files, nil return files, nil
} }
func (re *Render) createTemplateCache() (TemplateCache, error) { func (re *Render) createTemplateCache() (templateCache, error) {
myCache := TemplateCache{} myCache := templateCache{}
pagesTemplates, err := findHTMLFiles(re.PageTemplatesPath) pagesTemplates, err := findHTMLFiles(re.PageTemplatesPath)
if err != nil { if err != nil {
@ -164,10 +171,6 @@ func (re *Render) createTemplateCache() (TemplateCache, error) {
return myCache, err return myCache, err
} }
for function := range re.Functions {
slog.Info("function found", "function", function)
}
for _, file := range pagesTemplates { for _, file := range pagesTemplates {
name := filepath.Base(file) name := filepath.Base(file)
ts, err := template.New(name).Funcs(re.Functions).ParseFiles(append(files, file)...) ts, err := template.New(name).Funcs(re.Functions).ParseFiles(append(files, file)...)