diff --git a/app/controllers/settings/set_menu_items_controller.rb b/app/controllers/settings/set_menu_items_controller.rb
index eb9ea830..438b2f08 100644
--- a/app/controllers/settings/set_menu_items_controller.rb
+++ b/app/controllers/settings/set_menu_items_controller.rb
@@ -1,12 +1,12 @@
 class Settings::SetMenuItemsController < ApplicationController
   before_action :set_settings_menu_item, only: [:show, :edit, :update, :destroy]
-  before_action :set_settings_menu_category, only: [:index, :show, :edit, :new]
+  before_action :set_settings_menu_category, only: [:index, :show, :edit, :new, :update]
   # GET /settings/menu_items
   # GET /settings/menu_items.json
   def index
     @settings_menu_items = @category.menu_items
   end
-
+ 
   # GET /settings/menu_items/1
   # GET /settings/menu_items/1.json
   def show
@@ -25,7 +25,10 @@ class Settings::SetMenuItemsController < ApplicationController
   # POST /settings/menu_items.json
   def create
     @settings_menu_item = MenuItem.new(settings_menu_item_params)
-
+    if params[:simple_menu_item][:menu_item_id] == ''
+      @settings_menu_item.menu_category_id = params[:menu_category_id]
+    end
+    @settings_menu_item.created_by = current_login_employee.name
     respond_to do |format|
       if @settings_menu_item.save
         format.html { redirect_to settings_menu_category_set_menu_items_path, notice: 'Menu item was successfully created.' }
@@ -42,7 +45,7 @@ class Settings::SetMenuItemsController < ApplicationController
   def update
     respond_to do |format|
       if @settings_menu_item.update(settings_menu_item_params)
-        format.html { redirect_to settings_menu_category_set_menu_items_path(@settings_menu_item), notice: 'Menu item was successfully updated.' }
+        format.html { redirect_to settings_menu_category_set_menu_items_path, notice: 'Menu item was successfully updated.' }
         format.json { render :show, status: :ok, location: @settings_menu_item }
       else
         format.html { render :edit }
@@ -54,9 +57,9 @@ class Settings::SetMenuItemsController < ApplicationController
   # DELETE /settings/menu_items/1
   # DELETE /settings/menu_items/1.json
   def destroy
-    @settings_menu_item.destroy
+    # @settings_menu_item.destroy
     respond_to do |format|
-      format.html { redirect_to settings_menu_items_path, notice: 'Menu item was successfully destroyed.' }
+      format.html { redirect_to settings_menu_category_set_menu_items_path, notice: 'Menu item was successfully destroyed.' }
       format.json { head :no_content }
     end
   end
@@ -73,6 +76,6 @@ class Settings::SetMenuItemsController < ApplicationController
 
     # Never trust parameters from the scary internet, only allow the white list through.
     def settings_menu_item_params
-      params.require(:menu_item).permit(:item_code, :name, :alt_name, :type, :menu_category_id, :menu_item_id, :min_qty, :min_selectable_item, :max_selectable_item, :created_by)
+      params.require(:set_menu_item).permit(:item_code, :name, :alt_name, :type, :menu_category_id, :menu_item_id, :min_qty, :min_selectable_item, :max_selectable_item, :created_by)
     end
 end
diff --git a/app/controllers/settings/simple_menu_items_controller.rb b/app/controllers/settings/simple_menu_items_controller.rb
index e3ab6714..e81d6817 100644
--- a/app/controllers/settings/simple_menu_items_controller.rb
+++ b/app/controllers/settings/simple_menu_items_controller.rb
@@ -25,7 +25,7 @@ class Settings::SimpleMenuItemsController < ApplicationController
   # POST /settings/menu_items.json
   def create
     @settings_menu_item = MenuItem.new(settings_menu_item_params)
-    if params[:menu_item_id].nil?
+    if params[:simple_menu_item][:menu_item_id] == ''
       @settings_menu_item.menu_category_id = params[:menu_category_id]
     end
     @settings_menu_item.created_by = current_login_employee.name
diff --git a/app/views/settings/set_menu_items/_form.html.erb b/app/views/settings/set_menu_items/_form.html.erb
index 27127684..bdf0e97e 100644
--- a/app/views/settings/set_menu_items/_form.html.erb
+++ b/app/views/settings/set_menu_items/_form.html.erb
@@ -1,4 +1,4 @@
-<%= simple_form_for([:setting, @settings_menu_item]) do |f| %>
+<%= simple_form_for([:settings,@category, @settings_menu_item]) do |f| %>
   <%= f.error_notification %>
 
   <div class="form-inputs">
@@ -6,12 +6,11 @@
     <%= f.input :name %>
     <%= f.input :alt_name %>
     <%= f.input :type %>
-    <%= f.association :menu_category %>
-    <%= f.association :menu_item %>
+    <%= f.input :menu_item_id, :label => "Parent Menu Item", :collection => MenuItem.collection %>
+
     <%= f.input :min_qty %>
     <%= f.input :min_selectable_item %>
     <%= f.input :max_selectable_item %>
-    <%= f.input :created_by %>
   </div>
 
   <div class="form-actions">