enhancement: friendly_id native support

lib/avo/resources/base.rb

@@ -73,7 +73,19 @@ def current_user
         query
       }
       class_attribute :find_record_method, default: -> {
-        query.find id
+        # Check if the model uses FriendlyId and handle accordingly
+        if model_class.respond_to?(:friendly_id_config)
+          if id.is_a?(Array)
+            # For arrays, use the slug column
+            query.where(model_class.friendly_id_config.slug_column => id)
+          else
+            # For single values, use find_by_slug method
+            query.friendly.find(id)
+          end
+        else
+          # Standard Rails find behavior for non-FriendlyId models
+          query.find id
+        end
       }
       class_attribute :after_create_path, default: :show
       class_attribute :after_update_path, default: :show
@@ -236,7 +248,8 @@ def find_record(id, query: nil, params: nil)
             target: find_record_method,
             query: query,
             id: id,
-            params: params
+            params: params,
+            model_class:
           ).handle
         end
 

spec/dummy/app/avo/resources/compact_user.rb

@@ -1,8 +1,5 @@
 class Avo::Resources::CompactUser < Avo::BaseResource
   self.model_class = ::User
-  self.find_record_method = -> {
-    query.friendly.find id
-  }
 
   def fields
     field :personal_information, as: :heading

spec/dummy/app/avo/resources/field_discovery_user.rb

@@ -1,9 +1,6 @@
 class Avo::Resources::FieldDiscoveryUser < Avo::BaseResource
   self.model_class = ::User
   self.description = "This is a resource with discovered fields. It will show fields and associations as defined in the model."
-  self.find_record_method = -> {
-    query.friendly.find id
-  }
 
   def fields
     main_panel do

spec/dummy/app/avo/resources/post.rb

@@ -17,14 +17,10 @@ class Avo::Resources::Post < Avo::BaseResource
     mobile_user ? :table : :grid
   }
   self.find_record_method = -> {
-    # When using friendly_id, we need to check if the id is a slug or an id.
-    # If it's a slug, we need to use the find_by_slug method.
-    # If it's an id, we need to use the find method.
-    # If the id is an array, we need to use the where method in order to return a collection.
     if id.is_a?(Array)
-      id.first.to_i == 0 ? query.where(slug: id) : query.where(id: id)
+     query.where(slug: id)
     else
-      id.to_i == 0 ? query.find_by_slug(id) : query.find(id)
+      query.find_by_slug(id)
     end
   }
   self.view_types = [:grid, :table]

spec/dummy/app/avo/resources/team_user.rb

@@ -1,16 +1,5 @@
 class Avo::Resources::TeamUser < Avo::BaseResource
   self.model_class = "User"
-  self.find_record_method = -> {
-    # When using friendly_id, we need to check if the id is a slug or an id.
-    # If it's a slug, we need to use the find_by_slug method.
-    # If it's an id, we need to use the find method.
-    # If the id is an array, we need to use the where method in order to return a collection.
-    if id.is_a?(Array)
-      id.first.to_i == 0 ? query.where(slug: id) : query.where(id: id)
-    else
-      id.to_i == 0 ? query.find_by_slug(id) : query.find(id)
-    end
-  }
 
   def fields
     field :id, as: :id

spec/dummy/app/avo/resources/user.rb

@@ -20,19 +20,7 @@ class Avo::Resources::User < Avo::BaseResource
   self.index_query = -> do
     query.order(last_name: :asc)
   end
-  self.find_record_method = -> {
-    # When using friendly_id, we need to check if the id is a slug or an id.
-    # If it's a slug, we need to use the find_by_slug method.
-    # If it's an id, we need to use the find method.
-    # If the id is an array, we need to use the where method in order to return a collection.
-    if id.is_a?(Array)
-      first_is_number = true if Float(id.first, exception: false)
-      first_is_number ? query.where(id: id) : query.where(slug: id)
-    else
-      first_is_number = true if Float(id, exception: false)
-      first_is_number ? query.find(id) : query.find_by_slug(id)
-    end
-  }
+
   self.includes = [:posts, :post]
   self.attachments = [:cv]
   self.devise_password_optional = true

spec/dummy/app/avo/resources/z_post.rb

@@ -9,15 +9,11 @@ class Avo::Resources::ZPost < Avo::BaseResource
   self.default_view_type = :grid
   self.model_class = "Post"
   self.find_record_method = -> {
-    # When using friendly_id, we need to check if the id is a slug or an id.
-    # If it's a slug, we need to use the find_by_slug method.
-    # If it's an id, we need to use the find method.
-    # If the id is an array, we need to use the where method in order to return a collection.
     if id.is_a?(Array)
-      return id.first.to_i == 0 ? query.where(slug: id) : query.where(id: id)
+     query.where(slug: id)
+    else
+      query.find_by_slug(id)
     end
-
-    id.to_i == 0 ? query.find_by_slug(id) : query.find(id)
   }
   self.visible_on_sidebar = false