Skip to content

Commit

Permalink
fix(warn): fix component name inference in warning trace
Browse files Browse the repository at this point in the history
  • Loading branch information
yyx990803 committed Apr 20, 2020
1 parent 054ccec commit 0278992
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 12 deletions.
9 changes: 5 additions & 4 deletions packages/runtime-core/src/component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ export interface SFCInternalOptions {
__cssModules?: Data
__hmrId?: string
__hmrUpdated?: boolean
__file?: string
}

export interface FunctionalComponent<
Expand Down Expand Up @@ -540,16 +541,16 @@ const classify = (str: string): string =>

export function formatComponentName(
Component: Component,
file?: string
isRoot = false
): string {
let name = isFunction(Component)
? Component.displayName || Component.name
: Component.name
if (!name && file) {
const match = file.match(/([^/\\]+)\.vue$/)
if (!name && Component.__file) {
const match = Component.__file.match(/([^/\\]+)\.vue$/)
if (match) {
name = match[1]
}
}
return name ? classify(name) : 'Anonymous'
return name ? classify(name) : isRoot ? `App` : `Anonymous`
}
13 changes: 5 additions & 8 deletions packages/runtime-core/src/warning.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,7 @@ export function warn(msg: string, ...args: any[]) {
msg + args.join(''),
instance && instance.proxy,
trace
.map(
({ vnode }) =>
`at <${formatComponentName(vnode.type as Component)}>`
)
.map(({ vnode }) => `at <${formatComponentName(vnode.type)}>`)
.join('\n'),
trace
]
Expand Down Expand Up @@ -111,12 +108,12 @@ function formatTrace(trace: ComponentTraceStack): any[] {
function formatTraceEntry({ vnode, recurseCount }: TraceEntry): any[] {
const postfix =
recurseCount > 0 ? `... (${recurseCount} recursive calls)` : ``
const open = ` at <${formatComponentName(vnode)}`
const isRoot = vnode.component!.parent == null
const open = ` at <${formatComponentName(vnode.type, isRoot)}`
const close = `>` + postfix
const rootLabel = vnode.component!.parent == null ? `(Root)` : ``
return vnode.props
? [open, ...formatProps(vnode.props), close, rootLabel]
: [open + close, rootLabel]
? [open, ...formatProps(vnode.props), close]
: [open + close]
}

function formatProps(props: Data): any[] {
Expand Down

0 comments on commit 0278992

Please sign in to comment.