Update paths

Use the much nicer /project/:project_id/jobs format

app/controllers/jobs_controller.rb

@@ -19,7 +19,7 @@ class JobsController < ApplicationController
 
   def update
     if @job.update(remove_empty_time_budgets(job_params))
-      redirect_to job_path(@job, project_id: @job.project.id)
+      redirect_to project_job_path(@job.project, @job)
     else
       render :edit
     end
@@ -27,7 +27,7 @@ class JobsController < ApplicationController
 
   def create
     if @job = Job.create(job_params)
-      redirect_to job_path(@job, project_id: @job.project.id)
+      redirect_to project_job_path(@job.project, @job)
     else
       render :edit
     end
@@ -36,7 +36,7 @@ class JobsController < ApplicationController
   def destroy
     @project = @job.project
     if @job.destroy
-      redirect_to jobs_path(project_id: @project.id)
+      redirect_to project_jobs_path(@project)
     else
       render :show
     end

app/models/job.rb

@@ -49,6 +49,6 @@ class Job < ActiveRecord::Base
 
   def to_s
     ActionView::Base.send(:include, Rails.application.routes.url_helpers)
-    ActionController::Base.helpers.link_to name, ActionController::Base.helpers.job_path(self, project_id: project.id)
+    ActionController::Base.helpers.link_to name, ActionController::Base.helpers.project_job_path(project, self)
   end
 end

app/views/jobs/_job.html.erb

@@ -1,5 +1,5 @@
 <tr>
-  <td><%= link_to job.name, job_path(job, project_id: job.project_id) %></td>
+  <td><%= link_to job.name, project_job_path(job.project, job) %></td>
   <td><%= format_date(job.starts_on) %></td>
   <td><%= format_date(job.ends_on) %></td>
   <td><%= job.project_id %></td>

app/views/jobs/edit.html.erb

@@ -1,5 +1,5 @@
 <% html_title "Edit #{@job.name}" %>
 <h2>Edit <%= @job.name %></h2>
 
-<%= render partial: "form", locals: { form_target: job_path(@job, project_id: @job.project) } %>
+<%= render partial: "form", locals: { form_target: project_job_path(@job.project, @job) } %>
 

app/views/jobs/index.html.erb

@@ -1,6 +1,6 @@
 <% html_title "Jobs" %>
 <div class="contextual">
-  <%= link_to 'New job', new_job_path(project_id: @project.id), class: "icon icon-add new-job" %>
+  <%= link_to 'New job', new_project_job_path(project: @project), class: "icon icon-add new-job" %>
 </div>
 
 <h2>Jobs</h2>

app/views/jobs/new.html.erb

@@ -1,3 +1,3 @@
-<h2><%= link_to "Jobs", jobs_path(project_id: @project.id) %> » New</h2>
+<h2><%= link_to "Jobs", project_jobs_path(@project) %> » New</h2>
 
-<%= render partial: "form", locals: { form_target: jobs_path } %>
+<%= render partial: "form", locals: { form_target: project_jobs_path(@project) } %>

app/views/jobs/show.html.erb

@@ -1,9 +1,9 @@
 <% html_title @job.name %>
 <div class="contextual">
-  <%= link_to 'Edit', edit_job_path(project_id: @project.id), class: "icon icon-edit edit-job" %>
+  <%= link_to 'Edit', edit_project_job_path(@project, @job), class: "icon icon-edit edit-job" %>
   <%= actions_dropdown do %>
-    <%= copy_object_url_link(job_url(@job, project_id: @project.id, only_path: false)) %>
-    <%= link_to "Delete job", job_path(@job), data: { confirm: "Are you sure you want to delete this job?" }, method: :delete, class: 'icon icon-del' %>
+    <%= copy_object_url_link(project_job_url(@project, @job, only_path: false)) %>
+    <%= link_to "Delete job", project_job_path(@project, @job), data: { confirm: "Are you sure you want to delete this job?" }, method: :delete, class: 'icon icon-del' %>
   <% end %>
 </div>
 <h2>Job #<%= @job.id %></h2>

config/routes.rb

@@ -1 +1,3 @@
+resources :projects do
   resources :jobs
+end