fixed single atom results for boolean byte short real float

Author: ecmel

src/models/queryResult.ts

@@ -40,15 +40,16 @@ export const queryConstants = {
 };
 
 export interface StructuredTextColumns {
-  name: "values" | "value" | string;
+  name: string;
   type: string;
   values: string[] | string;
-  /* not the order for initial display */
-  order: number[] | "Not Yet Implemented for the input";
+  order:
+    | number[] /* ascending indexes if the column is sorted */
+    | string /* error message if sorting is not possible */;
 }
 
 export interface StructuredTextResults {
   columns: StructuredTextColumns[] | StructuredTextColumns;
-  /* values array length may differ */
+  /* note: values array length may differ */
   count: number;
 }

src/services/resultsPanelProvider.ts

@@ -227,7 +227,7 @@ export class KdbResultsViewProvider implements WebviewViewProvider {
 
     const columnDefs = columnsArray.map((column) => {
       const sanitizedKey = this.sanitizeString(column.name);
-      const cellDataType = this.kdbToAgGridCellType(column.type);
+      const cellDataType = "text";
       const headerName = column.type
         ? `${sanitizedKey} [${column.type}]`
         : sanitizedKey;
@@ -236,7 +236,7 @@ export class KdbResultsViewProvider implements WebviewViewProvider {
         field: column.name,
         headerName: headerName,
         cellDataType: cellDataType,
-        cellRendererParams: { disabled: cellDataType === "boolean" },
+        cellRendererParams: { disabled: false },
       };
     });
 

src/utils/queryUtils.ts

@@ -399,9 +399,10 @@ export function formatScratchpadStacktrace(stacktrace: ScratchpadStacktrace) {
 const PNG = ["0x89", "0x50", "0x4e", "0x47", "0x0d", "0x0a", "0x1a", "0x0a"];
 
 export function resultToBase64(result: any): string | undefined {
-  const bytes = Array.isArray(result?.columns)
-    ? result.columns[0].values
-    : result?.columns?.values || result;
+  const bytes =
+    (Array.isArray(result?.columns) && result.columns[0]?.values) ||
+    result?.columns?.values ||
+    result;
   if (Array.isArray(bytes) && bytes.length > 66) {
     for (let i = 0; i < PNG.length; i++) {
       if (bytes[i] !== PNG[i] && bytes[i] !== parseInt(PNG[i], 16)) {

test/suite/utils.test.ts

@@ -2035,7 +2035,7 @@ describe("Utils", () => {
     });
     it("should return undefined for bogus structuredText v2", () => {
       const result = queryUtils.resultToBase64({
-        columns: [{}],
+        columns: [],
       });
       assert.strictEqual(result, undefined);
     });