Merge pull request 'forgot password screns logic fixed' (#151) from fixed_forgot_password into main

Reviewed-on: https://repo.ravanertebat.com/TM/tm_app/pulls/151
This commit is contained in:
a.ghabeli
2025-09-22 14:26:36 +03:30
13 changed files with 209 additions and 133 deletions
+1
View File
@@ -44,6 +44,7 @@ class AuthService {
if (result is Ok<LoginResponseModel>) {
final prefs = await SharedPreferences.getInstance();
final token = result.value.data.accessToken;
AppLogger().info('🔑 Bearer token: $token');
final refreshToken = result.value.data.refreshToken;
await prefs.setString('bearer', token!);
+51 -9
View File
@@ -1,5 +1,3 @@
import 'dart:developer';
import 'package:flutter/material.dart';
import 'package:tm_app/data/services/model/customer/customer.dart';
import 'package:tm_app/data/services/model/forgot_password_response/forgot_password_response_model.dart';
@@ -49,13 +47,16 @@ class AuthViewModel with ChangeNotifier {
bool _isResetPasswordLoading = false;
String? _errorMessage;
String? _resetPasswordErrorMessage;
bool _resetPasswordSuccess = false;
Customer? _loggedInUser;
String? successMessage;
String? _successMessageOtp;
String? _forgotPasswordMessage;
bool _isLoadingForgot = false;
bool _isLoadingOtp = false;
String? _errorMessageForget;
String? _errorMessageOtp;
bool _verifyOtpSuccess = false;
bool get isLoading => _isLoading;
bool get isLoadingForgot => _isLoadingForgot;
@@ -65,10 +66,12 @@ class AuthViewModel with ChangeNotifier {
String? get errorMessageForget => _errorMessageForget;
String? get resetPasswordErrorMessage => _resetPasswordErrorMessage;
String? get errorMessageOtp => _errorMessageOtp;
String? get successMessageOtp => _successMessageOtp;
bool get verifyOtpSuccess => _verifyOtpSuccess;
Customer? get loggedInUser => _loggedInUser;
String? get forgotPasswordMessage => _forgotPasswordMessage;
String? resetToken;
bool get resetPasswordSuccess => _resetPasswordSuccess;
void togglePasswordVisibility() {
_obscurePassword = !_obscurePassword;
@@ -88,7 +91,7 @@ class AuthViewModel with ChangeNotifier {
bool get isOtpValid {
final otp = otpController.text.trim();
return otp.length == 5;
return otp.length == 6;
}
bool get canSubmitOtpPage {
@@ -182,8 +185,8 @@ class AuthViewModel with ChangeNotifier {
case Ok<ForgotPasswordResponseModel>():
_isLoadingForgot = false;
successMessage = result.value.message;
notifyListeners();
log('$successMessage');
break;
@@ -208,6 +211,7 @@ class AuthViewModel with ChangeNotifier {
Future<void> verifyOtp(BuildContext context) async {
_isLoadingOtp = true;
_errorMessageOtp = null;
_verifyOtpSuccess = false;
notifyListeners();
final result = await _authRepository.verifyOtp(
@@ -219,40 +223,78 @@ class AuthViewModel with ChangeNotifier {
case Ok<VerifyOtpResponseModel>():
_isLoadingOtp = false;
resetToken = result.value.data.token;
_successMessageOtp = result.value.message;
_verifyOtpSuccess = true;
notifyListeners();
break;
case Error<VerifyOtpResponseModel>():
_isLoadingOtp = false;
_errorMessageOtp = result.error.toString();
_verifyOtpSuccess = false;
notifyListeners();
break;
}
notifyListeners();
_errorMessageOtp = null;
_successMessageOtp = null;
notifyListeners();
}
void clearOtpFlow() {
_errorMessageOtp = null;
_successMessageOtp = null;
_isLoadingOtp = false;
_verifyOtpSuccess = false;
// Intentionally do not clear resetToken
otpController.clear();
notifyListeners();
}
Future<void> resetPassword() async {
_isResetPasswordLoading = true;
_resetPasswordErrorMessage = null;
_resetPasswordSuccess = false;
notifyListeners();
if (resetToken != null) {
final result = await _authRepository.resetPassword(
newPassword: newPasswordController.text.trim(),
token: '',
token: resetToken!,
);
switch (result) {
case Ok<ResetPasswordResponse>():
_resetPasswordSuccess = true;
break;
case Error<ResetPasswordResponse>():
_resetPasswordErrorMessage = result.error.toString();
_resetPasswordSuccess = false;
break;
}
_isResetPasswordLoading = false;
notifyListeners();
_resetPasswordErrorMessage = null;
notifyListeners();
} else {
_resetPasswordErrorMessage = 'Reset token is required';
_isResetPasswordLoading = false;
notifyListeners();
}
}
void clearResetPasswordSuccess() {
_resetPasswordSuccess = false;
notifyListeners();
}
void clearResetFlow() {
_resetPasswordErrorMessage = null;
_resetPasswordSuccess = false;
_isResetPasswordLoading = false;
// Intentionally do not clear resetToken
newPasswordController.clear();
notifyListeners();
}
@override
@@ -6,6 +6,7 @@ import 'package:tm_app/core/utils/size_config.dart';
import 'package:tm_app/views/forget_password_create/strings/forgot_password_create_strings.dart';
import '../../../core/constants/assets.dart';
import '../../../core/routes/app_routes.dart';
import '../../../core/theme/colors.dart';
import '../../../core/utils/app_toast.dart';
import '../../../view_models/auth_view_model.dart';
@@ -40,6 +41,10 @@ class _DesktopForgotPasswordCreatePageState
void _viewModelListener() {
if (viewModel.resetPasswordErrorMessage != null) {
AppToast.error(context, viewModel.resetPasswordErrorMessage!);
} else if (viewModel.isResetPasswordLoading == false &&
viewModel.resetPasswordSuccess) {
Router.neglect(context, () => const LoginScreenRoute().go(context));
viewModel.clearResetFlow();
}
}
@@ -146,10 +151,11 @@ class _DesktopForgotPasswordCreatePageState
),
SizedBox(height: 80.0.h()),
viewModel.isResetPasswordLoading
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isNewPasswordValid,
text: ForgotPasswordCreateStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isNewPasswordValid,
@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
import 'package:flutter_svg/svg.dart';
import 'package:go_router/go_router.dart';
import 'package:provider/provider.dart';
import 'package:tm_app/core/routes/app_routes.dart';
import 'package:tm_app/core/utils/size_config.dart';
import 'package:tm_app/views/forget_password_create/strings/forgot_password_create_strings.dart';
@@ -40,6 +41,10 @@ class _MobileForgotPasswordCreatePageState
void _viewModelListener() {
if (viewModel.resetPasswordErrorMessage != null) {
AppToast.error(context, viewModel.resetPasswordErrorMessage!);
} else if (viewModel.isResetPasswordLoading == false &&
viewModel.resetPasswordSuccess) {
Router.neglect(context, () => const LoginScreenRoute().go(context));
viewModel.clearResetFlow();
}
}
@@ -143,10 +148,11 @@ class _MobileForgotPasswordCreatePageState
),
SizedBox(height: 80.0.h()),
viewModel.isResetPasswordLoading
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isNewPasswordValid,
text: ForgotPasswordCreateStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isNewPasswordValid,
@@ -6,6 +6,7 @@ import 'package:tm_app/core/utils/size_config.dart';
import 'package:tm_app/views/forget_password_create/strings/forgot_password_create_strings.dart';
import '../../../core/constants/assets.dart';
import '../../../core/routes/app_routes.dart';
import '../../../core/theme/colors.dart';
import '../../../core/utils/app_toast.dart';
import '../../../view_models/auth_view_model.dart';
@@ -40,6 +41,10 @@ class _TabletForgotPasswordCreatePageState
void _viewModelListener() {
if (viewModel.resetPasswordErrorMessage != null) {
AppToast.error(context, viewModel.resetPasswordErrorMessage!);
} else if (viewModel.isResetPasswordLoading == false &&
viewModel.resetPasswordSuccess) {
Router.neglect(context, () => const LoginScreenRoute().go(context));
viewModel.clearResetFlow();
}
}
@@ -146,14 +151,16 @@ class _TabletForgotPasswordCreatePageState
),
SizedBox(height: 80.0.h()),
viewModel.isResetPasswordLoading
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isNewPasswordValid,
text: ForgotPasswordCreateStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isNewPasswordValid,
text: ForgotPasswordCreateStrings.resetLink,
onPressed:
viewModel.isNewPasswordValid
? () async {
@@ -6,12 +6,12 @@ import 'package:provider/provider.dart';
import 'package:tm_app/core/utils/size_config.dart';
import '../../../core/constants/assets.dart';
import '../../../core/routes/app_routes.dart';
import '../../../core/theme/colors.dart';
import '../../../core/utils/app_toast.dart';
import '../../../view_models/auth_view_model.dart';
import '../../shared/base_button.dart';
import '../strings/forgot_password_otp_strings.dart';
import '../../forget_password_create/pages/forgot_password_create_screen.dart';
class DesktopForgotPasswordOtpPage extends StatefulWidget {
const DesktopForgotPasswordOtpPage({super.key});
@@ -35,16 +35,14 @@ class _DesktopForgotPasswordOtpPageState
void _viewModelListener() {
if (viewModel.errorMessageOtp != null) {
AppToast.error(context, viewModel.errorMessageOtp!);
}
if (viewModel.resetToken != null) {
} else if (!viewModel.isLoadingOtp &&
viewModel.verifyOtpSuccess &&
viewModel.resetToken != null) {
Router.neglect(
context,
() => Navigator.push(
context,
MaterialPageRoute(builder: (_) => const ForgotPasswordCreateScreen()),
),
() => const ForgotPasswordCreateRouteData().pushReplacement(context),
);
viewModel.clearOtpFlow();
}
}
@@ -99,7 +97,7 @@ class _DesktopForgotPasswordOtpPageState
SizedBox(height: 40.0.h()),
Pinput(
controller: viewModel.otpController,
length: 5,
length: 6,
pinAnimationType: PinAnimationType.scale,
defaultPinTheme: PinTheme(
width: 50.0.w(),
@@ -117,15 +115,17 @@ class _DesktopForgotPasswordOtpPageState
),
SizedBox(height: 24.0.h()),
viewModel.isLoadingOtp
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isOtpValid,
text: ForgotPasswordOtpStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.canSubmitOtpPage,
text: ForgotPasswordOtpStrings.resetLink,
onPressed: viewModel.canSubmitOtpPage
onPressed:
viewModel.canSubmitOtpPage
? () async {
await viewModel.verifyOtp(context);
}
@@ -35,10 +35,11 @@ class _MobileForgotPasswordOtpPageState
void _viewModelListener() {
if (viewModel.errorMessageOtp != null) {
AppToast.error(context, viewModel.errorMessageOtp!);
}
if (viewModel.resetToken != null) {
const ForgotPasswordCreateRouteData().push(context);
} else if (!viewModel.isLoadingOtp &&
viewModel.verifyOtpSuccess &&
viewModel.resetToken != null) {
const ForgotPasswordCreateRouteData().pushReplacement(context);
viewModel.clearOtpFlow();
}
}
@@ -90,7 +91,7 @@ class _MobileForgotPasswordOtpPageState
SizedBox(height: 40.0.h()),
Pinput(
controller: viewModel.otpController,
length: 5,
length: 6,
pinAnimationType: PinAnimationType.scale,
defaultPinTheme: PinTheme(
width: 50.0.w(),
@@ -108,10 +109,11 @@ class _MobileForgotPasswordOtpPageState
),
SizedBox(height: 24.0.h()),
viewModel.isLoadingOtp
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isOtpValid,
text: ForgotPasswordOtpStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.canSubmitOtpPage,
@@ -6,12 +6,12 @@ import 'package:provider/provider.dart';
import 'package:tm_app/core/utils/size_config.dart';
import '../../../core/constants/assets.dart';
import '../../../core/routes/app_routes.dart';
import '../../../core/theme/colors.dart';
import '../../../core/utils/app_toast.dart';
import '../../../view_models/auth_view_model.dart';
import '../../shared/base_button.dart';
import '../strings/forgot_password_otp_strings.dart';
import '../../forget_password_create/pages/forgot_password_create_screen.dart';
class TabletForgotPasswordOtpPage extends StatefulWidget {
const TabletForgotPasswordOtpPage({super.key});
@@ -35,16 +35,14 @@ class _TabletForgotPasswordOtpPageState
void _viewModelListener() {
if (viewModel.errorMessageOtp != null) {
AppToast.error(context, viewModel.errorMessageOtp!);
}
if (viewModel.resetToken != null) {
} else if (!viewModel.isLoadingOtp &&
viewModel.verifyOtpSuccess &&
viewModel.resetToken != null) {
Router.neglect(
context,
() => Navigator.push(
context,
MaterialPageRoute(builder: (_) => const ForgotPasswordCreateScreen()),
),
() => const ForgotPasswordCreateRouteData().pushReplacement(context),
);
viewModel.clearOtpFlow();
}
}
@@ -99,7 +97,7 @@ class _TabletForgotPasswordOtpPageState
SizedBox(height: 40.0.h()),
Pinput(
controller: viewModel.otpController,
length: 5,
length: 6,
pinAnimationType: PinAnimationType.scale,
defaultPinTheme: PinTheme(
width: 50.0.w(),
@@ -117,15 +115,17 @@ class _TabletForgotPasswordOtpPageState
),
SizedBox(height: 24.0.h()),
viewModel.isLoadingOtp
? const BaseButton(
isEnabled: false,
? BaseButton(
isEnabled: viewModel.isOtpValid,
text: ForgotPasswordOtpStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.canSubmitOtpPage,
text: ForgotPasswordOtpStrings.resetLink,
onPressed: viewModel.canSubmitOtpPage
onPressed:
viewModel.canSubmitOtpPage
? () async {
await viewModel.verifyOtp(context);
}
@@ -33,7 +33,10 @@ class _DesktopForgotPasswordPageState extends State<DesktopForgotPasswordPage> {
void _viewModelListener() {
if (viewModel.successMessage != null) {
AppToast.success(context, viewModel.successMessage!);
const ForgotPasswordOtpRouteData().push(context);
Router.neglect(
context,
() => const ForgotPasswordOtpRouteData().pushReplacement(context),
);
viewModel.clearMessages();
}
@@ -106,11 +109,13 @@ class _DesktopForgotPasswordPageState extends State<DesktopForgotPasswordPage> {
isEnabled: false,
text: ForgotPasswordStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isEmailValid,
text: ForgotPasswordStrings.resetLink,
onPressed: viewModel.isEmailValid
onPressed:
viewModel.isEmailValid
? () async {
await viewModel.forgotPassword(context);
}
@@ -33,7 +33,7 @@ class _MobileForgotPasswordPageState extends State<MobileForgotPasswordPage> {
void _viewModelListener() {
if (viewModel.successMessage != null) {
AppToast.success(context, viewModel.successMessage!);
const ForgotPasswordOtpRouteData().push(context);
const ForgotPasswordOtpRouteData().pushReplacement(context);
viewModel.clearMessages();
}
@@ -101,11 +101,13 @@ class _MobileForgotPasswordPageState extends State<MobileForgotPasswordPage> {
isEnabled: false,
text: ForgotPasswordStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isEmailValid,
text: ForgotPasswordStrings.resetLink,
onPressed: viewModel.isEmailValid
onPressed:
viewModel.isEmailValid
? () async {
await viewModel.forgotPassword(context);
}
@@ -33,7 +33,10 @@ class _TabletForgotPasswordPageState extends State<TabletForgotPasswordPage> {
void _viewModelListener() {
if (viewModel.successMessage != null) {
AppToast.success(context, viewModel.successMessage!);
const ForgotPasswordOtpRouteData().push(context);
Router.neglect(
context,
() => const ForgotPasswordOtpRouteData().pushReplacement(context),
);
viewModel.clearMessages();
}
@@ -100,11 +103,13 @@ class _TabletForgotPasswordPageState extends State<TabletForgotPasswordPage> {
isEnabled: false,
text: ForgotPasswordStrings.resetLink,
onPressed: null,
isLoading: true,
)
: BaseButton(
isEnabled: viewModel.isEmailValid,
text: ForgotPasswordStrings.resetLink,
onPressed: viewModel.isEmailValid
onPressed:
viewModel.isEmailValid
? () async {
await viewModel.forgotPassword(context);
}
+14 -14
View File
@@ -106,20 +106,20 @@ class _LoginDesktopPageState extends State<LoginDesktopPage> {
}
},
),
// SizedBox(height: 32.0),
// TextButton(
// onPressed: () {
// ForgotPasswordRouteData().push(context);
// },
// child: Text(
// LoginStrings.forgotPassword,
// style: TextStyle(
// fontSize: 14.0.sp(),
// fontWeight: FontWeight.w500,
// color: AppColors.mainBlue,
// ),
// ),
// ),
const SizedBox(height: 32.0),
TextButton(
onPressed: () {
const ForgotPasswordRouteData().push(context);
},
child: Text(
LoginStrings.forgotPassword,
style: TextStyle(
fontSize: 14.0.sp(),
fontWeight: FontWeight.w500,
color: AppColors.mainBlue,
),
),
),
],
);
},
+14 -14
View File
@@ -99,20 +99,20 @@ class _LoginTabletPageState extends State<LoginTabletPage> {
}
: null,
),
// SizedBox(height: 32.0),
// TextButton(
// onPressed: () {
// ForgotPasswordRouteData().push(context);
// },
// child: Text(
// LoginStrings.forgotPassword,
// style: TextStyle(
// fontSize: 14.0.sp(),
// fontWeight: FontWeight.w500,
// color: AppColors.mainBlue,
// ),
// ),
// ),
const SizedBox(height: 32.0),
TextButton(
onPressed: () {
const ForgotPasswordRouteData().push(context);
},
child: Text(
LoginStrings.forgotPassword,
style: TextStyle(
fontSize: 14.0.sp(),
fontWeight: FontWeight.w500,
color: AppColors.mainBlue,
),
),
),
],
);
},