(Sinatra) Как загружать несколько изображений в CarrierWave?

Question

[Андрей]

Пробовал создавать еще одну модель (Attachment), но не сработало.
Думаю, в контроллере нужно что-то типа этого

post '/posts' do
  @post = Post.new(
    :image        => params[:image],
  )
  @post.image = params[:image]
  #binding.pry
  if @post.save!
    redirect :"/"
  else
    redirect :"/new"
  end
end

или в форме перебрать в name=""

Моя view

<form action="/posts" method="post" role="form" enctype="multipart/form-data">
    <input type='file' name='image' class='form-control'>
    <input type='file' name='image' class='form-control'>
    <input type='file' name='image' class='form-control'>
    <button type="submit" class="btn btn-success">Submit</button>
</form>

Моя миграция

class CreatePosts < ActiveRecord::Migration[5.0]
  def change
    create_table :posts do |t|
      t.string :image

      t.timestamps null: false 
    end
  end
end

Моя модель

class Post < ActiveRecord::Base
  mount_uploader :image, ImagesUploader
end

Мой action

post '/posts' do
  @post = Post.new(params[:post])
  @post.image = params[:image]
  #binding.pry
  if @post.save!
    redirect :"/"
  else
    redirect :"/new"
  end
end

Comments

[Andrey Demidenko]

Как вы хотите сохранять одно изображение в одно поле?
Как сохранить цифру 5 и 10 в переменную типа integer?

Answer 1

[Павел Грудинкин]

Без дополнительной таблицы обойтись может быть и можно, но решение будет слишком сложным. Поэтому нужно сделать таблицу post_attachments, в миграции связать с постами и естественно нужно поле image, должно получиться что-то вроде:

Сlass CreatePostAttachment < ActiveRecord::Migration
  def change
    create_table :post_attachments do |t|
      t.references :post, index: true
      t.string :image
      t.timestamps null: false
    end
    add_foreign_key :post_attachments, :posts
  end
end

Раз уж вы пользуетесь чистым html, то вьюха должна получиться примерно такой, могу ошибиться, привык к haml:

<label for="post_image">Выберите одну  или несколько фотографий</label>
<div class="field">
<input multiple="multiple" name="post_attachments[image][]" type="file" id="post_attachments_image">
</div>

Контроллер:

if @post.save
  unless params[:post_attachments].blank?
          params[:post_attachments]['image'].each do |a|
            @post.post_attachments.create!(:image => a)
          end
  redirect :"/"
  ....

@post.image = params[:image] и соответствующее поле соответственно нужно убрать из постов.

В моделях:

class PostAttachment < ActiveRecord::Base
  mount_uploader :image, ImagesUploader
  belongs_to :post
end

class Post < ActiveRecord::Base
  has_many :post_attachments, :dependent => :destroy
end

Comments

[Павел Грудинкин]

могут быть различия в контроллере, но тут уж вы сами подгоняйте решение из рельсы к синатре

[Андрей]

Спасибо, разбираюсь) но пока ошибку выдает uninitialized constant Post::Attachment:

[Павел Грудинкин]

Андрей: В каком конкретно месте ошибка?

[Андрей]

на строчке @ post.post_attachments.create!(:image => a) в контроллере

[Павел Грудинкин]

Андрей: Модели обновили? Без этого работать не будет.

[Андрей]

Вот такие сейчас модели

class Attachment < ActiveRecord::Base
  belongs_to :post
  mount_uploader :image, ImagesUploader
end

class Post < ActiveRecord::Base
  has_many :attachments, :dependent => :destroy
end

[Андрей]

Контроллер

# Сохранение публикации
post '/posts' do
  @post = Post.new(params[:post])
  #@post.image = params[:image]

  if @post.save
    unless params[:attachments].blank?
      binding.pry
      params[:attachments]['image'].each do |a|
        @post.attachments.create!(:image => a)
      end
    end

    redirect :"/"
  else
    redirect :"posts//new"
  end
end

[Павел Грудинкин]

Андрей: Со связями разберемся попозже, заменяем тогда эту строку на: "Attachment.create!(:image => a, post_id: post.id)"

[Павел Грудинкин]

@ post.id

[Андрей]

Контроллер проходит и удачно редеректит на главную, но там ошибка undefined method image for Post:0xbd49008

[Андрей]

<% @posts.each do |work| %>
<img src="<%= work.image.url %>" class="img-responsive" alt=""> </a>

[Павел Грудинкин]

Андрей: Вьюху то обновите, в посте image то теперь нет

[Павел Грудинкин]

<% @posts.each do |work| %>
  <% Attachment.where(post_id: work.id) do |at| %>
    <img src="<%= at.image.url %>" class="img-responsive" alt=""> </a>

В продакшене так делать нельзя, нужно разобраться со связями.

[Андрей]

да, понимаю) но так тоже не работает

<% @posts.attachments.each do |work| %>
<img src="<%= work.image.url %>" class="img-responsive" alt=""> </a>

[Павел Грудинкин]

Андрей: Как вы в данном случае получаете переменную @ posts?

[Андрей]

Спасибо, более-менее разобрался. Всё заработало..