From 0ca59c3eb254d5d2650948e3f72662678290d633 Mon Sep 17 00:00:00 2001
From: Robin Moussu <robin.moussu+git@gmail.com>
Date: Sun, 8 Apr 2018 07:50:54 +0200
Subject: Fix diceToText function() when multiple commands are involved

Currently their is no space between expressions in the breakdown of the dices.

Example:
    2d6;3d4*10;4d20*100
Current result (notice the absence of space between the '4' and '1', and
between the '3' and '17'):
    Result: 8,50,4300 - details:[2d6;3d4*10;4d20*100 (4,41,1,317,15,8,3)]
New result:
    Result: 10,80,4300 - details:[2d6;3d4*10;4d20*100 (4 4 - 1 1 3 - 17 15 8 3)]

NB: I used space instead of coma, since it's look better with conditions

Example:
    2d6a[>3];3d4a[>2]*10;4d20a[>10]*100
Current result (notice the abscence of space after each ']')
	8,[5,3],10,[4,6]7,[4,3],1,6,[3,3]3,33,[17,16],10,22,[14,8]
New result:
    8 [5,3] 10 [4,6] - 7 [4,3] 1 6 [3,3] - 3 33 [17,16] 10 22 [14,8]
---
 cli/displaytoolbox.cpp | 108 ++++++++++++++++++++++---------------------------
 1 file changed, 48 insertions(+), 60 deletions(-)

diff --git a/cli/displaytoolbox.cpp b/cli/displaytoolbox.cpp
index 0aef6a7..8a2a681 100644
--- a/cli/displaytoolbox.cpp
+++ b/cli/displaytoolbox.cpp
@@ -170,30 +170,34 @@ QString DisplayToolBox::colorToTermCode(QString str)
     {
         return QStringLiteral("\e[0;31m");
     }
-    if(str==QStringLiteral("white"))
+    else if(str==QStringLiteral("white"))
     {
         return  QStringLiteral("\e[97m");
     }
-    if(str==QStringLiteral("blue"))
+    else if(str==QStringLiteral("blue"))
     {
         return  QStringLiteral("\e[34m");
     }
-    if(str==QStringLiteral("red"))
+    else if(str==QStringLiteral("red"))
     {
         return  QStringLiteral("\e[31m");
     }
-    if(str==QStringLiteral("black"))
+    else if(str==QStringLiteral("black"))
     {
         return  QStringLiteral("\e[30m");
     }
-    if(str==QStringLiteral("green"))
+    else if(str==QStringLiteral("green"))
     {
         return  QStringLiteral("\e[32m");
     }
-    if(str==QStringLiteral("yellow"))
+    else if(str==QStringLiteral("yellow"))
     {
         return  QStringLiteral("\e[33m");
     }
+    else if(str==QStringLiteral("reset"))
+    {
+        return  QStringLiteral("\e[0m");
+    }
     return {};
 }
 
@@ -342,59 +346,43 @@ QString DisplayToolBox::diceResultToString(QJsonObject val)
 QString DisplayToolBox::diceToText(QJsonArray array, bool withColor,bool allSameFaceCount, bool allSameColor)
 {
     Q_UNUSED(allSameColor)
-    if(allSameFaceCount)
-    {
-        QStringList result;
-        for(auto item : array)
-        {
-            QStringList subResult;
-            auto obj = item.toObject();
-            auto values= obj["values"].toArray();
-            for(auto valRef : values)
-            {
-                subResult.append(diceResultToString(valRef.toObject()));
-            }
-            if(withColor)
-            {
-                result.append(DisplayToolBox::colorToTermCode(obj["color"].toString()));
-                result.append(subResult.join(','));
-                result.append(QStringLiteral("\e[0m"));
-            }
-            else
-            {
-                result.append(subResult.join(','));
-            }
-        }
-        return result.join("");
-    }
-    else
-    {
-        QStringList result;
-        for(auto item : array)
-        {
-            QStringList subResult;
-            auto obj = item.toObject();
-            auto values= obj["values"].toArray();
+	QStringList result;
+	for(auto item : array)
+	{
+		QString subResult;
+		auto obj = item.toObject();
+		auto values= obj["values"].toArray();
 
-            for(auto valRef : values)
-            {
-                subResult.append(diceResultToString(valRef.toObject()));
-            }
-            result.append(QStringLiteral("d%1:(").arg(obj["face"].toString()));
-            if(withColor)
-            {
-                result.append(DisplayToolBox::colorToTermCode(obj["color"].toString()));
-            }
-            result.append(subResult.join(','));
-            if(withColor)
-            {
-                result.append(QStringLiteral("\e[0m)"));
-            }
-            else
-            {
-                result.append(QStringLiteral(")"));
-            }
-        }
-        return result.join(' ');
-    }
+		QStringList diceResult;
+		for(auto valRef : values)
+		{
+			diceResult += diceResultToString(valRef.toObject());
+		}
+		if (!diceResult.isEmpty())
+		{
+			if(!allSameFaceCount)
+			{
+				subResult += QStringLiteral("d%1:(").arg(obj["face"].toString());
+			}
+			if(withColor)
+			{
+				subResult += DisplayToolBox::colorToTermCode(obj["color"].toString());
+			}
+			subResult += diceResult.join(" ");
+			if(withColor)
+			{
+				subResult += DisplayToolBox::colorToTermCode(QStringLiteral("reset"));
+			}
+			if(!allSameFaceCount)
+			{
+				subResult += QStringLiteral(")");
+			}
+		}
+
+		if (!subResult.isEmpty())
+		{
+			result += subResult;
+		}
+	}
+	return result.join(" - ");
 }
-- 
cgit v1.2.3-70-g09d2