From f0df6a80baa2e5b85b2e8f49a3f26e0589eef8d3 Mon Sep 17 00:00:00 2001 From: Aung Myo Date: Thu, 5 Jul 2018 12:50:16 +0630 Subject: [PATCH 1/3] update optimzation --- app/controllers/home_controller.rb | 1 + app/models/assigned_order_item.rb | 18 +++++---- app/models/sale.rb | 10 +++-- app/models/sale_order.rb | 12 +++--- app/views/layouts/login_dashboard.html.erb | 45 +++------------------- 5 files changed, 29 insertions(+), 57 deletions(-) diff --git a/app/controllers/home_controller.rb b/app/controllers/home_controller.rb index 09d6671e..4e0e89f3 100755 --- a/app/controllers/home_controller.rb +++ b/app/controllers/home_controller.rb @@ -20,6 +20,7 @@ class HomeController < ApplicationController def index # @employees = Employee.all_emp_except_waiter.order("name asc") @employees = Employee.all.where("is_active = true").order("name asc") + @roles = Lookup.collection_of("employee_roles") @login_form = LoginForm.new() render "layouts/login_dashboard", layout: false end diff --git a/app/models/assigned_order_item.rb b/app/models/assigned_order_item.rb index ea1b9b39..d5acc98a 100755 --- a/app/models/assigned_order_item.rb +++ b/app/models/assigned_order_item.rb @@ -8,14 +8,16 @@ class AssignedOrderItem < ApplicationRecord belongs_to :order_queue_station def self.assigned_order_item (order, item_code, instance_code, order_queue_station ) - assigned_order_item = AssignedOrderItem.new() - assigned_order_item.order = order - assigned_order_item.item_code = item_code - assigned_order_item.instance_code = instance_code - assigned_order_item.order_queue_station = order_queue_station - assigned_order_item.print_status = false - assigned_order_item.delivery_status = false - assigned_order_item.save + AssignedOrderItem.transaction do + assigned_order_item = AssignedOrderItem.new() + assigned_order_item.order = order + assigned_order_item.item_code = item_code + assigned_order_item.instance_code = instance_code + assigned_order_item.order_queue_station = order_queue_station + assigned_order_item.print_status = false + assigned_order_item.delivery_status = false + assigned_order_item.save! + end end def self.assigned_order_item_by_job(order_id) diff --git a/app/models/sale.rb b/app/models/sale.rb index 48dca4d8..574d303c 100644 --- a/app/models/sale.rb +++ b/app/models/sale.rb @@ -415,10 +415,12 @@ class Sale < ApplicationRecord shop = Shop.first #if tax is not apply create new record - SaleTax.where("sale_id='#{sale.sale_id}'").find_each do |existing_tax| - #delete existing and create new - existing_tax.delete - end + # SaleTax.where("sale_id='#{sale.sale_id}'").find_each do |existing_tax| + # #delete existing and create new + # existing_tax.delete + # end + taxes = SaleTax.where("sale_id='#{sale.sale_id}'").pluck(:sale_tax_id) + SaleTax.where("sale_tax_id IN (?)", taxes).destroy_all total_tax_amount = 0 tax_incl_exec = "exclusive" diff --git a/app/models/sale_order.rb b/app/models/sale_order.rb index 56a64353..773f2c06 100755 --- a/app/models/sale_order.rb +++ b/app/models/sale_order.rb @@ -7,11 +7,13 @@ class SaleOrder < ApplicationRecord belongs_to :sale belongs_to :order - def create_sale_order(sale, order) - sale_order = SaleOrder.new - sale_order.sale_id = sale - sale_order.order_id = order - sale_order.save! + def create_sale_order(sale, order) + SaleOrder.transaction do + sale_order = SaleOrder.new + sale_order.sale_id = sale + sale_order.order_id = order + sale_order.save! + end end private diff --git a/app/views/layouts/login_dashboard.html.erb b/app/views/layouts/login_dashboard.html.erb index 5db72f79..df2a35c2 100755 --- a/app/views/layouts/login_dashboard.html.erb +++ b/app/views/layouts/login_dashboard.html.erb @@ -112,49 +112,14 @@
From 58eb7ecb000b49d0c39081ac84f8f12f547ac8ab Mon Sep 17 00:00:00 2001 From: Jueli Moon Aung Date: Thu, 5 Jul 2018 10:55:32 +0000 Subject: [PATCH 2/3] .gitignore edited online with Bitbucket --- .gitignore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index 239ae203..df5bf8ff 100755 --- a/.gitignore +++ b/.gitignore @@ -42,7 +42,7 @@ pickle-email-*.html /config/initializers/* config/deploy/config/* config/puma.rb -#config/shops.json +config/shops.json !/log/.keep !/tmp/.keep .ruby-version From a5bb0a6746ad34406817e1b7e0aa6c34cbe89534 Mon Sep 17 00:00:00 2001 From: Aung Myo Date: Thu, 5 Jul 2018 18:19:16 +0630 Subject: [PATCH 3/3] update optimization --- app/assets/javascripts/addorder.js | 11 ++++--- .../origami/addorders_controller.rb | 7 ++++- .../origami/second_display_controller.rb | 31 ++++++++++--------- app/models/assigned_order_item.rb | 2 -- app/models/order_queue_station.rb | 4 +-- app/models/printer/order_queue_printer.rb | 4 +-- app/models/sale_order.rb | 2 -- app/views/origami/addorders/detail.html.erb | 1 + 8 files changed, 34 insertions(+), 28 deletions(-) diff --git a/app/assets/javascripts/addorder.js b/app/assets/javascripts/addorder.js index 07af7f41..d7c176ab 100755 --- a/app/assets/javascripts/addorder.js +++ b/app/assets/javascripts/addorder.js @@ -1,6 +1,7 @@ $(function() { var role = $('#role').val(); - + var second_display_lookup = $("#display_type"); + console.log(second_display_lookup) type = window.location.href.indexOf("quick_service"); modify_order = window.location.href.indexOf("modify_order"); @@ -428,7 +429,7 @@ $(function() { var items = $('.selected-set'); if (items.length >= min_qty) { - if ($('#server_mode').val() != "cloud") { + if ($('#server_mode').val() != "cloud" && second_display_lookup == 2) { item = get_set_item(items); customer_display_view(item,"set_add"); } @@ -756,7 +757,7 @@ $(function() { $('.add_to_order').attr('data-options',JSON.stringify(option_arr)); $('.add_to_order').attr('data-opt',JSON.stringify(option_arr)); var item_data = $(this); - if ($('#server_mode').val() != "cloud") { + if ($('#server_mode').val() != "cloud" && second_display_lookup == 2) { item = get_item(item_data,"add_to_order"); customer_display_view(item,"add"); } @@ -768,7 +769,7 @@ $(function() { // click plus icon for add $(document).on('click', '.add_icon', function(event){ var item_data = $(this); - if ($('#server_mode').val() != "cloud") { + if ($('#server_mode').val() != "cloud" && second_display_lookup == 2) { item = get_item(item_data,"add_icon"); customer_display_view(item,"add"); } @@ -895,7 +896,7 @@ $(function() { window.location.href = "/origami/room/" + table_id } } - if ($('#server_mode').val() != "cloud") { + if ($('#server_mode').val() != "cloud" && second_display_lookup == 2){ customer_display_view(null,"reload"); } } diff --git a/app/controllers/origami/addorders_controller.rb b/app/controllers/origami/addorders_controller.rb index 1d3bd755..025da23c 100755 --- a/app/controllers/origami/addorders_controller.rb +++ b/app/controllers/origami/addorders_controller.rb @@ -18,7 +18,12 @@ class Origami::AddordersController < BaseOrigamiController if check_mobile @webview = true end - + display_type = Lookup.find_by_lookup_type("display_type") + if !display_type.nil? && display_type.value==2 + @display_type = display_type.value + else + @display_type = nil + end # if params[:menu] == "true" @menus = [] @menu = [] diff --git a/app/controllers/origami/second_display_controller.rb b/app/controllers/origami/second_display_controller.rb index 5f88bdfc..0b1b967b 100644 --- a/app/controllers/origami/second_display_controller.rb +++ b/app/controllers/origami/second_display_controller.rb @@ -8,24 +8,27 @@ class Origami::SecondDisplayController < BaseOrigamiController end def customer_view - if params[:status]!= "billed" - tax_profiles = TaxProfile.all.order("order_by asc") - else - tax_profiles = nil - end - if ENV["SERVER_MODE"] == 'cloud' - from = request.subdomain + "." + request.domain - else - from = "" + display_type = Lookup.find_by_lookup_type("display_type") + if !display_type.nil? && display_type.value==2 + if params[:status]!= "billed" + tax_profiles = TaxProfile.all.order("order_by asc") + else + tax_profiles = nil + end + if ENV["SERVER_MODE"] == 'cloud' + from = request.subdomain + "." + request.domain + else + from = "" + end + ActionCable.server.broadcast "second_display_view_channel",data: params[:data],tax_profiles: tax_profiles,status:params[:status],from:from end - ActionCable.server.broadcast "second_display_view_channel",data: params[:data],tax_profiles: tax_profiles,status:params[:status],from:from # end end #Shop Name in Navbor - helper_method :shop_detail - def shop_detail - @shop = Shop.first - end + # helper_method :shop_detail + # def shop_detail + # @shop = Shop.first + # end end diff --git a/app/models/assigned_order_item.rb b/app/models/assigned_order_item.rb index d5acc98a..64570f7d 100755 --- a/app/models/assigned_order_item.rb +++ b/app/models/assigned_order_item.rb @@ -8,7 +8,6 @@ class AssignedOrderItem < ApplicationRecord belongs_to :order_queue_station def self.assigned_order_item (order, item_code, instance_code, order_queue_station ) - AssignedOrderItem.transaction do assigned_order_item = AssignedOrderItem.new() assigned_order_item.order = order assigned_order_item.item_code = item_code @@ -17,7 +16,6 @@ class AssignedOrderItem < ApplicationRecord assigned_order_item.print_status = false assigned_order_item.delivery_status = false assigned_order_item.save! - end end def self.assigned_order_item_by_job(order_id) diff --git a/app/models/order_queue_station.rb b/app/models/order_queue_station.rb index 19bf82be..a0372498 100755 --- a/app/models/order_queue_station.rb +++ b/app/models/order_queue_station.rb @@ -207,7 +207,7 @@ class OrderQueueStation < ApplicationRecord private #Print order_items in 1 slip def print_slip(oqs, order, order_items) - printer = PrintSetting.all + printer = PrintSetting.all.order("id ASC") unique_code="OrderSummaryPdf" if !printer.empty? @@ -245,7 +245,7 @@ class OrderQueueStation < ApplicationRecord #Print order_item in 1 slip per item def print_slip_item(oqs, order, assigned_items) - printer = PrintSetting.all + printer = PrintSetting.all.order("id ASC") unique_code="OrderItemPdf" if !printer.empty? diff --git a/app/models/printer/order_queue_printer.rb b/app/models/printer/order_queue_printer.rb index 03740f82..504dfab2 100755 --- a/app/models/printer/order_queue_printer.rb +++ b/app/models/printer/order_queue_printer.rb @@ -70,7 +70,7 @@ class Printer::OrderQueuePrinter < Printer::PrinterWorker print_settings.save! end - print_setting = PrintSetting.all + print_setting = PrintSetting.all.order("id ASC") order=print_query('order_summary', order_id) # For Print Per Item if oqs.cut_per_item @@ -170,7 +170,7 @@ class Printer::OrderQueuePrinter < Printer::PrinterWorker print_settings.save! end - print_setting = PrintSetting.all + print_setting = PrintSetting.all.order("id ASC") order=print_query('booking_summary', booking_id) # For Print Per Item diff --git a/app/models/sale_order.rb b/app/models/sale_order.rb index 773f2c06..631a5e62 100755 --- a/app/models/sale_order.rb +++ b/app/models/sale_order.rb @@ -8,12 +8,10 @@ class SaleOrder < ApplicationRecord belongs_to :order def create_sale_order(sale, order) - SaleOrder.transaction do sale_order = SaleOrder.new sale_order.sale_id = sale sale_order.order_id = order sale_order.save! - end end private diff --git a/app/views/origami/addorders/detail.html.erb b/app/views/origami/addorders/detail.html.erb index c20b53e1..bdcc9854 100644 --- a/app/views/origami/addorders/detail.html.erb +++ b/app/views/origami/addorders/detail.html.erb @@ -11,6 +11,7 @@
+