CB-6063 feat: add ability to import data without PK

webapp/packages/plugin-data-spreadsheet-new/src/DataGrid/CellRenderer/CellRenderer.tsx

@@ -103,7 +103,10 @@ export const CellRenderer = observer<CellRendererProps<IResultSetRowKey, unknown
   );
 
   function isEditable(column: CalculatedColumn<IResultSetRowKey>): boolean {
-    if (!cellContext.cell) {
+    if (
+      !cellContext.cell ||
+      (!dataGridContext.model.hasRowIdentifier(tableDataContext.view.resultIndex) && cellContext.editionState !== DatabaseEditChangeType.add)
+    ) {
       return false;
     }
 

webapp/packages/plugin-data-spreadsheet-new/src/DataGrid/DataGridContextMenu/DataGridContextMenuCellEditingService.ts

@@ -86,29 +86,32 @@ export class DataGridContextMenuCellEditingService {
         const isComplex = format.isBinary(key) || format.isGeometry(key);
         const isTruncated = content.isTextTruncated(key);
         const selectedElements = select?.getSelectedElements() || [];
+        const hasRowIdentifier = model.hasRowIdentifier(resultIndex);
+        // we can't edit table cells if table doesn't have row identifier, but we can edit new created rows before insert (CB-6063)
+        const canEdit = hasRowIdentifier || (!hasRowIdentifier && editor.getElementState(key) === DatabaseEditChangeType.add);
 
         if (action === ACTION_EDIT) {
-          if (!column || cellValue === undefined || format.isReadOnly(key) || isComplex || isTruncated) {
+          if (!column || cellValue === undefined || format.isReadOnly(key) || isComplex || isTruncated || !canEdit) {
             return false;
           }
 
           return !isBooleanValuePresentationAvailable(cellValue, column);
         }
 
         if (action === ACTION_DATA_GRID_EDITING_SET_TO_NULL) {
-          return cellValue !== undefined && !format.isReadOnly(key) && !view.getColumn(key.column)?.required && !format.isNull(key);
+          return cellValue !== undefined && !format.isReadOnly(key) && !view.getColumn(key.column)?.required && !format.isNull(key) && canEdit;
         }
 
         if (action === ACTION_DATA_GRID_EDITING_ADD_ROW || action === ACTION_DATA_GRID_EDITING_DUPLICATE_ROW) {
           return editor.hasFeature('add');
         }
 
         if (action === ACTION_DATA_GRID_EDITING_DELETE_ROW) {
-          return !format.isReadOnly(key) && editor.getElementState(key) !== DatabaseEditChangeType.delete;
+          return !format.isReadOnly(key) && canEdit && editor.getElementState(key) !== DatabaseEditChangeType.delete;
         }
 
         if (action === ACTION_DATA_GRID_EDITING_DELETE_SELECTED_ROW) {
-          if (model.isReadonly(resultIndex) || !editor.hasFeature('delete')) {
+          if (model.isReadonly(resultIndex) || !canEdit || !editor.hasFeature('delete')) {
             return false;
           }
 

webapp/packages/plugin-data-spreadsheet-new/src/DataGrid/DataGridTable.tsx

@@ -222,12 +222,16 @@ export const DataGridTable = observer<IDataPresentationProps>(function DataGridT
 
   function handleKeyDown(event: React.KeyboardEvent<HTMLDivElement>) {
     gridSelectedCellCopy.onKeydownHandler(event);
+    const cell = selectionAction.getFocusedElement();
+    // we can't edit table cells if table doesn't have row identifier, but we can edit new created rows before insert (CB-6063)
+    const canEdit =
+      model.hasRowIdentifier(resultIndex) ||
+      !!(!model.hasRowIdentifier(resultIndex) && cell && tableData.editor.getElementState(cell) === DatabaseEditChangeType.add);
 
-    if (EventContext.has(event, EventStopPropagationFlag) || tableData.isReadOnly() || model.isReadonly(resultIndex)) {
+    if (EventContext.has(event, EventStopPropagationFlag) || !canEdit || tableData.isReadOnly() || model.isReadonly(resultIndex)) {
       return;
     }
 
-    const cell = selectionAction.getFocusedElement();
     const activeElements = selectionAction.getActiveElements();
     const activeRows = selectionAction.getActiveRows();
 

webapp/packages/plugin-data-viewer/src/DatabaseDataModel/DatabaseDataModel.ts

@@ -59,6 +59,10 @@ export class DatabaseDataModel<TSource extends IDatabaseDataSource<any, any> = I
     return this.source.isReadonly(resultIndex);
   }
 
+  hasRowIdentifier(resultIndex: number): boolean {
+    return this.source.hasRowIdentifier(resultIndex);
+  }
+
   isDataAvailable(offset: number, count: number): boolean {
     return this.source.isDataAvailable(offset, count);
   }

webapp/packages/plugin-data-viewer/src/DatabaseDataModel/DatabaseDataSource.ts

@@ -206,6 +206,10 @@ export abstract class DatabaseDataSource<TOptions, TResult extends IDatabaseData
     return !this.isLoading() && !this.disabled;
   }
 
+  hasRowIdentifier(resultIndex: number): boolean {
+    return this.getResult(resultIndex)?.data?.hasRowIdentifier === true;
+  }
+
   isReadonly(resultIndex: number): boolean {
     return this.access === DatabaseDataAccessMode.Readonly || this.results.length > 1 || this.disabled;
   }

webapp/packages/plugin-data-viewer/src/DatabaseDataModel/IDatabaseDataModel.ts

@@ -31,6 +31,7 @@ export interface IDatabaseDataModel<TSource extends IDatabaseDataSource<any, any
 
   setName: (name: string | null) => this;
   isReadonly: (resultIndex: number) => boolean;
+  hasRowIdentifier: (resultIndex: number) => boolean;
   isDisabled: (resultIndex?: number) => boolean;
   isLoading: () => boolean;
   isDataAvailable: (offset: number, count: number) => boolean;

webapp/packages/plugin-data-viewer/src/DatabaseDataModel/IDatabaseDataSource.ts

@@ -71,6 +71,7 @@ export interface IDatabaseDataSource<TOptions = unknown, TResult extends IDataba
   isOutdated: () => boolean;
   isLoadable: () => boolean;
   isReadonly: (resultIndex: number) => boolean;
+  hasRowIdentifier: (resultIndex: number) => boolean;
   isDataAvailable: (offset: number, count: number) => boolean;
   isLoading: () => boolean;
   isDisabled: (resultIndex?: number) => boolean;

webapp/packages/plugin-data-viewer/src/ResultSet/ResultSetDataSource.ts

@@ -39,7 +39,7 @@ export abstract class ResultSetDataSource<TOptions = IDatabaseDataOptions> exten
   }
 
   override isReadonly(resultIndex: number): boolean {
-    return super.isReadonly(resultIndex) || !this.executionContext?.context || this.getResult(resultIndex)?.data?.hasRowIdentifier === false;
+    return super.isReadonly(resultIndex) || !this.executionContext?.context;
   }
 
   override async cancel(): Promise<void> {

webapp/packages/plugin-data-viewer/src/TableViewer/TableFooter/TableFooterMenu/TableFooterMenuService.ts

@@ -101,13 +101,17 @@ export class TableFooterMenuService {
           }
           case ACTION_DELETE: {
             const editor = model.source.getActionImplementation(resultIndex, DatabaseEditAction);
+            const selectedElements = getActiveElements(model, resultIndex);
 
-            if (!editor) {
+            // we can't edit table cells if table doesn't have row identifier, but we can edit new created rows before insert (CB-6063)
+            const canEdit =
+              model.hasRowIdentifier(resultIndex) ||
+              (!model.hasRowIdentifier(resultIndex) && selectedElements.every(key => editor?.getElementState(key) === DatabaseEditChangeType.add));
+
+            if (!editor || !canEdit) {
               return true;
             }
 
-            const selectedElements = getActiveElements(model, resultIndex);
-
             return selectedElements.length === 0 || !selectedElements.some(key => editor.getElementState(key) !== DatabaseEditChangeType.delete);
           }
           case ACTION_REVERT: {

webapp/packages/plugin-data-viewer/src/ValuePanelPresentation/BooleanValue/BooleanValuePresentation.tsx

@@ -51,7 +51,11 @@ export const BooleanValuePresentation: TabContainerPanelComponent<IDataValuePane
 
   const column = viewAction.getColumn(firstSelectedCell.column);
   const nullable = column?.required === false;
-  const readonly = model.isReadonly(resultIndex) || model.isDisabled(resultIndex) || formatAction.isReadOnly(firstSelectedCell);
+  const readonly =
+    model.isReadonly(resultIndex) ||
+    !model.hasRowIdentifier(resultIndex) ||
+    model.isDisabled(resultIndex) ||
+    formatAction.isReadOnly(firstSelectedCell);
 
   return (
     <div className={classes['container']}>

webapp/packages/plugin-data-viewer/src/ValuePanelPresentation/TextValue/isTextValueReadonly.ts

@@ -30,6 +30,7 @@ export function isTextValueReadonly({ contentAction, formatAction, model, result
     formatAction.isGeometry(cell) ||
     contentAction.isTextTruncated(cell) ||
     model.isReadonly(resultIndex) ||
-    model.isDisabled(resultIndex)
+    model.isDisabled(resultIndex) ||
+    !model.hasRowIdentifier(resultIndex)
   );
 }

webapp/packages/plugin-sql-generator/src/GeneratorMenuBootstrap.ts

@@ -43,6 +43,7 @@ export class GeneratorMenuBootstrap extends Bootstrap {
         const presentation = context.get(DATA_CONTEXT_DV_PRESENTATION);
         return (
           !model.isReadonly(resultIndex) &&
+          model.hasRowIdentifier(resultIndex) &&
           model.source.getResult(resultIndex)?.dataFormat === ResultDataFormat.Resultset &&
           !presentation?.readonly &&
           (!presentation || presentation.type === DataViewerPresentationType.Data)