Skip duplicated content.

Author: dkocher

core/src/main/java/ch/cyberduck/core/StringAppender.java

@@ -51,18 +51,20 @@ public StringAppender append(final String message) {
         if(StringUtils.isBlank(StringUtils.trim(message))) {
             return this;
         }
-        if(buffer.length() > 0) {
-            buffer.append(" ");
-        }
-        buffer.append(StringUtils.trim(message));
-        if(buffer.charAt(buffer.length() - 1) == '.') {
-            return this;
-        }
-        if(buffer.charAt(buffer.length() - 1) == ':') {
-            buffer.deleteCharAt(buffer.length() - 1);
-        }
-        if(!Pattern.matches("[.?!]", String.valueOf(buffer.charAt(buffer.length() - 1)))) {
-            buffer.append(suffix);
+        if(!buffer.toString().contains(StringUtils.trim(message))) {
+            if(buffer.length() > 0) {
+                buffer.append(" ");
+            }
+            buffer.append(StringUtils.trim(message));
+            if(buffer.charAt(buffer.length() - 1) == '.') {
+                return this;
+            }
+            if(buffer.charAt(buffer.length() - 1) == ':') {
+                buffer.deleteCharAt(buffer.length() - 1);
+            }
+            if(!Pattern.matches("[.?!]", String.valueOf(buffer.charAt(buffer.length() - 1)))) {
+                buffer.append(suffix);
+            }
         }
         return this;
     }

core/src/test/java/ch/cyberduck/core/StringAppenderTest.java

@@ -30,7 +30,7 @@ public void testAppend() {
         assertEquals("Message.", new StringAppender().append("Message.").toString());
         assertEquals("Message? t.", new StringAppender().append("Message?").append("t").toString());
         assertEquals("Message).", new StringAppender().append("Message)").toString());
-        assertEquals("m.", new StringAppender().append("m").append(" ").toString());
+        assertEquals("m.", new StringAppender().append("m").append("m").append(" ").toString());
     }
 
     @Test