CopyBot/Red-DiscordBot
1
0
Fork 0
mirror of https://github.com/Cog-Creators/Red-DiscordBot.git synced 2025-11-06 03:08:55 -05:00
Code Issues Packages Projects Releases Wiki Activity

[V3 Streams] Duplicate stream alerts fix (#1244)

Browse Source 
case insensitive on twitch streams, with a fast dedupe on load toany prior bad entries.
This commit is contained in:
Michael H 2018-01-14 23:17:57 -05:00 committed by palmtree5
parent 4ac606b468
commit f1a20e1677
1 changed files with 21 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
redbot/cogs/streams/streams.py
View File

@ -108,13 +108,13 @@ class Streams:
@_twitch.command(name="channel")
async def twitch_alert_channel(self, ctx: RedContext, channel_name: str):
"""Sets a Twitch stream alert notification in the channel"""
await self.stream_alert(ctx, TwitchStream, channel_name)
await self.stream_alert(ctx, TwitchStream, channel_name.lower())
@_twitch.command(name="community")
async def twitch_alert_community(self, ctx: RedContext, community: str):
"""Sets a Twitch stream alert notification in the channel
for the specified community."""
await self.community_alert(ctx, TwitchCommunity, community)
await self.community_alert(ctx, TwitchCommunity, community.lower())
@streamalert.command(name="hitbox")
async def hitbox_alert(self, ctx, channel_name: str):
@ -171,7 +171,7 @@ class Streams:
for stream in self.streams:
for channel_id in stream.channels:
if channel_id in guild_channels_ids:
streams_list[channel_id].append(stream.name)
streams_list[channel_id].append(stream.name.lower())
if not streams_list:
await ctx.send("There are no active stream alerts in this server.")
@ -185,7 +185,7 @@ class Streams:
await ctx.send(page)
async def stream_alert(self, ctx, _class, channel_name):
stream = self.get_stream(_class, channel_name)
stream = self.get_stream(_class, channel_name.lower())
if not stream:
token = await self.db.tokens.get_attr(_class.__name__)
stream = _class(name=channel_name,
@ -365,12 +365,12 @@ class Streams:
# isinstance will always return False
# As a workaround, we'll compare the class' name instead.
# Good enough.
if stream.type == _class.__name__ and stream.name == name:
if stream.type == _class.__name__ and stream.name.lower() == name.lower():
return stream
def get_community(self, _class, name):
for community in self.communities:
if community.type == _class.__name__ and community.name == name:
if community.type == _class.__name__ and community.name.lower() == name.lower():
return community
async def check_exists(self, stream):
@ -480,7 +480,14 @@ class Streams:
token = await self.db.tokens.get_attr(_class.__name__)
streams.append(_class(token=token, **raw_stream))
return streams
# issue 1191 extended resolution: Remove this after suitable period
# Fast dedupe below
seen = set()
seen_add = seen.add
return [x for x in streams
if not (x.name.lower() in seen or seen_add(x.name.lower()))]
# return streams
async def load_communities(self):
communities = []
@ -493,7 +500,13 @@ class Streams:
token = await self.db.tokens.get_attr(_class.__name__)
communities.append(_class(token=token, **raw_community))
return communities
# issue 1191 extended resolution: Remove this after suitable period
# Fast dedupe below
seen = set()
seen_add = seen.add
return [x for x in communities
if not (x.name.lower() in seen or seen_add(x.name.lower()))]
# return communities
async def save_streams(self):
raw_streams = []