优化代码

internal/theme/common/bodyclass.go

@@ -12,19 +12,19 @@ import (
 )
 
 var commonClass = map[int]string{
-	constraints.Home:     "home blog",
-	constraints.Archive:  "archive date",
+	constraints.Home:     "home blog ",
+	constraints.Archive:  "archive date ",
 	constraints.Category: "archive category ",
 	constraints.Tag:      "archive category ",
-	constraints.Search:   "search",
+	constraints.Search:   "search ",
 	constraints.Detail:   "post-template-default single single-post ",
 }
 
 func (h *Handle) CalBodyClass() {
-	h.GinH["bodyClass"] = h.bodyClass(h.Class...)
+	h.GinH["bodyClass"] = h.BodyClass(h.Class...)
 }
 
-func (h *Handle) bodyClass(class ...string) string {
+func (h *Handle) BodyClass(class ...string) string {
 	s := ""
 	if constraints.Ok != h.Stats {
 		return "error404"

internal/theme/common/common.go

@@ -2,6 +2,7 @@ package common
 
 import (
 	"github.com/fthvgb1/wp-go/helper/slice"
+	str "github.com/fthvgb1/wp-go/helper/strings"
 	"github.com/fthvgb1/wp-go/internal/cmd/reload"
 	"github.com/fthvgb1/wp-go/internal/pkg/constraints"
 	"github.com/fthvgb1/wp-go/internal/pkg/logs"
@@ -23,7 +24,7 @@ type Handle struct {
 	Templ     string
 	Class     []string
 	ThemeMods wpconfig.ThemeMods
-	Plugins   []HandlePluginFn[*Handle]
+	HandleFns []func(*Handle)
 }
 
 func NewHandle(c *gin.Context, scene int, theme string) *Handle {
@@ -61,19 +62,33 @@ func (h *Handle) GetPassword() {
 	}
 }
 
-func (h *Handle) ExecHandlePlugin() {
-	if len(h.Plugins) > 0 {
-		HandlePlugin(h.Plugins, h)
+func (h *Handle) Render() {
+	if h.Templ == "" {
+		h.Templ = str.Join(h.Theme, "/posts/index.gohtml")
+		if h.Scene == constraints.Detail {
+			h.Templ = str.Join(h.Theme, "/posts/detail.gohtml")
+		}
 	}
+	for _, fn := range h.HandleFns {
+		fn(h)
+	}
+	h.AutoCal("siteIcon", h.CalSiteIcon)
+	h.AutoCal("customLogo", h.CalCustomLogo)
+	h.AutoCal("customCss", h.CalCustomCss)
+	h.CalBodyClass()
+
+	h.C.HTML(h.Code, h.Templ, h.GinH)
 }
 
 type HandleFn[T any] func(T)
 
-type HandlePluginFn[T any] func(HandleFn[T], T) HandleFn[T]
+type HandlePipeFn[T any] func(HandleFn[T], T)
 
-// HandlePlugin 方便把功能写在其它包里
-func HandlePlugin[T any](fns []HandlePluginFn[T], h T) HandleFn[T] {
-	return slice.ReverseReduce(fns, func(t HandlePluginFn[T], r HandleFn[T]) HandleFn[T] {
-		return t(r, h)
-	}, func(t T) {})
+// HandlePipe  方便把功能写在其它包里
+func HandlePipe[T any](fns []HandlePipeFn[T], initial func(T)) HandleFn[T] {
+	return slice.ReverseReduce(fns, func(next HandlePipeFn[T], f func(t T)) func(t T) {
+		return func(t T) {
+			next(f, t)
+		}
+	}, initial)
 }

internal/theme/common/detail.go

@@ -17,20 +17,25 @@ type DetailHandle struct {
 	CommentRender plugins.CommentHtml
 	Comments      []models.Comments
 	Post          models.Posts
+	Pipes         []HandlePipeFn[*DetailHandle]
 }
 
 func NewDetailHandle(handle *Handle) *DetailHandle {
 	return &DetailHandle{Handle: handle}
 }
 
+func (d *DetailHandle) Pipe(calls ...HandlePipeFn[*DetailHandle]) {
+	HandlePipe[*DetailHandle](append(calls, d.Pipes...), func(d *DetailHandle) {
+		d.Render()
+	})(d)
+}
+
 func (d *DetailHandle) BuildDetailData() (err error) {
 	d.GinH["title"] = wpconfig.GetOption("blogname")
 	err = d.CheckAndGetPost()
 	if err != nil {
 		return
 	}
-	d.WidgetAreaData()
-	d.GetPassword()
 	d.Comment()
 	d.ContextPost()
 	return
@@ -73,6 +78,9 @@ func (d *DetailHandle) Comment() {
 }
 
 func (d *DetailHandle) RenderComment() {
+	if d.CommentRender == nil {
+		d.CommentRender = plugins.CommentRender()
+	}
 	ableComment := true
 	if d.Post.CommentStatus != "open" ||
 		(d.Post.PostPassword != "" && d.Password != d.Post.PostPassword) {
@@ -94,19 +102,8 @@ func (d *DetailHandle) ContextPost() {
 
 func (d *DetailHandle) Render() {
 	d.PasswordProject()
-	if d.CommentRender == nil {
-		d.CommentRender = plugins.CommentRender()
-	}
-	d.CalBodyClass()
-	d.AutoCal("siteIcon", d.CalSiteIcon)
-	d.AutoCal("customLogo", d.CalCustomLogo)
-	d.AutoCal("customCss", d.CalCustomCss)
 	d.RenderComment()
-	d.CalBodyClass()
-	if d.Templ == "" {
-		d.Templ = fmt.Sprintf("%s/posts/detail.gohtml", d.Theme)
-	}
-	d.C.HTML(d.Code, d.Templ, d.GinH)
+	d.Handle.Render()
 }
 
 func (d *DetailHandle) Details() {
@@ -117,6 +114,5 @@ func (d *DetailHandle) Details() {
 		d.C.HTML(d.Code, d.Templ, d.GinH)
 		return
 	}
-	d.ExecHandlePlugin()
 	d.Render()
 }

internal/theme/common/index.go

@@ -15,6 +15,26 @@ import (
 	"net/http"
 )
 
+type IndexHandle struct {
+	*Handle
+	Param        *IndexParams
+	Posts        []models.Posts
+	PageEle      pagination.Elements
+	TotalRows    int
+	PostsPlugins map[string]Plugin[models.Posts, *Handle]
+	Pipes        []HandlePipeFn[*IndexHandle]
+}
+
+func NewIndexHandle(handle *Handle) *IndexHandle {
+	return &IndexHandle{Handle: handle}
+}
+
+func (i *IndexHandle) Pipe(calls ...HandlePipeFn[*IndexHandle]) {
+	HandlePipe[*IndexHandle](append(calls, i.Pipes...), func(i *IndexHandle) {
+		i.Render()
+	})(i)
+}
+
 func (i *IndexHandle) ParseIndex(parm *IndexParams) (err error) {
 	i.Param = parm
 	switch i.Scene {
@@ -64,6 +84,9 @@ func (i *IndexHandle) GetIndexData() (posts []models.Posts, totalRaw int, err er
 }
 
 func (i *IndexHandle) Pagination() {
+	if i.PageEle == nil {
+		i.PageEle = plugins.TwentyFifteenPagination()
+	}
 	q := i.C.Request.URL.Query().Encode()
 	if q != "" {
 		q = fmt.Sprintf("?%s", q)
@@ -108,18 +131,8 @@ func (i *IndexHandle) ExecPostsPlugin(calls ...func(*models.Posts)) {
 
 func (i *IndexHandle) Render() {
 	i.ExecPostsPlugin()
-	if i.PageEle == nil {
-		i.PageEle = plugins.TwentyFifteenPagination()
-	}
 	i.Pagination()
-	i.AutoCal("siteIcon", i.CalSiteIcon)
-	i.AutoCal("customLogo", i.CalCustomLogo)
-	i.AutoCal("customCss", i.CalCustomCss)
-	i.CalBodyClass()
-	if i.Templ == "" {
-		i.Templ = fmt.Sprintf("%s/posts/index.gohtml", i.Theme)
-	}
-	i.C.HTML(i.Code, i.Templ, i.GinH)
+	i.Handle.Render()
 }
 
 func (i *IndexHandle) Indexs() {
@@ -130,6 +143,5 @@ func (i *IndexHandle) Indexs() {
 		i.C.HTML(i.Code, i.Templ, i.GinH)
 		return
 	}
-	i.ExecHandlePlugin()
 	i.Render()
 }

internal/theme/common/indexparams.go

@@ -11,10 +11,8 @@ import (
 	"github.com/fthvgb1/wp-go/internal/pkg/config"
 	"github.com/fthvgb1/wp-go/internal/pkg/constraints"
 	"github.com/fthvgb1/wp-go/internal/pkg/dao"
-	"github.com/fthvgb1/wp-go/internal/pkg/models"
 	"github.com/fthvgb1/wp-go/internal/wpconfig"
 	"github.com/fthvgb1/wp-go/model"
-	"github.com/fthvgb1/wp-go/plugin/pagination"
 	"github.com/gin-gonic/gin"
 	"strconv"
 	"strings"
@@ -55,19 +53,6 @@ type IndexParams struct {
 	BlogName          string
 }
 
-type IndexHandle struct {
-	*Handle
-	Param        *IndexParams
-	Posts        []models.Posts
-	PageEle      pagination.Elements
-	TotalRows    int
-	PostsPlugins map[string]Plugin[models.Posts, *Handle]
-}
-
-func NewIndexHandle(handle *Handle) *IndexHandle {
-	return &IndexHandle{Handle: handle}
-}
-
 var months = slice.SimpleToMap(number.Range(1, 12, 1), func(v int) int {
 	return v
 })

internal/theme/twentyseventeen/layout/base.gohtml

@@ -6,7 +6,7 @@
         {{block "head" .}}
         {{end}}
     </head>
-    <body class="{{.bodyClass}} wp-embed-responsive hfeed has-header-image has-sidebar colors-light">
+    <body class="{{.bodyClass}} wp-embed-responsive hfeed has-sidebar colors-light">
     {{template "svg"}}
     <div id="page" class="site">
         <a class="skip-link screen-reader-text" href="#content">跳至内容</a>

internal/theme/twentyseventeen/posts/detail.gohtml

@@ -1,7 +1,7 @@
 {{template "layout/base" .}}
 
 {{define "content"}}
-    {{ if .post.PostContent}}
+    {{ if  gt .post.Id 0}}
         {{if .post.Thumbnail.Path}}
             <div class="single-featured-image-header">
                 <img width="{{.post.Thumbnail.OriginAttachmentData.Width}}" height="{{.post.Thumbnail.OriginAttachmentData.Height}}" src="{{.post.Thumbnail.Path}}" class="attachment-twentyseventeen-featured-image size-twentyseventeen-featured-image wp-post-image" alt="{{.post.PostTitle}}" decoding="async" loading="lazy" srcset="{{.post.Thumbnail.Srcset}}" sizes="{{.post.Thumbnail.Sizes}}">

internal/theme/twentyseventeen/twentyseventeen.go

@@ -4,7 +4,6 @@ import (
 	"fmt"
 	"github.com/fthvgb1/wp-go/helper/maps"
 	"github.com/fthvgb1/wp-go/helper/slice"
-	str "github.com/fthvgb1/wp-go/helper/strings"
 	"github.com/fthvgb1/wp-go/internal/pkg/cache"
 	"github.com/fthvgb1/wp-go/internal/pkg/constraints"
 	"github.com/fthvgb1/wp-go/internal/pkg/logs"
@@ -29,41 +28,16 @@ var paginate = func() plugins.PageEle {
 	return p
 }()
 
-type handle struct {
-	*common.Handle
-}
-
-func newHandle(h *common.Handle) *handle {
-	return &handle{Handle: h}
-}
-
-type indexHandle struct {
-	*common.IndexHandle
-	h *handle
-}
-
-func newIndexHandle(iHandle *common.IndexHandle) *indexHandle {
-	return &indexHandle{IndexHandle: iHandle, h: newHandle(iHandle.Handle)}
-}
-
-type detailHandle struct {
-	*common.DetailHandle
-	h *handle
-}
-
-func newDetailHandle(dHandle *common.DetailHandle) *detailHandle {
-	return &detailHandle{DetailHandle: dHandle, h: newHandle(dHandle.Handle)}
-}
-
 func Hook(h *common.Handle) {
 	h.WidgetAreaData()
 	h.GetPassword()
+	h.HandleFns = append(h.HandleFns, calClass)
 	h.GinH["HeaderImage"] = getHeaderImage(h.C)
 	if h.Scene == constraints.Detail {
-		newDetailHandle(common.NewDetailHandle(h)).Detail()
+		common.NewDetailHandle(h).Pipe(detail)
 		return
 	}
-	newIndexHandle(common.NewIndexHandle(h)).Index()
+	common.NewIndexHandle(h).Pipe(index)
 }
 
 var pluginFns = func() map[string]common.Plugin[models.Posts, *common.Handle] {
@@ -72,31 +46,29 @@ var pluginFns = func() map[string]common.Plugin[models.Posts, *common.Handle] {
 	})
 }()
 
-func (i *indexHandle) Index() {
-	i.Templ = "twentyseventeen/posts/index.gohtml"
+func index(next common.HandleFn[*common.IndexHandle], i *common.IndexHandle) {
 	err := i.BuildIndexData(common.NewIndexParams(i.C))
 	if err != nil {
 		i.Stats = constraints.Error404
 		i.Code = http.StatusNotFound
-		i.GinH["bodyClass"] = i.h.bodyClass()
+		i.CalBodyClass()
 		i.C.HTML(i.Code, i.Templ, i.GinH)
 		return
 	}
 	i.PostsPlugins = pluginFns
 	i.PageEle = paginate
-	i.Render()
+	next(i)
 }
 
-func (d *detailHandle) Detail() {
+func detail(next common.HandleFn[*common.DetailHandle], d *common.DetailHandle) {
 	err := d.BuildDetailData()
 	if err != nil {
 		d.Code = http.StatusNotFound
 		d.Stats = constraints.Error404
-		d.GinH["bodyClass"] = d.h.bodyClass()
+		d.GinH["bodyClass"] = d.BodyClass()
 		d.C.HTML(d.Code, d.Templ, d.GinH)
 		return
 	}
-	d.GinH["bodyClass"] = d.h.bodyClass()
 	img := wpconfig.Thumbnail(d.Post.Thumbnail.OriginAttachmentData, "thumbnail", "", "thumbnail", "post-thumbnail")
 	img.Width = img.OriginAttachmentData.Width
 	img.Height = img.OriginAttachmentData.Height
@@ -105,8 +77,8 @@ func (d *detailHandle) Detail() {
 	d.Post.Thumbnail = img
 	d.CommentRender = commentFormat
 	d.GinH["post"] = d.Post
-	d.Render()
 
+	next(d)
 }
 
 var commentFormat = comment{}
@@ -148,46 +120,21 @@ func getHeaderImage(c *gin.Context) (r models.PostThumbnail) {
 		logs.ErrPrintln(err, "获取页眉背景图失败")
 	} else if len(hs) > 0 && err == nil {
 		_, r = slice.Rand(hs)
-
 	}
 	r.Sizes = "100vw"
 	return
 }
 
-func (i *handle) bodyClass() string {
-	s := ""
-	if constraints.Ok != i.Stats {
-		return "error404"
+func calClass(h *common.Handle) {
+	u := wpconfig.GetThemeModsVal(h.Theme, "header_image", h.ThemeMods.ThemeSupport.CustomHeader.DefaultImage)
+	if u != "" && u != "remove-header" {
+		h.Class = append(h.Class, "has-header-image")
 	}
-	switch i.Scene {
-	case constraints.Search:
-		s = "search-no-results"
-		if len(i.GinH["posts"].([]models.Posts)) > 0 {
-			s = "search-results"
+	if h.Scene == constraints.Archive {
+		if "one-column" == wpconfig.GetThemeModsVal(h.Theme, "page_layout", "") {
+			h.Class = append(h.Class, "page-one-column")
+		} else {
+			h.Class = append(h.Class, "page-two-column")
 		}
-	case constraints.Category, constraints.Tag:
-		cat := i.C.Param("category")
-		if cat == "" {
-			cat = i.C.Param("tag")
-		}
-		_, cate := slice.SearchFirst(cache.CategoriesTags(i.C, i.Scene), func(my models.TermsMy) bool {
-			return my.Name == cat
-		})
-		if cate.Slug[0] != '%' {
-			s = cate.Slug
-		}
-		s = fmt.Sprintf("category-%d %v", cate.Terms.TermId, s)
-	case constraints.Detail:
-		s = fmt.Sprintf("postid-%d", i.GinH["post"].(models.Posts).Id)
 	}
-	return str.Join(class[i.Scene], s)
-}
-
-var class = map[int]string{
-	constraints.Home:     "home blog ",
-	constraints.Archive:  "archive date page-two-column",
-	constraints.Category: "archive category page-two-column",
-	constraints.Tag:      "archive category page-two-column ",
-	constraints.Search:   "search ",
-	constraints.Detail:   "post-template-default single single-post single-format-standard ",
 }

model/parse.go

@@ -7,7 +7,7 @@ import (
 	"strings"
 )
 
-func (w SqlBuilder) parseField(ss []string, s *strings.Builder) {
+func (w SqlBuilder) parseWhereField(ss []string, s *strings.Builder) {
 	if strings.Contains(ss[0], ".") && !strings.Contains(ss[0], "(") {
 		x := slice.Map(strings.Split(ss[0], "."), func(t string) string {
 			return str.Join("`", t, "`")
@@ -81,11 +81,11 @@ func (w SqlBuilder) ParseWhere(in *[][]any) (string, []any, error) {
 			s.WriteString(ss[0])
 			s.WriteString(" and ")
 		case 2:
-			w.parseField(ss, &s)
+			w.parseWhereField(ss, &s)
 			s.WriteString("=? and ")
 			args = append(args, ss[1])
 		case 3, 4:
-			w.parseField(ss, &s)
+			w.parseWhereField(ss, &s)
 			s.WriteString(ss[1])
 			if w.parseIn(ss, &s, &c, &args, in) {
 				s.WriteString(" and ")
@@ -115,7 +115,7 @@ func (w SqlBuilder) ParseWhere(in *[][]any) (string, []any, error) {
 				if i == 0 {
 					s.WriteString("( ")
 				}
-				w.parseField(ss[start+1:end], &s)
+				w.parseWhereField(ss[start+1:end], &s)
 				s.WriteString(ss[start+2])
 				if w.parseIn(ss[start+1:end], &s, &c, &args, in) {
 					s.WriteString(" and ")