Sourcery refactored master branch by sourcery-ai[bot] · Pull Request #65 · CustomIcon/LyndaRobot

sourcery-ai · 2020-07-19T20:00:28Z

Branch master refactored by Sourcery.

If you're happy with these changes, merge this Pull Request using the Squash and merge strategy.

See our documentation here.

Run Sourcery locally

Reduce the feedback loop during development by using the Sourcery editor plugin:

Review changes via command line

To manually merge these changes, make sure you're on the master branch, then run:

git fetch origin sourcery/master
git merge --ff-only FETCH_HEAD
git reset HEAD^

sourcery-ai

Due to GitHub API limits, only the first 60 comments can be shown.

sourcery-ai · 2020-07-19T20:00:29Z

lynda/__main__.py

        imported_module.__mod_name__ = imported_module.__name__

-    if not imported_module.__mod_name__.lower() in IMPORTED:
+    if imported_module.__mod_name__.lower() not in IMPORTED:


Lines 75-75 refactored with the following changes:

Simplify logical expression using De Morgan identities (de-morgan)

sourcery-ai · 2020-07-19T20:00:29Z

lynda/__main__.py

-        if excp.message == "Message is not modified":
-            pass
-        elif excp.message == "Query_id_invalid":
-            pass
-        elif excp.message == "Message can't be deleted":
-            pass
-        else:
+        if (
+            excp.message != "Message is not modified"
+            and excp.message != "Query_id_invalid"
+            and excp.message != "Message can't be deleted"
+        ):


Function help_button refactored with the following changes:

Merge duplicate blocks in conditional (merge-duplicate-blocks)

Remove redundant conditional (remove-redundant-if)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:29Z

lynda/__main__.py

-        if excp.message == "Message is not modified":
-            pass
-        elif excp.message == "Query_id_invalid":
-            pass
-        elif excp.message == "Message can't be deleted":
-            pass
-        else:
+        if (
+            excp.message != "Message is not modified"
+            and excp.message != "Query_id_invalid"
+            and excp.message != "Message can't be deleted"
+        ):


Function settings_button refactored with the following changes:

Merge duplicate blocks in conditional (merge-duplicate-blocks)

Remove redundant conditional (remove-redundant-if)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:29Z

lynda/modules/admin.py

        return log_message

-    if user_member.status == 'administrator' or user_member.status == 'creator':
+    if user_member.status in ['administrator', 'creator']:


Function promote refactored with the following changes:

Hoist repeated code outside conditional statement (hoist-statement-from-if)

Replace multiple comparisons of same variable with in operator (merge-comparisons)

sourcery-ai · 2020-07-19T20:00:29Z

lynda/modules/admin.py

-    if not user_member.status == 'administrator':
+    if user_member.status != 'administrator':


Function demote refactored with the following changes:

Simplify logical expression using De Morgan identities (de-morgan)

sourcery-ai · 2020-07-19T20:00:32Z

lynda/modules/blacklistusers.py

-    else:
-        message += '\n'.join(users)
-
+    message += '\n'.join(users) if users else "Noone is being ignored as of yet."


Function bl_users refactored with the following changes:

Ensure first condition in if is positive (swap-if-else-branches)

Replace if statement with if expression (assign-if-exp)

sourcery-ai · 2020-07-19T20:00:32Z

lynda/modules/cleaner.py

-else:
-    CMD_STARTERS = '/'
-
+CMD_STARTERS = ('/', '!') if ALLOW_EXCL else '/'


Lines 13-17 refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

sourcery-ai · 2020-07-19T20:00:32Z

lynda/modules/cleaner.py

-    message = update.effective_message
+    if chat.get_member(bot.id).can_delete_messages and sql.is_enabled(chat.id):
+        message = update.effective_message

-    if chat.get_member(bot.id).can_delete_messages:
-        if sql.is_enabled(chat.id):
-            fst_word = message.text.strip().split(None, 1)[0]
+        fst_word = message.text.strip().split(None, 1)[0]

-            if len(fst_word) > 1 and any(fst_word.startswith(start)
-                                         for start in CMD_STARTERS):
+        if len(fst_word) > 1 and any(fst_word.startswith(start)
+                                     for start in CMD_STARTERS):

-                command = fst_word[1:].split('@')
-                chat = update.effective_chat
+            command = fst_word[1:].split('@')
+            chat = update.effective_chat

-                ignored = sql.is_command_ignored(chat.id, command[0])
-                if ignored:
-                    return
+            ignored = sql.is_command_ignored(chat.id, command[0])
+            if ignored:
+                return

-                if command[0] not in command_list:
-                    message.delete()
+            if command[0] not in command_list:
+                message.delete()


Function clean_blue_text_must_click refactored with the following changes:

Move assignments closer to their usage (move-assign)

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:32Z

lynda/modules/cleaner.py

-        if clean_status:
-            clean_status = "Enabled"
-        else:
-            clean_status = "Disabled"
+        clean_status = "Enabled" if clean_status else "Disabled"


Function set_blue_text_must_click refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

sourcery-ai · 2020-07-19T20:00:32Z

lynda/modules/cust_filters.py

-                    if excp.message == "Unsupported url protocol":
-                        message.reply_text(
-                            "You seem to be trying to use an unsupported url protocol. Telegram "
-                            "doesn't support buttons for some protocols, such as tg://. Please try "
-                            "again, or ask @Aman_Ahmed for help.")
-                    elif excp.message == "Reply message not found":
+                    if excp.message == "Reply message not found":
                        bot.send_message(
                            chat.id,
                            filt.reply,
                            parse_mode=ParseMode.MARKDOWN,
                            disable_web_page_preview=True,
                            reply_markup=keyboard)
+                    elif excp.message == "Unsupported url protocol":
+                        message.reply_text(
+                            "You seem to be trying to use an unsupported url protocol. Telegram "
+                            "doesn't support buttons for some protocols, such as tg://. Please try "
+                            "again, or ask @Aman_Ahmed for help.")


Function reply_filter refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai · 2020-07-19T20:00:34Z

lynda/modules/dbcleanup.py

-            pass
-
    try:
        progress_message.delete()
    except Exception as e:
        print(e)
-        pass
-
-    if not remove:
-        return kicked_chats
-    else:
+    if remove:
        for muted_chat in chat_list:
            sleep(0.1)
            user_sql.rem_chat(muted_chat)
-        return kicked_chats
+
+    return kicked_chats


Function get_invalid_chats refactored with the following changes:

Remove redundant pass statement (remove-redundant-pass)

Hoist repeated code outside conditional statement (hoist-statement-from-if)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:35Z

lynda/modules/dbcleanup.py

            gban_sql.ungban_user(user_id)
-        return ungbanned_users
+
+    return ungbanned_users


Function get_invalid_gban refactored with the following changes:

Remove redundant pass statement (remove-redundant-pass)

Hoist repeated code outside conditional statement (hoist-statement-from-if)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:35Z

lynda/modules/dbcleanup.py

                        progress_bar, chat_id, progress_message.message_id)
                except Exception as e:
                    print(e)
-                    pass


Function get_muted_chats refactored with the following changes:

Remove redundant pass statement (remove-redundant-pass)

Hoist repeated code outside conditional statement (hoist-statement-from-if)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:35Z

lynda/modules/dbcleanup.py

-    if query_type == "db_leave_chat":
-        if query.from_user.id in admin_list:
-            bot.editMessageText(
-                "Leaving chats ...",
-                chat_id,
-                message.message_id)
-            chat_count = get_muted_chats(bot, update, True)
-            bot.sendMessage(chat_id, f"Left {chat_count} chats.")
-        else:
-            query.answer("You are not allowed to use this.")
+    if query_type == "db_leave_chat" and query.from_user.id in admin_list:
+        bot.editMessageText(
+            "Leaving chats ...",
+            chat_id,
+            message.message_id)
+        chat_count = get_muted_chats(bot, update, True)
+        bot.sendMessage(chat_id, f"Left {chat_count} chats.")
+    elif (
+        query_type == "db_leave_chat"
+        or query_type == "db_cleanup"
+        and query.from_user.id not in admin_list
+    ):
+        query.answer("You are not allowed to use this.")
    elif query_type == "db_cleanup":
-        if query.from_user.id in admin_list:
-            bot.editMessageText(
-                "Cleaning up DB ...",
-                chat_id,
-                message.message_id)
-            invalid_chat_count = get_invalid_chats(bot, update, True)
-            invalid_gban_count = get_invalid_gban(bot, update, True)
-            reply = "Cleaned up {} chats and {} gbanned users from db.".format(
-                invalid_chat_count, invalid_gban_count)
-            bot.sendMessage(chat_id, reply)
-        else:
-            query.answer("You are not allowed to use this.")
+        bot.editMessageText(
+            "Cleaning up DB ...",
+            chat_id,
+            message.message_id)
+        invalid_chat_count = get_invalid_chats(bot, update, True)
+        invalid_gban_count = get_invalid_gban(bot, update, True)
+        reply = "Cleaned up {} chats and {} gbanned users from db.".format(
+            invalid_chat_count, invalid_gban_count)
+        bot.sendMessage(chat_id, reply)


Function callback_button refactored with the following changes:

Merge duplicate blocks in conditional (merge-duplicate-blocks)

Remove redundant conditional (remove-redundant-if)

sourcery-ai · 2020-07-19T20:00:35Z

lynda/modules/disable.py

-                    return False
-                else:
-                    return True
+                return not sql.is_command_disabled(chat.id, self.friendly)


Function DisableAbleMessageHandler.check_update refactored with the following changes:

Simplify conditional into return statement (return-identity)

sourcery-ai · 2020-07-19T20:00:37Z

lynda/modules/feds.py

-    if user.id in SUDO_USERS:
-        pass
-    else:
+    if user.id not in SUDO_USERS:
        for admin in administrators:
            status = admin.status
-            if status == "creator":
-                if str(admin.user.id) == str(user.id):
-                    pass
-                else:
-                    message.reply_text(
-                        "Only group creators can use this command!")
-                    return
+            if status == "creator" and str(admin.user.id) != str(user.id):
+                message.reply_text(
+                    "Only group creators can use this command!")
+                return


Function join_fed refactored with the following changes:

Merge nested if conditions (merge-nested-ifs)

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:37Z

lynda/modules/feds.py

-            if get_fedlog:
-                if eval(get_fedlog):
-                    bot.send_message(
-                        get_fedlog, "Chat *{}* has left the federation *{}*".format(
-                            chat.title, fed_info['fname']), parse_mode="markdown")
+            if get_fedlog and eval(get_fedlog):
+                bot.send_message(
+                    get_fedlog, "Chat *{}* has left the federation *{}*".format(
+                        chat.title, fed_info['fname']), parse_mode="markdown")


Function leave_fed refactored with the following changes:

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:37Z

lynda/modules/feds.py

-        elif not message.reply_to_message and (not args or (
-                len(args) >= 1 and not args[0].startswith("@") and not args[0].isdigit() and not message.parse_entities(
-            [MessageEntity.TEXT_MENTION]))):
+        elif (
+            not message.reply_to_message
+            and len(args) >= 1
+            and not args[0].startswith("@")
+            and not args[0].isdigit()
+            and not message.parse_entities([MessageEntity.TEXT_MENTION])
+        ):


Function user_join_fed refactored with the following changes:

Remove redundant conditional (remove-redundant-if)

sourcery-ai · 2020-07-19T20:00:37Z

lynda/modules/feds.py

-        elif not message.reply_to_message and (not args or (
-                len(args) >= 1 and not args[0].startswith("@") and not args[0].isdigit() and not message.parse_entities(
-            [MessageEntity.TEXT_MENTION]))):
+        elif (
+            not message.reply_to_message
+            and len(args) >= 1
+            and not args[0].startswith("@")
+            and not args[0].isdigit()
+            and not message.parse_entities([MessageEntity.TEXT_MENTION])
+        ):


Function user_demote_fed refactored with the following changes:

Remove redundant conditional (remove-redundant-if)

sourcery-ai · 2020-07-19T20:00:37Z

lynda/modules/feds.py

    message = update.effective_message
    if args:
        fed_id = args[0]
-        info = sql.get_fed_info(fed_id)


Function fed_info refactored with the following changes:

Hoist repeated code outside conditional statement (hoist-statement-from-if)

sourcery-ai · 2020-07-19T20:00:39Z

lynda/modules/feds.py

-        if get_fedlog:
-            if int(get_fedlog) != int(chat.id):
-                bot.send_message(get_fedlog,
-                                 "<b>FedBan reason updated</b>"
-                                 "\n<b>Federation:</b> {}"
-                                 "\n<b>Federation Admin:</b> {}"
-                                 "\n<b>User:</b> {}"
-                                 "\n<b>User ID:</b> <code>{}</code>"
-                                 "\n<b>Reason:</b> {}".format(fed_name,
-                                                              mention_html(user.id,
-                                                                           user.first_name),
-                                                              user_target,
-                                                              fban_user_id,
-                                                              reason),
-                                 parse_mode="HTML")
+        if get_fedlog and int(get_fedlog) != int(chat.id):
+            bot.send_message(get_fedlog,
+                             "<b>FedBan reason updated</b>"
+                             "\n<b>Federation:</b> {}"
+                             "\n<b>Federation Admin:</b> {}"
+                             "\n<b>User:</b> {}"
+                             "\n<b>User ID:</b> <code>{}</code>"
+                             "\n<b>Reason:</b> {}".format(fed_name,
+                                                          mention_html(user.id,
+                                                                       user.first_name),
+                                                          user_target,
+                                                          fban_user_id,
+                                                          reason),
+                             parse_mode="HTML")


Function fed_ban refactored with the following changes:

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:40Z

lynda/modules/feds.py

-    if get_fedlog:
-        if int(get_fedlog) != int(chat.id):
-            bot.send_message(get_fedlog,
-                             "<b>Un-FedBan</b>"
-                             "\n<b>Federation:</b> {}"
-                             "\n<b>Federation Admin:</b> {}"
-                             "\n<b>User:</b> {}"
-                             "\n<b>User ID:</b> <code>{}</code>".format(info['fname'],
-                                                                        mention_html(user.id,
-                                                                                     user.first_name),
-                                                                        user_target,
-                                                                        fban_user_id),
-                             parse_mode="HTML")
+    if get_fedlog and int(get_fedlog) != int(chat.id):
+        bot.send_message(get_fedlog,
+                         "<b>Un-FedBan</b>"
+                         "\n<b>Federation:</b> {}"
+                         "\n<b>Federation Admin:</b> {}"
+                         "\n<b>User:</b> {}"
+                         "\n<b>User ID:</b> <code>{}</code>".format(info['fname'],
+                                                                    mention_html(user.id,
+                                                                                 user.first_name),
+                                                                    user_target,
+                                                                    fban_user_id),
+                         parse_mode="HTML")


Function unfban refactored with the following changes:

Remove redundant pass statement (remove-redundant-pass)

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:40Z

lynda/modules/feds.py

+                "*{}* has updated federation rules for fed *{}*".format(
+                    user.first_name,
+                    getfed['fname']),
+                parse_mode="markdown")


Function set_frules refactored with the following changes:

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:40Z

lynda/modules/feds.py

-            if get_fedlog:
-                if eval(get_fedlog):
-                    teks = "Fed *{}* has successfully imported data. {} banned.".format(
-                        getfed['fname'], success)
-                    if failed >= 1:
-                        teks += " {} Failed to import.".format(failed)
-                    bot.send_message(get_fedlog, teks, parse_mode="markdown")
+            if get_fedlog and eval(get_fedlog):
+                teks = "Fed *{}* has successfully imported data. {} banned.".format(
+                    getfed['fname'], success)
+                if failed >= 1:
+                    teks += " {} Failed to import.".format(failed)
+                bot.send_message(get_fedlog, teks, parse_mode="markdown")


Function fed_import_bans refactored with the following changes:

Merge nested if conditions (merge-nested-ifs)

sourcery-ai · 2020-07-19T20:00:40Z

lynda/modules/feds.py

-    if args:
-        if args[0].isdigit():
-            user_id = args[0]
-        else:
-            user_id = extract_user(message, args)
+    if args and args[0].isdigit():
+        user_id = args[0]
    else:
        user_id = extract_user(message, args)
-


Function fed_stat_user refactored with the following changes:

Merge duplicate blocks in conditional (merge-duplicate-blocks)

sourcery-ai · 2020-07-19T20:00:42Z

lynda/modules/fun.py

-            if bool(random.getrandbits(1)):
-                reply_text += c.upper()
-            else:
-                reply_text += c.lower()
+            reply_text += c.upper() if bool(random.getrandbits(1)) else c.lower()


Function copypasta refactored with the following changes:

Replace if statement with if expression (assign-if-exp)

sourcery-ai · 2020-07-19T20:00:42Z

lynda/modules/global_bans.py

-            if excp.message in UNGBAN_ERRORS:
-                pass
-            else:
+            if excp.message not in UNGBAN_ERRORS:


Function ungban refactored with the following changes:

Swap if/else to remove empty if body (remove-pass-body)

sourcery-ai · 2020-07-19T20:00:42Z

lynda/modules/global_bans.py

-                    return
-                else:
-                    return
+                return
    except Exception as e:
        print(e)
-        pass
-


Function check_and_ban refactored with the following changes:

Remove redundant pass statement (remove-redundant-pass)

Hoist repeated code outside conditional statement (hoist-statement-from-if)

sourcery-ai · 2020-07-19T20:00:42Z

lynda/modules/global_bans.py

-    if sql.does_chat_gban(
-            update.effective_chat.id) and update.effective_chat.get_member(
-            bot.id).can_restrict_members:
-        user = update.effective_user
-        chat = update.effective_chat
-        msg = update.effective_message
+    if (
+        not sql.does_chat_gban(update.effective_chat.id)
+        or not update.effective_chat.get_member(bot.id).can_restrict_members
+    ):
+        return
+    user = update.effective_user
+    chat = update.effective_chat
+    msg = update.effective_message

-        if user and not is_user_admin(chat, user.id):
-            check_and_ban(update, user.id)
+    if user and not is_user_admin(chat, user.id):
+        check_and_ban(update, user.id)

-        if msg.new_chat_members:
-            new_members = update.effective_message.new_chat_members
-            for mem in new_members:
-                check_and_ban(update, mem.id)
+    if msg.new_chat_members:
+        new_members = update.effective_message.new_chat_members
+        for mem in new_members:
+            check_and_ban(update, mem.id)

-        if msg.reply_to_message:
-            user = msg.reply_to_message.from_user
-            if user and not is_user_admin(chat, user.id):
-                check_and_ban(update, user.id, should_message=False)
+    if msg.reply_to_message:
+        user = msg.reply_to_message.from_user
+        if user and not is_user_admin(chat, user.id):
+            check_and_ban(update, user.id, should_message=False)


Function enforce_gban refactored with the following changes:

Add guard clause (last-if-guard)

sourcery-ai · 2020-07-19T20:00:42Z

lynda/modules/global_bans.py

-    if len(args) > 0:
+    if args:


Function gbanstat refactored with the following changes:

Simplify sequence comparison (simplify-len-comparison)

sourcery-ai · 2020-07-19T20:00:53Z

Sourcery Code Quality Report (beta)

✅ Merging this PR will increase code quality in the affected files by 0.05 out of 10.

Quality metrics	Before	After	Change
Complexity	6.37	6.01	-0.36 🔵
Method Length	89.60	88.73	-0.87 🔵
Quality	7.70	7.75	0.05 🔵

Other metrics	Before	After	Change
Lines	13513	13307	-206

Changed files	Quality Before	Quality After	Quality Change
lynda/main.py	6.70	6.73	0.03 🔵
lynda/modules/admin.py	7.15	7.18	0.03 🔵
lynda/modules/anime.py	7.52	7.53	0.01 🔵
lynda/modules/antiflood.py	7.99	8.00	0.01 🔵
lynda/modules/bans.py	6.67	6.68	0.01 🔵
lynda/modules/blacklist.py	7.84	7.86	0.02 🔵
lynda/modules/blacklist_stickers.py	6.59	6.61	0.02 🔵
lynda/modules/blacklistusers.py	7.78	7.81	0.03 🔵
lynda/modules/cleaner.py	7.35	7.42	0.07 🔵
lynda/modules/cust_filters.py	7.45	7.45	0.00
lynda/modules/dbcleanup.py	7.33	7.41	0.08 🔵
lynda/modules/disable.py	8.34	8.38	0.04 🔵
lynda/modules/disasters.py	7.42	7.42	0.00
lynda/modules/extra.py	6.49	6.51	0.02 🔵
lynda/modules/feds.py	6.10	6.18	0.08 🔵
lynda/modules/fun.py	8.28	8.30	0.02 🔵
lynda/modules/global_bans.py	7.16	7.22	0.06 🔵
lynda/modules/gtranslator.py	6.48	6.50	0.02 🔵
lynda/modules/lastfm.py	8.24	8.24	0.00
lynda/modules/locks.py	7.98	8.02	0.04 🔵
lynda/modules/log_channel.py	8.17	8.34	0.17 🔵
lynda/modules/misc.py	7.58	7.60	0.02 🔵
lynda/modules/modules.py	6.82	6.82	0.00
lynda/modules/muting.py	7.08	7.12	0.04 🔵
lynda/modules/notes.py	8.12	8.12	0.00
lynda/modules/ping.py	8.23	8.29	0.06 🔵
lynda/modules/reporting.py	7.18	7.18	0.00
lynda/modules/rss.py	6.94	6.97	0.03 🔵
lynda/modules/rules.py	9.01	9.01	0.00
lynda/modules/sed.py	6.86	6.97	0.11 🔵
lynda/modules/special.py	8.36	8.41	0.05 🔵
lynda/modules/stickers.py	7.03	7.03	0.00
lynda/modules/userinfo.py	7.58	7.62	0.04 🔵
lynda/modules/users.py	8.45	8.46	0.01 🔵
lynda/modules/welcome.py	6.93	6.93	0.00
lynda/modules/helper_funcs/chat_status.py	8.40	8.42	0.02 🔵
lynda/modules/helper_funcs/extraction.py	7.65	7.68	0.03 🔵
lynda/modules/helper_funcs/handlers.py	8.50	8.53	0.03 🔵
lynda/modules/helper_funcs/msg_types.py	5.77	5.87	0.10 🔵
lynda/modules/helper_funcs/string_handling.py	7.42	7.56	0.14 🔵
lynda/modules/helper_funcs/telethn/chatstatus.py	9.20	9.39	0.19 🔵
lynda/modules/sql/chatbot_sql.py	9.27	9.30	0.03 🔵
lynda/modules/sql/cleaner_sql.py	8.88	8.90	0.02 🔵
lynda/modules/sql/feds_sql.py	8.41	8.49	0.08 🔵

Here are some functions in these files that still need a tune-up:

File	Function	Complexity	Length	Overall	Recommendation
lynda/modules/feds.py	fed_ban	135	895.27	0.05	Split out functionality. Reduce complexity
lynda/modules/feds.py	fed_import_bans	125	785.27	0.07	Split out functionality. Reduce complexity
lynda/modules/stickers.py	kang	76	791.95	0.46	Split out functionality. Reduce complexity
lynda/modules/gtranslator.py	totranslate	75	492.22	0.55	Split out functionality. Reduce complexity
lynda/modules/welcome.py	new_member	71	577.45	0.58	Split out functionality. Reduce complexity

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Let us know what you think of it via email or our Gitter!

'Refactored by Sourcery'

f9da035

sourcery-ai bot commented Jul 19, 2020

View reviewed changes

		if not user_member.status == 'administrator':
		if user_member.status != 'administrator':

Uh oh!

Conversation

sourcery-ai bot commented Jul 19, 2020

Uh oh!

sourcery-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020

Choose a reason for hiding this comment

Uh oh!

sourcery-ai bot Jul 19, 2020