Search code examples
ruby-on-rails-4argumentsrails-activerecordupdate-all

Rails 4 update_all syntax - Argument error


I'm getting the error:

Wrong number of arguments (2 for 1)

On my Task model when I defined my method to update all status of tasks. What is the correct syntax?

class Task < ActiveRecord::Base
  belongs_to :user

  def self.toggle(user, groups)
    groups.each do |status, ids|
      user.tasks.update_all({status: status.to_s}, {id: ids}) #=> error here
    end
  end
end

class GroupIdsByStatus
  def self.group(options = {})
    result = Hash.new {|h,k| h[k] = []} 
    options.reduce(result) do |buffer, (id, status)| 
      buffer[status.to_sym] << id
      buffer
    end
    result
  end
end

class TasksController < ApplicationController
  def toggle
    groups = GroupIdsByStatus.group(params[:tasks])
    Task.toggle(current_user, groups)

    redirect_to tasks_path
  end
end

Solution

  • The method update_all receives a single Hash as its sole argument. You need to put all of the updates into the single argument:

    user.tasks.update_all(status: status.to_s, id: ids)
    
    # The above line of code is identical to:
    # user.tasks.update_all( {status: status.to_s, id: ids} )  # < Notice the curly braces
    

    More information on this method is shown in the Rails Relation Docs