From fa5dabca1a02b9058ee4fa2e744f8bc28fa4e3fa Mon Sep 17 00:00:00 2001
From: Gavin D'souza <gavin18d@gmail.com>
Date: Mon, 3 May 2021 19:33:50 +0530
Subject: [PATCH] feat(cli): Format option in bench show-config

(cherry picked from commit 95db79a1d004d626890c4af15fd0b624ab8fa7f4)
---
 frappe/commands/utils.py | 34 +++++++++++++++++++++++-----------
 1 file changed, 23 insertions(+), 11 deletions(-)

diff --git a/frappe/commands/utils.py b/frappe/commands/utils.py
index 208d630c13..47676af69a 100644
--- a/frappe/commands/utils.py
+++ b/frappe/commands/utils.py
@@ -96,23 +96,25 @@ def destroy_all_sessions(context, reason=None):
 		raise SiteNotSpecifiedError
 
 @click.command('show-config')
+@click.option("--format", "-f", type=click.Choice(["text", "json"]), default="text")
 @pass_context
-def show_config(context):
-	"Print configuration file to STDOUT in text format"
+def show_config(context, format):
+	"Print configuration file to STDOUT in speified format"
 
 	if not context.sites:
 		raise SiteNotSpecifiedError
 
+	sites_config = {}
+
 	from frappe.utils.commands import render_table
 
-	def print_config(config, prefix=None, site_config=None):
-		if not site_config:
-			site_config = []
+	def transform_config(config, prefix=None):
 		prefix = f"{prefix}." if prefix else ""
+		site_config = []
 
 		for conf, value in config.items():
 			if isinstance(value, dict):
-				site_config += print_config(value, prefix=f"{prefix}{conf}")
+				site_config += transform_config(value, prefix=f"{prefix}{conf}")
 			else:
 				log_value = json.dumps(value) if isinstance(value, list) else value
 				site_config += [[f"{prefix}{conf}", log_value]]
@@ -122,17 +124,27 @@ def show_config(context):
 	for site in context.sites:
 		frappe.init(site)
 
-		if len(context.sites) != 1:
-			click.secho(f"\nSite {site}", fg="yellow")
+		if len(context.sites) != 1 and format == "text":
+			if context.sites.index(site) != 0:
+				click.echo()
+			click.secho(f"Site {site}", fg="yellow")
 
 		sites_path = frappe.get_site_path()
 		configuration = frappe.get_site_config(sites_path=sites_path, site_path=site)
-		data = print_config(configuration)
-		data.insert(0, ['Config','Value'])
-		render_table(data)
+
+		if format == "text":
+			data = transform_config(configuration)
+			data.insert(0, ['Config','Value'])
+			render_table(data)
+
+		if format == "json":
+			sites_config[site] = configuration
 
 		frappe.destroy()
 
+	if format == "json":
+		click.echo(sites_config)
+
 
 @click.command('reset-perms')
 @pass_context