Actually check that channel ids match in allChannelsHaveStats

Author: jonboiser

kolibri/plugins/management/assets/src/state/manageContentActions.js

@@ -20,11 +20,12 @@ export function refreshChannelList() {
  * @returns {Promise}
  */
 export function deleteChannel(store, channelId) {
-  return ChannelResource.getModel(channelId).delete()
+  return ChannelResource.getModel(channelId)
+    .delete()
     .then(() => {
       store.dispatch(mutationTypes.REMOVE_CHANNEL_FILE_SUMMARY, channelId);
     })
-    .then(refreshChannelList)
+    .then(refreshChannelList);
 }
 
 /**

kolibri/plugins/management/assets/src/state/manageContentMutations.js

@@ -28,4 +28,4 @@ function MANAGE_CONTENT_REMOVE_CHANNEL_FILE_SUMMARY(state, channelId) {
 export default {
   MANAGE_CONTENT_ADD_CHANNEL_FILE_SUMMARY,
   MANAGE_CONTENT_REMOVE_CHANNEL_FILE_SUMMARY,
-}
+};

kolibri/plugins/management/assets/src/views/manage-content-page/wizard-export.vue

@@ -65,7 +65,8 @@
       available: 'available',
       cancel: 'Cancel',
       export: 'Export',
-      exportPromptPrefix: 'You are about to export {numChannels, number} {numChannels, plural, one {channel} other {channels}}',
+      exportPromptPrefix:
+        'You are about to export {numChannels, number} {numChannels, plural, one {channel} other {channels}}',
       notWritable: 'Not writable',
       refresh: 'Refresh',
       title: 'Export to where?',
@@ -86,7 +87,9 @@
         return !this.drivesLoading && !this.wizardState.busy && this.selectedDrive !== '';
       },
       exportContentSize() {
-        const allChannelsHaveStats = this.allChannels.length === Object.keys(this.channelsWithStats).length;
+        const allChannelsHaveStats = this.allChannels.reduce((flag, channel) => {
+          return flag && Boolean(this.channelsWithStats[channel.id]);
+        }, true);
         if (allChannelsHaveStats) {
           const totalSize = sumBy(Object.values(this.channelsWithStats), 'totalFileSizeInBytes');
           return bytesForHumans(totalSize);
@@ -117,7 +120,7 @@
     },
     vuex: {
       getters: {
-        allChannels: (state) => state.core.channels.list,
+        allChannels: state => state.core.channels.list,
         channelsWithStats: state => state.pageState.channelFileSummaries,
         wizardState: state => state.pageState.wizardState,
       },