railsでformを送信したいところに送信できない

Question

前提・実現したいこと

deviseで実装した、ユーザーの編集フォームでユーザーを編集したい。

edit.html.erbで入力したnameの値をusersテーブルのnameカラムに保存したい。

発生している問題・エラーメッセージ

deviseにてログイン機能を実装したのちに、ユーザー詳細ページを実装、そこからユーザーの名前、メールアドレス、パスワードを変更できるフォームを作成（viewファイルは元々gemをインストールした際にあら鍵目あったものを使用）しました。

そのフォームでは、email、passwordは変更ができるのですが、nameのみ変更できませんでした。

そこで、rails consoleにてUser.allで調べてみると、nameがnilになっていました。

データベースのテーブルを間違えているかもしれないです。

該当のソースコード

以下にschema.rbのコードを表示します。
今回、フォームを送信したら、usersテーブルのnameに反映されるようにしたかったのですが、できませんでした。
userを新規登録する際にnameのフォームに入力された値はprofileのnameに保存されているようで、erbファイルでcurrent_user.profile.nameとすると表示されます。しかし、current_user.nameではもちろんnilですので表示されません。

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# Note that this schema.rb definition is the authoritative source for your
# database schema. If you need to create the application database on another
# system, you should be using db:schema:load, not running all the migrations
# from scratch. The latter is a flawed and unsustainable approach (the more migrations
# you'll amass, the slower it'll run and the greater likelihood for issues).
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2020_02_27_160636) do

  create_table "comments", force: :cascade do |t|
    t.integer "post_id"
    t.string "name"
    t.text "comment", null: false
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["post_id"], name: "index_comments_on_post_id"
  end

  create_table "posts", force: :cascade do |t|
    t.string "name"
    t.string "title"
    t.text "body"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "user_id"
    t.string "image_id"
    t.index ["user_id"], name: "index_posts_on_user_id"
  end

  create_table "profiles", force: :cascade do |t|
    t.integer "user_id"
    t.string "name", default: "", null: false
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["user_id"], name: "index_profiles_on_user_id"
  end

  create_table "users", force: :cascade do |t|
    t.string "email", default: "", null: false
    t.string "encrypted_password", default: "", null: false
    t.string "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "name"
    t.text "profile"
    t.index ["email"], name: "index_users_on_email", unique: true
    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
  end

end

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  has_many :posts, dependent: :destroy

  has_one :profile, dependent: :destroy

  include Gravtastic
  gravtastic

  validates :name, presence: true, length: { maximum: 20 } 
  validates :profile, length: { maximum: 200 } 
  validates :password, presence: true, length: { minimum: 6 }, allow_nil: true
end

class Profile < ApplicationRecord
  belongs_to :user
end

class UsersController < ApplicationController
  
  def show
    @user = User.find(params[:id])
  end

#   def new
#   end

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

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

  if @user.update(user_params)
    flash[:success] = "プロフィールを更新しました"
    redirect_to user_path
  else
    render user_edit_path
  end
end

  private

    def user_params
      params.require(:user).permit(:name, :email, :password,:password_confirmation)
    end
end

class Users::RegistrationsController < Devise::RegistrationsController
  # before_action :configure_sign_up_params, only: [:create]
  # before_action :configure_account_update_params, only: [:update]

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

  def new
  end

  def create
    super
    #ユーザーが無事に作られていたらprofileを作成
    if resource
      #resource==user
      profile = Profile.new
      profile.user_id = resource.id
      profile.name = params[:profile][:name]
      profile.save
    end 
  end

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

Rails.application.routes.draw do

 root 'posts#index'

 devise_for :users, :controllers => {
  :registrations => 'users/registrations',
  :sessions => 'users/sessions'
 }

 devise_scope :user do
  get 'users/:id/edit' => 'users/registrations#edit', as: :user_edit
 end

 resources :posts
 resources :comments, only: %i[create destroy]
 resources :testsessions, only: :create
 resources :users, only: %i[show update edit]
end

データベースがぐちゃぐちゃなのは承知です。

少し長くなりましたが、回答お待ちしております。

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

Comments

[winterboum]

コントローラーのcodeを載せてください

[kobaryo04ysh]

コードを追記しました。
回答、お待ちしております！！

Answer 1

devise のコントローラーは、セキュリティの都合上、デフォルトだと email や password など一部の情報しか受け取らないようになっています。
そのため、「name は安全なやつだよ」ということを devise に教えてあげなければいけません。

具体的には、 ApplicationController に以下のコードを追加すればOKです。

class ApplicationController < ActionController::Base
  before_action :configure_permitted_parameters, if: :devise_controller?

  protected

  def configure_permitted_parameters
    # 「登録時（sign_up）」に許可するパラメータを追加
    devise_parameter_sanitizer.permit(:sign_up, keys: [:name])

    # 「更新時（account_update）」に許可するパラメータを追加
    devise_parameter_sanitizer.permit(:account_update, keys: [:name])
  end
end

--

ちなみに、上記の回答は github の README を参考に書いています。
これ以外にも色々役立つ情報があるので、一度目を通すと今後の開発が捗るかと思います????

Strong Parameters - heartcombo/devise | github

Comments

[kobaryo04ysh]

無事に解決できました！ありがとうございます！！日本語の情報のみ追いかけていて、解決が出来なかったので、これからこのような公式のドキュメント含め、どんどん英語の情報にも触れるようにします！