Skip to content
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

Dart transformer ChangeDetectors do not have IDs #3726

Closed
mhevery opened this issue Aug 19, 2015 · 9 comments
Closed

Dart transformer ChangeDetectors do not have IDs #3726

mhevery opened this issue Aug 19, 2015 · 9 comments

Comments

@mhevery
Copy link
Contributor

mhevery commented Aug 19, 2015

When running WTF in Dart with transformers all ChangeDetector ids are null. The issue is here the this.id is null, it seems that ids are not generated in transformer mode. This makes it hard to do debugging.

@mhevery
Copy link
Contributor Author

mhevery commented Aug 19, 2015

/cc @kegluneq This is blocking getting proper traces in WTF for GT/AWS.

@kegluneq kegluneq self-assigned this Aug 19, 2015
@kegluneq kegluneq added this to the alpha-35 milestone Aug 19, 2015
@kegluneq
Copy link

I'll take a look right away

@kegluneq
Copy link

/cc @tbosch , @vsavkin

The change detector ids are coming from ChangeDetectorDefinition, which comes from ProtoViewFactory.

Any idea why these might be coming back empty?

@kegluneq
Copy link

@mhevery can you (or someone else) post some repro steps? I haven't been able to get this to happen so far.

@mhevery mhevery assigned mhevery and unassigned kegluneq Aug 19, 2015
@mhevery
Copy link
Contributor Author

mhevery commented Aug 19, 2015

@mhevery
Copy link
Contributor Author

mhevery commented Aug 20, 2015

@kegluneq the build is failing with some unrelated error. Could you have a look as to why?
error: https://travis-ci.org/angular/angular/jobs/76371909#L1113
on liner change: 3d237ab

rerun many times the failure is consistent.

@kegluneq
Copy link

/cc @rkirov, @vicb

@mhevery , this is coming from package:di

Transform InjectorGenerator on benchmarks_external|web/src/naive_infinite_scroll/index.dart threw error: <rethrow>
Caused by The null object does not have a getter 'importsAndExports'.

Here is InjectorGenerator

That transformer tries to resolve, and the error is cropping up in analyzer. Here is the getter I'm guessing we're trying to call.

Sorry I don't have more here, but I'm flying to MD tomorrow and will be OOO until Tuesday - hopefully someone with more di knowledge can track down the issue.

mhevery added a commit that referenced this issue Aug 20, 2015
@vicb
Copy link
Contributor

vicb commented Aug 20, 2015

The failure was related to #3698 which has been reverted from master for now.

Edit: it actually was not related to the router PR, see #3757 instead

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants