fix: 🐛 fix bug in getTaskPath - address case where inbox contains nes…

…ted tasks
ksalzke · Aug 22, 2023 · b7c95e2 · b7c95e2
1 parent 4b82044
commit b7c95e2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/fuzzySearchLib.omnifocusjs/Resources/fuzzySearchLib.js b/fuzzySearchLib.omnifocusjs/Resources/fuzzySearchLib.js
@@ -3,10 +3,10 @@
     lib.getTaskPath = function (task) {
         var getPath = function (task) {
             if (!task.parent)
-                return task.name; // project in inbox
-            if (task.parent === task.containingProject.task)
+                return task.name; // project in inbox, first level
+            if (task.containingProject && task.parent === task.containingProject.task)
                 return task.containingProject.name + " > " + task.name;
-            else if (task.parent === task.containingProject.task)
+            else if (task.containingProject && task.parent === task.containingProject.task)
                 return task.name;
             else
                 return getPath(task.parent) + " > " + task.name;

diff --git a/src/fuzzySearchLib.ts b/src/fuzzySearchLib.ts
@@ -18,9 +18,9 @@ interface FuzzySearchForm extends Form {
 
   lib.getTaskPath = (task: Task) => {
     const getPath = (task) => {
-      if (!task.parent) return task.name // project in inbox
-      if (task.parent === task.containingProject.task) return `${task.containingProject.name} > ${task.name}`
-      else if (task.parent === task.containingProject.task) return task.name
+      if (!task.parent) return task.name // project in inbox, first level
+      if (task.containingProject && task.parent === task.containingProject.task) return `${task.containingProject.name} > ${task.name}`
+      else if (task.containingProject && task.parent === task.containingProject.task) return task.name
       else return `${getPath(task.parent)} > ${task.name}`
     }
     return getPath(task)