From 732321818dd01a01a13ee23b76dc7f9c5dbd77fc Mon Sep 17 00:00:00 2001
From: Aditya Bansal <adi.bansal241996@gmail.com>
Date: Tue, 16 Jan 2018 05:15:17 +0530
Subject: [PATCH] scheduledmessage: Adjust schedule_message to accept
 callbacks.

---
 static/js/compose.js | 32 +++++++++++++++++++++-----------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/static/js/compose.js b/static/js/compose.js
index 15788079c0..b702ffad39 100644
--- a/static/js/compose.js
+++ b/static/js/compose.js
@@ -379,7 +379,7 @@ function patch_request_for_scheduling(request) {
     return new_request;
 }
 
-exports.schedule_message = function schedule_message(request) {
+exports.schedule_message = function schedule_message(request, success, error) {
     if (request === undefined) {
         request = create_message_object();
     }
@@ -389,15 +389,27 @@ exports.schedule_message = function schedule_message(request) {
     } else {
         request.to = JSON.stringify([request.to]);
     }
-    function success(data) {
-        notifications.notify_above_composebox('Scheduled your Message to be delivered at: ' + data.deliver_at);
-        $("#compose-textarea").attr('disabled', false);
-        clear_compose_box();
-    }
 
-    function error(response) {
-        $("#compose-textarea").attr('disabled', false);
-        compose_error(response, $('#compose-textarea'));
+    /* success and error callbacks are kind of a package deal here. When scheduling
+    a message either by means of slash command or from message feed, if we need to do
+    something special on success then we will also need to know if our request errored
+    and do something appropriate. Therefore we just check if success callback is not
+    defined and just assume request to be coming from compose box. This is correct
+    because we won't ever actually have success operate in different context than error. */
+    if (success === undefined) {
+        success = function (data) {
+            notifications.notify_above_composebox('Scheduled your Message to be delivered at: ' + data.deliver_at);
+            $("#compose-textarea").attr('disabled', false);
+            clear_compose_box();
+        };
+        error = function (response) {
+            $("#compose-textarea").attr('disabled', false);
+            compose_error(response, $('#compose-textarea'));
+        };
+        /* We are adding a disable on compose under this block since it actually
+        has its place with the branch of code which does stuff when slash command
+        is incoming from compose_box */
+        $("#compose-textarea").attr('disabled', true);
     }
 
     request = patch_request_for_scheduling(request);
@@ -406,9 +418,7 @@ exports.schedule_message = function schedule_message(request) {
        return;
     }
 
-    $("#compose-textarea").attr('disabled', true);
     exports.transmit_message(request, success, error);
-
 };
 
 exports.enter_with_preview_open = function () {