From 2785def67cc07c49eb617fde40603269077bd708 Mon Sep 17 00:00:00 2001
From: Mert Sincan <sincan.mert@gmail.com>
Date: Fri, 18 Oct 2024 17:22:56 +0100
Subject: [PATCH] Add form support to `SelectButton`

---
 .../src/selectbutton/BaseSelectButton.vue       | 10 ++--------
 .../primevue/src/selectbutton/SelectButton.vue  | 17 +++++++++--------
 .../src/selectbutton/style/SelectButtonStyle.js |  4 ++--
 3 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/packages/primevue/src/selectbutton/BaseSelectButton.vue b/packages/primevue/src/selectbutton/BaseSelectButton.vue
index 06ccc18dc..21cb4d672 100644
--- a/packages/primevue/src/selectbutton/BaseSelectButton.vue
+++ b/packages/primevue/src/selectbutton/BaseSelectButton.vue
@@ -1,12 +1,11 @@
 <script>
-import BaseComponent from '@primevue/core/basecomponent';
+import BaseEditableHolder from '@primevue/core/baseeditableholder';
 import SelectButtonStyle from 'primevue/selectbutton/style';
 
 export default {
     name: 'BaseSelectButton',
-    extends: BaseComponent,
+    extends: BaseEditableHolder,
     props: {
-        modelValue: null,
         options: Array,
         optionLabel: null,
         optionValue: null,
@@ -16,11 +15,6 @@ export default {
             type: Boolean,
             default: true
         },
-        invalid: {
-            type: Boolean,
-            default: false
-        },
-        disabled: Boolean,
         dataKey: null,
         ariaLabelledby: {
             type: String,
diff --git a/packages/primevue/src/selectbutton/SelectButton.vue b/packages/primevue/src/selectbutton/SelectButton.vue
index d87993003..64208988e 100755
--- a/packages/primevue/src/selectbutton/SelectButton.vue
+++ b/packages/primevue/src/selectbutton/SelectButton.vue
@@ -2,6 +2,7 @@
     <div :class="cx('root')" role="group" :aria-labelledby="ariaLabelledby" v-bind="ptmi('root')">
         <template v-for="(option, index) of options" :key="getOptionRenderKey(option)">
             <ToggleButton
+                :name="$formName"
                 :modelValue="isSelected(option)"
                 :onLabel="getOptionLabel(option)"
                 :offLabel="getOptionLabel(option)"
@@ -21,7 +22,7 @@
 </template>
 
 <script>
-import { resolveFieldData, equals } from '@primeuix/utils/object';
+import { equals, resolveFieldData } from '@primeuix/utils/object';
 import Ripple from 'primevue/ripple';
 import ToggleButton from 'primevue/togglebutton';
 import BaseSelectButton from './BaseSelectButton.vue';
@@ -30,7 +31,7 @@ export default {
     name: 'SelectButton',
     extends: BaseSelectButton,
     inheritAttrs: false,
-    emits: ['update:modelValue', 'change'],
+    emits: ['change'],
     methods: {
         getOptionLabel(option) {
             return this.optionLabel ? resolveFieldData(option, this.optionLabel) : option;
@@ -59,13 +60,13 @@ export default {
             let newValue;
 
             if (this.multiple) {
-                if (selected) newValue = this.modelValue.filter((val) => !equals(val, optionValue, this.equalityKey));
-                else newValue = this.modelValue ? [...this.modelValue, optionValue] : [optionValue];
+                if (selected) newValue = this.d_value.filter((val) => !equals(val, optionValue, this.equalityKey));
+                else newValue = this.d_value ? [...this.d_value, optionValue] : [optionValue];
             } else {
                 newValue = selected ? null : optionValue;
             }
 
-            this.$emit('update:modelValue', newValue);
+            this.updateValue(newValue, event);
             this.$emit('change', { event: event, value: newValue });
         },
         isSelected(option) {
@@ -73,8 +74,8 @@ export default {
             let optionValue = this.getOptionValue(option);
 
             if (this.multiple) {
-                if (this.modelValue) {
-                    for (let val of this.modelValue) {
+                if (this.d_value) {
+                    for (let val of this.d_value) {
                         if (equals(val, optionValue, this.equalityKey)) {
                             selected = true;
                             break;
@@ -82,7 +83,7 @@ export default {
                     }
                 }
             } else {
-                selected = equals(this.modelValue, optionValue, this.equalityKey);
+                selected = equals(this.d_value, optionValue, this.equalityKey);
             }
 
             return selected;
diff --git a/packages/primevue/src/selectbutton/style/SelectButtonStyle.js b/packages/primevue/src/selectbutton/style/SelectButtonStyle.js
index d228dc0e5..5ea5d598b 100644
--- a/packages/primevue/src/selectbutton/style/SelectButtonStyle.js
+++ b/packages/primevue/src/selectbutton/style/SelectButtonStyle.js
@@ -37,10 +37,10 @@ const theme = ({ dt }) => `
 `;
 
 const classes = {
-    root: ({ props }) => [
+    root: ({ instance }) => [
         'p-selectbutton p-component',
         {
-            'p-invalid': props.invalid
+            'p-invalid': instance.$invalid // @todo: check
         }
     ]
 };