From b78e2edfb845e5117fbec38f29f269af2e44e462 Mon Sep 17 00:00:00 2001 From: Daniel <845765@qq.com> Date: Wed, 13 Aug 2025 23:35:16 +0800 Subject: [PATCH] :art: Improve av --- kernel/model/export.go | 2 +- kernel/sql/av.go | 27 +++++++++++++++------------ kernel/sql/av_gallery.go | 5 +++-- kernel/sql/av_table.go | 5 +++-- 4 files changed, 22 insertions(+), 17 deletions(-) diff --git a/kernel/model/export.go b/kernel/model/export.go index f736484da..17745bb4f 100644 --- a/kernel/model/export.go +++ b/kernel/model/export.go @@ -3442,7 +3442,7 @@ func getAttrViewTable(attrView *av.AttributeView, view *av.View, query string) ( } depth := 1 - ret = sql.RenderAttributeViewTable(attrView, view, query, &depth) + ret = sql.RenderAttributeViewTable(attrView, view, query, &depth, map[string]*av.AttributeView{}) return } diff --git a/kernel/sql/av.go b/kernel/sql/av.go index b5d2ce83f..366841b70 100644 --- a/kernel/sql/av.go +++ b/kernel/sql/av.go @@ -72,10 +72,14 @@ func RenderGroupView(attrView *av.AttributeView, view, groupView *av.View, query func RenderView(attrView *av.AttributeView, view *av.View, query string) (ret av.Viewable) { depth := 1 - return renderView(attrView, view, query, &depth) + renderedAttrViews := map[string]*av.AttributeView{} + renderedAttrViews[attrView.ID] = attrView + ret = renderView(attrView, view, query, &depth, renderedAttrViews) + return } -func renderView(attrView *av.AttributeView, view *av.View, query string, depth *int) (ret av.Viewable) { +func renderView(attrView *av.AttributeView, view *av.View, query string, + depth *int, renderedAttrViews map[string]*av.AttributeView) (ret av.Viewable) { if 7 < *depth { return } @@ -83,9 +87,9 @@ func renderView(attrView *av.AttributeView, view *av.View, query string, depth * *depth++ switch view.LayoutType { case av.LayoutTypeTable: - ret = RenderAttributeViewTable(attrView, view, query, depth) + ret = RenderAttributeViewTable(attrView, view, query, depth, renderedAttrViews) case av.LayoutTypeGallery: - ret = RenderAttributeViewGallery(attrView, view, query, depth) + ret = RenderAttributeViewGallery(attrView, view, query, depth, renderedAttrViews) } return } @@ -321,9 +325,8 @@ func fillAttributeViewBaseValue(baseValue *av.BaseValue, fieldID, itemID string, } } -func fillAttributeViewAutoGeneratedValues(attrView *av.AttributeView, collection av.Collection, ials map[string]map[string]string, items map[string][]*av.KeyValues, depth *int) { - avCache := map[string]*av.AttributeView{} - avCache[attrView.ID] = attrView +func fillAttributeViewAutoGeneratedValues(attrView *av.AttributeView, collection av.Collection, ials map[string]map[string]string, items map[string][]*av.KeyValues, + depth *int, renderedAttrViews map[string]*av.AttributeView) { for _, item := range collection.GetItems() { for _, value := range item.GetValues() { itemID := item.GetID() @@ -354,11 +357,11 @@ func fillAttributeViewAutoGeneratedValues(attrView *av.AttributeView, collection break } - destAv := avCache[relKey.Relation.AvID] + destAv := renderedAttrViews[relKey.Relation.AvID] if nil == destAv { destAv, _ = av.ParseAttributeView(relKey.Relation.AvID) if nil != destAv { - avCache[relKey.Relation.AvID] = destAv + renderedAttrViews[relKey.Relation.AvID] = destAv } } if nil == destAv { @@ -373,7 +376,7 @@ func fillAttributeViewAutoGeneratedValues(attrView *av.AttributeView, collection var viewable av.Viewable if av.KeyTypeTemplate == destKey.Type { // 渲染目标视图，这样才能汇总渲染后的模板字段值 - viewable = renderView(destAv, destAv.Views[0], "", depth) + viewable = renderView(destAv, destAv.Views[0], "", depth, renderedAttrViews) } for _, blockID := range relVal.Relation.BlockIDs { @@ -409,11 +412,11 @@ func fillAttributeViewAutoGeneratedValues(attrView *av.AttributeView, collection value.Relation.Contents = nil relKey, _ := attrView.GetKey(value.KeyID) if nil != relKey && nil != relKey.Relation { - destAv := avCache[relKey.Relation.AvID] + destAv := renderedAttrViews[relKey.Relation.AvID] if nil == destAv { destAv, _ = av.ParseAttributeView(relKey.Relation.AvID) if nil != destAv { - avCache[relKey.Relation.AvID] = destAv + renderedAttrViews[relKey.Relation.AvID] = destAv } } if nil != destAv { diff --git a/kernel/sql/av_gallery.go b/kernel/sql/av_gallery.go index 9bbdc1c0b..5d4a709dc 100644 --- a/kernel/sql/av_gallery.go +++ b/kernel/sql/av_gallery.go @@ -18,7 +18,8 @@ import ( "github.com/siyuan-note/siyuan/kernel/util" ) -func RenderAttributeViewGallery(attrView *av.AttributeView, view *av.View, query string, depth *int) (ret *av.Gallery) { +func RenderAttributeViewGallery(attrView *av.AttributeView, view *av.View, query string, + depth *int, renderedAttrViews map[string]*av.AttributeView) (ret *av.Gallery) { ret = &av.Gallery{ BaseInstance: av.NewViewBaseInstance(view), CoverFrom: view.Gallery.CoverFrom, @@ -114,7 +115,7 @@ func RenderAttributeViewGallery(attrView *av.AttributeView, view *av.View, query ials := BatchGetBlockAttrsWitTrees(ialIDs, boundTrees) // 渲染自动生成的字段值，比如关联、汇总、创建时间和更新时间 - fillAttributeViewAutoGeneratedValues(attrView, ret, ials, cardsValues, depth) + fillAttributeViewAutoGeneratedValues(attrView, ret, ials, cardsValues, depth, renderedAttrViews) // 最后单独渲染模板字段，这样模板就可以使用汇总、关联、创建时间和更新时间的值了 renderTemplateErr := fillAttributeViewTemplateValues(attrView, ret, ials, cardsValues) diff --git a/kernel/sql/av_table.go b/kernel/sql/av_table.go index 3e33bd87c..0f2fecc08 100644 --- a/kernel/sql/av_table.go +++ b/kernel/sql/av_table.go @@ -22,7 +22,8 @@ import ( "github.com/siyuan-note/siyuan/kernel/util" ) -func RenderAttributeViewTable(attrView *av.AttributeView, view *av.View, query string, depth *int) (ret *av.Table) { +func RenderAttributeViewTable(attrView *av.AttributeView, view *av.View, query string, + depth *int, renderedAttrViews map[string]*av.AttributeView) (ret *av.Table) { ret = &av.Table{ BaseInstance: av.NewViewBaseInstance(view), Columns: []*av.TableColumn{}, @@ -107,7 +108,7 @@ func RenderAttributeViewTable(attrView *av.AttributeView, view *av.View, query s ials := BatchGetBlockAttrs(ialIDs) // 渲染自动生成的字段值，比如关联、汇总、创建时间和更新时间 - fillAttributeViewAutoGeneratedValues(attrView, ret, ials, rowsValues, depth) + fillAttributeViewAutoGeneratedValues(attrView, ret, ials, rowsValues, depth, renderedAttrViews) // 最后单独渲染模板字段，这样模板就可以使用汇总、关联、创建时间和更新时间的值了 renderTemplateErr := fillAttributeViewTemplateValues(attrView, ret, ials, rowsValues)