Add logic to display courses on home page

app/controllers/application_controller.rb

@@ -2,7 +2,7 @@ class ApplicationController < ActionController::Base
   before_action :authenticate_user!
   protect_from_forgery
 
-  after_action :user_activity
+  after_action :user_activity, if: :user_signed_in?
   allow_browser versions: :modern
 
   include Pundit::Authorization

app/controllers/courses_controller.rb

@@ -1,4 +1,5 @@
 class CoursesController < ApplicationController
+  skip_before_action :authenticate_user!, only: [ :show ]
   before_action :set_course, only: %i[ show edit update destroy approve unapprove ]
 
   # GET /courses or /courses.json

app/controllers/home_controller.rb

@@ -3,9 +3,9 @@ class HomeController < ApplicationController
   def index
     @courses = Course.all.limit(4)
     @latest_good_reviews = Enrollment.reviewed.latest_good_reviews
-    @latest = Course.latest
-    @top_rated = Course.top_rated
-    @popular = Course.popular
+    @latest = Course.latest.published.approved
+    @top_rated = Course.top_rated.published.approved
+    @popular = Course.popular.published.approved
     @purchased = Course.joins(:enrollments).where(enrollments: { user: current_user }).order(created_at: :desc).limit(3)
   end
 

app/helpers/courses_helper.rb

@@ -15,7 +15,7 @@ def enrollment_button(course)
         link_to "Free", new_course_enrollment_path(course), class: "btn btn-md btn-success"
       end
     else
-      link_to "Check price", course_path(course), class: "btn btn-md btn-success"
+      link_to "Check price", new_course_enrollment_path(course), class: "btn btn-md btn-success"
     end
   end
 

app/policies/course_policy.rb

@@ -13,8 +13,24 @@
   end
 
   def show?
-    @record.published && @record.approved ||@user.present? && @user.has_role?(:admin) || @user.present? && @record.user_id == @user.id || @record.bought(@user)
-  end
+    @record.published && @record.approved || @user.present? && @user.has_role?(:admin) || @user.present? && @record.user_id == @user.id || @user.present? && @record.bought(@user)
+  end
+
+  # another logic for show
+  # def show?
+  #   # Check if the record is published and approved
+  #   return true if @record.published && @record.approved
+
+  #   # Check user conditions if the user is present
+  #   return true if @user.present? && (
+  #     @user.has_role?(:admin) ||
+  #     @record.user_id == @user.id ||
+  #     @record.bought(@user)
+  #   )
+
+  #   # If none of the conditions are met, return false
+  #   false
+  # end
 
   def edit?
     @record.user == @user

app/policies/lesson_policy.rb

@@ -17,7 +17,7 @@ def show?
   end
 
   def edit?
-    @record.course.user_id == @user.id
+    @user.present? && @record.course.user_id == @user.id
   end
 
   def update?

app/views/home/index.html.haml

@@ -55,14 +55,16 @@
     .row
     Learn on your schedule
 
-%hr
-%h2
-  .fa.fa-user-graduate
-  Continue Learning
-.row.row-cols-lg-4
-  - @purchased.each do |course|
-    .col
-      = render 'courses/course', course: course
+
+- if current_user
+  %hr
+    %h2
+      .fa.fa-user-graduate
+      Continue Learning
+    .row.row-cols-lg-4
+      - @purchased.each do |course|
+        .col
+          = render 'courses/course', course: course
 %hr
 %h2
   .fa.fa-chart-line