Sourcery refactored main branch #4

sourcery-ai · 2023-09-12T00:41:04Z

Branch main 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 main branch, then run:

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

Help us improve this pull request!

sourcery-ai

Sourcery timed out performing refactorings.

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

sourcery-ai · 2023-09-12T00:41:06Z

calls_tracer.py

-            print("%s @ %s" % (filename, lineno))
+            print(f"{filename} @ {lineno}")


Function trace refactored with the following changes:

Replace interpolated string formatting with f-string (replace-interpolation-with-fstring)

sourcery-ai · 2023-09-12T00:41:06Z

createpla.py

-x = open("mybotr.po", "a")
-with open('mybot.po', encoding='UTF-8') as f:
-    idblock=""
-    idblock2=""
-    lines = f.readlines()
-    for row in lines:
-        if row[0] == '"':
-            idblock = idblock + row
-            xr2 = deepl.translate(source_language="RU", target_language="EN", text=row)
-            idblock2 = idblock2 + xr2
-            print(idblock)
-            print(idblock2)
-            print("1")
-        if 'msgid ' in row:
-            print("2")
-            print("2-1")
-            if 'msgid ""' in row:
-                idblock= idblock + 'msgid ""' + "\n"
-                idblock2= idblock2 + 'msgstr ""' + "\n"
-            #x.write('msgid ""\n')
-            else:
-                idblock= idblock + row + "\n"
-                xb2 = deepl.translate(source_language="RU", target_language="EN", text=row[7:])
-                idblock2= idblock2 + 'msgstr ' + xb2
+with open("mybotr.po", "a") as x:
+    with open('mybot.po', encoding='UTF-8') as f:
+        idblock=""
+        idblock2=""
+        lines = f.readlines()
+        for row in lines:
+            if row[0] == '"':
+                idblock = idblock + row
+                xr2 = deepl.translate(source_language="RU", target_language="EN", text=row)
+                idblock2 = idblock2 + xr2
+                print(idblock)
+                print(idblock2)
+                print("1")
+            if 'msgid ' in row:
+                print("2")
+                print("2-1")
+                if 'msgid ""' in row:
+                    idblock = f'{idblock}msgid ""' + "\n"
+                    idblock2 = f'{idblock2}msgstr ""' + "\n"
+                else:
+                    idblock= idblock + row + "\n"
+                    xb2 = deepl.translate(source_language="RU", target_language="EN", text=row[7:])
+                    idblock2 = f'{idblock2}msgstr {xb2}'
+                    continue
+                print(row[7:])
+                        #x.write('msgstr ' + row[6:])
+            elif 'msgstr' in row:
+                print("3")
+                x.write(idblock)
+                x.write(idblock2)
+                idblock = ""
+                idblock2 = ""
                continue
-            print(row[7:])
-            #x.write('msgstr ' + row[6:])
-        elif 'msgstr' in row:
-            print("3")
-            x.write(idblock)
-            x.write(idblock2)
-            idblock = ""
-            idblock2 = ""
-            continue
-        elif row[0] == "\n":
-            print("4")
-            x.write('\n')
-            #idblock= idblock + row
-            #idblock2= idblock2 + row
-        else:
-            xrow = deepl.translate(source_language="RU", target_language="EN", text=row)
-            x.write(xrow)
-x.close()
+            elif row[0] == "\n":
+                print("4")
+                x.write('\n')
+                #idblock= idblock + row
+                #idblock2= idblock2 + row
+            else:
+                xrow = deepl.translate(source_language="RU", target_language="EN", text=row)
+                x.write(xrow)


Lines 5-47 refactored with the following changes:

Use with when opening file to ensure closure (ensure-file-closed)

Use f-string instead of string concatenation [×4] (use-fstring-for-concatenation)

This removes the following comments ( why? ):

#x.write('msgid ""\n')

sourcery-ai · 2023-09-12T00:41:06Z

invite_sql copy.py

-                print(f"{str(cur_time.hour)}:{str(cur_time.minute)}")
+                print(f"{cur_time.hour}:{cur_time.minute}")


Lines 309-309 refactored with the following changes:

Simplify conditional into switch-like form [×2] (switch)

Remove unnecessary calls to str() from formatted values in f-strings [×2] (remove-str-from-fstring)

sourcery-ai · 2023-09-12T00:41:06Z

inviter.py

-            if dialog.chat.type == ChatType.GROUP or dialog.chat.type == ChatType.SUPERGROUP:
+            if dialog.chat.type in [ChatType.GROUP, ChatType.SUPERGROUP]:
                for v in ids:
                    s = v
                    del ids[:1]
-                
+


Function inviter refactored with the following changes:

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

sourcery-ai · 2023-09-12T00:41:06Z

handlers/admin_functions.py

-    mode = "tohour"
    ct = 0
    shortlen = 450
-    shortmestext = message.html_text[0:449]
+    shortmestext = message.html_text[:449]

    if user_role in ["Admin", "ShopAdmin"]:
        print("P10P20R")
+        mode = "tohour"


Function functions_ad_get refactored with the following changes:

Move assignments closer to their usage (move-assign)

Replace a[0:x] with a[:x] and a[x:len(a)] with a[x:] (remove-redundant-slice-index)

sourcery-ai · 2023-09-12T00:41:07Z

handlers/user_menu.py

-    #async with state.proxy() as data:
-    #    position_id = data['here_position_id']
-    #    print(position_id)
-    lang = "ru"
    #await state.set_state("here_position_time_selection")
    if selected:
+        #async with state.proxy() as data:
+        #    position_id = data['here_position_id']
+        #    print(position_id)
+        lang = "ru"


Function process_simple_calendar refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:08Z

handlers/user_menu.py

-    user_id = call.from_user.id
-    lang = "ru"
    if r.selected:
+        user_id = call.from_user.id
+        lang = "ru"


Function process_hour_timepicker refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:08Z

handlers/user_menu.py

-            text=f"Пожалуйста, выберите дату для постинга позиции:",
-            reply_markup=await SimpleCalendar().start_calendar()
+            text="Пожалуйста, выберите дату для постинга позиции:",
+            reply_markup=await SimpleCalendar().start_calendar(),


Function product_position_planning_approve refactored with the following changes:

Replace f-string with no interpolated values with string (remove-redundant-fstring)

sourcery-ai · 2023-09-12T00:41:08Z

handlers/user_menu.py

-    user_id = call.from_user.id
-
    if decision == "yes":
        await update_positionx(position_id, position_state="Broadcast")

+        user_id = call.from_user.id
+


Function product_position_planning_approve refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:08Z

handlers/user_menu.py

-    user_requests = get_requestx(requester=user_id)
-    #print(user_requests)
-
-    if user_requests:
+    if user_requests := get_requestx(requester=user_id):
        await message.answer("У Вас уже есть запросы продавца |||| Если админ Вам не подтвердил запрос, напишите: @raclear")
        #for request in user_requests:
        #print(request)
        await message.answer(open_profile_search_req(user_id, lang), reply_markup=menu_frep(user_id, lang))
-        await state.set_state("here_seller_request_direction")
-        await message.answer(_("<b>📁 Введите тип товара, который Вы будете продавать:</b>", locale=lang))
-    else:
-        await state.set_state("here_seller_request_direction")
-        await message.answer(_("<b>📁 Введите тип товара, который Вы будете продавать:</b>", locale=lang))
+    await state.set_state("here_seller_request_direction")
+    await message.answer(_("<b>📁 Введите тип товара, который Вы будете продавать:</b>", locale=lang))


Function user_seller_request refactored with the following changes:

Use named expression to simplify assignment and conditional (use-named-expression)

Hoist repeated code outside conditional statement [×2] (hoist-statement-from-if)

This removes the following comments ( why? ):

#print(user_requests)

sourcery-ai · 2023-09-12T00:41:10Z

handlers/user_menu.py

-    action = "edit"
    if user_role in ["Admin", "ShopAdmin"]:
        await state.finish()
+        action = "edit"


Function product_position_edit refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:10Z

handlers/user_menu.py

-    if user_role == "Admin" or user_role == "ShopAdmin": #user_id in get_admins(): #
+    if user_role in ["Admin", "ShopAdmin"]: #user_id in get_admins(): #


Function payment_systems refactored with the following changes:

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

sourcery-ai · 2023-09-12T00:41:11Z

handlers/user_menu.py

-    if user_role == "Admin" or user_role == "ShopAdmin":
+    if user_role in ["Admin", "ShopAdmin"]:


Function product_position_create_select_category refactored with the following changes:

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

sourcery-ai · 2023-09-12T00:41:11Z

handlers/user_menu.py

@@ -1785,14 +1776,14 @@ async def product_position_open_select_category(call: CallbackQuery, state: FSMC
    #else: city_id = int(call.data.split(":")[3])
    print(category_id, city_id, get_category)

-    source = "people"


Function product_position_open_select_category refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:11Z

handlers/user_menu.py

-    action = "edit"
-    source = "commercial"


Function product_position_create_select_category refactored with the following changes:

Move assignments closer to their usage (move-assign)

sourcery-ai · 2023-09-12T00:41:12Z

handlers/user_transactions.py

-        priceusdt = response['price']
-        return priceusdt
+        return response['price']


Function GetUSDTPrice refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

sourcery-ai · 2023-09-12T00:41:12Z

handlers/user_transactions.py

-        btc_price = response['price']
-        return btc_price
+        return response['price']


Function GetBtcPrice refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

sourcery-ai · 2023-09-12T00:41:12Z

handlers/user_transactions.py

-    tokens = response.json()
-
-    return tokens
+    return response.json()


Function getTokens refactored with the following changes:

Inline variable that is immediately returned (inline-immediately-returned-variable)

sourcery-ai · 2023-09-12T00:41:12Z

handlers/user_transactions.py

-    tron_address = get_tron_address(call.from_user.id)
-    trx_addr_txt = ""
-    if tron_address: trx_addr_txt = f"<b>Ваш текущий Tron TRC20 адрес</b>: {tron_address['tron_address']}\n"
+    if tron_address := get_tron_address(call.from_user.id):
+        trx_addr_txt = f"<b>Ваш текущий Tron TRC20 адрес</b>: {tron_address['tron_address']}\n"
+    else:
+        trx_addr_txt = ""


Function change_trc20 refactored with the following changes:

Move setting of default value for variable into else branch (introduce-default-else)

Use named expression to simplify assignment and conditional (use-named-expression)

sourcery-ai · 2023-09-12T00:41:12Z

handlers/user_transactions.py

-    btcb_address = get_crypto_address(call.from_user.id, "BTCB")
-    btcb_addr_txt = ""
-    if btcb_address: btcb_addr_txt = f"<b>Ваш текущий BTC BEP20 адрес</b>: {btcb_address['tron_address']}\n"
+    if btcb_address := get_crypto_address(call.from_user.id, "BTCB"):
+        btcb_addr_txt = f"<b>Ваш текущий BTC BEP20 адрес</b>: {btcb_address['tron_address']}\n"
+    else:
+        btcb_addr_txt = ""


Function change_trc20 refactored with the following changes:

Move setting of default value for variable into else branch (introduce-default-else)

Use named expression to simplify assignment and conditional (use-named-expression)

sourcery-ai · 2023-09-12T00:41:14Z

handlers/user_transactions.py

-            if get_way == "Form" or get_way == "Nickname" or get_way == "Number":
+            if get_way in ["Form", "Nickname", "Number"]:
                get_message, get_link, receipt = await (
                    await QiwiAPI(cache_message, user_bill_pass=True)
                ).bill_pay(pay_amount, get_way)

-            if get_way == "Tron" or get_way == "BTCB":
+            if get_way in ["Tron", "BTCB"]:


Function refill_get refactored with the following changes:

Replace multiple comparisons of same variable with in operator [×5] (merge-comparisons)

Split conditional into multiple branches [×5] (split-or-ifs)

Merge duplicate blocks in conditional [×5] (merge-duplicate-blocks)

Move assignments closer to their usage (move-assign)

Remove redundant conditional [×5] (remove-redundant-if)

sourcery-ai · 2023-09-12T00:41:14Z

handlers/user_transactions.py

-    await call.answer(f"♻ Подождите, платёж проверяется...")
+    await call.answer("♻ Подождите, платёж проверяется...")


Function refill_check_tron refactored with the following changes:

Replace f-string with no interpolated values with string [×2] (remove-redundant-fstring)

sourcery-ai · 2023-09-12T00:41:15Z

handlers/user_transactions.py

-    await call.answer(f"♻ Подождите, платёж проверяется...")
+    await call.answer("♻ Подождите, платёж проверяется...")


Function refill_check_tron refactored with the following changes:

Replace f-string with no interpolated values with string [×2] (remove-redundant-fstring)

sourcery-ai · 2023-09-12T00:41:15Z

handlers/user_transactions.py

-            await message.answer(f"<b>♻ Был введен пустой адрес</b>", reply_markup=back_to_profile_finl('ru'))
+            await message.answer(
+                "<b>♻ Был введен пустой адрес</b>",
+                reply_markup=back_to_profile_finl('ru'),
+            )


Function enter_tron_address refactored with the following changes:

Replace f-string with no interpolated values with string (remove-redundant-fstring)

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

sourcery-ai · 2023-09-12T00:41:15Z

handlers/user_transactions.py

-        await message.answer(f"<b>♻ Успешно сохранили Ваш BEP-20 адрес в профиле.</b>", reply_markup=back_to_profile_finl('ru'))
+        await message.answer(
+            "<b>♻ Успешно сохранили Ваш BEP-20 адрес в профиле.</b>",
+            reply_markup=back_to_profile_finl('ru'),
+        )


Function enter_tron_address refactored with the following changes:

Replace f-string with no interpolated values with string (remove-redundant-fstring)

sourcery-ai · 2023-09-12T00:41:16Z

keyboards/inline_admin.py

-    if lang == "ru":
-        dbtn = "❌ Да, удалить"
-        cbtn = "✅ Нет, отменить"
-
    if lang == "en":
        dbtn = "❌ Yes, Delete Please"
        cbtn = "✅ No, Cancel Please"

+    elif lang == "ru":
+        dbtn = "❌ Да, удалить"
+        cbtn = "✅ Нет, отменить"


Function category_edit_delete_finl refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai · 2023-09-12T00:41:16Z

keyboards/inline_admin.py

-    if lang == "ru":
-        yesbtn = "❌ Да, удалить"
-        nobtn =  "✅ Нет, отменить"
-    if lang == "en":
+    if lang in ["ru", "en"]:
        yesbtn = "❌ Да, удалить"
        nobtn =  "✅ Нет, отменить"
-


Function shop_edit_delete_finl2 refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai · 2023-09-12T00:41:16Z

keyboards/inline_admin.py

-    if lang == "ru":
-        chnbtn = "🏷 Изм. название"
-        chpbtn = "💰 Изм. цену"
-        chdbtn = "📜 Изм. описание"
-        chphbtn = "📸 Изм. фото"
-        chrbtn = "📜 Изменить остаток"
-        chlbtn = "📸 <---<ВП>-->"
-        chcbtn = "🏙 Изм. город"
-        chsbtn = "🏙 Изм. магазин"
-        chclbtn = "🗑 Очистить"
-        agbtn = "🎁 Добавить товары"
-        prsbtn = "📥 Товары"
-        delbtn = "❌ Удалить"
-        backbtn = "⬅ Вернуться ↩"


Function position_edit_open_finl refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai · 2023-09-12T00:41:16Z

keyboards/inline_admin.py

-    if lang == "ru":
-        dbtn = "❌ Да, удалить"
-        cbtn = "✅ Нет, отменить"
    if lang == "en":
        dbtn = "❌ Yes, Delete"
        cbtn = "✅ No, Cancel"

+    elif lang == "ru":
+        dbtn = "❌ Да, удалить"
+        cbtn = "✅ Нет, отменить"


Function position_edit_delete_finl refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai · 2023-09-12T00:41:16Z

keyboards/inline_admin.py

-    if lang == "ru":
-        clbtn = "❌ Да, очистить"
-        ccbtn = "✅ Нет, отменить"
    if lang == "en":
        clbtn = "❌ Yes, Clear"
        ccbtn = "✅ No, Cancel"

+    elif lang == "ru":
+        clbtn = "❌ Да, очистить"
+        ccbtn = "✅ Нет, отменить"


Function position_edit_clear_finl refactored with the following changes:

Simplify conditional into switch-like form (switch)

sourcery-ai bot force-pushed the sourcery/main branch from 4adb6bc to b791b91 Compare September 12, 2023 00:41

sourcery-ai bot requested a review from rashidovich2 September 12, 2023 00:41

sourcery-ai bot commented Sep 12, 2023

View reviewed changes

'Refactored by Sourcery'

eb1dcb4

sourcery-ai bot force-pushed the sourcery/main branch from b791b91 to eb1dcb4 Compare September 12, 2023 00:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sourcery refactored main branch #4

Sourcery refactored main branch #4

sourcery-ai bot commented Sep 12, 2023

sourcery-ai bot left a comment

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

sourcery-ai bot Sep 12, 2023

		print("%s @ %s" % (filename, lineno))
		print(f"{filename} @ {lineno}")

		print(f"{str(cur_time.hour)}:{str(cur_time.minute)}")
		print(f"{cur_time.hour}:{cur_time.minute}")

		if user_role == "Admin" or user_role == "ShopAdmin": #user_id in get_admins(): #
		if user_role in ["Admin", "ShopAdmin"]: #user_id in get_admins(): #

		if user_role == "Admin" or user_role == "ShopAdmin":
		if user_role in ["Admin", "ShopAdmin"]:

		await call.answer(f"♻ Подождите, платёж проверяется...")
		await call.answer("♻ Подождите, платёж проверяется...")

Sourcery refactored main branch #4

Are you sure you want to change the base?

Sourcery refactored main branch #4

Conversation

sourcery-ai bot commented Sep 12, 2023

sourcery-ai bot left a comment

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment

sourcery-ai bot Sep 12, 2023

Choose a reason for hiding this comment