refactor: Remplacement de l'accès direct à Firestore par des Cloud Functions
Migration complète du backend pour utiliser des Cloud Functions comme couche API sécurisée, en remplacement des appels directs à Firestore depuis le client.
**Backend (Cloud Functions):**
- **Centralisation CORS :** Ajout d'un middleware `withCors` et d'une configuration `httpOptions` pour gérer uniformément les en-têtes CORS et les requêtes `OPTIONS` sur toutes les fonctions.
- **Nouvelles Fonctions de Lecture (GET) :**
- `getEquipments`, `getContainers`, `getEvents`, `getUsers`, `getOptions`, `getEventTypes`, `getRoles`, `getMaintenances`, `getAlerts`.
- Ces fonctions gèrent les permissions côté serveur, masquant les données sensibles (ex: prix des équipements) pour les utilisateurs non-autorisés.
- `getEvents` retourne également une map des utilisateurs (`usersMap`) pour optimiser le chargement des données de la main d'œuvre.
- **Nouvelle Fonction de Recherche :**
- `getContainersByEquipment` : Endpoint dédié pour trouver efficacement tous les containers qui contiennent un équipement spécifique.
- **Nouvelles Fonctions d'Écriture (CRUD) :**
- Fonctions CRUD complètes pour `eventTypes` (`create`, `update`, `delete`), incluant la validation (unicité du nom, vérification des événements futurs avant suppression).
- **Mise à jour de Fonctions Existantes :**
- Toutes les fonctions CRUD existantes (`create/update/deleteEquipment`, `create/update/deleteContainer`, etc.) sont wrappées avec le nouveau gestionnaire CORS.
**Frontend (Flutter):**
- **Introduction du `DataService` :** Nouveau service centralisant tous les appels aux Cloud Functions, servant d'intermédiaire entre l'UI/Providers et l'API.
- **Refactorisation des Providers :**
- `EquipmentProvider`, `ContainerProvider`, `EventProvider`, `UsersProvider`, `MaintenanceProvider` et `AlertProvider` ont été refactorisés pour utiliser le `DataService` au lieu d'accéder directement à Firestore.
- Les `Stream` Firestore sont remplacés par des chargements de données via des méthodes `Future` (`loadEquipments`, `loadEvents`, etc.).
- **Gestion des Relations Équipement-Container :**
- Le modèle `EquipmentModel` ne stocke plus `parentBoxIds`.
- La relation est maintenant gérée par le `ContainerModel` qui contient `equipmentIds`.
- Le `ContainerEquipmentService` est introduit pour utiliser la nouvelle fonction `getContainersByEquipment`.
- L'affichage des boîtes parentes (`EquipmentParentContainers`) et le formulaire d'équipement (`EquipmentFormPage`) ont été mis à jour pour refléter ce nouveau modèle de données, synchronisant les ajouts/suppressions d'équipements dans les containers.
- **Amélioration de l'UI :**
- Nouveau widget `ParentBoxesSelector` pour une sélection améliorée et visuelle des boîtes parentes dans le formulaire d'équipement.
- Refonte visuelle de `EquipmentParentContainers` pour une meilleure présentation.
This commit is contained in:
ElPoyo
@@ -1,6 +1,7 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:cloud_firestore/cloud_firestore.dart';
|
||||
import 'package:em2rp/models/option_model.dart';
|
||||
import 'package:em2rp/services/data_service.dart';
|
||||
import 'package:em2rp/services/api_service.dart';
|
||||
|
||||
class OptionSelectorWidget extends StatefulWidget {
|
||||
final List<Map<String, dynamic>> selectedOptions;
|
||||
@@ -42,15 +43,20 @@ class _OptionSelectorWidgetState extends State<OptionSelectorWidget> {
|
||||
|
||||
Future<void> _fetchOptions() async {
|
||||
setState(() => _loading = true);
|
||||
final snapshot =
|
||||
await FirebaseFirestore.instance.collection('options').get();
|
||||
final options = snapshot.docs
|
||||
.map((doc) => EventOption.fromMap(doc.data(), doc.id))
|
||||
.toList();
|
||||
setState(() {
|
||||
_allOptions = options;
|
||||
_loading = false;
|
||||
});
|
||||
try {
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
final optionsData = await dataService.getOptions();
|
||||
final options = optionsData
|
||||
.map((data) => EventOption.fromMap(data, data['id'] as String))
|
||||
.toList();
|
||||
setState(() {
|
||||
_allOptions = options;
|
||||
_loading = false;
|
||||
});
|
||||
} catch (e) {
|
||||
setState(() => _loading = false);
|
||||
// Afficher une erreur silencieuse
|
||||
}
|
||||
}
|
||||
|
||||
// Méthode publique pour mettre à jour les options depuis l'extérieur
|
||||
@@ -258,27 +264,32 @@ class _OptionSelectorWidgetState extends State<OptionSelectorWidget> {
|
||||
);
|
||||
}
|
||||
|
||||
// Méthode pour charger les détails des options depuis Firebase
|
||||
// Méthode pour charger les détails des options via l'API
|
||||
Future<List<Map<String, dynamic>>> _loadOptionsWithDetails(List<Map<String, dynamic>> optionsData) async {
|
||||
List<Map<String, dynamic>> enrichedOptions = [];
|
||||
|
||||
// Charger toutes les options via l'API
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
final allOptionsData = await dataService.getOptions();
|
||||
|
||||
// Créer une map pour accès rapide par ID
|
||||
final optionsMap = {
|
||||
for (var opt in allOptionsData) opt['id']: opt
|
||||
};
|
||||
|
||||
for (final optionData in optionsData) {
|
||||
try {
|
||||
// Si l'option a un ID, récupérer les détails depuis Firestore
|
||||
// Si l'option a un ID, récupérer les détails depuis l'API
|
||||
if (optionData['id'] != null) {
|
||||
final doc = await FirebaseFirestore.instance
|
||||
.collection('options')
|
||||
.doc(optionData['id'])
|
||||
.get();
|
||||
final firestoreData = optionsMap[optionData['id']];
|
||||
|
||||
if (doc.exists) {
|
||||
final firestoreData = doc.data()!;
|
||||
if (firestoreData != null) {
|
||||
enrichedOptions.add({
|
||||
'id': optionData['id'],
|
||||
'code': firestoreData['code'] ?? optionData['id'], // Récupérer le code
|
||||
'code': firestoreData['code'] ?? optionData['id'],
|
||||
'name': firestoreData['code'] != null && firestoreData['code'].toString().isNotEmpty
|
||||
? '${firestoreData['code']} - ${firestoreData['name']}'
|
||||
: firestoreData['name'], // Affichage avec code
|
||||
: firestoreData['name'],
|
||||
'details': firestoreData['details'] ?? '',
|
||||
'price': optionData['price'],
|
||||
'quantity': optionData['quantity'] ?? 1,
|
||||
@@ -347,17 +358,22 @@ class _OptionPickerDialogState extends State<_OptionPickerDialog> {
|
||||
}
|
||||
|
||||
Future<void> _reloadOptions() async {
|
||||
final snapshot = await FirebaseFirestore.instance.collection('options').get();
|
||||
final updatedOptions = snapshot.docs
|
||||
.map((doc) => EventOption.fromMap(doc.data(), doc.id))
|
||||
.toList();
|
||||
try {
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
final optionsData = await dataService.getOptions();
|
||||
final updatedOptions = optionsData
|
||||
.map((data) => EventOption.fromMap(data, data['id'] as String))
|
||||
.toList();
|
||||
|
||||
setState(() {
|
||||
_currentOptions = updatedOptions;
|
||||
});
|
||||
setState(() {
|
||||
_currentOptions = updatedOptions;
|
||||
});
|
||||
|
||||
// Appeler le callback pour mettre à jour aussi le parent
|
||||
widget.onOptionsUpdated?.call(updatedOptions);
|
||||
// Appeler le callback pour mettre à jour aussi le parent
|
||||
widget.onOptionsUpdated?.call(updatedOptions);
|
||||
} catch (e) {
|
||||
// Erreur silencieuse
|
||||
}
|
||||
}
|
||||
|
||||
@override
|
||||
@@ -557,23 +573,34 @@ class _CreateOptionDialogState extends State<_CreateOptionDialog> {
|
||||
bool _loading = true;
|
||||
|
||||
Future<bool> _isCodeUnique(String code) async {
|
||||
// Vérifier si le document avec ce code existe déjà
|
||||
final doc = await FirebaseFirestore.instance
|
||||
.collection('options')
|
||||
.doc(code)
|
||||
.get();
|
||||
return !doc.exists;
|
||||
try {
|
||||
// Charger toutes les options via l'API
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
final optionsData = await dataService.getOptions();
|
||||
|
||||
// Vérifier si le code existe déjà
|
||||
return !optionsData.any((opt) => opt['id'] == code);
|
||||
} catch (e) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
Future<void> _fetchEventTypes() async {
|
||||
setState(() {
|
||||
_loading=true;
|
||||
});
|
||||
final snapshot = await FirebaseFirestore.instance.collection('eventTypes').get();
|
||||
setState(() {
|
||||
_allEventTypes = snapshot.docs.map((doc) => {'id': doc.id, 'name': doc['name']}).toList();
|
||||
_loading = false;
|
||||
_loading = true;
|
||||
});
|
||||
try {
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
final eventTypesData = await dataService.getEventTypes();
|
||||
setState(() {
|
||||
_allEventTypes = eventTypesData
|
||||
.map((data) => {'id': data['id'], 'name': data['name']})
|
||||
.toList();
|
||||
_loading = false;
|
||||
});
|
||||
} catch (e) {
|
||||
setState(() => _loading = false);
|
||||
}
|
||||
}
|
||||
|
||||
@override
|
||||
@@ -741,8 +768,9 @@ class _CreateOptionDialogState extends State<_CreateOptionDialog> {
|
||||
return;
|
||||
}
|
||||
try {
|
||||
// Utiliser le code comme identifiant du document
|
||||
await FirebaseFirestore.instance.collection('options').doc(code).set({
|
||||
// Créer via l'API
|
||||
final dataService = DataService(FirebaseFunctionsApiService());
|
||||
await dataService.createOption(code, {
|
||||
'code': code,
|
||||
'name': name,
|
||||
'details': _detailsController.text.trim(),
|
||||
@@ -751,7 +779,9 @@ class _CreateOptionDialogState extends State<_CreateOptionDialog> {
|
||||
'eventTypes': _selectedTypes,
|
||||
'isQuantitative': _isQuantitative,
|
||||
});
|
||||
Navigator.pop(context, true);
|
||||
if (mounted) {
|
||||
Navigator.pop(context, true);
|
||||
}
|
||||
} catch (e) {
|
||||
setState(() => _error = 'Erreur lors de la création : $e');
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user