質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.61%

ユーザーの更新で、なぜか@userがinvalidになる

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 112

score 11

前提・実現したいこと

Railsでwebアプリを作っています。

detail_user_pathで、Userモデルにおけるユーザーの詳細情報に関わるカラムをnil => (内容)に更新する機能を作りたいのですが、ソースコードで実行してもなぜか@userがinvalidになってしまいます。(password cant be blankのエラーメッセージがコンソールに出ました)
update_attributeも使っていますし、user_update_paramsも用意したのですが、何がダメなのか全くわからないです。。。

私のコードの修正すべき点を教えていただきたいです。

程度の低い質問でしたら申し訳ございません。
また、質問に必要な情報等に不足があれば、お手数ですがご指摘いただければ、追記いたします。

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

該当のソースコード

(コントローラ)
class UsersController < ApplicationController
  #before_action :logged_in_user, only: [:edit, :update, :destroy, :following, :followers]
  before_action :correct_user, only: [:edit, :update, :destroy]

  def show
    @user = User.find(params[:id])
    #@microposts = @user.microposts.paginate(page: params[:page])
  end

  def new
    @user = User.new
  end

  def signup_jp
    @user = User.new
  end

  def after_signup

  end

  def create
    @user = User.new(user_params)
    if @user.save
      @user.send_activation_email
      log_in @user
      flash[:info] = "Please check your email to activate your account."
      # redirect_to after_signup_path
      #redirect_to new_detail_path
      # TODO: ここでユーザー認証を促す画面にリダイレクトさせるようにあとで設定する
      # redirect_to root_url
      redirect_to root_path
    else
      render 'new'
    end
  end

  def detail
    @user = User.find(params[:id])
  end

  def detailcreate
    @user = User.find_by(id: params[:id])
    #binding.pry
    if @user.update_attribute(user_update_params)
      redirect_to @user
    else
      flash[:danger] = "Invalid information is included."
      render 'detail'
    end
  end



  def edit
    @user = User.find(params[:id])
    @detail = Detail.find_by(user_id: @user.id )
  end

  def update
    @user = User.find_by(id: params[:id])
    #binding.pry
    if @user.update_attribute(user_update_params)
      redirect_to @user
    else
      flash[:danger] = "Invalid information is included."
      render 'edit'
    end
  end


  # def edit_email edit_emailcontrollerに移行した
  #   @user = User.find(params[:id])
  # end


  def edit_password
    @user = User.find(params[:id])
  end




  private

    def user_params #strong parameterのネスト化実装
      params.require(:user).permit(
        :name, 
        :email, 
        :image, 
        :password, 
        :password_confirmation,
        :mother_tongue,
        :japanese_level,
        :english_level,
        :gender,
        :region,
        :purpose,
        :self_introduction,
        :skype,
        :discord,
        :other#,
        # :authenticity_token
          )
    end
    # .merge(user_id: User.find_by(id: params[:id]).id) #これはつけた方がいいですか?

    def user_update_params 
      params.permit(
        :mother_tongue,
        :japanese_level,
        :english_level,
        :gender,
        :region,
        :purpose,
        :self_introduction,
        :skype,
        :discord,
        :other
          )
    end



    def correct_user
      @user = User.find(params[:id])
      redirect_to(root_url) unless current_user?(@user)
    end

end
<% provide(:title, "user detail") %>

<div class="user-detail-wrapper">
  <div class="introduction">
    <h3>①Authorize your account via sent Email.</h3>
    <h3>②Let's register your user informations! You can change it later.</h3>
  </div>

  <div class="row user-detail-forms">
    <div class="col-md-6 col-md-offset-3">
      <%= form_with(url: detailcreate_user_path, method: "patch", local: true) do |f| %>


        <%= f.label :mother_tongue, "Mother Tongue" %>
        <%= f.select :mother_tongue, [["English", "English"], ["Japanese", "Japanese"], 
        ["Other","other"]], include_blank: "Please select.", class:"form-control" %>

        <%= f.label :japanese_level, "Japanese Level" %>
        <%= f.select :japanese_level, [["Beginner", "Beginner"], ["Capable of quite basic communication", "Capable of quite basic communication"], 
        ["Capable of daily conversation | 日常会話ができる","Capable of daily conversation | 日常会話ができる"], ["Capable of advanced conversation | 難しい内容の会話ができる","Capable of advanced conversation | 難しい内容の会話ができる"], ["Almost native level | ネイティブとほとんど同レベル","Almost native level | ネイティブとほとんど同レベル"],
        ["Native | ネイティブ","Native | ネイティブ"]], include_blank: "Please select.", class:"form-control" %>

        <%= f.label :english_level, "English Level | 英語のレベル" %>
        <%= f.select :english_level, [["Beginner | 初心者", "Beginner | 初心者"], ["Capable of quite basic communication | 簡単な会話ならできる", "Capable of quite basic communication | 簡単な会話ならできる"], 
        ["Capable of daily conversation | 日常会話ができる","Capable of daily conversation | 日常会話ができる"], ["Capable of advanced conversation | 難しい内容の会話ができる","Capable of advanced conversation | 難しい内容の会話ができる"], ["Almost native level | ネイティブとほとんど同レベル","Almost native level | ネイティブとほとんど同レベル"],
        ["Native | ネイティブ","Native | ネイティブ"]], include_blank: "Please select.", class:"form-control" %>



        <%= f.label :self_introduction, "Self Introduction | 自己紹介" %>
        <%= f.text_area :self_introduction, class: 'form-control self-intro', placeholder: "It is recommended that you write in simple English. 英語で書くことをお勧めします。簡単な文でも問題ありません。" %>

        <%= f.label :skype, "URL of Your SNS 1 (optional) | SNSリンク1(任意)" %>
        <%= f.text_field :skype, class: 'form-control' %>

        <%= f.label :discord, "URL of Your SNS 2 (optional) | SNSリンク2(任意)" %>
        <%= f.text_field :discord, class: 'form-control' %>

        <%= f.label :other, "URL of Your SNS 3 (optional) | SNSリンク3(任意)" %>
        <%= f.text_field :other, class: 'form-control' %>

        <%= f.text_field :name, type: "hidden", value: "#{@user.name}" %>
        <%= f.text_field :email, type: "hidden", value: "#{@user.email}" %>




        <%= f.submit "Send | 送信", class: "btn btn-primary create" %>
      <% end %>
    </div>
  </div>
</div> 
(user.rb)
class User < ApplicationRecord

  attr_accessor :remember_token, :activation_token, :reset_token
  before_save   :downcase_email
  before_create :create_activation_digest
  validates :name, presence: true, length: { maximum: 25 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 }, 
                    format: { with: VALID_EMAIL_REGEX }, uniqueness: true
  has_secure_password
  validates :password, presence: true, length: { minimum: 6 } 
  mount_uploader :image, ImageUploader

  # 渡された文字列のハッシュ値を返す
  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  def User.new_token
    SecureRandom.urlsafe_base64
  end

  # 永続セッションのためにユーザーをデータベースに記憶する
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  # 渡されたトークンがダイジェストと一致したらtrueを返す
  def authenticated?(attribute, token)
    digest = send("#{attribute}_digest")
    return false if digest.nil?
    BCrypt::Password.new(digest).is_password?(token)
  end

  def forget
    update_attribute(:remember_digest, nil)
  end

  # アカウントを有効にする
  def activate
    update_attribute(:activated,    true)
    update_attribute(:activated_at, Time.zone.now)
  end

  # 有効化用のメールを送信する
  def send_activation_email
    UserMailer.account_activation(self).deliver_now
  end

  # パスワード再設定の属性を設定する
  def create_reset_digest
    self.reset_token = User.new_token
    update_attribute(:reset_digest,  User.digest(reset_token))
    update_attribute(:reset_sent_at, Time.zone.now)
  end

  # パスワード再設定のメールを送信する
  def send_password_reset_email
    UserMailer.password_reset(self).deliver_now
  end

  def password_reset_expired?
    reset_sent_at < 2.hours.ago
  end

  private

    def downcase_email
      self.email = email.downcase
    end

    def create_activation_digest
      self.activation_token  = User.new_token
      self.activation_digest = User.digest(activation_token)
    end


end
(routes.rb)
Rails.application.routes.draw do
  get 'password_resets/new'
  get 'password_resets/edit'
  get 'sessions/new'
  root 'static_pages#index'
  resources :static_pages, only: [:index]
  resources :users do
    member do
      get 'detail'
      patch 'detailcreate'
    end
  end
  resources :sessions, only: [:index, :create, :destroy]
  resources :account_activations, only: [:edit]
  resources :password_resets,     only: [:new, :create, :edit, :update]
end

補足情報(FW/ツールのバージョンなど)

Rails 6.0.3
Ruby 2.6.3

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • winterboum

    2021/07/25 08:32

    エラーメッセージを載せてください

    キャンセル

回答 1

checkベストアンサー

+1

ああ、
validates :password, presence: true,
ですね。
create のときのみ 'on: create`  とかしてみて。

ただ、password変更のときに、長さのチェックは必要でしょうから、
presence と length は別のVALIDATIONにして

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る