New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(linker): minified umd fixes #41747
fix(linker): minified umd fixes #41747
Conversation
packages/compiler-cli/linker/src/ast/typescript/typescript_ast_host.ts
Outdated
Show resolved
Hide resolved
@@ -139,6 +139,8 @@ function getCalleeName(call: NodePath<t.CallExpression>): string|null { | |||
return callee.name; | |||
} else if (t.isMemberExpression(callee) && t.isIdentifier(callee.property)) { | |||
return callee.property.name; | |||
} else if (t.isMemberExpression(callee) && t.isStringLiteral(callee.property)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also support this case in AstHost.getSymbolName
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This case was not reported as problematic in the original issue, so I was being conservative in adding support. I would rather wait to see if we get any further issues. It is an easy fix to add in the future if it comes up.
Some partial libraries have been minified, which results in boolean literals being converted to `!0` and `!1`. This commit ensures that the linker can process these values. Fixes angular#41655
Some partial libraries have been minified, which results in the declaration calls being being converted from property accesses to indexed accesses. This commit ensures that the linker can process these calls. Fixes angular#41655
264b31d
to
f20d79f
Compare
f20d79f
to
3edd153
Compare
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Closes #41655