Revert "Enable the "limit" param in /api/search"

torkelo · torkelo · commit 10daf1d63a49 · 2016-06-17T11:22:39.000+02:00
This reverts commit 3d77685.
diff --git a/pkg/services/search/handlers.go b/pkg/services/search/handlers.go
@@ -44,7 +44,6 @@ func searchHandler(query *Query) error {
 		IsStarred:    query.IsStarred,
 		OrgId:        query.OrgId,
 		DashboardIds: query.DashboardIds,
-		Limit:        query.Limit,
 	}
 
 	if err := bus.Dispatch(&dashQuery); err != nil {
diff --git a/pkg/services/search/models.go b/pkg/services/search/models.go
@@ -42,7 +42,6 @@ type FindPersistedDashboardsQuery struct {
 	UserId       int64
 	IsStarred    bool
 	DashboardIds []int
-	Limit        int
 
 	Result HitList
 }
diff --git a/pkg/services/sqlstore/dashboard.go b/pkg/services/sqlstore/dashboard.go
@@ -123,11 +123,6 @@ type DashboardSearchProjection struct {
 }
 
 func SearchDashboards(query *search.FindPersistedDashboardsQuery) error {
-	limit := query.Limit
-	if limit == 0 {
-		limit = 1000
-	}
-
 	var sql bytes.Buffer
 	params := make([]interface{}, 0)
 
@@ -170,8 +165,7 @@ func SearchDashboards(query *search.FindPersistedDashboardsQuery) error {
 		params = append(params, "%"+query.Title+"%")
 	}
 
-	sql.WriteString(fmt.Sprintf(" ORDER BY dashboard.title ASC LIMIT ?"))
-	params = append(params, limit)
+	sql.WriteString(fmt.Sprintf(" ORDER BY dashboard.title ASC LIMIT 1000"))
 
 	var res []DashboardSearchProjection
 

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,6 @@ func searchHandler(query *Query) error {`
`44`	`44`	`IsStarred: query.IsStarred,`
`45`	`45`	`OrgId: query.OrgId,`
`46`	`46`	`DashboardIds: query.DashboardIds,`
`47`		`- Limit: query.Limit,`
`48`	`47`	`}`
`49`	`48`
`50`	`49`	`if err := bus.Dispatch(&dashQuery); err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,6 @@ type FindPersistedDashboardsQuery struct {`
`42`	`42`	`UserId int64`
`43`	`43`	`IsStarred bool`
`44`	`44`	`DashboardIds []int`
`45`		`- Limit int`
`46`	`45`
`47`	`46`	`Result HitList`
`48`	`47`	`}`