Merge pull request #6845 from Budibase/fix/public-app-api

Fixing issue with app DB not being in context for fetch.

packages/server/src/api/controllers/public/applications.ts

@@ -1,5 +1,5 @@
 const { getAllApps } = require("@budibase/backend-core/db")
-const { updateAppId } = require("@budibase/backend-core/context")
+const { doInAppContext } = require("@budibase/backend-core/context")
 import { search as stringSearch, addRev } from "./utils"
 import * as controller from "../application"
 import { Application } from "../../../definitions/common"
@@ -41,28 +41,31 @@ export async function create(ctx: any, next: any) {
 }
 
 export async function read(ctx: any, next: any) {
-  updateAppId(ctx.params.appId)
+  await doInAppContext(ctx.params.appId, async () => {
-  await setResponseApp(ctx)
+    await setResponseApp(ctx)
-  await next()
+    await next()
+  })
 }
 
 export async function update(ctx: any, next: any) {
   ctx.request.body = await addRev(fixAppID(ctx.request.body, ctx.params))
-  updateAppId(ctx.params.appId)
+  await doInAppContext(ctx.params.appId, async () => {
-  await controller.update(ctx)
+    await controller.update(ctx)
-  await setResponseApp(ctx)
+    await setResponseApp(ctx)
-  await next()
+    await next()
+  })
 }
 
 export async function destroy(ctx: any, next: any) {
-  updateAppId(ctx.params.appId)
+  await doInAppContext(ctx.params.appId, async () => {
-  // get the app before deleting it
+    // get the app before deleting it
-  await setResponseApp(ctx)
+    await setResponseApp(ctx)
-  const body = ctx.body
+    const body = ctx.body
-  await controller.destroy(ctx)
+    await controller.destroy(ctx)
-  // overwrite the body again
+    // overwrite the body again
-  ctx.body = body
+    ctx.body = body
-  await next()
+    await next()
+  })
 }
 
 export default {