fix(common): AsyncPipe should not throw when given observable of NaN#15723
fix(common): AsyncPipe should not throw when given observable of NaN#15723DzmitryShylovich wants to merge 1 commit intoangular:masterfrom
Conversation
| } | ||
|
|
||
| if (this._latestValue === this._latestReturnedValue) { | ||
| if (looseIdentical(this._latestValue, this._latestReturnedValue)) { |
There was a problem hiding this comment.
NaN !== NaN so AsyncPipe will return WrappedValue instead of NaN which results in expression has changed error.
|
@DzmitryShylovich what is the status of your pull request? |
|
This will not fix the issue, the test case passed only because it('should return NaN when given NaN and the latest returned value is NaN', () => {
const subject = new BehaviorSubject<any>(null);
pipe.transform(subject);
subject.next(NaN);
expect(pipe.transform(subject)).toBeNaN();
expect(pipe.transform(subject)).toBeNaN();
});And it will fail. |
|
You guys could retake this PR since Dzmitry was banned long ago. |
|
@ericmartinezr Just linked my PR here. |
|
@ericmartinezr Dzmitry is banned long ago, why? I do not know the history, what happened? |
|
@splincode Just a quick search is enough.
And you can then check any history message before that date. |
|
@trotyl Do you mean that they just blocked it? But for what? So you need to close all its PR? |
It's Angular team's job, nothing to do with me... |
|
@robwormald Could you close the PR's? Because the PR data will no longer be supported by anyone. About 10 open requests:
|
|
closing in favor of #22305 |
|
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 #15721