On 6 August 2016 at 08:51, Serguei Cambour <s.camb...@gmail.com> wrote: > I wonder what is the best way to delete all the records of the association > on a specified condition. > For example, you have User model that has many Post(s). How would keep all > the recent User's Posts and delete all the last Posts when a Users have more > than 25 ? > > There is a solution like that: > > def self.delete_old_posts > old_posts = [] > users = includes(:post) > users.each do |user| > old_posts =+ user.posts.offset(25) unless user.posts.empty? > end > > > old_posts.delete_all > end > > Any other ideas ? Thank you.
You need to order the posts otherwise there is no guarantee which ones will be deleted. I don't think you need includes(:post). Also you don't need the unless. Are you sure you want delete_all? That will not call callbacks. In fact I am not sure you can use delete_all in this situation. Colin > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Talk" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to rubyonrails-talk+unsubscr...@googlegroups.com. > To post to this group, send email to rubyonrails-talk@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/rubyonrails-talk/e72d40de-1f2b-494c-8e47-ff172c4cd817%40googlegroups.com. > For more options, visit https://groups.google.com/d/optout. -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscr...@googlegroups.com. To post to this group, send email to rubyonrails-talk@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/CAL%3D0gLujwZNO1RH68ketACmA1GOOgNputuAE7jjPhGuNLrq6MA%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.