前提・実現したいこと
プログラミング初心者です。
タスク管理アプリケーションの作成においてユーザが登録し、自身の登録したタスクのみにアクセスし新たなタスクの登録・削除・編集、自身のタスクのみを閲覧できるようにしたいと思っています。
ユーザが登録しサインイン、サインアウトし、タスクリスト画面において新規タスクの登録・削除・編集できる状態にはなっているのですが、登録されてるユーザ全てにおいて同じタスクを共有しており、一人のユーザが新しくタスクを登録すれば別のユーザが登録した際も同じくタスクリストにて自分のではないタスクが表示される状態となっており、一対多が実装出来ておりません。
「多」側に当たる方にbelongs_toのコードを記載しました。
class Task < ApplicationRecord belongs_to :user validates :status, presence: true, length: { maximum: 10} validates :content, presence: true, length: { maximum: 255 } end
class Tasklist < ApplicationRecord belongs_to :user validates :task, presence: true, length: {maximum: 255 } end
また、「1」側にhas_manyを次のように記載しているのですが…
class User < ApplicationRecord before_save { self.email.downcase! } validates :name, presence: true, length: { maximum: 50 } validates :email, presence: true, length: { maximum: 255 }, format: { with: /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i }, uniqueness: { case_sensitive: false } has_secure_password has_many :tasks end
この状態ではユーザに管理をさせたいstatus及びcontentをユーザと紐づけることが出来ずすべてのユーザで同じタスクリストを管理することになってしまっています。
status及びcontentをユーザに紐づけて一対多の関係を成り立たせるにはどうすればよいのでしょうか?
未熟な質問で申し訳ございませんがご教授頂けると助かります。
宜しくお願い致します。
頂いたご質問への追加です。
1)Tasklistは間違って作成してしまったモデルファイルかもしれません。
存在しているタスクモデルに追加でユーザとの一対多の関係を保持するカラムを用意する際に間違ってマイグレーション**rails g migration Add-To-**を行ってしまい、その際にモデルファイルの作成もも行ったとは思うのですが、はっきりどのファイルを間違って作成したのかは覚えておらず、rails migrate:resetは実行しているのと、覚えている限りの作成したファイルは削除しています。
2)tasks_controller.rb
class TasksController < ApplicationController before_action :require_user_logged_in before_action :correct_user, only: [:destroy] def index @tasklists = Task.order(id: :desc).page(params[:page]).per(3) end def show end def new @task = Task.new end def create @task = current_user.tasks.build(task_params) if @task.save flash[:success] = "タスクが正常に登録されました" redirect_to root_url else @tasks = current_user.tasks.order(id: :desc).page(params[:page]) flash.now[:danger] = "タスクが登録されませんでした" render :new end end def edit end def destroy @task.destroy flash[:success] = 'タスクを削除しました。' redirect_back(fallback_location: root_path) end private def set_task @task = Task.find(params[:id]) end def task_params params.require(:task).permit(:content, :status) end def correct_user @task = current_user.tasks.find_by(id: params[:id]) unless @task redirect_to root_url end end end
併せてマイグレーションファイル作成後に変更、作成したファイルのコードも載せさせてもらいます。
config/routes.rb
Rails.application.routes.draw do # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html root to: 'tasks#index' get 'login', to: 'sessions#new' post 'login', to: 'sessions#create' delete 'logout', to: 'sessions#destroy' resources :tasks get 'signup', to: 'users#new' resources :users, only: [:index, :show, :new, :create] resources :tasks, only: [:create, :destroy] end
tasks/_tasks.html.erb
<ul class="list-unstyled"> <% tasks.each do |task| %> <li class="media mb-3"> <img class="mr-2 rounded" src=" <%= gravater_url(task.user, { size: 50 }) %> " alt=""> <div class="media-body"> <div> <%= link_to task.user.name, user_path(task.user) %> <span class="text-muted">posted at <%= task.created_at %> </span> </div> <div> <p class="mb=0"> <%= task.content %> </p> </div> <div> <%= if current_user == task.user %> <%= link_to "Delete", task, method: :delete, data: { confirm: "You sure?" }, class: "btn btn-danger btn-sm" %> <% end %> </div> </div> </li> <% end %> <%= paginate tasks %> </ul>
toppages/index.html.erb
<% if logged_in? %> <div class="row"> <aside class="col-sm-4"> <%= form_with(model: @tasklist, local: true) do |f| %> <div class="form-group"> <%= f.text_area :content, class: 'form-control', rows: 5 %> </div> <%= f.submit 'Post', class: 'btn btn-primary btn-block' %> <% end %> </aside> <div class="col-sm-8"> <%= render 'tasklists/tasklists', tasklists: @tasklists %> </div> </div> <div class="center jumbotron"> <div class="text-center"> <hi>Welcome to the Tasklists</hi> <%= link_to 'Sign up now!', signup_path, class: 'btn btn-lg btn-primary' %> </div> </div> <% end %>
宜しくお願い致します。
補足情報(FW/ツールのバージョンなど)
Ruby 5.2.2
Rails 5.2.4.3
回答1件
あなたの回答
tips
プレビュー