removed param to send attachment over websocket

Itiviti · Feb 22, 2016 · 551612e · 551612e
1 parent ba8eab6
commit 551612e
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 8 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -0,0 +1 @@
+org.gradle.daemon=true
diff --git a/src/main/java/com/ullink/slack/simpleslackapi/SlackSession.java b/src/main/java/com/ullink/slack/simpleslackapi/SlackSession.java
@@ -74,7 +74,7 @@ public interface SlackSession {
 
     SlackMessageHandle<SlackMessageReply> updateMessage(String timeStamp, SlackChannel channel, String message);
 
-    SlackMessageHandle<SlackMessageReply> sendMessageOverWebSocket(SlackChannel channel, String message, SlackAttachment attachment);
+    SlackMessageHandle<SlackMessageReply> sendMessageOverWebSocket(SlackChannel channel, String message);
 
     SlackMessageHandle<SlackMessageReply> addReactionToMessage(SlackChannel channel, String messageTimeStamp, String emojiCode);
 

diff --git a/src/main/java/com/ullink/slack/simpleslackapi/impl/SlackWebSocketSessionImpl.java b/src/main/java/com/ullink/slack/simpleslackapi/impl/SlackWebSocketSessionImpl.java
@@ -674,7 +674,7 @@ private HttpClient getHttpClient()
     }
 
     @Override
-    public SlackMessageHandle<SlackMessageReply> sendMessageOverWebSocket(SlackChannel channel, String message, SlackAttachment attachment)
+    public SlackMessageHandle<SlackMessageReply> sendMessageOverWebSocket(SlackChannel channel, String message)
     {
         SlackMessageHandleImpl<SlackMessageReply> handle = new SlackMessageHandleImpl<SlackMessageReply>(getNextMessageId());
         try
@@ -683,10 +683,7 @@ public SlackMessageHandle<SlackMessageReply> sendMessageOverWebSocket(SlackChann
             messageJSON.put("type", "message");
             messageJSON.put("channel", channel.getId());
             messageJSON.put("text", message);
-            if (attachment != null)
-            {
-                messageJSON.put("attachments", SlackJSONAttachmentFormatter.encodeAttachments(attachment));
-            }
+
             websocketSession.getBasicRemote().sendText(messageJSON.toJSONString());
         }
         catch (Exception e)

diff --git a/src/test/java/com/ullink/slack/simpleslackapi/impl/TestAbstractSlackSessionImpl.java b/src/test/java/com/ullink/slack/simpleslackapi/impl/TestAbstractSlackSessionImpl.java
@@ -55,7 +55,7 @@ public SlackMessageHandle sendMessage(SlackChannel channel, String message, Slac
         }
 
         @Override
-        public SlackMessageHandle sendMessageOverWebSocket(SlackChannel channel, String message, SlackAttachment attachment)
+        public SlackMessageHandle sendMessageOverWebSocket(SlackChannel channel, String message)
         {
             return null;
         }

diff --git a/src/test/java/com/ullink/slack/simpleslackapi/impl/TestSlackJSONMessageParser.java b/src/test/java/com/ullink/slack/simpleslackapi/impl/TestSlackJSONMessageParser.java
@@ -86,7 +86,7 @@ public SlackMessageHandle sendMessage(SlackChannel channel, String message, Slac
             }
 
             @Override
-            public SlackMessageHandle sendMessageOverWebSocket(SlackChannel channel, String message, SlackAttachment attachment) {
+            public SlackMessageHandle sendMessageOverWebSocket(SlackChannel channel, String message) {
                 throw new UnsupportedOperationException();
             }