Skip to content

null safety on sentry_flutter #337

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.

Sign up for GitHub

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

Merged
marandaneto merged 17 commits into from
Mar 7, 2021
Merged

null safety on sentry_flutter #337

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
13eb5bf
migrate to null safety for sentry_flutter
fzyzcjy Mar 5, 2021
697f41c
make it compile on dart2.12/flutter2.0
fzyzcjy Mar 5, 2021
5cb2538
fix a few constructors
fzyzcjy Mar 6, 2021
07fcaee
avoid WidgetsBinding.instance! as it can be null
fzyzcjy Mar 6, 2021
b7b8e23
remove `as FutureOr Function()`
fzyzcjy Mar 6, 2021
5aeebd6
fix compile error
fzyzcjy Mar 6, 2021
8550c13
remove getter/setter which are only used for null checks
fzyzcjy Mar 6, 2021
9c3ccff
fixbug some warnings
fzyzcjy Mar 6, 2021
a443a43
generate mocks using mockito code generator
fzyzcjy Mar 6, 2021
68458a8
fix bugs about "!" on null value, and return null value
fzyzcjy Mar 6, 2021
20a9323
add a "when", otherwise mockito complains no stub value to return
fzyzcjy Mar 6, 2021
ad05773
make _observer nullable, since `call` may not be called
fzyzcjy Mar 6, 2021
10247d1
add changelog
fzyzcjy Mar 6, 2021
85b76b0
execute `flutter format .` on sentry-flutter
fzyzcjy Mar 6, 2021
fb2913a
fix analyze problems
fzyzcjy Mar 6, 2021
adb9c4a
change mockito version
fzyzcjy Mar 6, 2021
e3d8051
change signature of ensureInitialized
fzyzcjy Mar 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
make _observer nullable, since call may not be called
  • Loading branch information
@fzyzcjy
fzyzcjy committed Mar 6, 2021
commit ad057737fb9bdc30b18561d51797ea1a24865f17
8 changes: 4 additions & 4 deletions flutter/lib/src/default_integrations.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,7 @@ class NativeSdkIntegration extends Integration<SentryFlutterOptions> {
/// - [SentryWidgetsBindingObserver]
/// - [WidgetsBindingObserver](https://api.flutter.dev/flutter/widgets/WidgetsBindingObserver-class.html)
class WidgetsBindingIntegration extends Integration<SentryFlutterOptions> {
late SentryWidgetsBindingObserver _observer;
SentryWidgetsBindingObserver? _observer;

@override
FutureOr<void> call(Hub hub, SentryFlutterOptions options) {
Expand All @@ -215,7 +215,7 @@ class WidgetsBindingIntegration extends Integration<SentryFlutterOptions> {
// If the instance is not created, we skip it to keep going.
final instance = WidgetsBinding.instance;
if (instance != null) {
instance.addObserver(_observer);
instance.addObserver(_observer!);
options.sdk.addIntegration('widgetsBindingIntegration');
} else {
options.logger(
Expand All @@ -228,8 +228,8 @@ class WidgetsBindingIntegration extends Integration<SentryFlutterOptions> {
@override
void close() {
final instance = WidgetsBinding.instance;
if (instance != null) {
instance.removeObserver(_observer);
if (instance != null && _observer != null) {
instance.removeObserver(_observer!);
}
}
}
Expand Down