Edit object with HABTM association modify database before validation with ActiveAdmin

Question

I Having an issue when trying to update a model that has has_and_belongs_to_many association.

Let's say that Post has_and_belongs_to_many Tag, and Post validates the presence of title and Tags. If I update Post, removing its title and tags, I get validation error in title and tags, ok. But ActiveAdmin already removed the records that make association between Post and Tag, so, if I leave Post edit page, the post is left invalid on database, without tags.

Here my models:

class Tag < ActiveRecord::Base
  attr_accessible :label
  has_and_belongs_to_many :posts
end

---

class Post < ActiveRecord::Base
  attr_accessible :content, :title, :tag_ids
  has_and_belongs_to_many :tags
  validates_presence_of :content, :title, :tags
end

---

ActiveAdmin.register Post do

  form do |f|
    f.inputs do
        f.input :title
        f.input :content
        f.input :image
        f.input :tags
    end

    f.buttons
  end
end

I usign chosen-rails gem and it allows user to unselect all tags of post.

Summarizing, my problem is: ActiveAdmin updates relationships on database before perform model validations.

There a solution for this behavior or I doing something wrong?

Edit:

Here the request log when I trying to update post without title and tags:

Started PUT "/admin/posts/8" for 127.0.0.1 at 2013-04-01 10:32:07 -0300
Processing by Admin::PostsController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"amSbLlP/rgDrNn/N8lgq/KEaRXK1fMPShZDwpZ0QIJ4=", "post"=>{"title"=>"", "content"=>"content", "tag_ids"=>["", ""]}, "commit"=>"Update Post", "id"=>"8"}
  AdminUser Load (0.2ms)  SELECT `admin_users`.* FROM `admin_users` WHERE `admin_users`.`id` = 1 LIMIT 1
  Post Load (0.2ms)  SELECT `posts`.* FROM `posts` WHERE `posts`.`id` = 8 LIMIT 1
  Tag Load (0.2ms)  SELECT `tags`.* FROM `tags` INNER JOIN `posts_tags` ON `tags`.`id` = `posts_tags`.`tag_id` WHERE `posts_tags`.`post_id` = 8
   (0.1ms)  BEGIN
  SQL (12.3ms)  DELETE FROM `posts_tags` WHERE `posts_tags`.`post_id` = 8 AND `posts_tags`.`tag_id` IN (1, 2)
   (49.6ms)  COMMIT
   (0.1ms)  BEGIN
   (0.2ms)  ROLLBACK
  Post Load (0.3ms)  SELECT `posts`.* FROM `posts` WHERE `posts`.`id` = 8 LIMIT 1
  Tag Load (0.2ms)  SELECT `tags`.* FROM `tags` 
  Rendered /home/rodrigo/.rvm/gems/ruby-1.9.3-p125@blog/gems/activeadmin-0.5.1/app/views/active_admin/resource/edit.html.arb (192.3ms)
Completed 200 OK in 276ms (Views: 194.8ms | ActiveRecord: 63.3ms)

EDIT 2:

Ok, I sure that ActiveAdmin has this bug.

Looking at ActiveRecord behaviour, I think that validation flow is broken using only model class. See this example:

1.9.3p125 :064 > post = Post.find(8)
  Post Load (0.3ms)  SELECT `posts`.* FROM `posts` WHERE `posts`.`id` = 8 LIMIT 1
 => #<Post id: 8, title: "title", content: "content", created_at: "2013-03-27 13:13:20", updated_at: "2013-03-27 13:13:20", image: "extrato.bmp"> 
1.9.3p125 :065 > post.tags
  Tag Load (0.2ms)  SELECT `tags`.* FROM `tags` INNER JOIN `posts_tags` ON `tags`.`id` = `posts_tags`.`tag_id` WHERE `posts_tags`.`post_id` = 8
 => [#<Tag id: 1, label: "tag", created_at: "2013-02-25 18:32:45", updated_at: "2013-02-25 18:32:45">, #<Tag id: 2, label: "new", created_at: "2013-02-25 18:32:50", updated_at: "2013-02-25 18:32:50">] 
1.9.3p125 :066 > post.title = ""
 => "" 
1.9.3p125 :067 > post.save #<<<<<<< It's invalid on title
 => false 
1.9.3p125 :068 > post.tags = []  #<<<<<<< This shouldnt trigger database update
   (0.3ms)  BEGIN
  SQL (0.5ms)  DELETE FROM `posts_tags` WHERE `posts_tags`.`post_id` = 8 AND `posts_tags`.`tag_id` IN (1, 2)
   (55.5ms)  COMMIT
 => [] 
1.9.3p125 :069 > post.save #<<<<<<<  It's invalid on title AND TAGS
   (0.2ms)  BEGIN
   (0.2ms)  ROLLBACK
 => false 
1.9.3p125 :070 > post.reload
  Post Load (0.2ms)  SELECT `posts`.* FROM `posts` WHERE `posts`.`id` = 8 LIMIT 1
 => #<Post id: 8, title: "title", content: "content", created_at: "2013-03-27 13:13:20", updated_at: "2013-03-27 13:13:20", image: "extrato.bmp"> 
1.9.3p125 :071 > post.valid? #<<<<<<< Now, I have this model in invalid state 
  Tag Load (0.6ms)  SELECT `tags`.* FROM `tags` INNER JOIN `posts_tags` ON `tags`.`id` = `posts_tags`.`tag_id` WHERE `posts_tags`.`post_id` = 8
 => false 

Has any way to update post attributes(including tags) and validate the model before doing any database update?

Answer 1

You can use this gem: https://github.com/MartinKoerner/deferred_associations

The deferred associations will fix this bug.

Answer 2

@Rodrigo I was able to reproduce your issue locally without active admin, the issue is actually that is one of the default operations when using HABTM relationships if you see [here][1]

[1]: http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-has_and_belongs_to_many it says:

collection=objects

Replaces the collection’s content by deleting and adding objects as appropriate.

So apparently you will need to override this operation

Here is an example:

Override ActiveRecord << operator on has_many :through relationship, to accept data for the join model

Let me know how can I help you

Answer 3

I'd propose to add a tmp variable and store values in it. Then you should move them to database if validation passed.

Here is an example:

Your models/article.rb:

class Article < ActiveRecord::Base
  validate :author_presence
  has_and_belongs_to_many :authors

  attr_writer :tmp_author_ids

  def tmp_author_ids
    @tmp_author_ids || author_ids
  end

  def author_presence
    if tmp_author_ids.reject(&:blank?).empty?
      errors.add(:tmp_author_ids, 'Author is missing')
    else
      self.author_ids = tmp_author_ids
    end
  end
end

Your admin/article.rb, block form:

f.input :tmp_author_ids, as: :select, multiple: true, collection: Author.all, label: 'Authors'

That's it

Answer 4

So, for anyone having the same issue, I found a workaround: You can define a before_remove in the has_and_belongs_to_many association and raise an exception that then you can catch in ActiveAdmin.

For this it would be

class Post < ActiveRecord::Base
  attr_accessible :content, :title, :tag_ids
  has_and_belongs_to_many :tags, before_remove: :check_something
  validates_presence_of :content, :title, :tags
end

def check_something(agent)
    if self.tags.size == 1
      raise ActiveModel::MissingAttributeError.new 'something'
    end
end

more info: https://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html

Answer 5

if you want, you can redefine you update action in active_admin to prevent saving empty tags, in something like this style

ActiveAdmin.register Post do

  controller do
    def update
      if params[:post][:tag_ids] == ["", ""]
        flash.now[:alert] = "You can't remove all tags"
        render :edit
      else
        super
      end
    end
  end
...
end

and i think this stuff from model can be deleted

attr_accessor :new_tag_ids

validate :validate_new_tags_ids

after_save :update_tags

def update_tags
    self.tag_ids = @new_tag_ids if defined?(@new_tag_ids)
    @new_tag_ids = nil
end

private 
def validate_new_tags_ids
    errors[:tags] << "can't be blank (2)" if @new_tag_ids.blank? 
end