fix(ivy): align VE + Ivy #ref types in fullTemplateTypeCheck: false (#33261)
In View Engine, with fullTemplateTypeCheck mode disabled, the type of any inferred based on the entity being referenced. This is a bug, since the goal with fullTemplateTypeCheck: false is for Ivy and VE to be aligned in terms of type inference. This commit adds a 'checkTypeOfReference' flag in the TypeCheckingConfig to control this inference, and sets it to false when fullTemplateTypeCheck is disabled. PR Close #33261
This commit is contained in:
parent
ba29e4d953
commit
77240e1b60
@ -435,6 +435,7 @@ export class NgtscProgram implements api.Program {
|
|||||||
// - error TS2531: Object is possibly 'null'.
|
// - error TS2531: Object is possibly 'null'.
|
||||||
// - error TS2339: Property 'value' does not exist on type 'EventTarget'.
|
// - error TS2339: Property 'value' does not exist on type 'EventTarget'.
|
||||||
checkTypeOfDomEvents: false,
|
checkTypeOfDomEvents: false,
|
||||||
|
checkTypeOfReferences: true,
|
||||||
checkTypeOfPipes: true,
|
checkTypeOfPipes: true,
|
||||||
strictSafeNavigationTypes: true,
|
strictSafeNavigationTypes: true,
|
||||||
};
|
};
|
||||||
@ -449,6 +450,7 @@ export class NgtscProgram implements api.Program {
|
|||||||
checkTypeOfOutputEvents: false,
|
checkTypeOfOutputEvents: false,
|
||||||
checkTypeOfAnimationEvents: false,
|
checkTypeOfAnimationEvents: false,
|
||||||
checkTypeOfDomEvents: false,
|
checkTypeOfDomEvents: false,
|
||||||
|
checkTypeOfReferences: false,
|
||||||
checkTypeOfPipes: false,
|
checkTypeOfPipes: false,
|
||||||
strictSafeNavigationTypes: false,
|
strictSafeNavigationTypes: false,
|
||||||
};
|
};
|
||||||
|
@ -133,6 +133,15 @@ export interface TypeCheckingConfig {
|
|||||||
*/
|
*/
|
||||||
checkTypeOfDomEvents: boolean;
|
checkTypeOfDomEvents: boolean;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether to infer the type of local references.
|
||||||
|
*
|
||||||
|
* If this is `true`, the type of any `#ref` variable in the template will be determined by the
|
||||||
|
* referenced entity (either a directive or a DOM element). If set to `false`, the type of `ref`
|
||||||
|
* will be `any`.
|
||||||
|
*/
|
||||||
|
checkTypeOfReferences: boolean;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Whether to include type information from pipes in the type-checking operation.
|
* Whether to include type information from pipes in the type-checking operation.
|
||||||
*
|
*
|
||||||
|
@ -954,6 +954,11 @@ class TcbExpressionTranslator {
|
|||||||
addParseSpanInfo(expr, toAbsoluteSpan(ast.span, this.sourceSpan));
|
addParseSpanInfo(expr, toAbsoluteSpan(ast.span, this.sourceSpan));
|
||||||
return expr;
|
return expr;
|
||||||
} else if (binding instanceof TmplAstReference) {
|
} else if (binding instanceof TmplAstReference) {
|
||||||
|
if (!this.tcb.env.config.checkTypeOfReferences) {
|
||||||
|
// References are pinned to 'any'.
|
||||||
|
return NULL_AS_ANY;
|
||||||
|
}
|
||||||
|
|
||||||
const target = this.tcb.boundTarget.getReferenceTarget(binding);
|
const target = this.tcb.boundTarget.getReferenceTarget(binding);
|
||||||
if (target === null) {
|
if (target === null) {
|
||||||
throw new Error(`Unbound reference? ${binding.name}`);
|
throw new Error(`Unbound reference? ${binding.name}`);
|
||||||
|
@ -155,6 +155,7 @@ export const ALL_ENABLED_CONFIG: TypeCheckingConfig = {
|
|||||||
checkTypeOfOutputEvents: true,
|
checkTypeOfOutputEvents: true,
|
||||||
checkTypeOfAnimationEvents: true,
|
checkTypeOfAnimationEvents: true,
|
||||||
checkTypeOfDomEvents: true,
|
checkTypeOfDomEvents: true,
|
||||||
|
checkTypeOfReferences: true,
|
||||||
checkTypeOfPipes: true,
|
checkTypeOfPipes: true,
|
||||||
strictSafeNavigationTypes: true,
|
strictSafeNavigationTypes: true,
|
||||||
};
|
};
|
||||||
@ -198,6 +199,7 @@ export function tcb(
|
|||||||
checkTypeOfOutputEvents: true,
|
checkTypeOfOutputEvents: true,
|
||||||
checkTypeOfAnimationEvents: true,
|
checkTypeOfAnimationEvents: true,
|
||||||
checkTypeOfDomEvents: true,
|
checkTypeOfDomEvents: true,
|
||||||
|
checkTypeOfReferences: true,
|
||||||
checkTypeOfPipes: true,
|
checkTypeOfPipes: true,
|
||||||
checkTemplateBodies: true,
|
checkTemplateBodies: true,
|
||||||
strictSafeNavigationTypes: true,
|
strictSafeNavigationTypes: true,
|
||||||
|
@ -291,6 +291,7 @@ describe('type check blocks', () => {
|
|||||||
checkTypeOfOutputEvents: true,
|
checkTypeOfOutputEvents: true,
|
||||||
checkTypeOfAnimationEvents: true,
|
checkTypeOfAnimationEvents: true,
|
||||||
checkTypeOfDomEvents: true,
|
checkTypeOfDomEvents: true,
|
||||||
|
checkTypeOfReferences: true,
|
||||||
checkTypeOfPipes: true,
|
checkTypeOfPipes: true,
|
||||||
strictSafeNavigationTypes: true,
|
strictSafeNavigationTypes: true,
|
||||||
};
|
};
|
||||||
@ -416,6 +417,20 @@ describe('type check blocks', () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('config.checkTypeOfReferences', () => {
|
||||||
|
const TEMPLATE = `<input #ref>{{ref.value}}`;
|
||||||
|
|
||||||
|
it('should trace references when enabled', () => {
|
||||||
|
const block = tcb(TEMPLATE);
|
||||||
|
expect(block).toContain('(_t1).value');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should use any for reference types when disabled', () => {
|
||||||
|
const DISABLED_CONFIG: TypeCheckingConfig = {...BASE_CONFIG, checkTypeOfReferences: false};
|
||||||
|
const block = tcb(TEMPLATE, [], DISABLED_CONFIG);
|
||||||
|
expect(block).toContain('(null as any).value');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
describe('config.checkTypeOfPipes', () => {
|
describe('config.checkTypeOfPipes', () => {
|
||||||
const TEMPLATE = `{{a | test:b:c}}`;
|
const TEMPLATE = `{{a | test:b:c}}`;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user