несколько прикреплений скрепки с процессором водяного знака

Question

У меня куча проблем с загрузкой нескольких вложений с помощью скрепки и обработкой их водяным знаком.

У меня 2 модели, реклама и фото. Объявление has_many: фотографии и фото принадлежит_ к: объявление.

Чтобы быть более точным в /models/ad.rb, у меня есть:

class Ad < ActiveRecord::Base

  has_many :photos, :dependent => :destroy
  accepts_nested_attributes_for :photos, :allow_destroy => true  
end

и мой файл photo.rb выглядит так:

class Photo < ActiveRecord::Base

belongs_to :ad

has_attached_file :data,
:styles => {
  :thumb => "100x100#",
  :first => {
    :processors => [:watermark],
    :geometry => '300x250#',
    :watermark_path => ':rails_root/public/images/watermark.png',
    :position => 'SouthEast' },
  :large => {
    :processors => [:watermark],
    :geometry => '640x480#',
    :watermark_path => ':rails_root/public/images/watermark.png',
    :position => 'SouthEast' }
}
end

На мой взгляд, я использую это для добавления полей файла

<% f.fields_for :photos do |p| %>

  <%= p.label :data, 'Poza:' %> <%= p.file_field :data %>

<% end %>

В моем контроллере в действии редактирования я использую 4.times {@ad.photos.build} для генерации полей файла.

Все работает отлично, если я не использую процессор водяных знаков, если я использую обычное объявление has_attached_file, например:

has_attached_file :data,
:styles => {
  :thumb => "100x100#",
  :first => '300x250#',
  :large => '640x480#'
}

Но когда я использую процессор водяных знаков, я всегда получаю эту ошибку:

 NoMethodError in PublicController#update_ad

 You have a nil object when you didn't expect it!
 You might have expected an instance of ActiveRecord::Base.
 The error occurred while evaluating nil.[]
  ..............................
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/nested_attributes.rb:350:in `assign_nested_attributes_for_collection_association'
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/nested_attributes.rb:345:in `each'
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/nested_attributes.rb:345:in `assign_nested_attributes_for_collection_association'
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/nested_attributes.rb:243:in `photos_attributes='
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/base.rb:2746:in `send'
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/base.rb:2746:in `attributes='
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/base.rb:2742:in `each'
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/base.rb:2742:in `attributes='
 /usr/lib/ruby/gems/1.8/gems/activerecord-2.3.5/lib/active_record/base.rb:2628:in `update_attributes'
 /home/alexg/Sites/vandmasina/app/controllers/public_controller.rb:217
 /home/alexg/Sites/vandmasina/app/controllers/public_controller.rb:216:in `update_ad'

С параметрами все в порядке, насколько я могу сказать

 Parameters:

 {"commit"=>"Salveaza modificarile",
  "ad"=>{"price"=>"6000",
  "oras"=>"9",
  "photos_attributes"=>{"0"=>{"data"=>#<File:/tmp/RackMultipart20100928-5130-b42noe-0>},
  "1"=>{"data"=>#<File:/tmp/RackMultipart20100928-5130-r0ukcr-0>},
  "2"=>{"data"=>#<File:/tmp/RackMultipart20100928-5130-mb23ei-0>},
  "3"=>{"data"=>#<File:/tmp/RackMultipart20100928-5130-1bpkm3b-0>}},

Процессор Watermark /lib/paperclip_processors/watermark.rb выглядит следующим образом:

  module Paperclip
class Watermark < Processor

class InstanceNotGiven < ArgumentError; 
end

def initialize(file, options = {},attachment = nil)
  super
  @file = file
  @current_format = File.extname(@file.path)
  @basename = File.basename(@file.path, @current_format)
  @watermark = ':rails_root/public/images/watermark.png'
  @current_geometry = Geometry.from_file file # This is pretty slow
  @watermark_geometry = watermark_dimensions
end

def watermark_dimensions
  return @watermark_dimensions if @watermark_dimensions
  @watermark_dimensions = Geometry.from_file @watermark
end

def make
  dst = Tempfile.new([@basename, @format].compact.join("."))
  watermark = " \\( #{@watermark} -extract #{@current_geometry.width.to_i}x#{@current_geometry.height.to_i}+#{@watermark_geometry.height.to_i /
                2}+#{@watermark_geometry.width.to_i / 2} \\) "
  command = "-gravity center " + watermark + File.expand_path(@file.path) + " " +File.expand_path(dst.path)

  begin
    success = Paperclip.run("composite", command.gsub(/\s+/, " "))
  rescue PaperclipCommandLineError
    raise PaperclipError, "There was an error processing the watermark for #{@basename}" if @whiny_thumbnails
  end
  dst
end

end
end

Я попробовал процессор в обычном приложении, без нескольких вложений, и он отлично работает. Насколько я могу судить, он не работает с nested_attributes.

Приложение - рельсы 2.3.5 с ruby ​​1.8.7 и скрепкой 2.3.11

Если вы можете оказать какую-либо помощь, она будет очень признательна, так как я пытаюсь понять это уже 2 дня:)

Answer 1

Если вы используете направляющие 3 и скрепку> 2.3.3, попробуйте https://gist.github.com/843418 source.

Answer 2

О, чувак, это было непросто!

В вашем коде несколько ошибок, и ни одна из них не связана с вложенными моделями.Я объясняю это для Rails 3 и Paperclip 2.3.3

a) вещь: rails_root не работает.Эта интерполяция используется только в URL / пути, а не в пользовательских параметрах.Поэтому вы должны заменить его на Rails.root.join ("public / images ...")

b) вы просто игнорируете параметр: watermark_path и используете только жестко закодированный путь (в методе инициализации).Так что не имеет значения, что у вас есть в ваших: стилях, как это всегда делается для ... / images / watermark.png.Здесь снова: rails_root, поэтому он не может работать.

c) когда вы передаете параметр в Paperclip.run («Composite», «Foo Bar There»), он фактически выполняет эту команду:

composite 'foo bar there'

вы видите одинарные кавычки?Из-за этого составная команда видит ваши параметры как один огромный параметр и не понимает его вообще.Если вы передадите его как массив, то каждый элемент будет заключен в кавычки, а не массив в целом.

Итак, вот улучшенная версия процессора водяных знаков:

module Paperclip
  class Watermark < Processor

  class InstanceNotGiven < ArgumentError;
  end

  def initialize(file, options = {},attachment = nil)
    super
    @file = file
    @current_format = File.extname(@file.path)
    @basename = File.basename(@file.path, @current_format)
    # PAWIEN: use default value only if option is not specified
    @watermark = options[:watermark_path] || Rails.root.join('public/images/watermark.png')
    @current_geometry = Geometry.from_file file # This is pretty slow
    @watermark_geometry = watermark_dimensions
  end

  def watermark_dimensions
    return @watermark_dimensions if @watermark_dimensions
    @watermark_dimensions = Geometry.from_file @watermark
  end

  def make
    dst = Tempfile.new([@basename, @format].compact.join("."))
    dst.binmode

    begin
      # PAWIEN: change original "stringy" approach to arrayish approach
      # inspired by the thumbnail processor
      options = [
        "-gravity",
        "center",
        "#{@watermark}",
        "-extract",
        "#{@current_geometry.width.to_i}x#{@current_geometry.height.to_i}+#{@watermark_geometry.height.to_i / 2}+#{@watermark_geometry.width.to_i / 2}",
        File.expand_path(@file.path),
        File.expand_path(dst.path)
      ].flatten.compact.join(" ").strip.squeeze(" ")

      success = Paperclip.run("composite", options)
    rescue PaperclipCommandLineError
      raise PaperclipError, "There was an error processing the watermark for #{@basename}" if @whiny_thumbnails
    end
    dst
  end

  end
end

Надеждаэто помогло вам!

ОБНОВЛЕНИЕ : вам нужно использовать последнюю версию скрепки из github

gem 'paperclip', '>= 2.3.3', :git => "http://github.com/thoughtbot/paperclip.git"

В этой версии форматы #run были снова изменены, поэтому яобновил кодЭто действительно должно работать, как я создал тестовое приложение, и оно делает то, что предполагалось.

ОБНОВЛЕНИЕ 2 : Репо с рабочим примером:

git://repo.or.cz/paperclip-mass-example.git

Answer 3

На первый взгляд кажется, что watermark_path должен быть "# {Rails.root} / ...", хотя похоже, что у вас здесь много чего происходит.

Кроме того, я несмотрите вашу форму как в form_for.Убедитесь, что у вас есть {: multipart => true}