データの登録・編集・削除機能

プログラミング初心者で、現在タスクの管理アプリケーションを作成しております。

ユーザの登録、ログイン、ログアウトは現在可能なのですが、タスクの管理アプリで行えるステータス及びタスクの登録を行うことが出来ません。

タスク一覧画面から「新規タスク登録」ボタンを押し、タスクの新規作成ページに遷移し、そちらでステータスとタスク内容をフォームに入力後に「登録」ボタンを押すと何も登録されていない状態のタスク一覧へと戻ってしまう状態です。

元々は出来ていたので、gitで出来ていた状態の時のコードと比較検証したのですが、確実に不要なものを消去している以外は出来ていた時と変わらないコードに戻したつもりにも関わらず現在も出来ず、テキストを参照してもわからない為、こちらにて助言いただければと思い投稿させてもらいます。

現在cloud9上でタスク管理のアプリのために作成したフォルダ内の関連するコード一覧を下記に載せさせて頂きます。

宜しくお願い致します。

rails のバージョンは5.2.2です。

models/users.rb

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

models/tasks.rb

class Task < ApplicationRecord
  belongs_to :user
  
  validates :status, presence: true, length: { maximum: 10}
  validates :content, presence: true, length: { maximum: 255 }
end

application_controller.rb

class ApplicationController < ActionController::Base
  
  include SessionsHelper
  
  private
  
  def require_user_logged_in
    unless logged_in?
    redirect_to login_url
    end
  end
end

tasks_controller.rb

class TasksController < ApplicationController
  before_action :require_user_logged_in
  before_action :correct_user, only: [:create, :destroy]
  
  def index
      @tasklists = current_user.tasks
  end

  def show
    task_params
  end
  
  def new
    @task = current_user.tasks.new
  end
  
  def create
    @task = current_user.tasks.build(task_params)
    
    if @task.save
      flash[:success] = "タスクが正常に登録されました"
      redirect_to @task
    else
      flash.now[:danger] = "タスクが登録されませんでした"
      render :new
    end
  end
  
  def edit
    task_params
  end
  
  def update
    task_params
    if @task.update(task_params)
      flash[:success] = 'タスクは正常に更新されました。'
      redirect_to @task
    else
      flash.now[:danger] = 'タスクは更新されませんでした。'
      render :edit
    end
  end 
  
  def destroy
    task_params
    @task.destroy
    
    flash[:success] = 'タスクを削除しました。'
    redirect_to tasks_url
  end
  
  private
  
  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

tasks/edit.html

<h1>id:
<%= @task.id %>
のタスク編集ページ</h1>

<%= render 'form', task: @task %>

tasks/index.html

<h1>タスク一覧</h1>

<% if @tasklists.any? %>

  <table class="table table-striped">
    <thead>
      <tr>
        <th>id</th>
        <th>ステータス</th>
        <th>タスク</th>
      </tr>
    </thead>
    <tbody>
      
<% @tasklists.each do |task| %>

      <tr>
        <td>
<%= link_to task.id, task %>
</td>
        <td>
<%= task.status %>
</td>
        <td>
<%= task.content %>
</td>        
      </tr>
      
<% end %>
      
    </tbody>
  </table>
  <div>


  </div>
<% end %>

<ul>
  
<%= link_to '新規タスクの登録', new_task_path, class: "btn btn-primary" %>

tasks/new.html

<h1>タスク新規作成ページ</h1>

<%= render 'form', task: @task %>

tasks/show.html

<h1>id:
<%= @task.id %>
のタスク詳細ページ</h1>

<table class="table table-bordered">
  <tr>
    <th>id</th>
    <td>
<%= @task.id %>
</td>
  </tr>
  <tr>
    <th>ステータス</th>
    <td>
<%= @task.status %>
    </td>
  </tr>
  <tr>
    <th>タスク</th>
    <td>
<%= @task.content %>
</td>
  </tr>
</table>


<%= link_to 'このタスクを編集する', edit_task_path(@task), class: 'btn btn-light' %>

<%= link_to 'このタスクを削除する', @task, method: :delete, data: {confirm: '本当に削除してよろしいですか？' }, class: 'btn btn-danger' %>

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: [:show, :new, :create]
  
  resources :tasks, only: [:create, :destroy]
end

行動規範の内容に同意します

回答1件

ベストアンサー

気になるのはcorrect_userですね。

ruby
1  before_action :correct_user, only: [:create, :destroy]
2  def correct_user
3    @task = current_user.tasks.find_by(id: params[:id])
4    unless @task
5      redirect_to root_url
6    end
7  end