Louis | mild refactorings, add name method to bike

10 years ago · 47980d61b0
6 changed files with 36 additions and 40 deletions
--- a/app/controllers/bikes_controller.rb
+++ b/app/controllers/bikes_controller.rb
@ -18,30 +18,24 @@ class BikesController < ApplicationController

  def create
    @bike = Bike.new(bike_params)
-    respond_to do |format|
    if @bike.save
-        format.html { redirect_to @bike, notice: 'Bike was successfully created.' }
+      redirect_to @bike, notice: 'Bike was successfully created.'
    else
-        format.html { render action: 'new' }
-      end
+      render action: 'new'
    end
  end

  def update
-    respond_to do |format|
    if @bike.update(bike_params)
-        format.html { redirect_to @bike, notice: 'Bike was successfully updated.' }
+      redirect_to @bike, notice: 'Bike was successfully updated.'
    else
-        format.html { render action: 'edit' }
-      end
+      render action: 'edit'
    end
  end

  def destroy
    @bike.destroy
-    respond_to do |format|
-      format.html { redirect_to bikes_url }
-    end
+    redirect_to bikes_url
  end

  private
@ -56,7 +50,6 @@ class BikesController < ApplicationController
        :model, 
        :bike_type, 
        :color, 
-        :frame_size, 
        :log_number, 
        :purpose, 
        :serial_number, 
--- a/app/models/bike.rb
+++ b/app/models/bike.rb
@ -5,4 +5,8 @@ class Bike < ActiveRecord::Base
  validates :bike_type, presence: true
  validates :color, presence: true
  validates :serial_number, presence: true
+
+  def name
+    [self.seat_tube_size.to_s, self.brand, self.model] * ' '
+  end
 end
--- a/app/views/clients/_fields.html.haml
+++ b/app/views/clients/_fields.html.haml
@ -50,8 +50,7 @@
      = f.text_field :completion_date, class: "form-control datepicker", disabled: disabled

  .form-group
-    = f.label "Type:", class: "col-sm-2 control-label"
+    = f.label "Bike:", class: "col-sm-2 control-label"
    .col-sm-10
-      = f.select(:bike_id, Bike.all.collect {|b| [ b.model, b.id ] }, {include_blank: 'None'})
-  
+      = f.select(:bike_id, Bike.all.collect {|b| [ b.name, b.id ] }, {include_blank: 'None'})

--- a/app/views/clients/_form.html.haml
+++ b/app/views/clients/_form.html.haml
@ -1,5 +1,4 @@
-.container
-  = form_for @client, html: {class: 'form-horizontal'} do |f|
+= form_for @client, html: {class: 'form-horizontal'} do |f|
  - if @client.errors.any? 
    #error_explanation
      %h2= pluralize(@client.errors.count, "error") + " prohibited this bike from being saved:"
--- a/app/views/clients/new.html.haml
+++ b/app/views/clients/new.html.haml
@ -1,3 +1,4 @@
-%h1 New Client
-= render 'form'
-= link_to 'Back', clients_path
+.container
+  %h1 New Client
+  = render 'form'
+  = link_to 'Back', clients_path
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@ -1,10 +1,10 @@
 <!DOCTYPE html>
 <html lang="en">
 <%= render "layouts/head" %>
-<%= render "layouts/navbar" %>
-  <body>
+<body>
+  <%= render "layouts/navbar" %>
  <p class="notice"><%= notice %></p>
  <p class="alert"><%= alert %></p>
  <%= yield %>
-  </body>
+</body>
 </html>