From af7d4e0ec93a6c1b03a3440f88ac8e9a7f4afb43 Mon Sep 17 00:00:00 2001
From: Eugene Fox <eugene@xfox111.net>
Date: Thu, 22 Feb 2024 14:56:32 +0000
Subject: [PATCH] Minor fixes and refactoring

---
 backend/MuiCharts.Api/Controllers/ApiControllerBase.cs | 2 +-
 backend/MuiCharts.Api/Controllers/PointsController.cs  | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/backend/MuiCharts.Api/Controllers/ApiControllerBase.cs b/backend/MuiCharts.Api/Controllers/ApiControllerBase.cs
index 745970a..8c0ccc0 100644
--- a/backend/MuiCharts.Api/Controllers/ApiControllerBase.cs
+++ b/backend/MuiCharts.Api/Controllers/ApiControllerBase.cs
@@ -37,7 +37,7 @@ public abstract class ApiControllerBase<T>(ILogger<T> logger)
 
 		Error firstError = errors[0];
 
-		Logger.LogError("An error occured during request processing: {Error}", firstError);
+		Logger.LogWarning("An error occured during request processing: {Error}", firstError);
 
 		int statusCode = firstError.Type switch
 		{
diff --git a/backend/MuiCharts.Api/Controllers/PointsController.cs b/backend/MuiCharts.Api/Controllers/PointsController.cs
index 4801491..3dad5e7 100644
--- a/backend/MuiCharts.Api/Controllers/PointsController.cs
+++ b/backend/MuiCharts.Api/Controllers/PointsController.cs
@@ -103,9 +103,10 @@ public class PointsController(
 
 		PointResponse[] points = [
 			.. query
-			.Skip((request.Page - 1) * request.Count)
-			.Take(request.Count)
-			.Select(point => MapPointResponse(point))
+				.OrderBy(i => i.Id)
+				.Skip((request.Page - 1) * request.Count)
+				.Take(request.Count)
+				.Select(point => MapPointResponse(point))
 		];
 
 		GetPointsResponse response = new(