From b2b9a7a188c98a23db063a7c650d05964bffdbb8 Mon Sep 17 00:00:00 2001
From: ohsab <ohsab@github.com>
Date: Wed, 26 Nov 2014 11:58:51 +0100
Subject: [PATCH] clean up

---
 app/controllers/orders_controller.rb   |  4 ++--
 app/controllers/products_controller.rb |  4 +++-
 app/controllers/users_controller.rb    | 19 +++++++++++++++++++
 app/views/layouts/_header.html.erb     | 14 +++++++++-----
 app/views/orders/new.html.erb          | 10 +---------
 app/views/products/index.html.erb      |  4 +++-
 app/views/users/index.html.erb         |  4 +++-
 app/views/users/show.html.erb          |  3 +++
 8 files changed, 43 insertions(+), 19 deletions(-)

diff --git a/app/controllers/orders_controller.rb b/app/controllers/orders_controller.rb
index dbf0edb..0c974ce 100644
--- a/app/controllers/orders_controller.rb
+++ b/app/controllers/orders_controller.rb
@@ -5,14 +5,14 @@ class OrdersController < ApplicationController
 
 
   def new
-    @user = User.find(params[:id])
+    @user = User.find(params[:user_id])
     @order = @user.orders.build
     @products = Product.all
   end
 
 
   def create
-    @user = User.find(params[:id])
+    @user = User.find(params[:user_id])
     @order = @user.orders.build(order_params)
     if @order.save
       #flash[:success] = "order created!"
diff --git a/app/controllers/products_controller.rb b/app/controllers/products_controller.rb
index cd1d2bc..d376414 100644
--- a/app/controllers/products_controller.rb
+++ b/app/controllers/products_controller.rb
@@ -33,7 +33,9 @@ class ProductsController < ApplicationController
     end
   end
 
-  def delete
+  def destroy
+    Product.find(params[:id]).destroy
+    redirect_to products_path
   end
 
   private
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 354e835..8b43342 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -17,10 +17,29 @@ class UsersController < ApplicationController
     end
   end
 
+  def edit
+    @user = User.find(params[:id])
+  end
+
+  def update
+    @user = User.find(params[:id])
+    if @user.update_attributes(user_params)
+      redirect_to @user
+    else
+      render 'edit'
+    end
+  end
+
   def index
     @users = User.all
   end
 
+
+  def destroy
+    User.find(params[:id]).destroy
+    redirect_to users_path
+  end
+
   private
 
     def user_params
diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
index 869a955..9549175 100644
--- a/app/views/layouts/_header.html.erb
+++ b/app/views/layouts/_header.html.erb
@@ -8,7 +8,6 @@
     <!-- Collect the nav links, forms, and other content for toggling -->
     <div class="collapse navbar-collapse" id="bs-example-navbar-collapse-1">
       <ul class="nav navbar-nav">
-        <li><%= link_to "Index" , users_path %>  </li>
         <li><%= link_to "Overview" , overview_path %> </li> <!-- tijdelijk voor iedereen -->
         <li class="dropdown">
           <a href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">Products <span class="caret"></span></a>
@@ -20,15 +19,20 @@
         <li class="dropdown">
           <a href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">Users <span class="caret"></span></a>
           <ul class="dropdown-menu" role="menu">
-            <li><a href="#">new</a></li>
-            <li><a href="#">index</a></li>
-            <li><a href="#">edit</a></li>
+            <li><%= link_to "New User", new_user_path %></li>
+            <li><%= link_to "index" , users_path %></li>
           </ul>
         </li>
       </ul>
 
+      <ul class="nav navbar-nav navbar-center">
+        <li class="navbar-text">
+          ADMIN NAVBAR
+        </li>
+
+      </ul>
+
       <ul class="nav navbar-nav navbar-right">
-        <li><%= link_to "Index" , users_path %>  </li>
         <li class="dropdown">
           <a href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">Account <span class="caret"></span></a>
           <ul class="dropdown-menu" role="menu">
diff --git a/app/views/orders/new.html.erb b/app/views/orders/new.html.erb
index b484646..01be0fa 100644
--- a/app/views/orders/new.html.erb
+++ b/app/views/orders/new.html.erb
@@ -1,15 +1,7 @@
 <%= @user.name %>
 
 <% @products.each do |p| %>
-  <%#= render "orders/order_button" %>
-
-  <!-- dit moet in een andere file -->
-  <button class="btn btn-default product-btn"
-          data-name= '<%= p.name  %>'
-          date-price= '<%= p.sale_price %>'>
-    <%= p.name %>
-  </button>
-  <!-- tot hier -->
+  <%= render "orders/order_button" , p: p %>
 <% end %>
 
 <%= form_for( @order, url: user_orders_path(@user)) do |f| %>
diff --git a/app/views/products/index.html.erb b/app/views/products/index.html.erb
index 3e3c53d..4ee0122 100644
--- a/app/views/products/index.html.erb
+++ b/app/views/products/index.html.erb
@@ -3,7 +3,9 @@
 <ul class="products">
   <% @products.each do |product| %>
     <li>
-      <%= link_to product.name, product %>
+      <%= link_to product.name, product %> |
+      <%= link_to "delete", product, method: :delete,
+                                  data: { confirm: "You sure?" } %>
     </li>
   <% end %>
 </ul>
diff --git a/app/views/users/index.html.erb b/app/views/users/index.html.erb
index df651ab..e7b22da 100644
--- a/app/views/users/index.html.erb
+++ b/app/views/users/index.html.erb
@@ -3,7 +3,9 @@
 <ul class="users">
   <% @users.each do |user| %>
     <li>
-      <%= link_to user.name, user %>
+      <%= link_to user.name, user %> |
+      <%= link_to "delete", user, method: :delete,
+                                  data: { confirm: "You sure?" } %>
     </li>
   <% end %>
 </ul>
diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index 619f223..cbd8745 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -6,6 +6,9 @@
       <h1>Nickname: <%= @user.nickname %></h1>
       <h1>Balance: <%= @user.balance %></h1>
       <h1>User id: <%= @user.id %></h1>
+      <p>
+        <%= link_to "edit" , edit_user_path %>
+      </p>
 
     </section>
   </aside>