guard isDataArray function

Author: RafaelGB

src/components/cellTypes/TaskCell.tsx

@@ -14,14 +14,15 @@ const TaskCell = (taskProps: CellComponentProps) => {
   const { defaultCell } = taskProps;
   const { cell, column, table, row } = defaultCell;
   const { view } = table.options.meta;
+  const tableColumn = column.columnDef as TableColumn;
   useEffect(() => {
     let taskValue = cell.getValue();
     // Check if there are tasks in the cell
     if (taskValue) {
       taskRef.current.innerHTML = "";
       if (
-        (column.columnDef as TableColumn).config.task_hide_completed &&
-        DataviewService.getDataviewAPI().isDataArray(taskValue)
+        tableColumn.config.task_hide_completed &&
+        DataviewService.isDataArray(taskValue)
       ) {
         taskValue = taskValue
           .where((t: STask) => !t.completed)

src/services/DataviewService.ts

@@ -1,6 +1,7 @@
 import { Notice } from "obsidian";
-import { DataviewApi, getAPI, isPluginEnabled, STask } from "obsidian-dataview";
+import { DataArray, DataviewApi, getAPI, isPluginEnabled, STask } from "obsidian-dataview";
 import { Literal, WrappedLiteral } from "obsidian-dataview/lib/data-model/value";
+import { LOGGER } from "./Logger";
 class DataviewProxy {
 
     private static instance: DataviewProxy;
@@ -38,6 +39,14 @@ class DataviewProxy {
         return (value as STask[]).every(v => this.isStasks(v));
     }
 
+    isDataArray(value: Literal): value is DataArray<unknown> {
+        try {
+            return this.getDataviewAPI().isDataArray(value);
+        } catch (e) {
+            LOGGER.error(`Error while checking if value is DataArray: ${e.message}`)
+            return false;
+        }
+    }
     /**
      * Singleton instance
      * @returns {VaultManager}

src/services/ParseService.ts

@@ -42,7 +42,7 @@ class Parse {
             return literal;
         }
 
-        if (DataviewService.getDataviewAPI().isDataArray(literal)) {
+        if (DataviewService.isDataArray(literal)) {
             literal = literal.values
         }
 

src/services/parseServiceHelpers/SelectParser.ts

@@ -17,7 +17,7 @@ class SelectParser extends TypeParser<string> {
             case 'object':
                 if (DateTime.isDateTime(wrapped.value)) {
                     parsedValue = Db.coreFns.luxon.dateToString(wrapped.value, this.config.datetime_format);
-                } else if (DataviewService.getDataviewAPI().isDataArray(wrapped.value)) {
+                } else if (DataviewService.isDataArray(wrapped.value)) {
                     parsedValue = this.parseArrayToText(
                         (wrapped.value.values as Literal[])
                     );

src/services/parseServiceHelpers/TextParser.ts

@@ -19,7 +19,7 @@ class TextParser extends TypeParser<string | DataObject> {
                     return Db.coreFns.luxon.dateToString(wrapped.value, this.config.datetime_format);
                 }
 
-                if (DataviewService.getDataviewAPI().isDataArray(wrapped.value)) {
+                if (DataviewService.isDataArray(wrapped.value)) {
                     return this.parseArrayToText(
                         (wrapped.value.values as Literal[])
                     );