firebase / flutterfire

🔥 A collection of Firebase plugins for Flutter apps.
https://firebase.google.com/docs/flutter/setup
BSD 3-Clause "New" or "Revised" License
8.46k stars 3.91k forks source link

fix(firebase_analytics): fixed type assertion that ignored nullable fields #12792

Closed jorgecoca closed 2 weeks ago

jorgecoca commented 2 weeks ago

Description

In Firebase Analytics, when passing parameters to logPurchase, the signature allows for nullable values (Map<String, Object?>?). However, down the line, there is a method called _assertParameterTypesAreCorrect that asserts value types but ignored null values, creating a runtime issue.

More details in #12790

Related Issues

Fixes #12790

Checklist

Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?