Refactoring work assignment plugin

Changed privacy edition box to open on a new page. Fixed cancel buttom to redirect to back page. Signed-off-by: Tallys Martins <tallysmartins@gmail.com> Signed-off-by: Álvaro Fernando <alvarofernandoms@gmail.com> Signed-off-by: Eduardo Vital <vitaldu@gmail.com>

Refactoring work assignment plugin
Changed privacy edition box to open on a new page. Fixed cancel buttom to redirect to back page. Signed-off-by: Tallys Martins <tallysmartins@gmail.com> Signed-off-by: Álvaro Fernando <alvarofernandoms@gmail.com> Signed-off-by: Eduardo Vital <vitaldu@gmail.com>
Tallys Martins
1 parent dfed6005
Showing 4 changed files with 23 additions and 21 deletions Show diff stats
app/controllers/my_profile/cms_controller.rb
plugins/work_assignment/lib/ext/cms_controller.rb
plugins/work_assignment/lib/work_assignment_plugin/helper.rb
plugins/work_assignment/views/cms/edit_visibility.html.erb
@@ -34,8 +34,7 @@ class CmsController &lt; MyProfileController
     false 
   end
-  action_list = [:suggest_an_article, :set_home_page, :edit, :destroy, :publish, :upload_files, :new]
-  protect_if :except => action_list do |c, user, profile|
+  protect_if :except => [:suggest_an_article, :set_home_page, :edit, :destroy, :publish, :upload_files, :new] do |c, user, profile|
     add_as_exception?(c.action_name) || user && (user.has_permission?('post_content', profile) || user.has_permission?('publish_content', profile))
   end
@@ -8,19 +8,21 @@ protect_if :only =&gt; :edit_visibility do |c,user,profile|
 end
 def edit_visibility
-  @folder = profile.articles.find(params[:article_id])
-  @back_to = url_for(@folder.parent.url)
-  if request.post?
-    @folder.published = params[:article][:published]
-    unless params[:q].nil?
-      @folder.article_privacy_exceptions = params[:q].split(/,/).map{|n| environment.people.find n.to_i}
-      @folder.children.each do |c|
-        c.article_privacy_exceptions = params[:q].split(/,/).map{|n| environment.people.find n.to_i}
-        c.save!
+  unless params[:article_id].blank?
+    @folder = profile.articles.find(params[:article_id])
+    @back_to = url_for(@folder.parent.url)
+    unless params[:article].blank?
+      @folder.published = params[:article][:published]
+      unless params[:q].nil?
+        @folder.article_privacy_exceptions = params[:q].split(/,/).map{|n| environment.people.find n.to_i}
+        @folder.children.each do |c|
+          c.article_privacy_exceptions = params[:q].split(/,/).map{|n| environment.people.find n.to_i}
+          c.save!
+        end
       end
+      @folder.save!
+      redirect_to @back_to
     end
-    @folder.save!
-    redirect_to @back_to
   end
  end
@@ -68,7 +68,7 @@ module WorkAssignmentPlugin::Helper
   def display_delete_button(article)
     expirable_button article, :delete, _('Delete'), 
-    {:controller =>'cms', :action => 'destroy', :id => article.id }, 
+    {:controller =>'cms', :action => 'destroy', :id => article.id },
     :method => :post, :confirm => delete_article_message(article)
   end
@@ -76,15 +76,17 @@ module WorkAssignmentPlugin::Helper
     if author_folder
       @folder = environment.articles.find_by_id(author_folder.id)
       work_assignment = @folder.parent
+      @back_to = url_for(@folder.parent.url)
+      if(user && work_assignment.allow_privacy_edition &&
+        (author_folder.author_id == user.id || user.has_permission?('view_private_content', work_assignment.profile)))
-      if(user && work_assignment.allow_privacy_edition && (author_folder.author_id == user.id || user.has_permission?('view_private_content', work_assignment.profile))) 
         @tokenized_children = prepare_to_token_input(
                               profile.members.includes(:articles_with_access).find_all{ |m|
                                 m.articles_with_access.include?(@folder)
-                              }
-                            )
-        colorbox_button :edit, _('Edit'), { :controller => 'cms', 
-        :action => 'edit_visibility', :article_id => @folder.id, :tokenized_children => @tokenized_children}
+                              })
+        button :edit, _('Edit'), { :controller => 'cms',
+        :action => 'edit_visibility', :article_id => @folder.id,
+        :tokenized_children => @tokenized_children, :back_to => @back_to}, :method => :post
       end
     end
   end
-<!-- <div class='<%= (environment.enabled?('media_panel') ? 'with_media_panel' : 'no_media_panel') %>'> -->
 <div class="select-privacy-options">
 <%= labelled_form_for 'article', :html => { :multipart => true, :class => @type } do |f| %>
@@ -14,7 +13,7 @@
   <% button_bar do %>
     <%= submit_button :save, _('Save') %>
-    <%= colorbox_close_button(_('Cancel')) %>
+    <%= button :cancel, _('Cancel'), @back_to %>
   <% end %>
 <% end %>
 </div>
	@@ -34,8 +34,7 @@ class CmsController < MyProfileController		@@ -34,8 +34,7 @@ class CmsController < MyProfileController
34	false	34	false
35	end	35	end
36		36
37	- action_list = [:suggest_an_article, :set_home_page, :edit, :destroy, :publish, :upload_files, :new]
38	- protect_if :except => action_list do \|c, user, profile\|	37	+ protect_if :except => [:suggest_an_article, :set_home_page, :edit, :destroy, :publish, :upload_files, :new] do \|c, user, profile\|
39	add_as_exception?(c.action_name) \|\| user && (user.has_permission?('post_content', profile) \|\| user.has_permission?('publish_content', profile))	38	add_as_exception?(c.action_name) \|\| user && (user.has_permission?('post_content', profile) \|\| user.has_permission?('publish_content', profile))
40	end	39	end
41		40