## Description - Remove the primary column from the insert and update queries. - Save/Discard button isSaveDisabled and isDiscardDisabled properties should be in js mode. - Don't create insert and update query if datasource is read only #### PR fixes following issue(s) Fixes https://github.com/appsmithorg/appsmith/issues/24858 #### Type of change - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) > > > ## Testing > #### How Has This Been Tested? > Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration. > Delete anything that is not relevant - [x] Manual - [ ] Jest - [x] Cypress > > #### Test Plan > Add Testsmith test cases links that relate to this PR > > #### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) > > > ## Checklist: #### Dev activity - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag #### QA activity: - [ ] [Speedbreak features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-) have been covered - [ ] Test plan covers all impacted features and [areas of interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-) - [ ] Test plan has been peer reviewed by project stakeholders and other QA members - [ ] Manually tested functionality on DP - [ ] We had an implementation alignment call with stakeholders post QA Round 2 - [ ] Cypress test cases have been added and approved by SDET/manual QA - [ ] Added `Test Plan Approved` label after Cypress tests were reviewed - [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
246 lines
6.5 KiB
TypeScript
246 lines
6.5 KiB
TypeScript
import { BaseQueryGenerator } from "../BaseQueryGenerator";
|
|
import { formatDialect, postgresql } from "sql-formatter";
|
|
import { QUERY_TYPE } from "../types";
|
|
import type {
|
|
WidgetQueryGenerationConfig,
|
|
WidgetQueryGenerationFormConfig,
|
|
ActionConfigurationSQL,
|
|
} from "../types";
|
|
import { removeSpecialChars } from "utils/helpers";
|
|
import { without } from "lodash";
|
|
import { DatasourceConnectionMode } from "entities/Datasource";
|
|
export default abstract class PostgreSQL extends BaseQueryGenerator {
|
|
private static buildSelect(
|
|
widgetConfig: WidgetQueryGenerationConfig,
|
|
formConfig: WidgetQueryGenerationFormConfig,
|
|
) {
|
|
const { select } = widgetConfig;
|
|
//if no table name do not build query
|
|
if (!select || !formConfig.tableName) {
|
|
return;
|
|
}
|
|
|
|
const { limit, offset, orderBy, sortOrder, where } = select;
|
|
|
|
const querySegments = [
|
|
{
|
|
isValuePresent: formConfig.tableName,
|
|
template: "SELECT * FROM $1",
|
|
params: {
|
|
1: formConfig.tableName,
|
|
},
|
|
},
|
|
{
|
|
isValuePresent: formConfig.searchableColumn && where,
|
|
template: "WHERE $2 ilike $3",
|
|
params: {
|
|
2: `"${formConfig.searchableColumn}"`,
|
|
3: `'%{{${where}}}%'`,
|
|
},
|
|
},
|
|
formConfig.primaryColumn
|
|
? {
|
|
isValuePresent: orderBy,
|
|
template: `ORDER BY $4 $5`,
|
|
params: {
|
|
4: `"{{${orderBy} || '${formConfig.primaryColumn}'}}"`,
|
|
5: `{{${sortOrder} ? "" : "DESC"}}`,
|
|
},
|
|
}
|
|
: {
|
|
isValuePresent: orderBy,
|
|
template: "$4",
|
|
params: {
|
|
4: `{{${orderBy} ? "ORDER BY " + ${orderBy} + " " + (${sortOrder} ? "" : "DESC") : ""}}`,
|
|
},
|
|
},
|
|
{
|
|
isValuePresent: limit,
|
|
template: "LIMIT $6",
|
|
params: {
|
|
6: `{{${limit}}}`,
|
|
},
|
|
},
|
|
{
|
|
isValuePresent: offset,
|
|
template: "OFFSET $7",
|
|
params: {
|
|
7: `{{${offset}}}`,
|
|
},
|
|
},
|
|
];
|
|
|
|
const { params, template } = querySegments
|
|
//we need to filter out query segments which are not defined
|
|
.filter(({ isValuePresent }) => !!isValuePresent)
|
|
.reduce(
|
|
(acc, curr) => {
|
|
const { params, template } = curr;
|
|
return {
|
|
template: acc.template + " " + template,
|
|
params: { ...acc.params, ...params },
|
|
};
|
|
},
|
|
{ template: "", params: {} },
|
|
);
|
|
//formats sql string
|
|
const res = formatDialect(template, {
|
|
params,
|
|
dialect: postgresql,
|
|
});
|
|
|
|
return {
|
|
type: QUERY_TYPE.SELECT,
|
|
name: `Select_${removeSpecialChars(formConfig.tableName)}`,
|
|
payload: {
|
|
body: res,
|
|
},
|
|
dynamicBindingPathList: [
|
|
{
|
|
key: "body",
|
|
},
|
|
],
|
|
};
|
|
}
|
|
|
|
private static buildUpdate(
|
|
widgetConfig: WidgetQueryGenerationConfig,
|
|
formConfig: WidgetQueryGenerationFormConfig,
|
|
) {
|
|
const { update } = widgetConfig;
|
|
//if no table name do not build query
|
|
if (!update || !update.where || !formConfig.tableName) {
|
|
return;
|
|
}
|
|
|
|
const { value, where } = update;
|
|
|
|
const columns = without(formConfig.columns, formConfig.primaryColumn);
|
|
|
|
return {
|
|
type: QUERY_TYPE.UPDATE,
|
|
name: `Update_${removeSpecialChars(formConfig.tableName)}`,
|
|
payload: {
|
|
body: `UPDATE ${formConfig.tableName} SET ${columns
|
|
.map((column) => `"${column}"= '{{${value}.${column}}}'`)
|
|
.join(", ")} WHERE "${formConfig.primaryColumn}"= {{${where}.${
|
|
formConfig.primaryColumn
|
|
}}};`,
|
|
},
|
|
dynamicBindingPathList: [
|
|
{
|
|
key: "body",
|
|
},
|
|
],
|
|
};
|
|
}
|
|
|
|
private static buildInsert(
|
|
widgetConfig: WidgetQueryGenerationConfig,
|
|
formConfig: WidgetQueryGenerationFormConfig,
|
|
) {
|
|
const { create } = widgetConfig;
|
|
//if no table name do not build query
|
|
if (!create || !create.value || !formConfig.tableName) {
|
|
return;
|
|
}
|
|
|
|
const columns = without(formConfig.columns, formConfig.primaryColumn);
|
|
|
|
return {
|
|
type: QUERY_TYPE.CREATE,
|
|
name: `Insert_${removeSpecialChars(formConfig.tableName)}`,
|
|
payload: {
|
|
body: `INSERT INTO ${formConfig.tableName} (${columns.map(
|
|
(a) => `"${a}"`,
|
|
)}) VALUES (${columns
|
|
.map((d) => `'{{${create.value}.${d}}}'`)
|
|
.toString()})`,
|
|
},
|
|
dynamicBindingPathList: [
|
|
{
|
|
key: "body",
|
|
},
|
|
],
|
|
};
|
|
}
|
|
|
|
private static buildTotal(
|
|
widgetConfig: WidgetQueryGenerationConfig,
|
|
formConfig: WidgetQueryGenerationFormConfig,
|
|
) {
|
|
const { select, totalRecord } = widgetConfig;
|
|
//if no table name do not build query
|
|
if (!totalRecord) {
|
|
return;
|
|
}
|
|
|
|
return {
|
|
type: QUERY_TYPE.TOTAL_RECORD,
|
|
name: `Total_record_${removeSpecialChars(formConfig.tableName)}`,
|
|
payload: {
|
|
body: `SELECT COUNT(*) from ${formConfig.tableName}${
|
|
formConfig.searchableColumn
|
|
? ` where ${formConfig.searchableColumn} ilike '%{{${select?.where}}}%'`
|
|
: ""
|
|
};`,
|
|
},
|
|
dynamicBindingPathList: [
|
|
{
|
|
key: "body",
|
|
},
|
|
],
|
|
};
|
|
}
|
|
|
|
public static build(
|
|
widgetConfig: WidgetQueryGenerationConfig,
|
|
formConfig: WidgetQueryGenerationFormConfig,
|
|
pluginInitalValues: { actionConfiguration: ActionConfigurationSQL },
|
|
) {
|
|
const allBuildConfigs = [];
|
|
if (widgetConfig.select) {
|
|
allBuildConfigs.push(this.buildSelect(widgetConfig, formConfig));
|
|
}
|
|
|
|
if (
|
|
widgetConfig.update &&
|
|
formConfig.primaryColumn &&
|
|
formConfig.connectionMode === DatasourceConnectionMode.READ_WRITE
|
|
) {
|
|
allBuildConfigs.push(this.buildUpdate(widgetConfig, formConfig));
|
|
}
|
|
|
|
if (
|
|
widgetConfig.create &&
|
|
formConfig.primaryColumn &&
|
|
formConfig.connectionMode === DatasourceConnectionMode.READ_WRITE
|
|
) {
|
|
allBuildConfigs.push(this.buildInsert(widgetConfig, formConfig));
|
|
}
|
|
|
|
if (widgetConfig.totalRecord) {
|
|
allBuildConfigs.push(this.buildTotal(widgetConfig, formConfig));
|
|
}
|
|
|
|
return allBuildConfigs
|
|
.filter((val) => !!val)
|
|
.map((val) => ({
|
|
...val,
|
|
payload: {
|
|
...(val?.payload || {}),
|
|
...(pluginInitalValues?.actionConfiguration || {}),
|
|
pluginSpecifiedTemplates: [
|
|
{
|
|
value: false,
|
|
},
|
|
],
|
|
},
|
|
}));
|
|
}
|
|
|
|
static getTotalRecordExpression(binding: string) {
|
|
return `${binding}[0].count`;
|
|
}
|
|
}
|