diff options
author | Dylan Baker <dylan@pnwbakers.com> | 2017-08-16 11:37:31 -0700 |
---|---|---|
committer | Dylan Baker <dylan@pnwbakers.com> | 2017-08-19 12:52:26 -0700 |
commit | dee41bb04906d6976b4f7c05c222f7d95182b3ea (patch) | |
tree | 2dbe16e47e3b8fee167d9375e83bd42585e78464 /alot | |
parent | 188d79a0189b480656542a09be348fcbf506f33d (diff) |
Replace Exception.message with str(Exception)
In python3 Exception doesn't have a message attribute, the only way to
get the string output is to call str() on the Exception. This also works
in python 2.7, so go ahead and make that change.
Diffstat (limited to 'alot')
-rw-r--r-- | alot/commands/__init__.py | 2 | ||||
-rw-r--r-- | alot/commands/envelope.py | 10 | ||||
-rw-r--r-- | alot/commands/thread.py | 6 | ||||
-rw-r--r-- | alot/commands/utils.py | 2 | ||||
-rw-r--r-- | alot/crypto.py | 4 | ||||
-rw-r--r-- | alot/errors.py | 1 | ||||
-rw-r--r-- | alot/settings/manager.py | 2 | ||||
-rw-r--r-- | alot/settings/utils.py | 2 | ||||
-rw-r--r-- | alot/ui.py | 2 | ||||
-rw-r--r-- | alot/utils/configobj.py | 4 |
10 files changed, 18 insertions, 17 deletions
diff --git a/alot/commands/__init__.py b/alot/commands/__init__.py index 351fc993..0638926a 100644 --- a/alot/commands/__init__.py +++ b/alot/commands/__init__.py @@ -167,7 +167,7 @@ def commandfactory(cmdline, mode='global'): try: args = split_commandstring(cmdline) except ValueError as e: - raise CommandParseError(e.message) + raise CommandParseError(str(e)) args = [string_decode(x, 'utf-8') for x in args] logging.debug('ARGS: %s', args) cmdname = args[0] diff --git a/alot/commands/envelope.py b/alot/commands/envelope.py index b9ff51d6..4d2963ee 100644 --- a/alot/commands/envelope.py +++ b/alot/commands/envelope.py @@ -146,8 +146,8 @@ class SaveCommand(Command): ui.apply_command(globals.FlushCommand()) ui.apply_command(commands.globals.BufferCloseCommand()) except DatabaseError as e: - logging.error(e.message) - ui.notify('could not index message:\n%s' % e.message, + logging.error(e) + ui.notify('could not index message:\n%s' % e, priority='error', block=True) else: @@ -217,7 +217,7 @@ class SendCommand(Command): self.mail = email_as_string(self.mail) except GPGProblem as e: ui.clear_notify([clearme]) - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') return ui.clear_notify([clearme]) @@ -499,7 +499,7 @@ class SignCommand(Command): sign=True) except GPGProblem as e: envelope.sign = False - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') return else: try: @@ -571,7 +571,7 @@ class EncryptCommand(Command): tmp_key = crypto.get_key(keyid) del envelope.encrypt_keys[tmp_key.fpr] except GPGProblem as e: - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') if not envelope.encrypt_keys: envelope.encrypt = False ui.current_buffer.rebuild() diff --git a/alot/commands/thread.py b/alot/commands/thread.py index acbb4e28..d73ef622 100644 --- a/alot/commands/thread.py +++ b/alot/commands/thread.py @@ -186,7 +186,7 @@ class ReplyCommand(Command): try: from_header, _ = determine_sender(mail, 'reply') except AssertionError as e: - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') return envelope.add('From', from_header) @@ -394,7 +394,7 @@ class ForwardCommand(Command): try: from_header, _ = determine_sender(mail, 'reply') except AssertionError as e: - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') return envelope.add('From', from_header) @@ -441,7 +441,7 @@ class BounceMailCommand(Command): try: resent_from_header, account = determine_sender(mail, 'bounce') except AssertionError as e: - ui.notify(e.message, priority='error') + ui.notify(str(e), priority='error') return mail['Resent-From'] = resent_from_header diff --git a/alot/commands/utils.py b/alot/commands/utils.py index 0aa80656..1bf64cbb 100644 --- a/alot/commands/utils.py +++ b/alot/commands/utils.py @@ -86,7 +86,7 @@ def _get_keys(ui, encrypt_keyids, block_error=False, signed_only=False): encrypt_keyids.append(keyid) continue else: - ui.notify(e.message, priority='error', block=block_error) + ui.notify(str(e), priority='error', block=block_error) continue keys[key.fpr] = key returnValue(keys) diff --git a/alot/crypto.py b/alot/crypto.py index cd03d9e7..52eb8c58 100644 --- a/alot/crypto.py +++ b/alot/crypto.py @@ -196,7 +196,7 @@ def verify_detached(message, signature): except gpg.errors.BadSignatures as e: raise GPGProblem(str(e), code=GPGCode.BAD_SIGNATURE) except gpg.errors.GPGMEError as e: - raise GPGProblem(e.message, code=e.getcode()) + raise GPGProblem(str(e), code=e.getcode()) def decrypt_verify(encrypted): @@ -212,7 +212,7 @@ def decrypt_verify(encrypted): try: (plaintext, _, verify_result) = ctx.decrypt(encrypted, verify=True) except gpg.errors.GPGMEError as e: - raise GPGProblem(e.message, code=e.getcode()) + raise GPGProblem(str(e), code=e.getcode()) # what if the signature is bad? return verify_result.signatures, plaintext diff --git a/alot/errors.py b/alot/errors.py index c312e674..b9763192 100644 --- a/alot/errors.py +++ b/alot/errors.py @@ -19,6 +19,7 @@ class GPGCode(object): class GPGProblem(Exception): """GPG Error""" + def __init__(self, message, code): self.code = code super(GPGProblem, self).__init__(message) diff --git a/alot/settings/manager.py b/alot/settings/manager.py index 6f9273c2..6c9e80fe 100644 --- a/alot/settings/manager.py +++ b/alot/settings/manager.py @@ -123,7 +123,7 @@ class SettingsManager(object): except ConfigError as e: logging.warning( 'Theme file %s failed validation: %s', - themestring, str(e.message)) + themestring, e) else: break else: diff --git a/alot/settings/utils.py b/alot/settings/utils.py index 23278607..ea56b264 100644 --- a/alot/settings/utils.py +++ b/alot/settings/utils.py @@ -44,7 +44,7 @@ def read_config(configpath=None, specpath=None, checks=None): try: results = config.validate(validator, preserve_errors=True) except ConfigObjError as e: - raise ConfigError(e.message) + raise ConfigError(str(e)) if results is not True: error_msg = '' @@ -174,7 +174,7 @@ class UI(object): try: self.apply_commandline(cmdline) except CommandParseError as e: - self.notify(e.message, priority='error') + self.notify(str(e), priority='error') # move keys are always passed elif cmdline in ['move up', 'move down', 'move page up', 'move page down']: diff --git a/alot/utils/configobj.py b/alot/utils/configobj.py index 5e04409a..d7ce48a4 100644 --- a/alot/utils/configobj.py +++ b/alot/utils/configobj.py @@ -47,7 +47,7 @@ def attr_triple(value): normal = AttrSpec(acc['16fg'], acc['16bg'], 16) high = AttrSpec(acc['256fg'], acc['256bg'], 256) except AttrSpecError as e: - raise ValidateError(e.message) + raise ValidateError(str(e)) return mono, normal, high @@ -141,4 +141,4 @@ def gpg_key(value): try: return crypto.get_key(value) except GPGProblem as e: - raise ValidateError(e.message) + raise ValidateError(str(e)) |