From 6223d4cf16728715a2187ee5b6df1810fe74f89a Mon Sep 17 00:00:00 2001 From: Daniel Reed Date: Mon, 13 May 2019 15:40:10 -0700 Subject: [PATCH] Ignore whitespace-only lines in addition to purely blank lines. Fixes #38. --- metabot/modules/echo.py | 2 +- metabot/modules/test_echo.py | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/metabot/modules/echo.py b/metabot/modules/echo.py index 7552e5e..b157114 100644 --- a/metabot/modules/echo.py +++ b/metabot/modules/echo.py @@ -18,7 +18,7 @@ def moddispatch(ctx, msg, modconf): # pylint: disable=missing-docstring def echo(ctx, msg, message): # pylint: disable=missing-docstring - lines = [line for line in message.splitlines() if line] + lines = [line for line in message.splitlines() if line.strip()] page = ctx.text.isdigit() and int(ctx.text) or 1 for line in lines[:page]: msg.add('%s', line) diff --git a/metabot/modules/test_echo.py b/metabot/modules/test_echo.py index f4ff7f8..0e2d35e 100644 --- a/metabot/modules/test_echo.py +++ b/metabot/modules/test_echo.py @@ -31,6 +31,7 @@ def test_echo(conversation): # pylint: disable=redefined-outer-name conversation.multibot.conf['bots']['modulestestbot']['issue37']['echo']['about'] = ( 'First line.\n' 'Second line.\n' + ' \n' 'Last line.') assert conversation.message('/about') == """\