From b930d99708bbda0385bbf428fc2916a3d4a98d11 Mon Sep 17 00:00:00 2001
From: jonahtanjz <jonahtanjz4884@gmail.com>
Date: Fri, 11 Dec 2020 14:14:37 +0800
Subject: [PATCH] Person: Remove 'Tags: ' if there are no tags

An additional phrase 'Tags:' is always present even if the user did
not input any tags.

This is not intuitive as the result message is displaying
unnecessary information.

Let's remove 'Tags:' from the result message if tags are not present.
---
 src/main/java/seedu/address/model/person/Person.java | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/main/java/seedu/address/model/person/Person.java b/src/main/java/seedu/address/model/person/Person.java
index 6c14b7f96ac6..1be1387d96d2 100644
--- a/src/main/java/seedu/address/model/person/Person.java
+++ b/src/main/java/seedu/address/model/person/Person.java
@@ -111,9 +111,13 @@ public String toString() {
                 .append("; Email: ")
                 .append(getEmail())
                 .append("; Address: ")
-                .append(getAddress())
-                .append("; Tags: ");
-        getTags().forEach(builder::append);
+                .append(getAddress());
+
+        Set<Tag> tags = getTags();
+        if (!tags.isEmpty()) {
+            builder.append("; Tags: ");
+            tags.forEach(builder::append);
+        }
         return builder.toString();
     }