Merge pull request #139 from LeadOn/develop

5.2.3

Author: LeadOn

GameOn.Application/Common/Players/Commands/UpdateConnectedPlayer/UpdateConnectedPlayerCommandHandler.cs

@@ -35,7 +35,7 @@ public UpdateConnectedPlayerCommandHandler(IApplicationDbContext context, IAccou
         /// <inheritdoc />
         public async Task<Player> Handle(UpdateConnectedPlayerCommand request, CancellationToken cancellationToken)
         {
-            var playerInDb = await this.context.Players.FirstOrDefaultAsync(x => x.KeycloakId == request.Player.KeycloakId);
+            var playerInDb = await this.context.Players.FirstOrDefaultAsync(x => x.KeycloakId == request.Player.KeycloakId, cancellationToken);
 
             if (playerInDb == null)
             {
@@ -44,7 +44,15 @@ public async Task<Player> Handle(UpdateConnectedPlayerCommand request, Cancellat
 
             playerInDb.FullName = request.Player.FullName;
             playerInDb.Nickname = request.Player.Nickname;
-            playerInDb.ProfilePictureUrl = request.Player.ProfilePictureUrl;
+
+            if (request.Player.ProfilePictureUrl is not null && request.Player.ProfilePictureUrl != string.Empty)
+            {
+                playerInDb.ProfilePictureUrl = request.Player.ProfilePictureUrl;
+            }
+            else
+            {
+                playerInDb.ProfilePictureUrl = "https://gameon.valentinvirot.fr/assets/img/gameon-logo.webp";
+            }
 
             if (request.Player.RiotGamesNickname is not null && request.Player.RiotGamesTagLine is not null)
             {

GameOn.Application/Common/Players/Commands/UpdatePlayer/UpdatePlayerCommandHandler.cs

@@ -37,7 +37,7 @@ public async Task<Player> Handle(UpdatePlayerCommand request, CancellationToken
 
             playerInDb.FullName = request.Player.FullName;
             playerInDb.Nickname = request.Player.Nickname;
-            playerInDb.ProfilePictureUrl = request.Player.ProfilePictureUrl;
+            playerInDb.ProfilePictureUrl = request.Player.ProfilePictureUrl ?? "https://gameon.valentinvirot.fr/assets/img/gameon-logo.webp";
             playerInDb.KeycloakId = request.Player.KeycloakId;
             playerInDb.Archived = request.Player.Archived;
 

GameOn.Common/DTOs/UpdatePlayerDto.cs

@@ -42,7 +42,7 @@ public class UpdatePlayerDto
         /// <summary>
         /// Gets or sets Profile Picture URL.
         /// </summary>
-        public string ProfilePictureUrl { get; set; } = "INVALID URL";
+        public string? ProfilePictureUrl { get; set; }
 
         /// <summary>
         /// Gets or sets a value indicating whether the player is archived.

GameOn.Documentation/GameOn! API.postman_collection.json

@@ -1,7 +1,7 @@
 {
 	"info": {
 		"_postman_id": "1319992f-a208-4bcb-91f5-9faa28794297",
-		"name": "GameOn! 5.2.2",
+		"name": "GameOn! 5.2.3",
 		"description": "This API goal is to monitor players performance across real-world and virtual games.",
 		"schema": "https://schema.getpostman.com/json/collection/v2.1.0/collection.json",
 		"_exporter_id": "6882080"

GameOn.Documentation/GameOn!.swagger.json

@@ -7,7 +7,7 @@
       "name": "Valentin Virot",
       "email": "virot.valentin@gmail.com"
     },
-    "version": "5.2.2"
+    "version": "5.2.3"
   },
   "paths": {
     "/Admin/dashboard": {

GameOn.Presentation/Controllers/Common/PlayerController.cs

@@ -113,7 +113,7 @@ public async Task<IActionResult> GetPlayerStats(int playerId, int? seasonId)
         }
 
         /// <summary>
-        /// Update user (admin only).
+        /// Update connected user.
         /// </summary>
         /// <param name="update"><see cref="UpdatePlayerDto"/>.</param>
         /// <returns>IActionResult object.</returns>
@@ -135,7 +135,7 @@ public async Task<IActionResult> UpdateConnectedUser([FromBody] UpdatePlayerDto
         }
 
         /// <summary>
-        /// Update connected user.
+        /// Update user (admin only).
         /// </summary>
         /// <param name="update"><see cref="UpdatePlayerDto"/>.</param>
         /// <returns>IActionResult object.</returns>

GameOn.Presentation/Program.cs

@@ -23,7 +23,7 @@
     c.EnableAnnotations();
     c.SwaggerDoc("v1", new Microsoft.OpenApi.Models.OpenApiInfo
     {
-        Version = "5.2.2",
+        Version = "5.2.3",
         Title = "LeadOn's Corp - GameOn! API",
         Description = "This API goal is to monitor players performance across multiple games.",
         Contact = new Microsoft.OpenApi.Models.OpenApiContact