Code Challenge - Jared Murphy

Gemfile

@@ -47,6 +47,7 @@ group :development, :test do
   gem 'capybara'
   gem 'pry'
   gem 'rails-controller-testing'
+  gem 'faker'
 end
 
 group :development do

Gemfile.lock

@@ -82,6 +82,8 @@ GEM
     factory_bot_rails (1.0.0.alpha)
       factory_bot (~> 1.0.0.alpha)
       railties (>= 3.0.0)
+    faker (2.2.1)
+      i18n (>= 0.8)
     ffi (1.9.25)
     globalid (0.4.1)
       activesupport (>= 4.2.0)
@@ -233,6 +235,7 @@ DEPENDENCIES
   coffee-rails (~> 4.2)
   devise
   factory_bot_rails
+  faker
   jbuilder (~> 2.5)
   listen (>= 3.0.5, < 3.2)
   pry
@@ -254,4 +257,4 @@ RUBY VERSION
    ruby 2.4.4p296
 
 BUNDLED WITH
-   1.16.2
+   2.1.4

app/controllers/dogs_controller.rb

@@ -1,10 +1,11 @@
 class DogsController < ApplicationController
-  before_action :set_dog, only: [:show, :edit, :update, :destroy]
+  before_action :set_dogs, only: %i[index]
+  before_action :set_dog, only: %i[show edit update destroy]
+  before_action :redirect_unless_can_update, only: %i[edit update destroy]
 
   # GET /dogs
   # GET /dogs.json
   def index
-    @dogs = Dog.all
   end
 
   # GET /dogs/1
@@ -75,4 +76,21 @@ def set_dog
     def dog_params
       params.require(:dog).permit(:name, :description, :images)
     end
+
+    def redirect_unless_can_update
+      unless current_user.owner_of?(@dog)
+        flash[:notice] = "Oops you can't do that"
+        redirect_to dog_path(@dog)
+      end
+    end
+
+    def set_dogs
+      if params[:sort]
+        scope = params[:sort]
+      else
+        scope = :all
+      end
+
+      @dogs = Dog.send(scope)
+    end
 end

app/controllers/likes_controller.rb

@@ -0,0 +1,28 @@
+class LikesController < ApplicationController
+  before_action :set_dog, only: %i[create destroy]
+
+  def create
+    like = @dog.likes.where(user_id: current_user.id).first_or_initialize
+
+    respond_to do |format|
+      if like.save
+        format.js { render :create }
+      else
+        format.json { render json: { errors: like.errors }, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  def destroy
+    @dog.likes.where(user_id: current_user.id).destroy_all
+
+    respond_to do |format|
+      format.js { render :destroy }
+    end
+  end
+
+  private
+    def set_dog
+      @dog = Dog.find(params[:dog_id])
+    end
+end

app/helpers/users_helper.rb

@@ -0,0 +1,18 @@
+module UsersHelper
+  def can_edit?(dog)
+    logged_in? &&
+      current_user.owner_of?(dog)
+  end
+
+  def can_like?(dog)
+    logged_in? &&
+      !current_user.owner_of?(dog)
+  end
+
+  private
+
+  def logged_in?
+    current_user.present?
+  end
+end
+

app/models/dog.rb

@@ -1,3 +1,8 @@
 class Dog < ApplicationRecord
   has_many_attached :images
+  has_many :likes
+  belongs_to :user
+
+  # TODO: Still needs ordering by recent likes (in the last hour)
+  scope :recent_likes, -> { order("likes_count DESC") }
 end

app/models/like.rb

@@ -0,0 +1,4 @@
+class Like < ApplicationRecord
+  belongs_to :dog, counter_cache: true
+  belongs_to :user
+end

app/models/user.rb

@@ -3,4 +3,15 @@ class User < ApplicationRecord
   # :confirmable, :lockable, :timeoutable and :omniauthable
   devise :database_authenticatable, :registerable,
          :recoverable, :rememberable, :trackable, :validatable
+
+  has_many :dogs
+  has_many :likes
+
+  def owner_of?(dog)
+    id == dog.user_id
+  end
+
+  def likes?(dog)
+    dog.likes.where(user_id: id).any?
+  end
 end

app/views/dogs/_likes.html.erb

@@ -0,0 +1,9 @@
+<% if can_like?(dog) %>
+  <% if current_user.likes?(dog) %>
+    <%= link_to "Unlike", dog_like_path(dog), method: :delete, remote: true %>
+  <% else %>
+    <%= link_to "Like", dog_likes_path(dog), method: :post, remote: true %>
+  <% end %>
+<% end %>
+
+<p>Likes: <%= dog.likes.count %></p>

app/views/dogs/_thumbnail.html.erb

@@ -1,6 +1,13 @@
-<a href="<%= url_for(dog) %>">
-  <article>
-    <h2 class="dog-name"><%= dog.name %></h2>
-    <%= image_tag url_for(dog.images.first), class: "dog-photo", alt: "Photo of #{dog.name}" %>
-  </article>
-</a>
+<% if (dog_counter + 1) % 3 == 0 %>
+  <%= render "shared/ad" %>
+<% else %>
+  <a href="<%= url_for(dog) %>">
+    <article>
+      <h2 class="dog-name"><%= dog.name %></h2>
+      <div id="dog_<%= dog.id %>_likes">
+        <%= render "dogs/likes", dog: dog %>
+      </div>
+      <%= image_tag url_for(dog.images.first), class: "dog-photo", alt: "Photo of #{dog.name}" %>
+    </article>
+  </a>
+<% end %>

app/views/dogs/index.html.erb

@@ -1 +1,6 @@
+<%= form_tag dogs_path, method: :get do %>
+  <%= radio_button_tag :sort, "recent_likes", params[:sort] == "recent_likes" ? true : false %>
+  <%= label_tag :sort_likes, "Most likes in the last hour" %>
+  <%= submit_tag "Sort" %>
+<% end %>
 <%= render partial: 'thumbnail', collection: @dogs, as: :dog %>

app/views/dogs/show.html.erb

@@ -7,7 +7,13 @@
 
   <p><%= @dog.description %></p>
 
-  <%= link_to "Edit #{@dog.name}'s Profile", edit_dog_path %>
-  <br>
-  <%= link_to "Delete #{@dog.name}'s Profile", dog_path, method: :delete, data: { confirm: 'Are you sure?' } %>
+  <div id="dog_<%= @dog.id %>_likes">
+    <%= render "dogs/likes", dog: @dog %>
+  </div>
+
+  <% if can_edit?(@dog) %>
+    <%= link_to "Edit #{@dog.name}'s Profile", edit_dog_path %>
+    <br>
+    <%= link_to "Delete #{@dog.name}'s Profile", dog_path, method: :delete, data: { confirm: 'Are you sure?' } %>
+  <% end %>
 </section>

app/views/likes/create.js.erb

@@ -0,0 +1,3 @@
+document
+.getElementById("dog_<%= @dog.id %>_likes")
+.innerHTML = "<%= j render partial: "dogs/likes", locals: { dog: @dog } %>";

app/views/likes/destroy.js.erb

@@ -0,0 +1,3 @@
+document
+.getElementById("dog_<%= @dog.id %>_likes")
+.innerHTML = "<%= j render partial: "dogs/likes", locals: { dog: @dog } %>";

app/views/shared/_ad.html.erb

@@ -0,0 +1 @@
+<%= image_tag("ad.jpg") %>

config/routes.rb

@@ -1,6 +1,9 @@
 Rails.application.routes.draw do
   # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
   devise_for :users
-  resources :dogs
+  resources :dogs do
+    resources :likes, only: %i[create]
+    delete "/:dog_id", to: "likes#destroy", as: :like
+  end
   root to: "dogs#index"
 end

db/migrate/20200825221745_add_user_to_dogs.rb

@@ -0,0 +1,5 @@
+class AddUserToDogs < ActiveRecord::Migration[5.2]
+  def change
+    add_reference :dogs, :user, foreign_key: true
+  end
+end

db/migrate/20200826004158_create_likes.rb

@@ -0,0 +1,8 @@
+class CreateLikes < ActiveRecord::Migration[5.2]
+  def change
+    create_table :likes do |t|
+      t.references :user, foreign_key: true
+      t.references :dog, foreign_key: true
+    end
+  end
+end

db/migrate/20200826022636_add_likes_count_to_dogs.rb

@@ -0,0 +1,5 @@
+class AddLikesCountToDogs < ActiveRecord::Migration[5.2]
+  def change
+    add_column :dogs, :likes_count, :integer
+  end
+end

db/migrate/20200826023426_add_time_stamp_to_likes.rb

@@ -0,0 +1,5 @@
+class AddTimeStampToLikes < ActiveRecord::Migration[5.2]
+  def change
+    add_timestamps :likes, null: false, default: Time.now
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2018_06_07_114248) do
+ActiveRecord::Schema.define(version: 2020_08_26_023426) do
 
   create_table "active_storage_attachments", force: :cascade do |t|
     t.string "name", null: false
@@ -40,6 +40,18 @@
     t.text "description"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.integer "user_id"
+    t.integer "likes_count"
+    t.index ["user_id"], name: "index_dogs_on_user_id"
+  end
+
+  create_table "likes", force: :cascade do |t|
+    t.integer "user_id"
+    t.integer "dog_id"
+    t.datetime "created_at", default: "2020-08-26 02:36:39", null: false
+    t.datetime "updated_at", default: "2020-08-26 02:36:39", null: false
+    t.index ["dog_id"], name: "index_likes_on_dog_id"
+    t.index ["user_id"], name: "index_likes_on_user_id"
   end
 
   create_table "users", force: :cascade do |t|

db/seeds.rb

@@ -6,6 +6,9 @@
 #   movies = Movie.create([{ name: 'Star Wars' }, { name: 'Lord of the Rings' }])
 #   Character.create(name: 'Luke', movie: movies.first)
 
+Dog.destroy_all
+User.destroy_all
+
 dogs = [
   {
     name: 'Bowie',
@@ -49,8 +52,10 @@
   },
 ]
 
+5.times { FactoryBot.create(:user) }
+
 dogs.each do |dog|
-  dog = Dog.find_or_create_by(name: dog[:name], description: dog[:description])
+  dog = Dog.find_or_create_by(name: dog[:name], description: dog[:description], user: User.all.sample)
   directory_name = File.join(Rails.root, 'db', 'seed', "#{dog[:name].downcase}", "*")
 
   Dir.glob(directory_name).each do |filename|

spec/factories/dogs.rb

@@ -9,5 +9,7 @@
         filename: 'bowie_toys.jpeg',
         content_type: 'image/jpeg')
     end
+
+    user { create :user }
   end
 end

spec/factories/likes.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :like do
+    user { create :user }
+    dog { create :dog }
+  end
+end

spec/factories/users.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :user do
+    email { Faker::Internet.email }
+    password { Faker::Internet.password }
+  end
+end

spec/models/users_spec.rb

@@ -0,0 +1,33 @@
+require "rails_helper"
+
+describe User do
+  describe "#owner_of?" do
+    subject { user.owner_of? dog  }
+
+    let(:user) { create :user }
+
+    context "when the user owns the dog" do
+      let(:dog) { create :dog, user: user }
+      it { is_expected.to be true }
+    end
+
+    context "when the dog belongs to another user" do
+      let(:dog) { create :dog, user: create(:user) }
+      it { is_expected.to be false }
+    end
+  end
+
+  describe "#likes?" do
+    subject { user.likes? dog }
+
+    let(:dog) { create(:dog) }
+    let(:user) { dog.user }
+
+    it { is_expected.to be false }
+
+    context "when a user has liked a dog" do
+      before { create :like, user: user, dog: dog }
+      it { is_expected.to be true }
+    end
+  end
+end

spec/rails_helper.rb

@@ -56,4 +56,5 @@
   config.filter_rails_from_backtrace!
   # arbitrary gems may also be filtered via:
   # config.filter_gems_from_backtrace("gem name")
+  config.include Devise::Test::ControllerHelpers, type: :controller
 end