From 7a69c997523839c6374ab7964320a44b5544dd56 Mon Sep 17 00:00:00 2001 From: Sourcery AI Date: Mon, 6 Feb 2023 17:59:59 +0000 Subject: [PATCH] 'Refactored by Sourcery' --- runtests.py | 3 +-- simple_svg/templatetags/svg.py | 10 ++++------ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/runtests.py b/runtests.py index 99c0d62..bd6fc34 100644 --- a/runtests.py +++ b/runtests.py @@ -53,6 +53,5 @@ django.setup() test_runner = DiscoverRunner(verbosity=1) -failures = test_runner.run_tests(["simple_svg"], verbosity=1) -if failures: +if failures := test_runner.run_tests(["simple_svg"], verbosity=1): sys.exit(failures) diff --git a/simple_svg/templatetags/svg.py b/simple_svg/templatetags/svg.py index 277305b..c23b12a 100644 --- a/simple_svg/templatetags/svg.py +++ b/simple_svg/templatetags/svg.py @@ -39,12 +39,10 @@ def svg(filename, *args, **kwargs): if not path: message = "SVG '{filename}.svg' not found".format(filename=filename) - # Raise exception if DEBUG is True, else just log a warning. if settings.DEBUG: raise SVGNotFound(message) - else: - logger.warning(message) - return "" + logger.warning(message) + return "" # Sometimes path can be a list/tuple if there's more than one file found if isinstance(path, (list, tuple)): @@ -54,8 +52,8 @@ def svg(filename, *args, **kwargs): svg = svg_file.read() if kwargs: - attributes = " ".join(['{}="{}"'.format(k, v) for k, v in kwargs.items()]) - svg = svg.replace("