From 74561aac0964ca7bd9283ad0a9b017e02431f845 Mon Sep 17 00:00:00 2001
From: "Ira W. Snyder" <devel@irasnyder.com>
Date: Fri, 23 Nov 2007 00:01:14 -0800
Subject: [PATCH] Add list of just-purchased items (this transaction) to the
 purchase view

Signed-off-by: Ira W. Snyder <devel@irasnyder.com>
---
 app/controllers/purchase_controller.rb       |  12 ++++++++--
 app/models/merchandise.rb                    |   2 +-
 app/views/bitem/list.rhtml                   |   2 +-
 app/views/merchandise/_form.rhtml            |   4 ++--
 app/views/purchase/menu.rhtml                |  12 ++++++++++
 db/development.sqlite3                       | Bin 17408 -> 17408 bytes
 db/migrate/020_rebuild_merchandises_table.rb |  24 +++++++++++++++++++
 db/schema.rb                                 |   4 ++--
 8 files changed, 52 insertions(+), 8 deletions(-)
 create mode 100644 db/migrate/020_rebuild_merchandises_table.rb

diff --git a/app/controllers/purchase_controller.rb b/app/controllers/purchase_controller.rb
index 4445c07..704e38b 100644
--- a/app/controllers/purchase_controller.rb
+++ b/app/controllers/purchase_controller.rb
@@ -12,6 +12,7 @@ class PurchaseController < ApplicationController
     # enter a customer id here
     render :action => 'begin'
     session[:total] = 0.00
+    session[:items] = []
   end
 
   def customer_ok
@@ -27,6 +28,7 @@ class PurchaseController < ApplicationController
   def menu
     @customer = Customer.find_by_id(session[:customer_id])
     @total_price = session[:total]
+    @items = session[:items]
     render :action => 'menu'
   end
 
@@ -69,10 +71,13 @@ class PurchaseController < ApplicationController
     purchase.customer_id = session[:customer_id]
     purchase.date = Time.now.to_date
     purchase.price = @rentable.calculated_price
-    session[:total] += @rentable.calculated_price
     purchase.rentable = @rentable
     purchase.save!
 
+    # Add te session variables
+    session[:total] += @rentable.calculated_price
+    session[:items].push @rentable
+
     flash[:notice] = "Successfully made purchase"
     redirect_to :action => :menu
   end
@@ -108,11 +113,14 @@ class PurchaseController < ApplicationController
     purchase.customer_id = session[:customer_id]
     purchase.date = Time.now.to_date
     purchase.price = @merchandise.price
-    session[:total] += @merchandise.price
     purchase.merchandise = @merchandise
     purchase.quantity = 1
     @merchandise.quantity -= 1
 
+    # Add to session variables
+    session[:total] += @merchandise.price
+    session[:items].push @merchandise
+
     # Save both the merchandise (we changed the quantity) and the purchase to the log
     @merchandise.save!
     purchase.save!
diff --git a/app/models/merchandise.rb b/app/models/merchandise.rb
index 16fd1e8..efb7c7e 100644
--- a/app/models/merchandise.rb
+++ b/app/models/merchandise.rb
@@ -1,7 +1,7 @@
 class Merchandise < ActiveRecord::Base
   has_many :merchandise_purchases
 
-  validates_presence_of :name
+  validates_presence_of :title
   validates_numericality_of :quantity
   validates_numericality_of :price
 
diff --git a/app/views/bitem/list.rhtml b/app/views/bitem/list.rhtml
index 6b5bb14..81a7d7c 100644
--- a/app/views/bitem/list.rhtml
+++ b/app/views/bitem/list.rhtml
@@ -12,7 +12,7 @@
 <% for bitem in @bitems %>
   <tr>
   <td><%=h bitem.customer.name %></td>
-  <td><%=h bitem.merchandise.name %></td>
+  <td><%=h bitem.merchandise.title %></td>
   <% for column in Bitem.content_columns %>
     <td><%=h bitem.send(column.name) %></td>
   <% end %>
diff --git a/app/views/merchandise/_form.rhtml b/app/views/merchandise/_form.rhtml
index d7f7735..35db026 100644
--- a/app/views/merchandise/_form.rhtml
+++ b/app/views/merchandise/_form.rhtml
@@ -1,8 +1,8 @@
 <%= error_messages_for 'merchandise' %>
 
 <!--[form:merchandise]-->
-<p><label for="merchandise_name">Name</label><br/>
-<%= text_field 'merchandise', 'name'  %></p>
+<p><label for="merchandise_title">Title</label><br/>
+<%= text_field 'merchandise', 'title'  %></p>
 
 <p><label for="merchandise_quantity">Quantity</label><br/>
 <%= text_field 'merchandise', 'quantity'  %></p>
diff --git a/app/views/purchase/menu.rhtml b/app/views/purchase/menu.rhtml
index ce8a646..6f5abd1 100644
--- a/app/views/purchase/menu.rhtml
+++ b/app/views/purchase/menu.rhtml
@@ -1,11 +1,23 @@
 <h1>Purchase Menu</h1>
 
+<h3>Customer Information</h3>
 <ul>
   <li>Customer ID: <%= @customer.id %></li>
   <li>Customer Name: <%= @customer.name %></li>
   <li>Total Price: <%= @total_price %></li>
 </ul>
 
+<h3>Items Purchased</h3>
+<% if @items.empty? %>
+  None
+<% else %>
+<ul>
+<% for item in @items %>
+  <li><%= item.title %></li>
+<% end %>
+</ul>
+<% end %>
+
 <ul>
   <li><%=link_to 'Rent an Item', :action => 'rent_begin' %></li>
   <li><%=link_to 'Buy Merchandise', :action => 'buy_begin' %></li>
diff --git a/db/development.sqlite3 b/db/development.sqlite3
index f482dbeddf0f35aee6a15925c1f72df0c6032889..5ddceca9996e69b3e2e0569b4fba3083fb01c398 100644
GIT binary patch
delta 489
zcmX}o!An#@90&0E{(dv>>9cm<!)F~dXjylbmZW_iOqP9Ow}Jx0XuSxEERv{Cw&<F#
z(_VsN2>c5?)~@f=MLI|ak9o+O5CnDWKd85kIxx)V`(*}ZX19yoF5Zs^bHWZkUnrl=
z{@oK7;lYrU1;rbJfASIUbC);z=@NSr$`L@R1jqD>W5e<y;XJWTM0uJ3na{-=w{X$P
z>3aSCR1{6sr>``7;KRSpGlI|(Xwf&G>8Fp7wMu7@`CX1pqkQZ!9}51%Cw!O%vd0fb
zS&0hpBzgU!eS0B!`KsNKVMt0;fyb+z^+CC_mUOo3atwJ(Sln9O+`MeX$LwEKNgW^-
ze9FE2X)8R-KK-Vzv`0G>Q-ELif_G>m?x&+59hkH^DMuXbTcB1nPv@#CWYh||YUU`u
zqa>kr?Yz$wjTpocY0sV@J#4{x$U<t@!qtw2qm_ly@^TLkU_RY@UQi+t{EUg-pdpeE
m>9UtT^xd143f?Z7`SP;)Q3_43ypT1b@800|yv%R@b^imuRcUwt

delta 244
zcmZqZU~K4MoFFYI#Q*^UK#B>7tv2dhW@i-M#KFSC#mu~nf%y^h5$0u^1zBb=bICBX
zF)%VJDl#%p-oP)z$T)dDzZj70;+NSZz$m~A)XmHo&cGPXbd1?-W1}bI=5WRPj4X}r
zoa~dWluvOmF`s2%e#LwiXxKq!BWWg926aYupTwf{RE2>2g5><7JT?avR=FL4uUkbJ
zL6VHYxrsSBFv-%*|J7MRj_P1wzQMeKxdUiU6*H3>^W;VwmC5V$!+2O2tr(a?n3<UB
a8Lc)pYB6rMGQ7ma!d%Aew^_jE13LhPXF7fW

diff --git a/db/migrate/020_rebuild_merchandises_table.rb b/db/migrate/020_rebuild_merchandises_table.rb
new file mode 100644
index 0000000..e59901a
--- /dev/null
+++ b/db/migrate/020_rebuild_merchandises_table.rb
@@ -0,0 +1,24 @@
+class RebuildMerchandisesTable < ActiveRecord::Migration
+
+  # This whole thing is an awful way of doing a column rename
+
+  def self.up
+    drop_table :merchandises
+
+    create_table :merchandises do |t|
+      t.column :title, :string, :null => false
+      t.column :quantity, :integer, :null => false, :default => 0
+      t.column :price, :decimal, :precision => 8, :scale =>2, :default => 0
+    end
+  end
+
+  def self.down
+    drop_table :merchandises
+
+    create_table :merchandises do |t|
+      t.column :name, :string, :null => false
+      t.column :quantity, :integer, :null => false, :default => 0
+      t.column :price, :decimal, :precision => 8, :scale =>2, :default => 0
+    end
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 2e0f138..ae1459b 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -2,7 +2,7 @@
 # migrations feature of ActiveRecord to incrementally modify your database, and
 # then regenerate this schema definition.
 
-ActiveRecord::Schema.define(:version => 19) do
+ActiveRecord::Schema.define(:version => 20) do
 
   create_table "bitems", :force => true do |t|
     t.column "customer_id",    :integer, :null => false
@@ -33,7 +33,7 @@ ActiveRecord::Schema.define(:version => 19) do
   end
 
   create_table "merchandises", :force => true do |t|
-    t.column "name",     :string,                                                 :null => false
+    t.column "title",    :string,                                                 :null => false
     t.column "quantity", :integer,                               :default => 0,   :null => false
     t.column "price",    :decimal, :precision => 8, :scale => 2, :default => 0.0
   end
-- 
2.25.1