TeamInklings/Octobot
TeamInklings/Octobot
1
0
Fork 1
mirror of https://github.com/TeamOctolings/Octobot.git synced 2025-01-31 09:09:00 +03:00
Code Issues Wiki Activity

Sanitize scheduled event names (#159)

Browse source 
In this PR, I added `Markdown.Sanitize` for every `scheduledEvent.Name`
and `eventData.Name`. That means, every title in scheduled event embeds
will be sanitized.

I did that because in scheduled event list event title displays only as
sanitized string.

Signed-off-by: Macintosh II <mctaylxrs@outlook.com>
This commit is contained in:
Macintxsh 2023-10-12 13:56:24 +03:00 committed by GitHub
parent e21cefd79d
commit 4d526ad32f
Signed by: GitHub
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/Services/Update/ScheduledEventUpdateService.cs
View file

@ -200,7 +200,7 @@ public sealed class ScheduledEventUpdateService : BackgroundService
var embed = new EmbedBuilder() var embed = new EmbedBuilder()
.WithSmallTitle(string.Format(Messages.EventCreatedTitle, creator.GetTag()), creator) .WithSmallTitle(string.Format(Messages.EventCreatedTitle, creator.GetTag()), creator)
.WithTitle(scheduledEvent.Name) .WithTitle(Markdown.Sanitize(scheduledEvent.Name))
.WithDescription(embedDescription) .WithDescription(embedDescription)
.WithEventCover(scheduledEvent.ID, scheduledEvent.Image) .WithEventCover(scheduledEvent.ID, scheduledEvent.Image)
.WithCurrentTimestamp() .WithCurrentTimestamp()
@ -296,7 +296,7 @@ public sealed class ScheduledEventUpdateService : BackgroundService
return Result.FromError(embedDescriptionResult); return Result.FromError(embedDescriptionResult);
} }
var startedEmbed = new EmbedBuilder().WithTitle(string.Format(Messages.EventStarted, scheduledEvent.Name)) var startedEmbed = new EmbedBuilder().WithTitle(string.Format(Messages.EventStarted, Markdown.Sanitize(scheduledEvent.Name)))
.WithDescription(embedDescription) .WithDescription(embedDescription)
.WithColour(ColorsList.Green) .WithColour(ColorsList.Green)
.WithCurrentTimestamp() .WithCurrentTimestamp()
@ -321,7 +321,7 @@ public sealed class ScheduledEventUpdateService : BackgroundService
return Result.FromSuccess(); return Result.FromSuccess();
} }
var completedEmbed = new EmbedBuilder().WithTitle(string.Format(Messages.EventCompleted, eventData.Name)) var completedEmbed = new EmbedBuilder().WithTitle(string.Format(Messages.EventCompleted, Markdown.Sanitize(eventData.Name)))
.WithDescription( .WithDescription(
string.Format( string.Format(
Messages.EventDuration, Messages.EventDuration,
@ -358,7 +358,7 @@ public sealed class ScheduledEventUpdateService : BackgroundService
} }
var embed = new EmbedBuilder() var embed = new EmbedBuilder()
.WithSmallTitle(string.Format(Messages.EventCancelled, eventData.Name)) .WithSmallTitle(string.Format(Messages.EventCancelled, Markdown.Sanitize(eventData.Name)))
.WithDescription(":(") .WithDescription(":(")
.WithColour(ColorsList.Red) .WithColour(ColorsList.Red)
.WithCurrentTimestamp() .WithCurrentTimestamp()
@ -419,7 +419,7 @@ public sealed class ScheduledEventUpdateService : BackgroundService
var earlyResult = new EmbedBuilder() var earlyResult = new EmbedBuilder()
.WithDescription( .WithDescription(
string.Format(Messages.EventEarlyNotification, scheduledEvent.Name, string.Format(Messages.EventEarlyNotification, Markdown.Sanitize(scheduledEvent.Name),
Markdown.Timestamp(scheduledEvent.ScheduledStartTime, TimestampStyle.RelativeTime))) Markdown.Timestamp(scheduledEvent.ScheduledStartTime, TimestampStyle.RelativeTime)))
.WithColour(ColorsList.Default) .WithColour(ColorsList.Default)
.Build(); .Build();