From 225c48982b501597b2368b65400e6294eca3e261 Mon Sep 17 00:00:00 2001 From: kolaente Date: Thu, 1 Nov 2018 19:12:17 +0100 Subject: [PATCH] Fixed heatmap not working in mssql (#5248) --- models/user_heatmap.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/models/user_heatmap.go b/models/user_heatmap.go index f9575ed50fc..9e603f4fe17 100644 --- a/models/user_heatmap.go +++ b/models/user_heatmap.go @@ -19,6 +19,7 @@ type UserHeatmapData struct { func GetUserHeatmapDataByUser(user *User) ([]*UserHeatmapData, error) { hdata := make([]*UserHeatmapData, 0) var groupBy string + var groupByName = "timestamp" // We need this extra case because mssql doesn't allow grouping by alias switch { case setting.UseSQLite3: groupBy = "strftime('%s', strftime('%Y-%m-%d', created_unix, 'unixepoch'))" @@ -28,13 +29,14 @@ func GetUserHeatmapDataByUser(user *User) ([]*UserHeatmapData, error) { groupBy = "extract(epoch from date_trunc('day', to_timestamp(created_unix)))" case setting.UseMSSQL: groupBy = "dateadd(DAY,0, datediff(day,0, dateadd(s, created_unix, '19700101')))" + groupByName = groupBy } - err := x.Select(groupBy+" as timestamp, count(user_id) as contributions"). + err := x.Select(groupBy+" AS timestamp, count(user_id) as contributions"). Table("action"). Where("user_id = ?", user.ID). And("created_unix > ?", (util.TimeStampNow() - 31536000)). - GroupBy("timestamp"). + GroupBy(groupByName). OrderBy("timestamp"). Find(&hdata) return hdata, err