[Piuparts-commits] [SCM] piuparts git repository branch, develop, updated. 0.50-99-gdcbc09d

[Andreas Beckmann]

The following commit has been merged in the develop branch:
commit cb0e66e5377a74b031684cf0b002f03ac4d1c5fb
Author: Andreas Beckmann <anbe at debian.org>
Date:   Sun Apr 14 13:17:25 2013 +0200

    lib/db: ensure initialization in public methods
    
    always call _find_all_packages() or _compute_package_states()
    before accessing lazy initialized structures
    
    Signed-off-by: Andreas Beckmann <anbe at debian.org>

diff --git a/piupartslib/packagesdb.py b/piupartslib/packagesdb.py
index 713e669..7ef0da5 100644
--- a/piupartslib/packagesdb.py
+++ b/piupartslib/packagesdb.py
@@ -592,6 +592,7 @@ class PackagesDB:
         return name in self._packages
 
     def get_package(self, name, recurse=False, resolve_virtual=False):
+        self._find_all_packages()
         if name in self._packages:
             return self._packages[name]
         if recurse:
@@ -605,6 +606,7 @@ class PackagesDB:
         return None
 
     def get_providers(self, name, recurse=True):
+        self._find_all_packages()
         providers = []
         if name in self._virtual_packages:
             providers.extend(self._virtual_packages[name])
@@ -618,6 +620,7 @@ class PackagesDB:
         return self._packages
 
     def get_control_header(self, package_name, header):
+        self._find_all_packages()
         if header == "Source":
           # binary packages build from the source package with the same name
           # don't have a Source header, so let's try:
@@ -644,6 +647,7 @@ class PackagesDB:
           return self._packages[package_name][header]
 
     def get_package_state(self, package_name, resolve_virtual=True, recurse=True):
+        self._compute_package_states()
         if package_name in self._package_state:
             return self._package_state[package_name]
         if package_name in self._virtual_packages:

-- 
piuparts git repository