mirror of
https://github.com/hexastack/hexabot
synced 2025-06-26 18:27:28 +00:00
feat: add bulk delete functionality
This commit is contained in:
parent
ddb5e896de
commit
aa25ce6b03
@ -6,7 +6,7 @@
|
|||||||
* 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
|
* 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
|
||||||
*/
|
*/
|
||||||
|
|
||||||
import { NotFoundException } from '@nestjs/common';
|
import { BadRequestException, NotFoundException } from '@nestjs/common';
|
||||||
import { EventEmitter2 } from '@nestjs/event-emitter';
|
import { EventEmitter2 } from '@nestjs/event-emitter';
|
||||||
import { MongooseModule } from '@nestjs/mongoose';
|
import { MongooseModule } from '@nestjs/mongoose';
|
||||||
import { Test } from '@nestjs/testing';
|
import { Test } from '@nestjs/testing';
|
||||||
@ -143,6 +143,44 @@ describe('ContextVarController', () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('deleteMany', () => {
|
||||||
|
const deleteResult = { acknowledged: true, deletedCount: 2 };
|
||||||
|
|
||||||
|
it('should delete contextVars when valid IDs are provided', async () => {
|
||||||
|
jest
|
||||||
|
.spyOn(contextVarService, 'deleteMany')
|
||||||
|
.mockResolvedValue(deleteResult);
|
||||||
|
const result = await contextVarController.deleteMany([
|
||||||
|
contextVarToDelete.id,
|
||||||
|
contextVar.id,
|
||||||
|
]);
|
||||||
|
|
||||||
|
expect(contextVarService.deleteMany).toHaveBeenCalledWith({
|
||||||
|
_id: { $in: [contextVarToDelete.id, contextVar.id] },
|
||||||
|
});
|
||||||
|
expect(result).toEqual(deleteResult);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should throw BadRequestException when no IDs are provided', async () => {
|
||||||
|
await expect(contextVarController.deleteMany([])).rejects.toThrow(
|
||||||
|
new BadRequestException('No IDs provided for deletion.'),
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should throw NotFoundException when no contextVars are deleted', async () => {
|
||||||
|
jest.spyOn(contextVarService, 'deleteMany').mockResolvedValue({
|
||||||
|
acknowledged: true,
|
||||||
|
deletedCount: 0,
|
||||||
|
});
|
||||||
|
|
||||||
|
await expect(
|
||||||
|
contextVarController.deleteMany([contextVarToDelete.id, contextVar.id]),
|
||||||
|
).rejects.toThrow(
|
||||||
|
new NotFoundException('Context vars with provided IDs not found'),
|
||||||
|
);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
describe('updateOne', () => {
|
describe('updateOne', () => {
|
||||||
const contextVarUpdatedDto: ContextVarUpdateDto = {
|
const contextVarUpdatedDto: ContextVarUpdateDto = {
|
||||||
name: 'updated_context_var_name',
|
name: 'updated_context_var_name',
|
||||||
|
@ -7,6 +7,7 @@
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
import {
|
import {
|
||||||
|
BadRequestException,
|
||||||
Body,
|
Body,
|
||||||
Controller,
|
Controller,
|
||||||
Delete,
|
Delete,
|
||||||
@ -144,4 +145,31 @@ export class ContextVarController extends BaseController<ContextVar> {
|
|||||||
}
|
}
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Deletes multiple categories by their IDs.
|
||||||
|
* @param ids - IDs of categories to be deleted.
|
||||||
|
* @returns A Promise that resolves to the deletion result.
|
||||||
|
*/
|
||||||
|
@CsrfCheck(true)
|
||||||
|
@Delete('')
|
||||||
|
@HttpCode(204)
|
||||||
|
async deleteMany(@Body('ids') ids: string[]): Promise<DeleteResult> {
|
||||||
|
if (!ids || ids.length === 0) {
|
||||||
|
throw new BadRequestException('No IDs provided for deletion.');
|
||||||
|
}
|
||||||
|
const deleteResult = await this.contextVarService.deleteMany({
|
||||||
|
_id: { $in: ids },
|
||||||
|
});
|
||||||
|
|
||||||
|
if (deleteResult.deletedCount === 0) {
|
||||||
|
this.logger.warn(
|
||||||
|
`Unable to delete context vars with provided IDs: ${ids}`,
|
||||||
|
);
|
||||||
|
throw new NotFoundException('Context vars with provided IDs not found');
|
||||||
|
}
|
||||||
|
|
||||||
|
this.logger.log(`Successfully deleted context vars with IDs: ${ids}`);
|
||||||
|
return deleteResult;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
@ -8,9 +8,10 @@
|
|||||||
|
|
||||||
import { faAsterisk } from "@fortawesome/free-solid-svg-icons";
|
import { faAsterisk } from "@fortawesome/free-solid-svg-icons";
|
||||||
import AddIcon from "@mui/icons-material/Add";
|
import AddIcon from "@mui/icons-material/Add";
|
||||||
|
import DeleteIcon from "@mui/icons-material/Delete";
|
||||||
import { Button, Grid, Paper, Switch } from "@mui/material";
|
import { Button, Grid, Paper, Switch } from "@mui/material";
|
||||||
import { GridColDef } from "@mui/x-data-grid";
|
import { GridColDef, GridRowSelectionModel } from "@mui/x-data-grid";
|
||||||
import React from "react";
|
import React, { useState } from "react";
|
||||||
|
|
||||||
import { DeleteDialog } from "@/app-components/dialogs/DeleteDialog";
|
import { DeleteDialog } from "@/app-components/dialogs/DeleteDialog";
|
||||||
import { FilterTextfield } from "@/app-components/inputs/FilterTextfield";
|
import { FilterTextfield } from "@/app-components/inputs/FilterTextfield";
|
||||||
@ -21,6 +22,7 @@ import {
|
|||||||
import { renderHeader } from "@/app-components/tables/columns/renderHeader";
|
import { renderHeader } from "@/app-components/tables/columns/renderHeader";
|
||||||
import { DataGrid } from "@/app-components/tables/DataGrid";
|
import { DataGrid } from "@/app-components/tables/DataGrid";
|
||||||
import { useDelete } from "@/hooks/crud/useDelete";
|
import { useDelete } from "@/hooks/crud/useDelete";
|
||||||
|
import { useDeleteMany } from "@/hooks/crud/useDeleteMany";
|
||||||
import { useFind } from "@/hooks/crud/useFind";
|
import { useFind } from "@/hooks/crud/useFind";
|
||||||
import { useUpdate } from "@/hooks/crud/useUpdate";
|
import { useUpdate } from "@/hooks/crud/useUpdate";
|
||||||
import { getDisplayDialogs, useDialog } from "@/hooks/useDialog";
|
import { getDisplayDialogs, useDialog } from "@/hooks/useDialog";
|
||||||
@ -66,9 +68,24 @@ export const ContextVars = () => {
|
|||||||
},
|
},
|
||||||
onSuccess() {
|
onSuccess() {
|
||||||
deleteDialogCtl.closeDialog();
|
deleteDialogCtl.closeDialog();
|
||||||
|
setSelectedContextVars([]);
|
||||||
toast.success(t("message.item_delete_success"));
|
toast.success(t("message.item_delete_success"));
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
const { mutateAsync: deleteContextVars } = useDeleteMany(
|
||||||
|
EntityType.CONTEXT_VAR,
|
||||||
|
{
|
||||||
|
onError: (error) => {
|
||||||
|
toast.error(error.message || t("message.internal_server_error"));
|
||||||
|
},
|
||||||
|
onSuccess: () => {
|
||||||
|
deleteDialogCtl.closeDialog();
|
||||||
|
setSelectedContextVars([]);
|
||||||
|
toast.success(t("message.item_delete_success"));
|
||||||
|
},
|
||||||
|
},
|
||||||
|
);
|
||||||
|
const [selectedContextVars, setSelectedContextVars] = useState<string[]>([]);
|
||||||
const actionColumns = useActionColumns<IContextVar>(
|
const actionColumns = useActionColumns<IContextVar>(
|
||||||
EntityType.CONTEXT_VAR,
|
EntityType.CONTEXT_VAR,
|
||||||
[
|
[
|
||||||
@ -143,6 +160,9 @@ export const ContextVars = () => {
|
|||||||
},
|
},
|
||||||
actionColumns,
|
actionColumns,
|
||||||
];
|
];
|
||||||
|
const handleSelectionChange = (selection: GridRowSelectionModel) => {
|
||||||
|
setSelectedContextVars(selection as string[]);
|
||||||
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<Grid container gap={3} flexDirection="column">
|
<Grid container gap={3} flexDirection="column">
|
||||||
@ -152,7 +172,13 @@ export const ContextVars = () => {
|
|||||||
<DeleteDialog
|
<DeleteDialog
|
||||||
{...deleteDialogCtl}
|
{...deleteDialogCtl}
|
||||||
callback={() => {
|
callback={() => {
|
||||||
if (deleteDialogCtl?.data) deleteContextVar(deleteDialogCtl.data);
|
if (selectedContextVars.length > 0) {
|
||||||
|
deleteContextVars(selectedContextVars);
|
||||||
|
setSelectedContextVars([]);
|
||||||
|
deleteDialogCtl.closeDialog();
|
||||||
|
} else if (deleteDialogCtl?.data) {
|
||||||
|
deleteContextVar(deleteDialogCtl.data);
|
||||||
|
}
|
||||||
}}
|
}}
|
||||||
/>
|
/>
|
||||||
<PageHeader icon={faAsterisk} title={t("title.context_vars")}>
|
<PageHeader icon={faAsterisk} title={t("title.context_vars")}>
|
||||||
@ -179,12 +205,29 @@ export const ContextVars = () => {
|
|||||||
</Button>
|
</Button>
|
||||||
</Grid>
|
</Grid>
|
||||||
) : null}
|
) : null}
|
||||||
|
{selectedContextVars.length > 0 && (
|
||||||
|
<Grid item>
|
||||||
|
<Button
|
||||||
|
startIcon={<DeleteIcon />}
|
||||||
|
variant="contained"
|
||||||
|
color="error"
|
||||||
|
onClick={() => deleteDialogCtl.openDialog(undefined)}
|
||||||
|
>
|
||||||
|
{t("button.delete")}
|
||||||
|
</Button>
|
||||||
|
</Grid>
|
||||||
|
)}
|
||||||
</Grid>
|
</Grid>
|
||||||
</PageHeader>
|
</PageHeader>
|
||||||
<Grid item xs={12}>
|
<Grid item xs={12}>
|
||||||
<Paper sx={{ padding: 2 }}>
|
<Paper sx={{ padding: 2 }}>
|
||||||
<Grid>
|
<Grid>
|
||||||
<DataGrid columns={columns} {...dataGridProps} />
|
<DataGrid
|
||||||
|
columns={columns}
|
||||||
|
{...dataGridProps}
|
||||||
|
checkboxSelection
|
||||||
|
onRowSelectionModelChange={handleSelectionChange}
|
||||||
|
/>
|
||||||
</Grid>
|
</Grid>
|
||||||
</Paper>
|
</Paper>
|
||||||
</Grid>
|
</Grid>
|
||||||
|
Loading…
Reference in New Issue
Block a user