Refs #27: edit papers

app/controllers/papers_controller.rb

@@ -7,6 +7,10 @@ def new
     @paper = Paper.new
   end
 
+  def edit
+    @paper = Paper.find(params[:id])
+  end
+
   def show
     @paper = Paper.find(params[:id])
   end
@@ -20,6 +24,16 @@ def create
       render 'new'
     end
   end
+
+  def update
+    @paper = Paper.find(params[:id])
+
+    if @paper.update(paper_params)
+      redirect_to @paper
+    else
+      render 'edit'
+    end
+  end
 end
 
 private

app/views/papers/edit.html.erb

@@ -0,0 +1,40 @@
+<h1>Editing paper</h1>
+
+<%= form_for :paper, url: paper_path(@paper), method: :patch do |f| %>
+
+    <% if @paper.errors.any? %>
+        <div id="error_explanation">
+          <h2>
+            <%= pluralize(@paper.errors.count, 'error') %> prohibited
+            this paper from being saved:
+          </h2>
+          <ul>
+            <% @paper.errors.full_messages.each do |msg| %>
+                <li><%= msg %></li>
+            <% end %>
+          </ul>
+        </div>
+    <% end %>
+
+    <p>
+      <%= f.label :title %><br>
+      <%= f.text_field :title %>
+    </p>
+
+    <p>
+      <%= f.label :venue %><br>
+      <%= f.text_field :venue %>
+    </p>
+
+    <p>
+      <%= f.label :year %><br>
+      <%= f.text_field :year %>
+    </p>
+
+    <p>
+      <%= f.submit %>
+    </p>
+
+<% end %>
+
+<%= link_to 'Back', papers_path %>

app/views/papers/index.html.erb

@@ -9,6 +9,7 @@
     <th>Venue</th>
     <th>Year</th>
     <th>Show</th>
+    <th>Edit</th>
   </tr>
 
   <% @papers.each do |paper| %>
@@ -17,6 +18,7 @@
         <td><%= paper.venue %></td>
         <td><%= paper.year %></td>
         <td><%= link_to 'Show', paper %></td>
+        <td><%= link_to 'Edit', edit_paper_path(paper) %></td>
       </tr>
   <% end %>
 </table>

spec/features/paper/edit_paper_spec.rb

@@ -6,7 +6,7 @@
     visit edit_paper_path(@test_paper)
   end
   it 'should edit with no error' do
-    @test_paper = FactoryGirl.create :author
+    @test_paper = FactoryGirl.create :paper
     visit edit_paper_path(@test_paper)
 
     fill_in 'paper_year', with: 12_345