chore(TimePicker): revert to previous as no longer needed

Signed-off-by: Grigory Vodyanov <[email protected]>

src/components/Shared/TimePicker.vue

@@ -4,53 +4,62 @@
 -->
 
 <template>
-	<NcActions :manual-open="true"
-		:open="isListOpen"
-		@click="isListOpen = !isListOpen">
-		<template #icon>
-			<NcTextField :value.sync="date" :error="isInvalidTime">
-				<ClockOutline/>
-			</NcTextField>
-		</template>
-		<NcActionButton v-for="time in timeList" :key="time" @click="changeFromList(parse(time))">
-			<template #icon></template>
-			{{ time }}
-		</NcActionButton>
-	</NcActions>
+	<DateTimePicker ::clearable="false"
+		:first-day-of-week="firstDay"
+		:format="format"
+		:lang="lang"
+		:minute-step="5"
+		:show-second="false"
+		type="time"
+		:use12h="showAmPm"
+		:value="date"
+		v-bind="$attrs"
+		v-on="$listeners"
+		@change="change" />
 </template>
 
 <script>
-import { NcActions, NcTextField, NcActionButton } from '@nextcloud/vue'
-import ClockOutline from 'vue-material-design-icons/ClockOutline.vue'
+import { NcDateTimePicker as DateTimePicker } from '@nextcloud/vue'
 import moment from '@nextcloud/moment'
 import { mapState } from 'pinia'
+import {
+	getFirstDay,
+} from '@nextcloud/l10n'
+import { getLangConfigForVue2DatePicker } from '../../utils/localization.js'
 import useSettingsStore from '../../store/settings.js'
 
 export default {
 	name: 'TimePicker',
 	components: {
-		NcActions,
-		NcTextField,
-		NcActionButton,
-		ClockOutline,
+		DateTimePicker,
 	},
 	props: {
-		initialDate: {
+		date: {
 			type: Date,
 			required: true,
 		},
 	},
 	data() {
 		return {
-			date: '',
-			isInvalidTime: false,
-			isListOpen: false,
+			firstDay: getFirstDay() === 0 ? 7 : getFirstDay(),
+			format: {
+				stringify: this.stringify,
+				parse: this.parse,
+			},
 		}
 	},
 	computed: {
 		...mapState(useSettingsStore, {
 			locale: 'momentLocale',
 		}),
+		/**
+		 * Returns the lang config for vue2-datepicker
+		 *
+		 * @return {object}
+		 */
+		lang() {
+			return getLangConfigForVue2DatePicker(this.locale)
+		},
 		/**
 		 * Whether or not to offer am/pm in the timepicker
 		 *
@@ -62,51 +71,14 @@ export default {
 
 			return timeFormat.indexOf('a') !== -1
 		},
-
-		timeList() {
-			const times = []
-			let currentTime = moment(this.initialDate)
-
-			for (let i = 0; i < 10; i++) {
-				times.push(currentTime.format('LT'))
-				currentTime = currentTime.add(15, 'minutes')
-			}
-
-			return times
-		},
-	},
-	watch: {
-		date(value) {
-			let isValidTime = false
-			isValidTime = !isValidTime ? moment(value, 'LT', true).isValid() : isValidTime
-			isValidTime = !isValidTime ? moment(value, 'HH:mm', true).isValid() : isValidTime
-			isValidTime = !isValidTime ? moment(value, 'H:mm', true).isValid() : isValidTime
-
-			// Meaning it was changed through textfield
-			if (!(value instanceof Date) && isValidTime) {
-				this.isInvalidTime = false
-
-				const parsedDate = this.parse(value)
-				this.$emit('change', parsedDate)
-			} else if (!(value instanceof Date)) {
-				this.isInvalidTime = true
-			}
-		},
-	},
-	mounted() {
-		this.date = this.stringify(this.initialDate)
 	},
 	methods: {
 		/**
 		 * Emits a change event for the Date
 		 *
 		 * @param {Date} date The new Date object
 		 */
-		changeFromList(date) {
-			this.isInvalidTime = false
-			this.isListOpen = false
-
-			this.date = this.stringify(date)
+		change(date) {
 			this.$emit('change', date)
 		},
 		/**
@@ -125,26 +97,8 @@ export default {
 		 * @return {Date}
 		 */
 		parse(value) {
-			try {
-				return moment(value, 'LT', this.locale).toDate()
-			} catch (e) {
-				console.error(e)
-			}
+			return moment(value, 'LT', this.locale).toDate()
 		},
 	},
 }
 </script>
-
-<style scoped>
-:deep(.action-button__icon) {
-	display: none;
-}
-
-:deep(.action-button__text) {
-	margin: 0 8px;
-}
-
-:deep(.input-field__icon--trailing) {
-	display: none;
-}
-</style>