From 437d19a130f5337d676638a1c7e122d5177232a5 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Tue, 2 Oct 2012 00:47:38 -0400 Subject: [PATCH] Current if test is testing the existance of the property, test the value (True) instead --- apps/installation/models.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/apps/installation/models.py b/apps/installation/models.py index c11e5802ac..3fd1e230c0 100644 --- a/apps/installation/models.py +++ b/apps/installation/models.py @@ -143,7 +143,9 @@ class Installation(Singleton): else: try: dictionary = {} - if self.is_lsb: + if self.is_lsb == True: + # Explicit True test, if self.is_lsb alone would always + # return True as it exists as a Property instance dictionary.update( { 'is_lsb': unicode(self.is_lsb), @@ -169,7 +171,8 @@ class Installation(Singleton): 'fabfile': unicode(self.fabfile), } ) - if self.is_git_repo: + if self.is_git_repo == True: + # Same as above is_lsb test dictionary.update( { 'repo_remotes': unicode(self.repo_remotes),