From 1000975e52c529eaf5efaab0e30fdcb709d6f55c Mon Sep 17 00:00:00 2001
From: Simon Eskildsen <sirup@sirupsen.com>
Date: Fri, 14 Mar 2014 13:01:06 -0400
Subject: [PATCH] text_formatter: change to simpler k-v when no tty

---
 text_formatter.go | 87 +++++++++++++++++++++++++++++++++--------------
 1 file changed, 61 insertions(+), 26 deletions(-)

diff --git a/text_formatter.go b/text_formatter.go
index f6b9d14..b4936cf 100644
--- a/text_formatter.go
+++ b/text_formatter.go
@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"sort"
+	"strconv"
 	"strings"
 
 	"github.com/burke/ttyutils"
@@ -23,37 +24,71 @@ type TextFormatter struct {
 func (f *TextFormatter) Format(entry *Entry) ([]byte, error) {
 	var serialized []byte
 
-	levelText := strings.ToUpper(entry.Data["level"].(string))[0:4]
-	levelColor := blue
-
-	if entry.Data["level"] == "warning" {
-		levelColor = yellow
-	} else if entry.Data["level"] == "fatal" ||
-		entry.Data["level"] == "panic" {
-		levelColor = red
-	}
-
 	if ttyutils.IsTerminal(os.Stdout.Fd()) {
-		serialized = append(serialized, []byte(fmt.Sprintf("\x1b[%dm%s\x1b[0m[%04d] %-45s ", levelColor, levelText, miniTS(), entry.Data["msg"]))...)
-	}
+		levelText := strings.ToUpper(entry.Data["level"].(string))[0:4]
 
-	keys := make([]string, 0)
-	for k, _ := range entry.Data {
-		if k != "level" && k != "time" && k != "msg" {
-			keys = append(keys, k)
+		levelColor := blue
+
+		if entry.Data["level"] == "warning" {
+			levelColor = yellow
+		} else if entry.Data["level"] == "fatal" ||
+			entry.Data["level"] == "panic" {
+			levelColor = red
 		}
-	}
-	sort.Strings(keys)
-	first := true
-	for _, k := range keys {
-		v := entry.Data[k]
-		if first {
-			first = false
-		} else {
-			serialized = append(serialized, ' ')
+
+		serialized = append(serialized, []byte(fmt.Sprintf("\x1b[%dm%s\x1b[0m[%04d] %-45s ", levelColor, levelText, miniTS(), entry.Data["msg"]))...)
+
+		keys := make([]string, 0)
+		for k, _ := range entry.Data {
+			if k != "level" && k != "time" && k != "msg" {
+				keys = append(keys, k)
+			}
+		}
+		sort.Strings(keys)
+		first := true
+		for _, k := range keys {
+			v := entry.Data[k]
+			if first {
+				first = false
+			} else {
+				serialized = append(serialized, ' ')
+			}
+			serialized = append(serialized, []byte(fmt.Sprintf("\x1b[%dm%s\x1b[0m=%v", levelColor, k, v))...)
+		}
+	} else {
+		serialized = f.AppendKeyValue(serialized, "time", entry.Data["time"].(string))
+		serialized = f.AppendKeyValue(serialized, "level", entry.Data["level"].(string))
+		serialized = f.AppendKeyValue(serialized, "msg", entry.Data["msg"].(string))
+
+		for key, value := range entry.Data {
+			if key != "time" && key != "level" && key != "msg" {
+				serialized = append(serialized, []byte(fmt.Sprintf("%s=%s ",
+					f.ToString(key, false), f.ToString(value, true)))...)
+			}
 		}
-		serialized = append(serialized, []byte(fmt.Sprintf("\x1b[%dm%s\x1b[0m=%v", levelColor, k, v))...)
 	}
 
 	return append(serialized, '\n'), nil
 }
+
+func (f *TextFormatter) AppendKeyValue(serialized []byte, key, value string) []byte {
+	return append(serialized, []byte(fmt.Sprintf("%s=%s ",
+		f.ToString(key, false), f.ToString(value, true)))...)
+}
+
+func (f *TextFormatter) ToString(value interface{}, escapeStrings bool) string {
+	switch value.(type) {
+	default:
+		if escapeStrings {
+			return fmt.Sprintf("'%s'", value)
+		} else {
+			return fmt.Sprintf("%s", value)
+		}
+	case int:
+		return fmt.Sprintf("%s", strconv.Itoa(value.(int)))
+	case uint64:
+		return fmt.Sprintf("%s", strconv.FormatUint(value.(uint64), 10))
+	case bool:
+		return fmt.Sprintf("%s", strconv.FormatBool(value.(bool)))
+	}
+}