commit ba3a5202bad804b551865ef5cd6fcb6bbe3d4622
author John Ryan <ryjohn@google.com> Thu Sep 08 15:24:14 2022 -0700
committer GitHub <noreply@github.com> Thu Sep 08 22:24:14 2022 +0000
tree 1a57edc58d4ac40ac539a8e6418e3d0285f77e2f
parent ba51afa9fd90cdb2f12e582e4ce06d7248695206

[go_router] Don't clear listeners when logging is disabled (#2533)

packages/go_router/CHANGELOG.md

diff --git a/packages/go_router/CHANGELOG.md b/packages/go_router/CHANGELOG.md
index ef1978e..7ff717e 100644
--- a/packages/go_router/CHANGELOG.md
+++ b/packages/go_router/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 4.4.1
+
+- Fix an issue where disabling logging clears the root logger's listeners
+
 ## 4.4.0
 
 - Adds `buildPageWithState` to `GoRouteData`.

packages/go_router/lib/src/logging.dart

diff --git a/packages/go_router/lib/src/logging.dart b/packages/go_router/lib/src/logging.dart
index c66d075..ccbb09b 100644
--- a/packages/go_router/lib/src/logging.dart
+++ b/packages/go_router/lib/src/logging.dart
@@ -16,7 +16,6 @@
 void setLogging({bool enabled = false}) {
   _subscription?.cancel();
   if (!enabled) {
-    log.clearListeners();
     return;
   }
 

packages/go_router/pubspec.yaml

diff --git a/packages/go_router/pubspec.yaml b/packages/go_router/pubspec.yaml
index f93db6b..36f3145 100644
--- a/packages/go_router/pubspec.yaml
+++ b/packages/go_router/pubspec.yaml
@@ -1,7 +1,7 @@
 name: go_router
 description: A declarative router for Flutter based on Navigation 2 supporting
   deep linking, data-driven routes and more
-version: 4.4.0
+version: 4.4.1
 repository: https://github.com/flutter/packages/tree/main/packages/go_router
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+go_router%22
 

packages/go_router/test/logging_test.dart

diff --git a/packages/go_router/test/logging_test.dart b/packages/go_router/test/logging_test.dart
index 5273660..d6e2786 100644
--- a/packages/go_router/test/logging_test.dart
+++ b/packages/go_router/test/logging_test.dart
@@ -7,21 +7,11 @@
 import 'package:logging/logging.dart';
 
 void main() {
-  // Reset the logger before each test.
-  setUp(() {
-    setLogging();
-  });
-  test('setLogging enables log messages on the logger', () {
-    log.onRecord.listen(expectAsync1<void, LogRecord>((LogRecord r) {}));
-
+  test('setLogging does not clear listeners', () {
+    log.onRecord
+        .listen(expectAsync1<void, LogRecord>((LogRecord r) {}, count: 2));
     setLogging(enabled: true);
     log.info('message');
-  });
-
-  test('setLogging disables log messages on the logger', () {
-    log.onRecord
-        .listen(expectAsync1<void, LogRecord>((LogRecord r) {}, count: 0));
-
     setLogging();
     log.info('message');
   });