fix curators and admins management

Signed-off-by: Alexandre Arias <apa13@inf.ufpr.br>

app/controllers/management/users_controller.rb

@@ -71,6 +71,23 @@ class Management::UsersController < ManagementController
     end
   end
 
+
+  def curators
+    @curators = Array.new
+    @users = User.includes(:roles).all
+    @users.all.each do |user|
+     @curators << user if user.roles[0].name == "curator"
+    end
+  end
+
+  def admins
+    @admins = Array.new
+    @users = User.includes(:roles).all
+    @users.all.each do |user|
+      @admins << user if user.roles[0].name == "admin"
+    end
+  end
+
   private
 
   # Use callbacks to share common setup or constraints between actions.

app/views/management/users/admins.html.erb

@@ -4,12 +4,8 @@
   novo</a>
 
 
-<% admins = Array.new %>
-<% @users = User.includes(:roles).all%>
-<% @users.all.each do |user|%>
-  <% admins << user if user.roles[0].name == "admin" %>
-<% end %>
-<% if admins[0].nil? %>
+
+<% if @admins.empty? %>
   <BR><BR><h3>Não há adminstradores cadastrados</h3>
 <% else %>
 
@@ -51,7 +47,7 @@
   </thead>
   <tbody>
 
-    <% admins.each do |c| %>
+    <% @admins.each do |c| %>
         <tr>
           <td>
             <a href="#"><%= c.name %></a>

app/views/management/users/curators.html.erb

@@ -4,12 +4,8 @@
   novo</a>
 
 
-<% curators = Array.new %>
-<% @users = User.includes(:roles).all%>
-<% @users.all.each do |user|%>
-  <% curators << user if user.roles[0].name == "curator" %>
-<% end %>
-<% if curators[0].nil? %>
+
+<% if @curators.empty? %>
   <BR><BR><h3>Não há curadores cadastrados</h3>
 <% else %>
 
@@ -51,7 +47,7 @@
   </thead>
   <tbody>
 
-    <% curators.each do |c| %>
+    <% @curators.each do |c| %>
         <tr>
           <td>
             <a href="#"><%= c.name %></a>