Graham/angular-cli
1
0
Fork 0
mirror of https://github.com/angular/angular-cli.git synced 2025-05-17 02:54:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(@angular-devkit/build-optimizer): remove decorators calls when tslib helpers are inlined

Browse Source 
Closes #18682
This commit is contained in:
Alan Agius 2020-09-07 10:41:48 +02:00
parent 179b6caf58
commit 2586a0e9a2
2 changed files with 52 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
packages/angular_devkit/build_optimizer/src/transforms/scrub-file.ts
View File

@ -141,7 +141,7 @@ function isAngularCoreImport(node: ts.ImportDeclaration, isAngularCoreFile: bool
} }
// Relative imports from a Angular core file are also core imports. // Relative imports from a Angular core file are also core imports.
if (isAngularCoreFile && (importText.startsWith('./') || importText.startsWith('../'))) { if (isAngularCoreFile && importText.startsWith('.')) {
return true; return true;
} }
@ -571,11 +571,14 @@ function isTslibHelper(
return false; return false;
} }
for (const name of tslibImports) {
for (const dec of symbol.declarations) { for (const dec of symbol.declarations) {
if (ts.isImportSpecifier(dec) && name.elements.includes(dec)) { if (ts.isImportSpecifier(dec) && tslibImports.some(name => name.elements.includes(dec))) {
return true; return true;
} }
// Handle inline helpers `var __decorate = (this...`
if (ts.isVariableDeclaration(dec)) {
return true;
} }
} }

43
packages/angular_devkit/build_optimizer/src/transforms/scrub-file_spec.ts
View File

@ -118,6 +118,49 @@ describe('scrub-file', () => {
expect(tags.oneLine`${transform(input)}`).toEqual(tags.oneLine`${output}`); expect(tags.oneLine`${transform(input)}`).toEqual(tags.oneLine`${output}`);
}); });
it('removes Angular decorators calls when __decorate is inlined', () => {
const output = tags.stripIndent`
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
return c > 3 && r && Object.defineProperty(target, key, r), r;
};
import { Component, Injectable } from '@angular/core';
var Clazz = (function () {
function Clazz() { }
return Clazz;
}());
`;
const input = tags.stripIndent`
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
return c > 3 && r && Object.defineProperty(target, key, r), r;
};
import { Component, Injectable } from '@angular/core';
var Clazz = (function () {
function Clazz() { }
Clazz = __decorate([
Injectable(),
Component({
selector: 'app-root',
templateUrl: './app.component.html',
styleUrls: ['./app.component.css']
})
], Clazz);
return Clazz;
}());
`;
expect(testScrubFile(input)).toBeTruthy();
expect(tags.oneLine`${transform(input)}`).toEqual(tags.oneLine`${output}`);
});
it('removes constructor parameter metadata in __decorate', () => { it('removes constructor parameter metadata in __decorate', () => {
const output = tags.stripIndent` const output = tags.stripIndent`
import { __decorate, __metadata } from "tslib"; import { __decorate, __metadata } from "tslib";