From 299dcbab3d063516799863a7c17573e2297bba2c Mon Sep 17 00:00:00 2001
From: Andrew Kingston <aptkingston@ssl-mail.com>
Date: Thu, 12 Nov 2020 09:07:09 +0000
Subject: [PATCH] Add majority of API interactions to SDK

---
 .../userInterface/FlatButton.svelte           |  3 +-
 packages/client/src/render/screenRouter.js    |  2 +-
 packages/component-sdk/src/api/api.js         | 60 +++++++++++++++----
 packages/component-sdk/src/api/auth.js        |  4 --
 packages/component-sdk/src/api/datasources.js | 58 ++++++++++++++++++
 .../component-sdk/src/api/relationships.js    | 12 ++++
 packages/component-sdk/src/api/rows.js        | 16 +----
 packages/component-sdk/src/api/tables.js      | 16 +++++
 packages/component-sdk/src/api/views.js       | 22 +++++++
 packages/component-sdk/src/store/auth.js      |  4 --
 packages/component-sdk/src/store/config.js    |  4 --
 packages/standard-components/src/Card.svelte  |  2 +-
 .../src/CardHorizontal.svelte                 |  2 +-
 .../standard-components/src/Container.svelte  |  2 -
 packages/standard-components/src/Link.svelte  |  2 -
 packages/standard-components/src/Login.svelte |  2 -
 packages/standard-components/src/Nav.svelte   |  2 +-
 17 files changed, 166 insertions(+), 47 deletions(-)
 create mode 100644 packages/component-sdk/src/api/datasources.js
 create mode 100644 packages/component-sdk/src/api/relationships.js
 create mode 100644 packages/component-sdk/src/api/views.js

diff --git a/packages/builder/src/components/userInterface/FlatButton.svelte b/packages/builder/src/components/userInterface/FlatButton.svelte
index 4bac03d4a6..931b90a5a2 100644
--- a/packages/builder/src/components/userInterface/FlatButton.svelte
+++ b/packages/builder/src/components/userInterface/FlatButton.svelte
@@ -1,9 +1,8 @@
 <script>
-  import { buildStyle } from "../../helpers.js"
   export let value = ""
   export let text = ""
   export let icon = ""
-  export let onClick = value => {}
+  export let onClick = () => {}
   export let selected = false
 
   $: useIcon = !!icon
diff --git a/packages/client/src/render/screenRouter.js b/packages/client/src/render/screenRouter.js
index 7711b53acf..4d5579040f 100644
--- a/packages/client/src/render/screenRouter.js
+++ b/packages/client/src/render/screenRouter.js
@@ -1,6 +1,6 @@
 import regexparam from "regexparam"
 import appStore from "../state/store"
-import { getAppId } from "./getAppId"
+import { getAppId } from "../../../component-sdk/src/utils"
 
 export const screenRouter = ({ screens, onScreenSelected, window }) => {
   function sanitize(url) {
diff --git a/packages/component-sdk/src/api/api.js b/packages/component-sdk/src/api/api.js
index 10dede1c68..e7cc5261e0 100644
--- a/packages/component-sdk/src/api/api.js
+++ b/packages/component-sdk/src/api/api.js
@@ -2,22 +2,36 @@ import { get } from "svelte/store"
 import { getAppId } from "../utils"
 import { configStore } from "../store"
 
-const makeURL = path => {
+/**
+ * API cache for cached request responses.
+ */
+let cache = {}
+
+/**
+ * Makes a fully formatted URL based on the SDK configuration.
+ */
+const makeFullURL = path => {
   const { proto, domain, port } = get(configStore).config
   let url = `/${path}`.replace("//", "/")
   return domain ? `${proto}://${domain}:${port}${url}` : url
 }
 
+/**
+ * Handler for API errors.
+ */
 const handleError = error => {
-  store.actions.handleError(error)
+  configStore.actions.handleError(error)
   return { error }
 }
 
-const apiCall = method => async ({ url, body }) => {
+/**
+ * Performs an API call to the server.
+ * App ID header is always correctly set.
+ */
+const makeApiCall = async ({ method, url, body }) => {
   try {
-    const fullURL = makeURL(url)
-    const response = await fetch(fullURL, {
-      method: method,
+    const response = await fetch(url, {
+      method,
       headers: {
         "Content-Type": "application/json",
         "x-budibase-app-id": getAppId(window.document.cookie),
@@ -45,10 +59,36 @@ const apiCall = method => async ({ url, body }) => {
   }
 }
 
+/**
+ * Performs an API call to the server and caches the response.
+ * Future invocation for this URL will return the cached result instead of
+ * hitting the server again.
+ */
+const makeCachedApiCall = async params => {
+  const identifier = params.url
+  if (!identifier) {
+    return null
+  }
+  if (!cache[identifier]) {
+    cache[identifier] = makeApiCall(params)
+    cache[identifier] = await cache[identifier]
+  }
+  return await cache[identifier]
+}
+
+/**
+ * Constructs an API call function for a particular HTTP method.
+ */
+const requestApiCall = method => async ({ url, body, cache = false }) => {
+  const fullURL = makeFullURL(url)
+  const params = { method, url: fullURL, body }
+  return await (cache ? makeCachedApiCall : makeApiCall)(params)
+}
+
 export default {
-  post: apiCall("POST"),
-  get: apiCall("GET"),
-  patch: apiCall("PATCH"),
-  del: apiCall("DELETE"),
+  post: requestApiCall("POST"),
+  get: requestApiCall("GET"),
+  patch: requestApiCall("PATCH"),
+  del: requestApiCall("DELETE"),
   error: handleError,
 }
diff --git a/packages/component-sdk/src/api/auth.js b/packages/component-sdk/src/api/auth.js
index ba1cf51a75..04fc1169b9 100644
--- a/packages/component-sdk/src/api/auth.js
+++ b/packages/component-sdk/src/api/auth.js
@@ -2,10 +2,6 @@ import api from "./api"
 
 /**
  * Performs a log in request.
- *
- * @param username
- * @param password
- * @returns {Promise<{error: *}|any|{error: *}>}
  */
 export const logIn = async ({ username, password }) => {
   if (!username) {
diff --git a/packages/component-sdk/src/api/datasources.js b/packages/component-sdk/src/api/datasources.js
new file mode 100644
index 0000000000..acea8ab640
--- /dev/null
+++ b/packages/component-sdk/src/api/datasources.js
@@ -0,0 +1,58 @@
+import { fetchTableData, fetchTableDefinition } from "./tables"
+import { fetchViewData } from "./views"
+import { fetchRelationshipData } from "./relationships"
+
+/**
+ * Fetches all rows for a particular Budibase data source.
+ */
+export const fetchDatasource = async datasource => {
+  if (!datasource || !datasource.name) {
+    return []
+  }
+
+  // Fetch all rows in data source
+  const { type, name, tableId } = datasource
+  let rows = []
+  if (type === "table") {
+    rows = await fetchTableData(name)
+  } else if (type === "view") {
+    rows = await fetchViewData(datasource)
+  } else if (type === "link") {
+    rows = await fetchRelationshipData(tableId)
+  }
+
+  // Enrich rows
+  return await enrichDatasourceRows(rows, tableId)
+}
+
+/**
+ * Enriches data source rows which contain certain field types so that they can
+ * be properly displayed.
+ */
+const enrichDatasourceRows = async (rows, tableId) => {
+  if (rows && rows.length && tableId) {
+    // Fetch table schema so we can check column types
+    const tableDefinition = await fetchTableDefinition(tableId)
+    const schema = tableDefinition && tableDefinition.schema
+    if (schema) {
+      const keys = Object.keys(schema)
+      rows.forEach(row => {
+        for (let key of keys) {
+          const type = schema[key].type
+          if (type === "link") {
+            // Enrich row with the count of any relationship fields
+            row[`${key}_count`] = Array.isArray(row[key]) ? row[key].length : 0
+          } else if (type === "attachment") {
+            // Enrich row with the first image URL for any attachment fields
+            let url = null
+            if (Array.isArray(row[key]) && row[key][0] != null) {
+              url = row[key][0].url
+            }
+            row[`${key}_first`] = url
+          }
+        }
+      })
+    }
+  }
+  return rows
+}
diff --git a/packages/component-sdk/src/api/relationships.js b/packages/component-sdk/src/api/relationships.js
new file mode 100644
index 0000000000..a4d09f1250
--- /dev/null
+++ b/packages/component-sdk/src/api/relationships.js
@@ -0,0 +1,12 @@
+import api from "./api"
+
+/**
+ * Fetches related rows for a certain field of a certain row.
+ */
+export const fetchRelationshipData = async ({ tableId, rowId, fieldName }) => {
+  if (!tableId || !rowId) {
+    return []
+  }
+  const response = await api.get({ url: `/api/${tableId}/${rowId}/enrich` })
+  return response[fieldName] || []
+}
diff --git a/packages/component-sdk/src/api/rows.js b/packages/component-sdk/src/api/rows.js
index 239e33e23d..3267bfda3e 100644
--- a/packages/component-sdk/src/api/rows.js
+++ b/packages/component-sdk/src/api/rows.js
@@ -2,10 +2,6 @@ import api from "./api"
 
 /**
  * Creates a row in a table.
- *
- * @param params
- * @param state
- * @returns {Promise<any|{error: *}>}
  */
 export const saveRow = async (params, state) => {
   return await api.post({
@@ -16,10 +12,6 @@ export const saveRow = async (params, state) => {
 
 /**
  * Updates a row in a table.
- *
- * @param params
- * @param state
- * @returns {Promise<any|{error: *}>}
  */
 export const updateRow = async (params, state) => {
   const row = makeRowRequestBody(params, state)
@@ -32,11 +24,6 @@ export const updateRow = async (params, state) => {
 
 /**
  * Deletes a row from a table.
- *
- * @param tableId
- * @param rowId
- * @param revId
- * @returns {Promise<any|{error: *}>}
  */
 export const deleteRow = async ({ tableId, rowId, revId }) => {
   return await api.del({
@@ -44,6 +31,9 @@ export const deleteRow = async ({ tableId, rowId, revId }) => {
   })
 }
 
+/**
+ * Sanitises and parses column types when saving and updating rows.
+ */
 const makeRowRequestBody = (parameters, state) => {
   // start with the row thats currently in context
   const body = { ...(state.data || {}) }
diff --git a/packages/component-sdk/src/api/tables.js b/packages/component-sdk/src/api/tables.js
index e69de29bb2..f8259186d0 100644
--- a/packages/component-sdk/src/api/tables.js
+++ b/packages/component-sdk/src/api/tables.js
@@ -0,0 +1,16 @@
+import api from "./api"
+
+/**
+ * Fetches a table definition.
+ * Since definitions cannot change at runtime, the result is cached.
+ */
+export const fetchTableDefinition = async tableId => {
+  return await api.get({ url: `/api/tables/${tableId}`, cache: true })
+}
+
+/**
+ * Fetches all rows from a table.
+ */
+export const fetchTableData = async name => {
+  return await api.get({ url: `/api/views/${name}` })
+}
diff --git a/packages/component-sdk/src/api/views.js b/packages/component-sdk/src/api/views.js
new file mode 100644
index 0000000000..4f7acd2d8f
--- /dev/null
+++ b/packages/component-sdk/src/api/views.js
@@ -0,0 +1,22 @@
+import api from "./api"
+
+/**
+ * Fetches all rows in a view.
+ */
+export const fetchViewData = async ({ name, field, groupBy, calculation }) => {
+  const params = new URLSearchParams()
+
+  if (calculation) {
+    params.set("field", field)
+    params.set("calculation", calculation)
+  }
+  if (groupBy) {
+    params.set("group", groupBy)
+  }
+
+  const QUERY_VIEW_URL = field
+    ? `/api/views/${name}?${params}`
+    : `/api/views/${name}`
+
+  return await api.get({ url: QUERY_VIEW_URL })
+}
diff --git a/packages/component-sdk/src/store/auth.js b/packages/component-sdk/src/store/auth.js
index d492f65ef8..05f7709b4a 100644
--- a/packages/component-sdk/src/store/auth.js
+++ b/packages/component-sdk/src/store/auth.js
@@ -10,10 +10,6 @@ export const createAuthStore = () => {
 
   /**
    * Logs a user in.
-   *
-   * @param username
-   * @param password
-   * @returns {Promise<void>}
    */
   const logIn = async ({ username, password }) => {
     const user = await api.logIn({ username, password })
diff --git a/packages/component-sdk/src/store/config.js b/packages/component-sdk/src/store/config.js
index 5de7e73238..dbf9918ae3 100644
--- a/packages/component-sdk/src/store/config.js
+++ b/packages/component-sdk/src/store/config.js
@@ -12,8 +12,6 @@ export const createConfigStore = () => {
 
   /**
    * Sets the SDK configuration.
-   *
-   * @param config
    */
   const initialise = config => {
     store.update(state => {
@@ -33,8 +31,6 @@ export const createConfigStore = () => {
 
   /**
    * Store handler for errors which triggers the user defined error handler.
-   *
-   * @param error
    */
   const handleError = error => {
     const handler = get(store).onError
diff --git a/packages/standard-components/src/Card.svelte b/packages/standard-components/src/Card.svelte
index bfb0e0072f..63df22822a 100644
--- a/packages/standard-components/src/Card.svelte
+++ b/packages/standard-components/src/Card.svelte
@@ -1,5 +1,5 @@
 <script>
-  import { cssVars, createClasses } from "./cssVars"
+  import { cssVars } from "./helpers"
 
   export const className = ""
   export let imageUrl = ""
diff --git a/packages/standard-components/src/CardHorizontal.svelte b/packages/standard-components/src/CardHorizontal.svelte
index e3c010e507..33a5677a1c 100644
--- a/packages/standard-components/src/CardHorizontal.svelte
+++ b/packages/standard-components/src/CardHorizontal.svelte
@@ -1,5 +1,5 @@
 <script>
-  import { cssVars, createClasses } from "./cssVars"
+  import { cssVars } from "./helpers"
 
   export const className = ""
   export let imageUrl = ""
diff --git a/packages/standard-components/src/Container.svelte b/packages/standard-components/src/Container.svelte
index 0e4ff6f49b..9c2a259737 100644
--- a/packages/standard-components/src/Container.svelte
+++ b/packages/standard-components/src/Container.svelte
@@ -1,6 +1,4 @@
 <script>
-  import { cssVars, createClasses } from "./cssVars"
-
   export let className = ""
   export let onLoad
   export let type = "div"
diff --git a/packages/standard-components/src/Link.svelte b/packages/standard-components/src/Link.svelte
index ef0b71d62e..39c68d66b0 100644
--- a/packages/standard-components/src/Link.svelte
+++ b/packages/standard-components/src/Link.svelte
@@ -1,6 +1,4 @@
 <script>
-  import { cssVars, createClasses } from "./cssVars"
-
   export let url = ""
   export let text = ""
   export let openInNewTab = false
diff --git a/packages/standard-components/src/Login.svelte b/packages/standard-components/src/Login.svelte
index e125ee864b..ce4dde20d8 100644
--- a/packages/standard-components/src/Login.svelte
+++ b/packages/standard-components/src/Login.svelte
@@ -1,6 +1,4 @@
 <script>
-  import Button from "./Button.svelte"
-
   export let buttonText = "Log In"
   export let logo = ""
   export let title = ""
diff --git a/packages/standard-components/src/Nav.svelte b/packages/standard-components/src/Nav.svelte
index bc62cd4702..7619a440b4 100644
--- a/packages/standard-components/src/Nav.svelte
+++ b/packages/standard-components/src/Nav.svelte
@@ -1,5 +1,5 @@
 <script>
-  import cssVars from "./cssVars"
+  import { cssVars } from "./helpers"
 
   export let navBarBackground = ""
   export let navBarBorder = ""