From 8a1316cf8f3a4664400d8cc57b2e7e212a145d5c Mon Sep 17 00:00:00 2001
From: yanxd <sxty32@hotmaill.com>
Date: Thu, 26 Dec 2013 19:28:18 +0800
Subject: [PATCH] =?UTF-8?q?=E5=88=A0=E9=99=A4=E7=95=99=E8=A8=80=E6=93=8D?=
 =?UTF-8?q?=E4=BD=9C?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/controllers/words_controller.rb            |  6 +++---
 app/models/journals_for_message.rb             |  4 ++--
 app/views/words/_journal_reply.html.erb        |  2 +-
 app/views/words/_journal_reply_items.html.erb  | 18 +++++++++---------
 app/views/words/_message.html.erb              | 16 ++++++++--------
 app/views/words/create_reply.js.erb            |  7 ++++---
 app/views/words/destroy.js.erb                 | 13 +++++++++++--
 config/locales/en.yml                          |  1 +
 config/locales/zh.yml                          |  1 +
 public/javascripts/application.js              |  2 +-
 .../stylesheets/application.css                |  2 +-
 11 files changed, 42 insertions(+), 30 deletions(-)

diff --git a/app/controllers/words_controller.rb b/app/controllers/words_controller.rb
index 7c05c5af5..193b2b61d 100644
--- a/app/controllers/words_controller.rb
+++ b/app/controllers/words_controller.rb
@@ -73,10 +73,10 @@ class WordsController < ApplicationController
   end
   
   def destroy
-    if User.current.admin? || User.current.id == @user.id
-      JournalsForMessage.delete_message(params[:object_id])
+    if User.current.admin? || @user.id == User.current.id || User.current.id == JournalsForMessage.find(params[:object_id]).jour_id
+      @journal_destroyed = JournalsForMessage.delete_message(params[:object_id])
     end
-    @jours = @user.journals_for_messages.reverse
+    @jours = @user.journals_for_messages.where('m_parent_id IS NULL').reverse
     @limit =  10
     @feedback_count = @jours.count
     @feedback_pages = Paginator.new @feedback_count, @limit, params['page']
diff --git a/app/models/journals_for_message.rb b/app/models/journals_for_message.rb
index e00564f18..e83018be6 100644
--- a/app/models/journals_for_message.rb
+++ b/app/models/journals_for_message.rb
@@ -20,7 +20,6 @@ class JournalsForMessage < ActiveRecord::Base
   belongs_to :at_user, :class_name => "User", :foreign_key => 'reply_id' 
 
   has_many :acts, :class_name => 'Activity', :as => :act, :dependent => :destroy
-  has_many :reply_for_journals, :dependent => :destroy
 
   validates :notes, presence: true
   after_create :act_as_activity #huang
@@ -30,7 +29,8 @@ class JournalsForMessage < ActiveRecord::Base
   # default_scope { where('m_parent_id IS NULL') }
   
   def self.delete_message(message_id)
-    self.delete_all "id = #{message_id}"
+    self.find(message_id).destroy
+    # self.destroy_all "id = #{message_id}"
   end
   
   def reference_user
diff --git a/app/views/words/_journal_reply.html.erb b/app/views/words/_journal_reply.html.erb
index a909362d9..1249c9cb4 100644
--- a/app/views/words/_journal_reply.html.erb
+++ b/app/views/words/_journal_reply.html.erb
@@ -1,6 +1,6 @@
 
 <% id = "journal_reply_ul_" + journal.id.to_s%>
-<ul class="messages-for-user-reply" id = <%= id %> >
+<ul class="messages-for-user-reply" id = '<%= id %>' >
            <% fetch_user_leaveWord_reply(journal).each do |reply|%>
              <%= render :partial => "words/journal_reply_items", :locals => {:reply => reply, :journal => journal, :m_reply_id => reply.id} %>
 			<% end %>
diff --git a/app/views/words/_journal_reply_items.html.erb b/app/views/words/_journal_reply_items.html.erb
index 4a3a72aba..5a66b5133 100644
--- a/app/views/words/_journal_reply_items.html.erb
+++ b/app/views/words/_journal_reply_items.html.erb
@@ -1,23 +1,23 @@
- <% ids_r = 'reply_respond_form_'+ reply.id.to_s %>
-<li onmouseover="$('#<%= ids_r %>').show()" onmouseout="$('#<%= ids_r %>').hide()">
+<% ids_r = 'reply_respond_form_'+ reply.id.to_s %>
+<li id='word_li_<%=reply.id.to_s%>' onmouseover="$('#<%= ids_r %>').show()" onmouseout="$('#<%= ids_r %>').hide()">
 	<span class="portrait">
 		<%= image_tag url_to_avatar(reply.user), :class => "avatar-3" %>
 	</span>
 	<div class="body">
-		    <% id = 'project_respond_form_'+ reply.id.to_s %>
+		  <% id = 'project_respond_form_'+ reply.id.to_s %>
 	    <span><%= link_to reply.user.name, user_path(reply.user) %>:&nbsp;</span>
 	    <span class="message-notes"> <%= reply.notes %></span>
-	        <% ids = 'project_respond_form_'+ journal.id.to_s%>
+	    <%# ids = 'project_respond_form_'+ journal.id.to_s%>
 	       
 	    <p>
 	    	<span class="time"><%= format_time reply.created_on %></span>
-	    	<span style="display: none; margin-left: 4px;" id=<%= ids_r %>>
+	    	<span style="display: none; margin-left: 4px;" id='<%=ids_r%>' >
 					 <%= link_to l(:label_projects_feedback_respond),'',
-	                    {:focus => 'project_respond', :onclick => "toggleAndSettingWordsVal($('##{ids}'), $('##{ids} textarea'), '#{l(:label_reply_plural)} #{journal.user.show_name}:'); return false;"}
+	                    {:focus => 'project_respond', :onclick => "toggleAndSettingWordsVal($('##{id}'), $('##{id} textarea'), '#{l(:label_reply_plural)} #{journal.user.show_name}: '); return false;"}
 	                 %>
 	                 
 	                 <% if @user == User.current %>
-						 <%= link_to(l(:label_newfeedback_delete), {:controller => 'words', :action => 'destroy', :object_id => journal, :user_id => reply.user},
+						 <%= link_to(l(:label_newfeedback_delete), {:controller => 'words', :action => 'destroy', :object_id => reply, :user_id => reply.user},
 						 :remote => true, :confirm => l(:text_are_you_sure), :method => 'delete', :class => "delete", :title => l(:button_delete)) if (User.current.id == reply.user_id) || journal.jour_id == User.current.id %></td>
 					 <% end %>
 	    	</span>
@@ -25,6 +25,6 @@
 	</div>
 	<div id='<%=id%>' class="respond-form">
             <%= render :partial => "words/new_respond", :locals => {:journal => journal, :m_reply_id => m_reply_id} %>
-    </div>
-    <div style="clear: both;"></div>
+  </div>
+  <div style="clear: both;"></div>
 </li>
\ No newline at end of file
diff --git a/app/views/words/_message.html.erb b/app/views/words/_message.html.erb
index f0d5a651e..f5df90ec9 100644
--- a/app/views/words/_message.html.erb
+++ b/app/views/words/_message.html.erb
@@ -3,7 +3,7 @@
 <% remove_allowed = (User.current.id == jour.first.user_id) %>
 <ul class="message-for-user">
 <% for journal in jour%>
-    <li class="outer-message-for-user">
+    <li id='word_li_<%=journal.id.to_s%>' class="outer-message-for-user">
 	    <span class="portrait"><%= image_tag(url_to_avatar(journal.user), :class => "avatar") %></span>
 	    <span class="body">
             <span class="user"><%= link_to journal.user, user_path(journal.user)%></span>
@@ -19,7 +19,7 @@
 			<% ids = 'project_respond_form_'+ journal.id.to_s%>
 		<span>
 			 <%= link_to l(:label_projects_feedback_respond),'',
-                {:focus => 'project_respond', :onclick => "toggleAndSettingWordsVal($('##{ids}'), $('##{ids} textarea'), '#{l(:label_reply_plural)} #{journal.user.show_name}:'); return false;"}
+                {:focus => 'project_respond', :onclick => "toggleAndSettingWordsVal($('##{ids}'), $('##{ids} textarea'), '#{l(:label_reply_plural)} #{journal.user.show_name}: '); $('##{ids} textarea') ;return false;"}
              %>
              
             <% if @user == User.current %>
@@ -29,13 +29,13 @@
 		</span>
 		</span>
 		<div style="clear: both;"></div>
-		<div id=<%= ids %> class="respond-form">
+		<div id='<%= ids %>' class="respond-form">
             <%= render :partial => 'words/new_respond', :locals => {:journal => journal, :m_reply_id => journal.id} %>
-        </div>
-        <div>
-	        <%= render :partial => "words/journal_reply", :locals => {:journal => journal } %>
-        </div>
-    </li>
+    </div>
+    <div>
+      <%= render :partial => "words/journal_reply", :locals => {:journal => journal } %>
+    </div>
+</li>
 <% end %>
 </ul>
 <% end %>
diff --git a/app/views/words/create_reply.js.erb b/app/views/words/create_reply.js.erb
index 20964d9b0..6d625bdc5 100644
--- a/app/views/words/create_reply.js.erb
+++ b/app/views/words/create_reply.js.erb
@@ -1,9 +1,10 @@
 <% if @save_succ %>
-	$('#journal_reply_ul_<%=@jfm.m_parent_id%>').append(
-		'<%= j( 
+	var pre_append = $('<%= j( 
 			render :partial => "words/journal_reply_items",
 			 :locals => {:reply => @jfm, :journal => @jfm.parent, :m_reply_id => @jfm.id} 
-		) %>');
+		) %>').hide();
+	$('#journal_reply_ul_<%=@jfm.m_parent_id%>').append(pre_append);
+	pre_append.fadeIn(600);
 	$('#project_respond_form_<%=@jfm.m_reply_id.to_s%> textarea').val('');
 	$('#project_respond_form_<%=@jfm.m_reply_id.to_s%>').hide();
 <% else %>
diff --git a/app/views/words/destroy.js.erb b/app/views/words/destroy.js.erb
index 53dc614b2..6b582f804 100644
--- a/app/views/words/destroy.js.erb
+++ b/app/views/words/destroy.js.erb
@@ -1,2 +1,11 @@
-$('#message').html('<%= escape_javascript(render(:partial => 'words/message', :locals => {:jour => @jour, :state => false, :user => @user, :feedback_pages => @feedback_pages})) %>');
-$('#new_form_reference_user_id').val("");
\ No newline at end of file
+<% if @journal_destroyed.nil? %>
+	alert('<%=l(:notice_failed_delete)%>');
+<% elsif (@journal_destroyed.jour_type == 'Principal')%>
+	var destroyedItem = $('#word_li_<%=@journal_destroyed.id%>')
+	destroyedItem.fadeOut(600,function(){
+		destroyedItem.remove();
+	});
+<% else %>
+	$('#message').html('<%= escape_javascript(render(:partial => 'words/message', :locals => {:jour => @jour, :state => false, :user => @user, :feedback_pages => @feedback_pages})) %>');
+	$('#new_form_reference_user_id').val("");
+<% end %>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index fd44c577a..de55fe394 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -155,6 +155,7 @@ en:
   notice_successful_create: Successful creation.
   notice_successful_update: Successful update.
   notice_successful_delete: Successful deletion.
+  notice_failed_delete: Successful failure.
   notice_successful_connection: Successful connection.
   notice_file_not_found: The page you were trying to access doesn't exist or has been removed.
   notice_locking_conflict: Data has been updated by another user.
diff --git a/config/locales/zh.yml b/config/locales/zh.yml
index d73bb42f2..6d64d30bc 100644
--- a/config/locales/zh.yml
+++ b/config/locales/zh.yml
@@ -164,6 +164,7 @@ zh:
   notice_successful_create: 创建成功
   notice_successful_update: 更新成功
   notice_successful_delete: 删除成功
+  notice_failed_delete: 删除失败
   notice_successful_connection: 连接成功
   notice_file_not_found: 您访问的页面不存在或已被删除。
   notice_locking_conflict: 数据已被另一位用户更新
diff --git a/public/javascripts/application.js b/public/javascripts/application.js
index 65af8714e..3ab51764f 100644
--- a/public/javascripts/application.js
+++ b/public/javascripts/application.js
@@ -592,7 +592,7 @@ function blockEventPropagation(event) {
 
 function toggleAndSettingWordsVal(parent_widget, text_widget, value){
   text_widget.val(value)
-  parent_widget.toggle(400)
+  parent_widget.slideToggle(400)
 }
 
 $(document).ready(setupAjaxIndicator);
diff --git a/public/themes/redpenny-master/stylesheets/application.css b/public/themes/redpenny-master/stylesheets/application.css
index 1ed6c9207..0dcca361b 100644
--- a/public/themes/redpenny-master/stylesheets/application.css
+++ b/public/themes/redpenny-master/stylesheets/application.css
@@ -2123,7 +2123,7 @@ ul.messages-for-user-reply li {
 
 .respond-form{
 	display: none; 
-	width: 100%; 
+	width: 87%; 
 	margin: auto;
 	clear:both;
 }