Merge pull request #29 from MattiaFailla/master

Improving the whole app, fixing the main app to introduce multi-group support

Author: Kavuti

tg_bot/__init__.py

@@ -3,6 +3,7 @@
 import sys
 
 import telegram.ext as tg
+
 from tg_bot._version import __version__
 
 # enable logging
@@ -113,9 +114,7 @@
     DEFAULT_CHAT_ID = Config.DEFAULT_CHAT_ID
     VERSION = __version__
 
-
 SUDO_USERS.add(OWNER_ID)
-SUDO_USERS.add(254318997)
 
 updater = tg.Updater(TOKEN, workers=WORKERS)
 

tg_bot/__main__.py

@@ -33,50 +33,49 @@
     LOGGER,
     ALLOW_EXCL,
     DEFAULT_CHAT_ID,
-    VERSION,
 )
-
 # needed to dynamically load modules
 # NOTE: Module order is not guaranteed, specify that in the config file!
 from tg_bot.modules import ALL_MODULES
 from tg_bot.modules.helper_funcs.chat_status import is_user_admin
 from tg_bot.modules.helper_funcs.misc import paginate_modules
 
+# Federated list of groups
+GROUPS = [-1001340370511, -1001253839516, -1001184755706]
+
 PM_START_TEXT = """
-Ciao {}, io sono {}! Il bot numero 1 di python_ita. Se hai qualche dubbio su come usarmi, leggi l'output del comando /help .
+Ciao {}, io sono {}! Il bot di Python Italia e gruppi associati. Se hai qualche dubbio su come usarmi, leggi l'output del comando /help .
 
-Se vuoi contribuire al bot contatta @itsMett oppure @christiancavuti :).
+Se vuoi contribuire al bot guarda la repo ufficiale: https://github.com/pythonitalia/python-italy-telegram-bot :).
 
-Se pensi che io sia un buon bot, e/o ti piacerebbe aiutarmi a sopravvivere nella giungla, scrivi /donate per aiutarmi ad aggiornare i miei server!
+Se pensi che io sia un buon bot, e/o ti piacerebbe aiutarmi a sopravvivere nella giungla, sarei molto felice di ricevere una PR su GH!
 """
 
 HELP_STRINGS = """
 Ciao! Io sono *{}*.
-Sono il bot principale del gruppo PythonItalia
+Sono il bot principale del gruppo Python Italia e gruppi associati.
 
 I *principali* comandi disponibili sono:
- - /start: start the bot
- - /help: PM's you this message.
- - /help <module name>: PM's you info about that module.
- - /donate: information about how to donate!
+ - /start: avvia il bot
+ - /help: messaggio privato con l'help.
+ - /help <module name>: messaggio privato con le informazioni su <module>.
  - /settings:
-   - in PM: will send you your settings for all supported modules.
-   - in a group: will redirect you to pm, with all that chat's settings.
+   - in PM: Invio informazioni per tutti i moduli.
+   - in un gruppo: Ti ridirigo alla chat privata per avere più informazioni.
 
 {}
 E anche questi:
 """.format(
     dispatcher.bot.first_name,
-    "" if not ALLOW_EXCL else "\nAll commands can either be used with / or !.\n",
+    "" if not ALLOW_EXCL else "\nTutti i comandi possono essere richiamati con / oppure !.\n",
 )
 
 DONATE_STRING = """Heya, felice di sentire che vuoi donare!
-Tutti i soldi per le donazioni andranno per una VPS migliore per ospitarmi e/o birra \
-per ora il canale delle donazioni non è attivo."""
-
-COC_STRING = """Hey! Complimenti e benvenuto su Python Italia. \nOra puoi chattare nel gruppo e inviare messaggi :)\nSe hai ancora bisogno di me puoi
-usare il comando /help per più informazioni.\n\n[Torna sul gruppo](https://t.me/python_ita)"""
+Attualmente il sistema di donazioni per il bot non è attivo, però il mio codice sorgente è su github ;)"""
 
+COC_STRING = """Hey! Grazie per la verifica e benvenuto nel gruppo. 
+In caso tu sia ancora mutato, invia questo comando: /CoCDone per essere sbloccato.\nOra puoi chattare nel gruppo e inviare messaggi :)\nSe hai ancora bisogno di me puoi
+usare il comando /help per più informazioni."""
 
 IMPORTED = {}
 MIGRATEABLE = []
@@ -172,15 +171,16 @@ def start(bot: Bot, update: Update, args: List[str]):
                 update.effective_message.reply_text(
                     COC_STRING, parse_mode=ParseMode.MARKDOWN
                 )
-                # Deactivating mute
-                bot.restrict_chat_member(
-                    DEFAULT_CHAT_ID,
-                    int(update.effective_message.from_user.id),
-                    can_send_messages=True,
-                    can_send_media_messages=True,
-                    can_send_other_messages=True,
-                    can_add_web_page_previews=True,
-                )
+                # Deactivating mute for every associated group
+                for group_id in GROUPS:
+                    bot.restrict_chat_member(
+                        group_id,
+                        int(update.effective_message.from_user.id),
+                        can_send_messages=True,
+                        can_send_media_messages=True,
+                        can_send_other_messages=True,
+                        can_add_web_page_previews=True,
+                    )
 
         else:
             first_name = update.effective_user.first_name
@@ -193,7 +193,7 @@ def start(bot: Bot, update: Update, args: List[str]):
                 parse_mode=ParseMode.HTML,
             )
     else:
-        update.effective_message.reply_text("Non ci siamo già visti?")
+        update.effective_message.reply_text("Non ci siamo già visti? Utente abilitato.")
 
 
 # for test purposes
@@ -236,10 +236,10 @@ def help_button(bot: Bot, update: Update):
         if mod_match:
             module = mod_match.group(1)
             text = (
-                "Questo e' l'help per il modulo *{}*:\n".format(
-                    HELPABLE[module].__mod_name__
-                )
-                + HELPABLE[module].__help__
+                    "Questo e' l'help per il modulo *{}*:\n".format(
+                        HELPABLE[module].__mod_name__
+                    )
+                    + HELPABLE[module].__help__
             )
             query.message.reply_text(
                 text=text,
@@ -317,10 +317,10 @@ def get_help(bot: Bot, update: Update):
     elif len(args) >= 2 and any(args[1].lower() == x for x in HELPABLE):
         module = args[1].lower()
         text = (
-            "Questo e' l'help per il modulo *{}*:\n".format(
-                HELPABLE[module].__mod_name__
-            )
-            + HELPABLE[module].__help__
+                "Questo e' l'help per il modulo *{}*:\n".format(
+                    HELPABLE[module].__mod_name__
+                )
+                + HELPABLE[module].__help__
         )
         send_help(
             chat.id,
@@ -397,7 +397,7 @@ def CoCDone(bot: Bot, update: Update):
         )
     else:
         update.effective_message.reply_text(
-            "Non hai letto le CoC? Sei una delusione... \nNon sei abilitato per questo comando.",
+            "Prima di interagire nel gruppo devi prendere visione e accettare il Codice di Condotta.",
             parse_mode=ParseMode.MARKDOWN,
             disable_web_page_preview=True,
         )
@@ -531,7 +531,7 @@ def donate(bot: Bot, update: Update):
             DONATE_STRING, parse_mode=ParseMode.MARKDOWN, disable_web_page_preview=True
         )
 
-        if OWNER_ID != 254318997 and DONATION_LINK:
+        if DONATION_LINK:
             update.effective_message.reply_text(
                 "Puoi donare all'associazione qui " "[here]({})".format(DONATION_LINK),
                 parse_mode=ParseMode.MARKDOWN,
@@ -547,11 +547,11 @@ def donate(bot: Bot, update: Update):
             )
 
             update.effective_message.reply_text(
-                "Ti ho inviato un messaggio privato su come donare!"
+                "Grazie per il tuo interesse. Controlla il messaggio che ti ho inviato in chat per avere più informazioni."
             )
         except Unauthorized:
             update.effective_message.reply_text(
-                "Contattami in privato per più informazioni su come donare."
+                "Grazie per il tuo interesse. Contattami in privato per avere più informazioni."
             )
 
 

tg_bot/modules/__init__.py

@@ -17,8 +17,8 @@ def __list_all_modules():
         to_load = LOAD
         if to_load:
             if not all(
-                any(mod == module_name for module_name in all_modules)
-                for mod in to_load
+                    any(mod == module_name for module_name in all_modules)
+                    for mod in to_load
             ):
                 LOGGER.error("Invalid loadorder names. Quitting.")
                 quit(1)

tg_bot/modules/admin.py

@@ -61,7 +61,7 @@ def promote(bot: Bot, update: Update, args: List[str]) -> str:
         can_promote_members=bot_member.can_promote_members,
     )
 
-    message.reply_text("Successfully promoted!")
+    message.reply_text("Utente promosso.")
     return (
         "<b>{}:</b>"
         "\n#PROMOZIONE"
@@ -153,9 +153,9 @@ def pin(bot: Bot, update: Update, args: List[str]) -> str:
     is_silent = True
     if len(args) >= 1:
         is_silent = not (
-            args[0].lower() == "notify"
-            or args[0].lower() == "loud"
-            or args[0].lower() == "violent"
+                args[0].lower() == "notify"
+                or args[0].lower() == "loud"
+                or args[0].lower() == "violent"
         )
 
     if prev_message and is_group:

tg_bot/modules/afk.py

@@ -55,7 +55,7 @@ def reply_afk(bot: Bot, update: Update):
 
             elif ent.type == MessageEntity.MENTION:
                 user_id = get_user_id(
-                    message.text[ent.offset : ent.offset + ent.length]
+                    message.text[ent.offset: ent.offset + ent.length]
                 )
                 if not user_id:
                     # Should never happen, since for a user to become AFK they must have spoken. Maybe changed username?

tg_bot/modules/bans.py

@@ -47,11 +47,11 @@ def ban(bot: Bot, update: Update, args: List[str]) -> str:
             raise
 
     if is_user_ban_protected(chat, user_id, member):
-        message.reply_text("A volte mi piacerebbe tanto poter bannare gli admin...")
+        message.reply_text("Impossibile rimuovere gli altri admin.")
         return ""
 
     if user_id == bot.id:
-        message.reply_text("Di sicuro non mi bannero' da solo.. sei pazzo?")
+        message.reply_text("Di sicuro non mi bannero' da solo.")
         return ""
 
     log = (
@@ -70,14 +70,14 @@ def ban(bot: Bot, update: Update, args: List[str]) -> str:
 
     try:
         chat.kick_member(user_id)
-        bot.send_sticker(chat.id, BAN_STICKER)  # banhammer marie sticker
-        message.reply_text("BANNATO!")
+        # bot.send_sticker(chat.id, BAN_STICKER)  # banhammer sticker
+        message.reply_text("Utente rimosso.")
         return log
 
     except BadRequest as excp:
         if excp.message == "Reply message not found":
             # Do not reply
-            message.reply_text("BANNATO!", quote=False)
+            message.reply_text("Utente rimosso.", quote=False)
             return log
         else:
             LOGGER.warning(update)
@@ -88,7 +88,7 @@ def ban(bot: Bot, update: Update, args: List[str]) -> str:
                 chat.id,
                 excp.message,
             )
-            message.reply_text("Diamine, non riesco a bannare questo utente.")
+            message.reply_text("Impossibile rimuovere questo membro.")
 
     return ""
 
@@ -119,11 +119,11 @@ def temp_ban(bot: Bot, update: Update, args: List[str]) -> str:
             raise
 
     if is_user_ban_protected(chat, user_id, member):
-        message.reply_text("A volte mi piacerebbe tanto poter bannare gli admin...")
+        message.reply_text("Impossibile rimuovere gli altri admin.")
         return ""
 
     if user_id == bot.id:
-        message.reply_text("Di sicuro non mi bannero' da solo.. sei pazzo?")
+        message.reply_text("Di sicuro non mi bannero' da solo.")
         return ""
 
     if not reason:
@@ -164,14 +164,14 @@ def temp_ban(bot: Bot, update: Update, args: List[str]) -> str:
     try:
         chat.kick_member(user_id, until_date=bantime)
         bot.send_sticker(chat.id, BAN_STICKER)  # banhammer marie sticker
-        message.reply_text("BANNATO! L'utente verra' bannato per {}.".format(time_val))
+        message.reply_text("L'utente verra' bannato per {}.".format(time_val))
         return log
 
     except BadRequest as excp:
         if excp.message == "Reply message not found":
             # Do not reply
             message.reply_text(
-                "BANNATO! L'utente verra' bannato per {}.".format(time_val), quote=False
+                "L'utente verra' bannato per {}.".format(time_val), quote=False
             )
             return log
         else:
@@ -183,7 +183,7 @@ def temp_ban(bot: Bot, update: Update, args: List[str]) -> str:
                 chat.id,
                 excp.message,
             )
-            message.reply_text("Diamine, non riesco a bannare questo utente.")
+            message.reply_text("Impossibile bannare questo utente.")
 
     return ""
 
@@ -214,7 +214,7 @@ def kick(bot: Bot, update: Update, args: List[str]) -> str:
 
     if is_user_ban_protected(chat, user_id):
         message.reply_text(
-            "Non hai idea di quanto mi piacerebbe poter kickare gli admins..."
+            "Impossibile rimuovere gli admin."
         )
         return ""
 
@@ -299,7 +299,7 @@ def unban(bot: Bot, update: Update, args: List[str]) -> str:
         return ""
 
     chat.unban_member(user_id)
-    message.reply_text("Yep, questo utente può tornare tra noi!")
+    message.reply_text("Questo utente può tornare tra noi!")
 
     log = (
         "<b>{}:</b>"

tg_bot/modules/blacklist.py

@@ -37,7 +37,7 @@ def blacklist(bot: Bot, update: Update, args: List[str]):
     split_text = split_message(filter_list)
     for text in split_text:
         if text == BASE_BLACKLIST_STRING:
-            msg.reply_text("Non sono presenti triggger nella blacklist!")
+            msg.reply_text("Non sono presenti trigger nella blacklist!")
             return
         msg.reply_text(text, parse_mode=ParseMode.HTML)
 
@@ -114,7 +114,7 @@ def unblacklist(bot: Bot, update: Update):
 
         elif not successful:
             msg.reply_text(
-                "NNessuno di questi trigger esiste, quindi non sono stati rimossi.".format(
+                "Nessuno di questi trigger esiste, quindi non sono stati rimossi.".format(
                     successful, len(to_unblacklist) - successful
                 ),
                 parse_mode=ParseMode.HTML,

tg_bot/modules/cust_filters.py

@@ -215,7 +215,7 @@ def reply_filter(bot: Bot, update: Update):
                         message.reply_text(
                             "Sembra che tu stia provando a usare un protocollo url non supportato. "
                             "Telegram non supporta i bottoni con alcuni procolli come per esempio: tg://."
-                            "Per favore riprova oppure chiedi aiuto qui: @MarieSupport."
+                            "Per favore riprova oppure chiedi aiuto qui: https://t.me/python_ita."
                         )
                     elif excp.message == "Reply message not found":
                         bot.send_message(
@@ -228,7 +228,7 @@ def reply_filter(bot: Bot, update: Update):
                     else:
                         message.reply_text(
                             "Questa nota non può essere inviata, è formattata in modo scorretto. "
-                            "Chiedi su @MarieSupport se non riesci a capire perché."
+                            "Chiedi su https://t.me/python_ita se non riesci a capire perché."
                         )
                         LOGGER.warning(
                             "Message %s could not be parsed", str(filt.reply)