table-sort-js - npm Package Compare versions

Comparing version 1.12.0 to 1.12.1

package.json

 {
   "name": "table-sort-js",
-  "version": "1.12.0",
+  "version": "1.12.1",
   "description": "A JavaScript client-side HTML table sorting library with no dependencies required.",
@@ -5,0 +5,0 @@ "license": "MIT",

table-sort.js

@@ -180,26 +180,36 @@ /*
     function sortByRuntime(tableRows, columnData) {
-      for (let [i, tr] of tableRows.entries()) {
-        const regexMinutesAndSeconds = /^(\d+h)?\s?(\d+m)?\s?(\d+s)?$/i;
-        let columnOfTd = tr
-          .querySelectorAll("td")
-          .item(columnIndex).textContent;
-        let match = columnOfTd.match(regexMinutesAndSeconds);
-        let [minutesInSeconds, hours, seconds, timeinSeconds] = [0, 0, 0, 0];
-        if (match) {
-          const regexHours = match[1];
-          if (regexHours) {
-            hours = Number(regexHours.replace("h", "")) * 60 * 60;
+      try {
+        for (let [i, tr] of tableRows.entries()) {
+          const regexMinutesAndSeconds = /^(\d+h)?\s?(\d+m)?\s?(\d+s)?$/i;
+          let columnOfTd = "";
+          // TODO: github actions runtime didn't like textContent, tests didn't like innerText?
+          if (testingTableSortJS) {
+            columnOfTd = tr
+              .querySelectorAll("td")
+              .item(columnIndex).textContent;
+          } else {
+            columnOfTd = tr.querySelectorAll("td").item(columnIndex).innerText;
           }
-          const regexMinutes = match[2];
-          if (regexMinutes) {
-            minutesInSeconds = Number(regexMinutes.replace("m", "")) * 60;
+          let match = columnOfTd.match(regexMinutesAndSeconds);
+          let [minutesInSeconds, hours, seconds, timeinSeconds] = [0, 0, 0, 0];
+          if (match) {
+            const regexHours = match[1];
+            if (regexHours) {
+              hours = Number(regexHours.replace("h", "")) * 60 * 60;
+            }
+            const regexMinutes = match[2];
+            if (regexMinutes) {
+              minutesInSeconds = Number(regexMinutes.replace("m", "")) * 60;
+            }
+            const regexSeconds = match[3];
+            if (regexSeconds) {
+              seconds = Number(regexSeconds.replace("s", ""));
+            }
+            timeinSeconds = hours + minutesInSeconds + seconds;
           }
-          const regexSeconds = match[3];
-          if (regexSeconds) {
-            seconds = Number(regexSeconds.replace("s", ""));
-          }
-          timeinSeconds = hours + minutesInSeconds + seconds;
+          columnData.push(`${timeinSeconds}#${i}`);
+          columnIndexAndTableRow[columnData[i]] = tr.innerHTML;
         }
-        columnData.push(`${timeinSeconds}#${i}`);
-        columnIndexAndTableRow[columnData[i]] = tr.innerHTML;
+      } catch (e) {
+        console.log(e);
       }
@@ -206,0 +216,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

22062

increased by1.8%

Lines of code

408

increased by2.51%

No dependency changes