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

[cloud_firestore] Remove usage of invokeMapMethod #1435

Merged
merged 4 commits into from
Apr 3, 2019

Conversation

cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Apr 3, 2019

invokeMapMethod and invokeListMethod currently crashes if the return value from methodChannel is null. Remove usages of these 2 methods to prevent crashes.

@@ -1,3 +1,7 @@
## 0.9.8+1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be 0.9.9 rather than 0.9.8+1 since a behavior change is evident

@collinjackson
Copy link
Contributor

lgtm once bots are happy

@cyanglaz cyanglaz merged commit d8a9bd6 into flutter:master Apr 3, 2019
@cyanglaz cyanglaz deleted the cloud_firestore branch April 3, 2019 00:57
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
`invokeMapMethod` and `invokeListMethod`  currently crashes if the return value from methodChannel is null. Remove usages of these 2 methods to prevent crashes.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants