fix(database/gdb): Resolve the cache error overwriting caused by the use of fixed cache keys in pagination queries.

[LanceAdd]

func main() {
	adapter := gcache.NewAdapterRedis(g.Redis())
	g.DB().GetCache().SetAdapter(adapter)
	result, count, err := g.Model("TBL_USER").Cache(gdb.CacheOption{
		Duration: 100 * time.Minute,
		Name:     "VIP",
	}).AllAndCount(false)
	g.DumpJson(result)
	fmt.Println(count, err)
}

执行这段查询后g.DumpJson(result)的结果是[ { "COUNT(1)": 5 } ],但是正确结果应该是五条用户信息，查看源代码后发现先执行的count查询和后来select查询都是直接使用了VIP这个缓存key，在redis中实际缓存key是SelectCache:VIP，第二步查询select获得的是count查询的缓存，所以查询结果是错的。 因此为Model增加一个PageCache方法允许用户分别设置count query和data query的缓存参数

// PageCache sets the cache feature for pagination queries. It allows to configure
// separate cache options for count query and data query in pagination.
//
// Note that, the cache feature is disabled if the model is performing select statement
// on a transaction.
func (m *Model) PageCache(countOption CacheOption, dataOption CacheOption) *Model {
	model := m.getModel()
	model.pageCacheOption = []CacheOption{countOption, dataOption}
	model.cacheEnabled = true
	return model
}

然后AllAndCount在查询时分别给两个查询设置对应的缓存参数ScanAndCount同理

// AllAndCount retrieves all records and the total count of records from the model.
// If useFieldForCount is true, it will use the fields specified in the model for counting;
// otherwise, it will use a constant value of 1 for counting.
// It returns the result as a slice of records, the total count of records, and an error if any.
// The where parameter is an optional list of conditions to use when retrieving records.
//
// Example:
//
//	var model Model
//	var result Result
//	var count int
//	where := []any{"name = ?", "John"}
//	result, count, err := model.AllAndCount(true)
//	if err != nil {
//	    // Handle error.
//	}
//	fmt.Println(result, count)
func (m *Model) AllAndCount(useFieldForCount bool) (result Result, totalCount int, err error) {
	// Clone the model for counting
	countModel := m.Clone()

	// If useFieldForCount is false, set the fields to a constant value of 1 for counting
	if !useFieldForCount {
		countModel.fields = []any{Raw("1")}
	}
	if len(m.pageCacheOption) > 0 {
		countModel = countModel.Cache(m.pageCacheOption[0])
	}

	// Get the total count of records
	totalCount, err = countModel.Count()
	if err != nil {
		return
	}

	// If the total count is 0, there are no records to retrieve, so return early
	if totalCount == 0 {
		return
	}

	resultModel := m.Clone()
	if len(m.pageCacheOption) > 1 {
		resultModel = resultModel.Cache(m.pageCacheOption[1])
	}

	// Retrieve all records
	result, err = resultModel.doGetAll(m.GetCtx(), SelectTypeDefault, false)
	return
}

[Copilot]

Pull Request Overview

This PR fixes a cache key collision issue in the GoFrame database ORM where AllAndCount and ScanAndCount methods were incorrectly sharing the same cache key for both count and select operations, causing incorrect query results.

• Introduces a new cache prefix constant cachePrefixCountCache to differentiate count queries from select queries
• Modifies AllAndCount and ScanAndCount methods to use prefixed cache keys for count operations
• Ensures count queries use SelectCache:CountCache: prefix to avoid cache conflicts with select queries

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
database/gdb/gdb.go	Adds new cache prefix constant for count operations
database/gdb/gdb_model_select.go	Updates AllAndCount and ScanAndCount to use prefixed cache keys for count queries

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}