fix: fetch attendance status when calendars are loaded

One may have imported an event, but the attendance status is not properly fetched when rendering the email.
The reason is a timing / state problem.

- Loading the user principal and collections is initialized in https://github.com/nextcloud/mail/blob/6fc45eb0630b9065f9ccb4c1da5cc9557f7df834/src/App.vue#L49-L50.
- If the backend request for the message body is faster, than loading the principal and collections, then Imip.fetchExistingEvent runs without having calendars and changes existingEventFetched to true that prevents the method from running again.
- Solution: Render the imip component when principal and collections are fetched.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

src/App.vue

@@ -66,6 +66,7 @@ export default {
 		this.sync()
 		await this.$store.dispatch('fetchCurrentUserPrincipal')
 		await this.$store.dispatch('loadCollections')
+		this.$store.commit('hasCurrentUserPrincipalAndCollections', true)
 	},
 	methods: {
 		reload() {

src/components/Imip.vue

@@ -391,12 +391,6 @@ export default {
 				await this.fetchExistingEvent(this.attachedVEvent.uid)
 			},
 		},
-		clonedCalendars: {
-			immediate: true,
-			async handler() {
-				await this.fetchExistingEvent(this.attachedVEvent.uid)
-			},
-		},
 		calendarsForPicker: {
 			immediate: true,
 			handler(calendarsForPicker) {

src/components/Message.vue

@@ -36,7 +36,7 @@
 		<div v-if="itineraries.length > 0" class="message-itinerary">
 			<Itinerary :entries="itineraries" :message-id="message.messageId" />
 		</div>
-		<div v-if="message.scheduling.length > 0" class="message-imip">
+		<div v-if="hasCurrentUserPrincipalAndCollections && message.scheduling.length > 0" class="message-imip">
 			<Imip v-for="scheduling in message.scheduling"
 				:key="scheduling.id"
 				:scheduling="scheduling" />
@@ -144,6 +144,9 @@ export default {
 		itineraries() {
 			return this.message.itineraries ?? []
 		},
+		hasCurrentUserPrincipalAndCollections() {
+			return this.$store.getters.hasCurrentUserPrincipalAndCollections
+		},
 	},
 	methods: {
 		onReply(replyBody = '') {

src/store/getters.js

@@ -157,4 +157,5 @@ export const getters = {
 	},
 	isOneLineLayout: (state) => state.list,
 	hasFetchedInitialEnvelopes: (state) => state.hasFetchedInitialEnvelopes,
+	hasCurrentUserPrincipalAndCollections: (state) => state.hasCurrentUserPrincipalAndCollections,
 }

src/store/index.js

@@ -108,6 +108,7 @@ export default new Store({
 				calendars: [],
 				smimeCertificates: [],
 				hasFetchedInitialEnvelopes: false,
+				hasCurrentUserPrincipalAndCollections: false,
 			},
 			getters,
 			mutations,

src/store/mutations.js

@@ -512,4 +512,7 @@ export default {
 	setHasFetchedInitialEnvelopes(state, hasFetchedInitialEnvelopes) {
 		state.hasFetchedInitialEnvelopes = hasFetchedInitialEnvelopes
 	},
+	hasCurrentUserPrincipalAndCollections(state, hasCurrentUserPrincipalAndCollections) {
+		state.hasCurrentUserPrincipalAndCollections = hasCurrentUserPrincipalAndCollections
+	},
 }