INV: save UTC time & display according to locale timezone (+ linting)

shelfs/models.py

@@ -6,7 +6,7 @@ from products.models import CagetteProducts
 from inventory.models import CagetteInventory
 
 import os
-from datetime import date, datetime
+from datetime import date, datetime, timezone
 from openpyxl import Workbook
 from openpyxl.styles import Alignment, Font
 from statistics import *
@@ -196,7 +196,7 @@ class Shelf(models.Model):
 
     def set_begin_inventory_datetime(self):
         res = {}
-        now = datetime.now().isoformat()
+        now = datetime.now(timezone.utc).isoformat()  # save UTC time
         f = {'ongoing_inv_start_datetime': now}
 
         try:

shelfs/static/js/shelf_inventory.js

@@ -58,12 +58,13 @@ function back() {
 
 function get_added_qties_sum(item) {
     let total = null;
+
     function add(accumulator, a) { // for array sum
         result = 0;
         if (a) {
             if (item.uom_id[1] == "kg") {
                 if (typeof a === "string") {
-                    a = a.replace(',','.')
+                    a = a.replace(',', '.');
                 }
                 result = parseFloat(accumulator) + parseFloat(a);
                 result = result.toFixed(3);
@@ -71,6 +72,7 @@ function get_added_qties_sum(item) {
                 result = parseInt(accumulator, 10) + parseInt(a, 10);
             }
         }
+
         return result;
     }
     if (typeof item.added_qties != "undefined" && item.added_qties.length > 0) {
@@ -82,6 +84,7 @@ function get_added_qties_sum(item) {
 }
 function barcode_analyzer(chars) {
     let barcode = chars;
+
     if (barcode && barcode.length >=13) {
         barcode = barcode.substring(barcode.length-13);
     } else if (barcode && barcode.length == 12 && barcode.indexOf('0') !== 0) {
@@ -125,6 +128,7 @@ function refresh() {
 function select_product_from_bc(barcode) {
     var found = null,
         qty = null;
+
     if (isValidEAN13(barcode)) {
         var scannedProduct = barcodes.get_corresponding_odoo_product(barcode);
 
@@ -155,9 +159,13 @@ function select_product_from_bc(barcode) {
                     found = e;
                     if (qty) {
                         let message = "Attention, ce produit a déjà été compté.\n";
+
                         message += "La quantité " + qty + " n'a pas été ajoutée !\n";
                         // temporary add read qty and recorded one to added_qties to compute sum
-                        found.added_qties = [qty, found.qty]
+                        found.added_qties = [
+                            qty,
+                            found.qty
+                        ];
                         message += "Le total serait " + get_added_qties_sum(found);
                         alert(message);
                         qty = null;
@@ -173,17 +181,20 @@ function select_product_from_bc(barcode) {
             setLineEdition(found, qty);
             if (editing_origin === 'to_process') {
                 let row = table_to_process.row($('tr#'+found.id));
+
                 remove_from_toProcess(row);
             } else {
                 let row = table_processed.row($('tr#'+found.id));
+
                 remove_from_processed(row);
             }
         } else {
             console.log('Code barre introuvable');
         }
-    } else if (barcode == editing_item.barcode && qty){
+    } else if (barcode == editing_item.barcode && qty) {
         // We scan the same product as the current one
         let edition_input = $('#edition_input');
+
         if (typeof editing_item.added_qties == "undefined") {
             editing_item.added_qties = [edition_input.val()];
         }
@@ -377,9 +388,10 @@ function record_products_shelf_on_server(data) {
 // call on change_shelf_btn click action
 async function open_change_shelf_modal() {
     selected_products_for_shelf_change = [];
-    $('.select_product_cb:checked').each(function(idx,elt){
+    $('.select_product_cb:checked').each(function(idx, elt) {
         const row = $(elt).closest('tr');
-        selected_products_for_shelf_change.push(table_to_process.row(row).data())
+
+        selected_products_for_shelf_change.push(table_to_process.row(row).data());
     });
     if (selected_products_for_shelf_change.length > 0) {
         /*
@@ -388,6 +400,7 @@ async function open_change_shelf_modal() {
         */
 
         let shelfs = await get_all_shelfs();
+
         if (shelfs !== null) {
             let modal_content = $('#templates #change_shelf_form').clone(),
                 shelf_selector = $('<select>').addClass('shelf_selection'),
@@ -395,12 +408,13 @@ async function open_change_shelf_modal() {
 
             /* construct shelfs selector */
             // first of all, sort by name
-            shelfs.sort((a,b) => (a.name > b.name) ? 1 : ((b.name > a.name) ? -1 : 0));
+            shelfs.sort((a, b) => (a.name > b.name) ? 1 : ((b.name > a.name) ? -1 : 0));
             // if ahead_shelfs_ids is not empty, put them ahead
             if (ahead_shelfs_ids.length > 0) {
                 let to_move = {},
                     idx = 0;
                 // find index of shelfs to move
+
                 shelfs.forEach((shelf) => {
                     if (ahead_shelfs_ids.indexOf(shelf.id) > -1) {
                         to_move[shelf.id] = idx;
@@ -410,27 +424,30 @@ async function open_change_shelf_modal() {
                 // Respecting ahead_shelfs_ids order, move shelf ahead
                 // splice can not be used, since more than 1 elt could be involved
                 let ahead_elts = [];
+
                 ahead_shelfs_ids.forEach((shelf_id) => {
                     let shelf = shelfs[to_move[shelf_id]];
+
                     ahead_elts.push(shelf);
                 });
                 //remove ahead elts
-               shelfs = shelfs.filter((item) => {return !ahead_elts.includes(item.id)});
+                shelfs = shelfs.filter((item) => {
+                    return !ahead_elts.includes(item.id);
+                });
                 // put them ahead by concatenation
                 shelfs = ahead_elts.concat(shelfs);
             }
 
-            shelfs.forEach(
-                (shelf) => {
+            shelfs.forEach((shelf) => {
                 let option = $('<option>')
                     .val(shelf.id)
                     .text(shelf.name + ' (' + shelf.sort_order + ')');
+
                 shelf_selector.append(option);
             });
             /* add product rows */
-            selected_products_for_shelf_change.forEach(
-                (product) => {
-                    let tr = $('<tr>').attr('data-id',product.id)
+            selected_products_for_shelf_change.forEach((product) => {
+                let tr = $('<tr>').attr('data-id', product.id)
                     .append($('<td>').text(product.name))
                     .append($('<td>').append(shelf_selector.clone()));
 
@@ -444,19 +461,22 @@ async function open_change_shelf_modal() {
                         make_change = async () => {
                             // Prepare data to be transmitted to server to be recorded
                             let data = [];
-                            $('.overlay-content table tbody tr').each(function(idx,e){
+
+                            $('.overlay-content table tbody tr').each(function(idx, e) {
                                 data.push({
                                     product_id : $(e).data('id'),
-                                            shelf_id : $(e).find('select').val()
+                                    shelf_id : $(e).find('select')
+                                        .val()
                                 });
                             });
                             const update_result = await record_products_shelf_on_server(data);
+
                             if (update_result !== null) {
-                                update_result.forEach(
-                                    (product_id) => {
+                                update_result.forEach((product_id) => {
                                     remove_from_toProcess(table_to_process.row($('tr#'+product_id)));
                                 });
                                 let message = "L'opération a bien réussi.";
+
                                 if (update_result.length !== data.length) {
                                     message = "L'opération a partiellement réussi.\n";
                                     message += (data.length - update_result.length) + " produit(s) non déplacé(s).";
@@ -470,14 +490,14 @@ async function open_change_shelf_modal() {
                                     }
                                 );
                             }
-                        }
+                        };
                         make_change();
                     }
                 },
                 'Changer les produits de rayons'
             );
         } else {
-            alert("Les informations des autres rayons n'ont pas pu être récupérées.")
+            alert("Les informations des autres rayons n'ont pas pu être récupérées.");
         }
 
     }
@@ -1123,6 +1143,7 @@ function init() {
         });
         if ($(this).val().length > 0) {
             let reset_icon = $("#reset_to_previous_qty");
+
             reset_icon.show();
             reset_icon.off();
             reset_icon.on("click", reset_previous_value);
@@ -1132,6 +1153,7 @@ function init() {
     })
         .on('blur', function () {
             const current_val = $(this).val();
+
             $(this).off('wheel.disableScroll');
 
             if (editing_item !== null) {
@@ -1140,13 +1162,17 @@ function init() {
                     if (typeof editing_item.added_qties !== "undefined") {
                         // total may have been affected by manual typing
                         const total = get_added_qties_sum(editing_item);
+
                         if (current_val != total) {
                             // add difference in added_qties array
                             editing_item.added_qties.push(current_val - total);
                         }
                     } else {
                         // Init added_qties to take change into account
-                        editing_item.added_qties = [editing_item.qty, current_val - editing_item.qty];
+                        editing_item.added_qties = [
+                            editing_item.qty,
+                            current_val - editing_item.qty
+                        ];
                     }
                 }
 
@@ -1214,6 +1240,7 @@ function init() {
     $(document).on('scan.pos.barcode', function(event) {
         //access `event.code` - barcode data
         var barcode = event.code;
+
         barcode_analyzer(barcode);
 
     });

shelfs/static/js/shelfs_admin.js

@@ -357,7 +357,7 @@ var is_product_in_shelf_adding_queue_list = function(testing_pid) {
 var printProduct = function () {
     let clicked = $(this);
     let tr_to_print = clicked.closest('tr');
-  let barcode = tr_to_print.data('bc')
+    let barcode = tr_to_print.data('bc');
 
     openModal();
     try {
@@ -366,8 +366,9 @@ var printProduct = function () {
             data: {'barcode': barcode}
         })
             .done(function(res) {
-        var product = res.product
-        var product_tmpl_id = product.product_tmpl_id[0]
+                var product = res.product;
+                var product_tmpl_id = product.product_tmpl_id[0];
+
                 $.ajax({
                     url: '/products/label_print/' + product_tmpl_id
                 })
@@ -379,9 +380,9 @@ var printProduct = function () {
                         } else {
                             alert('Impression lancée');
                         }
-        })
-      })
-  } catch(e) {
+                    });
+            });
+    } catch (e) {
         closeModal();
         alert('Une erreur est survenue...');
     }

shelfs/static/js/shelfs_index.js

@@ -29,6 +29,13 @@ function init_datatable() {
                     else {
                         var date = new Date(data);
 
+                        // Get local timezone offset in minutes
+                        const offset = date.getTimezoneOffset();
+
+                        // Add offset to saved time.
+                        // offset is negative if the local time zone is ahead of UTC.
+                        date.setTime(date.getTime() - offset * 60 * 1000);
+
                         return `${date.toLocaleDateString('fr-FR')} ${date.toLocaleTimeString('fr-FR')}`;
                     }
                 }