Skip to content

Commit

Permalink
tell: avoid errors produced by empty tellee
Browse files Browse the repository at this point in the history
Co-authored-by: bronzeguy <[email protected]>
Co-authored-by: dgw <[email protected]>
  • Loading branch information
3 people committed Dec 11, 2023
1 parent 6a25d17 commit 47e30a7
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions sopel/builtins/tell.py
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,14 @@ def f_remind(bot, trigger):
verb = trigger.group(1)

if not trigger.group(3):
tellee = ""
else:
tellee = trigger.group(3).rstrip('.,:;').lstrip('@')

if not tellee:
bot.reply("%s whom?" % verb)
return

tellee = trigger.group(3).rstrip('.,:;')

# all we care about is having at least one non-whitespace
# character after the name
if not trigger.group(4):
Expand All @@ -204,9 +207,6 @@ def f_remind(bot, trigger):
bot.reply('That nickname is too long.')
return

if tellee[0] == '@':
tellee = tellee[1:]

if tellee == bot.nick:
bot.reply("I'm here now; you can %s me whatever you want!" % verb)
return
Expand Down

0 comments on commit 47e30a7

Please sign in to comment.