refactor: minor code style aligment

Author: J-Sek

packages/api-generator/src/locale/en/DataTable-group.json

@@ -2,8 +2,8 @@
   "props": {
     "groupBy": "Defines the grouping of the table items.",
     "groupKey": "Custom function to generate group IDs. Receives `{ key, value, parentKey }` where `parentKey` is `null` for top-level groups. Useful when group values contain special characters or are non-string types.",
-    "openAll": "Opens all groups by default. Synchronizes `v-model:opened`, so closed groups are not re-opened accidentally.",
-    "opened": "An array of group IDs that should be open. Supports two-way binding with `v-model:opened`.",
+    "openAll": "Opens all groups by default. Synchronizes with **v-model:opened**, so closed groups are not re-opened accidentally.",
+    "opened": "Array of group IDs that should be open. Can be bound to external variable using **v-model:opened**.",
     "pageBy": "Controls how pagination counts items.\n- **item** paginates by individual items,\n- **auto** paginates by top-level groups and falls back to regular items if **group-by** is empty,\n- **any** paginates by both items and groups combined"
   }
 }

packages/vuetify/src/components/VDataIterator/VDataIterator.tsx

@@ -1,7 +1,13 @@
 // Components
 import { VFadeTransition } from '@/components/transitions'
 import { makeDataTableExpandProps, provideExpanded } from '@/components/VDataTable/composables/expand'
-import { makeDataTableGroupProps, provideGroupBy, syncOpenedWithGroups, useGroupedItems } from '@/components/VDataTable/composables/group'
+import {
+  createGroupBy,
+  makeDataTableGroupProps,
+  provideGroupBy,
+  syncOpenedWithGroups,
+  useGroupedItems,
+} from '@/components/VDataTable/composables/group'
 import { useOptions } from '@/components/VDataTable/composables/options'
 import {
   createPagination,
@@ -17,7 +23,6 @@ import { makeDataIteratorItemsProps, useDataIteratorItems } from './composables/
 import { makeComponentProps } from '@/composables/component'
 import { makeFilterProps, useFilter } from '@/composables/filter'
 import { LoaderSlot } from '@/composables/loader'
-import { useProxiedModel } from '@/composables/proxiedModel'
 import { makeTagProps } from '@/composables/tag'
 import { useToggleScope } from '@/composables/toggleScope'
 import { makeTransitionProps, MaybeTransition } from '@/composables/transition'
@@ -110,10 +115,7 @@ export const VDataIterator = genericComponent<new <T> (
   },
 
   setup (props, { slots }) {
-    const groupBy = useProxiedModel(props, 'groupBy')
-    const openedModel = useProxiedModel(props, 'opened')
-    const openAll = toRef(() => props.openAll)
-    const groupKeyFn = toRef(() => props.groupKey)
+    const { groupBy, opened, openAll, groupKey } = createGroupBy(props)
     const search = toRef(() => props.search)
 
     const { items } = useDataIteratorItems(props)
@@ -125,15 +127,15 @@ export const VDataIterator = genericComponent<new <T> (
     const { toggleSort } = provideSort({ initialSortOrder, sortBy, multiSort, mustSort, page })
     const {
       sortByWithGroups,
-      opened,
+      opened: openedGroups,
       extractRows,
       isGroupOpen,
       toggleGroup,
-    } = provideGroupBy({ groupBy, sortBy, opened: openedModel })
+    } = provideGroupBy({ groupBy, sortBy, opened })
 
     const { sortedItems } = useSortedItems(props, filteredItems, sortByWithGroups, { transform: item => item.raw })
-    syncOpenedWithGroups(opened, openAll, sortedItems, groupBy, groupKeyFn)
-    const { flatItems } = useGroupedItems(sortedItems, groupBy, opened, false, isGroupOpen, groupKeyFn)
+    syncOpenedWithGroups(openedGroups, openAll, sortedItems, groupBy, groupKey)
+    const { flatItems } = useGroupedItems(sortedItems, groupBy, openedGroups, false, isGroupOpen, groupKey)
 
     const manualPagination = toRef(() => !isEmpty(props.itemsLength))
     const itemsLength = toRef(() => manualPagination.value ? Number(props.itemsLength) : flatItems.value.length)

packages/vuetify/src/components/VDataTable/composables/group.ts

@@ -33,7 +33,7 @@ export interface GroupSummary<T = any> {
   items: readonly (T | Group<T> | GroupSummary<T>)[]
 }
 
-export type GroupKeyFn = (options: { key: string, value: any, parentKey: string | null }) => string
+export type GroupKeyFunction = (options: { key: string, value: any, parentKey: string | null }) => string
 
 export const makeDataTableGroupProps = propsFactory({
   groupBy: {
@@ -45,7 +45,7 @@ export const makeDataTableGroupProps = propsFactory({
     default: () => ([]),
   },
   openAll: Boolean,
-  groupKey: Function as PropType<GroupKeyFn>,
+  groupKey: Function as PropType<GroupKeyFunction>,
 }, 'DataTable-group')
 
 const VDataTableGroupSymbol: InjectionKey<{
@@ -63,7 +63,7 @@ type GroupProps = {
   opened: readonly string[]
   'onUpdate:opened': ((value: string[]) => void) | undefined
   openAll: boolean
-  groupKey: GroupKeyFn | undefined
+  groupKey: GroupKeyFunction | undefined
 }
 
 export function createGroupBy (props: GroupProps) {
@@ -170,7 +170,7 @@ const defaultGroupId = (key: string, value: any, parentKey: string) => `${parent
 function groupItems <T extends GroupableItem> (
   items: readonly T[],
   groupBy: readonly string[],
-  groupKeyFn?: GroupKeyFn,
+  groupKeyFn?: GroupKeyFunction,
   depth = 0,
   parentKey = 'root',
 ) {
@@ -210,7 +210,7 @@ export function syncOpenedWithGroups (
   openAll: MaybeRefOrGetter<boolean>,
   items: MaybeRefOrGetter<readonly GroupableItem[]>,
   groupBy: Ref<readonly SortItem[]>,
-  groupKeyFn?: MaybeRefOrGetter<GroupKeyFn | undefined>,
+  groupKeyFn?: MaybeRefOrGetter<GroupKeyFunction | undefined>,
 ) {
   const allIds = computed(() => {
     if (!toValue(openAll) || !groupBy.value.length) return new Set<string>()
@@ -271,7 +271,7 @@ export function useGroupedItems <T extends GroupableItem> (
   opened: Ref<Set<string>>,
   hasSummary: MaybeRefOrGetter<boolean>,
   isGroupOpen?: (group: Group) => boolean,
-  groupKeyFn?: MaybeRefOrGetter<GroupKeyFn | undefined>,
+  groupKeyFn?: MaybeRefOrGetter<GroupKeyFunction | undefined>,
 ) {
   const groups = computed(() => {
     if (!groupBy.value.length) return []