style(log): more compact

This commit is contained in:
Anhgelus Morhtuuzh 2025-03-10 20:30:19 +01:00
parent 947b3289c8
commit 698292db49
Signed by: anhgelus
GPG key ID: CAD341EFA92DDDE5
2 changed files with 70 additions and 111 deletions

View file

@ -43,12 +43,9 @@ func OnMessage(s *discordgo.Session, m *discordgo.MessageCreate) {
c.AddXP(s, m.Member, xp, func(_ uint, _ uint) { c.AddXP(s, m.Member, xp, func(_ uint, _ uint) {
if err := s.MessageReactionAdd(m.ChannelID, m.Message.ID, "⬆"); err != nil { if err := s.MessageReactionAdd(m.ChannelID, m.Message.ID, "⬆"); err != nil {
utils.SendAlert( utils.SendAlert(
"xp/events.go - add reaction for new level", "xp/events.go - add reaction for new level", err.Error(),
err.Error(), "channel id", m.ChannelID,
"channel id", "message id", m.Message.ID,
m.ChannelID,
"message id",
m.Message.ID,
) )
} }
}) })
@ -110,8 +107,7 @@ func onDisconnect(s *discordgo.Session, e *discordgo.VoiceStateUpdate, client *r
// check validity of user (1) // check validity of user (1)
if errors.Is(res.Err(), redis.Nil) { if errors.Is(res.Err(), redis.Nil) {
utils.SendWarn(fmt.Sprintf( utils.SendWarn(fmt.Sprintf(
"User %s diconnect from a vocal but does not have a connected_since", "User %s diconnect from a vocal but does not have a connected_since", e.Member.DisplayName(),
e.Member.DisplayName(),
)) ))
return return
} }
@ -131,8 +127,7 @@ func onDisconnect(s *discordgo.Session, e *discordgo.VoiceStateUpdate, client *r
// check validity of user (2) // check validity of user (2)
if con == NotConnected { if con == NotConnected {
utils.SendWarn(fmt.Sprintf( utils.SendWarn(fmt.Sprintf(
"User %s diconnect from a vocal but was registered as not connected", "User %s diconnect from a vocal but was registered as not connected", e.Member.DisplayName(),
e.Member.DisplayName(),
)) ))
return return
} }
@ -155,9 +150,7 @@ func onDisconnect(s *discordgo.Session, e *discordgo.VoiceStateUpdate, client *r
c.AddXP(s, e.Member, XPVocal(uint(timeInVocal)), func(_ uint, newLevel uint) { c.AddXP(s, e.Member, XPVocal(uint(timeInVocal)), func(_ uint, newLevel uint) {
cfg := config.GetGuildConfig(e.GuildID) cfg := config.GetGuildConfig(e.GuildID)
_, err = s.ChannelMessageSend(cfg.FallbackChannel, fmt.Sprintf( _, err = s.ChannelMessageSend(cfg.FallbackChannel, fmt.Sprintf(
"%s est maintenant niveau %d", "%s est maintenant niveau %d", e.Member.Mention(), newLevel,
e.Member.Mention(),
newLevel,
)) ))
if err != nil { if err != nil {
utils.SendAlert("xp/events.go - Sending new level in fallback channel", err.Error()) utils.SendAlert("xp/events.go - Sending new level in fallback channel", err.Error())
@ -168,5 +161,11 @@ func onDisconnect(s *discordgo.Session, e *discordgo.VoiceStateUpdate, client *r
func OnLeave(_ *discordgo.Session, e *discordgo.GuildMemberRemove) { func OnLeave(_ *discordgo.Session, e *discordgo.GuildMemberRemove) {
utils.SendDebug("Leave event", "user_id", e.User.ID) utils.SendDebug("Leave event", "user_id", e.User.ID)
c := GetCopaing(e.User.ID, e.GuildID) c := GetCopaing(e.User.ID, e.GuildID)
gokord.DB.Where("guild_id = ?", e.GuildID).Delete(c) if err := gokord.DB.Where("guild_id = ?", e.GuildID).Delete(c).Error; err != nil {
utils.SendAlert(
"xp/events.go - deleting copaing from db", err.Error(),
"user_id", e.User.ID,
"guild_id", e.GuildID,
)
}
} }

View file

@ -10,35 +10,29 @@ import (
"time" "time"
) )
func onNewLevel(s *discordgo.Session, m *discordgo.Member, level uint) { func onNewLevel(dg *discordgo.Session, m *discordgo.Member, level uint) {
cfg := config.GetGuildConfig(m.GuildID) cfg := config.GetGuildConfig(m.GuildID)
xpForLevel := XPForLevel(level) xpForLevel := XPForLevel(level)
for _, role := range cfg.XpRoles { for _, role := range cfg.XpRoles {
if role.XP <= xpForLevel && !slices.Contains(m.Roles, role.RoleID) { if role.XP <= xpForLevel && !slices.Contains(m.Roles, role.RoleID) {
utils.SendDebug( utils.SendDebug(
"Add role", "Add role",
"role_id", "role_id", role.RoleID,
role.RoleID, "user_id", m.User.ID,
"user_id", "guild_id", m.GuildID,
m.User.ID,
"guild_id",
m.GuildID,
) )
err := s.GuildMemberRoleAdd(m.GuildID, m.User.ID, role.RoleID) err := dg.GuildMemberRoleAdd(m.GuildID, m.User.ID, role.RoleID)
if err != nil { if err != nil {
utils.SendAlert("xp/level.go - Adding role", err.Error(), "role_id", role.RoleID) utils.SendAlert("xp/level.go - Adding role", err.Error(), "role_id", role.RoleID)
} }
} else if role.XP > xpForLevel && slices.Contains(m.Roles, role.RoleID) { } else if role.XP > xpForLevel && slices.Contains(m.Roles, role.RoleID) {
utils.SendDebug( utils.SendDebug(
"Remove role", "Remove role",
"role_id", "role_id", role.RoleID,
role.RoleID, "user_id", m.User.ID,
"user_id", "guild_id", m.GuildID,
m.User.ID,
"guild_id",
m.GuildID,
) )
err := s.GuildMemberRoleRemove(m.GuildID, m.User.ID, role.RoleID) err := dg.GuildMemberRoleRemove(m.GuildID, m.User.ID, role.RoleID)
if err != nil { if err != nil {
utils.SendAlert("xp/level.go - Removing role", err.Error(), "role_id", role.RoleID) utils.SendAlert("xp/level.go - Removing role", err.Error(), "role_id", role.RoleID)
} }
@ -46,23 +40,20 @@ func onNewLevel(s *discordgo.Session, m *discordgo.Member, level uint) {
} }
} }
func (c *Copaing) OnNewLevel(s *discordgo.Session, level uint) { func (c *Copaing) OnNewLevel(dg *discordgo.Session, level uint) {
m, err := s.GuildMember(c.GuildID, c.DiscordID) m, err := dg.GuildMember(c.GuildID, c.DiscordID)
if err != nil { if err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Getting member for new level", "xp/level.go - Getting member for new level", err.Error(),
err.Error(), "discord_id", c.DiscordID,
"discord_id", "guild_id", c.GuildID,
c.DiscordID,
"guild_id",
c.GuildID,
) )
return return
} }
onNewLevel(s, m, level) onNewLevel(dg, m, level)
} }
func LastEventUpdate(s *discordgo.Session, c *Copaing) { func LastEventUpdate(dg *discordgo.Session, c *Copaing) {
h := c.HourSinceLastEvent() h := c.HourSinceLastEvent()
l := Lose(h, c.XP) l := Lose(h, c.XP)
xp := c.XPAlreadyRemoved() xp := c.XPAlreadyRemoved()
@ -83,34 +74,26 @@ func LastEventUpdate(s *discordgo.Session, c *Copaing) {
if Level(oldXP) != lvl { if Level(oldXP) != lvl {
utils.SendDebug( utils.SendDebug(
"Level changed", "Level changed",
"old", "old", Level(oldXP),
Level(oldXP), "new", lvl,
"new", "discord_id", c.DiscordID,
lvl, "guild_id", c.GuildID,
"discord_id",
c.DiscordID,
"guild_id",
c.GuildID,
) )
c.OnNewLevel(s, lvl) c.OnNewLevel(dg, lvl)
} }
if err := c.Save(); err != nil { if err := c.Save(); err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Saving copaing", "xp/level.go - Saving copaing", err.Error(),
err.Error(), "xp", c.XP,
"xp", "discord_id", c.DiscordID,
c.XP, "guild_id", c.GuildID,
"discord_id",
c.DiscordID,
"guild_id",
c.GuildID,
) )
} }
} }
c.SetLastEvent() c.SetLastEvent()
} }
func XPUpdate(s *discordgo.Session, c *Copaing) { func XPUpdate(dg *discordgo.Session, c *Copaing) {
oldXP := c.XP oldXP := c.XP
if oldXP == 0 { if oldXP == 0 {
return return
@ -136,36 +119,28 @@ func XPUpdate(s *discordgo.Session, c *Copaing) {
if Level(oldXP) != lvl { if Level(oldXP) != lvl {
utils.SendDebug( utils.SendDebug(
"Level updated", "Level updated",
"old", "old", Level(oldXP),
Level(oldXP), "new", lvl,
"new", "discord_id", c.DiscordID,
lvl, "guild_id", c.GuildID,
"discord_id",
c.DiscordID,
"guild_id",
c.GuildID,
) )
c.OnNewLevel(s, lvl) c.OnNewLevel(dg, lvl)
} }
utils.SendDebug("Save XP", "old", oldXP, "new", c.XP, "user", c.DiscordID) utils.SendDebug("Save XP", "old", oldXP, "new", c.XP, "user", c.DiscordID)
if err := c.Save(); err != nil { if err := c.Save(); err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Saving copaing", "xp/level.go - Saving copaing", err.Error(),
err.Error(), "xp", c.XP,
"xp", "discord_id", c.DiscordID,
c.XP, "guild_id", c.GuildID,
"discord_id",
c.DiscordID,
"guild_id",
c.GuildID,
) )
} }
} }
} }
func PeriodicReducer(s *discordgo.Session) { func PeriodicReducer(dg *discordgo.Session) {
var wg sync.WaitGroup var wg sync.WaitGroup
for _, g := range s.State.Guilds { for _, g := range dg.State.Guilds {
var cs []*Copaing var cs []*Copaing
err := gokord.DB.Where("guild_id = ?", g.ID).Find(&cs).Error err := gokord.DB.Where("guild_id = ?", g.ID).Find(&cs).Error
if err != nil { if err != nil {
@ -177,25 +152,19 @@ func PeriodicReducer(s *discordgo.Session) {
time.Sleep(15 * time.Second) // sleep prevents from spamming the Discord API and the database time.Sleep(15 * time.Second) // sleep prevents from spamming the Discord API and the database
} }
var u *discordgo.User var u *discordgo.User
u, err = s.User(c.DiscordID) u, err = dg.User(c.DiscordID)
if err != nil { if err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Fetching user", "xp/level.go - Fetching user", err.Error(),
err.Error(), "discord_id", c.DiscordID,
"discord_id", "guild_id", g.ID,
c.DiscordID,
"guild_id",
g.ID,
) )
utils.SendWarn("Removing user from database", "discord_id", c.DiscordID) utils.SendWarn("Removing user from database", "discord_id", c.DiscordID)
if gokord.DB.Delete(c).Error != nil { if err = gokord.DB.Delete(c).Error; err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Removing user from database", "xp/level.go - Removing user from database", err.Error(),
err.Error(), "discord_id", c.DiscordID,
"discord_id", "guild_id", g.ID,
c.DiscordID,
"guild_id",
g.ID,
) )
} }
continue continue
@ -203,38 +172,29 @@ func PeriodicReducer(s *discordgo.Session) {
if u.Bot { if u.Bot {
continue continue
} }
_, err = s.GuildMember(g.ID, c.DiscordID) if _, err = dg.GuildMember(g.ID, c.DiscordID); err != nil {
if err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Fetching member", "xp/level.go - Fetching member", err.Error(),
err.Error(), "discord_id", c.DiscordID,
"discord_id", "guild_id", g.ID,
c.DiscordID,
"guild_id",
g.ID,
) )
utils.SendWarn( utils.SendWarn(
"Removing user from guild in database", "Removing user from guild in database",
"discord_id", "discord_id", c.DiscordID,
c.DiscordID, "guild_id", g.ID,
"guild_id",
g.ID,
) )
if gokord.DB.Where("guild_id = ?", g.ID).Delete(c).Error != nil { if err = gokord.DB.Where("guild_id = ?", g.ID).Delete(c).Error; err != nil {
utils.SendAlert( utils.SendAlert(
"xp/level.go - Removing user from guild in database", "xp/level.go - Removing user from guild in database", err.Error(),
err.Error(), "discord_id", c.DiscordID,
"discord_id", "guild_id", g.ID,
c.DiscordID,
"guild_id",
g.ID,
) )
} }
continue continue
} }
wg.Add(1) wg.Add(1)
go func() { go func() {
XPUpdate(s, c) XPUpdate(dg, c)
wg.Done() wg.Done()
}() }()
} }
@ -242,5 +202,5 @@ func PeriodicReducer(s *discordgo.Session) {
utils.SendDebug("Periodic reduce, guild finished", "guild", g.Name) utils.SendDebug("Periodic reduce, guild finished", "guild", g.Name)
time.Sleep(15 * time.Second) // sleep prevents from spamming the Discord API and the database time.Sleep(15 * time.Second) // sleep prevents from spamming the Discord API and the database
} }
utils.SendDebug("Periodic reduce finished", "len(guilds)", len(s.State.Guilds)) utils.SendDebug("Periodic reduce finished", "len(guilds)", len(dg.State.Guilds))
} }