call reload before checking is user account is validates

FirebaseAuth.instance.currentUser() returns cached data, so reload() needs to be called if you want updated data
student-hub · Mar 19, 2020 · 508a8fd · 508a8fd
1 parent 7d49a3b
commit 508a8fd
Show file tree

Hide file tree

Showing 8 changed files with 34 additions and 64 deletions.
diff --git a/lib/authentication/service/auth_provider.dart b/lib/authentication/service/auth_provider.dart
@@ -102,6 +102,7 @@ class AuthProvider with ChangeNotifier {
 
   /// Check the network to see if there is a user authenticated
   Future<bool> get isVerifiedFromService async {
+    firebaseUser.reload();
     firebaseUser = await FirebaseAuth.instance.currentUser();
     return firebaseUser.isEmailVerified;
   }

diff --git a/lib/generated/intl/messages_en.dart b/lib/generated/intl/messages_en.dart
@@ -74,7 +74,6 @@ class MessageLookup extends MessageLookupByLibrary {
     "messageEmailNotVerified" : MessageLookupByLibrary.simpleMessage("Account is not verified."),
     "messageNewUser" : MessageLookupByLibrary.simpleMessage("New user?"),
     "messageResetPassword" : MessageLookupByLibrary.simpleMessage("Enter your e-mai in order to receive instructions on how to reset your password."),
-    "messageSignInAfterVerification" : MessageLookupByLibrary.simpleMessage("You may need to sign in again after verification."),
     "messageUnderConstruction" : MessageLookupByLibrary.simpleMessage("Under construction"),
     "messageWelcomeName" : m2,
     "messageWelcomeSimple" : MessageLookupByLibrary.simpleMessage("Welcome!"),

diff --git a/lib/generated/intl/messages_ro.dart b/lib/generated/intl/messages_ro.dart
@@ -74,7 +74,6 @@ class MessageLookup extends MessageLookupByLibrary {
     "messageEmailNotVerified" : MessageLookupByLibrary.simpleMessage("Contul nu este verificat."),
     "messageNewUser" : MessageLookupByLibrary.simpleMessage("Utilizator nou?"),
     "messageResetPassword" : MessageLookupByLibrary.simpleMessage("Introduceți mail-ul pentru a primi instrucțiuni de resetare a parolei."),
-    "messageSignInAfterVerification" : MessageLookupByLibrary.simpleMessage("Va trebui să vă conectați din nou după verificare."),
     "messageUnderConstruction" : MessageLookupByLibrary.simpleMessage("În construcție"),
     "messageWelcomeName" : m2,
     "messageWelcomeSimple" : MessageLookupByLibrary.simpleMessage("Bine ai venit!"),

diff --git a/lib/generated/l10n.dart b/lib/generated/l10n.dart
diff --git a/lib/l10n/intl_en.arb b/lib/l10n/intl_en.arb
@@ -75,7 +75,6 @@
   "messageEmailNotVerified": "Account is not verified.",
   "messageAccountCreated": "Account created successfully.",
   "messageCheckEmailVerification": "Please check your email for account verification.",
-  "messageSignInAfterVerification": "You may need to sign in again after verification.",
   "messageResetPassword": "Enter your e-mai in order to receive instructions on how to reset your password.",
   "messageUnderConstruction": "Under construction",
 

diff --git a/lib/l10n/intl_ro.arb b/lib/l10n/intl_ro.arb
@@ -75,7 +75,6 @@
   "messageEmailNotVerified": "Contul nu este verificat.",
   "messageAccountCreated": "Contul a fost creat cu succes.",
   "messageCheckEmailVerification": "Verificați-vă mail-ul pentru confirmarea contului.",
-  "messageSignInAfterVerification": "Va trebui să vă conectați din nou după verificare.",
   "messageResetPassword": "Introduceți mail-ul pentru a primi instrucțiuni de resetare a parolei.",
   "messageUnderConstruction": "În construcție",
 

diff --git a/lib/pages/profile/profile_page.dart b/lib/pages/profile/profile_page.dart
@@ -2,7 +2,6 @@ import 'package:acs_upb_mobile/authentication/service/auth_provider.dart';
 import 'package:acs_upb_mobile/generated/l10n.dart';
 import 'package:acs_upb_mobile/navigation/routes.dart';
 import 'package:acs_upb_mobile/widgets/scaffold.dart';
-import 'package:auto_size_text/auto_size_text.dart';
 import 'package:flutter/material.dart';
 import 'package:provider/provider.dart';
 
@@ -24,56 +23,39 @@ class ProfilePage extends StatelessWidget {
         }
         return Padding(
           padding: const EdgeInsets.all(8.0),
-          child: Column(
-            children: <Widget>[
-              ConstrainedBox(
-                constraints:
-                    BoxConstraints(maxWidth: MediaQuery.of(context).size.width),
-                child: Row(
-                  mainAxisAlignment: MainAxisAlignment.center,
-                  children: <Widget>[
-                    Icon(
-                      Icons.error_outline,
-                      size: Theme.of(context).textTheme.subtitle2.fontSize,
-                    ),
-                    SizedBox(width: 4),
-                    Text(
-                      S.of(context).messageEmailNotVerified,
-                      style: Theme.of(context)
-                          .textTheme
-                          .subtitle2
-                          .copyWith(fontWeight: FontWeight.w400),
-                      maxLines: 1,
-                    ),
-                    InkWell(
-                      onTap: () {
-                        authProvider.sendEmailVerification(context: context);
-                      },
-                      child: Text(
-                        ' ' + S.of(context).actionSendVerificationAgain,
-                        style: Theme.of(context)
-                            .accentTextTheme
-                            .subtitle2
-                            .copyWith(fontWeight: FontWeight.w500),
-                      ),
-                    ),
-                  ],
+          child: ConstrainedBox(
+            constraints:
+                BoxConstraints(maxWidth: MediaQuery.of(context).size.width),
+            child: Row(
+              mainAxisAlignment: MainAxisAlignment.center,
+              children: <Widget>[
+                Icon(
+                  Icons.error_outline,
+                  size: Theme.of(context).textTheme.subtitle2.fontSize,
                 ),
-              ),
-              SizedBox(height: 4),
-              AutoSizeText(
-                S.of(context).messageSignInAfterVerification,
-                style: Theme.of(context)
-                    .textTheme
-                    .subtitle2
-                    .copyWith(fontWeight: FontWeight.w400)
-                    .apply(
-                        fontSizeDelta: -2, color: Theme.of(context).hintColor),
-                maxLines: 1,
-                overflow: TextOverflow.visible,
-                minFontSize: 0,
-              )
-            ],
+                SizedBox(width: 4),
+                Text(
+                  S.of(context).messageEmailNotVerified,
+                  style: Theme.of(context)
+                      .textTheme
+                      .subtitle2
+                      .copyWith(fontWeight: FontWeight.w400),
+                  maxLines: 1,
+                ),
+                InkWell(
+                  onTap: () {
+                    authProvider.sendEmailVerification(context: context);
+                  },
+                  child: Text(
+                    ' ' + S.of(context).actionSendVerificationAgain,
+                    style: Theme.of(context)
+                        .accentTextTheme
+                        .subtitle2
+                        .copyWith(fontWeight: FontWeight.w500),
+                  ),
+                ),
+              ],
+            ),
           ),
         );
       },

diff --git a/pubspec.yaml b/pubspec.yaml
@@ -11,7 +11,7 @@ description: A mobile application for students at ACS UPB.
 # In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
 # Read more about iOS versioning at
 # https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
-version: 0.3.13+1
+version: 0.3.14+1
 
 environment:
   sdk: ">=2.6.0 <3.0.0"