Refactoring set quick edit modal mutations

Author: AlexVelezLl

contentcuration/contentcuration/frontend/channelEdit/components/ContentNodeEditListItem.vue

@@ -109,7 +109,7 @@
 
 <script>
 
-  import { mapGetters, mapActions, mapMutations } from 'vuex';
+  import { mapGetters, mapActions } from 'vuex';
 
   import ContentNodeListItem from './ContentNodeListItem';
   import ContentNodeOptions from './ContentNodeOptions';
@@ -249,12 +249,10 @@
         'updateContentNode',
         'waitForCopyingStatus',
         'deleteContentNode',
+        'setQuickEditModal',
       ]),
-      ...mapMutations('contentNode', {
-        openQuickEditModal: 'SET_QUICK_EDIT_MODAL_OPEN',
-      }),
       editTitleDescription() {
-        this.openQuickEditModal({
+        this.setQuickEditModal({
           modal: QuickEditModals.TITLE_DESCRIPTION,
           nodeIds: [this.nodeId],
         });

contentcuration/contentcuration/frontend/channelEdit/components/ContentNodeOptions.vue

@@ -43,7 +43,7 @@
 
 <script>
 
-  import { mapActions, mapGetters, mapMutations } from 'vuex';
+  import { mapActions, mapGetters } from 'vuex';
   import { RouteNames, TabNames, QuickEditModals } from '../constants';
   import MoveModal from './move/MoveModal';
   import { withChangeTracker } from 'shared/data/changes';
@@ -239,11 +239,9 @@
         'moveContentNodes',
         'copyContentNode',
         'waitForCopyingStatus',
+        'setQuickEditModal',
       ]),
       ...mapActions('clipboard', ['copy']),
-      ...mapMutations('contentNode', {
-        openQuickEditModal: 'SET_QUICK_EDIT_MODAL_OPEN',
-      }),
       async focusFirstOption() {
         const { optionsList } = this.$refs;
         const firstOption = optionsList.$el.querySelector('a');
@@ -313,7 +311,7 @@
       quickEditModalFactory(modal) {
         return $event => {
           $event.preventDefault();
-          this.openQuickEditModal({
+          this.setQuickEditModal({
             modal,
             nodeIds: [this.nodeId],
           });

contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/index.vue

@@ -53,7 +53,7 @@
 
 <script>
 
-  import { mapGetters, mapMutations } from 'vuex';
+  import { mapGetters, mapActions } from 'vuex';
   import { QuickEditModals } from '../../constants';
   import EditSourceModal from './EditSourceModal';
   import EditLevelsModal from './EditLevelsModal';
@@ -123,11 +123,9 @@
       },
     },
     methods: {
-      ...mapMutations('contentNode', {
-        setQuickEditModalOpen: 'SET_QUICK_EDIT_MODAL_OPEN',
-      }),
+      ...mapActions('contentNode', ['setQuickEditModal']),
       close() {
-        this.setQuickEditModalOpen(null);
+        this.setQuickEditModal(null);
       },
     },
   };

contentcuration/contentcuration/frontend/channelEdit/views/CurrentTopicView.vue

@@ -229,7 +229,7 @@
 
 <script>
 
-  import { mapActions, mapGetters, mapState, mapMutations } from 'vuex';
+  import { mapActions, mapGetters, mapState } from 'vuex';
   import get from 'lodash/get';
   import MoveModal from '../components/move/MoveModal';
   import ContentNodeOptions from '../components/ContentNodeOptions';
@@ -572,11 +572,9 @@
         'moveContentNodes',
         'copyContentNode',
         'waitForCopyingStatus',
+        'setQuickEditModal',
       ]),
       ...mapActions('clipboard', ['copyAll']),
-      ...mapMutations('contentNode', {
-        openQuickEditModal: 'SET_QUICK_EDIT_MODAL_OPEN',
-      }),
       clearSelections() {
         this.selected = [];
       },
@@ -665,7 +663,7 @@
       },
       quickEditModalFactory(modal) {
         return () => {
-          this.openQuickEditModal({
+          this.setQuickEditModal({
             modal,
             nodeIds: this.selected,
           });

contentcuration/contentcuration/frontend/channelEdit/vuex/contentNode/actions.js

@@ -545,3 +545,7 @@ export async function checkSavingProgress(
     .first();
   return Boolean(query);
 }
+
+export function setQuickEditModal(context, open) {
+  context.commit('SET_QUICK_EDIT_MODAL', open);
+}

contentcuration/contentcuration/frontend/channelEdit/vuex/contentNode/mutations.js

@@ -78,7 +78,7 @@ export function SET_MOVE_NODES(state, ids) {
   state.moveNodes = ids;
 }
 
-export function SET_QUICK_EDIT_MODAL_OPEN(state, quickEditModalOpen) {
+export function SET_QUICK_EDIT_MODAL(state, quickEditModalOpen) {
   state.quickEditModalOpen = quickEditModalOpen;
 }
 

contentcuration/contentcuration/frontend/shared/utils/helpers.js

@@ -533,7 +533,8 @@ export function isAudioVideoFile(file) {
     return false;
   }
 
-  const videoAllowedFormats = FormatPresetsMap.get(FormatPresetsNames.VIDEO).allowed_formats;
+  const videoAllowedFormats = FormatPresetsMap.get(FormatPresetsNames.HIGH_RES_VIDEO)
+    .allowed_formats;
   const audioAllowedFormats = FormatPresetsMap.get(FormatPresetsNames.AUDIO).allowed_formats;
   return (
     videoAllowedFormats.includes(file.file_format) || audioAllowedFormats.includes(file.file_format)