better user listing

app/controllers/users_controller.rb

@@ -8,7 +8,13 @@ class UsersController < ApplicationController
       if params[:role].downcase == "staff"
         @users = User.all.select {|u| u.role >= Role.get(:mod) }
       else
-        @users = User.where(role: Role.get(params[:role]))
+        if role = Role.get(params[:role])
+          @users = User.where(role: role)
+        else
+          flash[:alert] = "role '#{params[:role]}' does not exist!"
+          redirect_to users_path
+          return
+        end
       end
     else
       @users = User.where.not(id: User.first.id) #Remove first user

app/views/users/index.html.erb

@@ -1,9 +1,11 @@
 <% if params[:role] %>
-  <h1><%= title "All '#{params[:role]}' Users" %></h1>
+  <h1><%= title "All '#{params[:role]}' role users" %></h1>
   <%= link_to "show all", users_path %>
 <% else %>
   <h1><%= title "All Users" %></h1>
 <% end %>
+<p>User count: <%= @users.length %></p>
+
 <div id="userlist">
   <% @users.each do |u| %>
     <div class="list-user">