From 830d77976a87bd42a64931d972d4621c19da1f35 Mon Sep 17 00:00:00 2001
From: les <lesion@autistici.org>
Date: Fri, 4 Dec 2020 17:27:45 +0100
Subject: [PATCH] refactoring dateInput in add/edit event

---
 pages/add/DateInput.vue | 213 +++++++++++++++-------------------------
 pages/add/_edit.vue     |  82 ++++------------
 2 files changed, 94 insertions(+), 201 deletions(-)

diff --git a/pages/add/DateInput.vue b/pages/add/DateInput.vue
index c09f5079..08d72b39 100644
--- a/pages/add/DateInput.vue
+++ b/pages/add/DateInput.vue
@@ -1,41 +1,48 @@
 <template lang="pug">
 v-row
+  v-date-picker.col-md-6(
+    mode='dateTime'
+    :input-debounce="200"
+    :min='today'
+    :minute-increment="5"
+    is-dark
+    is-expanded
+    @input='v => change("from", v)'
+    :value="value.from"
+    :locale='$i18n.locale')
+    template(v-slot="{ inputValue, inputEvents }")
+      v-text-field(
+        :value='inputValue'
+        :label="$t('event.from')"
+        :rules="[$validators.required('common.when')]"
+        :hint="$t(`event.description`)"
+        ref='date'
+        prepend-icon='mdi-calendar'
+        persistent-hint
+        readonly
+        v-on="inputEvents")
 
-    v-btn-toggle.col-md-4(@change='changeType' color='primary' :value='value.type')
-      v-btn(value='normal') {{$t('event.normal')}}
-      v-btn(value='multidate') {{$t('event.multidate')}}
-      v-menu(v-if='settings.allow_recurrent_event' offset-y open-on-hover)
-        template(v-slot:activator="{ on, attrs }")
-          v-btn(value='recurrent' v-on='on') {{$t('event.recurrent')}}
-        v-list
-          v-list-item(v-for='f in frequencies' :key='f.value'
-            @click='selectFrequency(f.value)') {{f.text}}
+  v-date-picker.col-md-4(
+    mode='dateTime'
+    :minute-increment="5"
+    :input-debounce="200"
+    :min='today'
+    is-dark
+    is-expanded
+    @input='v => change("due", v)'
+    :value="value.due"
+    :locale='$i18n.locale')
+    template(v-slot="{ inputValue, inputEvents }")
+      v-text-field(
+        :value='inputValue'
+        :label="$t('event.from')"
+        :hint="$t(`event.description`)"
+        ref='date'
+        prepend-icon='mdi-calendar'
+        persistent-hint
+        readonly
+        v-on="inputEvents")
 
-    v-menu(
-      v-model="datePickerMenu"
-      :close-on-content-click="false"
-      :nudge-right="40"
-      transition="scale-transition"
-      offset-y
-      min-width="290px"
-    )
-      template(v-slot:activator="{ on, attrs }")
-        v-text-field.col-md-8(
-          :value='value.date'
-          :label="$t('common.when')"
-          :rules="[$validators.required('common.when')]"
-          prepend-icon='mdi-calendar'
-          readonly
-          v-bind="attrs"
-          v-on="on")
-      v-date-picker(
-        :min='today'
-        :value="value.date"
-        :range="value.type === 'multidate'"
-        :locale='settings.locale'
-        @input="pick")
-
-    p.col-12 {{$t(`event.${value.type}_description`)}}
     //- v-btn-toggle(v-if="type === 'recurrent'" v-model='value.recurrent.frequency' color='primary')
     //-   v-btn(v-for='f in frequencies' :value='f.value') {{f.text}}
 
@@ -50,11 +57,23 @@ v-row
     //-     is-expanded
     //-     :min-date='type !== "recurrent" && new Date()')
 
-    div.text-center.mb-2(v-if='value.type === "recurrent"')
-      span(v-if='value.recurrent.frequency !== "1m" && value.recurrent.frequency !== "2m"') {{whenPatterns}}
-      v-btn-toggle.mt-1(v-else v-model='value.recurrent.type' color='primary')
-        v-btn(v-for='whenPattern in whenPatterns' :value='whenPattern.key' :key='whenPatterns.key' small)
-          span {{whenPattern.label}}
+    //- v-btn-toggle.col-md-4(@change='changeType' color='primary' :value='value.type')
+    //-   v-btn(value='normal') {{$t('event.normal')}}
+    //-   v-btn(value='multidate') {{$t('event.multidate')}}
+  v-switch.col-md-2(v-model='is_recurrent' :label="$t('event.recurrent')" inset)
+  v-menu(v-if='settings.allow_recurrent_event && is_recurrent' offset-y open-on-hover)
+    template(v-slot:activator="{ on, attrs }")
+      v-btn.col-md-2.mt-2(color='primary' value='recurrent' v-on='on') {{$t('event.recurrent')}}
+    v-list
+      v-list-item-group(color='primary' v-model='frequency')
+        v-list-item(v-for='f in frequencies' :key='f.value'
+          @click='selectFrequency(f.value)') {{f.text}}
+
+    //- div.text-center.mb-2(v-if='value.type === "recurrent"')
+    //-   span(v-if='value.recurrent.frequency !== "1m" && value.recurrent.frequency !== "2m"') {{whenPatterns}}
+    //-   v-btn-toggle.mt-1(v-else v-model='value.recurrent.type' color='primary')
+    //-     v-btn(v-for='whenPattern in whenPatterns' :value='whenPattern.key' :key='whenPatterns.key' small)
+    //-       span {{whenPattern.label}}
 
     //- List(v-if='type==="normal" && todayEvents.length' :events='todayEvents' :title='$t("event.same_day")')
 
@@ -68,14 +87,13 @@ export default {
   name: 'WhenInput',
   components: { List },
   props: {
-    value: { type: Object, default: () => ({ date: null, type: 'normal', recurrent: {} }) }
+    value: { type: Object, default: () => ({ from: null, due: null, recurrent: null }) }
   },
   data () {
     return {
-      datePickerMenu: false,
+      is_recurrent: false,
       today: dayjs().format('YYYY-MM-DD'),
-      type: 'normal',
-      recurrent: { },
+      frequency: '',
       frequencies: [
         { value: '1w', text: this.$t('event.each_week') },
         { value: '2w', text: this.$t('event.each_2w') },
@@ -85,14 +103,6 @@ export default {
   },
   computed: {
     ...mapState(['settings', 'events']),
-    datePickerMode () {
-      const modeMap = {
-        multidate: 'range',
-        normal: 'single',
-        recurrent: 'single'
-      }
-      return modeMap[this.type]
-    },
     whenPatterns () {
       if (!this.value.date) { return }
       const date = dayjs(this.value.date)
@@ -138,97 +148,28 @@ export default {
       }
       return ''
     }
-    // todayEvents () {
-    //   if (this.type === 'multidate') {
-    //     if (!this.date || !this.date.start) { return }
-    //     const date_start = dayjs(this.date.start)
-    //     const date_end = dayjs(this.date.end)
-    //     return this.events.filter(e =>
-    //       !e.multidate
-    //         ? date_start.isSame(dayjs.unix(e.start_datetime), 'day') ||
-    //         (date_start.isBefore(dayjs.unix(e.start_dateime)) && date_end.isAfter(dayjs.unix(e.start_datetime)))
-    //         : date_start.isSame(dayjs.unix(e.start_datetime), 'day') || date_start.isSame(dayjs.unix(e.end_datetime)) ||
-    //         (date_start.isAfter(dayjs.unix(e.start_datetime)) && date_start.isBefore(dayjs.unix(e.end_datetime))))
-    //   } else if (this.type === 'recurrent') {
-    //     return []
-    //   } else {
-    //     const date = dayjs(this.date)
-    //     return this.events.filter(e =>
-    //       !e.multidate
-    //         ? !e.recurrent && date.isSame(dayjs.unix(e.start_datetime), 'day')
-    //         : dayjs.unix(e.start_datetime).isSame(date, 'day') ||
-    //           (dayjs.unix(e.start_datetime).isBefore(date) && dayjs.unix(e.end_datetime).isAfter(date))
-    //     )
-    //   }
-    // },
-    // attributes () {
-    //   let attributes = []
-    //   // attributes.push({ key: 'today', dates: new Date(), highlight: { color: 'red' } })
-
-    //   const date = dayjs(this.date)
-    //   const start = date.toDate()
-    //   attributes = attributes.concat(this.events
-    //     .filter(e => !e.multidate && (!e.parentId || this.type === 'recurrent'))
-    //     .map(e => ({ key: e.id, dot: { color: this.type === 'recurrent' ? 'orange' : 'green' }, dates: dayjs.unix(e.start_datetime).toDate() })))
-
-    //   if (this.type === 'recurrent' && this.date && this.date.length) {
-    //     let dates = {}
-    //     if (this.recurrent.frequency !== '1m') {
-    //       dates = {
-    //         weeklyInterval: this.recurrent.frequency === '1w' ? 1 : 2,
-    //         weekdays: date.day(),
-    //         start
-    //       }
-    //     } else if (this.recurrent.type === 'ordinal') {
-    //       const day = date.date()
-    //       dates = {
-    //         monthlyInterval: 1,
-    //         start,
-    //         days
-    //       }
-    //     } else if (this.recurrent.type === 'weekday') {
-    //       dates = {
-    //         monthlyInterval: 1,
-    //         start,
-    //         days: [Math.floor((date.day() - 1 / 7) + 1), date.day()]
-    //       }
-    //     } else {
-    //       dates = {
-    //         monthlyInterval: 1,
-    //         start,
-    //         days: [-1, date.day()]
-    //       }
-    //     }
-    //     attributes.push({
-    //       key: 'recurrent',
-    //       dot: { color: 'orange' },
-    //       dates
-    //     })
-    //   }
-    //   attributes = attributes.concat(this.events
-    //     .filter(e => e.multidate && !e.parentId)
-    //     .map(e => ({
-    //       key: e.id,
-    //       highlight: {},
-    //       dates: { start: dayjs.unix(e.start_datetime).toDate(), end: dayjs.unix(e.end_datetime).toDate() }
-    //     })))
-
-    //   return attributes
-    // }
   },
   methods: {
+    change (what, date) {
+      const v = this.value
+      v[what] = date
+      this.$emit('input', v)
+    },
     changeType (type) {
-      this.$emit('input', { type: type || 'normal', date: undefined })
+      this.$emit('input', { date: null, recurrent: {} })
+      if (type !== 'recurrent') {
+        this.frequency = ''
+      }
     },
     selectFrequency (f) {
-      this.$emit('input', { recurrent: { frequency: f }, date: null, type: 'recurrent' })
-    },
-    pick (date) {
-      if (this.value.type === 'normal' || this.value.type === 'recurrent' || this.value.date.length === 2) {
-        this.datePickerMenu = false
-      }
-      this.$emit('input', { date, type: this.value.type, recurrent: this.value.recurrent })
+      this.$emit('input', { recurrent: { frequency: f }, date: null })
     }
+    // pick (date) {
+    //   if (this.value.type === 'normal' || this.value.type === 'recurrent' || (this.value.date && this.value.date.length === 2)) {
+    //     this.datePickerMenu = false
+    //   }
+    //   this.$emit('input', { date, recurrent: this.value.recurrent })
+    // }
   }
 }
 </script>
diff --git a/pages/add/_edit.vue b/pages/add/_edit.vue
index 6dd51e17..207d244c 100644
--- a/pages/add/_edit.vue
+++ b/pages/add/_edit.vue
@@ -34,16 +34,15 @@
 
               //- When
               DateInput.col-12(v-model='date')
-              HourInput.col-12(v-model='time')
 
               //- Description
               Editor.col-12.mb-3(
+                :label="$t('event.description_description')"
                 v-model='event.description'
                 :placeholder="$t('event.description_description')"
                 max-height='400px')
 
               //- MEDIA / FLYER / POSTER
-
               v-file-input.col-6.mt-3(
                 :label="$t('common.media')"
                 :hint="$t('event.media_description')"
@@ -84,7 +83,7 @@ export default {
   },
   async asyncData ({ params, $axios, error, store }) {
     if (params.edit) {
-      const data = { time: {}, event: { place: {} } }
+      const data = { event: { place: {} } }
       data.id = params.edit
       data.edit = true
       let event
@@ -97,25 +96,12 @@ export default {
 
       data.event.place.name = event.place.name
       data.event.place.address = event.place.address || ''
-      data.date = {}
-      if (event.multidate) {
-        const start = dayjs.unix(event.start_datetime).format('YYYY-MM-DD')
-        const end = dayjs.unix(event.end_datetime).format('YYYY-MM-DD')
-        data.date = {
-          type: 'multidate',
-          date: [start, end]
-        }
-      } else if (event.recurrent) {
-        data.date.type = 'recurrent'
-        data.date.recurrent = event.recurrent
-      } else {
-        data.date.type = 'normal'
-        data.date.date = dayjs.unix(event.start_datetime).format('YYYY-MM-DD')
+      data.date = {
+        recurrent: event.recurrent,
+        from: new Date(dayjs.unix(event.start_datetime)),
+        due: new Date(dayjs.unix(event.end_datetime))
       }
 
-      data.time.start = dayjs.unix(event.start_datetime).format('HH:mm')
-      data.time.end = dayjs.unix(event.end_datetime).format('HH:mm')
-
       data.event.title = event.title
       data.event.description = event.description
       data.event.id = event.id
@@ -131,25 +117,27 @@ export default {
       valid: false,
       openImportDialog: false,
       event: {
-        type: 'normal',
         place: { name: '', address: '' },
         title: '',
         description: '',
         tags: [],
-        image: null,
-        recurrent: { frequency: '1m', days: [], type: 'weekday_desc' }
+        image: null
       },
       page: { month, year },
       fileList: [],
       id: null,
-      date: { type: 'normal', recurrent: {} },
-      time: { start: null, end: null },
+      date: { from: 0, due: 0, recurrent: null },
       edit: false,
       loading: false,
       mediaUrl: '',
       disableAddress: false
     }
   },
+  head () {
+    return {
+      title: `${this.settings.title} - ${this.$t('common.add_event')}`
+    }
+  },
   computed: {
     ...mapState(['tags', 'places', 'settings'])
   },
@@ -158,46 +146,16 @@ export default {
     eventImported (event) {
       this.event = Object.assign(this.event, event)
     },
-    // recurrentDays () {
-    //   if (this.event.type !== 'recurrent' || !this.date || !this.date.length) { return }
-    //   const type = this.event.recurrent.type
-    //   if (type === 'ordinal') { return map(this.date, d => dayjs(d).date()) } else if (type === 'weekday') { return map(this.date, d => dayjs(d).day() + 1) }
-    // },
-    // },
     cleanFile () {
       this.event.image = {}
     },
     async done () {
       if (!this.$refs.form.validate()) { return }
       this.loading = true
-      let start_datetime, end_datetime
-      const [start_hour, start_minute] = this.time.start.split(':')
-      if (!this.time.end) {
-        this.time.end = (Number(start_hour) + 2) + ':' + start_minute
-      }
-      const [end_hour, end_minute] = this.time.end.split(':')
 
       const formData = new FormData()
 
-      if (this.date.type === 'multidate') {
-        start_datetime = dayjs(this.date.date[0])
-          .hour(start_hour).minute(start_minute)
-        end_datetime = dayjs(this.date.date[1])
-          .hour(end_hour).minute(end_minute)
-      } else if (this.date.type === 'normal') {
-        start_datetime = dayjs(this.date.date).hour(start_hour).minute(start_minute)
-        end_datetime = dayjs(this.date.date).hour(end_hour).minute(end_minute)
-        if (end_hour < start_hour) {
-          end_datetime = end_datetime.add(1, 'day')
-        }
-      } else if (this.date.type === 'recurrent') {
-        start_datetime = dayjs(this.date.date).set('hour', start_hour).set('minute', start_minute)
-        end_datetime = dayjs(this.date.date).set('hour', end_hour).set('minute', end_minute)
-        if (end_hour < start_hour) {
-          end_datetime = end_datetime.add(1, 'day')
-        }
-        formData.append('recurrent', JSON.stringify(this.date.recurrent))
-      }
+      formData.append('recurrent', JSON.stringify(this.date.recurrent))
 
       if (this.event.image) {
         formData.append('image', this.event.image)
@@ -206,9 +164,9 @@ export default {
       formData.append('place_name', this.event.place.name)
       formData.append('place_address', this.event.place.address)
       formData.append('description', this.event.description)
-      formData.append('multidate', this.date.type === 'multidate')
-      formData.append('start_datetime', start_datetime.unix())
-      formData.append('end_datetime', end_datetime.unix())
+      // formData.append('multidate', this.date.type === 'multidate')
+      formData.append('start_datetime', dayjs(this.date.from).unix())
+      formData.append('end_datetime', this.date.due && dayjs(this.date.due).unix())
 
       if (this.edit) {
         formData.append('id', this.event.id)
@@ -222,7 +180,6 @@ export default {
         }
         this.updateMeta()
         this.$router.replace('/')
-        this.loading = false
         this.$root.$message(this.$auth.loggedIn ? 'event.added' : 'event.added_anon', { color: 'success' })
       } catch (e) {
         switch (e.request.status) {
@@ -235,11 +192,6 @@ export default {
         this.loading = false
       }
     }
-  },
-  head () {
-    return {
-      title: `${this.settings.title} - ${this.$t('common.add_event')}`
-    }
   }
 }
 </script>