Merge pull request #5858 from AlexVelezLl/add-guards-for-channel-version

Add guards for channel version computed values

Author: bjester

contentcuration/contentcuration/frontend/administration/components/sidePanels/ReviewSubmissionSidePanel.vue

@@ -351,7 +351,7 @@
         // state more accurately to the developer in case of debugging.
         // UI code should rely on XXXIsLoading and XXXIsFinished instead.
         if (!languageCodes) return undefined;
-        if (languageCodes.length === 0) return null;
+        if (!languageCodes?.length) return null;
 
         return languageCodes.map(code => LanguagesMap.get(code).readable_name).join(', ');
       });
@@ -362,7 +362,7 @@
 
       const categoriesString = computed(() => {
         if (!channelVersionIsFinished.value) return undefined;
-        if (channelVersionData.value.included_categories.length === 0) return null;
+        if (!channelVersionData.value.included_categories?.length) return null;
 
         return channelVersionData.value.included_categories
           .map(categoryId => categoryIdToName(categoryId))
@@ -371,7 +371,7 @@
 
       const licensesString = computed(() => {
         if (!channelVersionIsFinished.value) return undefined;
-        if (channelVersionData.value.included_licenses.length === 0) return null;
+        if (!channelVersionData.value.included_licenses?.length) return null;
 
         return channelVersionData.value.included_licenses
           .map(licenseId => LicensesMap.get(licenseId).license_name)

contentcuration/locale/ar/LC_MESSAGES/contentcuration-messages.json

@@ -2176,4 +2176,4 @@
   "sharedVue.masteryModelRequired": "الإتقان مطلوب",
   "sharedVue.shortActivityLteThirty": "يجب أن تكون القيمة مساوية أو أقل من 30",
   "sharedVue.titleRequired": "العنوان إلزامي"
-}
+}

contentcuration/locale/ar/LC_MESSAGES/django.po

@@ -739,4 +739,3 @@ msgstr "إصدار الـ API غير متوفر"
 #: kolibri_public/views_v1.py:170
 msgid "No channel matching {} found"
 msgstr "لم يتم العثور على قناة مطابقة {}"
-

contentcuration/locale/en/LC_MESSAGES/contentcuration-messages.json

@@ -2176,4 +2176,4 @@
   "sharedVue.masteryModelRequired": "Mastery is required",
   "sharedVue.shortActivityLteThirty": "Value must be equal or less than 30",
   "sharedVue.titleRequired": "Title is required"
-}
+}

contentcuration/locale/es_ES/LC_MESSAGES/contentcuration-messages.json

@@ -2176,4 +2176,4 @@
   "sharedVue.masteryModelRequired": "Tiene que seleccionar el criterio de dominio",
   "sharedVue.shortActivityLteThirty": "El valor debe ser igual o menor que 30",
   "sharedVue.titleRequired": "Este campo es obligatorio"
-}
+}

contentcuration/locale/es_ES/LC_MESSAGES/django.po

@@ -742,4 +742,3 @@ msgstr "La versión API no está disponible"
 #: kolibri_public/views_v1.py:170
 msgid "No channel matching {} found"
 msgstr "Ningún canal con {} encontrado"
-

contentcuration/locale/fr_FR/LC_MESSAGES/contentcuration-messages.json

@@ -2176,4 +2176,4 @@
   "sharedVue.masteryModelRequired": "Le critère de maîtrise est obligatoire",
   "sharedVue.shortActivityLteThirty": "La valeur doit être égale ou inférieure à 30",
   "sharedVue.titleRequired": "Le titre est obligatoire"
-}
+}

contentcuration/locale/fr_FR/LC_MESSAGES/django.po

@@ -740,4 +740,3 @@ msgstr "La version de l’API est indisponible"
 #: kolibri_public/views_v1.py:170
 msgid "No channel matching {} found"
 msgstr "Aucune chaîne correspondant à {} trouvée"
-

contentcuration/locale/hi_IN/LC_MESSAGES/contentcuration-messages.json

@@ -2176,4 +2176,4 @@
   "sharedVue.masteryModelRequired": "Mastery is required",
   "sharedVue.shortActivityLteThirty": "Value must be equal or less than 30",
   "sharedVue.titleRequired": "Title is required"
-}
+}

contentcuration/locale/hi_IN/LC_MESSAGES/django.po

@@ -723,4 +723,3 @@ msgstr ""
 #: kolibri_public/views_v1.py:170
 msgid "No channel matching {} found"
 msgstr ""
-