closes #62: booking_list show resume button now when user_id of curre…

…nt user and owner matches instead of relying on a parameter

app/models/time_booking.rb

@@ -159,6 +159,10 @@ def update_time(start, stop)
     self.time_entry.update_attributes!(:spent_on => start, :hours => self.hours_spent) #also update TimeEntry
   end
 
+  def user_id
+    self.time_entry.user_id
+  end
+
   def activity_id
     self.time_entry.activity_id
   end

app/views/time_bookings/_list_bookings.html.erb

@@ -55,7 +55,7 @@
                       <% end %>
 
                       <tr id="booking-entry-<%= entry.id %>" class="hascontextmenu entry <%= cycle('odd', 'even') %>">
-                        <%= render :partial => 'time_bookings/list_entry', :locals => {:entry => entry, :query => query, :show_resume => true} %>
+                        <%= render :partial => 'time_bookings/list_entry', :locals => {:entry => entry, :query => query} %>
                       </tr>
                   <% end -%>
                   </tbody>

app/views/time_bookings/_list_entry.html.erb

@@ -5,7 +5,7 @@
 <%= raw query.columns.map { |column| "<td class=\"#{column.css_classes}\">#{column_content(column, entry)}</td>" }.join %>
 <td>
   <% tt = TimeTracker.where(:user_id => User.current.id).first %>
-  <% if show_resume && tt.nil? && User.current.allowed_to_globally?({:controller => 'time_trackers', :action => 'start'}, {}) %>
+  <% if entry.user_id == User.current.id && tt.nil? && User.current.allowed_to_globally?({:controller => 'time_trackers', :action => 'start'}, {}) %>
       <%= link_to '',
                   {:controller => 'time_trackers', :action => 'start', :time_tracker => {:issue_id => entry.virtual? ? nil : entry.time_entry.issue.id, :comments => entry.comments}},
                   :class => 'icon-start tt_list_button'

app/views/tt_bookings_list/_list.html.erb

@@ -49,7 +49,7 @@
               <% end %>
 
               <tr id="booking-entry-<%= entry.id %>" class="hascontextmenu entry <%= cycle('odd', 'even') %>">
-                <%= render :partial => 'time_bookings/list_entry', :locals => {:entry => entry, :query => query, :show_resume => false} %>
+                <%= render :partial => 'time_bookings/list_entry', :locals => {:entry => entry, :query => query} %>
               </tr>
           <% end -%>
           </tbody>