From fab262721a448910bb25759a554b270103ef29ca Mon Sep 17 00:00:00 2001 From: Liam Young Date: Mon, 7 Dec 2015 10:57:58 +0000 Subject: [PATCH] Fix checking auto_accessors for none --- requires.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/requires.py b/requires.py index 8ba7b20..0909484 100644 --- a/requires.py +++ b/requires.py @@ -35,8 +35,6 @@ class KeystoneRequires(RelationBase): self.set_state('{relation_name}.available') else: self.remove_state('{relation_name}.available') - if not self.keystone_hosts(): - self.remove_state('{relation_name}.connected') @hook('{requires:keystone}-relation-changed') def changed(self): @@ -51,7 +49,8 @@ class KeystoneRequires(RelationBase): Get the connection string, if available, or None. """ for field in self.auto_accessors: - if not self.get(field): + attr_method = getattr(self, field.replace('-', '_')) + if not attr_method(): return False return True