Use patch when deleting components

This commit is contained in:
Andrew Kingston 2022-07-14 16:55:15 +01:00
parent 283de4b673
commit 64ca62f8d8
1 changed files with 23 additions and 28 deletions

View File

@ -431,7 +431,7 @@ export const getFrontendStore = () => {
return return
} }
// Create screen patch function // Patch screen
const patch = screen => { const patch = screen => {
// Find the selected component // Find the selected component
const currentComponent = findComponent( const currentComponent = findComponent(
@ -492,39 +492,34 @@ export const getFrontendStore = () => {
if (!component) { if (!component) {
return return
} }
const asset = get(currentAsset) const state = get(store)
if (!asset) { let parentId
return
}
// Fetch full definition // Patch screen
component = findComponent(asset.props, component._id) const patch = screen => {
// Check component exists
component = findComponent(screen.props, component._id)
if (!component) {
return false
}
// Ensure we aren't deleting the screen slot // Check component has a valid parent
if (component._component?.endsWith("/screenslot")) { const parent = findComponentParent(screen.props, component._id)
throw "You can't delete the screen slot" if (!parent) {
} return false
}
// Ensure we aren't deleting something that contains the screen slot parentId = parent._id
const screenslot = findComponentType(
component,
"@budibase/standard-components/screenslot"
)
if (screenslot != null) {
throw "You can't delete a component that contains the screen slot"
}
const parent = findComponentParent(asset.props, component._id)
if (parent) {
parent._children = parent._children.filter( parent._children = parent._children.filter(
child => child._id !== component._id child => child._id !== component._id
) )
store.update(state => {
state.selectedComponentId = parent._id
return state
})
} }
await store.actions.preview.saveSelected() await store.actions.screens.patch(state.selectedScreenId, patch)
// Select the deleted component's parent
store.update(state => {
state.selectedComponentId = parentId
return state
})
}, },
copy: (component, cut = false, selectParent = true) => { copy: (component, cut = false, selectParent = true) => {
const selectedAsset = get(currentAsset) const selectedAsset = get(currentAsset)