Skip to content

Commit

Permalink
Notify a user when their highscore is beaten (#1083)
Browse files Browse the repository at this point in the history
* Notify a user when their highscore is beaten

* formatting

Co-authored-by: sudokoko <sudo@koko.rip>

* noPrefix -> prefix

* documentation and fix closing tag

* better documentation

---------

Co-authored-by: sudokoko <sudo@koko.rip>
Co-authored-by: Josh <josh@slendy.pw>
  • Loading branch information
3 people authored Jan 11, 2025
1 parent 301e5d0 commit 4f9dc2a
Show file tree
Hide file tree
Showing 4 changed files with 41 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
using LBPUnion.ProjectLighthouse.Logging;
using LBPUnion.ProjectLighthouse.StorableLists.Stores;
using LBPUnion.ProjectLighthouse.Types.Entities.Level;
using LBPUnion.ProjectLighthouse.Types.Entities.Profile;
using LBPUnion.ProjectLighthouse.Types.Entities.Token;
using LBPUnion.ProjectLighthouse.Types.Levels;
using LBPUnion.ProjectLighthouse.Types.Logging;
Expand Down Expand Up @@ -159,7 +160,7 @@ public async Task<IActionResult> SubmitScore(string slotType, int id, int childI

await this.database.SaveChangesAsync();

return this.Ok(await this.GetScores(new LeaderboardOptions
ScoreboardResponse scores = await this.GetScores(new LeaderboardOptions
{
RootName = "scoreboardSegment",
PageSize = 5,
Expand All @@ -169,7 +170,17 @@ public async Task<IActionResult> SubmitScore(string slotType, int id, int childI
ScoreType = score.Type,
TargetUser = token.UserId,
TargetPlayerIds = null,
}));
});

if (score.Type == 1 && scores.YourRank == 1 && scores.Total > 1)

Check notice on line 175 in ProjectLighthouse.Servers.GameServer/Controllers/Slots/ScoreController.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Invert 'if' statement to reduce nesting

Invert 'if' statement to reduce nesting

Check notice on line 175 in ProjectLighthouse.Servers.GameServer/Controllers/Slots/ScoreController.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Merge null/pattern checks into complex pattern

Merge into pattern
{
GameScore? second = scores.Scores[1];
UserEntity? user = await this.database.UserFromGameToken(token);

await this.database.SendNotification(second.UserId, $"{user?.InfoXml} beat your highscore (<em>{second.Points}</em>) on {slot.InfoXml} with a score of <em>{score.Points}</em>.", true);

Check warning on line 180 in ProjectLighthouse.Servers.GameServer/Controllers/Slots/ScoreController.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Redundant argument with default value

The parameter 'prefix' has the same default value
}

return this.Ok(scores);
}

[HttpGet("scoreboard/{slotType}/{id:int}")]
Expand Down
23 changes: 14 additions & 9 deletions ProjectLighthouse/Database/DatabaseContext.Notifications.cs
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,10 @@ public partial class DatabaseContext
/// </summary>
/// <param name="userId">The user ID of the target user.</param>
/// <param name="text">The message to send.</param>
/// <param name="prefix">Prepend server name/timestamp.</param>
/// <param name="type">The <see cref="NotificationType"/> for the notification. Defaults to <c>ModerationNotification</c>.</param>
public async Task SendNotification
(int userId, string text, NotificationType type = NotificationType.ModerationNotification)
(int userId, string text, bool prefix = true, NotificationType type = NotificationType.ModerationNotification)
{
if (!await this.Users.AnyAsync(u => u.UserId == userId))
{
Expand All @@ -31,15 +32,19 @@ public async Task SendNotification

StringBuilder builder = new(text);

// Prepend server name to notification text if enabled
if (ServerConfiguration.Instance.NotificationConfiguration.ShowServerNameInText)
if (prefix)
{
builder.Insert(0, $"[{ServerConfiguration.Instance.Customization.ServerName}] ");
}
// Prepend timestamp to notification text if enabled
if (ServerConfiguration.Instance.NotificationConfiguration.ShowTimestampInText)
{
builder.Insert(0, $"[{DateTime.Now:g}] ");
// Prepend server name to notification text if enabled
if (ServerConfiguration.Instance.NotificationConfiguration.ShowServerNameInText)
{
builder.Insert(0, $"[{ServerConfiguration.Instance.Customization.ServerName}] ");
}

// Prepend timestamp to notification text if enabled
if (ServerConfiguration.Instance.NotificationConfiguration.ShowTimestampInText)
{
builder.Insert(0, $"[{DateTime.Now:g}] ");
}
}

NotificationEntity notification = new()
Expand Down
7 changes: 7 additions & 0 deletions ProjectLighthouse/Types/Entities/Level/SlotEntity.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,13 @@ public class SlotEntity

public string IconHash { get; set; } = "";

Check warning on line 30 in ProjectLighthouse/Types/Entities/Level/SlotEntity.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Possible performance issues caused by unlimited string length

Possible performance issues caused by unlimited string length

/// <summary>
/// Markup that displays the level name next to its badge.
/// This can be used everywhere markup works ingame, e.g. news or notifications
/// </summary>
[NotMapped]
public string InfoXml => $"<slot type=\"{this.Type}\" id=\"{this.SlotId}\" icon=\"{this.IconHash}\">{this.Name}</slot>";

public bool IsAdventurePlanet { get; set; }

public string RootLevel { get; set; } = "";

Check warning on line 41 in ProjectLighthouse/Types/Entities/Level/SlotEntity.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Possible performance issues caused by unlimited string length

Possible performance issues caused by unlimited string length
Expand Down
7 changes: 7 additions & 0 deletions ProjectLighthouse/Types/Entities/Profile/UserEntity.cs
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,13 @@ public class UserEntity

public string IconHash { get; set; }

Check warning on line 29 in ProjectLighthouse/Types/Entities/Profile/UserEntity.cs

View workflow job for this annotation

GitHub Actions / Qodana for .NET

Possible performance issues caused by unlimited string length

Possible performance issues caused by unlimited string length

/// <summary>
/// Markup that displays the username next to a polaroid with the user's icon.
/// This can be used everywhere markup works ingame, e.g. news or notifications
/// </summary>
[NotMapped]
public string InfoXml => $"<player icon=\"{this.IconHash}\">{this.Username}</player>";

/// <summary>
/// A user-customizable biography shown on the profile card
/// </summary>
Expand Down

0 comments on commit 4f9dc2a

Please sign in to comment.