Fix: Move author assignment logic to Article model

- Remove custom create/update methods from Avo resource
- Implement author_id= method in Article model to handle authorship
- Simplify Avo resource configuration
- Handle author assignment through model's virtual attribute

This approach is cleaner and follows Rails conventions better.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Shpigford

app/avo/resources/article.rb

@@ -15,60 +15,12 @@ def fields
     field :content, as: :easy_mde
     field :publish_at, as: :date_time
     field :author_name, as: :text, hide_on: [ :forms ]
-    field :author, as: :belongs_to, searchable: true, display_with_value: -> { record.name }, use_resource: Avo::Resources::Author
+    field :author, as: :belongs_to, searchable: true
 
     tabs do
       tab "Author Details" do
         field :authorship, as: :has_one
       end
     end
   end
-
-  def create_model_record(model, params, **args)
-    Rails.logger.debug "=== Article Create Params: #{params.inspect}"
-    
-    # Try different possible param keys
-    author_id = params.delete(:author_id) || params.delete("author_id") || params.dig(:author, :id) || params.dig("author", "id")
-    Rails.logger.debug "=== Author ID extracted: #{author_id.inspect}"
-
-    super(model, params, **args)
-
-    handle_author_assignment(model, author_id)
-
-    model
-  end
-
-  def update_model_record(model, params, **args)
-    Rails.logger.debug "=== Article Update Params: #{params.inspect}"
-    
-    # Try different possible param keys
-    author_id = params.delete(:author_id) || params.delete("author_id") || params.dig(:author, :id) || params.dig("author", "id")
-    Rails.logger.debug "=== Author ID extracted: #{author_id.inspect}"
-
-    super(model, params, **args)
-
-    handle_author_assignment(model, author_id)
-
-    model
-  end
-
-  private
-
-  def handle_author_assignment(model, author_id)
-    Rails.logger.debug "=== Handling author assignment with ID: #{author_id.inspect}"
-    
-    if author_id.present?
-      # Remove existing authorship if present
-      model.authorship&.destroy
-      Rails.logger.debug "=== Destroyed existing authorship"
-
-      # Create new authorship with the selected author
-      authorship = model.create_authorship(author_id: author_id, role: "primary")
-      Rails.logger.debug "=== Created new authorship: #{authorship.inspect}, valid: #{authorship.valid?}, errors: #{authorship.errors.full_messages}"
-    elsif author_id == ""
-      # If author_id is blank, remove the authorship
-      model.authorship&.destroy
-      Rails.logger.debug "=== Removed authorship (blank author_id)"
-    end
-  end
 end

app/models/article.rb

@@ -26,7 +26,15 @@ def author_id
   end
 
   def author_id=(id)
-    # This is handled in the Avo resource
+    if id.blank?
+      self.authorship&.destroy
+    else
+      if self.authorship
+        self.authorship.update(author_id: id)
+      else
+        self.create_authorship(author_id: id, role: "primary")
+      end
+    end
   end
 
   include MetaImage