Improved Object Validations with New ValidateObject

Author: Abhishek-Punhani

kolibri/plugins/learn/assets/src/views/HybridLearningLessonCard.vue

@@ -38,6 +38,7 @@
 <script>
 
   import { validateLinkObject } from 'kolibri/utils/validators';
+  import { validateObject } from 'kolibri/utils/objectSpecs';
   import commonCoreStrings from 'kolibri/uiText/commonCoreStrings';
   import ProgressBar from './ProgressBar';
   import LearningActivityLabel from './LearningActivityLabel';
@@ -56,6 +57,15 @@
       content: {
         type: Object,
         required: true,
+        validator: function (content) {
+          return validateObject(content, {
+            id: { type: String, required: true },
+            title: { type: String, required: true },
+            thumbnail: { type: String, required: false, default: '' },
+            description: { type: String, required: false, default: '' },
+            num_coach_contents: { type: Number, required: false, default: 0 },
+          });
+        },
       },
       link: {
         type: Object,

kolibri/plugins/learn/assets/src/views/LibraryAndChannelBrowserMainContent.vue

@@ -33,6 +33,7 @@
 <script>
 
   import useKResponsiveWindow from 'kolibri-design-system/lib/composables/useKResponsiveWindow';
+  import { validateObject } from 'kolibri/utils/objectSpecs';
   import useContentLink from '../composables/useContentLink';
   import CardGrid from './cards/CardGrid';
   import ResourceCard from './cards/ResourceCard';
@@ -66,14 +67,21 @@
       contents: {
         type: Array,
         required: true,
+        validator: function (contents) {
+          return contents.every(content =>
+            validateObject(content, {
+              id: { type: String, required: true },
+              title: { type: String, required: true },
+              is_leaf: { type: Boolean, required: true },
+              copies: { type: Array, required: false, default: () => [] },
+            }),
+          );
+        },
       },
       currentCardViewStyle: {
         type: String,
         required: true,
         default: 'card',
-        validator(value) {
-          return ['card', 'list'].includes(value);
-        },
       },
       keepCurrentBackLink: {
         type: Boolean,

packages/kolibri/components/DownloadButton.vue

@@ -22,6 +22,7 @@
 <script>
 
   import useUser from 'kolibri/composables/useUser';
+  import { validateObject } from 'kolibri/utils/objectSpecs';
   import { getRenderableFiles } from './internal/ContentRenderer/utils';
   import { getFilePresetString } from './internal/filePresetStrings';
 
@@ -38,6 +39,26 @@
       files: {
         type: Array,
         default: () => [],
+        validator: function (files) {
+          return files.every(file =>
+            validateObject(file, {
+              checksum: { type: String, required: true },
+              extension: { type: String, required: true },
+              preset: { type: String, required: true },
+              lang: {
+                type: Object,
+                required: false,
+                default: () => ({}),
+                validator: function (lang) {
+                  return validateObject(lang, {
+                    lang_name: { type: String, required: true },
+                  });
+                },
+              },
+              storage_url: { type: String, required: true },
+            }),
+          );
+        },
       },
       nodeTitle: {
         type: String,