From eb8b8c6939cbd98b18e7faace27f262b789eada4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Tue, 24 May 2022 22:45:43 +0200
Subject: [PATCH] calendar app: set mainUser and options for form

---
 .../Controller/CalendarController.php         |   4 +
 .../ChillCalendarBundle/Form/CalendarType.php |   6 +-
 .../MSGraphRemoteCalendarConnector.php        |  22 ++--
 .../Resources/public/vuejs/Calendar/App.vue   | 118 ++++++++++++------
 .../Resources/public/vuejs/Calendar/i18n.js   |   5 +-
 .../public/vuejs/Calendar/store/actions.js    |  37 +++---
 .../public/vuejs/Calendar/store/mutations.js  |  10 ++
 .../public/vuejs/Calendar/store/utils.js      |   1 +
 .../Resources/public/lib/api/apiMethods.js    |   2 +-
 .../public/vuejs/PickEntity/PickEntity.vue    |  17 ++-
 10 files changed, 149 insertions(+), 73 deletions(-)

diff --git a/src/Bundle/ChillCalendarBundle/Controller/CalendarController.php b/src/Bundle/ChillCalendarBundle/Controller/CalendarController.php
index f64a7a6bc..db121ea82 100644
--- a/src/Bundle/ChillCalendarBundle/Controller/CalendarController.php
+++ b/src/Bundle/ChillCalendarBundle/Controller/CalendarController.php
@@ -139,6 +139,10 @@ class CalendarController extends AbstractController
      */
     public function editAction(Calendar $entity, Request $request): Response
     {
+        if (!$this->remoteCalendarConnector->isReady()) {
+            return $this->remoteCalendarConnector->getMakeReadyResponse($request->getUri());
+        }
+
         $view = null;
         $em = $this->getDoctrine()->getManager();
 
diff --git a/src/Bundle/ChillCalendarBundle/Form/CalendarType.php b/src/Bundle/ChillCalendarBundle/Form/CalendarType.php
index 65c7d1320..987862195 100644
--- a/src/Bundle/ChillCalendarBundle/Form/CalendarType.php
+++ b/src/Bundle/ChillCalendarBundle/Form/CalendarType.php
@@ -81,10 +81,8 @@ class CalendarType extends AbstractType
                 'expanded' => true,
             ]);
 
-        if ($options['data'] instanceof Calendar && $options['data']->getId() === null) {
-            $builder->add('mainUser', HiddenType::class);
-            $builder->get('mainUser')->addModelTransformer($this->idToUserDataTransformer);
-        }
+        $builder->add('mainUser', HiddenType::class);
+        $builder->get('mainUser')->addModelTransformer($this->idToUserDataTransformer);
 
         $builder->add('startDate', HiddenType::class);
         $builder->get('startDate')
diff --git a/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraphRemoteCalendarConnector.php b/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraphRemoteCalendarConnector.php
index 7eb1b7c55..caca3cecb 100644
--- a/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraphRemoteCalendarConnector.php
+++ b/src/Bundle/ChillCalendarBundle/RemoteCalendar/Connector/MSGraphRemoteCalendarConnector.php
@@ -99,16 +99,18 @@ class MSGraphRemoteCalendarConnector implements RemoteCalendarConnectorInterface
             $ids = array_map(function ($item) { return $item['id']; }, $bareEvents['value']);
             $existingIdsInRange = $this->calendarRangeRepository->findRemoteIdsPresent($ids);
 
-            return array_map(
-                function ($item) {
-                    return $this->remoteEventConverter->convertToRemote($item);
-                },
-                // filter all event to keep only the one not in range
-                array_filter(
-                    $bareEvents['value'],
-                    function ($item) use ($existingIdsInRange) {
-                        return (!$existingIdsInRange[$item['id']]) ?? true;
-                    }
+            return array_values(
+                array_map(
+                    function ($item) {
+                        return $this->remoteEventConverter->convertToRemote($item);
+                    },
+                    // filter all event to keep only the one not in range
+                    array_filter(
+                        $bareEvents['value'],
+                        function ($item) use ($existingIdsInRange) {
+                            return (!$existingIdsInRange[$item['id']]) ?? true;
+                        }
+                    )
                 )
             );
         } catch (ClientExceptionInterface $e) {
diff --git a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/App.vue b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/App.vue
index 43d0020a9..d94718942 100644
--- a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/App.vue
+++ b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/App.vue
@@ -1,8 +1,34 @@
 <template>
+  <div class="row">
+    <div class="col-md-4">
+      <label>Utilisateur principal</label>
+    </div>
+    <div class="col-md-8 align-content-end">
+      <pick-entity
+        :multiple="false"
+        :types="['user']"
+        :uniqid="'main_user_calendar'"
+        :picked="[this.$store.getters.getMainUser]"
+        :removableIfSet="false"
+        @addNewEntity="setMainUser"
+      ></pick-entity>
+    </div>
+  </div>
+  <div class="row">
+    <select v-model="this.slotDuration">
+      <option value="00:05:00">5</option>
+      <option value="00:10:00">10</option>
+      <option value="00:15:00">15</option>
+      <option value="00:30:00">30</option>
+    </select>
+  </div>
+  <div>
+    <input type="checkbox" v-model="this.hideWeekEnds" /><label>Masquer les week-ends</label>
+  </div>
   <concerned-groups></concerned-groups>
     <div>
-      <template v-for="uid in this.$store.state.currentView.users.keys()" :key="uid">
-        <calendar-active :user="this.$store.getters.getUserDataById(uid).user" ></calendar-active>
+      <template v-for="u in getActiveUsers" :key="u.id">
+        <calendar-active :user="u" ></calendar-active>
       </template>
     </div>
     <teleport to="#fullCalendar">
@@ -28,7 +54,8 @@ import dayGridPlugin from '@fullcalendar/daygrid';
 import interactionPlugin from '@fullcalendar/interaction';
 import timeGridPlugin from '@fullcalendar/timegrid';
 import CalendarActive from './Components/CalendarActive';
-// import listPlugin from '@fullcalendar/list';
+import PickEntity from 'ChillMainAssets/vuejs/PickEntity/PickEntity.vue';
+import {mapGetters} from "vuex";
 
 export default {
   name: "App",
@@ -37,44 +64,25 @@ export default {
     Location,
     FullCalendar,
     CalendarActive,
+    PickEntity,
   },
   data() {
     return {
       errorMsg: [],
-      /*
-      users: {
-        loaded: [],
-        selected: [],
-        logged: null
-      },
-      calendarEvents: {
-         loaded: [],
-         selected: [],
-         user: [],
-         current: {
-            events: [{
-                title: 'plage prévue',
-                start: window.startDate,
-                end: window.endDate
-            }],
-            id: window.mainUser,
-            color: '#bbbbbb'
-          }
-      },*/
-      selectedEvent: null,
-      previousSelectedEvent: null,
-      previousSelectedEventColor: null,
       showMyCalendar: false,
+      slotDuration: '00:10:00',
+      hideWeekEnds: true,
     }
   },
   computed: {
+    ...mapGetters(['getMainUser']),
     events() {
       return this.$store.getters.getEventSources;
     },
     calendarOptions() {
       return {
         locale: frLocale,
-        plugins: [dayGridPlugin, interactionPlugin, timeGridPlugin],
+        plugins: [dayGridPlugin, interactionPlugin, timeGridPlugin, dayGridPlugin],
         initialView: 'timeGridWeek',
         initialDate: this.$store.getters.initialDate,
         eventSources: this.events,
@@ -85,29 +93,57 @@ export default {
         eventClick: this.onEventClick,
         selectMirror: true,
         editable: true,
-        weekends: false,
+        weekends: !this.hideWeekEnds,
         headerToolbar: {
           left: 'prev,next today',
           center: 'title',
-          right: 'dayGridMonth,timeGridWeek,timeGridDay,listMonth,listWeek,listDay'
-        }
+          right: 'dayGridMonth,timeGridWeek,dayGridThreeDays,timeGridDay',
+        },
+        views: {
+          timeGrid: {
+            slotEventOverlap: false,
+            slotDuration: this.slotDuration,
+            scrollTime: '10:00:00',
+          },
+          dayGridThreeDays: {
+            type: 'dayGridWeek',
+            duration: { days: 3},
+            buttonText: this.$t('list_three_days'),
+          },
+        },
       };
     },
     getActiveUsers() {
-      return this.$store.state.currentView.users.keys()
-        .filter(id => this.$store.getters.hasUserDataById(id))
-        .map(id => this.$store.getters.getUserDataById().user);
+      const users = [];
+      for (const id of this.$store.state.currentView.users.keys()) {
+        users.push(this.$store.getters.getUserDataById(id).user);
+      }
+      return users;
     }
   },
   methods: {
-    init() {
-      //this.updateEventsSource();
+    setMainUser(user) {
+      console.log('setMainUser APP', user);
+
+      if (user.id !== this.$store.getters.getMainUser && (
+          this.$store.state.activity.calendarRange !== null
+          || this.$store.state.activity.startDate !== null
+          || this.$store.state.activity.endDate !== null
+        )
+      ) {
+        if (!window.confirm(this.$t('change_main_user_will_reset_event_data'))) {
+          return;
+        }
+      }
+
+      this.$store.dispatch('setMainUser', user);
+      this.$store.commit('showUserOnCalendar', {user, ranges: true, remotes: true});
     },
-    toggleMyCalendar(value) {
-      this.showMyCalendar = value;
-    },
-    toggleWeekends: function () {
-      this.calendarOptions.weekends = !this.calendarOptions.weekends;
+    removeMainUser(user) {
+      console.log('removeMainUser APP', user);
+
+      window.alert(this.$t('main_user_is_mandatory'));
+      return;
     },
     onDatesSet(event) {
       console.log('onDatesSet', event);
@@ -143,7 +179,7 @@ export default {
 
       // show an alert if changing mainUser
       if (this.$store.getters.getMainUser !== null
-        && payload.event.extendedProps.userId !== this.$store.getMainUser.id) {
+        && payload.event.extendedProps.userId !== this.$store.getters.getMainUser.id) {
         if (!window.confirm(this.$t('this_calendar_range_will_change_main_user'))) {
           return;
         }
diff --git a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/i18n.js b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/i18n.js
index 22d9263ab..af8b59d7f 100644
--- a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/i18n.js
+++ b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/i18n.js
@@ -13,8 +13,11 @@ const appMessages = {
       bloc_thirdparty: "Tiers professionnels",
       bloc_users: "T(M)S",
     },
-    this_calendar_range_will_change_main_user: "Cette plage de disponibilité n'est pas celle de l'utilisateur principal. Si vous continuez, l'utilisateur principal sera adapté. Êtes-vous sur·e ?",
+    this_calendar_range_will_change_main_user: "Cette plage de disponibilité n'est pas celle de l'utilisateur principal. Si vous continuez, l'utilisateur principal sera adapté. Êtes-vous sûr·e ?",
     will_change_main_user_for_me: "Vous ne pouvez pas écrire dans le calendrier d'un autre utilisateur. Voulez-vous être l'utilisateur principal de ce rendez-vous ?",
+    main_user_is_mandatory: "L'utilisateur principal est requis. Vous pouvez le modifier, mais pas le supprimer",
+    change_main_user_will_reset_event_data: "Modifier l'utilisateur principal nécessite de choisir une autre plage de disponibilité ou un autre horaire. Ces informations seront perdues. Êtes-vous sûr·e de vouloir continuer ?",
+    list_three_days: 'Liste 3 jours',
   }
 }
 
diff --git a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/actions.js b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/actions.js
index 740562b06..d57592748 100644
--- a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/actions.js
+++ b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/actions.js
@@ -135,9 +135,9 @@ export default {
   setEventTimes({commit, state, getters}, {start, end}) {
     console.log('### action createEvent', {start, end});
     let startDateInput = document.getElementById("chill_activitybundle_activity_startDate");
-    startDateInput.value = datetimeToISO(start);
+    startDateInput.value = null !== start ? datetimeToISO(start) : '';
     let endDateInput = document.getElementById("chill_activitybundle_activity_endDate");
-    endDateInput.value = datetimeToISO(end);
+    endDateInput.value = null !== end ? datetimeToISO(end) : '';
     let calendarRangeInput = document.getElementById("chill_activitybundle_activity_calendarRange");
     calendarRangeInput.value = "";
 
@@ -152,31 +152,40 @@ export default {
   associateCalendarToRange({state, commit, dispatch}, {range}) {
     console.log('### action associateCAlendarToRange', range);
     let startDateInput = document.getElementById("chill_activitybundle_activity_startDate");
-    startDateInput.value = datetimeToISO(range.start);
+    startDateInput.value = null !== range ? datetimeToISO(range.start) : "";
     let endDateInput = document.getElementById("chill_activitybundle_activity_endDate");
-    endDateInput.value = datetimeToISO(range.end);
+    endDateInput.value = null !== range ? datetimeToISO(range.end) : "";
     let calendarRangeInput = document.getElementById("chill_activitybundle_activity_calendarRange");
-    calendarRangeInput.value = Number(range.extendedProps.calendarRangeId);
+    calendarRangeInput.value = null !== range ? Number(range.extendedProps.calendarRangeId) : "";
 
-    const userId = range.extendedProps.userId;
-    if (state.activity.mainUser !== null && state.activity.mainUser.id !== userId) {
-      dispatch('setMainUser', state.usersData.get(userId).user);
+    if (null !== range) {
+      const userId = range.extendedProps.userId;
+      if (state.activity.mainUser !== null && state.activity.mainUser.id !== userId) {
+        dispatch('setMainUser', state.usersData.get(userId).user);
 
-      // TODO: remove persons involved with this user
+        // TODO: remove persons involved with this user
+      }
     }
 
     commit('associateCalendarToRange', {range});
     return Promise.resolve();
   },
-  setMainUser({commit}, mainUser) {
+  setMainUser({commit, dispatch, state}, mainUser) {
     console.log('setMainUser', mainUser);
-    let mainUserInput = document.getElementById("chill_activitybundle_activity_mainUser");
-    mainUserInput.value = Number(mainUser.id);
+    if (state.activity.mainUser.id !== mainUser.id) {
+      let mainUserInput = document.getElementById("chill_activitybundle_activity_mainUser");
+      mainUserInput.value = Number(mainUser.id);
 
-    commit('setMainUser', mainUser);
+      if (state.activity.calendarRange !== null || state.activity.startDate !== null || state.acdtivity.endDate !== null) {
+        dispatch('associateCalendarToRange', { range: null });
+      }
+
+      commit('setMainUser', mainUser);
+    }
+
+    return Promise.resolve();
   },
 
-
   // Location
   updateLocation({commit}, value) {
     console.log('### action: updateLocation', value);
diff --git a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/mutations.js b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/mutations.js
index 67f4e8f74..957d09ede 100644
--- a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/mutations.js
+++ b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/mutations.js
@@ -48,6 +48,16 @@ export default {
    * @param range
    */
   associateCalendarToRange(state, {range}) {
+    console.log('associateCalendarToRange', range);
+
+    if (null === range) {
+      state.activity.calendarRange = null;
+      state.activity.startDate = null;
+      state.activity.endDate = null;
+
+      return;
+    }
+
     console.log('userId', range.extendedProps.userId);
 
     const r = state.usersData.get(range.extendedProps.userId).calendarRanges
diff --git a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/utils.js b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/utils.js
index 692d19ba2..7c36b4342 100644
--- a/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/utils.js
+++ b/src/Bundle/ChillCalendarBundle/Resources/public/vuejs/Calendar/store/utils.js
@@ -67,6 +67,7 @@ const calendarRangeToFullCalendarEvent = (entity) => {
 }
 
 const remoteToFullCalendarEvent = (entity) => {
+  console.log(entity);
   return {
     id: `range_${entity.id}`,
     title: entity.title,
diff --git a/src/Bundle/ChillMainBundle/Resources/public/lib/api/apiMethods.js b/src/Bundle/ChillMainBundle/Resources/public/lib/api/apiMethods.js
index 80e59005f..e15ae536a 100644
--- a/src/Bundle/ChillMainBundle/Resources/public/lib/api/apiMethods.js
+++ b/src/Bundle/ChillMainBundle/Resources/public/lib/api/apiMethods.js
@@ -57,7 +57,7 @@ const _fetchAction = (page, uri, params) => {
         },
     }).then(response => {
         if (response.ok) { return response.json(); }
-        throw Error({ m: response.statusText });
+        throw new Error({ m: response.statusText });
     });
 };
 
diff --git a/src/Bundle/ChillMainBundle/Resources/public/vuejs/PickEntity/PickEntity.vue b/src/Bundle/ChillMainBundle/Resources/public/vuejs/PickEntity/PickEntity.vue
index e69b26619..f688ff4f8 100644
--- a/src/Bundle/ChillMainBundle/Resources/public/vuejs/PickEntity/PickEntity.vue
+++ b/src/Bundle/ChillMainBundle/Resources/public/vuejs/PickEntity/PickEntity.vue
@@ -1,5 +1,5 @@
 <template>
-    <ul class="list-suggest remove-items" v-if="picked.length">
+    <ul :class="listClasses" v-if="picked.length">
         <li v-for="p in picked" @click="removeEntity(p)" :key="p.type+p.id">
             <span class="chill_denomination">{{ p.text }}</span>
         </li>
@@ -40,6 +40,10 @@ export default {
         uniqid: {
             type: String,
             required: true,
+        },
+        removableIfSet: {
+            type: Boolean,
+            default: true,
         }
     },
     emits: ['addNewEntity', 'removeEntity'],
@@ -78,7 +82,13 @@ export default {
             } else {
                 return appMessages.fr.pick_entity.modal_title_one + trans.join(', ');
             }
-        }
+        },
+        listClasses() {
+            return {
+                'list-suggest': true,
+                'remove-items': this.$props.removableIfSet,
+            };
+        },
     },
     methods: {
         addNewEntity({ selected, modal }) {
@@ -90,6 +100,9 @@ export default {
             modal.showModal = false;
         },
         removeEntity(entity) {
+            if (!this.$props.removableIfSet) {
+                return;
+            }
             this.$emit('removeEntity', entity);
         }
     },