Stuck on an issue?

Proposed change:

--- a/huey/api.py
+++ b/huey/api.py
@@ -356,13 +356,12 @@ class Huey(object):
         try:
             result = task.execute()
         except Exception as exc:
-            if self.result_store:
+            if self.result_store and self.store_errors:
                 metadata = self._get_task_metadata(task, True)
                 metadata['error'] = repr(exc)
                 metadata['traceback'] = traceback.format_exc()
                 self.put(task.task_id, Error(metadata))
-                if self.store_errors:
-                    self.put_error(metadata)
+                self.put_error(metadata)
             raise
 
         if self.result_store and not isinstance(task, PeriodicQueueTask):

So, here’s the relevant code (api.py):

    def execute(self, task):
        if not isinstance(task, QueueTask):
            raise TypeError('Unknown object: %s' % task)

        try:
            result = task.execute()
        except Exception as exc:
            if self.result_store:
                metadata = self._get_task_metadata(task, True)
                metadata['error'] = repr(exc)
                metadata['traceback'] = traceback.format_exc()
                self.put(task.task_id, Error(metadata))
                if self.store_errors:
                    self.put_error(metadata)
            raise

So we’re storing errors as results, but I think the fix would be to move the “store_errors” conditional up to the line reading “if self.result_store”.