Merge pull request #117 from nidhaloff/fix/file_translation

deep_translator/__init__.py

@@ -1,6 +1,5 @@
 """Top-level package for Deep Translator"""
 
-# TODO: Discussion: Do these need to be in __init__.py? Are they intended to be exportable?
 from .google_trans import GoogleTranslator
 from .pons import PonsTranslator
 from .linguee import LingueeTranslator
@@ -13,7 +12,6 @@ from .microsoft import MicrosoftTranslator
 from .papago import PapagoTranslator
 from .libre import LibreTranslator
 
-# TODO: Discussion: These should be declared in setup.cfg, setting them here is redundant
 __author__ = """Nidhal Baccouri"""
 __email__ = '[email protected]'
 __version__ = '1.5.0'
@@ -27,7 +25,8 @@ __all__ = [
     "MicrosoftTranslator",
     "QCRI",
     "DeepL",
-    "LibreTranslator"
+    "LibreTranslator",
+    "PapagoTranslator",
     "single_detection",
     "batch_detection"
-           ]
+    ]

deep_translator/google_trans.py

@@ -152,7 +152,7 @@ class GoogleTranslator(BaseTranslator):
         @return: str
         """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read().strip()
             return self.translate(text)
         except Exception as e:

deep_translator/libre.py

@@ -45,7 +45,7 @@ class LibreTranslator(BaseTranslator):
         @return: list or dict
         """
         return [*LibreTranslator._languages.keys()] if not as_dict else LibreTranslator._languages
-    
+
     def _map_language_to_code(self, language, **kwargs):
         """
         map language to its corresponding code (abbreviation) if the language was passed by its full name by the user
@@ -57,7 +57,7 @@ class LibreTranslator(BaseTranslator):
         elif language in self._languages.values():
             return language
         raise LanguageNotSupportedException(language)
-    
+
     def _is_language_supported(self, language, **kwargs):
         """
         check if the language is supported by the translator
@@ -68,7 +68,7 @@ class LibreTranslator(BaseTranslator):
             return True
         else:
             raise LanguageNotSupportedException(language)
-    
+
     def translate(self, text, **kwargs):
         """
         function that uses microsoft translate to translate a text
@@ -87,7 +87,7 @@ class LibreTranslator(BaseTranslator):
             "format": 'text'
         }
         # Add API Key if required
-        if self.api_key: 
+        if self.api_key:
             params["api_key"] = self.api_key
         # Do the request and check the connection.
         try:
@@ -95,7 +95,7 @@ class LibreTranslator(BaseTranslator):
         except ConnectionError:
             raise ServerException(503)
         # If the answer is not success, raise server exception.
-        
+
         if response.status_code == 403:
             raise AuthorizationException(self.api_key)
         elif response.status_code != 200:
@@ -116,7 +116,7 @@ class LibreTranslator(BaseTranslator):
         @return: str
         """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read().strip()
             return self.translate(text)
         except Exception as e:

deep_translator/microsoft.py

@@ -131,7 +131,7 @@ class MicrosoftTranslator:
         @return: translated text
         """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read().strip()
             return self.translate(text)
         except Exception as e:

deep_translator/mymemory.py

@@ -151,7 +151,7 @@ class MyMemoryTranslator(BaseTranslator):
          @return: str
          """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read().strip()
 
             return self.translate(text=text)

deep_translator/papago.py

@@ -105,7 +105,7 @@ class PapagoTranslator(object):
         @return: str
         """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read().strip()
             return self.translate(text)
         except Exception as e:

deep_translator/parent.py

@@ -3,6 +3,8 @@
 from .exceptions import NotValidPayload, NotValidLength, InvalidSourceOrTargetLanguage
 from abc import ABC, abstractmethod
 import string
+
+
 class BaseTranslator(ABC):
     """
     Abstract class that serve as a parent translator for other different translators

deep_translator/qcri.py

@@ -3,6 +3,7 @@ import requests
 from .constants import BASE_URLS, QCRI_LANGUAGE_TO_CODE
 from .exceptions import (ServerException, TranslationNotFound)
 
+
 class QCRI(object):
     """
     class that wraps functions, which use the QRCI translator under the hood to translate word(s)

deep_translator/yandex.py

@@ -119,7 +119,7 @@ class YandexTranslator(object):
         @return: translated text
         """
         try:
-            with open(path) as f:
+            with open(path, 'r', encoding='utf-8') as f:
                 text = f.read()
 
             return self.translate(text)