Merge pull request #32 from openHPI/30_duplicate_function

30 duplicate function

Author: Marcel Jankrift

app/controllers/exercises_controller.rb

@@ -27,6 +27,11 @@ def new
     @exercise.descriptions << Description.new
   end
 
+  def duplicate
+    @exercise = Exercise.new(Exercise.find(params[:id]).attributes)
+    render 'new'
+  end
+
   # GET /exercises/1/edit
   def edit
   end

app/models/ability.rb

@@ -8,6 +8,7 @@ def initialize(user)
       elsif user.role == 'teacher'
         can :show, Exercise
         can :create, Exercise
+        can :duplicate, Exercise
         can :read, Exercise
         can :edit, Exercise do |exercise|
           exercise.user == user

app/views/exercises/_test_form.html.erb

@@ -7,7 +7,7 @@
 </div>
 <div class="field">
   <%= f.label :feedback_message %><br>
-  <%= test.text_field :feedback_message %>
+  <%= test.text_field :feedback_message, size:80 %>
 </div>
 <div class="field">
   <%= f.label :testing_framework %><br>

app/views/exercises/show.html.erb

@@ -12,14 +12,14 @@
   <dt><%= 'Visibility' %>:</dt>
   <dd><%= @exercise.public ? 'public' : 'private' %></dd>
   <dt><%= 'Created by' %>:</dt>
-  <% if defined? @exercise.user %>
+  <% if @exercise.user %>
     <% if @exercise.user.first_name.nil?%>
       <dd><%= "User#{@exercise.user.id}"%></dd>
     <% else %>
-        <dd><%= @exercise.user.name %></dd>
+      <dd><%= @exercise.user.name %></dd>
     <% end %>
   <% else %>
-    <%= 'User undefined' %>
+    <dd><%= 'User undefined' %></dd>
   <% end %>
   <dt><%= 'Created at' %>:</dt>
   <dd><%= @exercise.created_at %></dd>

app/views/home/_exercises.html.erb

@@ -29,6 +29,9 @@
       <div class="btn-group-vertical pull-right" role="group" aria-label="...">
         <%= link_to 'Select', exercise_path(exercise), class:'btn btn-default' %>
         <%= link_to 'Show Comments', exercise_comments_path(exercise), class:'btn btn-default' %>
+        <% if can? :create, exercise %>
+            <%= link_to 'Duplicate', duplicate_exercise_path(exercise), class:'btn btn-default' %>
+        <% end %>
         <% if can? :edit, exercise %>
             <%= link_to 'Edit', edit_exercise_path(exercise), class:'btn btn-default' %>
             <%= link_to 'Delete', exercise_path(exercise), class:'btn btn-default', method: :delete, data: { confirm: 'Are you sure?' }  %>

config/routes.rb

@@ -16,7 +16,7 @@
 
   get 'comments/comments_all'
 
-  get 'exercises/exercises_all'
+  get 'exercises/:id/duplicate', to: 'exercises#duplicate', as: 'duplicate_exercise'
 
   resources :labels
   resources :label_categories
@@ -35,6 +35,9 @@
     end
   end
 
+  get 'exercises/exercises_all'
+
+
 
 
   # The priority is based upon order of creation: first created -> highest priority.