@saltcorn/server 1.1.1-beta.1 → 1.1.1-beta.2

package/help/Event types.tmd

@@ -55,6 +55,8 @@ user verification is enabled.
 
 **Startup**: run this whenever this saltcorn process initializes. 
 
+**AppChange**: the application build (views, pages, triggers etc.) changed
+
 ## Other events
 
 **Never**: this trigger is never run on its own. However triggers that are marked as never

package/locales/en.json

@@ -1520,7 +1520,13 @@
 	"OK": "OK",
 	"Step settings": "Step settings",
 	"Action settings": "Action settings",
+	"Keystore file is not applied for debug builds.": "Keystore file is not applied for debug builds.",
 	"Workflow": "Workflow",
 	"Previous runs": "Previous runs",
-	"The workflow the user will be interacting with.": "The workflow the user will be interacting with."
+	"The workflow the user will be interacting with.": "The workflow the user will be interacting with.",
+	"Delete old workflow runs with status after days": "Delete old workflow runs with status after days",
+	"Finished": "Finished",
+	"Error": "Error",
+	"Waiting": "Waiting",
+	"Running": "Running"
 }

package/package.json

@@ -1,20 +1,20 @@
 {
   "name": "@saltcorn/server",
-  "version": "1.1.1-beta.1",
+  "version": "1.1.1-beta.2",
   "description": "Server app for Saltcorn, open-source no-code platform",
   "homepage": "https://saltcorn.com",
   "main": "index.js",
   "license": "MIT",
   "dependencies": {
     "@aws-sdk/client-s3": "^3.451.0",
-    "@saltcorn/base-plugin": "1.1.1-beta.1",
-    "@saltcorn/builder": "1.1.1-beta.1",
-    "@saltcorn/data": "1.1.1-beta.1",
-    "@saltcorn/admin-models": "1.1.1-beta.1",
-    "@saltcorn/filemanager": "1.1.1-beta.1",
-    "@saltcorn/markup": "1.1.1-beta.1",
-    "@saltcorn/plugins-loader": "1.1.1-beta.1",
-    "@saltcorn/sbadmin2": "1.1.1-beta.1",
+    "@saltcorn/base-plugin": "1.1.1-beta.2",
+    "@saltcorn/builder": "1.1.1-beta.2",
+    "@saltcorn/data": "1.1.1-beta.2",
+    "@saltcorn/admin-models": "1.1.1-beta.2",
+    "@saltcorn/filemanager": "1.1.1-beta.2",
+    "@saltcorn/markup": "1.1.1-beta.2",
+    "@saltcorn/plugins-loader": "1.1.1-beta.2",
+    "@saltcorn/sbadmin2": "1.1.1-beta.2",
     "@socket.io/cluster-adapter": "^0.2.1",
     "@socket.io/sticky": "^1.0.1",
     "adm-zip": "0.5.10",

package/public/saltcorn.js

@@ -65,7 +65,7 @@ function updateQueryStringParameters(uri1, kvs) {
   return uri;
 }
 
-function removeQueryStringParameter(uri1, key) {
+function removeQueryStringParameter(uri1, key, value) {
   let hash = "";
   let uri = uri1;
   if (uri && uri.includes("#")) {
@@ -73,8 +73,12 @@ function removeQueryStringParameter(uri1, key) {
     hash = "#" + uris[1];
     uri = uris[0];
   }
-
-  var re = new RegExp("([?&])" + key + "=.*?(&|$)", "gi");
+  let re;
+  if (value) {
+    re = new RegExp("([?&])" + key + "=" + value + "?(&|$)", "gi");
+  } else {
+    re = new RegExp("([?&])" + key + "=.*?(&|$)", "gi");
+  }
   if (uri.match(re)) {
     uri = uri.replace(re, "$1" + "$2");
   }
@@ -84,6 +88,28 @@ function removeQueryStringParameter(uri1, key) {
   return uri + hash;
 }
 
+function addQueryStringParameter(uri1, key, value) {
+  let hash = "";
+  let uri = uri1;
+  if (uri && uri.includes("#")) {
+    let uris = uri1.split("#");
+    hash = "#" + uris[1];
+    uri = uris[0];
+  }
+  var re = new RegExp("([?&])" + key + "=" + value + "?(&|$)", "gi");
+  if (uri.match(re)) return uri1;
+
+  var separator = uri.indexOf("?") !== -1 ? "&" : "?";
+  if (Array.isArray(value))
+    return (
+      uri +
+      separator +
+      value.map((val) => key + "=" + encodeURIComponent(val)).join("&") +
+      hash
+    );
+  else return uri + separator + key + "=" + encodeURIComponent(value) + hash;
+}
+
 function select_id(id, e) {
   pjax_to(updateQueryStringParameter(get_current_state_url(e), "id", id), e);
 }
@@ -96,10 +122,10 @@ function check_state_field(that, e) {
   const checked = that.checked;
   const name = that.name;
   const value = encodeURIComponent(that.value);
-  var separator = get_current_state_url(e).indexOf("?") !== -1 ? "&" : "?";
   let dest;
-  if (checked) dest = get_current_state_url(e) + `${separator}${name}=${value}`;
-  else dest = get_current_state_url(e).replace(`${name}=${value}`, "");
+  if (checked)
+    dest = addQueryStringParameter(get_current_state_url(e), name, value);
+  else dest = removeQueryStringParameter(get_current_state_url(e), name, value);
   pjax_to(dest.replace("&&", "&").replace("?&", "?"), e);
 }
 
@@ -591,7 +617,7 @@ function updateViewPreview() {
 function ajaxSubmitForm(e, force_no_reload, event) {
   var form = $(e).closest("form");
   var url = form.attr("action");
-  if(event) event.preventDefault();
+  if (event) event.preventDefault();
   $.ajax(url, {
     type: "POST",
     headers: {
@@ -606,10 +632,10 @@ function ajaxSubmitForm(e, force_no_reload, event) {
         "data-on-close-reload-view"
       );
       $("#scmodal").modal("hide");
-      if (!force_no_reload && on_close_reload_view) {
+      if (on_close_reload_view) {
         const viewE = $(`[data-sc-embed-viewname="${on_close_reload_view}"]`);
         if (viewE.length) reload_embedded_view(on_close_reload_view);
-        else location.reload();
+        else if (!force_no_reload) location.reload();
       } else if (!force_no_reload && !no_reload) location.reload();
       else common_done(res, form.attr("data-viewname"));
     },
@@ -859,10 +885,16 @@ function builderMenuChanged(e) {
   });
 }
 
-function poll_mobile_build_finished(outDirName, pollCount, orginalBtnHtml) {
+function poll_mobile_build_finished(
+  outDirName,
+  buildDir,
+  mode,
+  pollCount,
+  orginalBtnHtml
+) {
   $.ajax("/admin/build-mobile-app/finished", {
     type: "GET",
-    data: { build_dir: outDirName },
+    data: { out_dir_name: outDirName, mode: mode },
     success: function (res) {
       if (!res.finished) {
         if (pollCount >= 100) {
@@ -873,14 +905,45 @@ function poll_mobile_build_finished(outDirName, pollCount, orginalBtnHtml) {
           });
         } else {
           setTimeout(() => {
-            poll_mobile_build_finished(outDirName, ++pollCount, orginalBtnHtml);
+            poll_mobile_build_finished(
+              outDirName,
+              buildDir,
+              mode,
+              ++pollCount,
+              orginalBtnHtml
+            );
           }, 5000);
         }
       } else {
         href_to(
-          `build-mobile-app/result?build_dir_name=${encodeURIComponent(
+          `/admin/build-mobile-app/result?out_dir_name=${encodeURIComponent(
             outDirName
-          )}`
+          )}&build_dir=${encodeURIComponent(buildDir)}&mode=${mode}`
+        );
+      }
+    },
+  });
+}
+
+function finish_mobile_app(button, outDirName, buildDir) {
+  $.ajax("/admin/build-mobile-app/finish", {
+    type: "POST",
+    headers: {
+      "CSRF-Token": _sc_globalCsrf,
+    },
+    data: { out_dir_name: outDirName, build_dir: buildDir },
+    success: function (data) {
+      if (data.success) {
+        notifyAlert("Finishing the app, please wait.", true);
+        for (const msg of data.msgs || []) notifyAlert(msg);
+        const orginalBtnHtml = $("#finishMobileAppBtnId").html();
+        press_store_button(button);
+        poll_mobile_build_finished(
+          outDirName,
+          buildDir,
+          "finish",
+          0,
+          orginalBtnHtml
         );
       }
     },
@@ -938,11 +1001,18 @@ function build_mobile_app(button) {
   ajax_post("/admin/build-mobile-app", {
     data: params,
     success: (data) => {
-      if (data.build_dir_name) {
-        handleMessages();
+      if (data.out_dir_name && data.build_dir) {
+        notifyAlert("Building the app, please wait.", true);
+        for (const msg of data.msgs || []) notifyAlert(msg);
         const orginalBtnHtml = $("#buildMobileAppBtnId").html();
         press_store_button(button);
-        poll_mobile_build_finished(data.build_dir_name, 0, orginalBtnHtml);
+        poll_mobile_build_finished(
+          data.out_dir_name,
+          data.build_dir,
+          data.mode,
+          0,
+          orginalBtnHtml
+        );
       }
     },
   });

package/routes/actions.js

@@ -14,6 +14,10 @@ const {
 const { ppVal, jsIdentifierValidator } = require("@saltcorn/data/utils");
 const { getState } = require("@saltcorn/data/db/state");
 const Trigger = require("@saltcorn/data/models/trigger");
+const View = require("@saltcorn/data/models/view");
+const {
+  getForm,
+} = require("@saltcorn/data/base-plugin/viewtemplates/viewable_fields");
 const FieldRepeat = require("@saltcorn/data/models/fieldrepeat");
 const { getTriggerList } = require("./common_lists");
 const TagEntry = require("@saltcorn/data/models/tag_entry");
@@ -441,6 +445,10 @@ router.post(
         const tr = await Trigger.create(form.values);
         id = tr.id;
       }
+      Trigger.emitEvent("AppChange", `Trigger ${form.values.name}`, req.user, {
+        entity_type: "Trigger",
+        entity_name: form.values.name,
+      });
       res.redirect(addOnDoneRedirect(`/actions/configure/${id}`, req));
     }
   })
@@ -482,6 +490,10 @@ router.post(
           ...form.values.configuration,
         };
       await Trigger.update(trigger.id, form.values); //{configuration: form.values});
+      Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+        entity_type: "Trigger",
+        entity_name: trigger.name,
+      });
       if (req.xhr) {
         res.json({ success: "ok" });
         return;
@@ -533,9 +545,11 @@ function genWorkflowDiagram(steps) {
     }
     if (step.action_name === "ForLoop") {
       linkLines.push(
-        `  ${step.mmname}-.->${WorkflowStep.mmescape(step.configuration.loop_body_initial_step)}`
+        `  ${step.mmname}-.->${WorkflowStep.mmescape(
+          step.configuration.loop_body_initial_step
+        )}`
       );
-    } 
+    }
     if (step.action_name === "EndForLoop") {
       // TODO this is not correct. improve.
       let forStep;
@@ -711,10 +725,20 @@ const getWorkflowStepForm = async (
             },
           };
         if (cfgFld.input_type === "code") cfgFld.input_type = "textarea";
-        actionConfigFields.push(cfgFld);
       }
     } catch {}
   }
+  actionConfigFields.push({
+    label: "Subcontext",
+    name: "subcontext",
+    type: "String",
+    sublabel:
+      "Optional. A key on the current workflow's context, the values of which will be the called workflow's context.",
+    showIf: {
+      wf_action_name: Trigger.find({ action: "Workflow" }).map((wf) => wf.name),
+    },
+  });
+
   const builtInActionExplainers = WorkflowStep.builtInActionExplainers();
   const actionsNotRequiringRow = Trigger.action_options({
     notRequireRow: true,
@@ -730,7 +754,9 @@ const getWorkflowStepForm = async (
     if (tr.description) actionExplainers[tr.name] = tr.description;
   });
   Object.assign(actionExplainers, builtInActionExplainers);
-  actionConfigFields.push(...(await WorkflowStep.builtInActionConfigFields()));
+  actionConfigFields.push(
+    ...(await WorkflowStep.builtInActionConfigFields({ trigger }))
+  );
 
   const form = new Form({
     action: addOnDoneRedirect(`/actions/stepedit/${trigger.id}`, req),
@@ -1142,6 +1168,10 @@ router.post(
       await Trigger.update(trigger.id, {
         configuration: { ...trigger.configuration, ...form.values },
       });
+      Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+        entity_type: "Trigger",
+        entity_name: trigger.name,
+      });
       if (req.xhr) {
         res.json({ success: "ok" });
         return;
@@ -1168,6 +1198,10 @@ router.post(
   error_catcher(async (req, res) => {
     const { id } = req.params;
     const trigger = await Trigger.findOne({ id });
+    Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+      entity_type: "Trigger",
+      entity_name: trigger.name,
+    });
     await trigger.delete();
     res.redirect(`/actions/`);
   })
@@ -1296,6 +1330,10 @@ router.post(
     const { id } = req.params;
     const trig = await Trigger.findOne({ id });
     const newtrig = await trig.clone();
+    Trigger.emitEvent("AppChange", `Trigger ${newtrig.name}`, req.user, {
+      entity_type: "Trigger",
+      entity_name: newtrig.name,
+    });
     req.flash(
       "success",
       req.__("Trigger %s duplicated as %s", trig.name, newtrig.name)
@@ -1425,6 +1463,10 @@ router.post(
           res.redirect(`/actions/configure/${step.trigger_id}`);
         }
       }
+      Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+        entity_type: "Trigger",
+        entity_name: trigger.name,
+      });
       if (_after_step && _after_step !== "undefined") {
         const astep = await WorkflowStep.findOne({
           id: _after_step,
@@ -1465,6 +1507,10 @@ router.post(
       step.trigger_id = trigger.id;
       await WorkflowStep.create(step);
     }
+    Trigger.emitEvent("AppChange", `Trigger ${trigger.name}`, req.user, {
+      entity_type: "Trigger",
+      entity_name: trigger.name,
+    });
     res.redirect(`/actions/configure/${trigger.id}`);
   })
 );
@@ -1687,6 +1733,21 @@ router.post(
 );
 
 const getWorkflowStepUserForm = async (run, trigger, step, req) => {
+  if (step.action_name === "EditViewForm") {
+    const view = View.findOne({ name: step.configuration.edit_view });
+    const table = Table.findOne({ id: view.table_id });
+    const form = await getForm(
+      table,
+      view.name,
+      view.configuration.columns,
+      view.configuration.layout,
+      null,
+      req
+    );
+    form.action = `/actions/fill-workflow-form/${run.id}`;
+    return form;
+  }
+
   let blurb = run.wait_info.output || step.configuration?.form_header || "";
   if (run.wait_info.markdown && run.wait_info.output) blurb = md.render(blurb);
   const form = new Form({
@@ -1819,12 +1880,10 @@ WORKFLOWS TODO
 
 help file to explain steps, and context
 
-workflow actions: ForLoop, EndForLoop, ReadFile, WriteFile, APIResponse
+workflow actions: Stop, RunEditView, ReadFile, WriteFile, APIResponse
 
-Error handlers
 other triggers can be steps
 interactive workflows for not logged in
-show end node in diagram
 actions can declare which variables they inject into scope
 
 show unconnected steps

package/routes/admin.js

@@ -19,6 +19,7 @@ const Plugin = require("@saltcorn/data/models/plugin");
 const File = require("@saltcorn/data/models/file");
 const { spawn, exec } = require("child_process");
 const User = require("@saltcorn/data/models/user");
+const Trigger = require("@saltcorn/data/models/trigger");
 const path = require("path");
 const { X509Certificate } = require("crypto");
 const { getAllTenants } = require("@saltcorn/admin-models/models/tenant");
@@ -146,6 +147,24 @@ const app_files_table = (files, buildDirName, req) =>
     ],
     files
   );
+const intermediate_build_result = (outDirName, buildDir, req) => {
+  return div(
+    h3("Intermediate build result"),
+    div(
+      button(
+        {
+          id: "finishMobileAppBtnId",
+          type: "button",
+          onClick: `finish_mobile_app(this, '${outDirName}', '${buildDir}');`,
+          class: "btn btn-warning",
+        },
+        i({ class: "fas fa-hammer pe-2" }),
+
+        req.__("Finish the build")
+      )
+    )
+  );
+};
 
 admin_config_route({
   router,
@@ -1993,9 +2012,6 @@ const buildDialogScript = (capacitorBuilderAvailable, isSbadmin2) =>
     $("#entryPointTypeID").attr("value", type);
   }
   
-  function handleMessages() {
-    notifyAlert("Building the app, please wait.", true)
-  }
   const versionPattern = /^\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}$/;
   ${domReady(`
   const versionInput = document.getElementById('appVersionInputId');
@@ -3077,6 +3093,48 @@ router.get(
                           )
                         )
                       )
+                      // Share Extension provisioning profile
+                      // disabled for now
+                      // div(
+                      //   { class: "row pb-3" },
+                      //   div(
+                      //     { class: "col-sm-8" },
+                      //     label(
+                      //       {
+                      //         for: "shareProvisioningProfileInputId",
+                      //         class: "form-label fw-bold",
+                      //       },
+                      //       req.__("Share Extension Provisioning Profile"),
+                      //       a(
+                      //         {
+                      //           href: "javascript:ajax_modal('/admin/help/Provisioning Profile?')",
+                      //         },
+                      //         i({ class: "fas fa-question-circle ps-1" })
+                      //       )
+                      //     ),
+                      //     select(
+                      //       {
+                      //         class: "form-select",
+                      //         name: "shareProvisioningProfile",
+                      //         id: "shareProvisioningProfileInputId",
+                      //       },
+                      //       [
+                      //         option({ value: "" }, ""),
+                      //         ...provisioningFiles.map((file) =>
+                      //           option(
+                      //             {
+                      //               value: file.location,
+                      //               selected:
+                      //                 builderSettings.shareProvisioningProfile ===
+                      //                 file.location,
+                      //             },
+                      //             file.filename
+                      //           )
+                      //         ),
+                      //       ].join("")
+                      //     )
+                      //   )
+                      // )
                     )
                   )
                 ),
@@ -3100,15 +3158,13 @@ router.get(
   })
 );
 
-const checkFiles = async (outDir, fileNames) => {
+const checkFiles = async (outDirName, fileNames) => {
   const rootFolder = await File.rootFolder();
-  const mobile_app_dir = path.join(rootFolder.location, "mobile_app", outDir);
+  const outDir = path.join(rootFolder.location, "mobile_app", outDirName);
   const unsafeFiles = await Promise.all(
     fs
-      .readdirSync(mobile_app_dir)
-      .map(
-        async (outFile) => await File.from_file_on_disk(outFile, mobile_app_dir)
-      )
+      .readdirSync(outDir)
+      .map(async (outFile) => await File.from_file_on_disk(outFile, outDir))
   );
   const entries = unsafeFiles
     .filter(
@@ -3127,9 +3183,18 @@ router.get(
   "/build-mobile-app/finished",
   isAdmin,
   error_catcher(async (req, res) => {
-    const { build_dir } = req.query;
+    const { out_dir_name, mode } = req.query;
+    const stepDesc =
+      mode === "prepare"
+        ? "_prepare_step"
+        : mode === "finish"
+        ? "_finish_step"
+        : "";
     res.json({
-      finished: await checkFiles(build_dir, ["logs.txt", "error_logs.txt"]),
+      finished: await checkFiles(out_dir_name, [
+        `logs${stepDesc}.txt`,
+        `error_logs${stepDesc}.txt`,
+      ]),
     });
   })
 );
@@ -3164,8 +3229,8 @@ router.get(
   "/build-mobile-app/result",
   isAdmin,
   error_catcher(async (req, res) => {
-    const { build_dir_name } = req.query;
-    if (!validateBuildDirName(build_dir_name)) {
+    const { out_dir_name, build_dir, mode } = req.query;
+    if (!validateBuildDirName(out_dir_name)) {
       return res.sendWrap(req.__(`Admin`), {
         above: [
           {
@@ -3177,11 +3242,7 @@ router.get(
       });
     }
     const rootFolder = await File.rootFolder();
-    const buildDir = path.join(
-      rootFolder.location,
-      "mobile_app",
-      build_dir_name
-    );
+    const buildDir = path.join(rootFolder.location, "mobile_app", out_dir_name);
     if (!validateBuildDir(buildDir, rootFolder.location)) {
       return res.sendWrap(req.__(`Admin`), {
         above: [
@@ -3199,7 +3260,15 @@ router.get(
         .readdirSync(buildDir)
         .map(async (outFile) => await File.from_file_on_disk(outFile, buildDir))
     );
-    const resultMsg = files.find((file) => file.filename === "logs.txt")
+    const stepDesc =
+      mode === "prepare"
+        ? "_prepare_step"
+        : mode === "finish"
+        ? "_finish_step"
+        : "";
+    const resultMsg = files.find(
+      (file) => file.filename === `logs${stepDesc}.txt`
+    )
       ? req.__("The build was successfully")
       : req.__("Unable to build the app");
     res.sendWrap(req.__(`Admin`), {
@@ -3209,11 +3278,98 @@ router.get(
           title: req.__("Build Result"),
           contents: div(resultMsg),
         },
-        files.length > 0 ? app_files_table(files, build_dir_name, req) : "",
+        files.length > 0 ? app_files_table(files, out_dir_name, req) : "",
+        mode === "prepare"
+          ? intermediate_build_result(out_dir_name, build_dir, req)
+          : "",
       ],
     });
   })
 );
+
+router.post(
+  "/build-mobile-app/finish",
+  isAdmin,
+  error_catcher(async (req, res) => {
+    const { out_dir_name, build_dir } = req.body;
+    const content = await fs.promises.readFile(
+      path.join(build_dir, "spawnParams.json")
+    );
+    const spawnParams = JSON.parse(content);
+    const rootFolder = await File.rootFolder();
+    const outDirFullPath = path.join(
+      rootFolder.location,
+      "mobile_app",
+      out_dir_name
+    );
+    res.json({
+      success: true,
+    });
+    const child = spawn(
+      getSafeSaltcornCmd(),
+      [...spawnParams, "-m", "finish"],
+      {
+        stdio: ["ignore", "pipe", "pipe"],
+        cwd: ".",
+      }
+    );
+    const childOutputs = [];
+    child.stdout.on("data", (data) => {
+      const outMsg = data.toString();
+      getState().log(5, outMsg);
+      if (data) childOutputs.push(outMsg);
+    });
+    child.stderr.on("data", (data) => {
+      const errMsg = data ? data.toString() : req.__("An error occurred");
+      getState().log(5, errMsg);
+      childOutputs.push(errMsg);
+    });
+    child.on("exit", async (exitCode, signal) => {
+      const logFile =
+        exitCode === 0 ? "logs_finish_step.txt" : "error_logs_finish_step.txt";
+      try {
+        const exitMsg = childOutputs.join("\n");
+        await fs.promises.writeFile(
+          path.join(outDirFullPath, logFile),
+          exitMsg
+        );
+        await File.set_xattr_of_existing_file(
+          logFile,
+          outDirFullPath,
+          req.user
+        );
+      } catch (error) {
+        console.log(`unable to write '${logFile}' to '${outDirFullPath}'`);
+        console.log(error);
+      }
+    });
+    child.on("error", (msg) => {
+      const message = msg.message ? msg.message : msg.code;
+      const stack = msg.stack ? msg.stack : "";
+      const logFile = "error_logs.txt";
+      const errMsg = [message, stack].join("\n");
+      getState().log(5, msg);
+      fs.writeFile(
+        path.join(outDirFullPath, "error_logs.txt"),
+        errMsg,
+        async (error) => {
+          if (error) {
+            console.log(`unable to write logFile to '${outDirFullPath}'`);
+            console.log(error);
+          } else {
+            // no transaction, '/build-mobile-app/finished' filters for valid attributes
+            await File.set_xattr_of_existing_file(
+              logFile,
+              outDirFullPath,
+              req.user
+            );
+          }
+        }
+      );
+    });
+  })
+);
+
 /**
  * Do Build Mobile App
  */
@@ -3222,6 +3378,8 @@ router.post(
   isAdmin,
   error_catcher(async (req, res) => {
     getState().log(2, `starting mobile build: ${JSON.stringify(req.body)}`);
+    const msgs = [];
+    let mode = "full";
     let {
       entryPoint,
       entryPointType,
@@ -3239,11 +3397,27 @@ router.post(
       synchedTables,
       includedPlugins,
       provisioningProfile,
+      shareProvisioningProfile,
       buildType,
       keystoreFile,
       keystoreAlias,
       keystorePassword,
     } = req.body;
+    // const receiveShareTriggers = Trigger.find({
+    //   when_trigger: "ReceiveMobileShareData",
+    // });
+    // disabeling share to support for now
+    let allowShareTo = false; // receiveShareTriggers.length > 0;
+    if (allowShareTo && iOSPlatform && !shareProvisioningProfile) {
+      allowShareTo = false;
+      msgs.push({
+        type: "warning",
+        text: req.__(
+          "A ReceiveMobileShareData trigger exists, but no Share Extension Provisioning Profile is provided. " +
+            "Building without share to support."
+        ),
+      });
+    }
     if (!includedPlugins) includedPlugins = [];
     if (!synchedTables) synchedTables = [];
     if (!entryPoint) {
@@ -3279,14 +3453,26 @@ router.post(
         ),
       });
     }
-    if (iOSPlatform && !provisioningProfile) {
-      return res.json({
-        error: req.__(
-          "Please provide a Provisioning Profile for the iOS build."
-        ),
+    if (iOSPlatform) {
+      if (!provisioningProfile)
+        return res.json({
+          error: req.__(
+            "Please provide a Provisioning Profile for the iOS build."
+          ),
+        });
+    }
+    if (buildType === "debug" && keystoreFile) {
+      msgs.push({
+        type: "warning",
+        text: req.__("Keystore file is not applied for debug builds."),
       });
     }
-    if (keystoreFile && (!keystoreAlias || !keystorePassword)) {
+
+    if (
+      buildType === "release" &&
+      keystoreFile &&
+      (!keystoreAlias || !keystorePassword)
+    ) {
       return res.json({
         error: req.__(
           "Please provide the keystore alias and password for the android build."
@@ -3294,8 +3480,9 @@ router.post(
       });
     }
     const outDirName = `build_${new Date().valueOf()}`;
+    const buildDir = `${os.userInfo().homedir}/mobile_app_build`;
     const rootFolder = await File.rootFolder();
-    const buildDir = path.join(rootFolder.location, "mobile_app", outDirName);
+    const outDir = path.join(rootFolder.location, "mobile_app", outDirName);
     await File.new_folder(outDirName, "/mobile_app");
     const spawnParams = [
       "build-app",
@@ -3304,9 +3491,9 @@ router.post(
       "-t",
       entryPointType === "pagegroup" ? "page" : entryPointType,
       "-c",
-      buildDir,
+      outDir,
       "-b",
-      `${os.userInfo().homedir}/mobile_app_build`,
+      buildDir,
       "-u",
       req.user.email, // ensured by isAdmin
     ];
@@ -3319,6 +3506,13 @@ router.post(
         "--provisioningProfile",
         provisioningProfile
       );
+      if (allowShareTo) {
+        mode = "prepare";
+        spawnParams.push(
+          "--shareExtensionProvisioningProfile",
+          shareProvisioningProfile
+        );
+      }
     }
     if (appName) spawnParams.push("--appName", appName);
     if (appId) spawnParams.push("--appId", appId);
@@ -3327,6 +3521,7 @@ router.post(
     if (serverURL) spawnParams.push("-s", serverURL);
     if (splashPage) spawnParams.push("--splashPage", splashPage);
     if (allowOfflineMode) spawnParams.push("--allowOfflineMode");
+    if (allowShareTo) spawnParams.push("--allowShareTo");
     if (autoPublicLogin) spawnParams.push("--autoPublicLogin");
     if (synchedTables.length > 0)
       spawnParams.push("--synchedTables", ...synchedTables.map((tbl) => tbl));
@@ -3348,10 +3543,15 @@ router.post(
       spawnParams.push("--androidKeyStoreAlias", keystoreAlias);
     if (keystorePassword)
       spawnParams.push("--androidKeystorePassword", keystorePassword);
-    // end http call, return the out directory name
+    // end http call, return the out directory name, the build directory path and the mode
     // the gui polls for results
-    res.json({ build_dir_name: outDirName });
-    const child = spawn(getSafeSaltcornCmd(), spawnParams, {
+    res.json({
+      out_dir_name: outDirName,
+      build_dir: buildDir,
+      mode: mode,
+      msgs,
+    });
+    const child = spawn(getSafeSaltcornCmd(), [...spawnParams, "-m", mode], {
       stdio: ["ignore", "pipe", "pipe"],
       cwd: ".",
     });
@@ -3366,18 +3566,29 @@ router.post(
       getState().log(5, errMsg);
       childOutputs.push(errMsg);
     });
-    child.on("exit", (exitCode, signal) => {
-      const logFile = exitCode === 0 ? "logs.txt" : "error_logs.txt";
-      const exitMsg = childOutputs.join("\n");
-      fs.writeFile(path.join(buildDir, logFile), exitMsg, async (error) => {
-        if (error) {
-          console.log(`unable to write '${logFile}' to '${buildDir}'`);
+    child.on("exit", async (exitCode, signal) => {
+      if (mode === "prepare" && exitCode === 0) {
+        try {
+          fs.promises.writeFile(
+            path.join(buildDir, "spawnParams.json"),
+            JSON.stringify(spawnParams)
+          );
+        } catch (error) {
+          console.log(`unable to write spawnParams to '${buildDir}'`);
           console.log(error);
-        } else {
-          // no transaction, '/build-mobile-app/finished' filters for valid attributes
-          await File.set_xattr_of_existing_file(logFile, buildDir, req.user);
         }
-      });
+      }
+      const stepDesc = mode === "prepare" ? "_prepare_step" : "";
+      const logFile =
+        exitCode === 0 ? `logs${stepDesc}.txt` : `error_logs${stepDesc}.txt`;
+      try {
+        const exitMsg = childOutputs.join("\n");
+        await fs.promises.writeFile(path.join(outDir, logFile), exitMsg);
+        await File.set_xattr_of_existing_file(logFile, outDir, req.user);
+      } catch (error) {
+        console.log(`unable to write '${logFile}' to '${outDir}'`);
+        console.log(error);
+      }
     });
     child.on("error", (msg) => {
       const message = msg.message ? msg.message : msg.code;
@@ -3386,15 +3597,15 @@ router.post(
       const errMsg = [message, stack].join("\n");
       getState().log(5, msg);
       fs.writeFile(
-        path.join(buildDir, "error_logs.txt"),
+        path.join(outDir, "error_logs.txt"),
         errMsg,
         async (error) => {
           if (error) {
-            console.log(`unable to write logFile to '${buildDir}'`);
+            console.log(`unable to write logFile to '${outDir}'`);
             console.log(error);
           } else {
             // no transaction, '/build-mobile-app/finished' filters for valid attributes
-            await File.set_xattr_of_existing_file(logFile, buildDir, req.user);
+            await File.set_xattr_of_existing_file(logFile, outDir, req.user);
           }
         }
       );
@@ -3462,7 +3673,7 @@ router.post(
         .filter(
           (plugin) =>
             ["base", "sbadmin2"].indexOf(plugin.name) < 0 &&
-            newCfg.includedPlugins.indexOf(plugin.name) < 0
+            (newCfg.includedPlugins || []).indexOf(plugin.name) < 0
         )
         .map((plugin) => plugin.name);
       newCfg.excludedPlugins = excludedPlugins;

package/routes/fields.js

@@ -12,6 +12,7 @@ const { getState } = require("@saltcorn/data/db/state");
 const { renderForm } = require("@saltcorn/markup");
 const Field = require("@saltcorn/data/models/field");
 const Table = require("@saltcorn/data/models/table");
+const Trigger = require("@saltcorn/data/models/trigger");
 const Form = require("@saltcorn/data/models/form");
 const Workflow = require("@saltcorn/data/models/workflow");
 const User = require("@saltcorn/data/models/user");
@@ -306,6 +307,10 @@ const fieldFlow = (req) =>
           }
 
           await field.update(fldRow);
+          Trigger.emitEvent("AppChange", `Field ${fldRow.name}`, req.user, {
+            entity_type: "Field",
+            entity_name: fldRow.name || fldRow.label,
+          });
         } catch (e) {
           return {
             redirect: `/table/${context.table_id}`,
@@ -315,6 +320,10 @@ const fieldFlow = (req) =>
       } else {
         try {
           await Field.create(fldRow);
+          Trigger.emitEvent("AppChange", `Field ${fldRow.name}`, req.user, {
+            entity_type: "Field",
+            entity_name: fldRow.name || fldRow.label,
+          });
         } catch (e) {
           return {
             redirect: `/table/${context.table_id}`,

package/routes/menu.js

@@ -571,6 +571,7 @@ router.post(
     const new_menu = req.body;
     const menu_items = jQMEtoMenu(new_menu);
     await save_menu_items(menu_items);
+    Trigger.emitEvent("AppChange", `Menu`, req.user, {});
 
     res.json({ success: true });
   })

package/routes/notifications.js

@@ -199,23 +199,31 @@ router.post(
   error_catcher(async (req, res) => {
     const role = req.user?.role_id || 100;
     if (role === 100) {
-      req.flash("error", req.__("You must be logged in to share"));
-      res.redirect("/auth/login");
+      const msg = req.__("You must be logged in to share");
+      if (!req.smr) {
+        req.flash("error", msg);
+        res.redirect("/auth/login");
+      } else res.json({ error: msg });
     } else if (!getState().getConfig("pwa_share_to_enabled", false)) {
-      req.flash("error", req.__("Sharing not enabled"));
-      res.redirect("/");
+      const msg = req.__("Sharing not enabled");
+      if (!req.smr) {
+        req.flash("error", msg);
+        res.redirect("/");
+      } else res.json({ error: msg });
     } else {
       Trigger.emitEvent("ReceiveMobileShareData", null, req.user, {
         row: req.body,
       });
-      req.flash(
-        "success",
-        req.__(
-          "Shared: %s",
-          req.body.title || req.body.text || req.body.url || ""
-        )
-      );
-      res.status(303).redirect("/");
+      if (!req.smr) {
+        req.flash(
+          "success",
+          req.__(
+            "Shared: %s",
+            req.body.title || req.body.text || req.body.url || ""
+          )
+        );
+        res.status(303).redirect("/");
+      } else res.json({ success: "ok" });
     }
   })
 );

package/routes/page.js

@@ -68,13 +68,13 @@ const runPage = async (page, req, res, tic) => {
           no_menu: page.attributes?.no_menu,
           requestFluidLayout: page.attributes?.request_fluid_layout,
         } || `${page.name} page`,
-        add_edit_bar({
+        req.smr ? contents : add_edit_bar({
           role,
           title: page.name,
           what: req.__("Page"),
           url: `/pageedit/edit/${encodeURIComponent(page.name)}`,
           contents,
-        })
+        }),
       );
   } else {
     getState().log(2, `Page ${page.name} not authorized`);

package/routes/pageedit.js

@@ -146,7 +146,9 @@ const pagePropertiesForm = async (req, isNew) => {
       {
         name: "request_fluid_layout",
         label: req.__("Fluid layout"),
-        sublabel: req.__("Request fluid layout from theme for a wider display for this page"),
+        sublabel: req.__(
+          "Request fluid layout from theme for a wider display for this page"
+        ),
         type: "Bool",
       },
     ],
@@ -503,12 +505,21 @@ router.post(
           pageRow.layout = {};
         }
         await Page.update(+id, pageRow);
+        Trigger.emitEvent("AppChange", `Page ${dbPage.name}`, req.user, {
+          entity_type: "Page",
+          entity_name: dbPage.name,
+        });
         if (req.xhr) res.json({ success: "ok" });
         else res.redirect(`/pageedit/`);
       } else {
         if (!pageRow.layout) pageRow.layout = {};
         if (!pageRow.fixed_states) pageRow.fixed_states = {};
         await Page.create(pageRow);
+        Trigger.emitEvent("AppChange", `Page ${pageRow.name}`, req.user, {
+          entity_type: "Page",
+          entity_name: pageRow.name,
+        });
+
         if (!html_file)
           res.redirect(
             addOnDoneRedirect(`/pageedit/edit/${pageRow.name}`, req)
@@ -679,6 +690,10 @@ router.post(
       await Page.update(page.id, {
         layout: decodeURIComponent(req.body.layout),
       });
+      Trigger.emitEvent("AppChange", `Page ${page.name}`, req.user, {
+        entity_type: "Page",
+        entity_name: page.name,
+      });
       req.flash("success", req.__(`Page %s saved`, pagename));
       res.redirect(redirectTarget);
     } else if (req.body.code) {
@@ -687,6 +702,10 @@ router.post(
         const file = await File.findOne(page.html_file);
         if (!file) throw new Error(req.__("File not found"));
         await fsp.writeFile(file.location, req.body.code);
+        Trigger.emitEvent("AppChange", `Page ${page.name}`, req.user, {
+          entity_type: "Page",
+          entity_name: page.name,
+        });
         if (!req.xhr) {
           req.flash("success", req.__(`Page %s saved`, pagename));
           res.redirect(redirectTarget);
@@ -723,6 +742,11 @@ router.post(
 
     if (id && req.body.layout) {
       await Page.update(+id, { layout: req.body.layout });
+      const page = await Page.findOne({ id });
+      Trigger.emitEvent("AppChange", `Page ${page.name}`, req.user, {
+        entity_type: "Page",
+        entity_name: page.name,
+      });
       res.json({
         success: "ok",
       });
@@ -744,6 +768,10 @@ router.post(
   error_catcher(async (req, res) => {
     const { id } = req.params;
     const page = await Page.findOne({ id });
+    Trigger.emitEvent("AppChange", `Page ${page.name}`, req.user, {
+      entity_type: "Page",
+      entity_name: page.name,
+    });
     await page.delete();
     req.flash("success", req.__(`Page deleted`));
     res.redirect(`/pageedit`);
@@ -796,6 +824,7 @@ router.post(
       min_role: page.min_role,
       pagename: page.name,
     });
+    Trigger.emitEvent("AppChange", `Menu`, req.user, {});
     req.flash(
       "success",
       req.__(
@@ -820,6 +849,10 @@ router.post(
     const { id } = req.params;
     const page = await Page.findOne({ id });
     const newpage = await page.clone();
+    Trigger.emitEvent("AppChange", `Page ${newpage.name}`, req.user, {
+      entity_type: "Page",
+      entity_name: newpage.name,
+    });
     req.flash(
       "success",
       req.__("Page %s duplicated as %s", page.name, newpage.name)

package/routes/utils.js

@@ -23,6 +23,7 @@ const Crash = require("@saltcorn/data/models/crash");
 const File = require("@saltcorn/data/models/file");
 const User = require("@saltcorn/data/models/user");
 const Page = require("@saltcorn/data/models/page");
+const Trigger = require("@saltcorn/data/models/trigger");
 const si = require("systeminformation");
 const {
   config_fields_form,
@@ -426,6 +427,9 @@ const admin_config_route = ({
         const restart_required = check_if_restart_required(form, req);
 
         await save_config_from_form(form);
+        Trigger.emitEvent("AppChange", `Config`, req.user, {
+          config_keys: Object.keys(form.values),
+        });
         if (!req.xhr) {
           if (restart_required) {
             flash_restart(req);

package/routes/view.js

@@ -139,18 +139,20 @@ router.get(
           : contents0;
       res.sendWrap(
         title,
-        add_edit_bar({
-          role,
-          title: view.name,
-          what: req.__("View"),
-          url: `/viewedit/edit/${encodeURIComponent(view.name)}`,
-          cfgUrl: `/viewedit/config/${encodeURIComponent(view.name)}`,
-          contents,
-          req,
-          view,
-          viewtemplate: view.viewtemplate,
-          table: view.table_id || view.exttable_name,
-        })
+        !req.smr
+          ? add_edit_bar({
+              role,
+              title: view.name,
+              what: req.__("View"),
+              url: `/viewedit/edit/${encodeURIComponent(view.name)}`,
+              cfgUrl: `/viewedit/config/${encodeURIComponent(view.name)}`,
+              contents,
+              req,
+              view,
+              viewtemplate: view.viewtemplate,
+              table: view.table_id || view.exttable_name,
+            })
+          : contents
       );
     }
   })

package/routes/viewedit.js

@@ -27,6 +27,7 @@ const Table = require("@saltcorn/data/models/table");
 const View = require("@saltcorn/data/models/view");
 const Workflow = require("@saltcorn/data/models/workflow");
 const User = require("@saltcorn/data/models/user");
+const Trigger = require("@saltcorn/data/models/trigger");
 const Page = require("@saltcorn/data/models/page");
 const File = require("@saltcorn/data/models/file");
 const Tag = require("@saltcorn/data/models/tag");
@@ -551,6 +552,10 @@ router.post(
           //console.log(v);
           await View.create(v);
         }
+        Trigger.emitEvent("AppChange", `View ${v.name}`, req.user, {
+          entity_type: "View",
+          entity_name: v.name,
+        });
         if (req.xhr) res.json({ success: "ok" });
         else
           res.redirect(
@@ -733,6 +738,10 @@ router.post(
         };
       else newcfg = { ...view.configuration, ...context };
       await View.update({ configuration: newcfg }, view.id);
+      Trigger.emitEvent("AppChange", `View ${view.name}`, req.user, {
+        entity_type: "View",
+        entity_name: view.name,
+      });
     };
     const wfres = await configFlow.run(req.body, req);
 
@@ -761,6 +770,7 @@ router.post(
       min_role: view.min_role,
       viewname: view.name,
     });
+    Trigger.emitEvent("AppChange", `Menu`, req.user, {});
     req.flash(
       "success",
       req.__(
@@ -790,6 +800,10 @@ router.post(
     const { id } = req.params;
     const view = await View.findOne({ id });
     const newview = await view.clone();
+    Trigger.emitEvent("AppChange", `View ${newview.name}`, req.user, {
+      entity_type: "View",
+      entity_name: newview.name,
+    });
     req.flash(
       "success",
       req.__("View %s duplicated as %s", view.name, newview.name)
@@ -841,6 +855,10 @@ router.post(
       const exview = await View.findOne({ id });
       let newcfg = { ...exview.configuration, ...req.body };
       await View.update({ configuration: newcfg }, +id);
+      Trigger.emitEvent("AppChange", `View ${exview.name}`, req.user, {
+        entity_type: "View",
+        entity_name: exview.name,
+      });
       res.json({ success: "ok" });
     } else {
       res.json({ error: req.__("Unable to save: No view") });
@@ -879,6 +897,10 @@ router.post(
             };
           else newcfg = { ...view.configuration, ...step.renderForm.values };
           await View.update({ configuration: newcfg }, view.id);
+          Trigger.emitEvent("AppChange", `View ${view.name}`, req.user, {
+            entity_type: "View",
+            entity_name: view.name,
+          });
           res.json({ success: "ok" });
         } else {
           res.json({ error: step.renderForm.errorSummary });
@@ -906,6 +928,10 @@ router.post(
     const role = req.body.role;
     await View.update({ min_role: role }, +id);
     const view = await View.findOne({ id });
+    Trigger.emitEvent("AppChange", `View ${view.name}`, req.user, {
+      entity_type: "View",
+      entity_name: view.name,
+    });
     const roles = await User.get_roles();
     const roleRow = roles.find((r) => r.id === +role);
     const message =

package/tests/clientjs.test.js

@@ -90,6 +90,25 @@ test("updateQueryStringParameter hash", () => {
     "/foo?name=Bar#Baz"
   );
 });
+test("addQueryStringParameter", () => {
+  expect(addQueryStringParameter("/foo", "age", 43)).toBe(
+    "/foo?age=43"
+  );
+  expect(addQueryStringParameter("/foo?age=43", "age", 44)).toBe(
+    "/foo?age=43&age=44"
+  );
+  expect(addQueryStringParameter("/foo?age=43", "age", 43)).toBe(
+    "/foo?age=43"
+  );
+});
+test("addQueryStringParameter hash", () => {
+  expect(addQueryStringParameter("/foo#baz", "age", 43)).toBe(
+    "/foo?age=43#baz"
+  );
+  expect(addQueryStringParameter("/foo?age=43#baz", "age", 44)).toBe(
+    "/foo?age=43&age=44#baz"
+  );
+});
 test("unique_field_from_rows test", () => {
   $("body").append(`<input id="mkuniq6" value="bar"></div>`);
   unique_field_from_rows(