Fix code and naming warnings

Author: lulunac27a

src/components/NewTodo.vue

@@ -73,7 +73,7 @@ export interface todoTask {
   repeatEvery: number; //task repetition number of days/weeks/months/years
   repeatInterval: number; //task repetition interval
   newId: number; //task id
-  completed: boolean; //task completed or not
+  isCompleted: boolean; //task completed or not
   timesCompleted: number; //number of times tasks has been completed
   streak: number; //task completion streak
   originalDueDate: Date; //task original due date in YYYY-MM-DD string
@@ -95,7 +95,7 @@ export default defineComponent({
       repeatEvery: 1, // set default task repetition number to 1
       repeatInterval: RepeatInterval.Once, //set default task repetition interval to one-time
       newId: 0, //initial task id is 0
-      completed: false, //task not completed if a task is created
+      isCompleted: false, //task not completed if a task is created
       timesCompleted: 0,
       streak: 0, //set default task streak to 0
       originalDueDate: currentLocalDate.toISOString().split("T")[0], //set a default original task due date to today
@@ -121,7 +121,7 @@ export default defineComponent({
       this.difficulty = Difficulty.Easy;
       this.repeatEvery = 1;
       this.repeatInterval = RepeatInterval.Once;
-      this.completed = false;
+      this.isCompleted = false;
       this.timesCompleted = 0;
       this.streak = 0;
       this.originalDueDate = currentLocalDate.toISOString().split("T")[0];

src/components/TodoList.vue

@@ -45,7 +45,7 @@
           ></span
         >
         <!--don't show complete button if one-time task is completed--><button
-          v-if="!todo.completed"
+          v-if="!todo.isCompleted"
           @click="completeTodo(todo.newId)"
         >
           Complete
@@ -90,7 +90,7 @@ export default defineComponent({
     repeatEvery: Number,
     level: Number,
     xp: Number,
-    completed: Boolean,
+    isCompleted: Boolean,
     timesCompleted: Number,
     streak: Number,
     dailyStreak: Number,

src/store/index.ts

@@ -1,10 +1,24 @@
 import { createStore } from "vuex";
 import createPersistedState from "vuex-persistedstate";
-
+interface Todo {
+  //todos task interface
+  newId: number;
+  task: string;
+  dueDate: string | Date;
+  priority: number;
+  difficulty: number;
+  xp: number;
+  isCompleted: boolean;
+  repeatEvery: number;
+  repeatInterval: number;
+  timesCompleted: number;
+  streak: number;
+  originalDueDate: string | Date;
+}
 export default createStore({
   state: {
     //eslint-disable-next-line
-    todos: [] as any[],
+    todos: [] as Todo[],
     user: {
       level: 1 as number, //set level to 1 as total XP is 0 when state is created
       xp: 0 as number,
@@ -34,9 +48,9 @@ export default createStore({
      * Update user XP state when a user presses Complete button to complete the task.
      */
     updateXp: (state, payload) => {
-      const task = state.todos.find(
+      const task: Todo = state.todos.find(
         (todo: { newId: number }) => todo.newId === payload,
-      );
+      ) as Todo;
       const daysToDue: number =
         (Number(new Date(task.dueDate + " 23:59:59.999")) -
           Number(new Date().setHours(23, 59, 59, 999))) /
@@ -64,7 +78,7 @@ export default createStore({
       let dayTasksMultiplier: number; //calculate XP and score multiplier for tasks completed in a day (today)
       let tasksMultiplier: number; //calculate score multiplier for total number of tasks completed
       const activeTasks: number = state.todos.filter(
-        (taskList) => !taskList.completed,
+        (taskList) => !taskList.isCompleted,
       ).length; //calculate the number of active tasks (tasks that are not completed) using array.filter
       let activeTasksMultiplier: number; //calculate score multiplier for number of active tasks (tasks that are not completed)
       //calculate task repetition XP multiplier
@@ -435,7 +449,7 @@ export default createStore({
         priority: payload.priority as number,
         difficulty: payload.difficulty as number,
         xp: payload.xp as number,
-        completed: payload.completed as boolean,
+        isCompleted: payload.isCompleted as boolean,
         repeatEvery: payload.repeatEvery as number,
         repeatInterval: payload.repeatInterval as number,
         timesCompleted: payload.timesCompleted as number,
@@ -448,12 +462,12 @@ export default createStore({
       /**
        * Complete the task when user presses the Complete button.
        */
-      const item = state.todos.find(
+      const item: Todo = state.todos.find(
         (todo: { newId: number }) => todo.newId === payload,
-      );
+      ) as Todo;
       if (Number(item.repeatInterval) === 5) {
         //if the task is a one-time only
-        item.completed = !item.completed; //complete task item (set completed task to true)
+        item.isCompleted = !item.isCompleted; //complete task item (set completed task to true)
       } else {
         item.timesCompleted++; //increment number of times tasks has been completed by 1
         if (Number(item.repeatInterval) === 1) {
@@ -592,13 +606,13 @@ export default createStore({
         (todo: { newId: number }) => todo.newId === payload,
       );
       let deleteTask;
-      if (!state.todos[index].completed) {
+      if (!state.todos[index].isCompleted) {
         //don't ask for confirmation when one-time task is completed
         deleteTask = confirm(
           `Do you want to delete the task ${state.todos[index].task}?\nThis action cannot be undone.`,
         ) as boolean; //ask user to confirm task deletion
       }
-      if (deleteTask || state.todos[index].completed) {
+      if (deleteTask || state.todos[index].isCompleted) {
         //delete task if one-time task is completed when the deleted button is clicked or when user confirms deletion alert
         state.todos.splice(index, 1); //delete task item
       }