diff --git a/app/controllers/children_controller.rb b/app/controllers/children_controller.rb index 0195aa0..cc642c5 100644 --- a/app/controllers/children_controller.rb +++ b/app/controllers/children_controller.rb @@ -4,5 +4,43 @@ class ChildrenController < ApplicationController respond_with @children end - def lookup; end + def show + @child = Child.includes(:parents).find_by_id(params[:id]) + @parents = Person.order(:first_name, :last_name) + end + + def new + @child = Child.new + end + + def edit + @child = Child.find_by_id(params[:id]) + end + + def add + @child = Child.create child_params + if @child.persisted? + redirect_to child_path(@child), notice: 'Created!' + else + render :new + end + end + + def update + @child = Child.find_by_id params[:id] + if @child.update child_params + redirect_to child_path(@child), notice: 'Updated!' + else + render :edit + end + end + + private + + def child_params + params.require(:child).permit( + :first_name, + :last_name + ) + end end diff --git a/app/controllers/docs_controller.rb b/app/controllers/docs_controller.rb index 7d77aef..cb686c8 100644 --- a/app/controllers/docs_controller.rb +++ b/app/controllers/docs_controller.rb @@ -1,8 +1,8 @@ class DocsController < ApplicationController def index @doc = { - name: "sms-pager-api", - documentation: "https://bitbucket.org/markamoser/sms-pager-api" + name: "sms-pager", + documentation: "https://bitbucket.org/markamoser/sms-pager" } respond_with @doc end diff --git a/app/controllers/parents_controller.rb b/app/controllers/parents_controller.rb index 573c3f3..765c800 100644 --- a/app/controllers/parents_controller.rb +++ b/app/controllers/parents_controller.rb @@ -15,7 +15,7 @@ class ParentsController < ApplicationController end def add - @parent = Person.create add_parent_params + @parent = Person.create parent_params if @parent.persisted? redirect_to parent_path(@parent), notice: 'Success!' @@ -30,9 +30,19 @@ class ParentsController < ApplicationController @more_children = Child.all end + def update + @parent = Person.find(params[:id]) + + if @parent.update(parent_params) + redirect_to parent_path(@parent), notice: 'Success!' + else + render :edit + end + end + private - def add_parent_params + def parent_params params.require(:parent).permit( :first_name, :last_name, diff --git a/app/models/person.rb b/app/models/person.rb index 65e08a4..07624de 100644 --- a/app/models/person.rb +++ b/app/models/person.rb @@ -13,7 +13,7 @@ class Person < ActiveRecord::Base scope :just_parents, lambda { joins(:children) - .order(:first_name) + .order(:first_name, :last_name) .uniq } diff --git a/app/views/children/_form.html.haml b/app/views/children/_form.html.haml new file mode 100644 index 0000000..ba8e1ff --- /dev/null +++ b/app/views/children/_form.html.haml @@ -0,0 +1,14 @@ +- if @child.errors.full_messages.any? + .errors + %h1 Uh oh! + - @child.errors.full_messages.each do |error| + %p= error + += form_for @child, as: :child, url: add_child_path do |f| + = f.label :first_name + = f.text_field :first_name + + = f.label :last_name + = f.text_field :last_name + + = f.submit diff --git a/app/views/children/_sub_nav.html.haml b/app/views/children/_sub_nav.html.haml index 0942d83..215428e 100644 --- a/app/views/children/_sub_nav.html.haml +++ b/app/views/children/_sub_nav.html.haml @@ -1,3 +1,4 @@ %ul.sub-nav - %li= link_to 'look up', lookup_child_path + %li= link_to 'List', list_children_path + %li= link_to 'New', new_child_path diff --git a/app/views/children/edit.html.haml b/app/views/children/edit.html.haml new file mode 100644 index 0000000..855edd0 --- /dev/null +++ b/app/views/children/edit.html.haml @@ -0,0 +1,5 @@ +%h2 Edit #{@child.name} + +%p= link_to 'back', child_path(@child) + += render partial: 'form' diff --git a/app/views/children/index.html.haml b/app/views/children/index.html.haml index 1dd0ccd..dae2f11 100644 --- a/app/views/children/index.html.haml +++ b/app/views/children/index.html.haml @@ -2,7 +2,11 @@ - @children.each do |child| %ul - %li= child.name + %li + = link_to child.name, child_path(child) + = link_to 'edit', edit_child_path(child) %ul - child.parents.each do |parent| - %li #{parent.name} #{page_link(parent)} + %li + = link_to parent.name, parent_path(parent) + = page_link(parent) diff --git a/app/views/children/new.html.haml b/app/views/children/new.html.haml new file mode 100644 index 0000000..7874876 --- /dev/null +++ b/app/views/children/new.html.haml @@ -0,0 +1,3 @@ +%h2 Add a New Child + += render partial: 'form' diff --git a/app/views/children/show.html.haml b/app/views/children/show.html.haml new file mode 100644 index 0000000..9ef8589 --- /dev/null +++ b/app/views/children/show.html.haml @@ -0,0 +1,18 @@ +%h2= @child.name +%p= link_to 'edit', edit_child_path(@child) + +- unless @child.parents.empty? + %p Parents: + %ul + - @child.parents.each do |parent| + %li + = link_to parent.name, parent_path(parent) + = link_to 'remove', del_parenthood_path(parent, @child), method: :delete + +%p Add Parent: += form_tag add_parent_to_child_path(@child) do + :ruby + select_options = options_from_collection_for_select(@parents, :id, :name) + html_options = { include_blank: true } + = select_tag(:parent, select_options, html_options) + = submit_tag 'Add Parent to Child' diff --git a/app/views/docs/index.html.haml b/app/views/docs/index.html.haml index 8052372..c5b419d 100644 --- a/app/views/docs/index.html.haml +++ b/app/views/docs/index.html.haml @@ -1,4 +1,2 @@ %h2 Something helpful later - %p= raw(ap @doc) - diff --git a/app/views/parents/edit.html.haml b/app/views/parents/edit.html.haml new file mode 100644 index 0000000..bf57a62 --- /dev/null +++ b/app/views/parents/edit.html.haml @@ -0,0 +1,24 @@ +%h2 Edit #{@parent.name} + +%p= link_to 'back', parent_path(@parent) + +- if @parent.errors.full_messages.any? + .errors + %h1 Uh oh! + - @parent.errors.full_messages.each do |e| + %p= e + += form_for @parent, as: :parent, url: update_parent_path do |f| + = f.label :first_name + = f.text_field :first_name + + = f.label :last_name + = f.text_field :last_name + + = f.label :phone + = f.phone_field :phone + + = f.label :email + = f.email_field :email + + = f.submit diff --git a/app/views/parents/index.html.haml b/app/views/parents/index.html.haml index a7af5d2..595add0 100644 --- a/app/views/parents/index.html.haml +++ b/app/views/parents/index.html.haml @@ -2,12 +2,12 @@ - @parents.each do |parent| %ul + %li= link_to parent.name, parent_path(parent) %li - = link_to parent.name, parent_path(parent) + = number_to_phone parent.phone = page_link(parent) - %li= number_to_phone parent.phone - %li= mail_to parent.email + %li= mail_to parent.email, nil, encode: 'hex' %li Children: %ul - parent.children.each do |child| - %li= child.name + %li= link_to child.name, child_path(child) diff --git a/app/views/parents/show.html.haml b/app/views/parents/show.html.haml index 0319f6d..30db3a0 100644 --- a/app/views/parents/show.html.haml +++ b/app/views/parents/show.html.haml @@ -8,7 +8,7 @@ %ul - @parent.children.each do |child| %li - = child.name + = link_to child.name, child_path(child) = link_to 'remove', del_parenthood_path(@parent, child), method: :delete %p Add Child: diff --git a/app/views/staff/_sub_nav.html.haml b/app/views/staff/_sub_nav.html.haml index 4ce1937..55e946f 100644 --- a/app/views/staff/_sub_nav.html.haml +++ b/app/views/staff/_sub_nav.html.haml @@ -1,2 +1 @@ -%ul.sub-nav - +-# %ul.sub-nav diff --git a/app/views/users/_sub_nav.html.haml b/app/views/users/_sub_nav.html.haml index 4ce1937..55e946f 100644 --- a/app/views/users/_sub_nav.html.haml +++ b/app/views/users/_sub_nav.html.haml @@ -1,2 +1 @@ -%ul.sub-nav - +-# %ul.sub-nav diff --git a/config/routes.rb b/config/routes.rb index 9ba9f83..1fcd5ea 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -1,16 +1,21 @@ Rails.application.routes.draw do get 'parents', to: 'parents#index', as: :list_parents - get 'parents/new', to: 'parents#new', as: :new_parent - post 'parents/new', to: 'parents#add', as: :add_parent - get 'parents/edit/:id', to: 'parents#edit', as: :edit_parent - get 'parents/:id', to: 'parents#show', as: :parent - post 'parents/:id', to: 'parents#update', as: :update_parent + get 'parent/new', to: 'parents#new', as: :new_parent + post 'parent/new', to: 'parents#add', as: :add_parent + get 'parent/edit/:id', to: 'parents#edit', as: :edit_parent + get 'parent/:id', to: 'parents#show', as: :parent + patch 'parent/:id', to: 'parents#update', as: :update_parent + post 'parenthood/child/:child', to: 'relationships#add_child', as: :add_parent_to_child post 'parenthood/:parent/', to: 'relationships#add_child', as: :add_parenthood delete 'parenthood/:parent/:child', to: 'relationships#del_child', as: :del_parenthood get 'children', to: 'children#index', as: :list_children - get 'children/lookup', to: 'children#lookup', as: :lookup_child + get 'child/new', to: 'children#new', as: :new_child + post 'child/new', to: 'children#add', as: :add_child + get 'childdit/:id', to: 'children#edit', as: :edit_child + get 'child/:id', to: 'children#show', as: :child + patch 'child/:id', to: 'children#update', as: :update_child get 'staff', to: 'staff#index', as: :list_staff