From 6ef32e4bea82d202e78aa08cd587d1e094154c88 Mon Sep 17 00:00:00 2001 From: Christoph Haas Date: Fri, 26 Jun 2015 21:18:56 +0200 Subject: [PATCH] Screenshot/Paperclip migration Rake task checks for validation errors --- lib/tasks/screenshots_to_paperclip.rake | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/lib/tasks/screenshots_to_paperclip.rake b/lib/tasks/screenshots_to_paperclip.rake index 282b57b..d0f0870 100644 --- a/lib/tasks/screenshots_to_paperclip.rake +++ b/lib/tasks/screenshots_to_paperclip.rake @@ -3,8 +3,8 @@ namespace :debshots do task :screenshots_to_paperclip => :environment do Rails.logger = Logger.new(STDOUT) - Rails.logger.level = Logger::INFO - #Rails.logger.level = Logger::DEBUG + # Rails.logger.level = Logger::INFO + Rails.logger.level = Logger::DEBUG base_path = Rails.root.join('public') @@ -21,13 +21,18 @@ namespace :debshots do # Has the screenshot been migrated already? if screenshot.image.exists? - Rails.logger.error "Screenshot already migrated. Skipping." + Rails.logger.info "Screenshot already migrated. Skipping." next end File.open(path, 'rb') do |image_file| screenshot.image = image_file + Rails.logger.debug "Adding image file: #{image_file}" + unless screenshot.valid? + Rails.logger.error "Adding image file failed. Errors: #{screenshot.errors[:image]}" + end screenshot.save + Rails.logger.debug "Screenshot object saved" end end end