Question

我正在使用Phusion Passenger部署的Rails站点上进行维护工作。工作流程与标准的三层Railsian测试开发生产安排略有不同;相反，对于并行Oracle数据库运行相同代码库的两个单独安装;开发网站位于qa.domain.com，网站位于www.domain.com

我在以下代码片段中遇到了不同的行为（来自'vendors_controller.rb'，它使用AuthenticatedSystem）在两个环境之间：

def create
  @user = current_user || User.new(params[:user])
  @registration = Registration.new(params[:registration])

  unless current_user

    @user.user_type = 'vendor'
    @user.active = 1

    if @user.save
      @user.activate!
      @user.has_role 'owner', @user
      @user.has_role 'vendor'
      self.current_user = user = @user

      @registration.active = 1
      @registration.email = @user.email
      @registration.user_id = @user.id
      if @registration.save
        send_confirmation(@user)
        send_solicitations_notifications(@registration) if @registration.notification_desired == true
        redirect_to thank_you_vendors_path
      else
        # THIS BEHAVIOR DIFFERS ACROSS PRODUCTION AND DEVELOPMENT
        @user.destroy
        self.current_user = user = nil
        # END DIFFERENCE
        respond_to do |format|
          format.html { render :action => 'new' }
          format.xml  { render :xml => @registration.errors, :status => :unprocessable_entity }
        end
      end

    else
      respond_to do |format|
        format.html  { render :action => 'new' }
        format.xml  { render :xml => @user.errors, :status => :unprocessable_entity }
      end
    end
...

如果系统无法创建相应的注册，则注释之间的代码会破坏刚刚创建的用户对象。它在开发服务器上运行良好，但在生产服务器上运行正常，即使保存注册失败，User对象也会固执地挂起数据库。将更改推送到生产环境只需上传控制器文件并通过shell执行touch tmp/restart.txt即可。这两个代码库在其他方面是相同的;可能导致这种差异的原因是什么？

感谢您的考虑！

贾斯汀

编辑：两个安装中的production.rb存在一些差异，可能有助于诊断问题。在制作上，

config.cache_classes = true

# Full error reports are disabled and caching is turned on
config.action_controller.consider_all_requests_local = false
config.action_controller.perform_caching             = true

在开发时，这三个标志被设置为它们的反向值。谢谢！

Answer 1

您应该考虑更改有关代码的一些事项：

您没有使用交易
你在控制器中做得太多了

说到你出现问题的原因可能是由于生产和开发之间的环境差异，很可能就是这样：

config.cache_classes = false

但是，我不认为你应该在生产中改变它，因为它会减慢你的所有行动。相反，我建议使用一个与您的生产环境非常匹配的临时环境。

要解决您的问题，我很可能会重写这样的操作：

# using before filters will keep your actions tight
before_filter :cannot_create_user, :if => :signed_in?

def create
  # setup all the objects
  @user = User.new(params[:user])

  @user.user_type = 'vendor'
  @user.active = 1

  @user.has_role 'owner', @user
  @user.has_role 'vendor'

  @registration = @user.registrations.build(params[:registration])
  @registration.active = 1
  @registration.email = @user.email

  # make sure everything is valid before trying to save and activate
  if @user.valid?
    @user.save!  # might not need this if activate calls save!
    @user.activate!

    # this should probably be a sign_in() method... 
    self.current_user = @user

    send_confirmation(@user)
    send_solicitations_notifications(@registration) if @registration.notification_desired?

    redirect_to thank_you_vendors_path
  else
    respond_to do |format|
      format.html { render :action => 'new' }
      format.xml  { render :xml => @registration.errors, :status => :unprocessable_entity }
    end
  end

...
end


protected 

def signed_in?
  !current_user.nil?
end

def cannot_create_user
  respond_to do |format|
    format.html  { render :action => 'new' }
    format.xml  { render :xml => @user.errors, :status => :unprocessable_entity }
  end
end

<强> n.b。我没有测试这个，它可能不起作用，但是你应该得到这个想法...如果你有单元测试（我希望你这样做......）你应该能够放入它并查看它是否有效！

您的下一步是使用accepts_nested_attribute_for作为注册对象，以便将其作为用户参数的一部分提交。

我还要重构这一点，以便所有角色设置等都在callbacks中完成。

此时，您的create操作很可能非常简单，您可以将控制器切换为使用inherited resources。

我希望这有帮助！

Rails：开发和生产的不同行为

1 个答案: