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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

3回答

1733閲覧

ユーザー情報を編集する際にエラーメッセージを表示させたいです。

ikt_erk122

総合スコア40

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2019/08/20 08:51

編集2019/08/22 06:10

<やりたいこと>
アコーディオンの編集フォームで誤ったユーザー情報で更新した際にエラーメッセージを表示させたい

<困っていること>
正しい情報での更新はできるのですが誤った情報だとエラーメッセージは表示されずエラーになる

<質問>
フォームの書き方が悪いのかコントローラが悪いのかわからないのですが参考サイトやアドバイスいただけないでしょうか?

users_contoller.rb

def index if params[:q] && params[:q].reject { |key, value| value.blank? }.present? @q = User.ransack(search_params, activated_true: true) @title = "検索結果" else @q = User.ransack(activated_true: true) @title = "ユーザー一覧" end @users = @q.result.paginate(page: params[:page]) end 〜 def update_basic_info @user = User.find(params[:id]) if @user.update_attributes(user_params) flash[:success] = "基本情報を更新しました。" redirect_to users_path else @users = User.paginate(page: params[:page]) flash[:danger] = "#{@user.name}さんの基本情報の更新に失敗しました。" render 'index' end end

index.html.erb

<% provide(:title, @title) %> <% provide(:button_text, '更新') %> <h1><%= @title %></h1> <div class="row"> <div class="search_form"> <%= render 'users/search_form' %> </div> </div> <%= will_paginate %> <% unless @users.empty? %> <ul class="users"> <li> <% @users.each do |user| %> <%= user.name %> <% if current_user.admin? %><br> <% if !current_user?(user) %> <%= link_to "削除", user, method: :delete, data: { confirm: "削除してよろしいですか?" },class: "btn btn-primary" %><br> <% end %> <div class="userslist"> <a class="btn btn-primary" data-toggle="collapse" href="#collapse<%= user.id %>">編集</a> <div class="collapse" id="collapse<%= user.id %>"> <div class="panel-body"> <%= render "user",user: user %> </div> </div> </div> <% end %> <% end %> </li> </ul> <% else %> <p class="search"> ユーザーが見つかりませんでした。<br> 探しているユーザー名を確認してください。 </p> <% end %> <%= will_paginate %>

_user.html.erb

<%= form_with model: user, url: update_basic_info_path(params: {id: user.id}), local: true, data: {remote: 'true'} do |f| %> <% if user.errors.any? %> <div class="alert alert-danger"> <ul> <% user.errors.full_messages.each do |msg| %> <li><%= msg %></li> <% end %> </ul> </div> <% end %> <%= f.label :name %> <%= f.text_field :name, class: 'form-control' %> <%= f.label :email %> <%= f.email_field :email, class: 'form-control' %> <%= f.label :department %> <%= f.text_field :department, class: 'form-control' %> <%= f.label :uid %> <%= f.text_field :uid, class: 'form-control' %> <%= f.label :employee_number %> <%= f.text_field :employee_number, class: 'form-control' %> <%= f.label :basic_time %> <%= f.time_field :basic_time, class: 'form-control' %> <%= f.label :work_time %> <%= f.time_field :work_time, class: 'form-control' %> <%= f.label :work_end_time %> <%= f.time_field :work_end_time, class: 'form-control' %> <%= f.label :password %> <%= f.password_field :password, class: 'form-control' %> <%= f.label :password_confirmation %> <%= f.password_field :password_confirmation, class: 'form-control' %> <%= f.submit yield(:button_text), class: "btn btn-primary" %> <% end %>

routes.rb

Rails.application.routes.draw do root 'static_pages#home' get '/signup', to: 'users#new' get '/login', to: 'sessions#new' post '/login', to: 'sessions#create' delete '/logout', to: 'sessions#destroy' # get '/edit-basic-info/:id', to: 'users#edit_basic_info', as: :basic_info #patch 'update-basic-info', to: 'users#update_basic_info' get 'users/:id/attendances/:date/edit', to: 'attendances#edit', as: :edit_attendances patch 'users/:id/attendances/:date/update', to: 'attendances#update', as: :update_attendances resources :users do member do patch 'update_basic_info' end resources :attendances, only: :create end end

schema.rb

create_table "users", force: :cascade do |t| t.string "name" t.string "email" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "password_digest" t.boolean "admin", default: false t.string "department" t.datetime "basic_time", default: "2019-02-19 22:30:00" t.datetime "work_time", default: "2019-02-19 23:00:00" t.string "remember_digest" t.datetime "work_end_time", default: "2019-02-20 08:00:00" t.string "employee_number" t.string "uid" t.boolean "superior", default: false t.index ["email"], name: "index_users_on_email", unique: true end

イメージ説明

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

8zca

2019/08/20 13:39

バリデーションをしたいということであっているでしょうか? 例えば email にメールアドレスのフォーマット以外の文字列が入ったときにエラーにしたい等
ikt_erk122

2019/08/21 02:22

はい、そうです。schemaとuserモデルを追加しておきました
guest

回答3

0

自己解決

usersコントローラーのStrongParametersを変えたら解決できました

def update_basic_info @user = User.find(params[:id]) if @user.update_attributes(basic_info_params) flash[:success] = "#{@user.name}さんの基本情報を更新しました。" else @user = User.find(params[:id]) flash[:danger] = "#{@user.name}さんの更新は失敗しました。<br>" + @user.errors.full_messages.join("<br>") end redirect_to users_url end  private  〜    def basic_info_params params.require(:user).permit(:name, :email, :department, :uid, :employee_number, :password, :password_confirmation, :basic_time, :work_time, :work_end_time) end

投稿2019/08/23 02:42

ikt_erk122

総合スコア40

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

Mugheatさんのに追加
form が remote: 'true' になっているので、defaultでは JSでの戻りが期待されてます。それが用意されていない。

それを用意するか remote: 'true' を削除する。
削除すると view update_basic_info がないというエラーになります。
エラーの時にどういう画面にしたいのか、redirect_to users_path がそれ?

投稿2019/08/21 03:13

winterboum

総合スコア23284

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Mugheart

2019/08/21 03:21

それ気になったんですけど local: true と data: {remote: 'true'} が同時に指定されているので、 なんやかんやで local: true が優先されてうまくいってるのかなぁとか思ってました。 そうでもないんですかね?同時に指定することがまずないのでよくわからない...。
Mugheart

2019/08/21 03:23

あとupdate_basic_infoの戻り値はいずれの場合もrenderもしくはredirectなので > view update_basic_info がないというエラー にはならないかなと思います。
ikt_erk122

2019/08/21 03:26

回答ありがとうございます。 remote: 'true'は消しました。 エラー時はユーザー一覧(users_path)画面上部にエラーメッセージを表示させたいです。
winterboum

2019/08/21 03:27

あ、render :index 見落としてました。 自分の習慣的に、rennder類は最後に書いているものだから。
Mugheart

2019/08/21 03:30

> ikt_erk122 さん > エラー時はユーザー一覧(users_path)画面上部にエラーメッセージを表示させたいです。 と言われてもここはあなたのために僕たちがコードを書いてあげる場ではないです。 サンプルコードも用意していますので、自分の力で取り組んでみてください。
guest

0

解決するにあたっていくつか問題があります。

  • エラーメッセージを表示するところがない。

erb

1<% if model.errors.any? %> 2 <div class="alert alert-warning"> 3 <ul> 4 <% model.errors.full_messages.each do |message| %> 5 <li><%= message %></li> 6 <% end %> 7 </ul> 8 </div> 9<% end %>
  • flashrenderする前に用意する必要がある

rb

1render 'index' 2flash[:danger] = "更新に失敗しました。"

rb

1flash.now[:danger] = "更新に失敗しました。" 2render 'index'
  • render :indexした後、indexのviewで使う@usersが用意されていない

rb

1def update_basic_info 2 @user = User.find(params[:id]) 3 if @user.update_attributes(user_params) 4 flash[:success] = "基本情報を更新しました。" 5 redirect_to users_path 6 else 7 # @users = ... 8 # @userのエラー情報を維持したまま@userを含む@usersを用意する必要があるのでやや複雑 9 render 'index' 10 flash[:danger] = "更新に失敗しました。" 11 end 12end

ひとつずつ潰していってみてください。

投稿2019/08/21 02:57

編集2019/08/21 02:59
Mugheart

総合スコア2340

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問