validationがかかりません。

Question

Ruby初心者です。
現在、インスタグラムのようなアプリを作成しており、
validationが上手く通らなく困っております。

インスタグラムのように、投稿するため、画像に対してvailidationをかけたいです。そこで以下のように実装したのですが、

photo.rb

class Photo < ApplicationRecord
    belongs_to :post
    default_scope -> {order(created_at: :desc)}  
    validates :image, presence: true
    mount_uploader :image, PhotoUploader

画像が空のままでも通ってしまいます。

ブラウザで投稿ボタンを押すと、以下の　createが機能します。

 def new
   @post = Post.new
 end

def create
        @post = Post.new(post_params)
     if @post.save       
       if params[:images]
            params[:images].each do |img|
             @post.photos.create(image: img)
            end
       end
          flash[:notice] = "投稿が保存されました"
          redirect_to posts_path
       else
          redirect_to posts_path
          flash[:alert] = "投稿に失敗しました"
       end
 end

些細なことでも、よろしいので回答して頂けると嬉しいです。

よろしくお願い致します。

Comments

[maisumakun]

ファイルアップロードに使っているライブラリは何でしょうか。

[ko-u]

carrierwaveを使用しています！

Answer 1

def create
@post = Post.new(post_params)
if params[:images]
if @post.save

        params[:images].each do |img|
          @post.photos.create(image: img)
        end
      else
      
      end
      redirect_to posts_path
      flash[:notice] = "投稿が保存されました"
    else
      redirect_to posts_path
      flash[:alert] = "投稿に失敗しました"
    end
  end

に変更したところ、エラーが消えました。

回答していただき、ありがとうございました。

Answer 2

carrierwaveのGitHubについたWikiを見てみると、ファイル列のバリデーションは、mount_uploaderよりあとでかけてあります。

# サンプルコードより引用
mount_uploader :avatar, AvatarUploader

# validates :avatar, presence: trueの古い書き方
validates_presence_of :avatar

Rubyではクラス宣言内もコードの順番があるので、もしかするとその影響かもしれません。

Comments

[ko-u]

ご回答ありがとうございます。
以上のように変更をしたのですが、validationをかけることが出来ませんでした。