Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fix(compiler-sfc): always generate runtime prop type for Function (#7112
)

fix #7111
  • Loading branch information
sxzz committed Nov 14, 2022
1 parent 6e8b6b3 commit 584eae6
Show file tree
Hide file tree
Showing 3 changed files with 61 additions and 6 deletions.
Expand Up @@ -1729,6 +1729,35 @@ const props = __props as {



return { props, get defaults() { return defaults } }
}

})"
`;

exports[`SFC compile <script setup> with TypeScript withDefaults (dynamic) w/ production mode 1`] = `
"import { mergeDefaults as _mergeDefaults, defineComponent as _defineComponent } from 'vue'
import { defaults } from './foo'

export default /*#__PURE__*/_defineComponent({
props: _mergeDefaults({
foo: { type: Function },
bar: { type: Boolean },
baz: { type: [Boolean, Function] },
qux: null
}, { ...defaults }),
setup(__props: any, { expose }) {
expose();

const props = __props as {
foo: () => void
bar: boolean
baz: boolean | (() => void)
qux: string | number
};



return { props, get defaults() { return defaults } }
}

Expand Down
29 changes: 29 additions & 0 deletions packages/compiler-sfc/__tests__/compileScript.spec.ts
Expand Up @@ -1144,6 +1144,35 @@ const emit = defineEmits(['a', 'b'])
)
})

// #7111
test('withDefaults (dynamic) w/ production mode', () => {
const { content } = compile(
`
<script setup lang="ts">
import { defaults } from './foo'
const props = withDefaults(defineProps<{
foo: () => void
bar: boolean
baz: boolean | (() => void)
qux: string | number
}>(), { ...defaults })
</script>
`,
{ isProd: true }
)
assertCode(content)
expect(content).toMatch(`import { mergeDefaults as _mergeDefaults`)
expect(content).toMatch(
`
_mergeDefaults({
foo: { type: Function },
bar: { type: Boolean },
baz: { type: [Boolean, Function] },
qux: null
}, { ...defaults })`.trim()
)
})

test('defineEmits w/ type', () => {
const { content } = compile(`
<script setup lang="ts">
Expand Down
9 changes: 3 additions & 6 deletions packages/compiler-sfc/src/compileScript.ts
Expand Up @@ -822,12 +822,9 @@ export function compileScript(
)}, required: ${required}${
defaultString ? `, ${defaultString}` : ``
} }`
} else if (
type.some(
el => el === 'Boolean' || (defaultString && el === 'Function')
)
) {
// #4783 production: if boolean or defaultString and function exists, should keep the type.
} else if (type.some(el => el === 'Boolean' || el === 'Function')) {
// #4783, #7111 for boolean or function, should keep the type
// in production
return `${key}: { type: ${toRuntimeTypeString(type)}${
defaultString ? `, ${defaultString}` : ``
} }`
Expand Down

0 comments on commit 584eae6

Please sign in to comment.