[IMP] pos_restaurant: better feedback for skipped orderlines (kitchen printing)

4 files changed, 40 insertions, 20 deletions

diff --git a/addons/pos_restaurant/static/src/css/restaurant.css b/addons/pos_restaurant/static/src/css/restaurant.css
index ec6c2f2f365..6e34f6aaa27 100644
--- a/addons/pos_restaurant/static/src/css/restaurant.css
+++ b/addons/pos_restaurant/static/src/css/restaurant.css
@@ -251,6 +251,9 @@
 .floor-map .table .order-count.notify-printing {
     background: red;
 }
+.floor-map .table .order-count.notify-skipped {
+    background: blue;
+}
 
 
 /* ------ FLOOR BUTTON IN THE ORDER SELECTOR ------- */
diff --git a/addons/pos_restaurant/static/src/js/floors.js b/addons/pos_restaurant/static/src/js/floors.js
index bbf102fc13f..e1075ab34d6 100644
--- a/addons/pos_restaurant/static/src/js/floors.js
+++ b/addons/pos_restaurant/static/src/js/floors.js
@@ -301,6 +301,8 @@ var TableWidget = PosBaseWidget.extend({
             if (orders[i].hasChangesToPrint()) {
                 notifications.printing = true;
                 break;
+            } else if (orders[i].hasSkippedChanges()) {
+                notifications.skipped  = true;
             }
         }
         return notifications;
diff --git a/addons/pos_restaurant/static/src/js/multiprint.js b/addons/pos_restaurant/static/src/js/multiprint.js
index 4f218dbdaf6..19c76a30a4c 100644
--- a/addons/pos_restaurant/static/src/js/multiprint.js
+++ b/addons/pos_restaurant/static/src/js/multiprint.js
@@ -49,13 +49,21 @@ models.load_models({
         }
 
         self.printers = [];
+        self.printers_categories = {}; // list of product categories that belong to
+                                       // one or more order printer
+
         for(var i = 0; i < printers.length; i++){
             if(active_printers[printers[i].id]){
                 var printer = new Printer(self,{url:'http://'+printers[i].proxy_ip+':8069'});
                 printer.config = printers[i];
                 self.printers.push(printer);
+
+                for (var j = 0; j < printer.config.product_categories_ids.length; j++) {
+                    self.printers_categories[printer.config.product_categories_ids[j]] = true;
+                }
             }
         }
+        self.printers_categories = _.keys(self.printers_categories);
         self.config.iface_printers = !!self.printers.length;
     },
 });
@@ -71,7 +79,10 @@ models.Orderline = models.Orderline.extend({
         if (typeof this.mp_dirty === 'undefined') {
             // mp dirty is true if this orderline has changed
             // since the last kitchen print
-            this.mp_dirty = true;
+            // it's left undefined if the orderline does not
+            // need to be printed to a printer. 
+
+            this.mp_dirty = this.printable() || undefined;
         } 
         if (!this.mp_skip) {
             // mp_skip is true if the cashier want this orderline
@@ -79,6 +90,10 @@ models.Orderline = models.Orderline.extend({
             this.mp_skip  = false;
         }
     },
+    // can this orderline be potentially printed ? 
+    printable: function() {
+        return this.pos.db.is_product_in_category(this.pos.printers_categories, this.get_product().id);
+    },
     init_from_JSON: function(json) {
         _super_orderline.init_from_JSON.apply(this,arguments);
         this.mp_dirty = json.mp_dirty;
@@ -91,7 +106,7 @@ models.Orderline = models.Orderline.extend({
         return json;
     },
     set_quantity: function(quantity) {
-        if (this.pos.config.iface_printers && quantity !== this.quantity) {
+        if (this.pos.config.iface_printers && quantity !== this.quantity && this.printable()) {
             this.mp_dirty = true;
         }
         _super_orderline.set_quantity.apply(this,arguments);
@@ -237,31 +252,19 @@ models.Order = models.Order.extend({
             // products that belong to one of the categories supplied as a parameter
 
             var self = this;
-            function product_in_category(product_id){
-                var cat = self.pos.db.get_product_by_id(product_id).pos_categ_id[0];
-                while(cat){
-                    for(var i = 0; i < categories.length; i++){
-                        if(cat === categories[i]){
-                            return true;
-                        }
-                    }
-                    cat = self.pos.db.get_category_parent_id(cat);
-                }
-                return false;
-            }
 
             var _add = [];
             var _rem = [];
             
             for(var i = 0; i < add.length; i++){
-                if(product_in_category(add[i].id)){
+                if(self.pos.db.is_product_in_category(categories,add[i].id)){
                     _add.push(add[i]);
                 }
             }
             add = _add;
 
             for(var i = 0; i < rem.length; i++){
-                if(product_in_category(rem[i].id)){
+                if(self.pos.db.is_product_in_category(categories,rem[i].id)){
                     _rem.push(rem[i]);
                 }
             }
@@ -307,6 +310,15 @@ models.Order = models.Order.extend({
         }
         return false;
     },
+    hasSkippedChanges: function() {
+        var orderlines = this.get_orderlines();
+        for (var i = 0; i < orderlines.length; i++) {
+            if (orderlines[i].mp_skip) {
+                return true;
+            }
+        }
+        return false;
+    },
     export_as_JSON: function(){
         var json = _super_order.export_as_JSON.apply(this,arguments);
         json.multiprint_resume = this.saved_resume;
@@ -341,9 +353,12 @@ screens.OrderWidget.include({
     update_summary: function(){
         this._super();
         var changes = this.pos.get_order().hasChangesToPrint();
-        if (this.getParent().action_buttons && 
-            this.getParent().action_buttons.submit_order) {
-            this.getParent().action_buttons.submit_order.highlight(changes);
+        var skipped = changes ? false : this.pos.get_order().hasSkippedChanges();
+        var buttons = this.getParent().action_buttons;
+
+        if (buttons && buttons.submit_order) {
+            buttons.submit_order.highlight(changes);
+            buttons.submit_order.altlight(skipped);
         }
     },
 });
diff --git a/addons/pos_restaurant/static/src/xml/floors.xml b/addons/pos_restaurant/static/src/xml/floors.xml
index 63a5574291e..39825fab86f 100644
--- a/addons/pos_restaurant/static/src/xml/floors.xml
+++ b/addons/pos_restaurant/static/src/xml/floors.xml
@@ -35,7 +35,7 @@
                     t-att-style='"height: " + Math.ceil(widget.fill * 100) + "%;"'
                     ></span>
                 <t t-if='widget.order_count'>
-                    <span t-att-class='"order-count " + (widget.notifications.printing ? "notify-printing":"")'><t t-esc='widget.order_count'/></span>
+                    <span t-att-class='"order-count " + (widget.notifications.printing ? "notify-printing":"") + (widget.notifications.skipped ? "notify-skipped" : "")'><t t-esc='widget.order_count'/></span>
                 </t>
                 <span class='label'>
                     <t t-esc='widget.table.name' />