Add eslint rule to find unused properties, data and computed properties

kolibri/core/assets/src/views/ContentIcon.vue

@@ -120,7 +120,7 @@
           return validateContentNodeKind(value, [USER]);
         },
       },
-      colorstyle: {
+      colorStyle: {
         type: String,
         default: 'action',
       },

kolibri/core/assets/src/views/ContentRenderer/index.vue

@@ -67,10 +67,6 @@
       UiAlert,
     },
     props: {
-      id: {
-        type: String,
-        required: true,
-      },
       kind: {
         type: String,
         required: true,
@@ -79,14 +75,6 @@
         type: Array,
         default: () => [],
       },
-      contentId: {
-        type: String,
-        default: '',
-      },
-      channelId: {
-        type: String,
-        default: '',
-      },
       available: {
         type: Boolean,
         default: false,

kolibri/core/assets/src/views/ExamReport/PageStatus.vue

@@ -43,7 +43,6 @@
   import KGridItem from 'kolibri.coreVue.components.KGridItem';
   import ProgressIcon from 'kolibri.coreVue.components.ProgressIcon';
   import ElapsedTime from 'kolibri.coreVue.components.ElapsedTime';
-  import { ContentNodeKinds } from 'kolibri.coreVue.vuex.constants';
   import KLabeledIcon from 'kolibri.coreVue.components.KLabeledIcon';
   import KIcon from 'kolibri.coreVue.components.KIcon';
 
@@ -96,9 +95,6 @@
         // Either return in completed or in progress
         return this.completed ? 1 : 0.1;
       },
-      kind() {
-        return ContentNodeKinds.EXAM;
-      },
     },
   };
 

kolibri/core/assets/src/views/KSelect/KeenUiSelect.vue

@@ -389,14 +389,6 @@
         return this.options.findIndex(option => looseEqual(this.highlightedOption, option));
       },
 
-      // Returns the index of the currently selected option, -1 if multi-select
-      selectedIndex() {
-        if (this.multiple) {
-          return -1;
-        }
-        return this.options.findIndex(option => looseEqual(this.value, option));
-      },
-
       // Returns an array containing the options and extra annotations
       annotatedOptions() {
         const options = JSON.parse(JSON.stringify(this.options));

kolibri/core/assets/src/views/KTooltip/Popper.vue

@@ -185,7 +185,6 @@
         referenceElm: null,
         popperJS: null,
         showPopper: false,
-        currentPlacement: '',
         popperOptions: {
           placement: 'bottom',
           computeStyle: {

kolibri/core/assets/src/views/icons/KLabeledIcon.vue

@@ -17,15 +17,6 @@
   export default {
     name: 'KLabeledIcon',
     components: {},
-    props: {
-      /**
-       * Whether the label should wrap
-       */
-      nowrap: {
-        type: Boolean,
-        default: false,
-      },
-    },
   };
 
 </script>

kolibri/core/assets/test/views/KTooltipExample.vue

@@ -32,11 +32,6 @@
         required: false,
         default: () => {},
       },
-      kTooltipText: {
-        type: String,
-        required: false,
-        default: '',
-      },
     },
   };
 

kolibri/plugins/coach/assets/src/views/common/HeaderTabs/HeaderTab.vue

@@ -33,11 +33,6 @@
         required: true,
       },
     },
-    data() {
-      return {
-        active: null,
-      };
-    },
     computed: {
       activeClasses() {
         // return both fixed and dynamic classes

kolibri/plugins/coach/assets/src/views/common/notifications/ActivityList.vue

@@ -104,7 +104,6 @@
     data() {
       return {
         loading: true,
-        error: false,
         moreResults: true,
         nextPage: 1,
         progressFilter: 'all',

kolibri/plugins/coach/assets/src/views/home/HomePage/ItemProgressDisplay.vue

@@ -58,10 +58,6 @@
         type: Object,
         required: true,
       },
-      isLast: {
-        type: Boolean,
-        default: false,
-      },
       to: {
         type: Object,
         required: false,

kolibri/plugins/coach/assets/src/views/plan/CoachExamsPage/index.vue

@@ -138,7 +138,7 @@
   import KRouterLink from 'kolibri.coreVue.components.KRouterLink';
   import KSelect from 'kolibri.coreVue.components.KSelect';
   import CoreInfoIcon from 'kolibri.coreVue.components.CoreInfoIcon';
-  import { ContentNodeKinds, ERROR_CONSTANTS } from 'kolibri.coreVue.vuex.constants';
+  import { ERROR_CONSTANTS } from 'kolibri.coreVue.vuex.constants';
   import { crossComponentTranslator } from 'kolibri.utils.i18n';
   import CatchErrors from 'kolibri.utils.CatchErrors';
   import KLabeledIcon from 'kolibri.coreVue.components.KLabeledIcon';
@@ -216,9 +216,6 @@
       manageExamModalStrings() {
         return manageExamModalStrings;
       },
-      examKind() {
-        return ContentNodeKinds.EXAM;
-      },
       sortedExams() {
         return this.exams.slice().reverse();
       },

kolibri/plugins/coach/assets/src/views/plan/CreateExamPage/CreateExamPreview.vue

@@ -300,13 +300,6 @@
       previewQuizStrings() {
         return previewQuizStrings;
       },
-      draggableOptions() {
-        return {
-          animation: 150,
-          touchStartThreshold: 3,
-          direction: 'vertical',
-        };
-      },
       currentQuestion() {
         return this.selectedQuestions[this.currentQuestionIndex] || {};
       },

kolibri/plugins/coach/assets/src/views/plan/GroupEnrollPage/index.vue

@@ -166,9 +166,6 @@
       usersNotInClass() {
         return differenceWith(this.classUsers, this.currentGroupUsers, (a, b) => a.id === b.id);
       },
-      filteredUsers() {
-        return this.usersNotInClass.filter(user => userMatchesFilter(user, this.filterInput));
-      },
       sortedFilteredUsers() {
         return filterAndSortUsers(this.usersNotInClass, user =>
           userMatchesFilter(user, this.filterInput)
@@ -195,9 +192,6 @@
       visibleFilteredUsers() {
         return this.sortedFilteredUsers.slice(this.startRange, this.endRange);
       },
-      showConfirmEnrollmentModal() {
-        return this.modalShown === true;
-      },
       emptyMessage() {
         if (this.classUsers.length === 0) {
           return this.$tr('noUsersExist');

kolibri/plugins/coach/assets/src/views/plan/LessonContentPreviewPage/ContentArea.vue

@@ -55,11 +55,6 @@
         default: '',
       },
     },
-    computed: {
-      hasHeader() {
-        return Boolean(this.header);
-      },
-    },
   };
 
 </script>

kolibri/plugins/coach/assets/src/views/plan/LessonContentPreviewPage/QuestionList.vue

@@ -60,13 +60,6 @@
         validator: value => typeof value(0) === 'string',
       },
     },
-    computed: {
-      buttonAndHeaderBorder() {
-        return {
-          borderBottom: `2px solid ${this.$coreTextDisabled}`,
-        };
-      },
-    },
   };
 
 </script>

kolibri/plugins/coach/assets/src/views/plan/LessonsRootPage/index.vue

@@ -109,11 +109,7 @@
   import KSelect from 'kolibri.coreVue.components.KSelect';
   import KLabeledIcon from 'kolibri.coreVue.components.KLabeledIcon';
   import KIcon from 'kolibri.coreVue.components.KIcon';
-  import {
-    ContentNodeKinds,
-    CollectionKinds,
-    ERROR_CONSTANTS,
-  } from 'kolibri.coreVue.vuex.constants';
+  import { CollectionKinds, ERROR_CONSTANTS } from 'kolibri.coreVue.vuex.constants';
   import CatchErrors from 'kolibri.utils.CatchErrors';
   import { crossComponentTranslator } from 'kolibri.utils.i18n';
   import LessonActive from '../../common/LessonActive';
@@ -148,7 +144,6 @@
     data() {
       return {
         showModal: false,
-        lessonKind: ContentNodeKinds.LESSON,
         filterSelection: {},
       };
     },

kolibri/plugins/coach/assets/src/views/reports/ReportsGroupHeader.vue

@@ -79,6 +79,7 @@
       group() {
         return this.groupMap[this.$route.params.groupId];
       },
+      /** TODO COACH
       recipients() {
         return this.group.member_ids;
       },
@@ -92,6 +93,7 @@
         }
         return this._.meanBy(statuses, 'score');
       },
+      */
     },
     $trs: {
       back: 'All groups',

kolibri/plugins/coach/assets/src/views/reports/ReportsGroupReportLessonExerciseLearnerListPage.vue

@@ -73,9 +73,6 @@
     },
     mixins: [commonCoach],
     computed: {
-      lesson() {
-        return this.lessonMap[this.$route.params.lessonId];
-      },
       recipients() {
         return this.getLearnersForGroups([this.$route.params.groupId]);
       },

kolibri/plugins/coach/assets/src/views/reports/ReportsGroupReportLessonPage.vue

@@ -93,6 +93,7 @@
     components: {},
     mixins: [commonCoach],
     computed: {
+      /** TODO COACH
       actionOptions() {
         return [
           { label: this.coachStrings.$tr('editDetailsAction'), value: 'ReportsLessonEditorPage' },
@@ -102,6 +103,7 @@
           },
         ];
       },
+      */
       lesson() {
         return this.lessonMap[this.$route.params.lessonId];
       },

kolibri/plugins/coach/assets/src/views/reports/ReportsGroupReportLessonResourceLearnerListPage.vue

@@ -13,7 +13,7 @@
       <p>
         <BackLink
           :to="classRoute('ReportsGroupReportLessonPage', {})"
-          :text="$tr('back', { lesson: 'Lesson 1' })"
+          :text="$tr('back', { lesson: lesson.title })"
         />
       </p>
       <h1>
@@ -98,9 +98,6 @@
       recipients() {
         return this.getLearnersForGroups([this.$route.params.groupId]);
       },
-      avgTime() {
-        return this.getContentAvgTimeSpent(this.$route.params.resourceId, this.recipients);
-      },
       tally() {
         return this.getContentStatusTally(this.$route.params.resourceId, this.recipients);
       },

kolibri/plugins/coach/assets/src/views/reports/ReportsGroupReportQuizHeader.vue

@@ -73,9 +73,6 @@
       recipients() {
         return this.getLearnersForGroups([this.$route.params.groupId]);
       },
-      tally() {
-        return this.getExamStatusTally(this.exam.id, this.recipients);
-      },
     },
     $trs: {
       back: 'All quizzes',

kolibri/plugins/coach/assets/src/views/reports/ReportsLessonEditorPage.vue

@@ -33,16 +33,6 @@
       LessonDetailEditor,
     },
     mixins: [commonCoach],
-    data() {
-      return {
-        lessonTitle: 'Lesson A',
-        lessonDescription: 'Ipsum lorem',
-        selectedCollectionIds: ['a'],
-        availableGroups: [{ name: 'Group A', id: 'a' }, { name: 'Group B', id: 'b' }],
-        selectedClassroomId: 'x',
-        isActive: true,
-      };
-    },
     $trs: {},
   };
 

kolibri/plugins/coach/assets/src/views/reports/ReportsLessonHeader.vue

@@ -79,6 +79,7 @@
     components: {},
     mixins: [commonCoach],
     computed: {
+      /** TODO COACH
       actionOptions() {
         return [
           { label: this.coachStrings.$tr('editDetailsAction'), value: 'ReportsLessonEditorPage' },
@@ -88,6 +89,7 @@
           },
         ];
       },
+      */
       lesson() {
         return this.lessonMap[this.$route.params.lessonId];
       },

kolibri/plugins/coach/assets/src/views/reports/ReportsLessonReportPage.vue

@@ -81,6 +81,7 @@
       emptyMessage() {
         return LessonSummaryPageStrings.$tr('noResourcesInLesson');
       },
+      /** TODO COACH
       actionOptions() {
         return [
           { label: this.coachStrings.$tr('editDetailsAction'), value: 'ReportsLessonEditorPage' },
@@ -90,6 +91,7 @@
           },
         ];
       },
+      */
       lesson() {
         return this.lessonMap[this.$route.params.lessonId];
       },

kolibri/plugins/coach/assets/src/views/reports/ReportsQuizEditorPage.vue

@@ -30,16 +30,6 @@
       QuizDetailEditor,
     },
     mixins: [commonCoach],
-    data() {
-      return {
-        lessonTitle: 'Lesson A',
-        lessonDescription: 'Ipsum lorem',
-        selectedCollectionIds: ['a'],
-        availableGroups: [{ name: 'Group A', id: 'a' }, { name: 'Group B', id: 'b' }],
-        selectedClassroomId: 'x',
-        isActive: true,
-      };
-    },
     $trs: {},
   };
 

kolibri/plugins/coach/assets/src/views/reports/ReportsQuizHeader.vue

@@ -80,12 +80,14 @@
       avgScore() {
         return this.getExamAvgScore(this.$route.params.quizId, this.recipients);
       },
+      /** TODO COACH
       actionOptions() {
         return [
           { label: this.coachStrings.$tr('previewAction'), value: 'ReportsQuizPreviewPage' },
           { label: this.coachStrings.$tr('editDetailsAction'), value: 'ReportsQuizEditorPage' },
         ];
       },
+      */
       exam() {
         return this.examMap[this.$route.params.quizId];
       },

kolibri/plugins/device_management/assets/src/views/ManageContentPage/SelectNetworkAddressModal/SelectAddressForm.vue

@@ -90,7 +90,6 @@
         selectedAddressId: '',
         showUiAlerts: false,
         stage: '',
-        Stages,
       };
     },
     computed: {