From dd5402b03748c3691237505c4ef0ce871e28b29d Mon Sep 17 00:00:00 2001
From: marco <marco@badshark.io>
Date: Fri, 21 Aug 2015 19:34:35 +0200
Subject: [PATCH] Newsletters can be sent!

---
 assets/js/src/newsletters_form.jsx |  3 +--
 assets/js/src/newsletters_list.jsx | 23 ++++++++++++++++++++---
 assets/js/src/settings.jsx         |  3 ++-
 lib/Router/Newsletters.php         |  9 +++++++++
 tests/unit/Mailer/BridgeCest.php   | 11 +++++++++++
 5 files changed, 43 insertions(+), 6 deletions(-)

diff --git a/assets/js/src/newsletters_form.jsx b/assets/js/src/newsletters_form.jsx
index 4eb38d9bbe..c7d1b5f066 100644
--- a/assets/js/src/newsletters_form.jsx
+++ b/assets/js/src/newsletters_form.jsx
@@ -7,7 +7,6 @@ define('newsletters_form', ['react', 'jquery', 'mailpoet'], function(React, jQue
         action: 'save',
         data: data,
         onSuccess: function(response) {
-          console.log(response);
         }.bind(this)
       })
     },
@@ -41,7 +40,7 @@ define('newsletters_form', ['react', 'jquery', 'mailpoet'], function(React, jQue
           <br />
           <textarea placeholder="Body" ref="body" />
           <br />
-          <input type="submit" value="Post" />
+          <input type="submit" value="Save" />
           </form>
           );
     }
diff --git a/assets/js/src/newsletters_list.jsx b/assets/js/src/newsletters_list.jsx
index 1c4b148121..3262d75c99 100644
--- a/assets/js/src/newsletters_list.jsx
+++ b/assets/js/src/newsletters_list.jsx
@@ -1,12 +1,29 @@
 define('newsletters_list', ['react', 'jquery', 'mailpoet'], function(React, jQuery, MailPoet) {
 
   var Newsletter = React.createClass({
+    send: function(e) {
+      e.preventDefault();
+      MailPoet.Ajax.post({
+        endpoint: 'newsletters',
+        action: 'send',
+        data: this.props.newsletter.id,
+        onSuccess: function(response) {
+          alert('Sent!');
+        },
+        onError: function(response) {
+          alert('Cannot send. Set the settings and add some subscribers!');
+        }
+      })
+    },
+
     render: function() {
       return (
         <div className="newsletter">
-          <h3 className="subject">
-            {this.props.newsletter.subject}
-          </h3>
+          <p className="subject">
+            {this.props.newsletter.subject} - <a href="" onClick={this.send}>
+            Send
+            </a>
+          </p>
         </div>
       );
     }
diff --git a/assets/js/src/settings.jsx b/assets/js/src/settings.jsx
index 83fc8c8914..982444ed7c 100644
--- a/assets/js/src/settings.jsx
+++ b/assets/js/src/settings.jsx
@@ -1,4 +1,5 @@
 define('settings', ['react/addons', 'jquery', 'mailpoet'], function(React, jQuery, MailPoet) {
+
   var SettingsForm = React.createClass({
     mixins: [React.addons.LinkedStateMixin],
     load: function() {
@@ -82,7 +83,7 @@ define('settings', ['react/addons', 'jquery', 'mailpoet'], function(React, jQuer
               From email:
               <input
                 type="text"
-                valueLink={this.linkSettingValue('from_email')} />
+                valueLink={this.linkSettingValue('from_address')} />
             </label>
           </p>
 
diff --git a/lib/Router/Newsletters.php b/lib/Router/Newsletters.php
index e8d9b9d488..d04505a172 100644
--- a/lib/Router/Newsletters.php
+++ b/lib/Router/Newsletters.php
@@ -1,6 +1,8 @@
 <?php
 namespace MailPoet\Router;
 use \MailPoet\Models\Newsletter;
+use \MailPoet\Models\Subscriber;
+use \MailPoet\Mailer\Bridge;
 
 if(!defined('ABSPATH')) exit;
 
@@ -32,4 +34,11 @@ class Newsletters {
   function delete($id) {
 
   }
+
+  function send($id) {
+    $newsletter = Newsletter::find_one($id)->as_array();
+    $subscribers = Subscriber::find_array();
+    $mailer = new Bridge($newsletter, $subscribers);
+    wp_send_json($mailer->send());
+  }
 }
diff --git a/tests/unit/Mailer/BridgeCest.php b/tests/unit/Mailer/BridgeCest.php
index 1552e2f166..b483bc6945 100644
--- a/tests/unit/Mailer/BridgeCest.php
+++ b/tests/unit/Mailer/BridgeCest.php
@@ -96,6 +96,17 @@ class BridgeCest {
       ->equals('application/json');
   }
 
+  function itCannotSendWithoutSubscribers() {
+    $subscribers = array();
+
+    $mailer = new Bridge(
+      $this->newsletter,
+      $subscribers
+    );
+
+    expect($mailer->send())->equals(false);
+  }
+
   function itCanSend() {
     /* $result = $this->mailer->send(); */
     /* expect($result)->equals(true); */