处理与虚拟属性对应的多参数属性的正确方法

时间:2013-07-05 06:33:56

标签: ruby-on-rails ruby

我有一个包含birthdate属性的模型的Rails应用。这对应于使用ActiveRecord date类型定义的数据库中的列。有了这个,我可以使用date_select表单帮助器方法在我的视图中将其渲染为三选输入。然后,与此字段对应的表单参数将序列化为birthdate(1i)birthdate(2i)birthdate(3i)。因此,我可以在模型的控制器中使用标准update_attributes方法来更新模型上的所有字段。

我现在正在尝试使用attr_encrypted gem加密此字段。虽然gem支持编组(这很好),但不再有birthdate类型的名称date的真实列 - 而是attr_encrypted将值公开为虚拟< / em>属性birthdate由真实encrypted_birthdate列支持。这意味着update_attributes无法执行先前的多参数属性分配来填充和保存此列。相反,我因调用内部MultiparameterAssignmentErrors方法而得到column_for_attribute错误,该方法返回此列的nil(来自execute_callstack_for_multiparameter_attributes内的某处)。

我目前正在努力解决以下问题:

我的模型app/models/person.rb

class Person < ActiveRecord::Base
  attr_encrypted :birthdate
end

app/controllers/people_controller.rb中的我的控制器:

class PeopleController < ApplicationController
  def update

    # This is the bit I would like to avoid having to do.
    params[:person] = munge_params(params[:person])

    respond_to do |format|
      if @person.update_attributes(params[:person])
        format.html { redirect_to @person, notice: 'Person was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: "edit" }
        format.json { render json: @person.errors, status: :unprocessable_entity }
      end
    end
  end

  private

  def munge_params(params)
    # This separates the "birthdate" parameters from the other parameters in the request.
    birthdate_params, munged_params = extract_multiparameter_attribute(params, :birthdate)

    # Now we place a scalar "birthdate" where the multiparameter attribute used to be.
    munged_params['birthdate'] = Date.new(
      birthdate_params[1],
      birthdate_params[2],
      birthdate_params[3]
    )

    munged_params
  end

  def extract_multiparameter_attribute(params, name)
    # This is sample code for demonstration purposes only and currently
    # only handles the integer "i" type.
    regex = /^#{Regexp.quote(name.to_s)}\((\d+)i)\)$/
    attribute_params, other_params = params.segment { |k, v| k =~ regex }
    attribute_params2 = Hash[attribute_params.collect do |key, value|
      key =~ regex or raise RuntimeError.new("Invalid key \"#{key}\"")
      index = Integer($1)
      [index, Integer(value)]
    end]
    [attribute_params2, other_params]
  end

  def segment(hash, &discriminator)
    hash.to_a.partition(&discriminator).map do |a|
      a.each_with_object(Hash.new) { |e, h| h[e.first] = e.last }
    end
  end
end

我的观点app/views/people/_form.html.erb

<%= form_for @person do |f| %>
    <%= f.label :birthdate %>
    <%= f.date_select :birthdate %>

    <% f.submit %>
<% end %>

处理这种类型属性的正确方法是什么,而不必像这样引入params数组的临时替换?

更新: 看起来this可能会引用相关问题。而且this也是。

另一个更新:

这是我目前的解决方案,基于Chris Heald的回答。此代码应添加到Person模型类:

class EncryptedAttributeClassWrapper
  attr_reader :klass
  def initialize(klass); @klass = klass; end
end

# TODO: Modify attr_encrypted to take a :class option in order
# to populate this hash.
ENCRYPTED_ATTRIBUTE_CLASS_WRAPPERS = {
  :birthdate => EncryptedAttributeClassWrapper.new(Date)
}

def column_for_attribute(attribute)
  attribute_sym = attribute.to_sym
  if encrypted = self.class.encrypted_attributes[attribute_sym]
    column_info = ENCRYPTED_ATTRIBUTE_CLASS_WRAPPERS[attribute_sym]
    column_info ||= super encrypted[:attribute]
    column_info
  else
    super
  end
end

此解决方案按原样运行,但如果attr_encrypted采用可动态构造:class哈希的ENCRYPTED_ATTRIBUTE_CLASS_WRAPPERS选项,则会更好。我将研究如何扩展/ monkeypatch attr_encrypted来实现这一目标。可在此处获取要点:https://gist.github.com/rcook/5992293

1 个答案:

答案 0 :(得分:1)

您可以对模型进行monkeypatch以通过column_for_attribute次调用。我没有对此进行测试,但它应该导致birthday字段上的反射而不是返回encrypted_birthday字段的反射,这应该导致多参数属性正确分配(因为AR将能够推断字段类型):

def column_for_attribute(attribute)
  if encrypted = encrypted_attributes[attribute.to_sym]
    super encrypted[:attribute]
  else
    super
  end
end

我们正在为this line修补column_for_attribute,以便AR可以推断出该列的正确类型。它需要弄清楚“birthday”的参数应该是DateTime类型的whatnot,并且不能从虚拟属性推断出。将反射映射到实际列应解决该问题。