2009-08-18 63 views
1

我一直在项目上使用attachment_fu很长一段时间,一切都很好,但现在正在尝试将项目升级到rails 2.3.3我遇到了一个奇怪的bug使我抓狂。在这种情况下,附件是一个标志,在创建时可以正确验证,但在更新时不会失败验证。我已经调试过它,并且初始验证失败,但似乎没有抛出异常,或者至少没有一个被我在控制器中救出的异常。似乎我已经尝试了一切,但无法弄清楚这一点。Attachment_fu无法验证更新

控制器:

# POST /tournaments 
    # POST /tournaments.xml 
    def create 
    # Build tournament 
    @tournament = Tournament.new(params[:tournament].merge(:user_id => current_user.id)) 

    # Save the uploaded attachments 
    params[:uploads].each do |upload| 
     @tournament.documents << Document.new({:uploaded_data => upload[:document]}.merge(:description => upload[:description])) 
    end unless params[:uploads].nil? 

    # if supplied save an event logo 
    @logo = Logo.new({:uploaded_data => params[:logo][:upload_data]}) unless params[:logo].nil? or params[:logo][:upload_data].blank? 
    @tournament.logo = @logo unless @logo.nil?  

    respond_to do |format| 
     begin 
     Tournament.transaction do  
      @tournament.logo.save! unless @tournament.logo.nil? 
      @tournament.save! 
     end 
     flash[:notice] = 'Tournament was successfully created.' 
     format.html { redirect_to tournament_url(@tournament) } 
     format.xml { head :created, :location => tournament_url(@tournament) }   
     rescue 
      flash[:notice] = 'Errors prevented your Tournament from being saved' 
     format.html { render :action => "new" } 
     format.xml { render :xml => @tournament.errors, :status => :unprocessable_entity } 
     end 
    end 
    end 

    # PUT /tournaments/1 
    # PUT /tournaments/1.xml 
    def update 
    @tournament = Tournament.find(params[:id]) 
    @tournament.user_id = session[:orig_user_id] 

    respond_to do |format| 
     begin 
     Tournament.transaction do 
      # Update Logo if necessary 
      unless params[:logo][:upload_data].blank? 
      @tournament.logo.destroy unless @tournament.logo.nil? 
      @tournament.logo = Logo.new({:uploaded_data => params[:logo][:upload_data]}.merge(:user_id => current_user.id)) 
      end 
      # Save any uploaded documents 
      params[:uploads].each do |upload| 
      @tournament.documents << Document.new({:uploaded_data => upload[:document]}.merge(:description => upload[:description])) 
      end unless params[:uploads].nil? 
      # Update Tournamnet Attributes 
      @tournament.attributes = params[:tournament] 
      # Save the Tournament 
      @tournament.save!   
     end 
     flash[:notice] = 'Tournament was successfully updated.' 
     format.html { redirect_to tournament_url(@tournament) } 
     format.xml { head :ok, :location => tournament_url(@tournament) }   
     rescue 
      flash[:notice] = 'Errors prevented your Tournament from being updated' 
     format.html { render :action => "edit" } 
     format.xml { render :xml => @tournament.errors, :status => :unprocessable_entity } 
     end 
    end 
    end 

标识型号:

class Logo < Asset  

    validate_on_create :attachment_valid? 

    has_attachment :content_type => :image, 
    :storage => :file_system, 
    :max_size => 4.megabytes, 
    :resize_to => '810x150>', 
    :processor => :ImageScience, 
    :thumbnails => { :thumb => '270x50>' } 


    def attachment_valid? 
    content_type = attachment_options[:content_type] 
    unless content_type.nil? || content_type.include?(self.content_type) 
     errors.add(:upload_data, " * must be an image file (jpg, gif, or png)") 
    end 
    size = attachment_options[:size] 
    unless size.nil? || size.include?(self.size)    
     errors.add(:upload_data, "* image must be 4MB or less") 
    end 
    end 


    before_thumbnail_saved do |thumbnail| 
    record = thumbnail.parent 
    thumbnail.user_id = record.user_id 
    thumbnail.listing_id = record.listing_id 
    end 

end  

我运行以下

的Rails 2.3.3

image_science 1.2.0

ŧ汉克斯 --Tim

回答

1

你也可以使用:before_save回调测试对象。如果无效,则引发异常。

+0

这似乎是最好的方式去 – triendeau 2009-09-16 13:42:15

+0

通过在我的自定义验证中引发RecordInvalid异常并将徽标的分配移动到tournamnet到事务块,我能够得到这个工作。谢谢您的帮助。 – triendeau 2009-10-04 14:41:55

0

尝试添加:

validate_on_update :attachment_valid? 
+0

这永远不会被称为 – triendeau 2009-09-16 13:41:40