From 3dbf99a7f29bdbc368ad2e818dc8dc4fc9616a60 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Daniel=20Mart=C3=AD?= <mvdan@mvdan.cc>
Date: Thu, 3 Jul 2014 17:35:28 +0200
Subject: [PATCH] Use shorter and non-redundant 'or' clauses for 'if True else'
 assignments

---
 fdroidserver/checkupdates.py | 6 +++---
 fdroidserver/common.py       | 6 +++---
 fdroidserver/import.py       | 4 ++--
 fdroidserver/lint.py         | 2 +-
 fdroidserver/metadata.py     | 2 +-
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/fdroidserver/checkupdates.py b/fdroidserver/checkupdates.py
index 7d9689ab..8325501c 100644
--- a/fdroidserver/checkupdates.py
+++ b/fdroidserver/checkupdates.py
@@ -90,7 +90,7 @@ def check_tags(app, pattern):
 
     try:
 
-        appid = app['Update Check Name'] if app['Update Check Name'] else app['id']
+        appid = app['Update Check Name'] or app['id']
         if app['Repo Type'] == 'srclib':
             build_dir = os.path.join('build', 'srclib', app['Repo'])
             repotype = common.getsrclibvcs(app['Repo'])
@@ -174,7 +174,7 @@ def check_repomanifest(app, branch=None):
 
     try:
 
-        appid = app['Update Check Name'] if app['Update Check Name'] else app['id']
+        appid = app['Update Check Name'] or app['id']
         if app['Repo Type'] == 'srclib':
             build_dir = os.path.join('build', 'srclib', app['Repo'])
             repotype = common.getsrclibvcs(app['Repo'])
@@ -312,7 +312,7 @@ def dirs_with_manifest(startdir):
 # subdir relative to the build dir if found, None otherwise.
 def check_changed_subdir(app):
 
-    appid = app['Update Check Name'] if app['Update Check Name'] else app['id']
+    appid = app['Update Check Name'] or app['id']
     if app['Repo Type'] == 'srclib':
         build_dir = os.path.join('build', 'srclib', app['Repo'])
     else:
diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 69e6e68b..6376b530 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -490,7 +490,7 @@ class vcs_git(vcs):
                 self.refreshed = True
         # origin/HEAD is the HEAD of the remote, e.g. the "default branch" on
         # a github repo. Most of the time this is the same as origin/master.
-        rev = str(rev if rev else 'origin/HEAD')
+        rev = rev or 'origin/HEAD'
         p = SilentPopen(['git', 'checkout', '-f', rev], cwd=self.local)
         if p.returncode != 0:
             raise VCSException("Git checkout of '%s' failed" % rev, p.output)
@@ -608,7 +608,7 @@ class vcs_gitsvn(vcs):
                     raise VCSException("Git svn rebase failed", p.output)
                 self.refreshed = True
 
-        rev = str(rev if rev else 'master')
+        rev = rev or 'master'
         if rev:
             nospaces_rev = rev.replace(' ', '%20')
             # Try finding a svn tag
@@ -723,7 +723,7 @@ class vcs_hg(vcs):
                     raise VCSException("Hg pull failed", p.output)
                 self.refreshed = True
 
-        rev = str(rev if rev else 'default')
+        rev = rev or 'default'
         if not rev:
             return
         p = SilentPopen(['hg', 'update', '-C', rev], cwd=self.local)
diff --git a/fdroidserver/import.py b/fdroidserver/import.py
index e6c2fa16..a241b2ba 100644
--- a/fdroidserver/import.py
+++ b/fdroidserver/import.py
@@ -285,8 +285,8 @@ def main():
 
     # Create a build line...
     build = {}
-    build['version'] = version if version else '?'
-    build['vercode'] = vercode if vercode else '?'
+    build['version'] = version or '?'
+    build['vercode'] = vercode or '?'
     build['commit'] = '?'
     build['disable'] = 'Generated by import.py - check/set version fields and commit id'
     if options.subdir:
diff --git a/fdroidserver/lint.py b/fdroidserver/lint.py
index 06e2f532..123a6441 100644
--- a/fdroidserver/lint.py
+++ b/fdroidserver/lint.py
@@ -189,7 +189,7 @@ def main():
 
         # Redundant summaries
         summary = app['Summary']
-        name = str(app['Name'] if app['Name'] else app['Auto Name'])
+        name = app['Name'] or app['Auto Name']
         if summary and name:
             summary_l = summary.lower()
             name_l = name.lower()
diff --git a/fdroidserver/metadata.py b/fdroidserver/metadata.py
index e99a6eca..6ed84105 100644
--- a/fdroidserver/metadata.py
+++ b/fdroidserver/metadata.py
@@ -794,7 +794,7 @@ def write_metadata(dest, app):
                 mf.write("%s\n" % comment)
                 written += 1
         if written > 0:
-            logging.debug("...writing comments for " + (key if key else 'EOF'))
+            logging.debug("...writing comments for " + (key or 'EOF'))
 
     def writefield(field, value=None):
         writecomments(field)
-- 
2.30.2