error handling

app/overrides/lib/kanamari/helpers/helper_methods_override.rb

@@ -2,24 +2,52 @@
 # [hyc-override] https://github.com/kaminari/kaminari/blob/v1.2.2/kaminari-core/lib/kaminari/helpers/helper_methods.rb
 Kaminari::Helpers::HelperMethods.module_eval do
     # Helper to generate a link to a specific page
-  def link_to_specific_page(scope, page, name, **options)
-    specific_page_path = path_to_specific_page(scope, page, options)
-
-    # Remove the :params and :param_name keys from the options hash before generating the link since they are irrelevant
-    options.except! :params, :param_name
-
-    # Setting aria instead of rel for accessibility
-    options[:aria] ||= { label: "Go to page #{page}" }
-
-    if specific_page_path
-      link_to(name || page, specific_page_path, options)
-    elsif block_given?
-      yield
+    def link_to_specific_page(scope, page, name, **options)
+      begin
+        # Validate inputs
+        raise ArgumentError, "Scope is required and must respond to :total_pages" unless scope&.respond_to?(:total_pages)
+        raise ArgumentError, "Page number must be a positive integer" unless page.is_a?(Integer) && page.positive?
+
+        specific_page_path = path_to_specific_page(scope, page, options)
+
+        # Remove the :params and :param_name keys from the options hash before generating the link since they are irrelevant
+        options.except! :params, :param_name
+
+        # Setting aria instead of rel for accessibility
+        options[:aria] ||= { label: "Go to page #{page}" }
+
+        if specific_page_path
+          link_to(name || page, specific_page_path, options)
+        elsif block_given?
+          yield
+        else
+          Rails.logger.warn "Specific page path could not be generated for page: #{page}"
+          nil
+        end
+      rescue ArgumentError => e
+        Rails.logger.error "Error in link_to_specific_page: #{e.message}"
+        nil
+      rescue StandardError => e
+        Rails.logger.error "Unexpected error in link_to_specific_page: #{e.message}"
+        nil
+      end
     end
-  end
-
+
     # Helper to generate the path for a specific page
-  def path_to_specific_page(scope, page, options = {})
-    Kaminari::Helpers::Page.new(self, **options.reverse_merge(current_page: page)).url if page.positive? && page <= scope.total_pages
+    def path_to_specific_page(scope, page, options = {})
+      begin
+        # Validate inputs
+        raise ArgumentError, "Page number must be a positive integer" unless page.is_a?(Integer) && page.positive?
+        raise ArgumentError, "Scope must respond to :total_pages" unless scope&.respond_to?(:total_pages)
+
+        Kaminari::Helpers::Page.new(self, **options.reverse_merge(current_page: page)).url if page <= scope.total_pages
+      rescue ArgumentError => e
+        Rails.logger.error "Error in path_to_specific_page: #{e.message}"
+        nil
+      rescue StandardError => e
+        Rails.logger.error "Unexpected error in path_to_specific_page: #{e.message}"
+        nil
+      end
+    end
   end
-end
+