Don't explode returned values in QgsTaskWrapper

nyalldawson · nyalldawson · commit fded1a1f394a · 2016-12-05T14:08:11.000+10:00
Breaks returning list values
diff --git a/python/core/__init__.py b/python/core/__init__.py
@@ -224,11 +224,7 @@ def finished(self, result):
 
         try:
             if self.returned_values:
-                # we want to support singular returned values which are not iterable
-                if hasattr(self.returned_values, '__iter__'):
-                    self.on_finished(result, *self.returned_values)
-                else:
-                    self.on_finished(result, self.returned_values)
+                self.on_finished(result, self.returned_values)
             else:
                 self.on_finished(result)
         except Exception as ex:
diff --git a/tests/src/python/test_qgstaskmanager.py b/tests/src/python/test_qgstaskmanager.py
@@ -80,9 +80,9 @@ def run_multiple_val_result(task):
     return 5, 'whoo'
 
 
-def finished_multiple_value_result(result, value, statement):
-    finished_multiple_value_result.value = value
-    finished_multiple_value_result.statement = statement
+def finished_multiple_value_result(result, results):
+    finished_multiple_value_result.value = results[0]
+    finished_multiple_value_result.statement = results[1]
     return
 
 
