Skip to content

Commit

Permalink
fix(lyrics): crash in some circumstances
Browse files Browse the repository at this point in the history
  • Loading branch information
zapteryx committed Jul 1, 2024
1 parent c639d13 commit d7c7a77
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/commands/lyrics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ export default {
romanizeFrom = 'chinese';
}
const lyricsFields = generateEmbedFieldsFromLyrics(json, lyrics);
if (lyricsFields.length === 0) {
if (lyricsFields.length === 0 || !lyricsFields[0].value) {
await interaction.replyHandler.locale(
'CMD.LYRICS.RESPONSE.NO_RESULTS',
{ type: MessageOptionsBuilderType.Error },
Expand Down
2 changes: 1 addition & 1 deletion src/components/buttons/lyrics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ export default {
.join('\n');
}
const lyricsFields = generateEmbedFieldsFromLyrics(json, lyrics);
if (lyricsFields.length === 0) {
if (lyricsFields.length === 0 || !lyricsFields[0].value) {
await interaction.replyHandler.locale(
'CMD.LYRICS.RESPONSE.NO_RESULTS',
{ type: MessageOptionsBuilderType.Error },
Expand Down
2 changes: 1 addition & 1 deletion src/events/music/trackStart.ts
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ export default {
json,
lyrics,
);
if (lyricsFields.length === 0) return;
if (lyricsFields.length === 0 || !lyricsFields[0].value) return;
await queue.player.handler.send(
new EmbedBuilder().setFields(lyricsFields),
{
Expand Down

0 comments on commit d7c7a77

Please sign in to comment.