dispose AnimationController before super.dispose()

Author: hoc081098

lib/pages/home/change_password/change_password_bottomsheet.dart

@@ -1,9 +1,9 @@
-import 'dart:async';
-
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc_pattern/flutter_bloc_pattern.dart';
+import 'package:flutter_disposebag/flutter_disposebag.dart';
 import 'package:node_auth/pages/home/change_password/change_password.dart';
 import 'package:node_auth/widgets/password_textfield.dart';
+import 'package:rxdart_ext/rxdart_ext.dart';
 
 class ChangePasswordBottomSheet extends StatefulWidget {
   const ChangePasswordBottomSheet({Key key}) : super(key: key);
@@ -14,11 +14,13 @@ class ChangePasswordBottomSheet extends StatefulWidget {
 }
 
 class _ChangePasswordBottomSheetState extends State<ChangePasswordBottomSheet>
-    with SingleTickerProviderStateMixin<ChangePasswordBottomSheet> {
+    with
+        SingleTickerProviderStateMixin<ChangePasswordBottomSheet>,
+        DisposeBagMixin {
   AnimationController fadeMessageController;
   Animation<double> messageOpacity;
+  Object listen;
 
-  StreamSubscription<void> subscription;
   final newPasswordFocusNode = FocusNode();
 
   @override
@@ -44,28 +46,28 @@ class _ChangePasswordBottomSheetState extends State<ChangePasswordBottomSheet>
   void didChangeDependencies() {
     super.didChangeDependencies();
 
-    subscription ??= BlocProvider.of<ChangePasswordBloc>(context)
+    listen ??= BlocProvider.of<ChangePasswordBloc>(context)
         .changePasswordState$
-        .listen((state) async {
-      if (state.message != null) {
-        final navigatorState = Navigator.of(context);
-
-        fadeMessageController.reset();
-        await fadeMessageController.forward();
-
-        if (state?.error == null) {
-          navigatorState.pop();
-        }
-      }
-    });
+        .flatMap((state) async* {
+          if (state.message != null) {
+            fadeMessageController.reset();
+            await fadeMessageController.forward();
+            yield null;
+
+            if (state?.error == null) {
+              Navigator.of(context).pop();
+            }
+          }
+        })
+        .collect()
+        .disposedBy(bag);
   }
 
   @override
   void dispose() {
-    super.dispose();
-    subscription.cancel();
     fadeMessageController.dispose();
     newPasswordFocusNode.dispose();
+    super.dispose();
   }
 
   @override

lib/pages/login/login_page.dart

@@ -72,9 +72,9 @@ class _MyLoginPageState extends State<LoginPage>
 
   @override
   void dispose() {
-    super.dispose();
     passwordFocusNode.dispose();
     loginButtonController.dispose();
+    super.dispose();
   }
 
   @override

lib/pages/register/register_page.dart

@@ -67,10 +67,10 @@ class _RegisterPageState extends State<RegisterPage>
 
   @override
   void dispose() {
-    super.dispose();
     registerButtonController.dispose();
     emailFocusNode.dispose();
     passwordFocusNode.dispose();
+    super.dispose();
   }
 
   @override

lib/pages/reset_password/input_token/input_token_and_reset_password_page.dart

@@ -79,10 +79,10 @@ class _InputTokenAndResetPasswordPageState
 
   @override
   void dispose() {
-    super.dispose();
     fadeController.dispose();
     tokenFocusNode.dispose();
     passwordFocusNode.dispose();
+    super.dispose();
   }
 
   @override

lib/pages/reset_password/reset_password_page.dart

@@ -91,8 +91,8 @@ class _ResetPasswordPageState extends State<ResetPasswordPage>
 
   @override
   void dispose() {
-    super.dispose();
     animationController.dispose();
+    super.dispose();
   }
 
   void onToggle() => requestEmailS.add(null);

lib/pages/reset_password/send_email/send_email_page.dart

@@ -58,8 +58,8 @@ class _SendEmailPageState extends State<SendEmailPage>
 
   @override
   void dispose() {
-    super.dispose();
     fadeController.dispose();
+    super.dispose();
   }
 
   @override