From a488e5475c18eaf7230cfdfc68f8e0270b1f9edf Mon Sep 17 00:00:00 2001
From: Johnny McQuade <jfmcquade@hotmail.com>
Date: Mon, 15 Jan 2024 16:34:49 +0000
Subject: [PATCH] chore: code tidy

---
 .../template/components/task-card/task-card.component.ts       | 3 ++-
 .../task-progress-bar/task-progress-bar.component.ts           | 2 +-
 src/app/shared/services/task/task.service.ts                   | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/app/shared/components/template/components/task-card/task-card.component.ts b/src/app/shared/components/template/components/task-card/task-card.component.ts
index 82294e7c26..02f4aff973 100644
--- a/src/app/shared/components/template/components/task-card/task-card.component.ts
+++ b/src/app/shared/components/template/components/task-card/task-card.component.ts
@@ -15,7 +15,8 @@ interface ITaskCardParams {
    */
   task_id: string;
   /**
-   * The ID of a task group, used to track whether it is the highlighted task group.
+   * The ID of a task group, within the config-level list app_config.TASKS.taskGroupsListName.
+   * Used to track whether it is the highlighted task group.
    * If provided, the task card will display a progress bar for the task group
    * **/
   task_group_id: string;
diff --git a/src/app/shared/components/template/components/task-progress-bar/task-progress-bar.component.ts b/src/app/shared/components/template/components/task-progress-bar/task-progress-bar.component.ts
index 7f01ad81d6..9b6162eeba 100644
--- a/src/app/shared/components/template/components/task-progress-bar/task-progress-bar.component.ts
+++ b/src/app/shared/components/template/components/task-progress-bar/task-progress-bar.component.ts
@@ -153,7 +153,7 @@ export class TmplTaskProgressBarComponent
   }
 
   private checkAndSetUseDynamicData() {
-    this.useDynamicData = this.dataRows[0].hasOwnProperty(this.params.completedColumnName);
+    this.useDynamicData = this.dataRows?.[0]?.hasOwnProperty(this.params.completedColumnName);
   }
 
   private async evaluateTaskGroupData() {
diff --git a/src/app/shared/services/task/task.service.ts b/src/app/shared/services/task/task.service.ts
index 1a73476be8..6bdfbcd1f3 100644
--- a/src/app/shared/services/task/task.service.ts
+++ b/src/app/shared/services/task/task.service.ts
@@ -170,7 +170,7 @@ export class TaskService extends AsyncServiceBase {
     if (subtasksCompleted === subtasksTotal) {
       progressStatus = "completed";
       // Check whether task group has already been completed
-      if (this.templateFieldService.getField(completedField) !== true) {
+      if (!this.templateFieldService.getField(completedField)) {
         // If not, set completed field to "true"
         await this.setTaskGroupCompletedStatus(completedField, true);
         newlyCompleted = true;