From 19ecc232aad03fb984b9a22eb20e4360c44f54fc Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Mon, 5 Feb 2024 14:05:45 +0100
Subject: [PATCH] chore(breakout): migrate to NcInputField to support number
 type

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../BreakoutRoomsEditor.vue                   | 26 +++++++++----------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue b/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
index 106b5c6698e..2a784c00b42 100644
--- a/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
+++ b/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
@@ -33,14 +33,13 @@
 					<p v-if="isInvalidAmount" class="breakout-rooms-editor__error-hint">
 						{{ t('spreed', 'You can create from 1 to 20 breakout rooms.') }}
 					</p>
-					<NcTextField id="room-number"
-						ref="textField"
-						:value="amount.toString()"
+					<NcInputField id="room-number"
+						ref="inputField"
+						:value.sync="amount"
 						class="breakout-rooms-editor__number-input"
 						type="number"
 						min="1"
-						max="20"
-						@update:value="setAmount" />
+						max="20" />
 
 					<label class="breakout-rooms-editor__caption">{{ t('spreed', 'Assignment method') }}</label>
 					<fieldset>
@@ -93,8 +92,8 @@
 <script>
 import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcCheckboxRadioSwitch from '@nextcloud/vue/dist/Components/NcCheckboxRadioSwitch.js'
+import NcInputField from '@nextcloud/vue/dist/Components/NcInputField.js'
 import NcModal from '@nextcloud/vue/dist/Components/NcModal.js'
-import NcTextField from '@nextcloud/vue/dist/Components/NcTextField.js'
 
 import BreakoutRoomsParticipantsEditor from './BreakoutRoomsParticipantsEditor.vue'
 
@@ -105,7 +104,7 @@ export default {
 		BreakoutRoomsParticipantsEditor,
 		NcButton,
 		NcCheckboxRadioSwitch,
-		NcTextField,
+		NcInputField,
 		NcModal,
 	},
 
@@ -140,6 +139,12 @@ export default {
 		},
 	},
 
+	watch: {
+		amount(value) {
+			this.isInvalidAmount = isNaN(value) || !this.$refs.inputField.$refs.input?.checkValidity()
+		},
+	},
+
 	methods: {
 		async handleCreateRooms() {
 			try {
@@ -153,13 +158,6 @@ export default {
 				console.debug(error)
 			}
 		},
-
-		// FIXME upstream: support of value type as Number should be added to NcInputField,
-		// now it breaks validation. Another option: Create NcNumberField component
-		setAmount(value) {
-			this.amount = parseFloat(value)
-			this.isInvalidAmount = isNaN(this.amount) || !this.$refs.textField.$refs.inputField.$refs.input?.checkValidity()
-		},
 	},
 }
 </script>