Added link to show reponses of proposals in content manager

Michel Felipe
1 parent b6e26e93
Showing 8 changed files with 84 additions and 8 deletions Show diff stats
lib/cms_helper.rb
lib/proposals_discussion_plugin/api_new_relic_instrumenter.rb
lib/proposals_discussion_plugin/response.rb
public/images/replyied-article.png
public/images/response-article.png
public/style.css
test/unit/proposal_test.rb
views/cms/proposals_discussion_plugin/_response.html.erb
@@ -10,7 +10,11 @@ module CmsHelper
         image_tag(icon_for_article(article)) + link_to(article_name, article.url)
       else
         if "ProposalsDiscussionPlugin::Proposal".eql? article.type
-          link_to article.abstract, article.url, :class => icon_for_article(article)
+          if article.children_count > 0
+            link_to article.abstract, {:action => 'view', :id => article.id}, :class => 'icon icon-replyied-article'
+          else
+            link_to article.abstract, article.url, :class => icon_for_article(article)
+          end
         else
           link_to article_name, article.url, :class => icon_for_article(article)
         end
@@ -0,0 +1,46 @@
+class ProposalsDiscussionPlugin::ApiNewRelicInstrumenter < Grape::Middleware::Base
+  include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+
+  def call_with_newrelic(&block)
+    trace_options = {
+      :category => :rack,
+      :path => "#{route_path}\##{route_method}",
+      :request => Grape::Request.new(@env)
+    }
+
+    perform_action_with_newrelic_trace(trace_options) do
+      result = yield
+      MetricFrame.abort_transaction! if result.first == 404 # ignore cascaded calls
+      result
+    end
+  end
+
+  def call(env)
+    @env = env
+    if ENV['NEW_RELIC_ID']
+      call_with_newrelic do
+        super
+      end
+    else
+      super
+    end
+  end
+
+  def env
+    @env
+  end
+
+  def route
+    env['api.endpoint'].routes.first
+  end
+
+  def route_method
+    route.route_method.downcase
+  end
+
+  def route_path
+    path = route.route_path.gsub(/^.+:version\/|^\/|:|\(.+\)/, '').tr('/', '-')
+    "api.#{route.route_version}.#{path}"
+  end
+
+end
@@ -12,16 +12,16 @@ class ProposalsDiscussionPlugin::Response &lt; TinyMceArticle
     _("The response of a Proposal")
   end
+  def icon_name
+    'response'
+  end
+
   protected
   def check_parent_type
     unless parent.is_a? ProposalsDiscussionPlugin::Proposal
       errors.add(:parent, N_('of Response needs be a Proposal'))
     end
-
-    # if self.body_changed? && (self.changed & attrs_validators).any?
-    #   errors.add(:response, N_('only have "body" field'))
-    # end
   end
 end
@@ -406,3 +406,11 @@ div.confirm_evaluation_button a.disabled {
 .icon-child-article {
   background-image: url('/plugins/proposals_discussion/images/reply.png');
 }
+
+.icon-response {
+  background: url('/plugins/proposals_discussion/images/response-article.png') no-repeat;
+}
+
+.icon-replyied-article {
+  background: url('/plugins/proposals_discussion/images/replyied-article.png') no-repeat;
+}
@@ -156,4 +156,20 @@ class ProposalTest &lt; ActiveSupport::TestCase
     assert_match "Body can't be blank", err.message
   end
+  should 'not add a response to a article that isnt a proposal' do
+    article = create(Article, :name => 'test article', :profile => @profile)
+    data = {
+      :name => 'Response',
+      :abstract => 'Test',
+      :parent => article,
+      :profile => profile
+    }
+
+    err = assert_raises ActiveRecord::RecordInvalid do
+      response = create(ProposalsDiscussionPlugin::Response, data)
+    end
+
+    assert_match "Parent of Response needs be a Proposal", err.message
+  end
+
 end
@@ -10,14 +10,16 @@
   <div class="title">
     <%= required labelled_form_field _('Title'), limited_text_area(:article, :name, title_limit, 'title_textarea', :rows => 1) %>
   </div>
-
-  <%= select_profile_folder(_('Parent folder:'), 'article[parent_id]', profile, @article.parent_id, {},{},{},:include_articles => true) %>
+  <% if @article.parent_id.nil? %>
+    <%= select_profile_folder(_('Parent folder:'), 'article[parent_id]', profile, @article.parent_id, {},{},{},:include_articles => true) %>
+  <% else %>
+    <%= hidden_field(:article, :parent_id) %>
+  <% end %>
   <div class="body">
     <%= labelled_form_field(_('Response'), text_area(:article, :body, :class => 'mceEditor')) %>
   </div>
-  <%= hidden_field(:article, :parent_id) %>
 </div>
 <script>
@@ -0,0 +1,46 @@		@@ -0,0 +1,46 @@
	1	+class ProposalsDiscussionPlugin::ApiNewRelicInstrumenter < Grape::Middleware::Base
	2	+ include NewRelic::Agent::Instrumentation::ControllerInstrumentation
	3	+
	4	+ def call_with_newrelic(&block)
	5	+ trace_options = {
	6	+ :category => :rack,
	7	+ :path => "#{route_path}\##{route_method}",
	8	+ :request => Grape::Request.new(@env)
	9	+ }
	10	+
	11	+ perform_action_with_newrelic_trace(trace_options) do
	12	+ result = yield
	13	+ MetricFrame.abort_transaction! if result.first == 404 # ignore cascaded calls
	14	+ result
	15	+ end
	16	+ end
	17	+
	18	+ def call(env)
	19	+ @env = env
	20	+ if ENV['NEW_RELIC_ID']
	21	+ call_with_newrelic do
	22	+ super
	23	+ end
	24	+ else
	25	+ super
	26	+ end
	27	+ end
	28	+
	29	+ def env
	30	+ @env
	31	+ end
	32	+
	33	+ def route
	34	+ env['api.endpoint'].routes.first
	35	+ end
	36	+
	37	+ def route_method
	38	+ route.route_method.downcase
	39	+ end
	40	+
	41	+ def route_path
	42	+ path = route.route_path.gsub(/^.+:version\/\|^\/\|:\|\(.+\)/, '').tr('/', '-')
	43	+ "api.#{route.route_version}.#{path}"
	44	+ end
	45	+
	46	+end
	@@ -406,3 +406,11 @@ div.confirm_evaluation_button a.disabled {		@@ -406,3 +406,11 @@ div.confirm_evaluation_button a.disabled {
406	.icon-child-article {	406	.icon-child-article {
407	background-image: url('/plugins/proposals_discussion/images/reply.png');	407	background-image: url('/plugins/proposals_discussion/images/reply.png');
408	}	408	}
		409	+
		410	+.icon-response {
		411	+ background: url('/plugins/proposals_discussion/images/response-article.png') no-repeat;
		412	+}
		413	+
		414	+.icon-replyied-article {
		415	+ background: url('/plugins/proposals_discussion/images/replyied-article.png') no-repeat;
		416	+}
	@@ -156,4 +156,20 @@ class ProposalTest < ActiveSupport::TestCase		@@ -156,4 +156,20 @@ class ProposalTest < ActiveSupport::TestCase
156	assert_match "Body can't be blank", err.message	156	assert_match "Body can't be blank", err.message
157	end	157	end
158		158
		159	+ should 'not add a response to a article that isnt a proposal' do
		160	+ article = create(Article, :name => 'test article', :profile => @profile)
		161	+ data = {
		162	+ :name => 'Response',
		163	+ :abstract => 'Test',
		164	+ :parent => article,
		165	+ :profile => profile
		166	+ }
		167	+
		168	+ err = assert_raises ActiveRecord::RecordInvalid do
		169	+ response = create(ProposalsDiscussionPlugin::Response, data)
		170	+ end
		171	+
		172	+ assert_match "Parent of Response needs be a Proposal", err.message
		173	+ end
		174	+
159	end	175	end