From aad1655f2e410c6cb971cd6cb75cba805b49a6bb Mon Sep 17 00:00:00 2001
From: "DESKTOP-T0O5CDB\\DESK-555BD" <ivancheahkf@gmail.com>
Date: Mon, 22 Jan 2024 10:52:30 -0700
Subject: [PATCH] added global parsefloat method which derives from C# culture.

---
 Views/Shared/_Layout.cshtml              |  11 +++
 Views/Vehicle/Index.cshtml               |   5 +
 Views/Vehicle/_Report.cshtml             |  10 +-
 Views/Vehicle/_ServiceRecordModal.cshtml |   2 +-
 Views/Vehicle/_SupplyStore.cshtml        |  49 +++++++++-
 Views/Vehicle/_SupplyUsage.cshtml        | 115 +++++++++++++++--------
 wwwroot/js/gasrecord.js                  |   4 +-
 wwwroot/js/reports.js                    |  10 +-
 wwwroot/js/shared.js                     |   7 ++
 wwwroot/js/supplyrecord.js               |   2 +-
 10 files changed, 154 insertions(+), 61 deletions(-)

diff --git a/Views/Shared/_Layout.cshtml b/Views/Shared/_Layout.cshtml
index 5694d37..146d012 100644
--- a/Views/Shared/_Layout.cshtml
+++ b/Views/Shared/_Layout.cshtml
@@ -6,6 +6,7 @@
     var useDarkMode = userConfig.UseDarkMode;
     var enableCsvImports = userConfig.EnableCsvImports;
     var shortDatePattern = System.Globalization.CultureInfo.CurrentCulture.DateTimeFormat.ShortDatePattern;
+    var numberFormat = System.Globalization.CultureInfo.CurrentCulture.NumberFormat;
     shortDatePattern = shortDatePattern.ToLower();
     if (!shortDatePattern.Contains("dd"))
     {
@@ -54,6 +55,16 @@
                 pattern: "@shortDatePattern"
             }
         }
+        function globalParseFloat(input){
+            //remove thousands separator.
+            var thousandSeparator = "@numberFormat.NumberGroupSeparator";
+            var decimalSeparator = "@numberFormat.NumberDecimalSeparator";
+            //strip thousands from input.
+            input = input.replace(thousandSeparator, "");
+            //convert to JS format where decimal is only separated by .
+            input = input.replace(decimalSeparator, ".");
+            return parseFloat(input);
+        }
     </script>
     @await RenderSectionAsync("Scripts", required: false)
 </head>
diff --git a/Views/Vehicle/Index.cshtml b/Views/Vehicle/Index.cshtml
index 0d7f3a5..3a3e42a 100644
--- a/Views/Vehicle/Index.cshtml
+++ b/Views/Vehicle/Index.cshtml
@@ -153,6 +153,11 @@
         </div>
     </div>
 </div>
+<div class="modal fade" data-bs-focus="false" id="inputSuppliesModal" tabindex="-1" role="dialog" aria-hidden="true" data-bs-backdrop="static" data-bs-keyboard="false">
+    <div class="modal-dialog modal-lg" role="document">
+        <div class="modal-content" id="inputSuppliesModalContent"></div>
+    </div>
+</div>
 <script>
     function GetVehicleId() {
         return { vehicleId: @Model.Id};
diff --git a/Views/Vehicle/_Report.cshtml b/Views/Vehicle/_Report.cshtml
index 4d05ac9..809c3b3 100644
--- a/Views/Vehicle/_Report.cshtml
+++ b/Views/Vehicle/_Report.cshtml
@@ -24,23 +24,23 @@
             <div class="col-md-1 d-sm-none d-md-block"></div>
             <div class="col-12 col-md-10 reportsCheckBoxContainer">
                 <div class="form-check form-check-inline">
-                    <input class="form-check-input" onChange="updateCheck(this)" type="checkbox" id="serviceExpenseCheck" value="1" checked>
+                    <input class="form-check-input" onChange="updateCheck()" type="checkbox" id="serviceExpenseCheck" value="1" checked>
                     <label class="form-check-label" for="serviceExpenseCheck">Service</label>
                 </div>
                 <div class="form-check form-check-inline">
-                    <input class="form-check-input" onChange="updateCheck(this)" type="checkbox" id="repairExpenseCheck" value="2" checked>
+                    <input class="form-check-input" onChange="updateCheck()" type="checkbox" id="repairExpenseCheck" value="2" checked>
                     <label class="form-check-label" for="repairExpenseCheck">Repairs</label>
                 </div>
                 <div class="form-check form-check-inline">
-                    <input class="form-check-input" onChange="updateCheck(this)" type="checkbox" id="upgradeExpenseCheck" value="3" checked>
+                    <input class="form-check-input" onChange="updateCheck()" type="checkbox" id="upgradeExpenseCheck" value="3" checked>
                     <label class="form-check-label" for="upgradeExpenseCheck">Upgrades</label>
                 </div>
                 <div class="form-check form-check-inline">
-                    <input class="form-check-input" onChange="updateCheck(this)" type="checkbox" id="gasExpenseCheck" value="4" checked>
+                    <input class="form-check-input" onChange="updateCheck()" type="checkbox" id="gasExpenseCheck" value="4" checked>
                     <label class="form-check-label" for="gasExpenseCheck">Gas</label>
                 </div>
                 <div class="form-check form-check-inline">
-                    <input class="form-check-input" onChange="updateCheck(this)" type="checkbox" id="taxExpenseCheck" value="5" checked>
+                    <input class="form-check-input" onChange="updateCheck()" type="checkbox" id="taxExpenseCheck" value="5" checked>
                     <label class="form-check-label" for="taxExpenseCheck">Tax</label>
                 </div>
             </div>
diff --git a/Views/Vehicle/_ServiceRecordModal.cshtml b/Views/Vehicle/_ServiceRecordModal.cshtml
index 541f68b..e43c759 100644
--- a/Views/Vehicle/_ServiceRecordModal.cshtml
+++ b/Views/Vehicle/_ServiceRecordModal.cshtml
@@ -23,7 +23,7 @@
                     <input type="text" id="serviceRecordDescription" class="form-control" placeholder="Description of item(s) serviced(i.e. Oil Change)" value="@Model.Description">
                     <label for="serviceRecordCost">Cost</label>
                     <input type="text" id="serviceRecordCost" class="form-control" placeholder="Cost of the service" value="@(isNew ? "" : Model.Cost)">
-                    @await Html.PartialAsync("_SupplyStore", "serviceRecordCost")
+                    @await Html.PartialAsync("_SupplyStore", "ServiceRecord")
                 </div>
                 <div class="col-md-6 col-12">
                     <label for="serviceRecordNotes">Notes(optional)</label>
diff --git a/Views/Vehicle/_SupplyStore.cshtml b/Views/Vehicle/_SupplyStore.cshtml
index efc0a2f..dd97f44 100644
--- a/Views/Vehicle/_SupplyStore.cshtml
+++ b/Views/Vehicle/_SupplyStore.cshtml
@@ -1,20 +1,59 @@
 @model string
 <a onclick="toggleSuppliesDiv()" class="btn btn-link">Available Supplies</a>
-<div id="inputSupplies"v style="max-height:30vh; overflow-x:hidden;"></div>
 <script>
+    resetSuppliesModal();
+    function GetCaller() {
+        return { tab: '@Model' };
+    }
+    function resetSuppliesModal() {
+        $("#inputSuppliesModalContent").html("");
+    }
+    function selectSupplies() {
+        var selectedSupplyResult = getSuppliesAndQuantity();
+        var caller = GetCaller().tab;
+        switch (caller) {
+            case "ServiceRecord":
+                $('#serviceRecordCost').val(selectedSupplyResult.totalSum);
+                break;
+        }
+        hideSuppliesModal();
+    }
+    function hideParentModal(){
+        var caller = GetCaller().tab;
+        switch (caller) {
+            case "ServiceRecord":
+                $('#serviceRecordModal').modal('hide');
+                break;
+        }
+    }
+    function showParentModal() {
+        var caller = GetCaller().tab;
+        switch (caller) {
+            case "ServiceRecord":
+                $('#serviceRecordModal').modal('show');
+                break;
+        }
+    }
     function toggleSuppliesDiv(){
-        if ($("#inputSupplies").html() != "") {
-            $("#inputSupplies").html("");
-        } else {
+        if ($("#inputSuppliesModalContent").html() == "") {
             getSupplies();
+        } else {
+            hideParentModal();
+            $('#inputSuppliesModal').modal('show');
         }
     }
     function getSupplies() {
         var vehicleId = GetVehicleId().vehicleId;
         $.get(`/Vehicle/GetSupplyRecordsForRecordsByVehicleId?vehicleId=${vehicleId}`, function (data) {
             if (data) {
-                $("#inputSupplies").html(data);
+                hideParentModal();
+                $("#inputSuppliesModalContent").html(data);
+                $('#inputSuppliesModal').modal('show');
             }
         })
     }
+    function hideSuppliesModal() {
+        $('#inputSuppliesModal').modal('hide');
+        showParentModal();
+    }
 </script>
\ No newline at end of file
diff --git a/Views/Vehicle/_SupplyUsage.cshtml b/Views/Vehicle/_SupplyUsage.cshtml
index ea08ac0..c58acdc 100644
--- a/Views/Vehicle/_SupplyUsage.cshtml
+++ b/Views/Vehicle/_SupplyUsage.cshtml
@@ -1,40 +1,57 @@
 @model List<SupplyRecord>
-<div class="row">
-    <div class="col-12">
-        <table class="table table-hover">
-            <thead class="sticky-top">
-                <tr class="d-flex">
-                    <th scope="col" class="col-1"></th>
-                    <th scope="col" class="col-3">Qty.</th>
-                    <th scope="col" class="col-2">In Stock</th>
-                    <th scope="col" class="col-4">Supply</th>
-                    <th scope="col" class="col-2">Unit Cost</th>
-                </tr>
-            </thead>
-            <tbody>
-                @foreach (SupplyRecord supplyRecord in Model)
-                {
-                    <tr class="d-flex" id="supplyRows">
-                        <td class="col-1"><input class="form-check-input" type="checkbox" onchange="toggleQuantityFieldDisabled(this)" value="@supplyRecord.Id"></td>
-                        <td class="col-3"><input type="text" disabled class="form-control"></td>
-                        <td class="col-2 supplyquantity">@supplyRecord.Quantity</td>
-                        <td class="col-4">@supplyRecord.Description</td>
-                        <td class="col-2 supplyprice">@((supplyRecord.Quantity > 0 ? supplyRecord.Cost / supplyRecord.Quantity : 0).ToString("F"))</td>
+<div class="modal-header">
+    <h5 class="modal-title">Select Supplies</h5>
+    <button type="button" class="btn-close" onclick="hideSuppliesModal()" aria-label="Close"></button>
+</div>
+<div class="modal-body">
+    <div class="row">
+        <div class="col-12">
+            <table class="table table-hover">
+                <thead class="sticky-top">
+                    <tr class="d-flex">
+                        <th scope="col" class="col-1"></th>
+                        <th scope="col" class="col-2">Quantity.</th>
+                        <th scope="col" class="col-2">In Stock</th>
+                        <th scope="col" class="col-5">Description</th>
+                        <th scope="col" class="col-2">Unit Cost</th>
                     </tr>
-                }
-            </tbody>
-        </table>
+                </thead>
+                <tbody>
+                    @foreach (SupplyRecord supplyRecord in Model)
+                    {
+                        <tr class="d-flex" id="supplyRows">
+                            <td class="col-1"><input class="form-check-input" type="checkbox" onchange="toggleQuantityFieldDisabled(this)" value="@supplyRecord.Id"></td>
+                            <td class="col-2"><input type="text" disabled onchange="recalculateTotal()" class="form-control"></td>
+                            <td class="col-2 supplyquantity">@supplyRecord.Quantity</td>
+                            <td class="col-5">@supplyRecord.Description</td>
+                            <td class="col-2 supplyprice">@((supplyRecord.Quantity > 0 ? supplyRecord.Cost / supplyRecord.Quantity : 0).ToString("F"))</td>
+                        </tr>
+                    }
+                </tbody>
+            </table>
+        </div>
     </div>
 </div>
+<div class="modal-footer">
+    <span id="supplySumLabel" style="margin-right:auto;">Total: 0.00</span>
+    <button type="button" class="btn btn-secondary" onclick="hideSuppliesModal()">Cancel</button>
+    <button type="button" class="btn btn-primary" disabled id="selectSuppliesButton" onclick="selectSupplies()">Select</button>
+</div>
 <script>
-    function toggleQuantityFieldDisabled(e){
+    function recalculateTotal() {
+        setDebounce(getSuppliesAndQuantity);
+    }
+    function toggleQuantityFieldDisabled(e) {
         var textField = getTextFieldFromCheckBox(e);
         var isChecked = $(e).is(":checked");
         textField.attr('disabled', !isChecked);
-        getSuppliesAndQuantity();
+        if (!isChecked){
+            textField.removeClass("is-invalid");
+        }
+        recalculateTotal();
     }
-    function getTextFieldFromCheckBox(elem){
-        var textField = $(elem.parentElement.parentElement).find('.col-3 > input[type=text]')[0];
+    function getTextFieldFromCheckBox(elem) {
+        var textField = $(elem.parentElement.parentElement).find('.col-2 > input[type=text]')[0];
         return $(textField);
     }
     function getInStockFieldFromCheckBox(elem) {
@@ -46,29 +63,47 @@
         return $(textField);
     }
     function getSuppliesAndQuantity() {
+        var totalSum = 0;
+        var hasError = false;
         var selectedSupplies = $("#supplyRows :checked").map(function () {
             var textField = getTextFieldFromCheckBox(this);
             var inStock = getInStockFieldFromCheckBox(this);
             var priceField = getPriceFieldFromCheckBox(this);
-            var requestedQuantity = parseFloat(textField.val());
-            var inStockQuantity = parseFloat(inStock.text());
-            var unitPrice = parseFloat(priceField.text());
+            var requestedQuantity = globalParseFloat(textField.val());
+            var inStockQuantity = globalParseFloat(inStock.text());
+            var unitPrice = globalParseFloat(priceField.text());
             //validation
-            if (isNaN(requestedQuantity) || requestedQuantity > inStockQuantity){
+            if (isNaN(requestedQuantity) || requestedQuantity > inStockQuantity) {
                 textField.addClass("is-invalid");
+                hasError = true;
             } else {
-                textField.removeClass("is-invalid");
+                textField.removeClass("is-invalid");    
             }
             //calculate sum.
-            console.log(requestedQuantity);
-            console.log(unitPrice);
             var sum = requestedQuantity * unitPrice;
-            return { 
-                supplyId: this.value, 
-                quantity: textField.val(),
-                sum: sum
+            totalSum += sum;
+            return {
+                supplyId: this.value,
+                quantity: textField.val()
             };
         });
-        return selectedSupplies.toArray();
+        if (isNaN(totalSum) || hasError) {
+            $("#supplySumLabel").text(`Total: 0.00`);
+        } else {
+            totalSum = totalSum.toFixed(2);
+            $("#supplySumLabel").text(`Total: ${totalSum}`);
+        }
+        $("#selectSuppliesButton").attr('disabled', (hasError || totalSum == 0));
+        if (!hasError) {
+            return {
+                totalSum: totalSum,
+                selectedSupplies: selectedSupplies.toArray()
+            };
+        } else {
+            return {
+                totalSum: 0,
+                selectedSupplies: []
+            }
+        }
     }
 </script>
\ No newline at end of file
diff --git a/wwwroot/js/gasrecord.js b/wwwroot/js/gasrecord.js
index 9524575..e7621e1 100644
--- a/wwwroot/js/gasrecord.js
+++ b/wwwroot/js/gasrecord.js
@@ -91,14 +91,14 @@ function getAndValidateGasRecordValues() {
     } else {
         $("#gasRecordMileage").removeClass("is-invalid");
     }
-    if (gasGallons.trim() == '' || parseFloat(gasGallons) <= 0) {
+    if (gasGallons.trim() == '' || globalParseFloat(gasGallons) <= 0) {
         hasError = true;
         $("#gasRecordGallons").addClass("is-invalid");
     } else {
         $("#gasRecordGallons").removeClass("is-invalid");
     }
     if (gasCostType != undefined && gasCostType == 'unit') {
-        var convertedGasCost = parseFloat(gasCost) * parseFloat(gasGallons);
+        var convertedGasCost = globalParseFloat(gasCost) * globalParseFloat(gasGallons);
         gasCost = convertedGasCost.toFixed(2).toString();
         if (isNaN(gasCost))
         {
diff --git a/wwwroot/js/reports.js b/wwwroot/js/reports.js
index 2a2a336..1c5bcbb 100644
--- a/wwwroot/js/reports.js
+++ b/wwwroot/js/reports.js
@@ -12,12 +12,8 @@ function generateVehicleHistoryReport() {
         }
     })
 }
-var debounce = null;
-function updateCheck(sender) {
-    clearTimeout(debounce);
-    debounce = setTimeout(function () {
-        refreshBarChart();
-    }, 1000);
+function updateCheck() {
+    setDebounce(refreshBarChart);
 }
 function refreshMPGChart() {
     var vehicleId = GetVehicleId().vehicleId;
@@ -26,7 +22,7 @@ function refreshMPGChart() {
         $("#monthFuelMileageReportContent").html(data);
     })
 }
-function refreshBarChart(callBack) {
+function refreshBarChart() {
     var selectedMetrics = [];
     var vehicleId = GetVehicleId().vehicleId;
     var year = getYear();
diff --git a/wwwroot/js/shared.js b/wwwroot/js/shared.js
index 85fe061..04904f6 100644
--- a/wwwroot/js/shared.js
+++ b/wwwroot/js/shared.js
@@ -147,4 +147,11 @@ function decodeHTMLEntities(text) {
     return $("<textarea/>")
         .html(text)
         .text();
+}
+var debounce = null;
+function setDebounce(callBack) {
+    clearTimeout(debounce);
+    debounce = setTimeout(function () {
+        callBack();
+    }, 1000);
 }
\ No newline at end of file
diff --git a/wwwroot/js/supplyrecord.js b/wwwroot/js/supplyrecord.js
index 82a1131..ba12e72 100644
--- a/wwwroot/js/supplyrecord.js
+++ b/wwwroot/js/supplyrecord.js
@@ -93,7 +93,7 @@ function getAndValidateSupplyRecordValues() {
     } else {
         $("#supplyRecordDescription").removeClass("is-invalid");
     }
-    if (supplyQuantity.trim() == '' || !isValidMoney(supplyQuantity) || parseFloat(supplyQuantity) < 0) {
+    if (supplyQuantity.trim() == '' || !isValidMoney(supplyQuantity) || globalParseFloat(supplyQuantity) < 0) {
         hasError = true;
         $("#supplyRecordQuantity").addClass("is-invalid");
     } else {