Fixed to_i position for - nil.presence.to_i || 1 => 0 https://stackoverflow.com/questions/59991786/rails-hash-null-check-which-is-better-if-condition-and-ternary-operator/59992604?noredirect=1#comment106298977_59992604

Author: x1wins

app/serializers/category_serializer.rb

@@ -3,12 +3,12 @@ class CategorySerializer < ActiveModel::Serializer
   has_one :user
   has_many :posts
   def posts
-    post_page = instance_options.dig(:pagaination_param, :post_page).presence.to_i || 1
-    post_per = instance_options.dig(:pagaination_param, :post_per).presence.to_i || 10
+    post_page = (instance_options.dig(:pagaination_param, :post_page).presence || 1).to_i
+    post_per = (instance_options.dig(:pagaination_param, :post_per).presence || 0).to_i
     object.posts.published.by_date.page(post_page).per(post_per)
   end
   def posts_pagination
-    post_per = instance_options.dig(:pagaination_param, :post_per).presence.to_i || 10
+    post_per = (instance_options.dig(:pagaination_param, :post_per).presence || 10).to_i
     Pagination.build_json(posts)[:posts_pagination] if post_per > 0
   end
 end

app/serializers/post_serializer.rb

@@ -11,12 +11,12 @@ def files
     file_urls
   end
   def comments
-    comment_page = instance_options.dig(:pagaination_param, :comment_page).presence.to_i || 1
-    comment_per = instance_options.dig(:pagaination_param, :comment_per).presence.to_i || 0
+    comment_page = (instance_options.dig(:pagaination_param, :comment_page).presence || 1).to_i
+    comment_per = (instance_options.dig(:pagaination_param, :comment_per).presence || 10).to_i
     object.comments.published.by_date.page(comment_page).per(comment_per)
   end
   def comments_pagination
-    comment_per = instance_options.dig(:pagaination_param, :comment_per).presence.to_i || 0
+    comment_per = (instance_options.dig(:pagaination_param, :comment_per).presence || 10).to_i
     Pagination.build_json(comments)[:comments_pagination] if comment_per > 0
   end
 end