#1481 refactor block and file cli to shows the whole notes in a format json output

Author: ATGE

SoftLayer/CLI/block/list.py

@@ -5,8 +5,7 @@
 import SoftLayer
 from SoftLayer.CLI import columns as column_helper
 from SoftLayer.CLI import environment
-from SoftLayer.CLI import formatting
-
+from SoftLayer.CLI import storage_utils
 
 COLUMNS = [
     column_helper.Column('id', ('id',), mask="id"),
@@ -18,7 +17,7 @@
         'storage_type',
         lambda b: b['storageType']['keyName'].split('_').pop(0)
         if 'storageType' in b and 'keyName' in b['storageType']
-        and isinstance(b['storageType']['keyName'], str)
+           and isinstance(b['storageType']['keyName'], str)
         else '-',
         mask="storageType.keyName"),
     column_helper.Column('capacity_gb', ('capacityGb',), mask="capacityGb"),
@@ -52,8 +51,6 @@
     'notes'
 ]
 
-DEFAULT_NOTES_SIZE = 20
-
 
 @click.command()
 @click.option('--username', '-u', help='Volume username')
@@ -78,24 +75,5 @@ def cli(env, sortby, columns, datacenter, username, storage_type, order):
                                                      order=order,
                                                      mask=columns.mask())
 
-    table = formatting.Table(columns.columns)
-    table.sortby = sortby
-
-    _reduce_notes(block_volumes)
-
-    for block_volume in block_volumes:
-        table.add_row([value or formatting.blank()
-                       for value in columns.row(block_volume)])
-
+    table = storage_utils.build_output_table(env, block_volumes, columns, sortby)
     env.fout(table)
-
-
-def _reduce_notes(block_volumes):
-    """Reduces the size of the notes in a volume list.
-
-    :param block_volumes: An list of block volumes
-    """
-    for block_volume in block_volumes:
-        if len(block_volume.get('notes', '')) > DEFAULT_NOTES_SIZE:
-            shortened_notes = block_volume['notes'][:DEFAULT_NOTES_SIZE]
-            block_volume['notes'] = shortened_notes

SoftLayer/CLI/environment.py

@@ -52,6 +52,10 @@ def fmt(self, output, fmt=None):
             fmt = self.format
         return formatting.format_output(output, fmt)
 
+    def format_output_is_json(self):
+        """Return True if format output is json or jsonraw"""
+        return 'json' in self.format
+
     def fout(self, output, newline=True):
         """Format the input and output to the console (stdout)."""
         if output is not None:

SoftLayer/CLI/file/list.py

@@ -5,7 +5,7 @@
 import SoftLayer
 from SoftLayer.CLI import columns as column_helper
 from SoftLayer.CLI import environment
-from SoftLayer.CLI import formatting
+from SoftLayer.CLI import storage_utils
 
 COLUMNS = [
     column_helper.Column('id', ('id',), mask="id"),
@@ -76,24 +76,5 @@ def cli(env, sortby, columns, datacenter, username, storage_type, order):
                                                   order=order,
                                                   mask=columns.mask())
 
-    table = formatting.Table(columns.columns)
-    table.sortby = sortby
-
-    _reduce_notes(file_volumes)
-
-    for file_volume in file_volumes:
-        table.add_row([value or formatting.blank()
-                       for value in columns.row(file_volume)])
-
+    table = storage_utils.build_output_table(env, file_volumes, columns, sortby)
     env.fout(table)
-
-
-def _reduce_notes(file_volumes):
-    """Reduces the size of the notes in a volume list.
-
-    :param file_volumes: An list of file volumes
-    """
-    for file_volume in file_volumes:
-        if len(file_volume.get('notes', '')) > DEFAULT_NOTES_SIZE:
-            shortened_notes = file_volume['notes'][:DEFAULT_NOTES_SIZE]
-            file_volume['notes'] = shortened_notes

SoftLayer/CLI/storage_utils.py

@@ -2,6 +2,43 @@
 # :license: MIT, see LICENSE for more details.
 
 from SoftLayer.CLI import columns as column_helper
+from SoftLayer.CLI import formatting
+
+DEFAULT_NOTES_SIZE = 20
+
+
+def reduce_notes(volumes, env):
+    """Reduces all long notes found in the volumes list just if the format output is different from a JSON format.
+
+    :param list volumes: An list of storage volumes
+    :param env :A environment console.
+    """
+    if env.format_output_is_json():
+        return
+
+    for volume in volumes:
+        if len(volume.get('notes', '')) > DEFAULT_NOTES_SIZE:
+            shortened_notes = volume['notes'][:DEFAULT_NOTES_SIZE]
+            volume['notes'] = shortened_notes
+
+
+def build_output_table(env, volumes, columns, sortby):
+    """Builds a formatting table for a list of volumes.
+
+        :param env :A Environment console.
+        :param list volumes: An list of storage volumes
+        :param columns :A ColumnFormatter for column names
+        :param str sortby :A string to sort by.
+        """
+    table = formatting.Table(columns.columns)
+    if sortby in table.columns:
+        table.sortby = sortby
+
+    reduce_notes(volumes, env)
+    for volume in volumes:
+        table.add_row([value or formatting.blank()
+                       for value in columns.row(volume)])
+    return table
 
 
 def _format_name(obj):
@@ -96,7 +133,6 @@ def _format_name(obj):
 """),
 ]
 
-
 DEFAULT_COLUMNS = [
     'id',
     'name',