go-gin-example icon indicating copy to clipboard operation
go-gin-example copied to clipboard

bug: 分页查询错误

Open lionelStuart opened this issue 4 years ago • 4 comments

models/tag.go line 52-53

if pageSize > 0 && pageNum >= 0 {
	err = db.Where(maps).Offset(pageNum).Limit(pageSize).Find(&tags).Error

lionelStuart avatar Feb 27 '20 13:02 lionelStuart

@lionelStuart 请更进一步阐述是什么引起的 BUG?

eddycjy avatar Mar 16 '20 04:03 eddycjy

pageNum 可以为0, Find位于函数调用末尾

lionelStuart avatar Mar 23 '20 12:03 lionelStuart

models/tag.go line 52-53

if pageSize > 0 && pageNum >= 0 {
	err = db.Where(maps).Offset(pageNum).Limit(pageSize).Find(&tags).Error

@lionelStuart 请问你是在哪里看到的这块有问题的代码呢,我看了下,我在 2018 年 6 月的时候就已经是 if pageSize > 0 && pageNum > 0{ 了,传送门:https://github.com/eddycjy/go-gin-example/commit/9aa3ebf0476ba24a197bb0cc0bec8cbb7535b984#diff-abaed573adbc4a7465b02d108bdab248R45 。

另外 Find 方法的调用也是没问题的。

eddycjy avatar May 05 '20 10:05 eddycjy

models/tag.go line 52-53

if pageSize > 0 && pageNum >= 0 {
	err = db.Where(maps).Offset(pageNum).Limit(pageSize).Find(&tags).Error

@lionelStuart 请问你是在哪里看到的这块有问题的代码呢,我看了下,我在 2018 年 6 月的时候就已经是 if pageSize > 0 && pageNum > 0{ 了,传送门:9aa3ebf#diff-abaed573adbc4a7465b02d108bdab248R45

另外 Find 方法的调用也是没问题的。 image

master上面的代码还是有问题的(位于model下tag.go)

deadly-poison avatar Aug 01 '20 08:08 deadly-poison