Decrease mpog_software_plugin block complexity.

Signed-off-by: David Carlos <ddavidcarlos1392@gmail.com> Signed-off-by: Fabio Teixeira <fabio1079@gmail.com>

Decrease mpog_software_plugin block complexity.
Signed-off-by: David Carlos <ddavidcarlos1392@gmail.com> Signed-off-by: Fabio Teixeira <fabio1079@gmail.com>
David Silva
1 parent e999d703
Showing 1 changed file with 34 additions and 23 deletions Show diff stats
lib/mpog_software_plugin.rb
@@ -71,17 +71,9 @@ class MpogSoftwarePlugin &lt; Noosfero::Plugin
   end
   def profile_tabs
-    if context.profile.community? && context.profile.software?
-        { :title => _("Software"),
-        :id => 'mpog-fields',
-        :content => Proc::new do render :partial => 'software_tab' end,
-        :start => true }
-    elsif context.profile.community? && context.profile.institution?
-      { :title => _("Institution"),
-        :id => 'mpog-fields',
-        :content => Proc::new do render :partial => 'institution_tab' end,
-        :start => true
-      }
+    if context.profile.community?
+      return profile_tabs_software if context.profile.software?
+      profile_tabs_institution
     end
   end
@@ -396,18 +388,7 @@ class MpogSoftwarePlugin &lt; Noosfero::Plugin
       end
     end
-    leave_communities = (old_communities - new_communities)
-    enter_communities = (new_communities - old_communities)
-
-    leave_communities.each do |community|
-      community.remove_member(user.person)
-      user.institutions.delete(community.institution)
-    end
-
-    enter_communities.each do |community|
-      community.add_member(user.person)
-      user.institutions << community.institution
-    end
+    manage_user_institutions(user, leave_communities, enter_communities)
   end
   def show_sisp_field
@@ -457,4 +438,34 @@ class MpogSoftwarePlugin &lt; Noosfero::Plugin
       }
     }
   end
+
+  def manage_user_institutions(leave_communities,enter_communities)
+    leave_communities = (old_communities - new_communities)
+    enter_communities = (new_communities - old_communities)
+
+    leave_communities.each do |community|
+      community.remove_member(user.person)
+      user.institutions.delete(community.institution)
+    end
+
+    enter_communities.each do |community|
+      community.add_member(user.person)
+      user.institutions << community.institution
+    end
+  end
+
+  def profile_tabs_software
+    { :title => _("Software"),
+      :id => 'mpog-fields',
+      :content => Proc::new do render :partial => 'software_tab' end,
+      :start => true }
+  end
+
+  def profile_tabs_institution
+    { :title => _("Institution"),
+      :id => 'mpog-fields',
+      :content => Proc::new do render :partial => 'institution_tab' end,
+      :start => true
+    }
+  end
 end