added emails and sending emails also test emails

app/controllers/admin/e_paper_criterias_controller.rb

@@ -1,3 +1,4 @@
+require 'rake'
 class Admin::EPaperCriteriasController < OrbitAdminController
 
 def initialize
@@ -46,10 +47,17 @@ def initialize
   
   def send_test_email
     paper_criteria = PaperCriteria.find(params[:id])
-    page = Page.where(:module_app => "e_paper").first.url rescue ""     
+    page = Page.where(:module => "e_paper").first.url rescue ""     
-    email = Email.new(:mail_subject => paper_criteria.title, :mail_lang => :zh_tw, :template => "admin/e_paper_criterias/emailer", :template_data => {"paper" => paper_criteria, "page" => "http://" + request.host_with_port + page}, :mail_to => params[:email])     
+    email = Email.new(:mail_subject => paper_criteria.sending_title, :mail_lang => :zh_tw, :template => "admin/e_paper_criterias/emailer", :template_data => {"paper" => paper_criteria, "page" => "http://" + request.host_with_port + page}, :mail_to => params[:email])     
     email.deliver
+    render :json => {"success" => true}.to_json
+  end
 
+  def send_newsletter
+    Orbit::Application.load_tasks
+    rake = Rake::Task['epaper:send_email'].execute(OpenStruct.new({:paper_criteria_id => params[:id], :domain => request.host_with_port}))
+    #  rake = Rake::Task['epaper:send_email'].invoke([params[:id], request.host_with_port])
+    render :json => {"success" => true}.to_json
   end
 
   private

app/models/paper_criteria.rb

@@ -10,8 +10,8 @@ class PaperCriteria
   field :sending_time, type: DateTime
   field :sending_title, localize: true
   field :sending_notes
-  field :language_to_send, type: Array, :default => []
+  field :language_to_send, type: Array, :default => ["en","zh_tw"]
-  field :receivers, type: Array, :default => []
+  field :receivers, type: Array, :default => ["subscriber"]
   field :other_emails
 
 
@@ -19,4 +19,12 @@ class PaperCriteria
     EPaperTopic.where(:period.gte => self.start_date, :period.lte => self.end_date).can_display
   end
 
+  def other_emails_ids
+    if !self.other_emails.nil?
+      return self.other_emails.split(",")
+    else
+      return []
+    end
+  end
+  
 end

app/views/admin/e_paper_criterias/_form.html.erb

@@ -27,13 +27,13 @@
     <div class="tab-pane fade in active" id="basic">
 
       <!-- Date Time Picker -->
-      <div class="control-group">
+   <!--   <div class="control-group">
         <label class="control-label muted"><%= t(:start_date) %></label>
         <div class="controls">
           <%= f.datetime_picker :start_date, :no_label => true, :new_record => @paper_criteria.new_record?, :data=>{"picker-type" => "range", "range" => "start"} %>
         </div>
       </div>
-
+-->
       <div class="control-group">
         <label class="control-label muted"><%= t(:end_date) %></label>
         <div class="controls">
@@ -77,7 +77,7 @@
           </div>
         <% end %>
       </div>
-       <div class="control-group">
+      <!-- <div class="control-group">
         <label class="control-label muted"><%= t("e_paper.receivers") %></label>
         <div class="controls">
             <label>
@@ -89,7 +89,7 @@
             <input type="checkbox" <%= @paper_criteria.receivers.include?("website_member") ? "checked=checked" : "" %>  name="paper_criteria[receivers][]" value="member"> <%= t("e_paper.website_member") %>    
           <label>        
         </div>
-      </div>
+      </div> -->
       <div class="control-group">
         <label class="control-label muted"><%= t("e_paper.other_emails") %></label>
         <div class="controls">

app/views/admin/e_paper_criterias/_index.html.erb

@@ -15,6 +15,7 @@
   			    <ul class="nav nav-pills">
   			      <li><a href="<%= edit_admin_e_paper_criteria_path(criteria.id) %>"><%= t(:edit) %></a></li>
   			      <li><a class="test-email" href="<%= send_test_email_admin_e_paper_criteria_path(criteria.id) %>"><%= t('e_paper.test_mail') %></a></li>
+              <li><a class="send-newsletter" href="<%= send_newsletter_admin_e_paper_criteria_path(criteria.id) %>"><%= t('e_paper.send_newsletter') %></a></li>
   			      <li><a class="delete text-error" href="<%= admin_e_paper_criteria_path(criteria.id) %>" data-method="delete" data-confirm="Are you sure?"><%= t(:delete_) %></a></li>
             </ul>
           </div>
@@ -48,4 +49,17 @@
     }
     return false;
   })
+   $(".send-newsletter").on("click",function(){
+    var el = $(this);
+    if(confirm("Are you sure?")){
+      $.ajax({
+        url : el.attr("href"),
+        dataType : "json",
+        type : "get"
+      }).done(function(){
+        alert("Newsletter sent.")
+      })
+    }
+    return false;
+  })
 </script>

app/views/admin/e_paper_criterias/emailer.html.erb

@@ -150,19 +150,18 @@
 	<div class="epaper-show">
 		<h3 class="epaper-show-title"><%= t('e_paper.intro') %></h3>
 		<div class="ep-info">
-			<div class="ep-info-title"><%= t('e_paper.title') %>：<span><%= paper.title %></span></div>
+			<div class="ep-info-title"><%= t('e_paper.title') %>：<a class="ep-content-title" href="<%= page + "/" + paper.to_param + "?method=topics" %>" target="_blank"><span><%= paper.title %></span></a></div>
 			<div class="ep-info-date"><%= t('e_paper.date') %>：<span><%= paper.created_at.strftime("%Y-%m-%d") %></span></div>
-			<div class="ep-info-period"><%= t('e_paper.period') %>：<span><%= paper.period.strftime("%Y-%m-%d") %></span></div>
 			<div class="ep-info-description"><%= t('e_paper.description') %><span><%= paper.description %></span></div>
 		</div>
 
 		<div class="ep-content">
 			<% paper.epaper_topics.desc(:created_at).each do |topic| %>
 			<div class="ep-content-item">
-				<a class="ep-content-title" href="<%= page + "/" + paper.to_param + "?method=showtopic" %>" target="_blank"><%= topic.episode %></a>
+				<a class="ep-content-title" href="<%= page + "/" + topic.to_param %>" target="_blank"><%= topic.title %></a>
 				<% t = strip_tags(topic.content) %>
 				<p class="ep-content-text"><%= t[0..100] %></p>
-				<a class="ep-content-more" href="<%= page + "/" + paper.to_param + "?method=showtopic" %>" target="_blank">more</a>
+				<a class="ep-content-more" href="<%= page + "/" + topic.to_param %>" target="_blank">more</a>
 			</div>
 			<% end %>
 		</div>

config/locales/en.yml

@@ -34,3 +34,4 @@ en:
     website_member: Website Members
     other_email: Other Email
     receivers: Receivers
+    send_newsletter: Send Newsletter

config/locales/zh_tw.yml

@@ -35,3 +35,4 @@ zh_tw:
     website_member: 網站會員
     other_email: 其他Email
     receivers: 收件人
+    send_newsletter: 發送時事通訊

config/routes.rb

@@ -7,6 +7,7 @@ Rails.application.routes.draw do
       resources :e_paper_criterias do
         member do
 	    		get "send_test_email"
+          get "send_newsletter"
         end
       end
       resources :e_paper_subscribers

lib/tasks/e_paper_tasks.rake

@@ -1,4 +1,44 @@
 # desc "Explaining what the task does"
-# task :e_paper do
+namespace :epaper do 
-#   # Task goes here
+	task :send_email, [:paper_criteria_id, :domain] => :environment do |task,args|
-# end
+    paper = PaperCriteria.find(args[:paper_criteria_id]) rescue nil
+		page = Page.where(:module => "e_paper").first.url rescue ""  
+		domain = args[:domain]
+		if !paper.nil?
+			subscribers = EPaperSubscriber.all
+			emails = []
+			mail_limit = 25
+			subscribers.each do |subscriber|
+				emails << subscriber.email if paper.language_to_send.include? subscriber.language
+				subscriber.last_paper_sent = paper.id.to_s
+				subscriber.save
+				if emails.count == mail_limit
+					puts "Sending #{mail_limit} mails"
+          email = Email.new(:mail_subject => paper.sending_title, :mail_lang => :zh_tw, :template => "admin/e_paper_criterias/emailer", :template_data => {"paper" => paper, "page" => "http://" + domain + page}, :mail_to => emails)     
+          email.deliver
+	  			emails = []
+	  			sleep 0.5
+				end
+			end
+			other_emails = paper.other_emails_ids
+			if !other_emails.blank?
+				other_emails.each do |email|
+					emails << email
+					if emails.count == mail_limit
+						puts "Sending #{mail_limit} mails"
+						email = Email.new(:mail_subject => paper.sending_title, :mail_lang => :zh_tw, :template => "admin/e_paper_criterias/emailer", :template_data => {"paper" => paper, "page" => "http://" + domain + page}, :mail_to => emails)     
+						email.deliver
+						emails = []
+						sleep 0.5
+					end
+				end
+			end
+			if !emails.empty?
+				puts "Sending last emails"
+			  email = Email.new(:mail_subject => paper.sending_title, :mail_lang => :zh_tw, :template => "admin/e_paper_criterias/emailer", :template_data => {"paper" => paper, "page" => "http://" + domain + page}, :mail_to => emails)   
+		  	email.deliver
+		  end
+			paper.save
+		end
+	end
+end