fix: common bug

lib/src/providers/auth_provider.dart

@@ -19,10 +19,10 @@ class AuthProvider with ChangeNotifier {
   bool filterByUpcoming = false;
   bool filterByOverdued = false;
 
-  int? userIdResetPw;
   bool isLoading = false;
   bool resetPasswordTokenSended = false;
   bool resetPasswordSucced = false;
+  bool loanBookSuccess = false;
 
   List<dynamic>? loans;
   List<dynamic>? nearOutstandingLoans;
@@ -38,6 +38,7 @@ class AuthProvider with ChangeNotifier {
 
   void setLoading(bool value) {
     isLoading = value;
+    notifyListeners();
   }
 
   Future<void> signIn(String username, String password) async {
@@ -66,7 +67,7 @@ class AuthProvider with ChangeNotifier {
         debugPrint("Login failed $code");
       }
 
-      setLoading(false);
+      setLoading(true);
       notifyListeners();
     } catch (error) {
       debugPrint("Login failed $error");
@@ -90,6 +91,17 @@ class AuthProvider with ChangeNotifier {
         await storage.delete(key: 'token');
         isAuthenticated = false;
         user = null;
+        filterByUpcoming = false;
+        filterByOverdued = false;
+        memberLoans = null;
+
+        isLoading = false;
+        resetPasswordTokenSended = false;
+        resetPasswordSucced = false;
+
+        loans = null;
+        nearOutstandingLoans = null;
+        overduedLoans = null;
       } else {
         debugPrint("Logout failed: ${response.statusCode} ${response.body}");
       }
@@ -102,9 +114,8 @@ class AuthProvider with ChangeNotifier {
   }
 
   Future<void> signUp(String username, String email, String password) async {
-    isLoading = true;
-
     try {
+      setLoading(true);
       final body = {
         "username": username,
         "email": email,
@@ -128,7 +139,7 @@ class AuthProvider with ChangeNotifier {
             "Error: sign up failed, ${response.statusCode}: ${response.body}");
       }
 
-      isLoading = false;
+      setLoading(false);
       notifyListeners();
     } catch (error) {
       debugPrint("Error: sign up failed, $error");
@@ -136,11 +147,11 @@ class AuthProvider with ChangeNotifier {
   }
 
   Future<void> getUserDetail() async {
-    isLoading = true;
     final token = await getAccessToken();
 
     if (token != null) {
       try {
+        setLoading(true);
         final response = await http.get(
           Uri.parse('$baseUrl/user'),
           headers: {
@@ -156,7 +167,7 @@ class AuthProvider with ChangeNotifier {
           debugPrint('Error fetching user details: ${response.statusCode}');
         }
 
-        isLoading = true;
+        setLoading(false);
         notifyListeners();
       } catch (error) {
         debugPrint('Error user details: $error');
@@ -172,7 +183,7 @@ class AuthProvider with ChangeNotifier {
     String? lastName,
     bool isStaff,
   ) async {
-    isLoading = true;
+    setLoading(true);
     final token = await getAccessToken();
 
     if (token != null) {
@@ -200,7 +211,7 @@ class AuthProvider with ChangeNotifier {
               'Error update user details: ${response.statusCode}, ${response.body}');
         }
 
-        isLoading = false;
+        setLoading(false);
         notifyListeners();
       } catch (error) {
         debugPrint("Error update user details: $error");
@@ -209,9 +220,8 @@ class AuthProvider with ChangeNotifier {
   }
 
   Future<void> resetPassword(String email) async {
-    isLoading = true;
-
     try {
+      setLoading(true);
       final response = await http.post(
         Uri.parse('$baseUrl/reset-password/request-token'),
         body: jsonEncode({"email": email}),
@@ -219,15 +229,13 @@ class AuthProvider with ChangeNotifier {
       );
 
       if (response.statusCode == 200) {
-        final data = jsonDecode(response.body);
         resetPasswordTokenSended = true;
-        userIdResetPw = data["user_id_reset_pw"];
       } else {
         debugPrint(
             'Error reset user password: ${response.statusCode}, ${response.body}');
       }
 
-      isLoading = false;
+      setLoading(false);
       notifyListeners();
     } catch (error) {
       debugPrint("Error reset user password: $error");
@@ -236,7 +244,7 @@ class AuthProvider with ChangeNotifier {
 
   Future<void> confirmResetPassword(
       int pin, String password1, String password2) async {
-    isLoading = true;
+    setLoading(true);
     final body = jsonEncode({
       "pin": pin,
       "password1": password1,
@@ -257,7 +265,7 @@ class AuthProvider with ChangeNotifier {
             'Error confirm reset user password: ${response.statusCode}, ${response.body}');
       }
 
-      isLoading = false;
+      setLoading(false);
       notifyListeners();
     } catch (error) {
       debugPrint("Error confirm reset user password: $error");
@@ -265,7 +273,7 @@ class AuthProvider with ChangeNotifier {
   }
 
   Future<void> getMemberLoan() async {
-    isLoading = true;
+    setLoading(true);
     final token = await getAccessToken();
 
     String url = '$baseUrl/members/${user?.accountId}/loans/';
@@ -294,7 +302,7 @@ class AuthProvider with ChangeNotifier {
             "Failed to get member loan. ${response.statusCode}: ${response.body}");
       }
 
-      isLoading = false;
+      setLoading(false);
       notifyListeners();
     } catch (error) {
       debugPrint("Failed to get member loan. $error");
@@ -302,21 +310,20 @@ class AuthProvider with ChangeNotifier {
   }
 
   void setFilterUpcoming() {
-    isLoading = true;
+    setLoading(true);
     filterByUpcoming = !filterByUpcoming;
-    isLoading = false;
+    setLoading(false);
     notifyListeners();
   }
 
   void setFilterOverdued() {
-    isLoading = true;
+    setLoading(true);
     filterByOverdued = !filterByOverdued;
+    setLoading(false);
     notifyListeners();
-    isLoading = false;
   }
 
   Future<void> createMemberLoan(int memberId, int bookId, int loanDay) async {
-    isLoading = true;
     final token = await getAccessToken();
 
     final now = DateTime.now();
@@ -329,6 +336,7 @@ class AuthProvider with ChangeNotifier {
     };
 
     try {
+      setLoading(true);
       final response = await http.post(
         Uri.parse('$baseUrl/members/$memberId/loans/'),
         body: jsonEncode(body),
@@ -346,7 +354,8 @@ class AuthProvider with ChangeNotifier {
             "Failed to create member loan. ${response.statusCode}: ${response.body}");
       }
 
-      isLoading = false;
+      loanBookSuccess = true;
+      setLoading(false);
       notifyListeners();
     } catch (error) {
       debugPrint("Failed to create member loan. $error");
@@ -355,7 +364,6 @@ class AuthProvider with ChangeNotifier {
 
   // for admin or librarian
   Future<void> getLoans(String? type) async {
-    isLoading = true;
     final token = await storage.read(key: 'access_token');
     String url = baseUrl;
     if (type == "upcoming") {
@@ -368,6 +376,7 @@ class AuthProvider with ChangeNotifier {
 
     if (token != null) {
       try {
+        setLoading(true);
         final response = await http.get(
           Uri.parse(url),
           headers: {
@@ -390,7 +399,7 @@ class AuthProvider with ChangeNotifier {
           debugPrint("Error: Fetch upcoming loans failed, $code");
         }
 
-        isLoading = false;
+        setLoading(false);
         notifyListeners();
       } catch (error) {
         debugPrint("Error: Fetch upcoming loans failed, $error");

lib/src/widgets/forms/book_loan_form.dart

 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:library_app/src/providers/auth_provider.dart';
+import 'package:library_app/src/screens/list/list_screen.dart';
 import 'package:provider/provider.dart';
 
 class LoanBookForm extends StatefulWidget {
@@ -24,6 +25,20 @@ class _LoanBookForm extends State<LoanBookForm> {
   @override
   Widget build(BuildContext context) {
     return Consumer<AuthProvider>(builder: (context, authProvider, child) {
+      if (authProvider.loanBookSuccess) {
+        return AlertDialog(
+          title: const Text(
+            'Loan book succed!',
+            style: TextStyle(fontSize: 20.0),
+          ),
+          content: FilledButton(
+            onPressed: () {
+              Navigator.of(context).pop();
+            },
+            child: const Text('Continue'),
+          ),
+        );
+      }
       return Container(
         padding: const EdgeInsets.symmetric(horizontal: 20.0, vertical: 10.0),
         child: FilledButton(

lib/src/widgets/forms/reset_password_form.dart

@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter_svg/flutter_svg.dart';
 import 'package:library_app/src/providers/auth_provider.dart';
 import 'package:library_app/src/screens/form_screen.dart';
+import 'package:library_app/src/widgets/loading.dart';
 import 'package:provider/provider.dart';
 
 class ResetPasswordForm extends StatefulWidget {
@@ -101,7 +102,9 @@ class _ResetPasswordForm extends State<ResetPasswordForm> {
                                   },
                                 );
                               },
-                              child: const Text("Submit"),
+                              child: authProvider.isLoading
+                                  ? const Loading()
+                                  : const Text("Submit"),
                             ),
                           ),
                         ],
@@ -139,6 +142,12 @@ class _ConfirmResetPasswordForm extends State<ConfirmResetPasswordForm> {
     const String formText = "Enter the pin that we just sent to your email";
 
     return Consumer<AuthProvider>(builder: (context, authProvider, child) {
+      if (authProvider.isLoading) {
+        return const Center(
+          child: CircularProgressIndicator(),
+        );
+      }
+
       return Column(
         children: [
           Container(

lib/src/widgets/loans/admin_loan_list.dart

@@ -26,8 +26,13 @@ class _AdminLoanList extends State<AdminLoanList> {
 
   @override
   void initState() {
-    super.initState();
+    Future.delayed(
+      Duration.zero,
+      () {
         Provider.of<AuthProvider>(context, listen: false).getLoans(type);
+      },
+    );
+    super.initState();
   }
 
   @override

lib/src/widgets/loans/loan_list.dart

@@ -17,8 +17,13 @@ class LoanList extends StatefulWidget {
 class _LoanList extends State<LoanList> {
   @override
   void initState() {
-    super.initState();
+    Future.delayed(
+      Duration.zero,
+      () {
         Provider.of<AuthProvider>(context, listen: false).getMemberLoan();
+      },
+    );
+    super.initState();
   }
 
   @override

lib/src/widgets/profile.dart

@@ -18,7 +18,11 @@ class _Profile extends State<Profile> {
     super.initState();
     Future.delayed(
       Duration.zero,
-      () => Provider.of<AuthProvider>(context, listen: false).getUserDetail(),
+      () {
+        if (Provider.of<AuthProvider>(context, listen: false).user == null) {
+          Provider.of<AuthProvider>(context, listen: false).getUserDetail();
+        }
+      },
     );
   }
 
@@ -26,13 +30,16 @@ class _Profile extends State<Profile> {
   Widget build(BuildContext context) {
     return Consumer<AuthProvider>(
       builder: (context, authProvider, child) {
+        final isLoading = authProvider.isLoading;
         final user = authProvider.user;
         final firstName = user?.firstName ?? "";
         final lastName = user?.lastName ?? "";
         final fullName = "$firstName $lastName";
 
         if (user == null) {
-          return const Center(child: CircularProgressIndicator());
+          return const Center(child: Text("Failed to fetch user data"));
+        } else if (isLoading) {
+          return const Center(child: CircularProgressIndicator.adaptive());
         } else {
           return NestedScrollView(
             headerSliverBuilder: