make _ always be the gettext function, nothing else

This avoids hard bugs where the _() function gets overidden by a str or
something else.

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 9129dfdf3b3d49639f04bf6245cb7c03992b4966..634de8b5f9847af00b8f60cd85e1fcf7a35ec848 100644 (file)
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -519,7 +519,7 @@ def get_extension(filename):
 
 
 def has_extension(filename, ext):
-    _, f_ext = get_extension(filename)
+    _ignored, f_ext = get_extension(filename)
     return ext == f_ext
 
 
@@ -1745,7 +1745,7 @@ class KnownApks:
                 default_date = datetime.utcnow()
             self.apks[apkName] = (app, default_date)
             self.changed = True
-        _, added = self.apks[apkName]
+        _ignored, added = self.apks[apkName]
         return added
 
     def getapp(self, apkname):

diff --git a/fdroidserver/metadata.py b/fdroidserver/metadata.py
index 16c40a88a38da114e7250dfbe1e136c6ea463794..cc0942582010b9ef5920a940246115b8f2b76893 100644 (file)
--- a/fdroidserver/metadata.py
+++ b/fdroidserver/metadata.py
@@ -921,7 +921,7 @@ def _decode_bool(s):
 def parse_metadata(metadatapath, check_vcs=False):
     '''parse metadata file, optionally checking the git repo for metadata first'''
 
-    _, ext = fdroidserver.common.get_extension(metadatapath)
+    _ignored, ext = fdroidserver.common.get_extension(metadatapath)
     accepted = fdroidserver.common.config['accepted_formats']
     if ext not in accepted:
         warn_or_exception('"%s" is not an accepted format, convert to: %s' % (
@@ -970,7 +970,7 @@ def parse_metadata(metadatapath, check_vcs=False):
             else:
                 root_dir = '.'
             paths = fdroidserver.common.manifest_paths(root_dir, build.gradle)
-            _, _, app.id = fdroidserver.common.parse_androidmanifests(paths, app)
+            _ignored, _ignored, app.id = fdroidserver.common.parse_androidmanifests(paths, app)
 
     return app
 
@@ -1498,7 +1498,7 @@ def write_txt(mf, app):
 
 
 def write_metadata(metadatapath, app):
-    _, ext = fdroidserver.common.get_extension(metadatapath)
+    _ignored, ext = fdroidserver.common.get_extension(metadatapath)
     accepted = fdroidserver.common.config['accepted_formats']
     if ext not in accepted:
         warn_or_exception('Cannot write "%s", not an accepted format, use: %s'

diff --git a/fdroidserver/scanner.py b/fdroidserver/scanner.py
index 030ac7c443edb7c7759a75f6fde70e04c0107827..cad0026b1b1a91ad32ce0f50baf75f52bccd10b2 100644 (file)
--- a/fdroidserver/scanner.py
+++ b/fdroidserver/scanner.py
@@ -185,7 +185,7 @@ def scan_source(build_dir, build):
                 continue
 
             path_in_build_dir = os.path.relpath(filepath, build_dir)
-            _, ext = common.get_extension(path_in_build_dir)
+            _ignored, ext = common.get_extension(path_in_build_dir)
 
             if ext == 'so':
                 count += handleproblem('shared library', path_in_build_dir, filepath)

diff --git a/fdroidserver/update.py b/fdroidserver/update.py
index 6b31f1622360e8155208803c4ec805a23f9da57c..83018a753dfde47b0c98b7ca10f2da6f61fbf5e7 100644 (file)
--- a/fdroidserver/update.py
+++ b/fdroidserver/update.py
@@ -824,7 +824,7 @@ def insert_localized_app_metadata(apps):
                         locale = segments[-2]
                         destdir = os.path.join('repo', packageName, locale)
                     for f in glob.glob(os.path.join(root, d, '*.*')):
-                        _, extension = common.get_extension(f)
+                        _ignored, extension = common.get_extension(f)
                         if extension in ALLOWED_EXTENSIONS:
                             screenshotdestdir = os.path.join(destdir, d)
                             os.makedirs(screenshotdestdir, mode=0o755, exist_ok=True)