TX-16599: Native push --verbose command group created, updated, skipped, failed strings

Author: foteinigk

transifex/native/django/management/utils/push.py

@@ -3,6 +3,7 @@
 import os
 import sys
 import time
+import json
 
 from django import VERSION as DJANGO_VERSION
 from django.conf import settings
@@ -295,59 +296,128 @@ def _show_collect_results(self):
             )
             Color.echo(file_list)
 
+    # ---------------- New helpers for verbose group printing ---------------- #
+
+    def _normalize_array(self, maybe):
+        """Normalize possibly-stringified arrays into a python list."""
+        if maybe is None:
+            return []
+        if isinstance(maybe, list):
+            return maybe
+        if isinstance(maybe, (str, bytes)):
+            try:
+                parsed = json.loads(maybe)
+                return parsed if isinstance(parsed, list) else []
+            except Exception:
+                return []
+        return []
+
+    def _print_verbose_group(self, label, items):
+        """Print one verbose group (Created/Updated/etc) in a readable format."""
+        arr = self._normalize_array(items)
+        if not arr:
+            return
+        Color.echo('  [high]{label}:[end] [warn]{n}[end]'.format(
+            label=label, n=len(arr)
+        ))
+        for item in arr:
+            string = item.get('string', '')
+            key = item.get('key', '')
+            occurrences = item.get('occurrences', [])
+            context = item.get('context', [])
+            if not isinstance(occurrences, list):
+                occurrences = []
+            if not isinstance(context, list):
+                context = []
+
+            Color.echo('    └─ {}'.format(json.dumps(string)))
+            Color.echo('      └─ key: {}'.format(json.dumps(key)))
+            Color.echo('      └─ occurrences: {}'.format(json.dumps(occurrences)))
+            Color.echo('      └─ context: {}'.format(json.dumps(context)))
+
+    # ------------------------------------------------------------------------ #
+
     def _show_push_results(self, status_code, response_content):
         """Display results of pushing the source strings to CDS.
 
         :param int status_code: the HTTP status code
         :param dict response_content: the content of the response
         """
         try:
-            data = response_content.get('data')
+            data = response_content.get('data', {})
             status = data.get('status')
-            errors = data.get('errors', [])
+            errors = data.get('errors', []) or []
             if status == 'completed':
-                details = data.get('details')
-                created = details.get('created')
-                updated = details.get('updated')
-                skipped = details.get('skipped')
-                deleted = details.get('deleted')
-                failed = details.get('failed')
+                details = data.get('details', {}) or {}
+                created = details.get('created', 0)
+                updated = details.get('updated', 0)
+                skipped = details.get('skipped', 0)
+                deleted = details.get('deleted', 0)
+                failed = details.get('failed', 0)
+                verbose = details.get('verbose', {}) or {}
+
                 Color.echo(
                     '[green]\nSuccessfully pushed strings to Transifex.[end]'
                 )
 
-                if created > 0:
-                    Color.echo(
-                        '[high]Created strings:[end] '
-                        '[warn]{created}[end]'.format(created=created))
-
-                if updated > 0:
-                    Color.echo(
-                        '[high]Updated strings:[end] '
-                        '[warn]{updated}[end]'.format(updated=updated))
-
-                if skipped > 0:
-                    Color.echo(
-                        '[high]Skipped strings:[end] '
-                        '[warn]{skipped}[end]'.format(skipped=skipped))
-
-                if deleted > 0:
-                    Color.echo(
-                        '[high]Deleted strings:[end] '
-                        '[warn]{deleted}[end]'.format(deleted=deleted))
-
-                if failed > 0:
-                    Color.echo(
-                        '[high]Failed strings:[end] '
-                        '[warn]{failed}[end]'.format(failed=failed))
+                if verbose:
+                    self._print_verbose_group(
+                        'Created strings',
+                        verbose.get('created')
+                    )
+                    self._print_verbose_group(
+                        'Updated strings',
+                        verbose.get('updated')
+                    )
+                    self._print_verbose_group(
+                        'Skipped strings',
+                        verbose.get('skipped')
+                    )
+                    self._print_verbose_group(
+                        'Deleted strings',
+                        verbose.get('deleted')
+                    )
+                    self._print_verbose_group(
+                        'Failed strings',
+                        verbose.get('failed')
+                    )
+                else:
+
+                    if created > 0:
+                        Color.echo(
+                            '[high]Created strings:[end] '
+                            '[warn]{created}[end]'.format(created=created))
+
+                    if updated > 0:
+                        Color.echo(
+                            '[high]Updated strings:[end] '
+                            '[warn]{updated}[end]'.format(updated=updated))
+
+                    if skipped > 0:
+                        Color.echo(
+                            '[high]Skipped strings:[end] '
+                            '[warn]{skipped}[end]'.format(skipped=skipped))
+
+                    if deleted > 0:
+                        Color.echo(
+                            '[high]Deleted strings:[end] '
+                            '[warn]{deleted}[end]'.format(deleted=deleted))
+
+                    if failed > 0:
+                        Color.echo(
+                            '[high]Failed strings:[end] '
+                            '[warn]{failed}[end]'.format(failed=failed))
+
             else:
                 Color.echo(
                     '[error]\nCould not push strings to Transifex.[end]')
 
-            if len(errors) > 0:
+            if errors:
                 Color.echo(
                     '[high]Errors:[end] {errors}[end]\n'.format(
-                        errors='\n'.join(errors)
+                        errors='\n'.join(
+                            [json.dumps(e) if not isinstance(e, str) else e for e in errors]
+                        )
                     )
                 )
         except Exception: