Refactor facilityconfig facility actions into composables

kolibri/core/assets/src/state/modules/core/actions.js

@@ -3,8 +3,6 @@ import pick from 'lodash/pick';
 import client from 'kolibri/client';
 import heartbeat from 'kolibri/heartbeat';
 import logger from 'kolibri-logging';
-import FacilityResource from 'kolibri-common/apiResources/FacilityResource';
-import FacilityDatasetResource from 'kolibri-common/apiResources/FacilityDatasetResource';
 import UserSyncStatusResource from 'kolibri-common/apiResources/UserSyncStatusResource';
 import { setServerTime } from 'kolibri/utils/serverClock';
 import urls from 'kolibri/urls';
@@ -163,42 +161,6 @@ export function setPageVisibility(store) {
   _setPageVisibility(store, document.visibilityState === 'visible');
 }
 
-export function getFacilities(store) {
-  return FacilityResource.fetchCollection({ force: true }).then(facilities => {
-    store.commit('CORE_SET_FACILITIES', [...facilities]);
-  });
-}
-
-export function getFacilityConfig(store, facilityId) {
-  const { userFacilityId, selectedFacility } = store.getters;
-  const facId = facilityId || userFacilityId;
-  if (!facId) {
-    // No facility Id, so nothing good is going to happen here.
-    // Redirect and let Kolibri sort it out.
-    return Promise.resolve(redirectBrowser());
-  }
-  let datasetPromise;
-  if (selectedFacility && typeof selectedFacility.dataset !== 'object') {
-    datasetPromise = Promise.resolve([selectedFacility.dataset]);
-  } else {
-    datasetPromise = FacilityDatasetResource.fetchCollection({
-      getParams: {
-        // fetchCollection for currentSession's facilityId if none was passed
-        facility_id: facId,
-      },
-    });
-  }
-
-  return datasetPromise.then(facilityConfig => {
-    let config = {};
-    const facility = facilityConfig[0];
-    if (facility) {
-      config = { ...facility };
-    }
-    store.commit('CORE_SET_FACILITY_CONFIG', config);
-  });
-}
-
 export function loading(store) {
   return new Promise(resolve => {
     store.commit('CORE_SET_PAGE_LOADING', true);

kolibri/core/assets/src/state/modules/core/getters.js

@@ -1,14 +1,6 @@
 import useUser from 'kolibri/composables/useUser';
 import { get } from '@vueuse/core';
 
-export function facilityConfig(state) {
-  return state.facilityConfig;
-}
-
-export function facilities(state) {
-  return state.facilities;
-}
-
 export function pageSessionId(state) {
   return state.pageSessionId;
 }

kolibri/core/assets/src/state/modules/core/index.js

@@ -13,8 +13,6 @@ export default {
       notifications: [],
       allowRemoteAccess: plugin_data.allowRemoteAccess,
       // facility
-      facilityConfig: {},
-      facilities: [],
       pageVisible: true,
     };
   },

kolibri/core/assets/src/state/modules/core/mutations.js

@@ -1,10 +1,4 @@
 export default {
-  CORE_SET_FACILITY_CONFIG(state, facilityConfig) {
-    state.facilityConfig = facilityConfig;
-  },
-  CORE_SET_FACILITIES(state, facilities) {
-    state.facilities = facilities;
-  },
   CORE_SET_PAGE_LOADING(state, value) {
     const update = { loading: value };
     if (value) {

kolibri/core/assets/src/state/modules/session.js

@@ -1,5 +1,6 @@
 import some from 'lodash/some';
 import { UserKinds } from 'kolibri/constants';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 
 export const baseSessionState = {
   app_context: false,
@@ -46,8 +47,9 @@ export default {
       return state.kind.includes(UserKinds.ADMIN);
     },
     // An "Multi-Facility Admin" is a superuser for a device with 2+ facilities
-    userIsMultiFacilityAdmin(state, getters, rootState) {
-      return getters.isSuperuser && rootState.core.facilities.length > 1;
+    userIsMultiFacilityAdmin(state, getters) {
+      const { facilities } = useFacilities();
+      return getters.isSuperuser && facilities.value.length > 1;
     },
     getUserPermissions(state) {
       const permissions = {};

kolibri/plugins/coach/assets/src/app.js

@@ -5,11 +5,14 @@ import router from 'kolibri/router';
 import ChannelResource from 'kolibri-common/apiResources/ChannelResource';
 import KolibriApp from 'kolibri-app';
 import useSnackbar from 'kolibri/composables/useSnackbar';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import { PageNames } from './constants';
 import routes from './routes';
 import pluginModule from './modules/pluginModule';
 import HomeActivityPage from './views/home/HomeActivityPage';
 
+const { getFacilities, facilities } = useFacilities();
+
 function _channelListState(data) {
   return data.map(channel => ({
     id: channel.id,
@@ -137,8 +140,8 @@ class CoachToolsModule extends KolibriApp {
         promises.push(this.store.dispatch('initClassInfo', to.params.classId));
       }
 
-      if (get(isSuperuser) && this.store.state.core.facilities.length === 0) {
-        promises.push(this.store.dispatch('getFacilities').catch(() => {}));
+      if (get(isSuperuser) && facilities.value.length === 0) {
+        promises.push(getFacilities().catch(() => {}));
       }
 
       if (promises.length > 0) {

kolibri/plugins/coach/assets/src/composables/useCoreCoach.js

@@ -4,6 +4,7 @@ import { get } from '@vueuse/core';
 import { computed, getCurrentInstance } from 'vue';
 import { currentLanguage, isRtl } from 'kolibri/utils/i18n';
 import useUser from 'kolibri/composables/useUser';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import { coachStrings } from '../views/common/commonCoachStrings';
 
 const logging = logger.getLogger(__filename);
@@ -17,14 +18,15 @@ export default function useCoreCoach(store) {
   const classId = computed(() => get(route).params.classId);
   const groups = computed(() => store.getters['classSummary/groups']);
   const { isSuperuser } = useUser();
+  const { facilities } = useFacilities();
 
   function getAppBarTitle() {
     let facilityName;
     // Using coachStrings.$tr() here because mixins are not applied
     // prior to props being processed.
     const { facility_id, name } = store.state.classSummary;
-    if (facility_id && store.state.core.facilities.length > 1 && get(isSuperuser)) {
-      const match = find(store.state.core.facilities, { id: facility_id }) || {};
+    if (facility_id && get(facilities).length > 1 && get(isSuperuser)) {
+      const match = find(get(facilities), { id: facility_id }) || {};
       facilityName = match.name;
     }
     if (facilityName && name) {

kolibri/plugins/coach/assets/src/composables/useGroups.js

@@ -3,9 +3,11 @@ import samePageCheckGenerator from 'kolibri-common/utils/samePageCheckGenerator'
 import LearnerGroupResource from 'kolibri-common/apiResources/LearnerGroupResource';
 import FacilityUserResource from 'kolibri-common/apiResources/FacilityUserResource';
 import useUser from 'kolibri/composables/useUser';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 
 // Place outside the function to keep the state
 const groupsAreLoading = ref(false);
+const { getFacilities, facilities } = useFacilities();
 
 export function useGroups() {
   function setGroupsLoading(loading) {
@@ -15,8 +17,8 @@ export function useGroups() {
   async function showGroupsPage(store, classId) {
     const initClassInfoPromise = store.dispatch('initClassInfo', classId);
     const getFacilitiesPromise =
-      useUser().isSuperuser.value && store.state.core.facilities.length === 0
-        ? store.dispatch('getFacilities').catch(() => {})
+      useUser().isSuperuser.value && facilities.value.length === 0
+        ? getFacilities().catch(() => {})
         : Promise.resolve();
 
     await Promise.all([initClassInfoPromise, getFacilitiesPromise]);

kolibri/plugins/coach/assets/src/composables/useLessons.js

@@ -1,10 +1,12 @@
 import { ref } from 'vue';
 import LearnerGroupResource from 'kolibri-common/apiResources/LearnerGroupResource';
 import useUser from 'kolibri/composables/useUser';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import { PageNames } from '../constants';
 
 // Place outside the function to keep the state
 const lessonsAreLoading = ref(false);
+const { getFacilities, facilities } = useFacilities();
 
 export function useLessons() {
   function setLessonsLoading(loading) {
@@ -15,8 +17,8 @@ export function useLessons() {
   async function showLessonsRootPage(store, classId) {
     const initClassInfoPromise = store.dispatch('initClassInfo', classId);
     const getFacilitiesPromise =
-      useUser().isSuperuser.value && store.state.core.facilities.length === 0
-        ? store.dispatch('getFacilities').catch(() => {})
+      useUser().isSuperuser.value && facilities.value.length === 0
+        ? getFacilities().catch(() => {})
         : Promise.resolve();
 
     await Promise.all([initClassInfoPromise, getFacilitiesPromise]);

kolibri/plugins/coach/assets/src/modules/lessonResources/handlers.js

@@ -6,8 +6,11 @@ import { ContentNodeKinds } from 'kolibri/constants';
 import chunk from 'lodash/chunk';
 import useUser from 'kolibri/composables/useUser';
 import { get } from '@vueuse/core';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import { PageNames } from '../../constants';
 
+const { getFacilities, facilities } = useFacilities();
+
 async function showResourceSelectionPage(store, params) {
   const {
     lessonId,
@@ -22,8 +25,8 @@ async function showResourceSelectionPage(store, params) {
   const initClassInfoPromise = store.dispatch('initClassInfo', params.classId);
   const { isSuperuser } = useUser();
   const getFacilitiesPromise =
-    get(isSuperuser) && store.state.core.facilities.length === 0
-      ? store.dispatch('getFacilities').catch(() => {})
+    get(isSuperuser) && get(facilities).length === 0
+      ? getFacilities().catch(() => {})
       : Promise.resolve();
 
   await Promise.all([initClassInfoPromise, getFacilitiesPromise]);
@@ -186,8 +189,8 @@ export async function showLessonResourceContentPreview(store, params) {
   const initClassInfoPromise = store.dispatch('initClassInfo', classId);
   const { isSuperuser } = useUser();
   const getFacilitiesPromise =
-    get(isSuperuser) && store.state.core.facilities.length === 0
-      ? store.dispatch('getFacilities').catch(() => {})
+    get(isSuperuser) && get(facilities).length === 0
+      ? getFacilities().catch(() => {})
       : Promise.resolve();
 
   await Promise.all([initClassInfoPromise, getFacilitiesPromise]);
@@ -203,8 +206,8 @@ export async function showLessonSelectionContentPreview(store, params, query = {
   const initClassInfoPromise = store.dispatch('initClassInfo', classId);
   const { isSuperuser } = useUser();
   const getFacilitiesPromise =
-    get(isSuperuser) && store.state.core.facilities.length === 0
-      ? store.dispatch('getFacilities').catch(() => {})
+    get(isSuperuser) && get(facilities).length === 0
+      ? getFacilities().catch(() => {})
       : Promise.resolve();
 
   await Promise.all([initClassInfoPromise, getFacilitiesPromise]);

kolibri/plugins/coach/assets/src/modules/lessonSummary/handlers.js

@@ -1,9 +1,11 @@
 import LearnerGroupResource from 'kolibri-common/apiResources/LearnerGroupResource';
 import useUser from 'kolibri/composables/useUser';
 import { get } from '@vueuse/core';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import { PageNames } from '../../constants';
 
 export async function setLessonSummaryState(store, params) {
+  const { getFacilities, facilities } = useFacilities();
   const { classId, lessonId } = params;
   store.commit('lessonSummary/resources/RESET_STATE');
   store.commit('lessonSummary/SET_STATE', {
@@ -16,8 +18,8 @@ export async function setLessonSummaryState(store, params) {
   const initClassInfoPromise = store.dispatch('initClassInfo', classId);
   const { isSuperuser } = useUser();
   const getFacilitiesPromise =
-    get(isSuperuser) && store.state.core.facilities.length === 0
-      ? store.dispatch('getFacilities').catch(() => {})
+    get(isSuperuser) && get(facilities).length === 0
+      ? getFacilities().catch(() => {})
       : Promise.resolve();
 
   await Promise.all([initClassInfoPromise, getFacilitiesPromise]);

kolibri/plugins/coach/assets/src/routes/index.js

@@ -2,6 +2,7 @@ import store from 'kolibri/store';
 import router from 'kolibri/router';
 import useUser from 'kolibri/composables/useUser';
 import { get } from '@vueuse/core';
+import useFacilities from 'kolibri-common/composables/useFacilities';
 import AllFacilitiesPage from '../views/AllFacilitiesPage';
 import CoachClassListPage from '../views/CoachClassListPage';
 import ClassLearnersListPage from '../views/ClassLearnersListPage';
@@ -20,9 +21,11 @@ import groupsRoutes from './groupsRoutes';
 function showHomePage(toRoute) {
   const initClassInfoPromise = store.dispatch('initClassInfo', toRoute.params.classId);
   const { isSuperuser } = useUser();
+  const { getFacilities, facilities } = useFacilities();
+
   const getFacilitiesPromise =
-    get(isSuperuser) && store.state.core.facilities.length === 0
-      ? store.dispatch('getFacilities').catch(() => {})
+    get(isSuperuser) && get(facilities).length === 0
+      ? getFacilities().catch(() => {})
       : Promise.resolve();
 
   return Promise.all([initClassInfoPromise, getFacilitiesPromise]);

kolibri/plugins/coach/assets/src/views/AllFacilitiesPage.vue

@@ -42,6 +42,7 @@
   import CoreTable from 'kolibri/components/CoreTable';
   import commonCoreStrings from 'kolibri/uiText/commonCoreStrings';
   import useUser from 'kolibri/composables/useUser';
+  import useFacilities from 'kolibri-common/composables/useFacilities';
   import commonCoach from './common';
   import CoachAppBarPage from './CoachAppBarPage';
 
@@ -54,7 +55,8 @@
     mixins: [commonCoach, commonCoreStrings],
     setup() {
       const { facility_id, userIsMultiFacilityAdmin } = useUser();
-      return { facility_id, userIsMultiFacilityAdmin };
+      const { facilities } = useFacilities();
+      return { facility_id, userIsMultiFacilityAdmin, facilities };
     },
     props: {
       subtopicName: {
@@ -63,11 +65,6 @@
         default: null,
       },
     },
-    computed: {
-      facilities() {
-        return this.$store.state.core.facilities;
-      },
-    },
     beforeMount() {
       if (!this.userIsMultiFacilityAdmin) {
         const singleFacility = { id: this.facility_id };

kolibri/plugins/coach/assets/src/views/CoachClassListPage.vue

@@ -70,6 +70,7 @@
   import commonCoreStrings from 'kolibri/uiText/commonCoreStrings';
   import urls from 'kolibri/urls';
   import useUser from 'kolibri/composables/useUser';
+  import useFacilities from 'kolibri-common/composables/useFacilities';
   import { PageNames } from '../constants';
   import CoachAppBarPage from './CoachAppBarPage';
   import commonCoach from './common';
@@ -82,7 +83,8 @@
     mixins: [commonCoach, commonCoreStrings],
     setup() {
       const { isClassCoach, isFacilityCoach, userIsMultiFacilityAdmin } = useUser();
-      return { isClassCoach, isFacilityCoach, userIsMultiFacilityAdmin };
+      const { facilities } = useFacilities();
+      return { isClassCoach, isFacilityCoach, userIsMultiFacilityAdmin, facilities };
     },
     props: {
       subtopicName: {
@@ -127,7 +129,7 @@
         const { facility_id } = this.$route.params;
 
         if (facility_id) {
-          const match = find(this.$store.state.core.facilities, { id: facility_id }) || {};
+          const match = find(this.facilities, { id: facility_id }) || {};
           facilityName = match.name;
         }
         if (facilityName) {

kolibri/plugins/coach/assets/src/views/lessons/LessonCreationPage.vue

@@ -30,6 +30,7 @@
   import { ERROR_CONSTANTS } from 'kolibri/constants';
   import CatchErrors from 'kolibri/utils/CatchErrors';
   import commonCoreStrings from 'kolibri/uiText/commonCoreStrings';
+  import useFacilities from 'kolibri-common/composables/useFacilities';
   import AssignmentDetailsModal from '../common/assignments/AssignmentDetailsModal';
   import commonCoach from '../common';
   import CoachImmersivePage from '../CoachImmersivePage';
@@ -47,10 +48,11 @@
       },
     },
     created() {
+      const { getFacilities, facilities } = useFacilities();
       const initClassInfoPromise = this.$store.dispatch('initClassInfo', this.classId);
       const getFacilitiesPromise =
-        this.isSuperuser && this.$store.state.core.facilities.length === 0
-          ? this.$store.dispatch('getFacilities').catch(() => {})
+        this.isSuperuser && facilities.length === 0
+          ? getFacilities().catch(() => {})
           : Promise.resolve();
 
       Promise.all([initClassInfoPromise, getFacilitiesPromise]);