Finish user edit, update, index, and destroy actions

Author: LeSa22

Gemfile

@@ -6,6 +6,9 @@ gem 'rails', '4.2.6'
 gem 'bcrypt',               '3.1.7'
 gem 'bootstrap-sass',       '3.2.0.0'
 gem 'puma',           '3.1.0'
+gem 'faker',                '1.4.2'
+gem 'will_paginate',           '3.0.7'
+gem 'bootstrap-will_paginate', '0.0.10'
 # Use sqlite3 as the database for Active Record
 gem 'sqlite3'
 # Use SCSS for stylesheets

Gemfile.lock

@@ -42,6 +42,8 @@ GEM
       debug_inspector (>= 0.0.1)
     bootstrap-sass (3.2.0.0)
       sass (~> 3.2)
+    bootstrap-will_paginate (0.0.10)
+      will_paginate
     builder (3.2.2)
     byebug (9.0.5)
     coffee-rails (4.1.1)
@@ -55,6 +57,8 @@ GEM
     debug_inspector (0.0.2)
     erubis (2.7.0)
     execjs (2.7.0)
+    faker (1.4.2)
+      i18n (~> 0.5)
     globalid (0.3.7)
       activesupport (>= 4.1.0)
     i18n (0.7.0)
@@ -145,15 +149,18 @@ GEM
       binding_of_caller (>= 0.7.2)
       railties (>= 4.0)
       sprockets-rails (>= 2.0, < 4.0)
+    will_paginate (3.0.7)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   bcrypt (= 3.1.7)
   bootstrap-sass (= 3.2.0.0)
+  bootstrap-will_paginate (= 0.0.10)
   byebug
   coffee-rails (~> 4.1.0)
+  faker (= 1.4.2)
   jbuilder (~> 2.0)
   jquery-rails
   puma (= 3.1.0)
@@ -165,6 +172,7 @@ DEPENDENCIES
   turbolinks
   uglifier (>= 1.3.0)
   web-console (~> 2.0)
+  will_paginate (= 3.0.7)
 
 BUNDLED WITH
    1.12.5

app/assets/stylesheets/custom.css.scss

@@ -2,6 +2,15 @@
 @import "bootstrap";
 
 $gray-medium-light: #eaeaea;
+.users {
+  list-style: none;
+  margin: 0;
+  li {
+    overflow: auto;
+    padding: 10px 0;
+    border-bottom: 1px solid $gray-lighter;
+  }
+}
 .checkbox {
   margin-top: -10px;
   margin-bottom: 10px;

app/controllers/sessions_controller.rb

@@ -6,7 +6,7 @@ def create
   	if user && user.authenticate(params[:session][:password])
   		log_in user
       params[:session][:remember_me] == '1' ? remember(user) : forget(user)
-  		redirect_to user
+  		redirect_back_or user
   	else
   		flash.now[:danger] = 'Invalid email/password combination' # Not quite right!
   		render 'new'

app/controllers/users_controller.rb

@@ -1,5 +1,18 @@
 class UsersController < ApplicationController
-	before_action :logged_in_user, only: [:edit, :update]
+	before_action :logged_in_user, only: [:index, :edit, :update, :destroy]
+  	before_action :correct_user,   only: [:edit, :update]
+  	before_action :admin_user,     only: :destroy
+  def admin_user
+     redirect_to(root_url) unless current_user.admin?
+   end
+  def index
+    @users = User.paginate(page: params[:page]) 
+  end
+  def destroy
+    User.find(params[:id]).destroy
+    flash[:success] = "User deleted"
+    redirect_to users_url
+  end
   def show
   	@user = User.find(params[:id]) 
   #	debugger
@@ -36,8 +49,13 @@ def user_params
   end
   def logged_in_user
   	unless logged_in?
+  		store_location
         flash[:danger] = "Please log in."
         redirect_to login_url
       end
   end
+   def correct_user
+      @user = User.find(params[:id])
+      redirect_to(root_url) unless current_user?(@user)
+    end
 end

app/helpers/sessions_helper.rb

@@ -12,6 +12,9 @@ def log_out
 		session.delete(:user_id)
 		@current_user = nil
 	end
+  def current_user?(user)
+    user == current_user
+  end
 	def remember(user)
 		user.remember
 		cookies.permanent.signed[:user_id] = user.id
@@ -39,4 +42,13 @@ def log_out
     session.delete(:user_id)
     @current_user = nil
   end
+ def redirect_back_or(default)
+    redirect_to(session[:forwarding_url] || default)
+    session.delete(:forwarding_url)
+  end
+
+  # Stores the URL trying to be accessed.
+  def store_location
+    session[:forwarding_url] = request.url if request.get?
+  end
 end

app/helpers/users_helper.rb

@@ -1,5 +1,5 @@
 module UsersHelper
-	def gravatar_for(user)
+	def gravatar_for(user, options = { size: 50 })
 		gravatar_id = Digest::MD5::hexdigest(user.email.downcase)
 		gravatar_url = "https://secure.gravatar.com/avatar/#{gravatar_id}"
 		image_tag(gravatar_url, alt: user.name, class: "gravatar")

app/views/layouts/_header.html.erb

@@ -6,7 +6,7 @@
         <li><%= link_to "Home", root_path %></li>
         <li><%= link_to "Help", help_path %></li>
         <% if logged_in? %>
-          <li><%= link_to "Users", '#' %></li>
+          <li><%= link_to "Users", users_path %></li>
           <li class="dropdown">
             <a href="#" class="dropdown-toggle" data-toggle="dropdown">
               Account <b class="caret"></b>

app/views/users/_user.html.erb

@@ -0,0 +1,8 @@
+<li>
+  <%= gravatar_for user, size: 50 %>
+  <%= link_to user.name, user %>
+  <% if current_user.admin? && !current_user?(user) %>
+    | <%= link_to "delete", user, method: :delete,
+                                  data: { confirm: "You sure?" } %>
+  <% end %>
+</li>

app/views/users/index.html.erb

@@ -0,0 +1,13 @@
+<% provide(:title, 'All users') %>
+<h1>All users</h1>
+<%= will_paginate %>
+<ul class="users">
+	<%= render @users %>
+  <!-- <% @users.each do |user| %>
+    <li>
+      <%= gravatar_for user, size: 50 %>
+      <%= link_to user.name, user %>
+    </li>
+  <% end %> -->
+</ul>
+<%= will_paginate %>

config/routes.rb

@@ -1,11 +1,5 @@
 Rails.application.routes.draw do
-  resources :users
-  get 'users/new'
-
-  get 'users/new'
-
-  get 'users/new'
-
+  
   get 'users/new'
 
   root 'static_pages#home'
@@ -16,7 +10,7 @@
   get    'login'   => 'sessions#new'
   post   'login'   => 'sessions#create'
   delete 'logout'  => 'sessions#destroy'
- 
+ resources :users
 
   # The priority is based upon order of creation: first created -> highest priority.
   # See how all your routes lay out with "rake routes".

db/migrate/20160921080444_add_admin_to_users.rb

@@ -0,0 +1,5 @@
+class AddAdminToUsers < ActiveRecord::Migration
+  def change
+    add_column :users, :admin, :boolean
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160920063233) do
+ActiveRecord::Schema.define(version: 20160921080444) do
 
   create_table "users", force: :cascade do |t|
     t.string   "name"
@@ -20,6 +20,7 @@
     t.datetime "updated_at",      null: false
     t.string   "password_digest"
     t.string   "remember_digest"
+    t.boolean  "admin"
   end
 
   add_index "users", ["email"], name: "index_users_on_email", unique: true

db/seeds.rb

@@ -5,3 +5,18 @@
 #
 #   cities = City.create([{ name: 'Chicago' }, { name: 'Copenhagen' }])
 #   Mayor.create(name: 'Emanuel', city: cities.first)
+User.create!(name:  "Example User",
+             email: "[email protected]",
+             password:              "foobar",
+             password_confirmation: "foobar",
+             admin: true)
+
+99.times do |n|
+  name  = Faker::Name.name
+  email = "example-#{n+1}@railstutorial.org"
+  password = "password"
+  User.create!(name:  name,
+               email: email,
+               password:              password,
+               password_confirmation: password)
+end

test/fixtures/users.yml

@@ -2,4 +2,5 @@ michael:
   name: Michael Example
   email: [email protected]
   password_digest: <%= User.digest('password') %>
+  admin: true
   #empty