Merge pull request #124 from planetlabs/output-report-persistence

Display report with asset locations for all CLI downloads

Author: sarasafavi

planet/scripts/util.py

@@ -201,6 +201,16 @@ class _BaseOutput(object):
 
     refresh_rate = 1
 
+    def _report_complete(self, item, asset, path=None):
+        msg = {
+            'item': item['id'],
+            'asset': asset['type'],
+            'location': path or asset['location']
+        }
+        # cancel() allows report log to persist for both ANSI & regular output
+        self.cancel()
+        click.echo(json.dumps(msg))
+
     def __init__(self, thread, dl):
         self._thread = thread
         self._timer = None
@@ -231,14 +241,6 @@ def cancel(self):
 
 class Output(_BaseOutput):
 
-    def _report_complete(self, item, asset, path=None):
-        msg = {
-            'item': item['id'],
-            'asset': asset['type'],
-            'location': path or asset['location']
-        }
-        click.echo(json.dumps(msg))
-
     def _output(self, stats):
         logging.info('%s', stats)
 
@@ -276,9 +278,6 @@ def _output(self, stats):
             self._stats.update(stats)
             self._do_output()
 
-    def _report_complete(self, item, asset, path=None):
-        pass
-
     def _do_output(self):
         # renders a terminal like:
         # highlighted status rows