diff options
author | dol-sen <brian.dolbec@gmail.com> | 2011-07-12 12:01:43 -0700 |
---|---|---|
committer | dol-sen <brian.dolbec@gmail.com> | 2011-07-12 12:01:43 -0700 |
commit | 36b0e7f01af24ed622d7ebab1bda9ad00ab8dcfc (patch) | |
tree | c57081584095f603963ee6e3c29d0eb6ab3a418d /layman/api.py | |
parent | 925f99c614b8708e1b79681ee7ae166152f0c396 (diff) | |
download | layman-36b0e7f01af24ed622d7ebab1bda9ad00ab8dcfc.tar.gz layman-36b0e7f01af24ed622d7ebab1bda9ad00ab8dcfc.tar.bz2 layman-36b0e7f01af24ed622d7ebab1bda9ad00ab8dcfc.zip |
fix an errant mistracking of the success of adding an overlay.
Diffstat (limited to 'layman/api.py')
-rw-r--r-- | layman/api.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/layman/api.py b/layman/api.py index 8e54d7d..b494b58 100644 --- a/layman/api.py +++ b/layman/api.py @@ -115,9 +115,9 @@ class LaymanAPI(object): results.append(False) continue try: - self._get_installed_db().delete( + success = self._get_installed_db().delete( self._get_installed_db().select(ovl)) - results.append(True) + results.append(success) except Exception as e: self._error( "Failed to disable repository '"+ovl+"':\n"+str(e)) @@ -147,12 +147,10 @@ class LaymanAPI(object): self._error(UnknownOverlayMessage(ovl)) results.append(False) continue - try: - self._get_installed_db().add(self._get_remote_db().select(ovl), quiet=True) - results.append(True) - except Exception as e: + success = self._get_installed_db().add(self._get_remote_db().select(ovl), quiet=True) + results.append(success) + if not success: self._error("Failed to enable repository '"+ovl+"' : "+str(e)) - results.append(False) self.get_installed(dbreload=True) if False in results: return False |