diff --git a/app/controllers/books_controller.rb b/app/controllers/books_controller.rb
index d13ffd9..065113f 100644
--- a/app/controllers/books_controller.rb
+++ b/app/controllers/books_controller.rb
@@ -50,7 +50,8 @@ def create
format.xml { render xml: @book, status: :created, location: @book }
format.json { render json: @book, status: :created, location: @book }
else
- format.html { render action: "new" }
+# format.html { render action: "new" }
+ format.html { render :new }
format.xml { render xml: @book.errors, status: :unprocessable_entity }
format.json { render json: @book.errors, status: :unprocessable_entity }
end
@@ -66,7 +67,8 @@ def update
format.xml { head :no_content }
format.json { head :no_content }
else
- format.html { render action: "edit" }
+# format.html { render action: "edit" }
+ format.html { render :edit }
format.xml { render xml: @book.errors, status: :unprocessable_entity }
format.json { render json: @book.errors, status: :unprocessable_entity }
end
diff --git a/app/controllers/memos_controller.rb b/app/controllers/memos_controller.rb
new file mode 100644
index 0000000..fe41343
--- /dev/null
+++ b/app/controllers/memos_controller.rb
@@ -0,0 +1,38 @@
+class MemosController < ApplicationController
+ def new
+ @book = Book.find(params[:book_id])
+ @memo = @book.memos.new
+ end
+
+ def create
+ @book = Book.find(params[:book_id])
+ @memo = @book.memos.new(params[:memo])
+ if @memo.save
+ redirect_to book_path(@book), :notice => "Registered the memo."
+ else
+ render :new
+ end
+ end
+
+ def edit
+ @book = Book.find(params[:book_id])
+ @memo = @book.memos.find(params[:id])
+ end
+
+ def update
+ @book = Book.find(params[:book_id])
+ @memo = @book.memos.find(params[:id])
+ if @memo.update_attributes(params[:memo])
+ redirect_to book_path(@book), :notice => "Update a memo."
+ else
+ render :action => "Edit"
+ end
+ end
+
+ def destroy
+ @book = Book.find(params[:book_id])
+ @memo = @book.memos.find(params[:id])
+ @memo.destroy
+ redirect_to book_path(@book)
+ end
+end
diff --git a/app/models/book.rb b/app/models/book.rb
index 95c42dd..c5f099f 100644
--- a/app/models/book.rb
+++ b/app/models/book.rb
@@ -1,5 +1,6 @@
# encoding: UTF-8
class Book < ActiveRecord::Base
+ has_many :memos, :dependent => :destroy
attr_accessible :memo, :purchased_on, :title
validates :title, :presence => true
diff --git a/app/models/memo.rb b/app/models/memo.rb
new file mode 100644
index 0000000..f186e1f
--- /dev/null
+++ b/app/models/memo.rb
@@ -0,0 +1,4 @@
+class Memo < ActiveRecord::Base
+ attr_accessible :body, :book_id
+ validates :body, :presence => true, :length => { :maximum => 100 }
+end
diff --git a/app/views/books/show.html.erb b/app/views/books/show.html.erb
index 23005ab..d076385 100644
--- a/app/views/books/show.html.erb
+++ b/app/views/books/show.html.erb
@@ -15,6 +15,15 @@
<%= @book.purchased_on %>
+
+ <%= link_to 'Add a memo', new_book_memo_path(@book) %>
+
+<% @book.memos.each do |memo| %>
+ <%= memo.body %> |
+ <%= link_to "Edit", edit_book_memo_path(:book_id => @book.id, :id => memo.id) %> |
+ <%= link_to "Delete", book_memo_path(:book_id => @book.id, :id => memo.id),
+ :confirm => "Delete a memo?", :method => :delete %>
+<% end %>
<%= link_to 'Edit', edit_book_path(@book) %> |
<%= link_to 'Back', books_path %>
diff --git a/app/views/memos/edit.html.erb b/app/views/memos/edit.html.erb
new file mode 100644
index 0000000..023ebf7
--- /dev/null
+++ b/app/views/memos/edit.html.erb
@@ -0,0 +1,10 @@
+Title: <%= @book.title %>
+Memo: <%= @book.memo %>
+Purchased on: <%= @book.purchased_on %>
+
+
+<%= form_for @memo, :url => book_memo_path(:book_id => @book.id,
+ :id => @memo.id), :method => :put do |f| %>
+ Additional memo: <%= f.text_field :body %>
+ <%= f.submit %>
+<% end %>
diff --git a/app/views/memos/new.html.erb b/app/views/memos/new.html.erb
new file mode 100644
index 0000000..51f820d
--- /dev/null
+++ b/app/views/memos/new.html.erb
@@ -0,0 +1,8 @@
+Title: <%= @book.title %>
+Memo: <%= @book.memo %>
+Purchased on: <%= @book.purchased_on %>
+
+<%= form_for @memo, :url => book_memos_path do |f| %>
+ Memo: <%= f.text_field :body %>
+ <%= f.submit %>
+<% end %>
diff --git a/config/routes.rb b/config/routes.rb
index f3d38ab..17d97c2 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -1,6 +1,8 @@
BookMemo2::Application.routes.draw do
- resources :books
-
+ resources :books do
+ resources :memos
+ end
+
# The priority is based upon order of creation:
# first created -> highest priority.
diff --git a/db/migrate/20120615012627_create_memos.rb b/db/migrate/20120615012627_create_memos.rb
new file mode 100644
index 0000000..6a31f6f
--- /dev/null
+++ b/db/migrate/20120615012627_create_memos.rb
@@ -0,0 +1,14 @@
+class CreateMemos < ActiveRecord::Migration
+ def change
+ create_table :memos do |t|
+ t.integer :book_id
+ t.text :body
+
+ t.timestamps
+ end
+ end
+
+ def self.down
+ drop_table :memos
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 5fc61c2..2ffe2f9 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -11,7 +11,7 @@
#
# It's strongly recommended to check this file into your version control system.
-ActiveRecord::Schema.define(:version => 20120526050801) do
+ActiveRecord::Schema.define(:version => 20120615012627) do
create_table "books", :force => true do |t|
t.string "title"
@@ -21,4 +21,11 @@
t.datetime "updated_at", :null => false
end
+ create_table "memos", :force => true do |t|
+ t.integer "book_id"
+ t.text "body"
+ t.datetime "created_at", :null => false
+ t.datetime "updated_at", :null => false
+ end
+
end
diff --git a/spec/factories/memos.rb b/spec/factories/memos.rb
new file mode 100644
index 0000000..8d3526d
--- /dev/null
+++ b/spec/factories/memos.rb
@@ -0,0 +1,8 @@
+# Read about factories at https://github.com/thoughtbot/factory_girl
+
+FactoryGirl.define do
+ factory :memo do
+ book_id 1
+ body "MyText"
+ end
+end
diff --git a/spec/helpers/books_helper_spec.rb b/spec/helpers/books_helper_spec.rb
index 24456a3..b3c5992 100644
--- a/spec/helpers/books_helper_spec.rb
+++ b/spec/helpers/books_helper_spec.rb
@@ -11,5 +11,4 @@
# end
# end
describe BooksHelper do
- pending "add some examples to (or delete) #{__FILE__}"
end
diff --git a/spec/models/memo_spec.rb b/spec/models/memo_spec.rb
new file mode 100644
index 0000000..85d081e
--- /dev/null
+++ b/spec/models/memo_spec.rb
@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Memo do
+ context "character length is below 100." do
+ body = "x" * 100
+ subject { Memo.new(:body => body).valid? }
+ it { should be_true }
+ end
+
+ context "character length is over 100." do
+ body = "x" * (100 + 1)
+ subject { Memo.new(:body => body).valid? }
+ it { should be_false }
+ end
+end
diff --git a/spec/requests/books_spec.rb b/spec/requests/books_spec.rb
index 37522b7..f4d4a1d 100644
--- a/spec/requests/books_spec.rb
+++ b/spec/requests/books_spec.rb
@@ -37,4 +37,45 @@
end
end
end
+
+ describe '/books/:id/memos/new' do
+ let!(:book){ FactoryGirl.create :book }
+ subject { page }
+
+ before { visit "/books/#{book.id}/memos/new" }
+
+ context "Write a memo first" do
+ let(:memo_body){ 'first memo!' }
+
+ before do
+ fill_in "memo[body]", with: memo_body
+ click_on 'Create Memo'
+ end
+
+ it "Page transition check" do
+ current_path.should == book_path(book)
+ end
+
+ it "Input name check" do
+ should have_content memo_body
+ end
+ end
+
+ context "Write a memo second" do
+ let(:memo_body){ 'second memo!' }
+
+ before do
+ fill_in "memo[body]", with: memo_body
+ click_on 'Create Memo'
+ end
+
+ it "Page transition check" do
+ current_path.should == book_path(book)
+ end
+
+ it "Input name check" do
+ should have_content memo_body
+ end
+ end
+ end
end