From 283a368edd5d5c13aa8653156dd601505326864e Mon Sep 17 00:00:00 2001
From: HombreLaser <sebastian-440@live.com>
Date: Sat, 1 Apr 2023 13:15:14 -0600
Subject: Corregidos errores de ProductReviewsSerializer

---
 app/controllers/api/reviews_controller.rb | 54 ++++++++++++++++---------------
 1 file changed, 28 insertions(+), 26 deletions(-)

(limited to 'app/controllers')

diff --git a/app/controllers/api/reviews_controller.rb b/app/controllers/api/reviews_controller.rb
index 8a5de64..353e382 100644
--- a/app/controllers/api/reviews_controller.rb
+++ b/app/controllers/api/reviews_controller.rb
@@ -1,41 +1,43 @@
 # frozen_string_literal: true
 
-# ReviewController
-class ReviewsController < AuthenticatedController
-  skip_before_action only: [:index]
+module Api
+  # ReviewsController
+  class ReviewsController < AuthenticatedController
+    skip_before_action :validate_jwt, only: :index
 
-  def index
-    @product = Product.find_by(public_id: params[:product_id])
+    def index
+      @product = Product.find_by(public_id: params[:product_id])
 
-    render status: :not_found if @product.nil?
+      render status: :not_found and return if @product.nil?
 
-    @reviews = @product.product_reviews.joins(:user_account).select('product_reviews.*',
+      @reviews = @product.product_reviews.joins(:user_account).select('product_reviews.*',
                                                                     'user_accounts.first_name as author_name')
-    render json: serialized_collection.serializable_hash, status: :ok
-  end
+      render json: serialized_collection.serializable_hash, status: :ok
+    end
 
-  def create
-    @review = Review.new(permitted_params.merge({ product_id: params[:product_id],
-                                                  user_account_id: current_user_account.id }))
+    def create
+      @review = Review.new(permitted_params.merge({ product_id: params[:product_id],
+                                                    user_account_id: current_user_account.id }))
 
-    if @review.save
-      render json: serialized_object.serializable_hash, status: :ok
-    else
-      render json: { error_messages: @review.errors.full_messages }, status: :unprocessable_entity
+      if @review.save
+        render json: serialized_object.serializable_hash, status: :ok
+      else
+        render json: { error_messages: @review.errors.full_messages }, status: :unprocessable_entity
+      end
     end
-  end
 
-  private
+    private
 
-  def permitted_params
-    params.permit(:review, :rating)
-  end
+    def permitted_params
+      params.permit(:review, :rating)
+    end
 
-  def serialized_collection
-    @serialized_collection ||= ProductReviewSerializer.new(@reviews)
-  end
+    def serialized_collection
+      @serialized_collection ||= ProductReviewSerializer.new(@reviews)
+    end
 
-  def serialized_object
-    @serialized_object ||= ProductReviewSerializer.new(@review)
+    def serialized_object
+      @serialized_object ||= ProductReviewSerializer.new(@review)
+    end
   end
 end
-- 
cgit v1.2.3