@saltcorn/server 0.8.0-beta.3 → 0.8.0

package/routes/tables.js

@@ -83,33 +83,33 @@ const tableForm = async (table, req) => {
     fields: [
       ...(!table.external
         ? [
-          {
-            label: req.__("Ownership field"),
-            name: "ownership_field_id",
-            sublabel: req.__(
-              "The user referred to in this field will be the owner of the row"
-            ),
-            input_type: "select",
-            options: [
-              { value: "", label: req.__("None") },
-              ...userFields,
-              { value: "_formula", label: req.__("Formula") },
-            ],
-          },
-          {
-            name: "ownership_formula",
-            label: req.__("Ownership formula"),
-            validator: expressionValidator,
-            type: "String",
-            class: "validate-expression",
-            sublabel:
-              req.__("User is treated as owner if true. In scope: ") +
-              ["user", ...fields.map((f) => f.name)]
-                .map((fn) => code(fn))
-                .join(", "),
-            showIf: { ownership_field_id: "_formula" },
-          },
-        ]
+            {
+              label: req.__("Ownership field"),
+              name: "ownership_field_id",
+              sublabel: req.__(
+                "The user referred to in this field will be the owner of the row"
+              ),
+              input_type: "select",
+              options: [
+                { value: "", label: req.__("None") },
+                ...userFields,
+                { value: "_formula", label: req.__("Formula") },
+              ],
+            },
+            {
+              name: "ownership_formula",
+              label: req.__("Ownership formula"),
+              validator: expressionValidator,
+              type: "String",
+              class: "validate-expression",
+              sublabel:
+                req.__("User is treated as owner if true. In scope: ") +
+                ["user", ...fields.map((f) => f.name)]
+                  .map((fn) => code(fn))
+                  .join(", "),
+              showIf: { ownership_field_id: "_formula" },
+            },
+          ]
         : []),
       // description of table
       {
@@ -129,29 +129,29 @@ const tableForm = async (table, req) => {
         name: "min_role_read",
         input_type: "select",
         options: roleOptions,
-        attributes: { asideNext: !table.external }
+        attributes: { asideNext: !table.external },
       },
       ...(table.external
         ? []
         : [
-          {
-            label: req.__("Minimum role to write"),
-            name: "min_role_write",
-            input_type: "select",
-            sublabel: req.__(
-              "User must have this role or higher to edit or create new rows in the table, unless they are the owner"
-            ),
-            options: roleOptions,
-          },
-          {
-            label: req.__("Version history"),
-            sublabel: req.__(
-              "Version history allows to track table data changes"
-            ),
-            name: "versioned",
-            type: "Bool",
-          },
-        ]),
+            {
+              label: req.__("Minimum role to write"),
+              name: "min_role_write",
+              input_type: "select",
+              sublabel: req.__(
+                "User must have this role or higher to edit or create new rows in the table, unless they are the owner"
+              ),
+              options: roleOptions,
+            },
+            {
+              label: req.__("Version history"),
+              sublabel: req.__(
+                "Version history allows to track table data changes"
+              ),
+              name: "versioned",
+              type: "Bool",
+            },
+          ]),
     ],
   });
   if (table) {
@@ -218,11 +218,11 @@ const discoverForm = (tables, req) => {
     blurb:
       tables.length > 0
         ? req.__(
-          "The following tables in your database can be imported into Saltcorn:"
-        )
+            "The following tables in your database can be imported into Saltcorn:"
+          )
         : req.__(
-          "There are no tables in the database that can be imported into Saltcorn."
-        ),
+            "There are no tables in the database that can be imported into Saltcorn."
+          ),
     submitLabel: req.__("Import"),
     fields: tables.map((t) => ({
       name: t.table_name,
@@ -328,7 +328,7 @@ router.get(
                     name: "name",
                     input_type: "text",
                   },
-                    // todo implement file mask filter like , accept: "text/csv"
+                  // todo implement file mask filter like , accept: "text/csv"
                   { label: req.__("File"), name: "file", input_type: "file" },
                 ],
               }),
@@ -516,7 +516,12 @@ const attribBadges = (f) => {
   let s = "";
   if (f.attributes) {
     Object.entries(f.attributes).forEach(([k, v]) => {
-      if (["summary_field", "default"].includes(k)) return;
+      if (
+        ["summary_field", "default", "on_delete_cascade", "on_delete"].includes(
+          k
+        )
+      )
+        return;
       if (v || v === 0) s += badge("secondary", k);
     });
   }
@@ -575,11 +580,11 @@ router.get(
             key: (r) =>
               r.type === "Key"
                 ? `Key to ` +
-                a({ href: `/table/${r.reftable_name}` }, r.reftable_name)
+                  a({ href: `/table/${r.reftable_name}` }, r.reftable_name)
                 : (r.type && r.type.name) ||
-                r.type ||
-                r.typename +
-                span({ class: "badge bg-danger ms-1" }, "Unknown type"),
+                  r.type ||
+                  r.typename +
+                    span({ class: "badge bg-danger ms-1" }, "Unknown type"),
           },
           {
             label: "",
@@ -593,23 +598,23 @@ router.get(
           ...(table.external
             ? []
             : [
-              {
-                label: req.__("Edit"),
-                key: (r) => link(`/field/${r.id}`, req.__("Edit")),
-              },
-            ]),
+                {
+                  label: req.__("Edit"),
+                  key: (r) => link(`/field/${r.id}`, req.__("Edit")),
+                },
+              ]),
           ...(table.external || db.isSQLite
             ? []
             : [
-              {
-                label: req.__("Delete"),
-                key: (r) =>
-                  (table.name === "users" && r.name === "email") ||
+                {
+                  label: req.__("Delete"),
+                  key: (r) =>
+                    (table.name === "users" && r.name === "email") ||
                     r.primary_key
-                    ? ""
-                    : post_delete_btn(`/field/delete/${r.id}`, req, r.name),
-              },
-            ]),
+                      ? ""
+                      : post_delete_btn(`/field/delete/${r.id}`, req, r.name),
+                },
+              ]),
         ],
         fields,
         { hover: true }
@@ -618,17 +623,17 @@ router.get(
         tableHtml,
         inbound_refs.length > 0
           ? req.__("Inbound keys: ") +
-          inbound_refs.map((tnm) => link(`/table/${tnm}`, tnm)).join(", ") +
-          "<br>"
+            inbound_refs.map((tnm) => link(`/table/${tnm}`, tnm)).join(", ") +
+            "<br>"
           : "",
         !table.external &&
-        a(
-          {
-            href: `/field/new/${table.id}`,
-            class: "btn btn-primary add-field mt-2",
-          },
-          req.__("Add field")
-        ),
+          a(
+            {
+              href: `/field/new/${table.id}`,
+              class: "btn btn-primary add-field mt-2",
+            },
+            req.__("Add field")
+          ),
       ];
     }
     var viewCard;
@@ -700,8 +705,8 @@ router.get(
               table.name === "users"
                 ? `/useradmin/`
                 : fields.length === 1
-                  ? `javascript:;` // Fix problem with edition of table with only one column ID / Primary Key
-                  : `/list/${table.name}`,
+                ? `javascript:;` // Fix problem with edition of table with only one column ID / Primary Key
+                : `/list/${table.name}`,
           },
           i({ class: "fas fa-2x fa-edit" }),
           "<br/>",
@@ -721,75 +726,75 @@ router.get(
         )
       ),
       !table.external &&
-      div(
-        { class: "mx-auto" },
-        form(
-          {
-            method: "post",
-            action: `/table/upload_to_table/${table.name}`,
-            encType: "multipart/form-data",
-            acceptCharset: "UTF-8",
-          },
-          input({ type: "hidden", name: "_csrf", value: req.csrfToken() }),
-          label(
-            { class: "btn-link", for: "upload_to_table" },
-            i({ class: "fas fa-2x fa-upload" }),
-            "<br/>",
-            req.__("Upload CSV")
-          ),
-          input({
-            id: "upload_to_table",
-            name: "file",
-            type: "file",
-            accept: "text/csv,.csv",
-            onchange: "this.form.submit();",
-          })
-        )
-      ),
-      // only if table is not external
-      !table.external &&
-      div(
-        { class: "mx-auto" },
-        settingsDropdown(`dataMenuButton`, [
-          a(
+        div(
+          { class: "mx-auto" },
+          form(
             {
-              class: "dropdown-item",
-              href: `/table/constraints/${table.id}`,
+              method: "post",
+              action: `/table/upload_to_table/${table.name}`,
+              encType: "multipart/form-data",
+              acceptCharset: "UTF-8",
             },
-            '<i class="fas fa-ban"></i>&nbsp;' + req.__("Constraints")
-          ),
-          // rename table doesnt supported for sqlite
-          !db.isSQLite &&
-          table.name !== "users" &&
-          a(
-            {
-              class: "dropdown-item",
-              href: `/table/rename/${table.id}`,
-            },
-            '<i class="fas fa-edit"></i>&nbsp;' + req.__("Rename table")
-          ),
-          post_dropdown_item(
-            `/table/recalc-stored/${table.name}`,
-            '<i class="fas fa-sync"></i>&nbsp;' +
-            req.__("Recalculate stored fields"),
-            req
-          ),
-          post_dropdown_item(
-            `/table/delete-all-rows/${table.name}`,
-            '<i class="far fa-trash-alt"></i>&nbsp;' +
-            req.__("Delete all rows"),
-            req,
-            true
-          ),
-          table.name !== "users" &&
-          post_dropdown_item(
-            `/table/forget-table/${table.id}`,
-            '<i class="fas fa-recycle"></i>&nbsp;' + req.__("Forget table"),
-            req,
-            true
-          ),
-        ])
-      )
+            input({ type: "hidden", name: "_csrf", value: req.csrfToken() }),
+            label(
+              { class: "btn-link", for: "upload_to_table" },
+              i({ class: "fas fa-2x fa-upload" }),
+              "<br/>",
+              req.__("Upload CSV")
+            ),
+            input({
+              id: "upload_to_table",
+              name: "file",
+              type: "file",
+              accept: "text/csv,.csv",
+              onchange: "this.form.submit();",
+            })
+          )
+        ),
+      // only if table is not external
+      !table.external &&
+        div(
+          { class: "mx-auto" },
+          settingsDropdown(`dataMenuButton`, [
+            a(
+              {
+                class: "dropdown-item",
+                href: `/table/constraints/${table.id}`,
+              },
+              '<i class="fas fa-ban"></i>&nbsp;' + req.__("Constraints")
+            ),
+            // rename table doesnt supported for sqlite
+            !db.isSQLite &&
+              table.name !== "users" &&
+              a(
+                {
+                  class: "dropdown-item",
+                  href: `/table/rename/${table.id}`,
+                },
+                '<i class="fas fa-edit"></i>&nbsp;' + req.__("Rename table")
+              ),
+            post_dropdown_item(
+              `/table/recalc-stored/${table.name}`,
+              '<i class="fas fa-sync"></i>&nbsp;' +
+                req.__("Recalculate stored fields"),
+              req
+            ),
+            post_dropdown_item(
+              `/table/delete-all-rows/${table.name}`,
+              '<i class="far fa-trash-alt"></i>&nbsp;' +
+                req.__("Delete all rows"),
+              req,
+              true
+            ),
+            table.name !== "users" &&
+              post_dropdown_item(
+                `/table/forget-table/${table.id}`,
+                '<i class="fas fa-recycle"></i>&nbsp;' + req.__("Forget table"),
+                req,
+                true
+              ),
+          ])
+        )
     );
     // add table form
     if (table.ownership_formula && !table.ownership_field_id)
@@ -812,12 +817,12 @@ router.get(
         },
         ...(fields.length > 0
           ? [
-            {
-              type: "card",
-              title: req.__("Table data"),
-              contents: dataCard,
-            },
-          ]
+              {
+                type: "card",
+                title: req.__("Table data"),
+                contents: dataCard,
+              },
+            ]
           : []),
         ...(viewCard ? [viewCard] : []),
         {
@@ -864,7 +869,7 @@ router.post(
       // todo check that works after where change
       // todo findOne can be have parameter for external table here
       //we can only save min role
-      const table = await Table.findOne( { name : v.name });
+      const table = await Table.findOne({ name: v.name });
       if (table) {
         const exttables_min_role_read = getState().getConfigCopy(
           "exttables_min_role_read",
@@ -883,13 +888,13 @@ router.post(
       const table = await Table.findOne({ id: parseInt(id) });
       const old_versioned = table.versioned;
       let hasError = false;
-      let notify = ""
+      let notify = "";
       if (!rest.versioned) rest.versioned = false;
       if (rest.ownership_field_id === "_formula") {
         rest.ownership_field_id = null;
         const fmlValidRes = expressionValidator(rest.ownership_formula);
         if (typeof fmlValidRes === "string") {
-          notify = req.__(`Invalid ownership formula: %s`, fmlValidRes)
+          notify = req.__(`Invalid ownership formula: %s`, fmlValidRes);
           hasError = true;
         }
       } else rest.ownership_formula = null;
@@ -1023,16 +1028,18 @@ router.get(
         req.__("Create from CSV upload")
       ),
       !db.isSQLite &&
-      a(
-        {
-          href: `/table/discover`,
-          class: "btn btn-secondary mt-1",
-          title: req.__("Discover tables that are already in the Database, but not known to Saltcorn"),
-        },
-        i({ class: "fas fa-map-signs me-1" }),
+        a(
+          {
+            href: `/table/discover`,
+            class: "btn btn-secondary mt-1",
+            title: req.__(
+              "Discover tables that are already in the Database, but not known to Saltcorn"
+            ),
+          },
+          i({ class: "fas fa-map-signs me-1" }),
 
-        req.__("Discover tables")
-      )
+          req.__("Discover tables")
+        )
     );
     res.sendWrap(req.__("Tables"), {
       above: [

package/routes/tag_entries.js

@@ -81,7 +81,7 @@ const formOptions = async (type, tag_id) => {
     case "triggers": {
       const ids = await tag.getTriggerIds();
       return {
-        triggers: (Trigger.find()).filter(
+        triggers: Trigger.find().filter(
           (value) => ids.indexOf(value.id) === -1
         ),
       };

package/routes/utils.js

@@ -218,7 +218,9 @@ const scan_for_page_title = (contents, viewname) => {
   try {
     scanstr =
       typeof contents === "string" ? contents : JSON.stringify(contents);
-  } catch {}
+  } catch {
+    //ignore
+  }
   if (scanstr.includes("<!--SCPT:")) {
     const start = scanstr.indexOf("<!--SCPT:");
     const end = scanstr.indexOf("-->", start);

package/routes/view.js

@@ -71,8 +71,11 @@ router.get(
         title: view.name,
         what: req.__("View"),
         url: `/viewedit/edit/${encodeURIComponent(view.name)}`,
+        cfgUrl: `/viewedit/config/${encodeURIComponent(view.name)}`,
         contents,
         req,
+        viewtemplate: view.viewtemplate,
+        table: view.table_id || view.exttable_name,
       })
     );
   })
@@ -104,13 +107,17 @@ router.post(
         if (!row) {
           if (!table)
             // todo check after where change
-            table = await Table.findOne(view.table_id ? { id : view.table_id } : {name: view.exttable_name});
+            table = await Table.findOne(
+              view.table_id
+                ? { id: view.table_id }
+                : { name: view.exttable_name }
+            );
           row = await table.getRow({});
         }
         if (row) query[sf.name] = row[sf.name];
       }
     }
-    const contents = await view.run(query, { req, res });
+    const contents = await view.run(query, { req, res, isPreview: true });
 
     res.send(contents);
   })

package/s3storage.js

@@ -5,7 +5,7 @@ const { getState } = require("@saltcorn/data/db/state");
 const fileUpload = require("express-fileupload");
 const { v4: uuidv4 } = require("uuid");
 const contentDisposition = require("content-disposition");
-
+const fs = require("fs");
 function createS3Client() {
   return new aws.S3({
     secretAccessKey: getState().getConfig("storage_s3_access_secret"),
@@ -52,19 +52,18 @@ module.exports = {
         createParentPath: true,
         tempFileDir: "/tmp/",
         // set to true - if you want to have debug
-        debug: getState().getConfig("file_upload_debug",false),
+        debug: getState().getConfig("file_upload_debug", false),
         //uriDecodeFileNames: true,
         //safeFileNames: true,
-        defCharset: 'utf8',
-        defParamCharset: 'utf8',
+        defCharset: "utf8",
+        defParamCharset: "utf8",
         // 0 - means no upload limit check
         limits: {
           fileSize: fileSizeLimit,
         },
         abortOnLimit: fileSizeLimit !== 0,
         // 0 - means no upload limit check
-        uploadTimeout: getState().getConfig("file_upload_timeout",0),
-
+        uploadTimeout: getState().getConfig("file_upload_timeout", 0),
       })(req, res, next);
     }
   },
@@ -146,7 +145,7 @@ module.exports = {
       // Forward the object
       s3.getObject(params)
         .on("httpHeaders", function (statusCode, headers) {
-          if (!!download)
+          if (download)
             res.set("Content-Disposition", contentDisposition(file.filename));
           res.set("Content-Length", headers["content-length"]);
           this.response.httpResponse.createUnbufferedStream().pipe(res);

package/serve.js

@@ -30,7 +30,11 @@ const { getConfig } = require("@saltcorn/data/models/config");
 const { migrate } = require("@saltcorn/data/migrate");
 const socketio = require("socket.io");
 const { createAdapter, setupPrimary } = require("@socket.io/cluster-adapter");
-const { setTenant, getSessionStore, get_tenant_from_req } = require("./routes/utils");
+const {
+  setTenant,
+  getSessionStore,
+  get_tenant_from_req,
+} = require("./routes/utils");
 const passport = require("passport");
 const { authenticate } = require("passport");
 const View = require("@saltcorn/data/models/view");
@@ -106,8 +110,8 @@ const workerDispatchMsg = ({ tenant, ...msg }) => {
     });
   }
   if (!getState()) {
-    console.error("no State for tenant", tenant)
-    return
+    console.error("no State for tenant", tenant);
+    return;
   }
   if (msg.refresh) getState()[`refresh_${msg.refresh}`](true);
   if (msg.createTenant) {
@@ -142,33 +146,33 @@ const workerDispatchMsg = ({ tenant, ...msg }) => {
  */
 const onMessageFromWorker =
   (masterState, { port, watchReaper, disableScheduler, pid }) =>
-    (msg) => {
-      //console.log("worker msg", typeof msg, msg);
-      if (msg === "Start" && !masterState.started) {
-        masterState.started = true;
-        runScheduler({
-          port,
-          watchReaper,
-          disableScheduler,
-          eachTenant,
-          auto_backup_now,
-          take_snapshot,
-        });
-        require("./systemd")({ port });
-        return true;
-      } else if (msg === "RestartServer") {
-        process.exit(0);
-        return true;
-      } else if (msg.tenant || msg.createTenant) {
-        ///ie from saltcorn
-        //broadcast
-        Object.entries(cluster.workers).forEach(([wpid, w]) => {
-          if (wpid !== pid) w.send(msg);
-        });
-        workerDispatchMsg(msg); //also master
-        return true;
-      }
-    };
+  (msg) => {
+    //console.log("worker msg", typeof msg, msg);
+    if (msg === "Start" && !masterState.started) {
+      masterState.started = true;
+      runScheduler({
+        port,
+        watchReaper,
+        disableScheduler,
+        eachTenant,
+        auto_backup_now,
+        take_snapshot,
+      });
+      require("./systemd")({ port });
+      return true;
+    } else if (msg === "RestartServer") {
+      process.exit(0);
+      return true;
+    } else if (msg.tenant || msg.createTenant) {
+      ///ie from saltcorn
+      //broadcast
+      Object.entries(cluster.workers).forEach(([wpid, w]) => {
+        if (wpid !== pid) w.send(msg);
+      });
+      workerDispatchMsg(msg); //also master
+      return true;
+    }
+  };
 
 module.exports =
   /**
@@ -381,7 +385,7 @@ const setupSocket = (...servers) => {
         } catch (err) {
           getState().log(1, `Socket join_room error: ${err.stack}`);
         }
-      }
+      };
       if (ten && ten !== "public") db.runWithTenant(ten, f);
       else f();
     });