Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

fix NoSuchMethodError in filter #1685

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

michilu
Copy link
Contributor

@michilu michilu commented Mar 9, 2015

I got errors when using filter for objects that is nullable.
I seem to be no substantial problem.
However, I want to suppress unwanted errors.

The null object does not have a method 'toList'.

NoSuchMethodError: method not found: 'toList'
Receiver: null
Arguments: [growable: false]

STACKTRACE:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
#1      Filter.call (package:angular/formatter/filter.dart:193:26)
...

Review on Reviewable

I got errors when using `filter` for objects that is nullable.
I seem to be no substantial problem.
However, I want to suppress unwanted errors.

```
The null object does not have a method 'toList'.

NoSuchMethodError: method not found: 'toList'
Receiver: null
Arguments: [growable: false]

STACKTRACE:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
#1      Filter.call (package:angular/formatter/filter.dart:193:26)
...
```
@rkirov
Copy link
Contributor

rkirov commented Mar 20, 2015

Can you add a test for this case?

@michilu
Copy link
Contributor Author

michilu commented Mar 22, 2015

@rkirov OK. I will try to write a test case. But I do not have any time, now. Please wait until April or later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants