ROR：arguments 的编号错误（给定 2，预期 1）

Question

在下面的一段代码上

def initialize(clause)
      clause =~ /^([\w\_\.]+)(->'\w+')?_(desc|asc)$|^([\w\_\.]+->>'[\w\_]+')(->'\w+')?_(desc|asc)$/
      @column = $1 || $4
      @op = $2 || $5
      @order = $3 || $6
      @field = [@column, @op].compact.join
    end

我收到以下错误

ArgumentError (wrong number of arguments (given 2, expected 1)):
  config/initializers/order_clause.rb:5:in `initialize' 

有谁知道如何解决它？ 请帮忙！

编辑- 我正在添加完整的代码以进行更多说明。

module ActiveAdmin
  class OrderClause
    attr_reader :field, :order

    def initialize(clause)
      clause =~ /^([\w\_\.]+)(->'\w+')?_(desc|asc)$|^([\w\_\.]+->>'[\w\_]+')(->'\w+')?_(desc|asc)$/
      @column = $1 || $4
      @op = $2 || $5
      @order = $3 || $6
      @field = [@column, @op].compact.join
    end

    def valid?
      @field.present? && @order.present?
    end

    def to_sql(active_admin_config)
      table = column_in_table?(active_admin_config.resource_column_names, @column) ? active_admin_config.resource_table_name : nil
      if json_column?(@column)
        table_column = (@column =~ /\./) ? @column : [table, @column].compact.join(".")
        ['(', table_column, @op, ' ',')::numeric ', @order].compact.join
      else
        table_column = (@column =~ /\./) ? @column : [table, active_admin_config.resource_quoted_column_name(@column)].compact.join(".")
        [table_column, @op, ' ', @order].compact.join
      end
    end

    private

    def json_column?(column)
      column.include?('->>')
    end

    def column_in_table?(names, column)
      column = json_column?(column) ? column.split('->>')[0].strip : column
      names.include?(column)
    end
  end
end

Answer 1

如您在此处和此处所见， activeadmin gem 使用几个 arguments（ active_admin_config和order ）实例化ActiveAdmin::OrderClause class。 修复initialize方法 arguments。

def initialize(active_admin_config, clause)
  # ... 
end

您还应该从to_sql方法中删除active_admin_config参数，因为它是在没有 arguments 的情况下调用的。 您可以在initialize方法中设置@active_admin_config并将:active_admin_config添加到attr_reader调用以在to_sql方法中使用。

module ActiveAdmin
  class OrderClause
    attr_reader :field, :order, :active_admin_config

    def initialize(active_admin_config, clause)
      @active_admin_config = active_admin_config
      # rest of the code
    end

    def to_sql
      # ...
    end
  end
end

我建议您创建一个继承自 gem 的ActiveAdmin::OrderClause CustomOrderClause的 CustomOrderClause class 并仅覆盖必要的方法。 然后，您可以在初始化程序中配置 activeadmin 时使用config.order_clause = CustomOrderClause 。

Answer 2

我认为您正在尝试覆盖ActiveAdmin::OrderClause ，如果您看到 ActivAdmin 的代码，您会发现：

module ActiveAdmin
  class OrderClause
    attr_reader :field, :order, :active_admin_config

    def initialize(active_admin_config, clause)
      clause =~ /^([\w\.]+)(->'\w+')?_(desc|asc)$/
      @column = $1
      @op = $2
      @order = $3
      @active_admin_config = active_admin_config
      @field = [@column, @op].compact.join
    end

这意味着无论在哪里初始化它都会用两个 arguments 进行初始化，因为它们的构造函数接受它。 因此，您必须更改代码中的initialize方法以接受两个 arguments 或仅从调用它的任何位置传递一个参数，这将是乏味的。

Answer 3

而不是覆盖整个 class 您应该能够使用Module#prepend来更改您有兴趣更改的功能，而不必担心实现的 rest。 例如

module OrderClauseExtension 
  def initialize(active_admin_config, clause)
      super
      if field.empty?
        clause =~ /^([\w\_\.]+->>'[\w\_]+')(->'\w+')?_(desc|asc)$/
        @column = $1 
        @op = $2 
        @order = $3
        @field = [@column, @op].compact.join
      end
  end
end 

ActiveAdmin::OrderClause.prepend(OrderClauseExtension) 

这将首先尝试原始实现，并在field属性导致空String的情况下使用您的实现作为后备，这将是[nil,nil].compact.join #=> ""中的情况

此 class 中包含的功能的所有 rest 将保持不变，您不必担心 gem 本身的任何更改（除非初始化方法和/或此处分配的属性发生更改）

您可以将此文件放在config/initializers中。