ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-17 02:51:36 -05:00
Code Issues Packages Projects Releases Wiki Activity

Revert the changes of row fileGroup table which was added accidentally (#24339)

Browse Source
This commit is contained in:
Sai Avishkar Sreerama
2023-09-08 12:00:48 -05:00
committed by GitHub
parent 5fdac2054d
commit 118e726fab
2 changed files with 60 additions and 173 deletions
Download Patch File Download Diff File Expand all files Collapse all files

188
extensions/mssql/src/objectManagement/ui/databaseDialog.ts
View File

@@ -20,11 +20,6 @@ const MAXDOP_Max_Limit = 32767;
const PAUSED_RESUMABLE_INDEX_Max_Limit = 71582; const PAUSED_RESUMABLE_INDEX_Max_Limit = 71582;
const DscTableRowLength = 15; const DscTableRowLength = 15;
export const tableHeaderCssStylings = { 'border': 'solid 1px #C8C8C8' };
export const tableRowCssStylings = { 'border': 'solid 1px #C8C8C8' }
// export const tableHeaderCssStylings = { 'border-top': 'solid 1px #C8C8C8', 'border-bottom': 'none', 'border-left': 'none', 'border-right': 'none' };
// export const tableRowCssStylings = { 'border-top': 'solid 1px #ccc', 'border-bottom': 'none', 'border-left': 'none', 'border-right': 'none' }
export class DatabaseDialog extends ObjectManagementDialogBase<Database, DatabaseViewInfo> { export class DatabaseDialog extends ObjectManagementDialogBase<Database, DatabaseViewInfo> {
// Database Properties tabs // Database Properties tabs
private generalTab: azdata.Tab; private generalTab: azdata.Tab;
@@ -60,7 +55,7 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
private filestreamDatafileGroupsOptions: string[]; private filestreamDatafileGroupsOptions: string[];
// fileGroups Tab // fileGroups Tab
private readonly fileGroupsTabId: string = 'fileGroupsDatabaseId'; private readonly fileGroupsTabId: string = 'fileGroupsDatabaseId';
private rowsFilegroupsTable: azdata.DeclarativeTableComponent; private rowsFilegroupsTable: azdata.TableComponent;
private filestreamFilegroupsTable: azdata.TableComponent; private filestreamFilegroupsTable: azdata.TableComponent;
private memoryOptimizedFilegroupsTable: azdata.TableComponent; private memoryOptimizedFilegroupsTable: azdata.TableComponent;
private rowsFilegroupNameInput: azdata.InputBoxComponent; private rowsFilegroupNameInput: azdata.InputBoxComponent;
@@ -663,8 +658,8 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
*/ */
private async updateFileGroupsTablesfileCount(fileType: string): Promise<void> { private async updateFileGroupsTablesfileCount(fileType: string): Promise<void> {
if (fileType === localizedConstants.RowsDataFileType) { if (fileType === localizedConstants.RowsDataFileType) {
// let data = this.getTableData(FileGroupType.RowsFileGroup); let data = this.getTableData(FileGroupType.RowsFileGroup);
// await this.setTableData(this.rowsFilegroupsTable, data); await this.setTableData(this.rowsFilegroupsTable, data);
} }
else if (fileType === localizedConstants.FilestreamFileType) { else if (fileType === localizedConstants.FilestreamFileType) {
let data = this.getTableData(FileGroupType.FileStreamDataFileGroup); let data = this.getTableData(FileGroupType.FileStreamDataFileGroup);
@@ -696,13 +691,13 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
}); });
} }
} }
// else if (table === this.rowsFilegroupsTable && this.rowsFilegroupsTable.selectedRows !== undefined && this.rowsFilegroupsTable.selectedRows.length === 1) { else if (table === this.rowsFilegroupsTable && this.rowsFilegroupsTable.selectedRows !== undefined && this.rowsFilegroupsTable.selectedRows.length === 1) {
// const selectedRow = this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRows[0]]; const selectedRow = this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRows[0]];
// // Cannot delete a row file if the fileGroup is Primary. // Cannot delete a row file if the fileGroup is Primary.
// if (selectedRow.name === 'PRIMARY' && selectedRow.id > 0) { if (selectedRow.name === 'PRIMARY' && selectedRow.id > 0) {
// isEnabled = false; isEnabled = false;
// } }
// } }
return isEnabled; return isEnabled;
} }
@@ -760,62 +755,49 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
*/ */
private async initializeRowsFileGroupSection(): Promise<azdata.GroupContainer> { private async initializeRowsFileGroupSection(): Promise<azdata.GroupContainer> {
const data = this.getTableData(FileGroupType.RowsFileGroup); const data = this.getTableData(FileGroupType.RowsFileGroup);
this.rowsFilegroupsTable = this.modelView.modelBuilder.declarativeTable().withProps({ this.rowsFilegroupsTable = this.modelView.modelBuilder.table().withProps({
columns: [{ columns: [{
valueType: azdata.DeclarativeDataType.string, type: azdata.ColumnType.text,
width: 120, value: localizedConstants.NameText,
isReadOnly: false, width: 120
displayName: localizedConstants.NameText,
headerCssStyles: tableHeaderCssStylings,
rowCssStyles: tableRowCssStylings
}, { }, {
valueType: azdata.DeclarativeDataType.string, type: azdata.ColumnType.text,
width: 60, value: localizedConstants.FilesText,
isReadOnly: true, width: 60
displayName: localizedConstants.FilesText,
headerCssStyles: tableHeaderCssStylings,
rowCssStyles: tableRowCssStylings
}, { }, {
valueType: azdata.DeclarativeDataType.boolean, type: azdata.ColumnType.checkBox,
width: 80, value: localizedConstants.ReadOnlyText,
isReadOnly: false, width: 80
displayName: localizedConstants.ReadOnlyText,
headerCssStyles: tableHeaderCssStylings,
rowCssStyles: tableRowCssStylings
}, { }, {
valueType: azdata.DeclarativeDataType.boolean, type: azdata.ColumnType.checkBox,
width: 80, value: localizedConstants.DefaultText,
isReadOnly: false, width: 80
displayName: localizedConstants.DefaultText,
headerCssStyles: tableHeaderCssStylings,
rowCssStyles: tableRowCssStylings
}, { }, {
valueType: azdata.DeclarativeDataType.boolean, type: azdata.ColumnType.checkBox,
isReadOnly: false, value: localizedConstants.AutogrowAllFilesText,
displayName: localizedConstants.AutogrowAllFilesText, width: 110
width: 110,
headerCssStyles: tableHeaderCssStylings,
rowCssStyles: tableRowCssStylings
}], }],
data: data, data: data,
height: getTableHeight(data.length, DefaultMinTableRowCount, DefaultMaxTableRowCount), height: getTableHeight(data.length, DefaultMinTableRowCount, DefaultMaxTableRowCount),
width: DefaultTableWidth, width: DefaultTableWidth,
forceFitColumns: azdata.ColumnSizingMode.DataFit,
CSSStyles: { CSSStyles: {
'margin-left': '10px' 'margin-left': '10px'
} }
}).component(); }).component();
this.rowsFilegroupNameInput = this.getFilegroupNameInput(this.rowsFilegroupsTable, FileGroupType.RowsFileGroup);
const addButtonComponent: DialogButton = { const addButtonComponent: DialogButton = {
buttonAriaLabel: localizedConstants.AddFilegroupText, buttonAriaLabel: localizedConstants.AddFilegroupText,
buttonHandler: () => this.onAddDatabaseFileGroupsButtonClicked1(this.rowsFilegroupsTable) buttonHandler: () => this.onAddDatabaseFileGroupsButtonClicked(this.rowsFilegroupsTable)
}; };
const removeButtonComponent: DialogButton = { const removeButtonComponent: DialogButton = {
buttonAriaLabel: localizedConstants.RemoveButton, buttonAriaLabel: localizedConstants.RemoveButton,
buttonHandler: () => this.onRemoveDatabaseFileGroupsButtonClicked1(this.rowsFilegroupsTable) buttonHandler: () => this.onAddDatabaseFileGroupsButtonClicked(this.rowsFilegroupsTable)
}; };
const rowsFileGroupButtonContainer = this.addButtonsForTable1(this.rowsFilegroupsTable, addButtonComponent, removeButtonComponent); const rowsFileGroupButtonContainer = this.addButtonsForTable(this.rowsFilegroupsTable, addButtonComponent, removeButtonComponent);
this.disposables.push( this.disposables.push(
this.rowsFilegroupsTable.onDataChanged(async (arg: azdata.ICheckboxCellActionEventArgs) => { this.rowsFilegroupsTable.onCellAction(async (arg: azdata.ICheckboxCellActionEventArgs) => {
let filegroup = this.rowDataFileGroupsTableRows[arg.row]; let filegroup = this.rowDataFileGroupsTableRows[arg.row];
// Read-Only column // Read-Only column
if (arg.column === 2) { if (arg.column === 2) {
@@ -831,14 +813,14 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
} }
// Refresh the table with updated data // Refresh the table with updated data
// let data = this.getTableData(FileGroupType.RowsFileGroup); let data = this.getTableData(FileGroupType.RowsFileGroup);
//await this.setTableData(this.rowsFilegroupsTable, data); await this.setTableData(this.rowsFilegroupsTable, data);
this.onFormFieldChange(); this.onFormFieldChange();
}), }),
this.rowsFilegroupsTable.onRowSelected( this.rowsFilegroupsTable.onRowSelected(
async () => { async () => {
if (this.rowsFilegroupsTable.selectedRow === 1) { if (this.rowsFilegroupsTable.selectedRows.length === 1) {
const fileGroup = this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRow]; const fileGroup = this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRows[0]];
await this.rowsFilegroupNameInput.updateCssStyles({ 'visibility': fileGroup.id < 0 ? 'visible' : 'hidden' }); await this.rowsFilegroupNameInput.updateCssStyles({ 'visibility': fileGroup.id < 0 ? 'visible' : 'hidden' });
this.rowsFilegroupNameInput.value = fileGroup.name; this.rowsFilegroupNameInput.value = fileGroup.name;
this.onFormFieldChange(); this.onFormFieldChange();
@@ -846,7 +828,9 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
} }
) )
); );
return this.createGroup(localizedConstants.RowsFileGroupsSectionText, [this.rowsFilegroupsTable, rowsFileGroupButtonContainer], true); const rowContainer = this.modelView.modelBuilder.flexContainer().withItems([this.rowsFilegroupNameInput]).component();
rowContainer.addItems([rowsFileGroupButtonContainer], { flex: '0 0 auto' });
return this.createGroup(localizedConstants.RowsFileGroupsSectionText, [this.rowsFilegroupsTable, rowContainer], true);
} }
/** /**
@@ -1005,16 +989,15 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
isDefault: false, isDefault: false,
autogrowAllFiles: false autogrowAllFiles: false
}; };
// if (table === this.rowsFilegroupsTable) { if (table === this.rowsFilegroupsTable) {
// newRow.type = FileGroupType.RowsFileGroup; newRow.type = FileGroupType.RowsFileGroup;
// newRow.isReadOnly = false; newRow.isReadOnly = false;
// newRow.isDefault = false; newRow.isDefault = false;
// newRow.autogrowAllFiles = false newRow.autogrowAllFiles = false
// this.objectInfo.filegroups?.push(newRow); this.objectInfo.filegroups?.push(newRow);
// newData = this.getTableData(FileGroupType.RowsFileGroup); newData = this.getTableData(FileGroupType.RowsFileGroup);
// } }
// else else if (table === this.filestreamFilegroupsTable) {
if (table === this.filestreamFilegroupsTable) {
newRow.type = FileGroupType.FileStreamDataFileGroup; newRow.type = FileGroupType.FileStreamDataFileGroup;
newRow.isReadOnly = false; newRow.isReadOnly = false;
newRow.isDefault = false; newRow.isDefault = false;
@@ -1034,55 +1017,6 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
} }
} }
private async onAddDatabaseFileGroupsButtonClicked1(table: azdata.DeclarativeTableComponent): Promise<void> {
let newData: any[] | undefined;
let newRow: FileGroup = {
id: --this.newFileGroupTemporaryId,
name: '',
type: undefined,
isReadOnly: false,
isDefault: false,
autogrowAllFiles: false
};
if (table === this.rowsFilegroupsTable) {
newRow.type = FileGroupType.RowsFileGroup;
newRow.isReadOnly = false;
newRow.isDefault = false;
newRow.autogrowAllFiles = false
this.objectInfo.filegroups?.push(newRow);
newData = this.getTableData(FileGroupType.RowsFileGroup);
}
if (newData !== undefined) {
// Refresh the table with new row data
this.updateFileGroupsOptionsAndTableRows();
// await this.setTableData(table, newData, DefaultMaxTableRowCount);
await table.updateProperties({
data: newData,
height: getTableHeight(newData.length, DefaultMinTableRowCount)
});
}
}
private async onRemoveDatabaseFileGroupsButtonClicked1(table: azdata.DeclarativeTableComponent): Promise<void> {
if (table === this.rowsFilegroupsTable) {
if (this.rowsFilegroupsTable.selectedRow === 1) {
const removeFilegroupIndex = this.objectInfo.filegroups.indexOf(this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRow]);
this.objectInfo.filegroups?.splice(removeFilegroupIndex, 1);
var newData = this.getTableData(FileGroupType.RowsFileGroup);
await this.rowsFilegroupNameInput.updateCssStyles({ 'visibility': 'hidden' });
}
}
// Refresh the individual table rows object and table with updated data
this.updateFileGroupsOptionsAndTableRows();
await table.updateProperties({
data: newData,
height: getTableHeight(newData.length, DefaultMinTableRowCount)
});
}
/** /**
* Prepares the individual table rows for each filegroup type and list of filegroups options * Prepares the individual table rows for each filegroup type and list of filegroups options
* This will be useful to get the selected row data from the table to get the filegroup property details, helps when have duplicate rows added * This will be useful to get the selected row data from the table to get the filegroup property details, helps when have duplicate rows added
@@ -1107,16 +1041,15 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
* @param table table component * @param table table component
*/ */
private async onRemoveDatabaseFileGroupsButtonClicked(table: azdata.TableComponent): Promise<void> { private async onRemoveDatabaseFileGroupsButtonClicked(table: azdata.TableComponent): Promise<void> {
// if (table === this.rowsFilegroupsTable) { if (table === this.rowsFilegroupsTable) {
// if (this.rowsFilegroupsTable.selectedRows.length === 1) { if (this.rowsFilegroupsTable.selectedRows.length === 1) {
// const removeFilegroupIndex = this.objectInfo.filegroups.indexOf(this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRows[0]]); const removeFilegroupIndex = this.objectInfo.filegroups.indexOf(this.rowDataFileGroupsTableRows[this.rowsFilegroupsTable.selectedRows[0]]);
// this.objectInfo.filegroups?.splice(removeFilegroupIndex, 1); this.objectInfo.filegroups?.splice(removeFilegroupIndex, 1);
// var newData = this.getTableData(FileGroupType.RowsFileGroup); var newData = this.getTableData(FileGroupType.RowsFileGroup);
// await this.rowsFilegroupNameInput.updateCssStyles({ 'visibility': 'hidden' }); await this.rowsFilegroupNameInput.updateCssStyles({ 'visibility': 'hidden' });
// } }
// } }
// else else if (table === this.filestreamFilegroupsTable) {
if (table === this.filestreamFilegroupsTable) {
if (this.filestreamFilegroupsTable.selectedRows.length === 1) { if (this.filestreamFilegroupsTable.selectedRows.length === 1) {
const removeFilegroupIndex = this.objectInfo.filegroups.indexOf(this.filestreamDataFileGroupsTableRows[this.filestreamFilegroupsTable.selectedRows[0]]); const removeFilegroupIndex = this.objectInfo.filegroups.indexOf(this.filestreamDataFileGroupsTableRows[this.filestreamFilegroupsTable.selectedRows[0]]);
this.objectInfo.filegroups?.splice(removeFilegroupIndex, 1); this.objectInfo.filegroups?.splice(removeFilegroupIndex, 1);
@@ -1148,10 +1081,9 @@ export class DatabaseDialog extends ObjectManagementDialogBase<Database, Databas
return this.createInputBox(async (value) => { return this.createInputBox(async (value) => {
if (table.selectedRows.length === 1) { if (table.selectedRows.length === 1) {
let fg = null; let fg = null;
// if (table === this.rowsFilegroupsTable) { if (table === this.rowsFilegroupsTable) {
// fg = this.rowDataFileGroupsTableRows[table.selectedRows[0]]; fg = this.rowDataFileGroupsTableRows[table.selectedRows[0]];
// } else } else if (table === this.filestreamFilegroupsTable) {
if (table === this.filestreamFilegroupsTable) {
fg = this.filestreamDataFileGroupsTableRows[table.selectedRows[0]]; fg = this.filestreamDataFileGroupsTableRows[table.selectedRows[0]];
} else if (table === this.memoryOptimizedFilegroupsTable) { } else if (table === this.memoryOptimizedFilegroupsTable) {
fg = this.memoryoptimizedFileGroupsTableRows[table.selectedRows[0]]; fg = this.memoryoptimizedFileGroupsTableRows[table.selectedRows[0]];

45
extensions/mssql/src/ui/dialogBase.ts
View File

@@ -328,51 +328,6 @@ export abstract class DialogBase<DialogResult> {
return this.createButtonContainer(buttonComponents) return this.createButtonContainer(buttonComponents)
} }
protected addButtonsForTable1(table: azdata.DeclarativeTableComponent, addbutton: DialogButton, removeButton: DialogButton, editButton: DialogButton = undefined): azdata.FlexContainer {
let addButtonComponent: azdata.ButtonComponent;
let editButtonComponent: azdata.ButtonComponent;
let removeButtonComponent: azdata.ButtonComponent;
let buttonComponents: azdata.ButtonComponent[] = [];
const updateButtons = (isRemoveEnabled: boolean = undefined) => {
this.onFormFieldChange();
const tableSelectedRowsLengthCheck = table.selectedRow === 1 && table.selectedRow < table.data.length;
if (editButton !== undefined) {
editButtonComponent.enabled = tableSelectedRowsLengthCheck;
}
removeButtonComponent.enabled = !!isRemoveEnabled && tableSelectedRowsLengthCheck;
}
addButtonComponent = this.createButton(uiLoc.AddText, addbutton.buttonAriaLabel, async () => {
await addbutton.buttonHandler(addButtonComponent);
updateButtons();
});
buttonComponents.push(addButtonComponent);
if (editButton !== undefined) {
editButtonComponent = this.createButton(uiLoc.EditText, editButton.buttonAriaLabel, async () => {
await editButton.buttonHandler(editButtonComponent);
updateButtons();
}, false);
buttonComponents.push(editButtonComponent);
}
removeButtonComponent = this.createButton(uiLoc.RemoveText, removeButton.buttonAriaLabel, async () => {
await removeButton.buttonHandler(removeButtonComponent);
if (table.selectedRow === 1) {
// table.selectedRow = [table.data.length - 1];
}
updateButtons();
}, false);
buttonComponents.push(removeButtonComponent);
this.disposables.push(table.onRowSelected(() => {
const isRemoveButtonEnabled = true;
updateButtons(isRemoveButtonEnabled);
}));
return this.createButtonContainer(buttonComponents)
}
protected createDropdown(ariaLabel: string, handler: (newValue: string) => Promise<void>, values: string[], value: string | undefined, enabled: boolean = true, width: number = DefaultInputWidth, editable?: boolean, strictSelection?: boolean): azdata.DropDownComponent { protected createDropdown(ariaLabel: string, handler: (newValue: string) => Promise<void>, values: string[], value: string | undefined, enabled: boolean = true, width: number = DefaultInputWidth, editable?: boolean, strictSelection?: boolean): azdata.DropDownComponent {
// Automatically add an empty item to the beginning of the list if the current value is not specified. // Automatically add an empty item to the beginning of the list if the current value is not specified.
// This is needed when no meaningful default value can be provided. // This is needed when no meaningful default value can be provided.