From: Hans-Christoph Steiner <hans@eds.org>
Date: Fri, 18 Nov 2016 22:17:19 +0000 (+0100)
Subject: App.get_last_build() method to replace duplicated code
X-Git-Tag: 0.8~141^2~3
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=b91cdffe1797c5f6663efaff1fd4bc676f6d9e2b;p=fdroidserver.git

App.get_last_build() method to replace duplicated code

Look @mvdan, I added a method to the App class!
---

diff --git a/fdroidserver/checkupdates.py b/fdroidserver/checkupdates.py
index 43447ca8..7f5314b8 100644
--- a/fdroidserver/checkupdates.py
+++ b/fdroidserver/checkupdates.py
@@ -109,9 +109,7 @@ def check_tags(app, pattern):
 
         vcs.gotorevision(None)
 
-        last_build = metadata.Build()
-        if len(app.builds) > 0:
-            last_build = app.builds[-1]
+        last_build = app.get_last_build()
 
         if last_build.submodules:
             vcs.initsubmodules()
@@ -322,9 +320,7 @@ def possible_subdirs(app):
     else:
         build_dir = os.path.join('build', app.id)
 
-    last_build = metadata.Build()
-    if len(app.builds) > 0:
-        last_build = app.builds[-1]
+    last_build = app.get_last_build()
 
     for d in dirs_with_manifest(build_dir):
         m_paths = common.manifest_paths(d, last_build.gradle)
@@ -351,9 +347,7 @@ def fetch_autoname(app, tag):
     except VCSException:
         return None
 
-    last_build = metadata.Build()
-    if len(app.builds) > 0:
-        last_build = app.builds[-1]
+    last_build = app.get_last_build()
 
     logging.debug("...fetch auto name from " + build_dir)
     new_name = None
diff --git a/fdroidserver/metadata.py b/fdroidserver/metadata.py
index 198bdea2..212128f3 100644
--- a/fdroidserver/metadata.py
+++ b/fdroidserver/metadata.py
@@ -239,6 +239,12 @@ class App():
                 self.__dict__[k] = v
                 self._modified.add(k)
 
+    def get_last_build(self):
+        if len(self.builds) > 0:
+            return self.builds[-1]
+        else:
+            return Build()
+
 
 TYPE_UNKNOWN = 0
 TYPE_OBSOLETE = 1