Skip to content

Instantly share code, notes, and snippets.

@EvieePy
Last active March 7, 2024 21:00
Show Gist options
  • Star 81 You must be signed in to star a gist
  • Fork 20 You must be signed in to fork a gist
  • Save EvieePy/ab667b74e9758433b3eb806c53a19f34 to your computer and use it in GitHub Desktop.
Save EvieePy/ab667b74e9758433b3eb806c53a19f34 to your computer and use it in GitHub Desktop.
Basic music with playlist support on Rewrite
"""
Please understand Music bots are complex, and that even this basic example can be daunting to a beginner.
For this reason it's highly advised you familiarize yourself with discord.py, python and asyncio, BEFORE
you attempt to write a music bot.
This example makes use of: Python 3.6
For a more basic voice example please read:
https://github.com/Rapptz/discord.py/blob/rewrite/examples/basic_voice.py
This is a very basic playlist example, which allows per guild playback of unique queues.
The commands implement very basic logic for basic usage. But allow for expansion. It would be advisable to implement
your own permissions and usage logic for commands.
e.g You might like to implement a vote before skipping the song or only allow admins to stop the player.
Music bots require lots of work, and tuning. Goodluck.
If you find any bugs feel free to ping me on discord. @Eviee#0666
"""
import discord
from discord.ext import commands
import asyncio
import itertools
import sys
import traceback
from async_timeout import timeout
from functools import partial
from youtube_dl import YoutubeDL
ytdlopts = {
'format': 'bestaudio/best',
'outtmpl': 'downloads/%(extractor)s-%(id)s-%(title)s.%(ext)s',
'restrictfilenames': True,
'noplaylist': True,
'nocheckcertificate': True,
'ignoreerrors': False,
'logtostderr': False,
'quiet': True,
'no_warnings': True,
'default_search': 'auto',
'source_address': '0.0.0.0' # ipv6 addresses cause issues sometimes
}
ffmpegopts = {
'before_options': '-nostdin',
'options': '-vn'
}
ytdl = YoutubeDL(ytdlopts)
class VoiceConnectionError(commands.CommandError):
"""Custom Exception class for connection errors."""
class InvalidVoiceChannel(VoiceConnectionError):
"""Exception for cases of invalid Voice Channels."""
class YTDLSource(discord.PCMVolumeTransformer):
def __init__(self, source, *, data, requester):
super().__init__(source)
self.requester = requester
self.title = data.get('title')
self.web_url = data.get('webpage_url')
# YTDL info dicts (data) have other useful information you might want
# https://github.com/rg3/youtube-dl/blob/master/README.md
def __getitem__(self, item: str):
"""Allows us to access attributes similar to a dict.
This is only useful when you are NOT downloading.
"""
return self.__getattribute__(item)
@classmethod
async def create_source(cls, ctx, search: str, *, loop, download=False):
loop = loop or asyncio.get_event_loop()
to_run = partial(ytdl.extract_info, url=search, download=download)
data = await loop.run_in_executor(None, to_run)
if 'entries' in data:
# take first item from a playlist
data = data['entries'][0]
await ctx.send(f'```ini\n[Added {data["title"]} to the Queue.]\n```', delete_after=15)
if download:
source = ytdl.prepare_filename(data)
else:
return {'webpage_url': data['webpage_url'], 'requester': ctx.author, 'title': data['title']}
return cls(discord.FFmpegPCMAudio(source), data=data, requester=ctx.author)
@classmethod
async def regather_stream(cls, data, *, loop):
"""Used for preparing a stream, instead of downloading.
Since Youtube Streaming links expire."""
loop = loop or asyncio.get_event_loop()
requester = data['requester']
to_run = partial(ytdl.extract_info, url=data['webpage_url'], download=False)
data = await loop.run_in_executor(None, to_run)
return cls(discord.FFmpegPCMAudio(data['url']), data=data, requester=requester)
class MusicPlayer:
"""A class which is assigned to each guild using the bot for Music.
This class implements a queue and loop, which allows for different guilds to listen to different playlists
simultaneously.
When the bot disconnects from the Voice it's instance will be destroyed.
"""
__slots__ = ('bot', '_guild', '_channel', '_cog', 'queue', 'next', 'current', 'np', 'volume')
def __init__(self, ctx):
self.bot = ctx.bot
self._guild = ctx.guild
self._channel = ctx.channel
self._cog = ctx.cog
self.queue = asyncio.Queue()
self.next = asyncio.Event()
self.np = None # Now playing message
self.volume = .5
self.current = None
ctx.bot.loop.create_task(self.player_loop())
async def player_loop(self):
"""Our main player loop."""
await self.bot.wait_until_ready()
while not self.bot.is_closed():
self.next.clear()
try:
# Wait for the next song. If we timeout cancel the player and disconnect...
async with timeout(300): # 5 minutes...
source = await self.queue.get()
except asyncio.TimeoutError:
return self.destroy(self._guild)
if not isinstance(source, YTDLSource):
# Source was probably a stream (not downloaded)
# So we should regather to prevent stream expiration
try:
source = await YTDLSource.regather_stream(source, loop=self.bot.loop)
except Exception as e:
await self._channel.send(f'There was an error processing your song.\n'
f'```css\n[{e}]\n```')
continue
source.volume = self.volume
self.current = source
self._guild.voice_client.play(source, after=lambda _: self.bot.loop.call_soon_threadsafe(self.next.set))
self.np = await self._channel.send(f'**Now Playing:** `{source.title}` requested by '
f'`{source.requester}`')
await self.next.wait()
# Make sure the FFmpeg process is cleaned up.
source.cleanup()
self.current = None
try:
# We are no longer playing this song...
await self.np.delete()
except discord.HTTPException:
pass
def destroy(self, guild):
"""Disconnect and cleanup the player."""
return self.bot.loop.create_task(self._cog.cleanup(guild))
class Music:
"""Music related commands."""
__slots__ = ('bot', 'players')
def __init__(self, bot):
self.bot = bot
self.players = {}
async def cleanup(self, guild):
try:
await guild.voice_client.disconnect()
except AttributeError:
pass
try:
del self.players[guild.id]
except KeyError:
pass
async def __local_check(self, ctx):
"""A local check which applies to all commands in this cog."""
if not ctx.guild:
raise commands.NoPrivateMessage
return True
async def __error(self, ctx, error):
"""A local error handler for all errors arising from commands in this cog."""
if isinstance(error, commands.NoPrivateMessage):
try:
return await ctx.send('This command can not be used in Private Messages.')
except discord.HTTPException:
pass
elif isinstance(error, InvalidVoiceChannel):
await ctx.send('Error connecting to Voice Channel. '
'Please make sure you are in a valid channel or provide me with one')
print('Ignoring exception in command {}:'.format(ctx.command), file=sys.stderr)
traceback.print_exception(type(error), error, error.__traceback__, file=sys.stderr)
def get_player(self, ctx):
"""Retrieve the guild player, or generate one."""
try:
player = self.players[ctx.guild.id]
except KeyError:
player = MusicPlayer(ctx)
self.players[ctx.guild.id] = player
return player
@commands.command(name='connect', aliases=['join'])
async def connect_(self, ctx, *, channel: discord.VoiceChannel=None):
"""Connect to voice.
Parameters
------------
channel: discord.VoiceChannel [Optional]
The channel to connect to. If a channel is not specified, an attempt to join the voice channel you are in
will be made.
This command also handles moving the bot to different channels.
"""
if not channel:
try:
channel = ctx.author.voice.channel
except AttributeError:
raise InvalidVoiceChannel('No channel to join. Please either specify a valid channel or join one.')
vc = ctx.voice_client
if vc:
if vc.channel.id == channel.id:
return
try:
await vc.move_to(channel)
except asyncio.TimeoutError:
raise VoiceConnectionError(f'Moving to channel: <{channel}> timed out.')
else:
try:
await channel.connect()
except asyncio.TimeoutError:
raise VoiceConnectionError(f'Connecting to channel: <{channel}> timed out.')
await ctx.send(f'Connected to: **{channel}**', delete_after=20)
@commands.command(name='play', aliases=['sing'])
async def play_(self, ctx, *, search: str):
"""Request a song and add it to the queue.
This command attempts to join a valid voice channel if the bot is not already in one.
Uses YTDL to automatically search and retrieve a song.
Parameters
------------
search: str [Required]
The song to search and retrieve using YTDL. This could be a simple search, an ID or URL.
"""
await ctx.trigger_typing()
vc = ctx.voice_client
if not vc:
await ctx.invoke(self.connect_)
player = self.get_player(ctx)
# If download is False, source will be a dict which will be used later to regather the stream.
# If download is True, source will be a discord.FFmpegPCMAudio with a VolumeTransformer.
source = await YTDLSource.create_source(ctx, search, loop=self.bot.loop, download=False)
await player.queue.put(source)
@commands.command(name='pause')
async def pause_(self, ctx):
"""Pause the currently playing song."""
vc = ctx.voice_client
if not vc or not vc.is_playing():
return await ctx.send('I am not currently playing anything!', delete_after=20)
elif vc.is_paused():
return
vc.pause()
await ctx.send(f'**`{ctx.author}`**: Paused the song!')
@commands.command(name='resume')
async def resume_(self, ctx):
"""Resume the currently paused song."""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently playing anything!', delete_after=20)
elif not vc.is_paused():
return
vc.resume()
await ctx.send(f'**`{ctx.author}`**: Resumed the song!')
@commands.command(name='skip')
async def skip_(self, ctx):
"""Skip the song."""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently playing anything!', delete_after=20)
if vc.is_paused():
pass
elif not vc.is_playing():
return
vc.stop()
await ctx.send(f'**`{ctx.author}`**: Skipped the song!')
@commands.command(name='queue', aliases=['q', 'playlist'])
async def queue_info(self, ctx):
"""Retrieve a basic queue of upcoming songs."""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently connected to voice!', delete_after=20)
player = self.get_player(ctx)
if player.queue.empty():
return await ctx.send('There are currently no more queued songs.')
# Grab up to 5 entries from the queue...
upcoming = list(itertools.islice(player.queue._queue, 0, 5))
fmt = '\n'.join(f'**`{_["title"]}`**' for _ in upcoming)
embed = discord.Embed(title=f'Upcoming - Next {len(upcoming)}', description=fmt)
await ctx.send(embed=embed)
@commands.command(name='now_playing', aliases=['np', 'current', 'currentsong', 'playing'])
async def now_playing_(self, ctx):
"""Display information about the currently playing song."""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently connected to voice!', delete_after=20)
player = self.get_player(ctx)
if not player.current:
return await ctx.send('I am not currently playing anything!')
try:
# Remove our previous now_playing message.
await player.np.delete()
except discord.HTTPException:
pass
player.np = await ctx.send(f'**Now Playing:** `{vc.source.title}` '
f'requested by `{vc.source.requester}`')
@commands.command(name='volume', aliases=['vol'])
async def change_volume(self, ctx, *, vol: float):
"""Change the player volume.
Parameters
------------
volume: float or int [Required]
The volume to set the player to in percentage. This must be between 1 and 100.
"""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently connected to voice!', delete_after=20)
if not 0 < vol < 101:
return await ctx.send('Please enter a value between 1 and 100.')
player = self.get_player(ctx)
if vc.source:
vc.source.volume = vol / 100
player.volume = vol / 100
await ctx.send(f'**`{ctx.author}`**: Set the volume to **{vol}%**')
@commands.command(name='stop')
async def stop_(self, ctx):
"""Stop the currently playing song and destroy the player.
!Warning!
This will destroy the player assigned to your guild, also deleting any queued songs and settings.
"""
vc = ctx.voice_client
if not vc or not vc.is_connected():
return await ctx.send('I am not currently playing anything!', delete_after=20)
await self.cleanup(ctx.guild)
def setup(bot):
bot.add_cog(Music(bot))
@RedstonedLife
Copy link

Its with ffmpeg, Its multiple reasons tbh, Its also YouTube, your bandwidth, ffmpeg and discord.py[voice]

@quinton-oglesby
Copy link

How would I modify this to play local files?

@RedstonedLife
Copy link

I wouldn't use this code if I were you, It is good it shows the basics of Discord Music Clients, but in reality this script is jack
YouTube links/gates have expiry times so if you try to play anything after a set amount of 25-30 seconds the gate expiry's

@EvieePy
Copy link
Author

EvieePy commented Mar 6, 2021

I wouldn't use this code if I were you, It is good it shows the basics of Discord Music Clients, but in reality this script is jack
YouTube links/gates have expiry times so if you try to play anything after a set amount of 25-30 seconds the gate expiry's

You do realise it handles that?

@EvieePy
Copy link
Author

EvieePy commented Mar 6, 2021

Although even though it does handle it. I would suggest using Lavalink. If you can handle a few Linux commands and have a VPS to run it on you will have a much better time with running a music bot. You can also checkout my lib, WaveLink, which is a powerful wrapper around Lavalink for discord.py.

@justanothernoob4648
Copy link

Although even though it does handle it. I would suggest using Lavalink. If you can handle a few Linux commands and have a VPS to run it on you will have a much better time with running a music bot. You can also checkout my lib, WaveLink, which is a powerful wrapper around Lavalink for discord.py.

I’m using lavalink, but after a while I get NodeNotFound error

@RedstonedLife
Copy link

how can i add a loop command?

If its loop as in replay Queue. I would suggest just setting queue position to 0

How would you add a loop command for a single song?

Don't clear the queue, and repeat it. even if its 1 song, you can add it to queue. OR
You can play the last-played song at the end of each song, If its repeated it will play the same song.

@RedstonedLife
Copy link

how can i add a loop command?

If its loop as in replay Queue. I would suggest just setting queue position to 0

How would you add a loop command for a single song?

Don't clear the queue, and repeat it. even if its 1 song, you can add it to queue. OR
You can play the last-played song at the end of each song, If its repeated it will play the same song.

I've been trying to code it where it doesn't clear the queue, and it repeats it instead. However, I have been unable to do this. Can you please tell me specifically the code that would be required to do this?

Mate, I'm just saying how you can do it. I'm not gonna now do a full tutorial just for you so you can copy & paste it.
I have enough stuff to do.

@exosane
Copy link

exosane commented Apr 4, 2021

Pls help
Command raised an exception: RuntimeError: PyNaCl library needed in order to ise voice

@RedstonedLife
Copy link

Pls help
Command raised an exception: RuntimeError: PyNaCl library needed in order to ise voice

mate it literally tells you what you need, Install PyNaCI
using python -m pip install PyNaCI

@IsmailSultan
Copy link

so im getting the error in the command play. would appreciate if someone were to explain this to me

Ignoring exception in command play:
Traceback (most recent call last):
  File "/home/mysteri0us/Code/discord bot/music.py", line 232, in get_player
    player = self.players[ctx.guild.id]
KeyError: 835749310712119307

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/home/mysteri0us/Code/discord bot/music.py", line 293, in play_
    player = self.get_player(ctx)
  File "/home/mysteri0us/Code/discord bot/music.py", line 234, in get_player
    player = MusicPlayer(ctx)
NameError: name 'MusicPlayer' is not defined

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: NameError: name 'MusicPlayer' is not defined
Ignoring exception in command play:
Traceback (most recent call last):
  File "/home/mysteri0us/Code/discord bot/music.py", line 232, in get_player
    player = self.players[ctx.guild.id]
KeyError: 835749310712119307

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/home/mysteri0us/Code/discord bot/music.py", line 293, in play_
    player = self.get_player(ctx)
  File "/home/mysteri0us/Code/discord bot/music.py", line 234, in get_player
    player = MusicPlayer(ctx)
NameError: name 'MusicPlayer' is not defined

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: NameError: name 'MusicPlayer' is not defined
^Cmysteri0us@Mysteri0us-Computer:~/Code$ /usr/bin/python3 "/home/mysteri0us/Code/discord bot/index.py"
We have logged in as testiu887878#2253
Ignoring exception in command play:
Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 855, in invoke
    await self.prepare(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 789, in prepare
    await self._parse_arguments(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 706, in _parse_arguments
    kwargs[name] = await self.transform(ctx, param)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 542, in transform
    raise MissingRequiredArgument(param)
discord.ext.commands.errors.MissingRequiredArgument: search is a required argument that is missing.
Ignoring exception in command play:
Traceback (most recent call last):
  File "/home/mysteri0us/Code/discord bot/music.py", line 232, in get_player
    player = self.players[ctx.guild.id]
KeyError: 835749310712119307

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/home/mysteri0us/Code/discord bot/music.py", line 293, in play_
    player = self.get_player(ctx)
  File "/home/mysteri0us/Code/discord bot/music.py", line 234, in get_player
    player = MusicPlayer(ctx)
NameError: name 'MusicPlayer' is not defined

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: NameError: name 'MusicPlayer' is not defined
Ignoring exception in command play:
Traceback (most recent call last):
  File "/home/mysteri0us/Code/discord bot/music.py", line 232, in get_player
    player = self.players[ctx.guild.id]
KeyError: 835749310712119307

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/home/mysteri0us/Code/discord bot/music.py", line 293, in play_
    player = self.get_player(ctx)
  File "/home/mysteri0us/Code/discord bot/music.py", line 234, in get_player
    player = MusicPlayer(ctx)
NameError: name 'MusicPlayer' is not defined

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/home/mysteri0us/.local/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: NameError: name 'MusicPlayer' is not defined

it might be caused due to something in get_player i think

@GGReric
Copy link

GGReric commented Aug 1, 2021

can you do a loop command?

@WarStylez
Copy link

Please make a loop command :)

@JorgeMagdaleno
Copy link

JorgeMagdaleno commented Aug 28, 2021

Hi, I'm having this error
TypeError: 'coroutine' object is not iterable
in the line 358
fmt = '\n'.join(f'**{_["title"]}**' for _ in upcoming)

I tried to search why this happens but most sources I've found say that it must have await outside the function but if I'm not wrong it already is in here, any help is welcome :)

edit.
Nvm apparently any having @bot.command which also iterates list brake the thing. I solved it by encasing those commands in a class and making them @commands.command.

@ryvegit
Copy link

ryvegit commented Sep 28, 2021

I added bot = commands.Bot(prefix='!') and did bot.run with my token at the bottom but when I tried to use the commands I get "no such command" error. What could be causing that?

@JSac1
Copy link

JSac1 commented Oct 9, 2021

fsr it gives me a error: ERROR: No video formats found
can someone help me?

@JustPhantom50
Copy link

Hello, Im using your code which is amazing very smart by the way. Im having some troubles where it brings up that play is not an attribute, i have the error code posted below this. This only happens once and while and its normally in the middle of a song. If someone could please help that would be great.

Task exception was never retrieved
future: <Task finished name='Task-180' coro=<MusicPlayer.player_loop() done, defined at /home/runner/Fusion-Music/cogs/music.py:118> exception=AttributeError("'NoneType' object has no attribute 'play'")>
Traceback (most recent call last):
File "/home/runner/Fusion-Music/cogs/music.py", line 145, in player_loop
self._guild.voice_client.play(source, after=lambda _: self.bot.loop.call_soon_threadsafe(self.next.set))
AttributeError: 'NoneType' object has no attribute 'play'

@davidesidoti
Copy link

Hi, your code works perfectly, fast and easy to use. I'm only having a problem with playing playlists. If I link a playlist with the play command it only queue the first song. Looking at the console, it returns this error:

Ignoring exception in command play:
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/Users/admin/Documents/GitHub/chickennuggetsbot/app.py", line 296, in play_
    source = await YTDLSource.create_source(ctx, search, loop=self.bot.loop, download=False)
  File "/Users/admin/Documents/GitHub/chickennuggetsbot/app.py", line 86, in create_source
    if 'entries' in data:
TypeError: argument of type 'NoneType' is not iterable

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: argument of type 'NoneType' is not iterable

Doing a print of data variable it seems to be None value. What should I do?

@moviebrain
Copy link

Hi, your code works perfectly, fast and easy to use. I'm only having a problem with playing playlists. If I link a playlist with the play command it only queue the first song. Looking at the console, it returns this error:

Ignoring exception in command play:
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "/Users/admin/Documents/GitHub/chickennuggetsbot/app.py", line 296, in play_
    source = await YTDLSource.create_source(ctx, search, loop=self.bot.loop, download=False)
  File "/Users/admin/Documents/GitHub/chickennuggetsbot/app.py", line 86, in create_source
    if 'entries' in data:
TypeError: argument of type 'NoneType' is not iterable

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/bot.py", line 939, in invoke
    await ctx.command.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 863, in invoke
    await injected(*ctx.args, **ctx.kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/discord/ext/commands/core.py", line 94, in wrapped
    raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: argument of type 'NoneType' is not iterable

Doing a print of data variable it seems to be None value. What should I do?

What URL are you passing? There are some types of YouTube playlists that aren’t collections of songs and ytdl doesn’t recognize them as a list to iterate through. I fixed this by being more selective about the urls I passed into it.

@davidesidoti
Copy link

davidesidoti commented Jul 15, 2022

I'm testing with this one:
https://youtube.com/playlist?list=PLov8ST38VcFSYhPl56Ggba12RFA0mjueG
i tried with hidden and public @moviebrain

@moviebrain
Copy link

@davidesidoti I tested it in my fork of music.py and everything loads except the second song in your playlist Jonas Blue. That song appears to be blocked in the USA and is why YTDL is throwing the error. Give music py a minute or two to ingest the whole 91 other songs and it’ll start playing without a problem (or, rather, did for me)

@davidesidoti
Copy link

@moviebrain After waiting a little bit it actually load but queue only the first song for me. I am in Europe, could it be that all the other songs are blocked here?

@moviebrain
Copy link

@davidesidoti this gets off topic for this python program, but you can pass the playlist url to YouTube-dl and list the stream formats for every playlist item. If the song is unavailable from where ever you are it’ll output “Error: Video Unavailable” instead of the webm/mp4/etc info by using the following command in a terminal:

youtube-dl —list-formats {playlist url}

Or you can click each of those individual videos and see if YouTube gets mad.

@davidesidoti
Copy link

@moviebrain I'll try, thank you for your help 😄

@RegretDNM
Copy link

Added a command with bot.add_command(Music.play_), but when I try to use the command, I get

discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: Music.play_() missing 1 required positional argument: 'ctx'

Any ideas?

@into-the-night
Copy link

Added a command with bot.add_command(Music.play_), but when I try to use the command, I get

discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: Music.play_() missing 1 required positional argument: 'ctx'

Any ideas?

Maybe try passing (ctx) as an argument to the function. That's what the error suggests atleast.

@davidesidoti
Copy link

Anyone else have the bot skip to the next song in queue at the middle of the song?

@Wolff1117
Copy link

Anyone else have the bot skip to the next song in queue at the middle of the song?

I am experiencing that also, I have no idea how to fix it. For me it cuts out partially through at roughly the same spot in the song and its different depending on the song, and it does not continue to the next song in the queue.

@moviebrain
Copy link

Anyone else have the bot skip to the next song in queue at the middle of the song?

I am experiencing that also, I have no idea how to fix it. For me it cuts out partially through at roughly the same spot in the song and its different depending on the song, and it does not continue to the next song in the queue.

I believe this behavior is because YouTube has changed how streaming links expire, and the mechanism by which the stream links are regathered in this code no longer work due to this aggressive new policy. I’ve modified the behavior to prefer downloads and play “locally” with ffmpeg transformer instead of streaming links until it’s discovered why streaming can’t be regathered or refreshed.

@JordanSchmelzer
Copy link

Thanks for posting this! This helped me build my music bot cog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment