diff --git a/app/controllers/subprincipleitems_controller.rb b/app/controllers/subprincipleitems_controller.rb
index aa12d91..4da584f 100644
--- a/app/controllers/subprincipleitems_controller.rb
+++ b/app/controllers/subprincipleitems_controller.rb
@@ -1,5 +1,6 @@
 class SubprincipleitemsController < ApplicationController
     before_action :set_subprincipleitem, only: %i[ edit update ]
+    before_action :set_caf, only: %i[ edit update ]
 
     def edit
     end
@@ -7,7 +8,7 @@ class SubprincipleitemsController < ApplicationController
     def update
         respond_to do |format|
             if @subprincipleitem.update(subprincipleitem_params)
-                format.html { redirect_to company_url(@company), notice: "Company was successfully updated." }
+                format.html { redirect_to (company_caf_path(@caf.company, @caf) + "#subprincipleitem_#{@subprincipleitem.id}"),  notice: "CAF item was updated." }
                 format.json { render :show, status: :ok, location: @company }
             else
                 format.html { render :edit, status: :unprocessable_entity }
@@ -21,8 +22,12 @@ class SubprincipleitemsController < ApplicationController
             @subprincipleitem = Subprincipleitem.find(params[:id])
         end
 
+        def set_caf
+            @caf = Caf.find(params[:caf_id])
+        end
+
         # Only allow a list of trusted parameters through.
         def subprincipleitem_params
-            params.require(:subprincipleitem).permit(:affirmative)
+            params.require(:subprincipleitem).permit(:affirmative,:evidence)
         end
 end
diff --git a/app/views/cafs/_caf.html.erb b/app/views/cafs/_caf.html.erb
index 906ab3f..c987ff2 100644
--- a/app/views/cafs/_caf.html.erb
+++ b/app/views/cafs/_caf.html.erb
@@ -27,7 +27,12 @@
           <% subprinciple.subprincipleitemgroups.each do |subprincipleitemgroup| %>
             <td>
             <% subprincipleitemgroup.subprincipleitems.each do |subprincipleitem| %>
-              <div style="background-color:<%= subprincipleitem.affirmative ? '#05aa05' : '#aa3030'  %>;" id="<%= "subprincipleitemid_#{subprincipleitem.id}" %>" class="subprincipleitem"><%= subprincipleitem.description %><%= link_to "Change", edit_caf_subprincipleitem_path(subprincipleitem) %>
+              <div style="background-color:<%= subprincipleitem.affirmative ? '#05aa05' : '#aa3030'  %>;" id="<%=  dom_id(subprincipleitem) %>" class="subprincipleitem">
+                <%= subprincipleitem.description %>
+                <br/>
+                <%= link_to "Change", edit_caf_subprincipleitem_path(@caf,subprincipleitem) %>
+                <br/>
+                <hr/>
               </div>
 
             <% end %>
diff --git a/app/views/subprincipleitems/edit.html.erb b/app/views/subprincipleitems/edit.html.erb
index cd6c7cb..7cf454d 100644
--- a/app/views/subprincipleitems/edit.html.erb
+++ b/app/views/subprincipleitems/edit.html.erb
@@ -3,7 +3,8 @@
 
 <p><%= @subprincipleitem.description %><p>
 
-<%= form_with(model: @subprincipleitem) do |form| %>
+<%#= form_for( [company, caf] ) do |form| %>
+<%= form_for( [ @caf, @subprincipleitem] ) do |form| %>
     <div>
         <%= form.label :affirmative %> <%= form.check_box :affirmative, {}, true, false %>
     </div>
@@ -12,6 +13,6 @@
         <%= form.rich_text_area :evidence %>
     </div>
     <div>
-        <%= form.submit %>
+        <%= form.submit 'Update'  %>
     </div>
 <% end %>