@saltcorn/history-control 0.5.0 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saltcorn/history-control",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Allow users to interact with row history",
   "main": "index.js",
   "dependencies": {

package/rowdiffview.js

@@ -23,6 +23,7 @@ const {
   button,
 } = require("@saltcorn/markup/tags");
 const { radio_group, checkbox_group } = require("@saltcorn/markup/helpers");
+const { picked_fields_to_query } = require("@saltcorn/data/plugin-helper");
 const moment = require("moment");
 
 const get_state_fields = () => [
@@ -49,7 +50,23 @@ const configuration_workflow = (req) =>
               state_fields.some((sf) => sf.name === "id")
           );
           const show_view_opts = show_views.map((v) => v.select_option);
+          let blurb;
+          if (table.provider_name)
+            blurb = [
+              div(
+                { class: "alert alert-danger fst-normal" },
+                `Use this view on a standard database table, not a provided table`
+              ),
+            ];
+          else if (!table.versioned)
+            blurb = [
+              div(
+                { class: "alert alert-danger fst-normal" },
+                `Version history is not enabled for this table`
+              ),
+            ];
           return new Form({
+            blurb,
             fields: [
               {
                 name: "show_view",
@@ -70,6 +87,11 @@ const configuration_workflow = (req) =>
                   options: show_view_opts,
                 },
               },
+              {
+                name: "comparison",
+                label: "Side-by-side comparison",
+                type: "Bool",
+              },
               {
                 name: "min_interval_secs",
                 label: "Minimum interval (s)",
@@ -91,7 +113,7 @@ const configuration_workflow = (req) =>
 const run = async (
   table_id,
   viewname,
-  { show_view, min_interval_secs, date_format },
+  { show_view, min_interval_secs, date_format, comparison },
   state,
   extraArgs
 ) => {
@@ -104,29 +126,55 @@ const run = async (
 
   let hist = await table.get_history(id);
 
+  if (!hist || !hist.length) return "No versions recorded";
+
   let last = 0;
   let last_changed_by = undefined;
-  hist = hist.filter((row) => {
-    const myEpoch = Math.round(new Date(row._time).getTime() / 1000);
-    if (
-      myEpoch - last > min_interval_secs ||
-      (row._userid && row._userid !== last_changed_by)
-    ) {
-      //include
-      last = myEpoch;
-      last_changed_by = row._userid;
-      return true;
-    } else return false;
-  });
 
-  if (!hist || !hist.length) return "No versions recorded";
-  hist = hist.reverse();
+  hist = hist
+    .filter((row) => {
+      const myEpoch = Math.round(new Date(row._time).getTime() / 1000);
+      if (
+        myEpoch - last > min_interval_secs ||
+        (row._userid && row._userid !== last_changed_by)
+      ) {
+        //include
+        last = myEpoch;
+        last_changed_by = row._userid;
+        return true;
+      } else return false;
+    })
+    .reverse();
+  const view = View.findOne({ name: show_view });
+
+  const { joinFields } = picked_fields_to_query(
+    view.configuration.columns,
+    table.fields,
+    view.configurationlayout,
+    extraArgs.req,
+    table
+  );
+  for (const [nm, jf] of Object.entries(joinFields)) {
+    const refVals = new Set(hist.map((r) => r[jf.ref]));
+    const refField = table.getField(jf.ref);
+    const reftable = Table.findOne(refField.reftable_name);
+    const rows = await reftable.getRows({
+      id: { in: [...refVals] },
+    });
+    const rowsByVal = {};
+    rows.forEach((r) => {
+      rowsByVal[r[reftable.pk_name]] = r;
+    });
+
+    hist.forEach((h) => {
+      h[nm] = rowsByVal[h[jf.ref]]?.[jf.target];
+    });
+  }
 
   const userIds = new Set(hist.map((h) => h._userid));
   const users = await User.find({ id: { in: [...userIds] } });
   const emails = {};
   users.forEach((u) => (emails[u.id] = u.email));
-  const view = View.findOne({ name: show_view });
   const rendered = await view.viewtemplateObj.renderRows(
     table,
     view.name,
@@ -169,7 +217,17 @@ const run = async (
             "aria-labelledby": `a${stateHash}head${ix}`,
             "data-bs-parent": `#top${stateHash}`,
           },
-          div({ class: ["accordion-body"] }, html)
+          div(
+            { class: ["accordion-body"] },
+            comparison && ix < rendered.length - 1
+              ? div(
+                  { class: "d-flex align-middle" },
+                  div({ class: "border p-1 m-1" }, rendered[ix + 1]),
+                  i({ class: "m-2 fas fa-arrow-right" }),
+                  div({ class: "border p-1 m-1" }, html)
+                )
+              : html
+          )
         )
       );
     })