2009-08-18 79 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