]> git.decadent.org.uk Git - dak.git/commitdiff
get_source_in_suite: return None if suite does not exist
authorAnsgar Burchardt <ansgar@debian.org>
Wed, 7 May 2014 17:57:29 +0000 (19:57 +0200)
committerAnsgar Burchardt <ansgar@debian.org>
Wed, 7 May 2014 17:57:29 +0000 (19:57 +0200)
daklib/dbconn.py

index 988309361b3bef1fa2295c8184bedc8f60eddc31..624ff8b5c83e6b2dbf30186f147bbf4f8d941cef 100644 (file)
@@ -2089,27 +2089,28 @@ __all__.append('get_sources_from_name')
 # FIXME: This function fails badly if it finds more than 1 source package and
 # its implementation is trivial enough to be inlined.
 @session_wrapper
-def get_source_in_suite(source, suite, session=None):
+def get_source_in_suite(source, suite_name, session=None):
     """
-    Returns a DBSource object for a combination of C{source} and C{suite}.
+    Returns a DBSource object for a combination of C{source} and C{suite_name}.
 
       - B{source} - source package name, eg. I{mailfilter}, I{bbdb}, I{glibc}
-      - B{suite} - a suite name, eg. I{unstable}
+      - B{suite_name} - a suite name, eg. I{unstable}
 
     @type source: string
     @param source: source package name
 
-    @type suite: string
+    @type suite_name: string
     @param suite: the suite name
 
     @rtype: string
     @return: the version for I{source} in I{suite}
 
     """
-
-    q = get_suite(suite, session).get_sources(source)
+    suite = get_suite(suite_name, session)
+    if suite is None:
+        return None
     try:
-        return q.one()
+        return suite.get_sources(source).one()
     except NoResultFound:
         return None