refactor: Amélioration des performances et migration des Cloud Functions
Cette mise à jour majeure vise à améliorer significativement les performances de l'application, en particulier au démarrage, et à standardiser l'infrastructure backend. Les principaux changements incluent la migration de toutes les Cloud Functions vers une région européenne (`europe-west9`), l'optimisation du chargement des données, et l'introduction d'un moniteur de performance pour le débogage.
**Changements Backend (Cloud Functions) :**
- **Migration de la Région :**
- Toutes les Cloud Functions ont été déplacées de `us-central1` à `europe-west9` (Paris) pour réduire la latence pour les utilisateurs européens. Cela concerne les appels depuis le frontend (ex: `api_config.dart`, `email_service.dart`) et les définitions des fonctions elles-mêmes (`index.js`, etc.).
- **Standardisation des Fonctions :**
- La plupart des fonctions `onCall` (v1) ont été migrées vers le format `onRequest` (v2) avec une gestion d'authentification et de CORS unifiée, améliorant la robustesse et la cohérence.
- Les triggers Firestore (`onDocumentCreated`, `onDocumentUpdated`) et les tâches planifiées (`onSchedule`) ont été mis à jour pour spécifier explicitement la région `europe-west9`.
- **Mise à jour des Index Firestore :**
- Les index `firestore.indexes.json` ont été mis à jour pour supporter les nouvelles requêtes de l'application et optimiser les performances de filtrage.
**Améliorations des Performances Frontend :**
- **Chargement Asynchrone et Mis en Cache :**
- Le chargement des données utilisateur (`LocalUserProvider`) et des événements (`EventProvider`) a été optimisé pour utiliser un cache local à court terme (5 minutes pour l'utilisateur, 30 secondes pour les événements).
- Les données ne sont rechargées que si le cache a expiré ou si un rechargement est forcé, évitant des appels réseau redondants et accélérant la navigation.
- **Démarrage de l'Application Optimisé :**
- Le processus de connexion automatique (`main.dart`) a été revu. L'application navigue désormais immédiatement vers la page demandée sans attendre la fin du chargement des données utilisateur, qui s'effectue en arrière-plan.
- Un écran de chargement plus esthétique avec le logo de l'entreprise a été ajouté, remplaçant l'indicateur de chargement simple.
- **Chargement de la Page Calendrier :**
- Le chargement et la sélection de l'événement par défaut sur la page `CalendarPage` sont maintenant entièrement asynchrones, rendant l'affichage de la page quasi instantané.
**Nouveaux Outils et Améliorations UX :**
- **Moniteur de Performance :**
- Ajout d'un nouvel outil `PerformanceMonitor` (`lib/utils/performance_monitor.dart`) pour mesurer précisément le temps d'exécution des opérations critiques (appels API, parsing, etc.) en mode débogage. Il aide à identifier les goulots d'étranglement.
- **Amélioration du Formulaire de Connexion :**
- Les champs "Email" et "Mot de passe" sur la page de connexion (`LoginPage`) supportent désormais l'autocomplétion du navigateur (`AutofillGroup`).
- Appuyer sur "Entrée" dans l'un des champs déclenche désormais la connexion, améliorant l'ergonomie.
**Mise à jour de la version :**
- La version de l'application a été incrémentée à `1.0.9`.
This commit is contained in:
ElPoyo
@@ -3,6 +3,7 @@ import 'package:cloud_firestore/cloud_firestore.dart';
|
||||
import 'package:em2rp/models/event_model.dart';
|
||||
import 'package:em2rp/services/data_service.dart';
|
||||
import 'package:em2rp/services/api_service.dart';
|
||||
import 'package:em2rp/utils/performance_monitor.dart';
|
||||
|
||||
class EventProvider with ChangeNotifier {
|
||||
final DataService _dataService = DataService(FirebaseFunctionsApiService());
|
||||
@@ -15,19 +16,43 @@ class EventProvider with ChangeNotifier {
|
||||
// Cache des utilisateurs chargés depuis getEvents
|
||||
Map<String, Map<String, dynamic>> _usersCache = {};
|
||||
|
||||
// Cache pour éviter les rechargements inutiles
|
||||
DateTime? _lastLoadTime;
|
||||
String? _lastUserId;
|
||||
bool _lastCanViewAll = false;
|
||||
|
||||
/// Vérifie si les données doivent être rechargées (cache de 30 secondes)
|
||||
bool _shouldReload(String userId, bool canViewAllEvents) {
|
||||
if (_lastLoadTime == null) return true;
|
||||
if (_lastUserId != userId || _lastCanViewAll != canViewAllEvents) return true;
|
||||
|
||||
final now = DateTime.now();
|
||||
final difference = now.difference(_lastLoadTime!);
|
||||
return difference.inSeconds > 30;
|
||||
}
|
||||
|
||||
/// Charger les événements d'un utilisateur via l'API
|
||||
Future<void> loadUserEvents(String userId, {bool canViewAllEvents = false}) async {
|
||||
Future<void> loadUserEvents(String userId, {bool canViewAllEvents = false, bool forceReload = false}) async {
|
||||
PerformanceMonitor.start('EventProvider.loadUserEvents');
|
||||
|
||||
// Éviter les rechargements inutiles
|
||||
if (!forceReload && !_shouldReload(userId, canViewAllEvents)) {
|
||||
print('Using cached events (loaded ${DateTime.now().difference(_lastLoadTime!).inSeconds}s ago)');
|
||||
PerformanceMonitor.end('EventProvider.loadUserEvents');
|
||||
return;
|
||||
}
|
||||
|
||||
_isLoading = true;
|
||||
notifyListeners();
|
||||
|
||||
// Sauvegarder les paramètres
|
||||
_saveLastLoadParams(userId, canViewAllEvents);
|
||||
|
||||
try {
|
||||
print('Loading events for user: $userId (canViewAllEvents: $canViewAllEvents)');
|
||||
|
||||
PerformanceMonitor.start('EventProvider.getEvents_API');
|
||||
// Charger via l'API - les permissions sont vérifiées côté serveur
|
||||
final result = await _dataService.getEvents(userId: userId);
|
||||
PerformanceMonitor.end('EventProvider.getEvents_API');
|
||||
|
||||
final eventsData = result['events'] as List<Map<String, dynamic>>;
|
||||
final usersData = result['users'] as Map<String, dynamic>;
|
||||
|
||||
@@ -38,6 +63,7 @@ class EventProvider with ChangeNotifier {
|
||||
|
||||
print('Found ${eventsData.length} events from API');
|
||||
|
||||
PerformanceMonitor.start('EventProvider.parseEvents');
|
||||
List<EventModel> allEvents = [];
|
||||
int failedCount = 0;
|
||||
|
||||
@@ -51,23 +77,30 @@ class EventProvider with ChangeNotifier {
|
||||
failedCount++;
|
||||
}
|
||||
}
|
||||
PerformanceMonitor.end('EventProvider.parseEvents');
|
||||
|
||||
_events = allEvents;
|
||||
_lastLoadTime = DateTime.now();
|
||||
_lastUserId = userId;
|
||||
_lastCanViewAll = canViewAllEvents;
|
||||
|
||||
print('Successfully loaded ${_events.length} events (${failedCount} failed)');
|
||||
|
||||
_isLoading = false;
|
||||
notifyListeners();
|
||||
PerformanceMonitor.end('EventProvider.loadUserEvents');
|
||||
} catch (e) {
|
||||
print('Error loading events: $e');
|
||||
_isLoading = false;
|
||||
notifyListeners();
|
||||
PerformanceMonitor.end('EventProvider.loadUserEvents');
|
||||
rethrow;
|
||||
}
|
||||
}
|
||||
|
||||
/// Recharger les événements (utilise le dernier userId)
|
||||
Future<void> refreshEvents(String userId, {bool canViewAllEvents = false}) async {
|
||||
await loadUserEvents(userId, canViewAllEvents: canViewAllEvents);
|
||||
await loadUserEvents(userId, canViewAllEvents: canViewAllEvents, forceReload: true);
|
||||
}
|
||||
|
||||
/// Récupérer un événement spécifique par ID
|
||||
@@ -157,16 +190,9 @@ class EventProvider with ChangeNotifier {
|
||||
/// Vider la liste des événements
|
||||
void clearEvents() {
|
||||
_events = [];
|
||||
_lastLoadTime = null;
|
||||
_lastUserId = null;
|
||||
_lastCanViewAll = false;
|
||||
notifyListeners();
|
||||
}
|
||||
|
||||
// Variables pour stocker le dernier appel
|
||||
String? _lastUserId;
|
||||
bool _lastCanViewAll = false;
|
||||
|
||||
/// Sauvegarder les paramètres du dernier chargement
|
||||
void _saveLastLoadParams(String userId, bool canViewAllEvents) {
|
||||
_lastUserId = userId;
|
||||
_lastCanViewAll = canViewAllEvents;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -7,6 +7,7 @@ import '../models/notification_preferences_model.dart';
|
||||
import '../utils/firebase_storage_manager.dart';
|
||||
import '../services/api_service.dart';
|
||||
import '../services/data_service.dart';
|
||||
import '../utils/performance_monitor.dart';
|
||||
|
||||
class LocalUserProvider with ChangeNotifier {
|
||||
UserModel? _currentUser;
|
||||
@@ -15,6 +16,9 @@ class LocalUserProvider with ChangeNotifier {
|
||||
final FirebaseStorageManager _storageManager = FirebaseStorageManager();
|
||||
final DataService _dataService = DataService(apiService);
|
||||
|
||||
bool _isLoadingUserData = false;
|
||||
DateTime? _lastUserDataLoad;
|
||||
|
||||
UserModel? get currentUser => _currentUser;
|
||||
String? get uid => _currentUser?.uid;
|
||||
String? get firstName => _currentUser?.firstName;
|
||||
@@ -25,18 +29,46 @@ class LocalUserProvider with ChangeNotifier {
|
||||
String? get phoneNumber => _currentUser?.phoneNumber;
|
||||
RoleModel? get currentRole => _currentRole;
|
||||
List<String> get permissions => _currentRole?.permissions ?? [];
|
||||
bool get isLoadingUserData => _isLoadingUserData;
|
||||
|
||||
/// Vérifie si les données utilisateur doivent être rechargées
|
||||
bool _shouldReloadUserData() {
|
||||
if (_currentUser == null) return true;
|
||||
if (_lastUserDataLoad == null) return true;
|
||||
|
||||
final now = DateTime.now();
|
||||
final difference = now.difference(_lastUserDataLoad!);
|
||||
return difference.inMinutes > 5; // Cache de 5 minutes pour les données utilisateur
|
||||
}
|
||||
|
||||
/// Charge les données de l'utilisateur actuel via Cloud Function
|
||||
Future<void> loadUserData() async {
|
||||
Future<void> loadUserData({bool forceReload = false}) async {
|
||||
if (_auth.currentUser == null) {
|
||||
print('No current user in Auth');
|
||||
return;
|
||||
}
|
||||
|
||||
// Éviter les rechargements inutiles
|
||||
if (!forceReload && !_shouldReloadUserData()) {
|
||||
print('Using cached user data');
|
||||
return;
|
||||
}
|
||||
|
||||
// Éviter les appels simultanés
|
||||
if (_isLoadingUserData) {
|
||||
print('User data already loading, skipping');
|
||||
return;
|
||||
}
|
||||
|
||||
_isLoadingUserData = true;
|
||||
PerformanceMonitor.start('LocalUserProvider.loadUserData');
|
||||
print('Loading user data for: ${_auth.currentUser!.uid}');
|
||||
try {
|
||||
// Utiliser la Cloud Function getCurrentUser
|
||||
PerformanceMonitor.start('LocalUserProvider.getCurrentUser_API');
|
||||
final result = await apiService.call('getCurrentUser', {});
|
||||
PerformanceMonitor.end('LocalUserProvider.getCurrentUser_API');
|
||||
|
||||
final userData = result['user'] as Map<String, dynamic>;
|
||||
|
||||
print('User data loaded from API: ${userData['uid']}');
|
||||
@@ -59,9 +91,14 @@ class LocalUserProvider with ChangeNotifier {
|
||||
);
|
||||
|
||||
print('User data loaded successfully');
|
||||
_lastUserDataLoad = DateTime.now();
|
||||
_isLoadingUserData = false;
|
||||
notifyListeners();
|
||||
PerformanceMonitor.end('LocalUserProvider.loadUserData');
|
||||
} catch (e) {
|
||||
print('Error loading user data: $e');
|
||||
_isLoadingUserData = false;
|
||||
PerformanceMonitor.end('LocalUserProvider.loadUserData');
|
||||
rethrow;
|
||||
}
|
||||
}
|
||||
@@ -76,6 +113,8 @@ class LocalUserProvider with ChangeNotifier {
|
||||
void clearUser() {
|
||||
_currentUser = null;
|
||||
_currentRole = null;
|
||||
_lastUserDataLoad = null;
|
||||
_isLoadingUserData = false;
|
||||
notifyListeners();
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user