Commit 68c02ba1 authored Jan 25, 2019 by Andrei Mihu

Correctly handle leaderboard records previous page cursor.

parent 07e315dc

CHANGELOG.md

+1 −0

Original line number	Diff line number	Diff line
		@@ -6,6 +6,7 @@ The format is based on [keep a changelog](http://keepachangelog.com) and this pr
		## [Unreleased]
		### Fixed
		- Correctly return group user results in Lua runtime listing operation.
		- Correctly handle leaderboard records previous page cursor.

		## [2.3.2] - 2019-01-17
		### Fixed

server/core_leaderboard.go

+9 −1

Original line number	Diff line number	Diff line
		@@ -203,7 +203,15 @@ func LeaderboardRecordsList(ctx context.Context, logger zap.Logger, db sql.DB,

		if incomingCursor != nil && !incomingCursor.IsNext {
		// If this was a previous page listing, flip the results to their normal order and swap the cursors.
		if nextCursor != nil && prevCursor != nil {
		nextCursor, nextCursor.IsNext, nextCursor.Rank, prevCursor, prevCursor.IsNext, prevCursor.Rank = prevCursor, prevCursor.IsNext, prevCursor.Rank, nextCursor, nextCursor.IsNext, nextCursor.Rank
		} else if nextCursor != nil {
		nextCursor, prevCursor = nil, nextCursor
		prevCursor.IsNext = !prevCursor.IsNext
		} else if prevCursor != nil {
		nextCursor, prevCursor = prevCursor, nil
		nextCursor.IsNext = !nextCursor.IsNext
		}

		for i, j := 0, len(records)-1; i < j; i, j = i+1, j-1 {
		records[i], records[i].Rank, records[j], records[j].Rank = records[j], records[j].Rank, records[i], records[i].Rank