Fix bugs

www/assets/js/map.js

@@ -45,40 +45,46 @@ $("#app").on("click", "#package-info-sheet .package-info-toggle-status", functio
             openPackageInfoSheet(packages[i], true);
             return;
         }
-    } 
+    }
 });
 
 $("#app").on("click", "#package-info-sheet #package-info-get-directions", function () {
     window.open($(this).attr("href"), "_system");
 });
 
-function openPackageInfoSheet(package, refreshOnly) {
+function openPackageInfoSheet(coordid, refreshOnly) {
     if (typeof refreshOnly == "undefined") {
         refreshOnly = false;
     }
-    
-    $("#package-info-get-directions").attr("href", "geo:" + package.coords[0] + "," + package.coords[1]);
-    $("#package-info-sheet-inner").html("");
 
-    for (var i = 0; i < package.items.length; i++) {
+    for (var i = 0; i < packages.length; i++) {
-        $("#package-info-sheet-inner").append(''
+        if (packages[i].id == coordid) {
-                + '<div class="block margin-top-half">'
+            package = packages[i];
-                + '  <div>'
+
-                + '    <span class="package-info-delivery-status">'
+            $("#package-info-get-directions").attr("href", "geo:" + package.coords[0] + "," + package.coords[1]);
-                + '      ' +  + (package.items[i].delivered ? '<span class="text-color-green"><i class="fas fa-check-circle"></i> Delivered</span>' : '<i class="fas fa-circle"></i> Not delivered')
+            $("#package-info-sheet-inner").html("");
-                + '    </span>'
+
-                + '  </div>'
+            for (var i = 0; i < package.items.length; i++) {
-                + '  <h3>'
+                $("#package-info-sheet-inner").append(''
-                + '    <span class="package-info-address">' + package.items[i].address + '</span>'
+                        + '<div class="block margin-top-half">'
-                + '  </h3>'
+                        + '  <div>'
-                + '  <div class="button button-fill margin-top" class="package-info-toggle-status" data-id="' + package.items[i].id + '" data-coordid=' + package.id + '>'
+                        + '    <span class="package-info-delivery-status">'
-                + '    ' + (package.items[i].delivered ? "Undeliver" : "Deliver")
+                        + '      ' + +(package.items[i].delivered ? '<span class="text-color-green"><i class="fas fa-check-circle"></i> Delivered</span>' : '<i class="fas fa-circle"></i> Not delivered')
-                + '  </div>'
+                        + '    </span>'
-                + '</div>');
+                        + '  </div>'
-    }
+                        + '  <h3>'
-    
+                        + '    <span class="package-info-address">' + package.items[i].address + '</span>'
-    if (!refreshOnly) {
+                        + '  </h3>'
-        app.sheet.create({el: "#package-info-sheet"}).open();
+                        + '  <div class="button button-fill margin-top" class="package-info-toggle-status" data-id="' + package.items[i].id + '" data-coordid=' + package.id + '>'
+                        + '    ' + (package.items[i].delivered ? "Undeliver" : "Deliver")
+                        + '  </div>'
+                        + '</div>');
+            }
+
+            if (!refreshOnly) {
+                app.sheet.create({el: "#package-info-sheet"}).open();
+            }
+        }
     }
 }
 

www/assets/js/map_leaflet.js

@@ -78,7 +78,7 @@ function leafletMap() {
                         icon: icon
                     })
                     .on("click", function () {
-                        openPackageInfoSheet(data[i]);
+                        openPackageInfoSheet(data[i].id);
                     });
 
             map.packagelayer.addLayer(marker);

www/assets/js/packages.js

@@ -17,7 +17,7 @@ if (localStorage.getItem("packages") != null) {
  */
 function getUndeliveredCount(address) {
     var undelivered = 0;
-    for (var i = 0; i < address.items.length; j++) {
+    for (var i = 0; i < address.items.length; i++) {
         if (!address.items[i].delivered) {
             undelivered++;
         }
@@ -28,7 +28,7 @@ function getUndeliveredCount(address) {
 function addPackage(address, latitude, longitude) {
     var added = false;
     for (var i = 0; i < packages.length; i++) {
-        if (packages[i].coords == [latitude, longitude] && packages[i].address == address) {
+        if (packages[i].coords[0] == latitude && packages[i].coords[1] == longitude && packages[i].address == address) {
             packages[i].items.push({
                 address: address,
                 delivered: false,