Fix pep8 failures, and apply black formatting where applicable.

kivy · Dec 8, 2023 · c46bca7 · c46bca7
1 parent d16addd
commit c46bca7
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 12 deletions.
diff --git a/docs/source/conf.py b/docs/source/conf.py
@@ -56,6 +56,7 @@
 # built documents.
 #
 
+
 # Lookup the version from the pyjnius module, without installing it
 # since readthedocs.org may have issue to install it.
 # Read the version from the __init__.py file, without importing it.
@@ -64,10 +65,14 @@ def get_version():
         os.path.join(os.path.abspath("../.."), "plyer", "__init__.py")
     ) as fp:
         for line in fp:
-            m = re.search(r'^\s*__version__\s*=\s*([\'"])([^\'"]+)\1\s*$', line)
+            m = re.search(
+                r'^\s*__version__\s*=\s*([\'"])([^\'"]+)\1\s*$',
+                line,
+            )
             if m:
                 return m.group(2)
 
+
 # The short X.Y version.
 version = get_version()
 # The full version, including alpha/beta/rc tags.

diff --git a/plyer/platforms/android/filechooser.py b/plyer/platforms/android/filechooser.py
@@ -143,9 +143,11 @@ def _open_file(self, **kwargs):
 
         # create Intent for opening
         file_intent = Intent(Intent.ACTION_GET_CONTENT)
-        if not self.selected_mime_type or \
-            type(self.selected_mime_type) != str or \
-                self.selected_mime_type not in self.mime_type:
+        if (
+            not self.selected_mime_type
+            or not isinstance(self.selected_mime_type, str)
+            or self.selected_mime_type not in self.mime_type
+        ):
             file_intent.setType("*/*")
         else:
             file_intent.setType(self.mime_type[self.selected_mime_type])
@@ -176,9 +178,11 @@ def _save_file(self, **kwargs):
             kwargs.pop("filters")[0] if "filters" in kwargs else ""
 
         file_intent = Intent(Intent.ACTION_CREATE_DOCUMENT)
-        if not self.selected_mime_type or \
-            type(self.selected_mime_type) != str or \
-                self.selected_mime_type not in self.mime_type:
+        if (
+            not self.selected_mime_type
+            or not isinstance(self.selected_mime_type, str)
+            or self.selected_mime_type not in self.mime_type
+        ):
             file_intent.setType("*/*")
         else:
             file_intent.setType(self.mime_type[self.selected_mime_type])

diff --git a/plyer/platforms/linux/filechooser.py b/plyer/platforms/linux/filechooser.py
@@ -122,7 +122,7 @@ def _gen_cmdline(self):
         if self.icon:
             cmdline += ["--window-icon", self.icon]
         for f in self.filters:
-            if type(f) == str:
+            if isinstance(f, str):
                 cmdline += ["--file-filter", f]
             else:
                 cmdline += [
@@ -150,7 +150,7 @@ def _gen_cmdline(self):
         filt = []
 
         for f in self.filters:
-            if type(f) == str:
+            if isinstance(f, str):
                 filt += [f]
             else:
                 filt += list(f[1:])
@@ -215,7 +215,7 @@ def _gen_cmdline(self):
         if self.icon:
             cmdline += ["--window-icon", self.icon]
         for f in self.filters:
-            if type(f) == str:
+            if isinstance(f, str):
                 cmdline += ["--file-filter", f]
             else:
                 cmdline += [

diff --git a/plyer/platforms/macosx/filechooser.py b/plyer/platforms/macosx/filechooser.py
@@ -80,7 +80,7 @@ def run(self):
         if self.filters:
             filthies = []
             for f in self.filters:
-                if type(f) == str:
+                if isinstance(f, str):
                     f = (None, f)
                 for s in f[1:]:
                     if not self.use_extensions:

diff --git a/plyer/platforms/win/filechooser.py b/plyer/platforms/win/filechooser.py
@@ -84,7 +84,7 @@ def run(self):
                 # e.g. open_file(filters=['*.txt', '*.py'])
                 filters = ""
                 for f in self.filters:
-                    if type(f) == str:
+                    if isinstance(f, str):
                         filters += (f + "\x00") * 2
                     else:
                         filters += f[0] + "\x00" + ";".join(f[1:]) + "\x00"