edit a task complete with tests passing

Author: ChantalDemissie

app/controllers/tasks_controller.rb

@@ -1,27 +1,81 @@
-# class Task
-#   def initialize(name)
-#     @name = name
-#     @complete = false
-#   end
-# end
-
-TASKS = [
-  {name: 'Buy Jesse a toy', complete: false},
-  {name: 'Percy needs flea pills', complete: true},
-  {name: 'Reload chantals coffee card', complete: true}
-]
-
 class TasksController < ApplicationController
   def root
     @message = 'Hello world!'
   end
 
   def show
-    @task = Task.find(params['id'])
+    # This is the ruby try-catch.
+    # If Task.find throws an exception
+    # (e.g., can't find a task with that id)
+    # then we can catch the exception and
+    # redirect the user to the homepage with a message.
+    begin
+      @task = Task.find(params['id'])
+    rescue
+      flash[:error] = "Could not find task with id: #{params['id']}" #not printing??? why!
+      redirect_to tasks_path
+      return
+    end
   end
 
   def index
     @tasks = Task.all
-    #@tasks = TASKS
   end
+
+  def create
+    unless params.key?(:task)
+      @task = Task.new
+    else
+      @task = Task.new(
+        name: params[:task][:name],
+        description: params[:task][:description]
+      )
+      if @task.save
+        new_task = Task.find_by(name: @task.name)
+        redirect_to task_path(new_task.id)
+      else
+        render :new
+      end
+    end
+  end
+
+  def edit
+    begin
+      @task = Task.find(params['id'])
+    rescue
+      flash[:error] = "Could not find task with id: #{params['id']}"
+      redirect_to tasks_path
+      return
+    end
+  end
+
+  def update
+    begin
+      @task = Task.find(params[:id])
+    rescue
+      flash[:error] = "Could not find task with id: #{params['id']}"
+      redirect_to tasks_path
+      return
+    end
+
+    begin
+      if params[:task].key?(:completion_date)
+        @task.update(
+          name: params[:task][:name],
+          description: params[:task][:description],
+          completion_date: params[:task][:completion_date]
+        )
+      else
+        @task.update(
+          name: params[:task][:name],
+          description: params[:task][:description]
+        )
+      end
+      redirect_to task_path(params[:id])
+    rescue
+      render :new
+      return
+    end
+  end
+
 end

app/views/tasks/create.html.erb

@@ -0,0 +1,9 @@
+<h1>New Task</h1>
+<%= link_to 'Back to tasks', tasks_path %>
+<%= form_with model: @task do |f| %>
+<ul>
+  <li>Name: <%= f.text_field :name %></li>
+  <li>Description: <%= f.text_field :description %></li>
+</ul>
+<%= f.submit "Save task" %>
+<% end %>

app/views/tasks/edit.html.erb

@@ -0,0 +1,10 @@
+<h1>Form for editing an existing Task</h1>
+<%= link_to 'Back to all tasks', tasks_path %>
+<%= form_with model: @task, class: 'edit-book' do |f| %>
+<p> Please fill out this form to edit a task </p>
+<ul>
+  <li>Name: <%= f.text_field :name %></li>
+  <li>Description: <%= f.text_field :description %></li>
+  <%= f.submit "Save Task", class: "task-form_submit-btn"%>
+</ul>
+<% end %>

app/views/tasks/index.html.erb

@@ -1,12 +1,13 @@
 <h1>Tasks</h1>
+<%= link_to 'Create task', create_task_path %>
 <ul>
   <% @tasks.each do |task| %>
     <li>
-          <%= link_to task.name, task_path(task.id) %>
       <% if task.completion_date %>
-        <s><%= task.name %></s>
+        <s><%= link_to task.name, task_path(task.id) %></s>
       <% else %>
-        <%= task.name %>
+        <%= link_to task.name, task_path(task.id) %>
       <% end %>
     </li>
   <% end %>
+</ul>

app/views/tasks/show.html.erb

@@ -16,5 +16,6 @@
     <li>completed on <%= @task.completion_date %></li>
   <% end %>
 </ul>
-
+<%= link_to "Edit", edit_task_path %>
+<br />
 <%= link_to "Back to all Tasks", tasks_path %>

config/routes.rb

@@ -2,6 +2,9 @@
   # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
   root :to => 'tasks#root' #re-read ada lesson this looks weird
   get '/tasks', to: 'tasks#index', as: 'tasks'
+  get '/tasks/create', to: 'tasks#create', as: 'create_task'
   get '/tasks/:id', to: 'tasks#show', as: 'task'
-
+  post '/tasks', to: 'tasks#create'
+  get '/tasks/:id/edit', to: 'tasks#edit', as: 'edit_task'
+  patch '/tasks/:id', to: 'tasks#update', as: 'update_task'
 end