mirror of https://github.com/vuejs/core.git
chore(reactivity): replace console.warn() to warn function (#10394)
This commit is contained in:
parent
ff130c4702
commit
411596c07b
|
@ -7,6 +7,7 @@ import {
|
||||||
} from './reactiveEffect'
|
} from './reactiveEffect'
|
||||||
import { ReactiveFlags, TrackOpTypes, TriggerOpTypes } from './constants'
|
import { ReactiveFlags, TrackOpTypes, TriggerOpTypes } from './constants'
|
||||||
import { capitalize, hasChanged, hasOwn, isMap, toRawType } from '@vue/shared'
|
import { capitalize, hasChanged, hasOwn, isMap, toRawType } from '@vue/shared'
|
||||||
|
import { warn } from './warning'
|
||||||
|
|
||||||
type CollectionTypes = IterableCollections | WeakCollections
|
type CollectionTypes = IterableCollections | WeakCollections
|
||||||
|
|
||||||
|
@ -223,7 +224,7 @@ function createReadonlyMethod(type: TriggerOpTypes): Function {
|
||||||
return function (this: CollectionTypes, ...args: unknown[]) {
|
return function (this: CollectionTypes, ...args: unknown[]) {
|
||||||
if (__DEV__) {
|
if (__DEV__) {
|
||||||
const key = args[0] ? `on key "${args[0]}" ` : ``
|
const key = args[0] ? `on key "${args[0]}" ` : ``
|
||||||
console.warn(
|
warn(
|
||||||
`${capitalize(type)} operation ${key}failed: target is readonly.`,
|
`${capitalize(type)} operation ${key}failed: target is readonly.`,
|
||||||
toRaw(this),
|
toRaw(this),
|
||||||
)
|
)
|
||||||
|
@ -397,7 +398,7 @@ function checkIdentityKeys(
|
||||||
const rawKey = toRaw(key)
|
const rawKey = toRaw(key)
|
||||||
if (rawKey !== key && has.call(target, rawKey)) {
|
if (rawKey !== key && has.call(target, rawKey)) {
|
||||||
const type = toRawType(target)
|
const type = toRawType(target)
|
||||||
console.warn(
|
warn(
|
||||||
`Reactive ${type} contains both the raw and reactive ` +
|
`Reactive ${type} contains both the raw and reactive ` +
|
||||||
`versions of the same object${type === `Map` ? ` as keys` : ``}, ` +
|
`versions of the same object${type === `Map` ? ` as keys` : ``}, ` +
|
||||||
`which can lead to inconsistencies. ` +
|
`which can lead to inconsistencies. ` +
|
||||||
|
|
|
@ -13,6 +13,7 @@ import {
|
||||||
} from './collectionHandlers'
|
} from './collectionHandlers'
|
||||||
import type { RawSymbol, Ref, UnwrapRefSimple } from './ref'
|
import type { RawSymbol, Ref, UnwrapRefSimple } from './ref'
|
||||||
import { ReactiveFlags } from './constants'
|
import { ReactiveFlags } from './constants'
|
||||||
|
import { warn } from './warning'
|
||||||
|
|
||||||
export interface Target {
|
export interface Target {
|
||||||
[ReactiveFlags.SKIP]?: boolean
|
[ReactiveFlags.SKIP]?: boolean
|
||||||
|
@ -247,7 +248,7 @@ function createReactiveObject(
|
||||||
) {
|
) {
|
||||||
if (!isObject(target)) {
|
if (!isObject(target)) {
|
||||||
if (__DEV__) {
|
if (__DEV__) {
|
||||||
console.warn(`value cannot be made reactive: ${String(target)}`)
|
warn(`value cannot be made reactive: ${String(target)}`)
|
||||||
}
|
}
|
||||||
return target
|
return target
|
||||||
}
|
}
|
||||||
|
|
|
@ -25,6 +25,7 @@ import type { ShallowReactiveMarker } from './reactive'
|
||||||
import { type Dep, createDep } from './dep'
|
import { type Dep, createDep } from './dep'
|
||||||
import { ComputedRefImpl } from './computed'
|
import { ComputedRefImpl } from './computed'
|
||||||
import { getDepFromReactive } from './reactiveEffect'
|
import { getDepFromReactive } from './reactiveEffect'
|
||||||
|
import { warn } from './warning'
|
||||||
|
|
||||||
declare const RefSymbol: unique symbol
|
declare const RefSymbol: unique symbol
|
||||||
export declare const RawSymbol: unique symbol
|
export declare const RawSymbol: unique symbol
|
||||||
|
@ -345,7 +346,7 @@ export type ToRefs<T = any> = {
|
||||||
*/
|
*/
|
||||||
export function toRefs<T extends object>(object: T): ToRefs<T> {
|
export function toRefs<T extends object>(object: T): ToRefs<T> {
|
||||||
if (__DEV__ && !isProxy(object)) {
|
if (__DEV__ && !isProxy(object)) {
|
||||||
console.warn(`toRefs() expects a reactive object but received a plain one.`)
|
warn(`toRefs() expects a reactive object but received a plain one.`)
|
||||||
}
|
}
|
||||||
const ret: any = isArray(object) ? new Array(object.length) : {}
|
const ret: any = isArray(object) ? new Array(object.length) : {}
|
||||||
for (const key in object) {
|
for (const key in object) {
|
||||||
|
|
Loading…
Reference in New Issue