Merge pull request #5557 from learningequality/hotfixes

marcellamaki · web-flow · commit e5d1416a9fc9 · 2025-12-02T17:08:59.000-05:00
Studio patch release v2025.12.02
diff --git a/.github/workflows/deploytest.yml b/.github/workflows/deploytest.yml
@@ -61,7 +61,9 @@ jobs:
           ${{ runner.os }}-pyprod-
     - name: Install pip-tools and python dependencies
       run: |
-        python -m pip install --upgrade pip
+        # Pin pip to 25.2 to avoid incompatibility with pip-tools and 25.3
+        # see https://github.com/jazzband/pip-tools/issues/2252
+        python -m pip install pip==25.2
         pip install pip-tools
         pip-sync requirements.txt
     - name: Use pnpm
diff --git a/.github/workflows/pythontest.yml b/.github/workflows/pythontest.yml
@@ -82,7 +82,9 @@ jobs:
           ${{ runner.os }}-pytest-
     - name: Install pip-tools and python dependencies
       run: |
-        python -m pip install --upgrade pip
+        # Pin pip to 25.2 to avoid incompatibility with pip-tools and 25.3
+        # see https://github.com/jazzband/pip-tools/issues/2252
+        python -m pip install pip==25.2
         pip install pip-tools
         pip-sync requirements.txt requirements-dev.txt
     - name: Test pytest
diff --git a/Makefile b/Makefile
@@ -38,7 +38,7 @@ migrate:
 # 4) Remove the management command from this `deploy-migrate` recipe
 # 5) Repeat!
 deploy-migrate:
-	python contentcuration/manage.py set_file_duration
+	echo "Nothing to do here!"
 
 contentnodegc:
 	python contentcuration/manage.py garbage_collect
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditBooleanMapModal.vue b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditBooleanMapModal.vue
@@ -95,7 +95,7 @@
     },
     data() {
       return {
-        updateDescendants: false,
+        updateDescendants: true,
         error: '',
         /**
          * selectedValues is an object with the following structure:
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditLanguageModal.vue b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditLanguageModal.vue
@@ -92,7 +92,7 @@
       return {
         selectedLanguage: '',
         searchQuery: '',
-        updateDescendants: false,
+        updateDescendants: true,
         isMultipleNodeLanguages: false,
         changed: false,
       };
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditTitleDescriptionModal.vue b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/EditTitleDescriptionModal.vue
@@ -98,6 +98,7 @@
           id: nodeId,
           title: title.trim(),
           description: description.trim(),
+          checkComplete: true,
         });
         /* eslint-disable-next-line kolibri/vue-no-undefined-string-uses */
         this.$store.dispatch('showSnackbarSimple', commonStrings.$tr('changesSaved'));
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditBooleanMapModal.spec.js b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditBooleanMapModal.spec.js
@@ -275,10 +275,11 @@ describe('EditBooleanMapModal', () => {
       expect(wrapper.find('[data-test="update-descendants-checkbox"]').exists()).toBeFalsy();
     });
 
-    test('should call updateContentNode on success submit if the user does not check the update descendants checkbox', async () => {
+    test('should call updateContentNode on success submit if the user uncheck the update descendants checkbox', async () => {
       nodes['node1'].kind = ContentKindsNames.TOPIC;
 
       const wrapper = makeWrapper({ nodeIds: ['node1'], isDescendantsUpdatable: true });
+      wrapper.find('[data-test="update-descendants-checkbox"]').element.click();
       await wrapper.vm.handleSave();
 
       expect(contentNodeActions.updateContentNode).toHaveBeenCalledWith(expect.anything(), {
@@ -287,11 +288,10 @@ describe('EditBooleanMapModal', () => {
       });
     });
 
-    test('should call updateContentNodeDescendants on success submit if the user checks the descendants checkbox', async () => {
+    test('should call updateContentNodeDescendants on success submit if the user does not uncheck the update descendants checkbox', async () => {
       nodes['node1'].kind = ContentKindsNames.TOPIC;
 
       const wrapper = makeWrapper({ nodeIds: ['node1'], isDescendantsUpdatable: true });
-      wrapper.find('[data-test="update-descendants-checkbox"]').element.click();
       await wrapper.vm.handleSave();
 
       expect(contentNodeActions.updateContentNodeDescendants).toHaveBeenCalledWith(
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditLanguageModal.spec.js b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditLanguageModal.spec.js
@@ -220,8 +220,8 @@ describe('EditLanguageModal', () => {
   });
 
   describe('topic nodes present', () => {
-    it('should display the checkbox to apply change to descendants if a topic is present', () => {
-      [wrapper] = makeWrapper(['test-en-topic', 'test-en-res']);
+    test('should display a selected checkbox to apply change to descendants if a topic is present', () => {
+      const [wrapper] = makeWrapper(['test-en-topic', 'test-en-res']);
 
       expect(
         wrapper.findComponent('[data-test="update-descendants-checkbox"]').exists(),
@@ -236,30 +236,33 @@ describe('EditLanguageModal', () => {
       ).toBeFalsy();
     });
 
-    it('should call updateContentNode with the right language on success submit if the user does not check the checkbox', async () => {
-      [wrapper, mocks] = makeWrapper(['test-en-topic', 'test-en-res']);
+    test('should call updateContentNodeDescendants with the right language on success submit by default', async () => {
+      const [wrapper, mocks] = makeWrapper(['test-en-topic', 'test-en-res']);
 
       await chooseLanguage(wrapper, 'es');
       await wrapper.vm.handleSave();
-      await wrapper.vm.$nextTick();
 
-      expect(mocks.updateContentNode).toHaveBeenCalledWith({
+      expect(mocks.updateContentNodeDescendants).toHaveBeenCalledWith({
         id: 'test-en-topic',
         language: 'es',
       });
     });
 
-    it('should call updateContentNodeDescendants with the right language on success submit if the user checks the checkbox', async () => {
-      [wrapper, mocks] = makeWrapper(['test-en-topic', 'test-en-res']);
+    test('should call updateContentNode with the right language on success submit if the user unchecks check the checkbox', async () => {
+      const [wrapper, mocks] = makeWrapper(['test-en-topic', 'test-en-res']);
 
       await chooseLanguage(wrapper, 'es');
-      wrapper.findComponent('[data-test="update-descendants-checkbox"]').vm.$emit('change', true);
+
+      // Uncheck the descendants checkbox
+      const descendantsCheckbox = wrapper.findComponent(
+        '[data-test="update-descendants-checkbox"]',
+      );
+      descendantsCheckbox.vm.$emit('change', false);
       await wrapper.vm.$nextTick();
-      expect(wrapper.vm.updateDescendants).toBe(true);
+
       await wrapper.vm.handleSave();
-      await wrapper.vm.$nextTick();
 
-      expect(mocks.updateContentNodeDescendants).toHaveBeenCalledWith({
+      expect(mocks.updateContentNode).toHaveBeenCalledWith({
         id: 'test-en-topic',
         language: 'es',
       });
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditTitleDescriptionModal.spec.js b/contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/__tests__/EditTitleDescriptionModal.spec.js
@@ -33,9 +33,7 @@ describe('EditTitleDescriptionModal', () => {
           },
         },
       }),
-      propsData: {
-        nodeId,
-      },
+      propsData: { nodeId },
     });
 
     updateContentNode = jest.spyOn(wrapper.vm, 'updateContentNode').mockImplementation(() => {});
@@ -70,7 +68,8 @@ describe('EditTitleDescriptionModal', () => {
     expect(updateContentNode).toHaveBeenCalledWith({
       id: nodeId,
       title: newTitle,
-      description: newDescription,
+      description: newDescription ?? '',
+      checkComplete: true,
     });
   });
 
@@ -80,11 +79,11 @@ describe('EditTitleDescriptionModal', () => {
     descriptionInput.vm.$emit('input', '');
 
     modal.vm.$emit('submit');
-
     expect(updateContentNode).toHaveBeenCalledWith({
       id: nodeId,
       title: newTitle,
       description: '',
+      checkComplete: true,
     });
   });
 
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue b/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
@@ -74,7 +74,8 @@
         slider-color="primary"
       >
         <VTab
-          class="px-2"
+          href="#questions"
+          class="pa-1"
           exact
           @change="tab = 'questions'"
         >
@@ -86,7 +87,8 @@
           />
         </VTab>
         <VTab
-          class="px-2"
+          href="#details"
+          class="pa-1"
           exact
           @change="tab = 'details'"
         >
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/edit/DetailsTabView.vue b/contentcuration/contentcuration/frontend/channelEdit/components/edit/DetailsTabView.vue
@@ -499,6 +499,7 @@
   } from 'shared/constants';
   import { constantsTranslationMixin, metadataTranslationMixin } from 'shared/mixins';
   import { crossComponentTranslator } from 'shared/i18n';
+  import { LanguagesNames } from 'shared/leUtils/Languages';
 
   function getValueFromResults(results) {
     if (results.length === 0) {
@@ -715,7 +716,17 @@
         },
       },
       role: generateGetterSetter('role_visibility'),
-      language: generateGetterSetter('language'),
+      language: {
+        get() {
+          const value = this.getValueFromNodes('language');
+          return value === nonUniqueValue ? LanguagesNames.MUL : value;
+        },
+        set(value) {
+          if (!(value === LanguagesNames.MUL && this.language === LanguagesNames.MUL)) {
+            this.update({ language: value });
+          }
+        },
+      },
       accessibility: generateNestedNodesGetterSetter('accessibility_labels'),
       contentLevel: generateNestedNodesGetterSetterObject('grade_levels'),
       resourcesNeeded: generateNestedNodesGetterSetterObject('learner_needs'),
diff --git a/contentcuration/contentcuration/frontend/channelEdit/views/ImportFromChannels/SearchResultsList.vue b/contentcuration/contentcuration/frontend/channelEdit/views/ImportFromChannels/SearchResultsList.vue
@@ -3,13 +3,22 @@
   <div>
     <!-- Filters -->
     <SearchFilterBar />
-    <VLayout row>
-      <VFlex shrink>
-        <SearchFilters :searchResults="nodes" />
-      </VFlex>
-
-      <!-- Main area with cards -->
-      <VFlex class="pl-4">
+    <KGrid>
+      <KGridItem
+        :layout4="{ span: 4 }"
+        :layout8="{ span: 3 }"
+        :layout12="{ span: 5 }"
+      >
+        <SearchFilters
+          style="width: 100%"
+          :searchResults="nodes"
+        />
+      </KGridItem>
+      <KGridItem
+        :layout4="{ span: 4 }"
+        :layout8="{ span: 5 }"
+        :layout12="{ span: 7 }"
+      >
         <VContainer v-if="loadFailed">
           <p class="text-xs-center">
             <Icon icon="error" />
@@ -105,8 +114,8 @@
             </div>
           </div>
         </VContainer>
-      </VFlex>
-    </VLayout>
+      </KGridItem>
+    </KGrid>
   </div>
 
 </template>
diff --git a/contentcuration/contentcuration/frontend/shared/views/__tests__/languageDropdown.spec.js b/contentcuration/contentcuration/frontend/shared/views/__tests__/languageDropdown.spec.js
@@ -1,4 +1,4 @@
-import { mount } from '@vue/test-utils';
+import { mount, shallowMount } from '@vue/test-utils';
 import LanguageDropdown from '../LanguageDropdown.vue';
 import TestForm from './TestForm.vue';
 import { LanguagesList } from 'shared/leUtils/Languages';
@@ -61,4 +61,24 @@ describe('languageDropdown', () => {
     await wrapper.vm.$nextTick();
     expect(wrapper.find('.error--text').exists()).toBe(true);
   });
+
+  it('returns formatted language text when native_name is present', () => {
+    const wrapper = shallowMount(LanguageDropdown, {
+      mocks: {
+        $tr: (key, params) => `${params.language} (${params.code})`,
+      },
+    });
+    const item = { native_name: 'Español,Spanish', id: 'es' };
+    expect(wrapper.vm.languageText(item)).toBe('Español (es)');
+  });
+
+  it('returns formatted language text when native_name is an empty string', () => {
+    const wrapper = shallowMount(LanguageDropdown, {
+      mocks: {
+        $tr: (key, params) => `${params.language} (${params.code})`,
+      },
+    });
+    const item = { native_name: '', id: 'de' };
+    expect(wrapper.vm.languageText(item)).toBe(' (de)');
+  });
 });

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,8 @@`
`74`	`74`	`slider-color="primary"`
`75`	`75`	`>`
`76`	`76`	`<VTab`
`77`		`- class="px-2"`
	`77`	`+ href="#questions"`
	`78`	`+ class="pa-1"`
`78`	`79`	`exact`
`79`	`80`	`@change="tab = 'questions'"`
`80`	`81`	`>`
`@@ -86,7 +87,8 @@`
`86`	`87`	`/>`
`87`	`88`	`</VTab>`
`88`	`89`	`<VTab`
`89`		`- class="px-2"`
	`90`	`+ href="#details"`
	`91`	`+ class="pa-1"`
`90`	`92`	`exact`
`91`	`93`	`@change="tab = 'details'"`
`92`	`94`	`>`