@saltcorn/server 1.1.4-beta.8 → 1.1.4

package/docs/variables/_saltcorn_data.utils.sleep.html

@@ -19,7 +19,7 @@
 <h1>Variable sleep</h1></div>
 <div class="tsd-signature"><span class="tsd-kind-variable">sleep</span><span class="tsd-signature-symbol">:</span> <span class="tsd-signature-type">any</span></div><aside class="tsd-sources">
 <ul>
-<li>Defined in <a href="https://github.com/saltcorn/saltcorn/blob/b0f1710efb/packages/saltcorn-data/index.ts#L20">saltcorn-data/index.ts:20</a></li></ul></aside></div>
+<li>Defined in <a href="https://github.com/saltcorn/saltcorn/blob/99d8abda8b/packages/saltcorn-data/index.ts#L20">saltcorn-data/index.ts:20</a></li></ul></aside></div>
 <div class="col-sidebar">
 <div class="page-menu">
 <div class="tsd-navigation settings">

package/docs/variables/_saltcorn_db_common.multi_tenant.tenantNamespace.html

@@ -19,7 +19,7 @@
 <h1>Variable tenantNamespace<code class="tsd-tag ts-flagConst">Const</code> </h1></div>
 <div class="tsd-signature"><span class="tsd-kind-variable">tenant<wbr/>Namespace</span><span class="tsd-signature-symbol">:</span> <span class="tsd-signature-type ">AsyncLocalStorage</span><span class="tsd-signature-symbol">&lt;</span><span class="tsd-signature-type ">RequestContext</span><span class="tsd-signature-symbol">&gt;</span><span class="tsd-signature-symbol"> = ...</span></div><aside class="tsd-sources">
 <ul>
-<li>Defined in <a href="https://github.com/saltcorn/saltcorn/blob/b0f1710efb/packages/db-common/multi-tenant.ts#L17">db-common/multi-tenant.ts:17</a></li></ul></aside></div>
+<li>Defined in <a href="https://github.com/saltcorn/saltcorn/blob/99d8abda8b/packages/db-common/multi-tenant.ts#L17">db-common/multi-tenant.ts:17</a></li></ul></aside></div>
 <div class="col-sidebar">
 <div class="page-menu">
 <div class="tsd-navigation settings">

package/locales/da.json

@@ -860,5 +860,61 @@
 	"Always show create view": "Vis altid opret visning",
 	"If off, only show create view if the query state is about the current user": "Hvis slukket, vis kun opret visning, hvis forespørgselstilstanden handler om den nuværende bruger.",
 	"The current theme has no user specific settings": "The current theme has no user specific settings",
-	"View '%s' was loaded": "View '%s' was loaded"
+	"View '%s' was loaded": "View '%s' was loaded",
+	"After delete": "After delete",
+	"Working directory": "Working directory",
+	"All tags": "All tags",
+	"Manage tags...": "Manage tags...",
+	"Triggers on table": "Triggers on table",
+	"Create trigger": "Create trigger",
+	"Create model": "Create model",
+	"Delete table": "Delete table",
+	"Delete table+views+triggers": "Delete table+views+triggers",
+	"Sync information": "Sync information",
+	"Sync information tracks the last modification or deletion timestamp so that the table data can be synchronized with the mobile app": "Sync information tracks the last modification or deletion timestamp so that the table data can be synchronized with the mobile app",
+	"Models": "Models",
+	"Row inclusion formula": "Row inclusion formula",
+	"Only include rows where this formula is true. ": "Only include rows where this formula is true. ",
+	"In scope:": "In scope:",
+	"Error message": "Error message",
+	"Error shown to user if uniqueness is violated": "Error shown to user if uniqueness is violated",
+	"Calculated field will be stored in Database": "Calculated field will be stored in Database",
+	"Protected": "Protected",
+	"Set role to access": "Set role to access",
+	"User must have this role or higher to update or create field values": "User must have this role or higher to update or create field values",
+	"The field that will be shown to the user when choosing a value": "The field that will be shown to the user when choosing a value",
+	"Include in full-text search": "Include in full-text search",
+	"On delete": "On delete",
+	"Prevent any deletion of parent rows": "Prevent any deletion of parent rows",
+	"If the parent row is deleted, automatically delete the child rows.": "If the parent row is deleted, automatically delete the child rows.",
+	"If the parent row is deleted, set key fields on child rows to null": "If the parent row is deleted, set key fields on child rows to null",
+	"If the parent row is deleted, do this to the child rows.": "If the parent row is deleted, do this to the child rows.",
+	"Installed theme": "Installed theme",
+	"Theme for role": "Theme for role",
+	"Set theme for each user role »": "Set theme for each user role »",
+	"Available themes": "Available themes",
+	"Install more themes »": "Install more themes »",
+	"The view name is part of the URL when it is shown alone.": "The view name is part of the URL when it is shown alone.",
+	"Some view patterns accept interpolations. Ex: <code>{{ name }}</code> or <code>{{ row ? `Edit ${row.name}` : `New person` }}</code>": "Some view patterns accept interpolations. Ex: <code>{{ name }}</code> or <code>{{ row ? `Edit ${row.name}` : `New person` }}</code>",
+	"Page description": "Page description",
+	"For search engines. Some view patterns accept interpolations.": "For search engines. Some view patterns accept interpolations.",
+	"No menu": "No menu",
+	"Omit the menu from this view": "Omit the menu from this view",
+	"Popup width": "Popup width",
+	"Popup min width": "Popup min width",
+	"Show an icon in the title bar to indicate when form data is being saved": "Show an icon in the title bar to indicate when form data is being saved",
+	"Link out?": "Link out?",
+	"Show a link to open popup contents in new tab": "Show a link to open popup contents in new tab",
+	"Search for...": "Search for...",
+	"Clean modules and restart": "Clean modules and restart",
+	"Mobile": "Mobile",
+	"Create new row": "Create new row",
+	"Locale identifier short code, e.g. en, zh, fr, ar etc. ": "Locale identifier short code, e.g. en, zh, fr, ar etc. ",
+	"Is this the default language in which the application is built?": "Is this the default language in which the application is built?",
+	"Translate with LLM": "Translate with LLM",
+	"Pagegroups": "Pagegroups",
+	"Registry editor": "Registry editor",
+	"Choose File": "Choose File",
+	"No file chosen": "No file chosen",
+	"Fixed and blocked fields": "Fixed and blocked fields"
 }

package/locales/en.json

@@ -1602,5 +1602,7 @@
 	"Translated %s strings. Click 'Translate with LLM' again to continue": "Translated %s strings. Click 'Translate with LLM' again to continue",
 	"Translate with LLM": "Translate with LLM",
 	"You may need to adjust view relations associated with this table after renaming it": "You may need to adjust view relations associated with this table after renaming it",
-	"Error running workflow": "Error running workflow"
+	"Error running workflow": "Error running workflow",
+	"Choose File": "Choose File",
+	"No file chosen": "No file chosen"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saltcorn/server",
-  "version": "1.1.4-beta.8",
+  "version": "1.1.4",
   "description": "Server app for Saltcorn, open-source no-code platform",
   "homepage": "https://saltcorn.com",
   "main": "index.js",
@@ -8,14 +8,14 @@
   "dependencies": {
     "@aws-sdk/client-s3": "^3.735.0",
     "@dr.pogodin/csurf": "^1.14.1",
-    "@saltcorn/base-plugin": "1.1.4-beta.8",
-    "@saltcorn/builder": "1.1.4-beta.8",
-    "@saltcorn/data": "1.1.4-beta.8",
-    "@saltcorn/admin-models": "1.1.4-beta.8",
-    "@saltcorn/filemanager": "1.1.4-beta.8",
-    "@saltcorn/markup": "1.1.4-beta.8",
-    "@saltcorn/plugins-loader": "1.1.4-beta.8",
-    "@saltcorn/sbadmin2": "1.1.4-beta.8",
+    "@saltcorn/base-plugin": "1.1.4",
+    "@saltcorn/builder": "1.1.4",
+    "@saltcorn/data": "1.1.4",
+    "@saltcorn/admin-models": "1.1.4",
+    "@saltcorn/filemanager": "1.1.4",
+    "@saltcorn/markup": "1.1.4",
+    "@saltcorn/plugins-loader": "1.1.4",
+    "@saltcorn/sbadmin2": "1.1.4",
     "@socket.io/cluster-adapter": "^0.2.1",
     "@socket.io/sticky": "^1.0.1",
     "adm-zip": "0.5.16",

package/public/saltcorn-common.js

@@ -736,6 +736,14 @@ function escapeHtml(text) {
   });
 }
 
+function unescapeHtml(str) {
+  if (!str || !str.replace) return str;
+  return str
+    .replaceAll("&lt;", "<")
+    .replaceAll("&gt;", ">")
+    .replaceAll("&amp;", "&");
+}
+
 function reload_on_init() {
   localStorage.setItem("reload_on_init", true);
 }
@@ -1018,7 +1026,8 @@ function initialize_page() {
       );
     }
   });
-  $("[data-inline-edit-dest-url]").click(function () {
+  $("[data-inline-edit-dest-url]").click(function (event) {
+    event.stopPropagation();
     var url = $(this).attr("data-inline-edit-dest-url");
     var current =
       $(this).attr("data-inline-edit-current") ||
@@ -1042,7 +1051,7 @@ function initialize_page() {
           })
         );
         $(this).replaceWith(
-          `<form method="post" action="/field/save-click-edit" onsubmit="inline_ajax_submit_with_fielddata(event, '${opts}')"        
+          `<form method="post" action="/field/save-click-edit" onclick="event.stopPropagation()" onsubmit="inline_ajax_submit_with_fielddata(event, '${opts}')"        
       <input type="hidden" name="_csrf" value="${_sc_globalCsrf}">
       <input type="hidden" name="_fielddata" value="${fielddata}">
       <div class="input-group">
@@ -1124,7 +1133,8 @@ function initialize_page() {
     } else if (is_key) {
       const [tblName, target] = type.replace("Key:", "").split(".");
       doAjaxOptionsFetch(tblName, target);
-    } else
+    } else {
+      const parent = $(this).parent();
       $(this).replaceWith(
         `<form method="post" action="${url}" ${
           ajax
@@ -1154,17 +1164,18 @@ function initialize_page() {
                   : "any"
               }"`
             : ""
-        } name="${key}" ${
-          type === "Bool"
-            ? current
-              ? "checked"
-              : ""
-            : `value="${escapeHtml(current)}"`
-        }>
+        } name="${key}" ${type === "Bool" ? (current ? "checked" : "") : ``}>
       <button type="submit" class="btn btn-sm btn-primary">OK</button>
-      <button onclick="cancel_inline_edit(event, '${opts}')" type="button" class="btn btn-sm btn-danger"><i class="fas fa-times"></i></button>
+      <button onclick="cancel_inline_edit(event, '${opts.replaceAll("'", "\\'")}')" type="button" class="btn btn-sm btn-danger"><i class="fas fa-times"></i></button>
       </form>`
       );
+      if (type !== "Bool") {
+        const newVal = $(this).attr("data-inline-edit-unescape")
+          ? unescapeHtml(current)
+          : current;
+        parent.find(`[name="${key}"]`).val(newVal);
+      }
+    }
   });
   if (!isNode) {
     doMobileTransforms();

package/public/saltcorn.js

@@ -215,6 +215,7 @@ function pjax_to(href, e) {
         $dest.html(res);
         if (localizer.length) localizer.attr("data-sc-local-state", href);
         initialize_page();
+        document.dispatchEvent(new Event("pjax-loaded"));
       },
       error: function (res) {
         if (!checkNetworkError(res))
@@ -722,9 +723,15 @@ function checkNetworkError(e) {
 }
 
 function ajax_post_btn(e, reload_on_done, reload_delay) {
-  var form = $(e).closest("form");
-  var url = form.attr("action");
-  var form_data = form.serialize();
+  let form_data = "";
+  let url;
+  if (typeof e === "string") url = e;
+  else if (e) {
+    var form = $(e).closest("form");
+    url = form.attr("action");
+    form_data = form.serialize();
+  }
+
   $.ajax(url, {
     type: "POST",
     headers: {

package/routes/actions.js

@@ -930,7 +930,12 @@ router.get(
         '&nbsp;<i class="fas fa-edit"></i>'
       ) +
       a(
-        { href: `/actions/testrun/${id}`, class: "ms-2" },
+        {
+          href: `/actions/testrun/${id}`,
+          class: "ms-2",
+          onclick: `press_store_button(this)`,
+        },
+
         req.__("Test run") + "&nbsp;&raquo;"
       );
     if (trigger.action === "Workflow") {
@@ -1277,7 +1282,7 @@ router.get(
         sub2_page: "Test run output",
         contents: {
           type: "card",
-          title: req.__("Test run output"),
+          title: req.__("Test run output") + " - " + trigger.name,
           contents: div(
             div({ class: "testrunoutput" }, output),
             runres
@@ -1298,6 +1303,7 @@ router.get(
               {
                 href: `/actions/testrun/${id}`,
                 class: "ms-1 mt-4 btn btn-primary",
+                onclick: `press_store_button(this)`,
               },
               i({ class: "fas fa-redo me-1" }),
               req.__("Re-run")

package/routes/admin.js

@@ -97,7 +97,10 @@ const {
 const packagejson = require("../package.json");
 const Form = require("@saltcorn/data/models/form");
 const { get_latest_npm_version } = require("@saltcorn/data/models/config");
-const { getMailTransport } = require("@saltcorn/data/models/email");
+const {
+  getMailTransport,
+  getOauth2Client,
+} = require("@saltcorn/data/models/email");
 const {
   getBaseDomain,
   hostname_matches_baseurl,
@@ -216,8 +219,22 @@ admin_config_route({
   flash: "Email settings updated",
   field_names: [
     "smtp_host",
-    "smtp_username",
-    "smtp_password",
+    "smtp_auth_method",
+    { name: "smtp_username", showIf: { smtp_auth_method: "password" } },
+    { name: "smtp_password", showIf: { smtp_auth_method: "password" } },
+    {
+      name: "smtp_api_option",
+      showIf: {
+        smtp_host: "outlook.office365.com",
+        smtp_auth_method: "oauth2",
+      },
+    },
+    { name: "smtp_authorize_url", showIf: { smtp_auth_method: "oauth2" } },
+    { name: "smtp_token_url", showIf: { smtp_auth_method: "oauth2" } },
+    { name: "smtp_redirect_uri", showIf: { smtp_auth_method: "oauth2" } },
+    { name: "smtp_client_id", showIf: { smtp_auth_method: "oauth2" } },
+    { name: "smtp_client_secret", showIf: { smtp_auth_method: "oauth2" } },
+    { name: "smtp_outh_scopes", showIf: { smtp_auth_method: "oauth2" } },
     "smtp_port",
     "smtp_secure",
     "smtp_allow_self_signed",
@@ -243,6 +260,40 @@ admin_config_route({
             },
             req.__("Send test email")
           ),
+          a(
+            {
+              id: "authenticate_oauth",
+              href: "/admin/authorize-mail-oauth",
+              class: "btn btn-primary ms-2",
+              onclick: "spin_action_link(this)",
+            },
+            req.__("Authorize")
+          ),
+          script(`
+  function authMethodChange(method) {
+    const btn = document.getElementById("authenticate_oauth");
+    if (method === "oauth2") {
+      btn.classList.remove("d-none");
+      btn.classList.add("d-inline");
+      // get 
+    }
+    else {
+      btn.classList.add("d-none");
+      btn.classList.remove("d-inline");
+    }
+
+  }
+  ${domReady(`
+  const authMethod = document.getElementById('inputsmtp_auth_method');
+  if (authMethod) {
+    const authMethodValue = authMethod.value;
+    authMethodChange(authMethodValue);
+    authMethod.addEventListener('change', (e) => {
+      const authMethod = e.target.value;
+      authMethodChange(authMethod);
+    });
+  }`)}
+`),
         ],
       },
     });
@@ -266,7 +317,7 @@ router.get(
       html: req.__("Hello from Saltcorn"),
     };
     try {
-      const sendres = await getMailTransport().sendMail(email);
+      const sendres = await (await getMailTransport()).sendMail(email);
       getState().log(6, sendres);
       req.flash(
         "success",
@@ -281,6 +332,27 @@ router.get(
   })
 );
 
+/**
+ * do the redirect to the oauth2 provider
+ */
+router.get(
+  "/authorize-mail-oauth",
+  isAdmin,
+  error_catcher(async (req, res) => {
+    const client = getOauth2Client();
+    const smtpRedirectUri = getState().getConfig("smtp_redirect_uri");
+    const scopes = getState().getConfig("smtp_outh_scopes");
+    const scopeArray = scopes.split(" ").map((s) => s.trim());
+
+    // Mail.Send SMTP.Send profile openid email https://outlook.office365.com/IMAP.AccessAsUser.All
+    const authorizeUrl = client.authorizeURL({
+      redirect_uri: smtpRedirectUri,
+      scope: scopeArray,
+    });
+    res.redirect(authorizeUrl);
+  })
+);
+
 /**
  * @name get/send-test-email
  * @function
@@ -288,7 +360,16 @@ router.get(
  */
 router.get(
   "/help/:topic",
-  isAdmin,
+  isAdminOrHasConfigMinRole([
+    "min_role_edit_tables",
+    "min_role_inspect_tables",
+    "min_role_edit_views",
+    "min_role_edit_pages",
+    "min_role_edit_triggers",
+    "min_role_edit_search",
+    "min_role_edit_menu",
+    "min_role_edit_files",
+  ]),
   error_catcher(async (req, res) => {
     const { topic } = req.params;
     const { markup } = await get_help_markup(topic, req.query, req);
@@ -438,8 +519,9 @@ router.get(
                     {
                       btnClass: "btn-outline-primary",
                       klass: "backupadminbtn",
-                      //TODO not sure how to tell when backup done 
-                      onClick: "spin_action_link($('.backupadminbtn'));setTimeout(()=> reset_spinners($('.backupadminbtn')), 3000)"
+                      //TODO not sure how to tell when backup done
+                      onClick:
+                        "spin_action_link($('.backupadminbtn'));setTimeout(()=> reset_spinners($('.backupadminbtn')), 3000)",
                     }
                   )
                 ),

package/routes/api.js

@@ -426,10 +426,16 @@ router.get(
             }
           }
           if (tabulator_pagination_format) {
-            res.json({
-              last_page: Math.ceil((await table.countRows()) / +tabulator_size),
-              data: rows.map(limitFields(fields)),
-            });
+            const count = await table.countRows();
+            if (count === null)
+              res.json({
+                data: rows.map(limitFields(fields)),
+              });
+            else
+              res.json({
+                last_page: Math.ceil(count / +tabulator_size),
+                data: rows.map(limitFields(fields)),
+              });
           } else res.json({ success: rows.map(limitFields(fields)) });
         } else {
           getState().log(3, `API get ${table.name} not authorized`);

package/routes/common_lists.js

@@ -737,7 +737,10 @@ const getTriggerList = async (
         key: (r) =>
           link(
             `/actions/testrun/${r.id}${on_done_redirect_str}`,
-            req.__("Test run")
+            req.__("Test run"),
+            {
+              onclick: `press_store_button(this)`,
+            }
           ),
       },
       {

package/routes/fields.js

@@ -307,12 +307,7 @@ const fieldFlow = (req) =>
       try {
         await db.withTransaction(async () => {
           if (context.id) {
-            const field = await Field.findOne({ id: context.id });
-
-            if (fldRow.label && field.label != fldRow.label) {
-              fldRow.name = Field.labelToName(fldRow.label);
-            }
-
+            const field = await Field.findOne({ id: context.id });          
             await field.update(fldRow);
             Trigger.emitEvent(
               "AppChange",

package/routes/infoarch.js

@@ -19,13 +19,19 @@ const {
   upload_language_pack,
 } = require("../markup/admin.js");
 const { getState } = require("@saltcorn/data/db/state");
-const { div, a, i, text, button } = require("@saltcorn/markup/tags");
-const { mkTable, renderForm, post_delete_btn } = require("@saltcorn/markup");
+const { span, div, a, i, text, button } = require("@saltcorn/markup/tags");
+const {
+  mkTable,
+  renderForm,
+  post_delete_btn,
+  post_btn,
+} = require("@saltcorn/markup");
 const Form = require("@saltcorn/data/models/form");
 const Snapshot = require("@saltcorn/admin-models/models/snapshot");
 const { stringify } = require("csv-stringify");
 const csvtojson = require("csvtojson");
 const { hasLLM, translate } = require("@saltcorn/data/translate");
+const { escapeHtml } = require("@saltcorn/data/utils");
 
 /**
  * @type {object}
@@ -365,18 +371,35 @@ router.get(
                 [
                   {
                     label: req.__("In default language"),
-                    key: "in_default",
+                    key: (r) => escapeHtml(r.in_default),
                   },
                   {
                     label: req.__("In %s", form.values.name),
                     key: (r) =>
-                      div(
-                        {
-                          "data-inline-edit-dest-url": `/site-structure/localizer/save-string/${lang}/${encodeURIComponent(
-                            r.in_default
-                          )}`,
-                        },
-                        r.translated
+                      span(
+                        div(
+                          {
+                            "data-inline-edit-dest-url": `/site-structure/localizer/save-string/${lang}/${encodeURIComponent(
+                              r.in_default
+                            )}`,
+                            "data-inline-edit-unescape": "true",
+                            class: "d-inline",
+                          },
+                          escapeHtml(r.translated)
+                        ),
+                        r.in_default !== r.translated &&
+                          post_btn(
+                            `/site-structure/localizer/delete-string/${lang}/${encodeURIComponent(
+                              r.in_default
+                            )}`,
+                            "",
+                            req.csrfToken(),
+                            {
+                              icon: "fas fa-trash-alt",
+                              btnClass: "btn-sm btn-xs btn-link link-danger",
+                              formClass: "d-inline",
+                            }
+                          )
                       ),
                   },
                 ],
@@ -431,10 +454,12 @@ router.post(
       req.flash(
         "success",
         req.__(
-          `Translated %s strings. Click 'Translate with LLM' again to continue`, count
+          `Translated %s strings. Click 'Translate with LLM' again to continue`,
+          count
         )
       );
-    else req.flash("success", req.__(`Finished translating %s strings.`, count));
+    else
+      req.flash("success", req.__(`Finished translating %s strings.`, count));
     res.redirect(`/site-structure/localizer/edit/${lang}`);
   })
 );
@@ -459,9 +484,28 @@ router.post(
       return;
     }
     const cfgStrings = getState().getConfigCopy("localizer_strings");
-    if (cfgStrings[lang])
-      cfgStrings[lang][defstring] = text((req.body || {}).value);
-    else cfgStrings[lang] = { [defstring]: text((req.body || {}).value) };
+    if (cfgStrings[lang]) cfgStrings[lang][defstring] = (req.body || {}).value;
+    else cfgStrings[lang] = { [defstring]: (req.body || {}).value };
+    await getState().setConfig("localizer_strings", cfgStrings);
+    res.redirect(`/site-structure/localizer/edit/${lang}`);
+  })
+);
+
+router.post(
+  "/localizer/delete-string/:lang/:defstring",
+  isAdmin,
+  error_catcher(async (req, res) => {
+    const { lang, defstring } = req.params;
+    if (
+      lang === "__proto__" ||
+      defstring === "__proto__" ||
+      lang === "constructor"
+    ) {
+      res.redirect(`/`);
+      return;
+    }
+    const cfgStrings = getState().getConfigCopy("localizer_strings");
+    if (cfgStrings[lang]) delete cfgStrings[lang][defstring];
     await getState().setConfig("localizer_strings", cfgStrings);
     res.redirect(`/site-structure/localizer/edit/${lang}`);
   })

package/routes/menu.js

@@ -413,7 +413,7 @@ const menuForm = async (req) => {
         //default: "Standard",
         attributes: {
           inline: true,
-          options: "Standard, Mobile Bottom, Secondary Menu",
+          options: ["Standard", "Mobile Bottom", "Secondary Menu"],
         },
       },
     ],

package/routes/plugins.js

@@ -102,7 +102,7 @@ const pluginForm = (req, plugin) => {
         name: "source",
         type: getState().types.String,
         required: true,
-        attributes: { options: "npm,local,github,git" },
+        attributes: { options: ["npm", "local", "github", "git"] },
         sublabel: req.__(
           "Source of module for install. Few options:" +
             "npm - download from npm repository," +

package/routes/registry.js

@@ -374,6 +374,23 @@ router.post(
     await db.withTransaction(async () => {
       await install_pack(pack);
     });
+    switch (etype) {
+      case "table":
+        await getState().refresh_tables();
+        break;
+      case "view":
+        await getState().refresh_views();
+        break;
+      case "page":
+        await getState().refresh_pages();
+        break;
+      case "trigger":
+        await getState().refresh_triggers();
+        break;
+      case "config":
+        await getState().refresh_config();
+        break;
+    }
     res.redirect(
       `/registry-editor?etype=${etype}&ename=${encodeURIComponent(
         ename

package/routes/tables.js

@@ -1018,7 +1018,8 @@ router.get(
     // Table Data card
     const dataCard = div(
       { class: "d-flex text-center" },
-      div({ class: "mx-auto" }, h4(`${nrows}`), req.__("Rows")),
+      nrows !== null &&
+        div({ class: "mx-auto" }, h4(`${nrows}`), req.__("Rows")),
       div(
         { class: "mx-auto" },
         a(