Merge pull request #493 from hargata/Hargata/persist.dashboard

check against null instead of undefined,

Helper/StaticHelper.cs

@@ -8,7 +8,7 @@ namespace CarCareTracker.Helper
     /// </summary>
     public static class StaticHelper
     {
-        public static string VersionNumber = "1.3.0";
+        public static string VersionNumber = "1.3.1";
         public static string DbName = "data/cartracker.db";
         public static string UserConfigPath = "config/userConfig.json";
         public static string GenericErrorMessage = "An error occurred, please try again later";

Views/Home/_Settings.cshtml

@@ -219,7 +219,7 @@
         </p>
         <p class="lead">
             If you enjoyed using this app, please consider spreading the good word.<br />
-            If you are a commercial user, or if you just want to support the development of this project, consider subscribing to <a class="link-light link-offset-2 link-underline-opacity-25 link-underline-opacity-100-hover" href="https://www.patreon.com/LubeLogger" target="_blank">our Patreon</a> or make a <a class="link-light link-offset-2 link-underline-opacity-25 link-underline-opacity-100-hover" href="https://buy.stripe.com/aEU9Egc8DdMc9bO144" target="_blank">donation</a>
+            If you are a commercial user, or if you just want to support the development of this project, consider subscribing to <a class="link-body-emphasis link-offset-2 link-underline-opacity-25 link-underline-opacity-100-hover" href="https://www.patreon.com/LubeLogger" target="_blank">our Patreon</a> or make a <a class="link-body-emphasis link-offset-2 link-underline-opacity-25 link-underline-opacity-100-hover" href="https://buy.stripe.com/aEU9Egc8DdMc9bO144" target="_blank">donation</a>
         </p>
         <div class="d-flex justify-content-center">
             <h6 class="display-7 mt-2">Hometown Shoutout</h6>

Views/Vehicle/_Report.cshtml

@@ -146,4 +146,8 @@
         </div>
     </div>
 </div>
-<div id="vehicleHistoryReport" class="showOnPrint"></div>
+<div id="vehicleHistoryReport" class="showOnPrint"></div>
+
+<script>
+    getSelectedMetrics();
+</script>

wwwroot/js/gasrecord.js

@@ -175,28 +175,28 @@ function convertGasConsumptionUnits(currentUnit, destinationUnit, save) {
             case "l":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 3.785;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "l"));
                     sender.attr("data-unit", "l");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 3.785;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
             case "imp gal":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 1.201;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "imp gal"));
                     sender.attr("data-unit", "imp gal");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 1.201;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
@@ -206,28 +206,28 @@ function convertGasConsumptionUnits(currentUnit, destinationUnit, save) {
             case "US gal":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 3.785;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "US gal"));
                     sender.attr("data-unit", "US gal");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 3.785;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
             case "imp gal":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 4.546;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "imp gal"));
                     sender.attr("data-unit", "imp gal");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 4.546;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
@@ -237,28 +237,28 @@ function convertGasConsumptionUnits(currentUnit, destinationUnit, save) {
             case "US gal":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 1.201;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "US gal"));
                     sender.attr("data-unit", "US gal");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 1.201;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
             case "l":
                 $("[data-gas-type='consumption']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) * 4.546;
-                    elem.innerText = convertedAmount.toFixed(2);
+                    elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     sender.text(sender.text().replace(sender.attr("data-unit"), "l"));
                     sender.attr("data-unit", "l");
                 });
                 $("[data-gas-type='unitcost']").map((index, elem) => {
                     var convertedAmount = globalParseFloat(elem.innerText) / 4.546;
                     var decimalPoints = getGlobalConfig().useThreeDecimals ? 3 : 2;
-                    elem.innerText = `${getGlobalConfig().currencySymbol}${convertedAmount.toFixed(decimalPoints)}`;
+                    elem.innerText = `${getGlobalConfig().currencySymbol}${globalFloatToString(convertedAmount.toFixed(decimalPoints))}`;
                 });
                 if (save) { setDebounce(saveUserGasTabPreferences); }
                 break;
@@ -275,7 +275,7 @@ function convertFuelMileageUnits(currentUnit, destinationUnit, save) {
                     var convertedAmount = globalParseFloat(elem.innerText);
                     if (convertedAmount > 0) {
                         convertedAmount = 100 / convertedAmount;
-                        elem.innerText = convertedAmount.toFixed(2);
+                        elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     }
                 });
                 //update labels up top.
@@ -283,19 +283,19 @@ function convertFuelMileageUnits(currentUnit, destinationUnit, save) {
                 if (newAverage > 0) {
                     newAverage = 100 / newAverage;
                     var averageLabel = $("#averageFuelMileageLabel");
-                    averageLabel.text(`${averageLabel.text().split(':')[0]}: ${newAverage.toFixed(2)}`);
+                    averageLabel.text(`${averageLabel.text().split(':')[0]}: ${globalFloatToString(newAverage.toFixed(2))}`);
                 }
                 var newMin = globalParseFloat($("#minFuelMileageLabel").text().split(":")[1].trim());
                 if (newMin > 0) {
                     newMin = 100 / newMin;
                     var minLabel = $("#minFuelMileageLabel");
-                    minLabel.text(`${minLabel.text().split(':')[0]}: ${newMin.toFixed(2)}`);
+                    minLabel.text(`${minLabel.text().split(':')[0]}: ${globalFloatToString(newMin.toFixed(2))}`);
                 }
                 var newMax = globalParseFloat($("#maxFuelMileageLabel").text().split(":")[1].trim());
                 if (newMax > 0) {
                     newMax = 100 / newMax;
                     var maxLabel = $("#maxFuelMileageLabel");
-                    maxLabel.text(`${maxLabel.text().split(':')[0]}: ${newMax.toFixed(2)}`);
+                    maxLabel.text(`${maxLabel.text().split(':')[0]}: ${globalFloatToString(newMax.toFixed(2))}`);
                 }
                 sender.text(sender.text().replace(sender.attr("data-unit"), "km/l"));
                 sender.attr("data-unit", "km/l");
@@ -309,27 +309,26 @@ function convertFuelMileageUnits(currentUnit, destinationUnit, save) {
                     var convertedAmount = globalParseFloat(elem.innerText);
                     if (convertedAmount > 0) {
                         convertedAmount = 100 / convertedAmount;
-                        elem.innerText = convertedAmount.toFixed(2);
+                        elem.innerText = globalFloatToString(convertedAmount.toFixed(2));
                     }
                 });
                 var newAverage = globalParseFloat($("#averageFuelMileageLabel").text().split(":")[1].trim());
                 if (newAverage > 0) {
                     newAverage = 100 / newAverage;
-
                     var averageLabel = $("#averageFuelMileageLabel");
-                    averageLabel.text(`${averageLabel.text().split(':')[0]}: ${newAverage.toFixed(2)}`);
+                    averageLabel.text(`${averageLabel.text().split(':')[0]}: ${globalFloatToString(newAverage.toFixed(2))}`);
                 }
                 var newMin = globalParseFloat($("#minFuelMileageLabel").text().split(":")[1].trim());
                 if (newMin > 0) {
                     newMin = 100 / newMin;
                     var minLabel = $("#minFuelMileageLabel");
-                    minLabel.text(`${minLabel.text().split(':')[0]}: ${newMin.toFixed(2)}`);
+                    minLabel.text(`${minLabel.text().split(':')[0]}: ${globalFloatToString(newMin.toFixed(2))}`);
                 }
                 var newMax = globalParseFloat($("#maxFuelMileageLabel").text().split(":")[1].trim());
                 if (newMax > 0) {
                     newMax = 100 / newMax;
                     var maxLabel = $("#maxFuelMileageLabel");
-                    maxLabel.text(`${maxLabel.text().split(':')[0]}: ${newMax.toFixed(2)}`);
+                    maxLabel.text(`${maxLabel.text().split(':')[0]}: ${globalFloatToString(newMax.toFixed(2))}`);
                 }
                 sender.text(sender.text().replace(sender.attr("data-unit"), "l/100km"));
                 sender.attr("data-unit", "l/100km");
@@ -358,17 +357,17 @@ function updateMPGLabels() {
             if (!getGlobalConfig().useMPG && $("[data-gas='fueleconomy']").attr("data-unit") != 'km/l' && averageMPG > 0) {
                 averageMPG = 100 / averageMPG;
             }
-            averageLabel.text(`${averageLabel.text().split(':')[0]}: ${averageMPG.toFixed(2)}`);
+            averageLabel.text(`${averageLabel.text().split(':')[0]}: ${globalFloatToString(averageMPG.toFixed(2))}`);
         } else {
             averageLabel.text(`${averageLabel.text().split(':')[0]}: 0.00`);
         }
         if (!getGlobalConfig().useMPG && $("[data-gas='fueleconomy']").attr("data-unit") != 'km/l') {
-            maxLabel.text(`${maxLabel.text().split(':')[0]}: ${minMPG.toFixed(2)}`);
-            minLabel.text(`${minLabel.text().split(':')[0]}: ${maxMPG.toFixed(2)}`);
+            maxLabel.text(`${maxLabel.text().split(':')[0]}: ${globalFloatToString(minMPG.toFixed(2))}`);
+            minLabel.text(`${minLabel.text().split(':')[0]}: ${globalFloatToString(maxMPG.toFixed(2))}`);
         }
         else {
-            minLabel.text(`${minLabel.text().split(':')[0]}: ${minMPG.toFixed(2)}`);
-            maxLabel.text(`${maxLabel.text().split(':')[0]}: ${maxMPG.toFixed(2)}`);
+            minLabel.text(`${minLabel.text().split(':')[0]}: ${globalFloatToString(minMPG.toFixed(2))}`);
+            maxLabel.text(`${maxLabel.text().split(':')[0]}: ${globalFloatToString(maxMPG.toFixed(2))}`);
         }
     }
 }

wwwroot/js/reports.js

@@ -29,6 +29,40 @@ function refreshMPGChart() {
         $("#monthFuelMileageReportContent").html(data);
     })
 }
+function setSelectedMetrics() {
+    var selectedMetricCheckBoxes = [];
+    $(".reportCheckBox:checked").map((index, elem) => {
+        selectedMetricCheckBoxes.push(elem.id);
+    });
+    var yearMetric = $('#yearOption').val();
+    var reminderMetric = $("#reminderOption").val();
+    sessionStorage.setItem("selectedMetricCheckBoxes", JSON.stringify(selectedMetricCheckBoxes));
+    sessionStorage.setItem("yearMetric", yearMetric);
+    sessionStorage.setItem("reminderMetric", reminderMetric);
+}
+function getSelectedMetrics() {
+    var selectedMetricCheckBoxes = sessionStorage.getItem("selectedMetricCheckBoxes");
+    var yearMetric = sessionStorage.getItem("yearMetric");
+    var reminderMetric = sessionStorage.getItem("reminderMetric");
+    if (selectedMetricCheckBoxes != null && yearMetric != null && reminderMetric != null) {
+        selectedMetricCheckBoxes = JSON.parse(selectedMetricCheckBoxes);
+        $(".reportCheckBox").prop('checked', false);
+        $("#selectAllExpenseCheck").prop("checked", false);
+        selectedMetricCheckBoxes.map(x => {
+            $(`#${x}`).prop('checked', true);
+        });
+        if (selectedMetricCheckBoxes.length == 6) {
+            $("#selectAllExpenseCheck").prop("checked", true);
+        }
+        $('#yearOption').val(yearMetric);
+        $("#reminderOption").val(reminderMetric);
+        //retrieve data.
+        yearUpdated();
+        updateReminderPie();
+        return true;
+    }
+    return false;
+}
 function refreshBarChart() {
     var selectedMetrics = [];
     var vehicleId = GetVehicleId().vehicleId;
@@ -61,11 +95,13 @@ function refreshBarChart() {
         }, function (data) {
             $("#gasCostByMonthReportContent").html(data);
                 refreshMPGChart();
-        });
+    });
+    setSelectedMetrics();
 }
 function updateReminderPie() {
     var vehicleId = GetVehicleId().vehicleId;
     var daysToAdd = $("#reminderOption").val();
+    setSelectedMetrics();
     $.get(`/Vehicle/GetReminderMakeUpByVehicle?vehicleId=${vehicleId}`, { daysToAdd: daysToAdd }, function (data) {
         $("#reminderMakeUpReportContent").html(data);
     });

wwwroot/js/vehicle.js

@@ -486,13 +486,13 @@ function getRecordsDeltaStats(recordIds) {
     costSum = costSum.toFixed(2);
     Swal.fire({
         title: "Record Statistics",
-        html: `<p>Average Distance Traveled between Records: ${averageOdo}</p>
+        html: `<p>Average Distance Traveled between Records: ${globalFloatToString(averageOdo)}</p>
                 <br />
                 <p>Average Days between Records: ${averageDays}</p>
                 <br />
-                <p>Total Cost: ${getGlobalConfig().currencySymbol} ${costSum}</p>
+                <p>Total Cost: ${getGlobalConfig().currencySymbol} ${globalFloatToString(costSum)}</p>
                 <br />
-                <p>Average Cost: ${getGlobalConfig().currencySymbol} ${averageSum}</p>`
+                <p>Average Cost: ${getGlobalConfig().currencySymbol} ${globalFloatToString(averageSum)}</p>`
         ,
         icon: "info"
     });